Skip to content

Commit ad50827

Browse files
gimlergimler
committed
[FrameworkBundle] add class description to debug:container command
1 parent a1aee05 commit ad50827

File tree

12 files changed

+57
-3
lines changed

12 files changed

+57
-3
lines changed

‎src/Symfony/Bundle/FrameworkBundle/Console/Descriptor/Descriptor.php‎

Lines changed: 26 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -11,6 +11,7 @@
1111

1212
namespace Symfony\Bundle\FrameworkBundle\Console\Descriptor;
1313

14+
use phpDocumentor\Reflection\DocBlockFactory;
1415
use Symfony\Component\Console\Descriptor\DescriptorInterface;
1516
use Symfony\Component\Console\Output\OutputInterface;
1617
use Symfony\Component\DependencyInjection\Alias;
@@ -284,4 +285,29 @@ protected function sortServiceIds(array $serviceIds)
284285

285286
return $serviceIds;
286287
}
288+
289+
/**
290+
* Get class description from doc block.
291+
*
292+
* @param string $class
293+
*
294+
* @return string
295+
*/
296+
protected function getClassDescription($class)
297+
{
298+
if (!interface_exists('phpDocumentor\Reflection\DocBlockFactoryInterface')) {
299+
return '';
300+
}
301+
302+
try {
303+
$reflectionProperty = new \ReflectionClass($class);
304+
305+
return DocBlockFactory::createInstance()
306+
->create($reflectionProperty->getDocComment())
307+
->getSummary();
308+
} catch (\ReflectionException $e) {
309+
}
310+
311+
return '';
312+
}
287313
}

‎src/Symfony/Bundle/FrameworkBundle/Console/Descriptor/JsonDescriptor.php‎

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -220,6 +220,10 @@ private function getContainerDefinitionData(Definition $definition, bool $omitTa
220220
'autoconfigure' => $definition->isAutoconfigured(),
221221
);
222222

223+
if ('' !== $classDescription = $this->getClassDescription($definition->getClass())) {
224+
$data['description'] = $classDescription;
225+
}
226+
223227
if ($showArguments) {
224228
$data['arguments'] = $this->describeValue($definition->getArguments(), $omitTags, $showArguments);
225229
}

‎src/Symfony/Bundle/FrameworkBundle/Console/Descriptor/MarkdownDescriptor.php‎

Lines changed: 7 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -181,7 +181,13 @@ protected function describeContainerServices(ContainerBuilder $builder, array $o
181181
*/
182182
protected function describeContainerDefinition(Definition $definition, array $options = array())
183183
{
184-
$output = '- Class: `'.$definition->getClass().'`'
184+
$output = '';
185+
186+
if ('' !== $classDescription = $this->getClassDescription($definition->getClass())) {
187+
$output .= '- Description: `'.$classDescription.'`'."\n";
188+
}
189+
190+
$output .= '- Class: `'.$definition->getClass().'`'
185191
."\n".'- Public: '.($definition->isPublic() && !$definition->isPrivate() ? 'yes' : 'no')
186192
."\n".'- Synthetic: '.($definition->isSynthetic() ? 'yes' : 'no')
187193
."\n".'- Lazy: '.($definition->isLazy() ? 'yes' : 'no')

‎src/Symfony/Bundle/FrameworkBundle/Console/Descriptor/TextDescriptor.php‎

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -260,6 +260,10 @@ protected function describeContainerDefinition(Definition $definition, array $op
260260
$options['output']->title(sprintf('Information for Service "<info>%s</info>"', $options['id']));
261261
}
262262

263+
if ('' !== $classDescription = $this->getClassDescription($definition->getClass())) {
264+
$options['output']->text($classDescription."\n");
265+
}
266+
263267
$tableHeaders = array('Option', 'Value');
264268

265269
$tableRows[] = array('Service ID', isset($options['id']) ? $options['id'] : '-');

‎src/Symfony/Bundle/FrameworkBundle/Console/Descriptor/XmlDescriptor.php‎

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -304,6 +304,11 @@ private function getContainerDefinitionDocument(Definition $definition, string $
304304
$serviceXML->setAttribute('id', $id);
305305
}
306306

307+
if ('' !== $classDescription = $this->getClassDescription($definition->getClass())) {
308+
$serviceXML->appendChild($descriptionXML = $dom->createElement('description'));
309+
$descriptionXML->appendChild($dom->createCDATASection($classDescription));
310+
}
311+
307312
$serviceXML->setAttribute('class', $definition->getClass());
308313

309314
if ($factory = $definition->getFactory()) {

‎src/Symfony/Bundle/FrameworkBundle/Tests/Fixtures/Descriptor/builder_1_arguments.json‎

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -76,6 +76,7 @@
7676
"abstract": false,
7777
"autowire": false,
7878
"autoconfigure": false,
79+
"description": "ContainerInterface is the interface implemented by service container classes.",
7980
"arguments": [],
8081
"file": null,
8182
"tags": []

‎src/Symfony/Bundle/FrameworkBundle/Tests/Fixtures/Descriptor/builder_1_arguments.md‎

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -20,6 +20,7 @@ Definitions
2020

2121
### service_container
2222

23+
- Description: `ContainerInterface is the interface implemented by service container classes.`
2324
- Class: `Symfony\Component\DependencyInjection\ContainerInterface`
2425
- Public: yes
2526
- Synthetic: yes

‎src/Symfony/Bundle/FrameworkBundle/Tests/Fixtures/Descriptor/builder_1_arguments.xml‎

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -23,5 +23,7 @@
2323
<argument type="service" id=".definition_2"/>
2424
</argument>
2525
</definition>
26-
<definition id="service_container" class="Symfony\Component\DependencyInjection\ContainerInterface" public="true" synthetic="true" lazy="false" shared="true" abstract="false" autowired="false" autoconfigured="false" file=""/>
26+
<definition id="service_container" class="Symfony\Component\DependencyInjection\ContainerInterface" public="true" synthetic="true" lazy="false" shared="true" abstract="false" autowired="false" autoconfigured="false" file="">
27+
<description><![CDATA[ContainerInterface is the interface implemented by service container classes.]]></description>
28+
</definition>
2729
</container>

‎src/Symfony/Bundle/FrameworkBundle/Tests/Fixtures/Descriptor/builder_1_public.json‎

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -23,6 +23,7 @@
2323
"abstract": false,
2424
"autowire": false,
2525
"autoconfigure": false,
26+
"description": "ContainerInterface is the interface implemented by service container classes.",
2627
"file": null,
2728
"tags": []
2829
}

‎src/Symfony/Bundle/FrameworkBundle/Tests/Fixtures/Descriptor/builder_1_public.md‎

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -19,6 +19,7 @@ Definitions
1919

2020
### service_container
2121

22+
- Description: `ContainerInterface is the interface implemented by service container classes.`
2223
- Class: `Symfony\Component\DependencyInjection\ContainerInterface`
2324
- Public: yes
2425
- Synthetic: yes

0 commit comments

Comments
 (0)