Don't show wanna-be-private services as public in debug:container

Robin Chalas · Robin Chalas · commit 4fbc63af1290 · 2018-01-31T18:08:05.000+01:00
diff --git a/Console/Descriptor/JsonDescriptor.php b/Console/Descriptor/JsonDescriptor.php
@@ -113,11 +113,11 @@ protected function describeContainerServices(ContainerBuilder $builder, array $o
             $service = $this->resolveServiceDefinition($builder, $serviceId);
 
             if ($service instanceof Alias) {
-                if ($showPrivate || $service->isPublic()) {
+                if ($showPrivate || ($service->isPublic() && !$service->isPrivate())) {
                     $data['aliases'][$serviceId] = $this->getContainerAliasData($service);
                 }
             } elseif ($service instanceof Definition) {
-                if (($showPrivate || $service->isPublic())) {
+                if (($showPrivate || ($service->isPublic() && !$service->isPrivate()))) {
                     $data['definitions'][$serviceId] = $this->getContainerDefinitionData($service, $omitTags, $showArguments);
                 }
             } else {
@@ -217,7 +217,7 @@ private function getContainerDefinitionData(Definition $definition, $omitTags =
     {
         $data = array(
             'class' => (string) $definition->getClass(),
-            'public' => $definition->isPublic(),
+            'public' => $definition->isPublic() && !$definition->isPrivate(),
             'synthetic' => $definition->isSynthetic(),
             'lazy' => $definition->isLazy(),
             'shared' => $definition->isShared(),
@@ -281,7 +281,7 @@ private function getContainerAliasData(Alias $alias)
     {
         return array(
             'service' => (string) $alias,
-            'public' => $alias->isPublic(),
+            'public' => $alias->isPublic() && !$alias->isPrivate(),
         );
     }
 
diff --git a/Console/Descriptor/MarkdownDescriptor.php b/Console/Descriptor/MarkdownDescriptor.php
@@ -139,11 +139,11 @@ protected function describeContainerServices(ContainerBuilder $builder, array $o
             $service = $this->resolveServiceDefinition($builder, $serviceId);
 
             if ($service instanceof Alias) {
-                if ($showPrivate || $service->isPublic()) {
+                if ($showPrivate || ($service->isPublic() && !$service->isPrivate())) {
                     $services['aliases'][$serviceId] = $service;
                 }
             } elseif ($service instanceof Definition) {
-                if (($showPrivate || $service->isPublic())) {
+                if (($showPrivate || ($service->isPublic() && !$service->isPrivate()))) {
                     $services['definitions'][$serviceId] = $service;
                 }
             } else {
@@ -182,7 +182,7 @@ protected function describeContainerServices(ContainerBuilder $builder, array $o
     protected function describeContainerDefinition(Definition $definition, array $options = array())
     {
         $output = '- Class: `'.$definition->getClass().'`'
-            ."\n".'- Public: '.($definition->isPublic() ? 'yes' : 'no')
+            ."\n".'- Public: '.($definition->isPublic() && !$definition->isPrivate() ? 'yes' : 'no')
             ."\n".'- Synthetic: '.($definition->isSynthetic() ? 'yes' : 'no')
             ."\n".'- Lazy: '.($definition->isLazy() ? 'yes' : 'no')
             ."\n".'- Shared: '.($definition->isShared() ? 'yes' : 'no')
@@ -246,7 +246,7 @@ protected function describeContainerDefinition(Definition $definition, array $op
     protected function describeContainerAlias(Alias $alias, array $options = array(), ContainerBuilder $builder = null)
     {
         $output = '- Service: `'.$alias.'`'
-            ."\n".'- Public: '.($alias->isPublic() ? 'yes' : 'no');
+            ."\n".'- Public: '.($alias->isPublic() && !$alias->isPrivate() ? 'yes' : 'no');
 
         if (!isset($options['id'])) {
             return $this->write($output);
diff --git a/Console/Descriptor/TextDescriptor.php b/Console/Descriptor/TextDescriptor.php
@@ -194,7 +194,7 @@ protected function describeContainerServices(ContainerBuilder $builder, array $o
             $definition = $this->resolveServiceDefinition($builder, $serviceId);
             if ($definition instanceof Definition) {
                 // filter out private services unless shown explicitly
-                if (!$showPrivate && !$definition->isPublic()) {
+                if (!$showPrivate && (!$definition->isPublic() || $definition->isPrivate())) {
                     unset($serviceIds[$key]);
                     continue;
                 }
@@ -212,7 +212,7 @@ protected function describeContainerServices(ContainerBuilder $builder, array $o
                     }
                 }
             } elseif ($definition instanceof Alias) {
-                if (!$showPrivate && !$definition->isPublic()) {
+                if (!$showPrivate && (!$definition->isPublic() || $definition->isPrivate())) {
                     unset($serviceIds[$key]);
                     continue;
                 }
@@ -302,7 +302,7 @@ protected function describeContainerDefinition(Definition $definition, array $op
             $tableRows[] = array('Calls', implode(', ', $callInformation));
         }
 
-        $tableRows[] = array('Public', $definition->isPublic() ? 'yes' : 'no');
+        $tableRows[] = array('Public', $definition->isPublic() && !$definition->isPrivate() ? 'yes' : 'no');
         $tableRows[] = array('Synthetic', $definition->isSynthetic() ? 'yes' : 'no');
         $tableRows[] = array('Lazy', $definition->isLazy() ? 'yes' : 'no');
         $tableRows[] = array('Shared', $definition->isShared() ? 'yes' : 'no');
diff --git a/Console/Descriptor/XmlDescriptor.php b/Console/Descriptor/XmlDescriptor.php
@@ -318,7 +318,7 @@ private function getContainerServicesDocument(ContainerBuilder $builder, $tag =
         foreach ($this->sortServiceIds($serviceIds) as $serviceId) {
             $service = $this->resolveServiceDefinition($builder, $serviceId);
 
-            if (($service instanceof Definition || $service instanceof Alias) && !($showPrivate || $service->isPublic())) {
+            if (($service instanceof Definition || $service instanceof Alias) && !($showPrivate || ($service->isPublic() && !$service->isPrivate()))) {
                 continue;
             }
 
@@ -364,7 +364,7 @@ private function getContainerDefinitionDocument(Definition $definition, $id = nu
             }
         }
 
-        $serviceXML->setAttribute('public', $definition->isPublic() ? 'true' : 'false');
+        $serviceXML->setAttribute('public', $definition->isPublic() && !$definition->isPrivate() ? 'true' : 'false');
         $serviceXML->setAttribute('synthetic', $definition->isSynthetic() ? 'true' : 'false');
         $serviceXML->setAttribute('lazy', $definition->isLazy() ? 'true' : 'false');
         $serviceXML->setAttribute('shared', $definition->isShared() ? 'true' : 'false');
@@ -469,7 +469,7 @@ private function getContainerAliasDocument(Alias $alias, $id = null)
         }
 
         $aliasXML->setAttribute('service', (string) $alias);
-        $aliasXML->setAttribute('public', $alias->isPublic() ? 'true' : 'false');
+        $aliasXML->setAttribute('public', $alias->isPublic() && !$alias->isPrivate() ? 'true' : 'false');
 
         return $dom;
     }
diff --git a/Tests/Fixtures/Descriptor/builder_1_arguments.json b/Tests/Fixtures/Descriptor/builder_1_arguments.json
@@ -17,7 +17,7 @@
                 "%parameter%",
                 {
                     "class": "inline_service",
-                    "public": true,
+                    "public": false,
                     "synthetic": false,
                     "lazy": false,
                     "shared": true,
@@ -39,7 +39,7 @@
                     },
                     {
                         "class": "inline_service",
-                        "public": true,
+                        "public": false,
                         "synthetic": false,
                         "lazy": false,
                         "shared": true,
diff --git a/Tests/Fixtures/Descriptor/builder_1_arguments.xml b/Tests/Fixtures/Descriptor/builder_1_arguments.xml
@@ -6,7 +6,7 @@
     <argument type="service" id="definition2"/>
     <argument>%parameter%</argument>
     <argument>
-      <definition class="inline_service" public="true" synthetic="false" lazy="false" shared="true" abstract="false" autowired="false" autoconfigured="false" file="">
+      <definition class="inline_service" public="false" synthetic="false" lazy="false" shared="true" abstract="false" autowired="false" autoconfigured="false" file="">
         <argument>arg1</argument>
         <argument>arg2</argument>
       </definition>
@@ -15,7 +15,7 @@
       <argument>foo</argument>
       <argument type="service" id="definition2"/>
       <argument>
-        <definition class="inline_service" public="true" synthetic="false" lazy="false" shared="true" abstract="false" autowired="false" autoconfigured="false" file=""/>
+        <definition class="inline_service" public="false" synthetic="false" lazy="false" shared="true" abstract="false" autowired="false" autoconfigured="false" file=""/>
       </argument>
     </argument>
     <argument type="iterator">
diff --git a/Tests/Fixtures/Descriptor/definition_arguments_1.json b/Tests/Fixtures/Descriptor/definition_arguments_1.json
@@ -15,7 +15,7 @@
         "%parameter%",
         {
             "class": "inline_service",
-            "public": true,
+            "public": false,
             "synthetic": false,
             "lazy": false,
             "shared": true,
@@ -37,7 +37,7 @@
             },
             {
                 "class": "inline_service",
-                "public": true,
+                "public": false,
                 "synthetic": false,
                 "lazy": false,
                 "shared": true,
diff --git a/Tests/Fixtures/Descriptor/definition_arguments_1.xml b/Tests/Fixtures/Descriptor/definition_arguments_1.xml
@@ -4,7 +4,7 @@
   <argument type="service" id="definition2"/>
   <argument>%parameter%</argument>
   <argument>
-    <definition class="inline_service" public="true" synthetic="false" lazy="false" shared="true" abstract="false" autowired="false" autoconfigured="false" file="">
+    <definition class="inline_service" public="false" synthetic="false" lazy="false" shared="true" abstract="false" autowired="false" autoconfigured="false" file="">
       <argument>arg1</argument>
       <argument>arg2</argument>
     </definition>
@@ -13,7 +13,7 @@
     <argument>foo</argument>
     <argument type="service" id="definition2"/>
     <argument>
-      <definition class="inline_service" public="true" synthetic="false" lazy="false" shared="true" abstract="false" autowired="false" autoconfigured="false" file=""/>
+      <definition class="inline_service" public="false" synthetic="false" lazy="false" shared="true" abstract="false" autowired="false" autoconfigured="false" file=""/>
     </argument>
   </argument>
   <argument type="iterator">

Original file line number	Diff line number	Diff line change
`@@ -113,11 +113,11 @@ protected function describeContainerServices(ContainerBuilder $builder, array $o`
`113`	`113`	`$service = $this->resolveServiceDefinition($builder, $serviceId);`
`114`	`114`
`115`	`115`	`if ($service instanceof Alias) {`
`116`		`- if ($showPrivate \|\| $service->isPublic()) {`
	`116`	`+ if ($showPrivate \|\| ($service->isPublic() && !$service->isPrivate())) {`
`117`	`117`	`$data['aliases'][$serviceId] = $this->getContainerAliasData($service);`
`118`	`118`	`}`
`119`	`119`	`} elseif ($service instanceof Definition) {`
`120`		`- if (($showPrivate \|\| $service->isPublic())) {`
	`120`	`+ if (($showPrivate \|\| ($service->isPublic() && !$service->isPrivate()))) {`
`121`	`121`	`$data['definitions'][$serviceId] = $this->getContainerDefinitionData($service, $omitTags, $showArguments);`
`122`	`122`	`}`
`123`	`123`	`} else {`
`@@ -217,7 +217,7 @@ private function getContainerDefinitionData(Definition $definition, $omitTags =`
`217`	`217`	`{`
`218`	`218`	`$data = array(`
`219`	`219`	`'class' => (string) $definition->getClass(),`
`220`		`- 'public' => $definition->isPublic(),`
	`220`	`+ 'public' => $definition->isPublic() && !$definition->isPrivate(),`
`221`	`221`	`'synthetic' => $definition->isSynthetic(),`
`222`	`222`	`'lazy' => $definition->isLazy(),`
`223`	`223`	`'shared' => $definition->isShared(),`
`@@ -281,7 +281,7 @@ private function getContainerAliasData(Alias $alias)`
`281`	`281`	`{`
`282`	`282`	`return array(`
`283`	`283`	`'service' => (string) $alias,`
`284`		`- 'public' => $alias->isPublic(),`
	`284`	`+ 'public' => $alias->isPublic() && !$alias->isPrivate(),`
`285`	`285`	`);`
`286`	`286`	`}`
`287`	`287`
Original file line number	Diff line number	Diff line change
`@@ -194,7 +194,7 @@ protected function describeContainerServices(ContainerBuilder $builder, array $o`
`194`	`194`	`$definition = $this->resolveServiceDefinition($builder, $serviceId);`
`195`	`195`	`if ($definition instanceof Definition) {`
`196`	`196`	`// filter out private services unless shown explicitly`
`197`		`- if (!$showPrivate && !$definition->isPublic()) {`
	`197`	`+ if (!$showPrivate && (!$definition->isPublic() \|\| $definition->isPrivate())) {`
`198`	`198`	`unset($serviceIds[$key]);`
`199`	`199`	`continue;`
`200`	`200`	`}`
`@@ -212,7 +212,7 @@ protected function describeContainerServices(ContainerBuilder $builder, array $o`
`212`	`212`	`}`
`213`	`213`	`}`
`214`	`214`	`} elseif ($definition instanceof Alias) {`
`215`		`- if (!$showPrivate && !$definition->isPublic()) {`
	`215`	`+ if (!$showPrivate && (!$definition->isPublic() \|\| $definition->isPrivate())) {`
`216`	`216`	`unset($serviceIds[$key]);`
`217`	`217`	`continue;`
`218`	`218`	`}`
`@@ -302,7 +302,7 @@ protected function describeContainerDefinition(Definition $definition, array $op`
`302`	`302`	`$tableRows[] = array('Calls', implode(', ', $callInformation));`
`303`	`303`	`}`
`304`	`304`
`305`		`- $tableRows[] = array('Public', $definition->isPublic() ? 'yes' : 'no');`
	`305`	`+ $tableRows[] = array('Public', $definition->isPublic() && !$definition->isPrivate() ? 'yes' : 'no');`
`306`	`306`	`$tableRows[] = array('Synthetic', $definition->isSynthetic() ? 'yes' : 'no');`
`307`	`307`	`$tableRows[] = array('Lazy', $definition->isLazy() ? 'yes' : 'no');`
`308`	`308`	`$tableRows[] = array('Shared', $definition->isShared() ? 'yes' : 'no');`
Original file line number	Diff line number	Diff line change
`@@ -318,7 +318,7 @@ private function getContainerServicesDocument(ContainerBuilder $builder, $tag =`
`318`	`318`	`foreach ($this->sortServiceIds($serviceIds) as $serviceId) {`
`319`	`319`	`$service = $this->resolveServiceDefinition($builder, $serviceId);`
`320`	`320`
`321`		`- if (($service instanceof Definition \|\| $service instanceof Alias) && !($showPrivate \|\| $service->isPublic())) {`
	`321`	`+ if (($service instanceof Definition \|\| $service instanceof Alias) && !($showPrivate \|\| ($service->isPublic() && !$service->isPrivate()))) {`
`322`	`322`	`continue;`
`323`	`323`	`}`
`324`	`324`
`@@ -364,7 +364,7 @@ private function getContainerDefinitionDocument(Definition $definition, $id = nu`
`364`	`364`	`}`
`365`	`365`	`}`
`366`	`366`
`367`		`- $serviceXML->setAttribute('public', $definition->isPublic() ? 'true' : 'false');`
	`367`	`+ $serviceXML->setAttribute('public', $definition->isPublic() && !$definition->isPrivate() ? 'true' : 'false');`
`368`	`368`	`$serviceXML->setAttribute('synthetic', $definition->isSynthetic() ? 'true' : 'false');`
`369`	`369`	`$serviceXML->setAttribute('lazy', $definition->isLazy() ? 'true' : 'false');`
`370`	`370`	`$serviceXML->setAttribute('shared', $definition->isShared() ? 'true' : 'false');`
`@@ -469,7 +469,7 @@ private function getContainerAliasDocument(Alias $alias, $id = null)`
`469`	`469`	`}`
`470`	`470`
`471`	`471`	`$aliasXML->setAttribute('service', (string) $alias);`
`472`		`- $aliasXML->setAttribute('public', $alias->isPublic() ? 'true' : 'false');`
	`472`	`+ $aliasXML->setAttribute('public', $alias->isPublic() && !$alias->isPrivate() ? 'true' : 'false');`
`473`	`473`
`474`	`474`	`return $dom;`
`475`	`475`	`}`