diff --git a/src/Symfony/Bundle/FrameworkBundle/Console/Descriptor/JsonDescriptor.php b/src/Symfony/Bundle/FrameworkBundle/Console/Descriptor/JsonDescriptor.php
index d22ebcd181785..a6167b9a48aa9 100644
--- a/src/Symfony/Bundle/FrameworkBundle/Console/Descriptor/JsonDescriptor.php
+++ b/src/Symfony/Bundle/FrameworkBundle/Console/Descriptor/JsonDescriptor.php
@@ -217,29 +217,18 @@ private function getContainerDefinitionData(Definition $definition, $omitTags =
'public' => $definition->isPublic(),
'synthetic' => $definition->isSynthetic(),
'lazy' => $definition->isLazy(),
+ 'shared' => $definition->isShared(),
+ 'synchronized' => $definition->isSynchronized(false),
+ 'abstract' => $definition->isAbstract(),
+ 'autowire' => $definition->isAutowired(),
+ 'autowiring_types' => array(),
+ 'file' => $definition->getFile(),
);
- if (method_exists($definition, 'isShared')) {
- $data['shared'] = $definition->isShared();
+ foreach ($definition->getAutowiringTypes() as $autowiringType) {
+ $data['autowiring_types'][] = $autowiringType;
}
- if (method_exists($definition, 'isSynchronized')) {
- $data['synchronized'] = $definition->isSynchronized(false);
- }
-
- $data['abstract'] = $definition->isAbstract();
-
- if (method_exists($definition, 'isAutowired')) {
- $data['autowire'] = $definition->isAutowired();
-
- $data['autowiring_types'] = array();
- foreach ($definition->getAutowiringTypes() as $autowiringType) {
- $data['autowiring_types'][] = $autowiringType;
- }
- }
-
- $data['file'] = $definition->getFile();
-
if ($definition->getFactoryClass(false)) {
$data['factory_class'] = $definition->getFactoryClass(false);
}
@@ -269,11 +258,9 @@ private function getContainerDefinitionData(Definition $definition, $omitTags =
if (!$omitTags) {
$data['tags'] = array();
- if (count($definition->getTags())) {
- foreach ($definition->getTags() as $tagName => $tagData) {
- foreach ($tagData as $parameters) {
- $data['tags'][] = array('name' => $tagName, 'parameters' => $parameters);
- }
+ foreach ($definition->getTags() as $tagName => $tagData) {
+ foreach ($tagData as $parameters) {
+ $data['tags'][] = array('name' => $tagName, 'parameters' => $parameters);
}
}
}
diff --git a/src/Symfony/Bundle/FrameworkBundle/Console/Descriptor/MarkdownDescriptor.php b/src/Symfony/Bundle/FrameworkBundle/Console/Descriptor/MarkdownDescriptor.php
index 4795331d0f653..d9d3278842eff 100644
--- a/src/Symfony/Bundle/FrameworkBundle/Console/Descriptor/MarkdownDescriptor.php
+++ b/src/Symfony/Bundle/FrameworkBundle/Console/Descriptor/MarkdownDescriptor.php
@@ -183,24 +183,14 @@ protected function describeContainerDefinition(Definition $definition, array $op
."\n".'- Public: '.($definition->isPublic() ? 'yes' : 'no')
."\n".'- Synthetic: '.($definition->isSynthetic() ? 'yes' : 'no')
."\n".'- Lazy: '.($definition->isLazy() ? 'yes' : 'no')
+ ."\n".'- Shared: '.($definition->isShared() ? 'yes' : 'no')
+ ."\n".'- Synchronized: '.($definition->isSynchronized(false) ? 'yes' : 'no')
+ ."\n".'- Abstract: '.($definition->isAbstract() ? 'yes' : 'no')
+ ."\n".'- Autowired: '.($definition->isAutowired() ? 'yes' : 'no')
;
- if (method_exists($definition, 'isShared')) {
- $output .= "\n".'- Shared: '.($definition->isShared() ? 'yes' : 'no');
- }
-
- if (method_exists($definition, 'isSynchronized')) {
- $output .= "\n".'- Synchronized: '.($definition->isSynchronized(false) ? 'yes' : 'no');
- }
-
- $output .= "\n".'- Abstract: '.($definition->isAbstract() ? 'yes' : 'no');
-
- if (method_exists($definition, 'isAutowired')) {
- $output .= "\n".'- Autowired: '.($definition->isAutowired() ? 'yes' : 'no');
-
- foreach ($definition->getAutowiringTypes() as $autowiringType) {
- $output .= "\n".'- Autowiring Type: `'.$autowiringType.'`';
- }
+ foreach ($definition->getAutowiringTypes() as $autowiringType) {
+ $output .= "\n".'- Autowiring Type: `'.$autowiringType.'`';
}
if ($definition->getFile()) {
@@ -371,7 +361,7 @@ protected function describeCallable($callable, array $options = array())
*/
private function formatRouterConfig(array $array)
{
- if (!count($array)) {
+ if (!$array) {
return 'NONE';
}
diff --git a/src/Symfony/Bundle/FrameworkBundle/Console/Descriptor/TextDescriptor.php b/src/Symfony/Bundle/FrameworkBundle/Console/Descriptor/TextDescriptor.php
index 4aa00a85dbb17..1b35f8c565fff 100644
--- a/src/Symfony/Bundle/FrameworkBundle/Console/Descriptor/TextDescriptor.php
+++ b/src/Symfony/Bundle/FrameworkBundle/Console/Descriptor/TextDescriptor.php
@@ -255,8 +255,7 @@ protected function describeContainerDefinition(Definition $definition, array $op
$tableRows[] = array('Service ID', isset($options['id']) ? $options['id'] : '-');
$tableRows[] = array('Class', $definition->getClass() ?: '-');
- $tags = $definition->getTags();
- if (count($tags)) {
+ if ($tags = $definition->getTags()) {
$tagInformation = '';
foreach ($tags as $tagName => $tagData) {
foreach ($tagData as $tagParameters) {
@@ -281,24 +280,12 @@ protected function describeContainerDefinition(Definition $definition, array $op
$tableRows[] = array('Public', $definition->isPublic() ? 'yes' : 'no');
$tableRows[] = array('Synthetic', $definition->isSynthetic() ? 'yes' : 'no');
$tableRows[] = array('Lazy', $definition->isLazy() ? 'yes' : 'no');
-
- if (method_exists($definition, 'isSynchronized')) {
- $tableRows[] = array('Synchronized', $definition->isSynchronized(false) ? 'yes' : 'no');
- }
+ $tableRows[] = array('Synchronized', $definition->isSynchronized(false) ? 'yes' : 'no');
$tableRows[] = array('Abstract', $definition->isAbstract() ? 'yes' : 'no');
+ $tableRows[] = array('Autowired', $definition->isAutowired() ? 'yes' : 'no');
- if (method_exists($definition, 'isAutowired')) {
- $tableRows[] = array('Autowired', $definition->isAutowired() ? 'yes' : 'no');
-
- $autowiringTypes = $definition->getAutowiringTypes();
- if (count($autowiringTypes)) {
- $autowiringTypesInformation = implode(', ', $autowiringTypes);
- } else {
- $autowiringTypesInformation = '-';
- }
-
- $tableRows[] = array('Autowiring Types', $autowiringTypesInformation);
- }
+ $autowiringTypes = $definition->getAutowiringTypes();
+ $tableRows[] = array('Autowiring Types', $autowiringTypes ? implode(', ', $autowiringTypes) : '-');
if ($definition->getFile()) {
$tableRows[] = array('Required File', $definition->getFile() ? $definition->getFile() : '-');
diff --git a/src/Symfony/Bundle/FrameworkBundle/Console/Descriptor/XmlDescriptor.php b/src/Symfony/Bundle/FrameworkBundle/Console/Descriptor/XmlDescriptor.php
index 62fb688e3c06f..0f0533573d4c7 100644
--- a/src/Symfony/Bundle/FrameworkBundle/Console/Descriptor/XmlDescriptor.php
+++ b/src/Symfony/Bundle/FrameworkBundle/Console/Descriptor/XmlDescriptor.php
@@ -187,7 +187,7 @@ private function getRouteDocument(Route $route, $name = null)
$methodXML->appendChild(new \DOMText($method));
}
- if (count($route->getDefaults())) {
+ if ($route->getDefaults()) {
$routeXML->appendChild($defaultsXML = $dom->createElement('defaults'));
foreach ($route->getDefaults() as $attribute => $value) {
$defaultsXML->appendChild($defaultXML = $dom->createElement('default'));
@@ -198,7 +198,7 @@ private function getRouteDocument(Route $route, $name = null)
$requirements = $route->getRequirements();
unset($requirements['_scheme'], $requirements['_method']);
- if (count($requirements)) {
+ if ($requirements) {
$routeXML->appendChild($requirementsXML = $dom->createElement('requirements'));
foreach ($requirements as $attribute => $pattern) {
$requirementsXML->appendChild($requirementXML = $dom->createElement('requirement'));
@@ -207,7 +207,7 @@ private function getRouteDocument(Route $route, $name = null)
}
}
- if (count($route->getOptions())) {
+ if ($route->getOptions()) {
$routeXML->appendChild($optionsXML = $dom->createElement('options'));
foreach ($route->getOptions() as $name => $value) {
$optionsXML->appendChild($optionXML = $dom->createElement('option'));
@@ -331,18 +331,16 @@ private function getContainerDefinitionDocument(Definition $definition, $id = nu
$serviceXML->setAttribute('class', $definition->getClass());
- if (method_exists($definition, 'getFactoryMethod')) {
- if ($definition->getFactoryClass(false)) {
- $serviceXML->setAttribute('factory-class', $definition->getFactoryClass(false));
- }
+ if ($definition->getFactoryClass(false)) {
+ $serviceXML->setAttribute('factory-class', $definition->getFactoryClass(false));
+ }
- if ($definition->getFactoryService(false)) {
- $serviceXML->setAttribute('factory-service', $definition->getFactoryService(false));
- }
+ if ($definition->getFactoryService(false)) {
+ $serviceXML->setAttribute('factory-service', $definition->getFactoryService(false));
+ }
- if ($definition->getFactoryMethod(false)) {
- $serviceXML->setAttribute('factory-method', $definition->getFactoryMethod(false));
- }
+ if ($definition->getFactoryMethod(false)) {
+ $serviceXML->setAttribute('factory-method', $definition->getFactoryMethod(false));
}
if ($factory = $definition->getFactory()) {
@@ -366,24 +364,14 @@ private function getContainerDefinitionDocument(Definition $definition, $id = nu
$serviceXML->setAttribute('public', $definition->isPublic() ? 'true' : 'false');
$serviceXML->setAttribute('synthetic', $definition->isSynthetic() ? 'true' : 'false');
$serviceXML->setAttribute('lazy', $definition->isLazy() ? 'true' : 'false');
- if (method_exists($definition, 'isShared')) {
- $serviceXML->setAttribute('shared', $definition->isShared() ? 'true' : 'false');
- }
- if (method_exists($definition, 'isSynchronized')) {
- $serviceXML->setAttribute('synchronized', $definition->isSynchronized(false) ? 'true' : 'false');
- }
+ $serviceXML->setAttribute('shared', $definition->isShared() ? 'true' : 'false');
+ $serviceXML->setAttribute('synchronized', $definition->isSynchronized(false) ? 'true' : 'false');
$serviceXML->setAttribute('abstract', $definition->isAbstract() ? 'true' : 'false');
-
- if (method_exists($definition, 'isAutowired')) {
- $serviceXML->setAttribute('autowired', $definition->isAutowired() ? 'true' : 'false');
- }
-
+ $serviceXML->setAttribute('autowired', $definition->isAutowired() ? 'true' : 'false');
$serviceXML->setAttribute('file', $definition->getFile());
if (!$omitTags) {
- $tags = $definition->getTags();
-
- if (count($tags) > 0) {
+ if ($tags = $definition->getTags()) {
$serviceXML->appendChild($tagsXML = $dom->createElement('tags'));
foreach ($tags as $tagName => $tagData) {
foreach ($tagData as $parameters) {
pFad - Phonifier reborn
Pfad - The Proxy pFad of © 2024 Garber Painting. All rights reserved.
Note: This service is not intended for secure transactions such as banking, social media, email, or purchasing. Use at your own risk. We assume no liability whatsoever for broken pages.
Alternative Proxies:
Alternative Proxy
pFad Proxy
pFad v3 Proxy
pFad v4 Proxy