/**
* Processes the ContainerBuilder to remove unused definitions.
- *
- * @param ContainerBuilder $container
*/
public function process(ContainerBuilder $container)
{
- $compiler = $container->getCompiler();
- $formatter = $compiler->getLoggingFormatter();
- $graph = $compiler->getServiceReferenceGraph();
+ $graph = $container->getCompiler()->getServiceReferenceGraph();
$hasChanged = false;
foreach ($container->getDefinitions() as $id => $definition) {
- if ($definition->isPublic()) {
+ if ($definition->isPublic() || $definition->isPrivate()) {
continue;
}
$referencingAliases = array();
$sourceIds = array();
foreach ($edges as $edge) {
+ if ($edge->isWeak()) {
+ continue;
+ }
$node = $edge->getSourceNode();
$sourceIds[] = $node->getId();
if (1 === count($referencingAliases) && false === $isReferenced) {
$container->setDefinition((string) reset($referencingAliases), $definition);
- $definition->setPublic(true);
+ $definition->setPublic(!$definition->isPrivate());
+ $definition->setPrivate(reset($referencingAliases)->isPrivate());
$container->removeDefinition($id);
- $compiler->addLogMessage($formatter->formatRemoveService($this, $id, 'replaces alias '.reset($referencingAliases)));
+ $container->log($this, sprintf('Removed service "%s"; reason: replaces alias %s.', $id, reset($referencingAliases)));
} elseif (0 === count($referencingAliases) && false === $isReferenced) {
$container->removeDefinition($id);
$container->resolveEnvPlaceholders(serialize($definition));
- $compiler->addLogMessage($formatter->formatRemoveService($this, $id, 'unused'));
+ $container->log($this, sprintf('Removed service "%s"; reason: unused.', $id));
$hasChanged = true;
}
}