Merge branch '2.6' into 2.7

* 2.6:
  [2.3] CS And DocBlock Fixes
  [2.3] CS Fixes
  [FrameworkBundle] Fixed Translation loader and update translation command.
  [Console] remove « use » statement for PHP built-in exception classes.
  [SecurityBundle] adds unit tests suite for SecurityDataCollector class.

Conflicts:
	src/Symfony/Bundle/SecurityBundle/DataCollector/SecurityDataCollector.php

src/Symfony/Bridge/Doctrine/ContainerAwareEventManager.php

@@ -24,6 +24,7 @@ class ContainerAwareEventManager extends EventManager
 {
     /**
      * Map of registered listeners.
+     *
      * <event> => <listeners>
      *
      * @var array

src/Symfony/Bridge/Doctrine/DependencyInjection/CompilerPass/DoctrineValidationPass.php

@@ -16,7 +16,7 @@ use Symfony\Component\DependencyInjection\Compiler\CompilerPassInterface;
 use Symfony\Component\Config\Resource\FileResource;
 
 /**
- * Registers additional validators
+ * Registers additional validators.
  *
  * @author Benjamin Eberlei <kontakt@beberlei.de>
  */
@@ -43,7 +43,7 @@ class DoctrineValidationPass implements CompilerPassInterface
 
     /**
      * Gets the validation mapping files for the format and extends them with
-     * files matching a doctrine search pattern (Resources/config/validation.orm.xml)
+     * files matching a doctrine search pattern (Resources/config/validation.orm.xml).
      *
      * @param ContainerBuilder $container
      * @param string           $mapping

src/Symfony/Bridge/Doctrine/DependencyInjection/CompilerPass/RegisterMappingsPass.php

@@ -35,12 +35,14 @@ abstract class RegisterMappingsPass implements CompilerPassInterface
     /**
      * DI object for the driver to use, either a service definition for a
      * private service or a reference for a public service.
+     *
      * @var Definition|Reference
      */
     protected $driver;
 
     /**
-     * List of namespaces handled by the driver
+     * List of namespaces handled by the driver.
+     *
      * @var string[]
      */
     protected $namespaces;
@@ -48,14 +50,16 @@ abstract class RegisterMappingsPass implements CompilerPassInterface
     /**
      * List of potential container parameters that hold the object manager name
      * to register the mappings with the correct metadata driver, for example
-     * array('acme.manager', 'doctrine.default_entity_manager')
+     * array('acme.manager', 'doctrine.default_entity_manager').
+     *
      * @var string[]
      */
     protected $managerParameters;
 
     /**
      * Naming pattern of the metadata chain driver service ids, for example
-     * 'doctrine.orm.%s_metadata_driver'
+     * 'doctrine.orm.%s_metadata_driver'.
+     *
      * @var string
      */
     protected $driverPattern;
@@ -64,6 +68,7 @@ abstract class RegisterMappingsPass implements CompilerPassInterface
      * A name for a parameter in the container. If set, this compiler pass will
      * only do anything if the parameter is present. (But regardless of the
      * value of that parameter.
+     *
      * @var string
      */
     protected $enabledParameter;

src/Symfony/Bridge/Doctrine/Form/ChoiceList/EntityChoiceList.php

@@ -18,7 +18,7 @@ use Doctrine\Common\Persistence\ObjectManager;
 use Symfony\Component\PropertyAccess\PropertyAccessorInterface;
 
 /**
- * A choice list presenting a list of Doctrine entities as choices
+ * A choice list presenting a list of Doctrine entities as choices.
  *
  * @author Bernhard Schussek <bschussek@gmail.com>
  */
@@ -41,7 +41,7 @@ class EntityChoiceList extends ObjectChoiceList
 
     /**
      * Contains the query builder that builds the query for fetching the
-     * entities
+     * entities.
      *
      * This property should only be accessed through queryBuilder.
      *
@@ -50,21 +50,21 @@ class EntityChoiceList extends ObjectChoiceList
     private $entityLoader;
 
     /**
-     * The identifier field, if the identifier is not composite
+     * The identifier field, if the identifier is not composite.
      *
      * @var array
      */
     private $idField = null;
 
     /**
-     * Whether to use the identifier for index generation
+     * Whether to use the identifier for index generation.
      *
      * @var bool
      */
     private $idAsIndex = false;
 
     /**
-     * Whether to use the identifier for value generation
+     * Whether to use the identifier for value generation.
      *
      * @var bool
      */
@@ -98,7 +98,7 @@ class EntityChoiceList extends ObjectChoiceList
      *                                                     the choices are given as flat array.
      * @param PropertyAccessorInterface $propertyAccessor  The reflection graph for reading property paths.
      */
-    public function __construct(ObjectManager $manager, $class, $labelPath = null, EntityLoaderInterface $entityLoader = null, $entities = null,  array $preferredEntities = array(), $groupPath = null, PropertyAccessorInterface $propertyAccessor = null)
+    public function __construct(ObjectManager $manager, $class, $labelPath = null, EntityLoaderInterface $entityLoader = null, $entities = null, array $preferredEntities = array(), $groupPath = null, PropertyAccessorInterface $propertyAccessor = null)
     {
         $this->em = $manager;
         $this->entityLoader = $entityLoader;
@@ -128,7 +128,7 @@ class EntityChoiceList extends ObjectChoiceList
     }
 
     /**
-     * Returns the list of entities
+     * Returns the list of entities.
      *
      * @return array
      *
@@ -144,7 +144,7 @@ class EntityChoiceList extends ObjectChoiceList
     }
 
     /**
-     * Returns the values for the entities
+     * Returns the values for the entities.
      *
      * @return array
      *

src/Symfony/Bridge/Doctrine/Form/ChoiceList/ORMQueryBuilderLoader.php

@@ -17,13 +17,13 @@ use Doctrine\DBAL\Connection;
 use Doctrine\ORM\EntityManager;
 
 /**
- * Getting Entities through the ORM QueryBuilder
+ * Getting Entities through the ORM QueryBuilder.
  */
 class ORMQueryBuilderLoader implements EntityLoaderInterface
 {
     /**
      * Contains the query builder that builds the query for fetching the
-     * entities
+     * entities.
      *
      * This property should only be accessed through queryBuilder.
      *
@@ -32,7 +32,7 @@ class ORMQueryBuilderLoader implements EntityLoaderInterface
     private $queryBuilder;
 
     /**
-     * Construct an ORM Query Builder Loader
+     * Construct an ORM Query Builder Loader.
      *
      * @param QueryBuilder|\Closure $queryBuilder
      * @param EntityManager         $manager

src/Symfony/Bridge/Doctrine/RegistryInterface.php

@@ -39,7 +39,7 @@ interface RegistryInterface extends ManagerRegistryInterface
     public function getEntityManager($name = null);
 
     /**
-     * Gets an array of all registered entity managers
+     * Gets an array of all registered entity managers.
      *
      * @return array An array of EntityManager instances
      */

src/Symfony/Bridge/Doctrine/Security/RememberMe/DoctrineTokenProvider.php

@@ -39,14 +39,14 @@ class DoctrineTokenProvider implements TokenProviderInterface
 {
     /**
      * Doctrine DBAL database connection
-     * F.ex. service id: doctrine.dbal.default_connection
+     * F.ex. service id: doctrine.dbal.default_connection.
      *
      * @var \Doctrine\DBAL\Connection
      */
     private $conn;
 
     /**
-     * new DoctrineTokenProvider for the RememberMe authentication service
+     * new DoctrineTokenProvider for the RememberMe authentication service.
      *
      * @param \Doctrine\DBAL\Connection $conn
      */

src/Symfony/Bridge/Doctrine/Tests/DoctrineOrmTestCase.php

@@ -14,7 +14,7 @@ namespace Symfony\Bridge\Doctrine\Tests;
 use Symfony\Bridge\Doctrine\Test\DoctrineTestHelper;
 
 /**
- * Class DoctrineOrmTestCase
+ * Class DoctrineOrmTestCase.
  *
  * @deprecated Deprecated as of Symfony 2.4, to be removed in Symfony 3.0.
  *             Use {@link DoctrineTestHelper} instead.

src/Symfony/Bridge/Doctrine/Tests/Fixtures/AssociationEntity.php

@@ -27,6 +27,7 @@ class AssociationEntity
 
     /**
      * @ORM\ManyToOne(targetEntity="SingleIntIdEntity")
+     *
      * @var \Symfony\Bridge\Doctrine\Tests\Fixtures\SingleIntIdEntity
      */
     public $single;
@@ -37,6 +38,7 @@ class AssociationEntity
      *  @ORM\JoinColumn(name="composite_id1", referencedColumnName="id1"),
      *  @ORM\JoinColumn(name="composite_id2", referencedColumnName="id2")
      * })
+     *
      * @var \Symfony\Bridge\Doctrine\Tests\Fixtures\CompositeIntIdEntity
      */
     public $composite;

src/Symfony/Bridge/Doctrine/Validator/Constraints/UniqueEntity.php

@@ -14,7 +14,7 @@ namespace Symfony\Bridge\Doctrine\Validator\Constraints;
 use Symfony\Component\Validator\Constraint;
 
 /**
- * Constraint for the Unique Entity validator
+ * Constraint for the Unique Entity validator.
  *
  * @Annotation
  * @Target({"CLASS", "ANNOTATION"})

src/Symfony/Bridge/Monolog/Handler/ChromePhpHandler.php

@@ -33,7 +33,7 @@ class ChromePhpHandler extends BaseChromePhpHandler
     private $response;
 
     /**
-     * Adds the headers to the response once it's created
+     * Adds the headers to the response once it's created.
      */
     public function onKernelResponse(FilterResponseEvent $event)
     {
@@ -72,7 +72,7 @@ class ChromePhpHandler extends BaseChromePhpHandler
     }
 
     /**
-     * Override default behavior since we check it in onKernelResponse
+     * Override default behavior since we check it in onKernelResponse.
      */
     protected function headersAccepted()
     {

src/Symfony/Bridge/Monolog/Handler/FirePHPHandler.php

@@ -33,7 +33,7 @@ class FirePHPHandler extends BaseFirePHPHandler
     private $response;
 
     /**
-     * Adds the headers to the response once it's created
+     * Adds the headers to the response once it's created.
      */
     public function onKernelResponse(FilterResponseEvent $event)
     {
@@ -73,7 +73,7 @@ class FirePHPHandler extends BaseFirePHPHandler
     }
 
     /**
-     * Override default behavior since we check the user agent in onKernelResponse
+     * Override default behavior since we check the user agent in onKernelResponse.
      */
     protected function headersAccepted()
     {

src/Symfony/Bridge/Monolog/Handler/SwiftMailerHandler.php

@@ -16,7 +16,7 @@ use Symfony\Component\Console\Event\ConsoleTerminateEvent;
 use Symfony\Component\HttpKernel\Event\PostResponseEvent;
 
 /**
- * Extended SwiftMailerHandler that flushes mail queue if necessary
+ * Extended SwiftMailerHandler that flushes mail queue if necessary.
  *
  * @author Philipp Kräutli <pkraeutli@astina.ch>
  */
@@ -35,7 +35,7 @@ class SwiftMailerHandler extends BaseSwiftMailerHandler
     }
 
     /**
-     * After the kernel has been terminated we will always flush messages
+     * After the kernel has been terminated we will always flush messages.
      *
      * @param PostResponseEvent $event
      */
@@ -67,7 +67,7 @@ class SwiftMailerHandler extends BaseSwiftMailerHandler
     }
 
     /**
-     * Flushes the mail queue if a memory spool is used
+     * Flushes the mail queue if a memory spool is used.
      */
     private function flushMemorySpool()
     {

src/Symfony/Bridge/Monolog/Processor/WebProcessor.php

@@ -15,7 +15,7 @@ use Monolog\Processor\WebProcessor as BaseWebProcessor;
 use Symfony\Component\HttpKernel\Event\GetResponseEvent;
 
 /**
- * WebProcessor override to read from the HttpFoundation's Request
+ * WebProcessor override to read from the HttpFoundation's Request.
  *
  * @author Jordi Boggiano <j.boggiano@seld.be>
  */

src/Symfony/Bridge/Propel1/DataCollector/PropelDataCollector.php

@@ -24,21 +24,21 @@ use Symfony\Component\HttpKernel\DataCollector\DataCollector;
 class PropelDataCollector extends DataCollector
 {
     /**
-     * Propel logger
+     * Propel logger.
      *
      * @var \Symfony\Bridge\Propel1\Logger\PropelLogger
      */
     private $logger;
 
     /**
-     * Propel configuration
+     * Propel configuration.
      *
      * @var \PropelConfiguration
      */
     protected $propelConfiguration;
 
     /**
-     * Constructor
+     * Constructor.
      *
      * @param PropelLogger         $logger              A Propel logger.
      * @param \PropelConfiguration $propelConfiguration The Propel configuration object.

src/Symfony/Bridge/Propel1/Form/ChoiceList/ModelChoiceList.php

@@ -29,7 +29,7 @@ use Symfony\Component\PropertyAccess\PropertyAccessorInterface;
 class ModelChoiceList extends ObjectChoiceList
 {
     /**
-     * The fields of which the identifier of the underlying class consists
+     * The fields of which the identifier of the underlying class consists.
      *
      * This property should only be accessed through identifier.
      *

src/Symfony/Bridge/Propel1/Form/EventListener/TranslationCollectionFormListener.php

@@ -16,7 +16,7 @@ use Symfony\Component\EventDispatcher\EventSubscriberInterface;
 use Symfony\Component\Form\FormEvent;
 
 /**
- * listener class for propel1_translatable_collection
+ * listener class for propel1_translatable_collection.
  *
  * @author Patrick Kaufmann
  */

src/Symfony/Bridge/Propel1/Form/EventListener/TranslationFormListener.php

@@ -15,7 +15,7 @@ use Symfony\Component\Form\FormEvent;
 use Symfony\Component\Form\FormEvents;
 
 /**
- * Event Listener class for propel1_translation
+ * Event Listener class for propel1_translation.
  *
  * @author Patrick Kaufmann
  */

src/Symfony/Bridge/Propel1/Form/Type/TranslationCollectionType.php

@@ -18,7 +18,7 @@ use Symfony\Component\OptionsResolver\OptionsResolverInterface;
 use Symfony\Bridge\Propel1\Form\EventListener\TranslationCollectionFormListener;
 
 /**
- * form type for i18n-columns in propel
+ * form type for i18n-columns in propel.
  *
  * @author Patrick Kaufmann
  */

src/Symfony/Bridge/Propel1/Form/Type/TranslationType.php

@@ -17,7 +17,7 @@ use Symfony\Component\OptionsResolver\OptionsResolverInterface;
 use Symfony\Bridge\Propel1\Form\EventListener\TranslationFormListener;
 
 /**
- * Translation type class
+ * Translation type class.
  *
  * @author Patrick Kaufmann
  */

src/Symfony/Bridge/Propel1/Security/User/PropelUserProvider.php

@@ -45,7 +45,7 @@ class PropelUserProvider implements UserProviderInterface
     protected $property;
 
     /**
-     * Default constructor
+     * Default constructor.
      *
      * @param string      $class    The User model class.
      * @param string|null $property The property to use to retrieve a user.

src/Symfony/Bridge/Propel1/Tests/Fixtures/ItemQuery.php

@@ -57,7 +57,7 @@ class ItemQuery
     }
 
     /**
-     * Method from the TableMap API
+     * Method from the TableMap API.
      */
     public function hasColumn($column)
     {
@@ -65,7 +65,7 @@ class ItemQuery
     }
 
     /**
-     * Method from the TableMap API
+     * Method from the TableMap API.
      */
     public function getColumn($column)
     {
@@ -75,7 +75,7 @@ class ItemQuery
     }
 
     /**
-     * Method from the TableMap API
+     * Method from the TableMap API.
      */
     public function hasColumnByInsensitiveCase($column)
     {

src/Symfony/Bridge/ProxyManager/LazyProxy/Instantiator/RuntimeInstantiator.php

@@ -32,7 +32,7 @@ class RuntimeInstantiator implements InstantiatorInterface
     private $factory;
 
     /**
-     * Constructor
+     * Constructor.
      */
     public function __construct()
     {

src/Symfony/Bridge/ProxyManager/LazyProxy/PhpDumper/ProxyDumper.php

@@ -37,7 +37,7 @@ class ProxyDumper implements DumperInterface
     private $classGenerator;
 
     /**
-     * Constructor
+     * Constructor.
      */
     public function __construct()
     {

src/Symfony/Bridge/ProxyManager/Tests/LazyProxy/ContainerBuilderTest.php

@@ -18,7 +18,7 @@ use Symfony\Component\DependencyInjection\ContainerBuilder;
 
 /**
  * Integration tests for {@see \Symfony\Component\DependencyInjection\ContainerBuilder} combined
- * with the ProxyManager bridge
+ * with the ProxyManager bridge.
  *
  * @author Marco Pivetta <ocramius@gmail.com>
  */

src/Symfony/Bridge/ProxyManager/Tests/LazyProxy/Dumper/PhpDumperTest.php

@@ -17,7 +17,7 @@ use Symfony\Component\DependencyInjection\Dumper\PhpDumper;
 
 /**
  * Integration tests for {@see \Symfony\Component\DependencyInjection\Dumper\PhpDumper} combined
- * with the ProxyManager bridge
+ * with the ProxyManager bridge.
  *
  * @author Marco Pivetta <ocramius@gmail.com>
  */
@@ -45,7 +45,7 @@ class PhpDumperTest extends \PHPUnit_Framework_TestCase
     }
 
     /**
-     * Verifies that the generated container retrieves the same proxy instance on multiple subsequent requests
+     * Verifies that the generated container retrieves the same proxy instance on multiple subsequent requests.
      */
     public function testDumpContainerWithProxyServiceWillShareProxies()
     {

src/Symfony/Bridge/ProxyManager/Tests/LazyProxy/Fixtures/php/lazy_service.php

@@ -11,7 +11,7 @@ use Symfony\Component\DependencyInjection\Parameter;
 use Symfony\Component\DependencyInjection\ParameterBag\FrozenParameterBag;
 
 /**
- * ProjectServiceContainer
+ * ProjectServiceContainer.
  *
  * This class has been auto-generated
  * by the Symfony Dependency Injection Component.

src/Symfony/Bridge/ProxyManager/Tests/LazyProxy/Instantiator/RuntimeInstantiatorTest.php

@@ -17,7 +17,7 @@ use Symfony\Component\DependencyInjection\ContainerInterface;
 use Symfony\Component\DependencyInjection\Definition;
 
 /**
- * Tests for {@see \Symfony\Bridge\ProxyManager\LazyProxy\Instantiator\RuntimeInstantiator}
+ * Tests for {@see \Symfony\Bridge\ProxyManager\LazyProxy\Instantiator\RuntimeInstantiator}.
  *
  * @author Marco Pivetta <ocramius@gmail.com>
  *

src/Symfony/Bridge/ProxyManager/Tests/LazyProxy/PhpDumper/ProxyDumperTest.php

@@ -15,7 +15,7 @@ use Symfony\Bridge\ProxyManager\LazyProxy\PhpDumper\ProxyDumper;
 use Symfony\Component\DependencyInjection\Definition;
 
 /**
- * Tests for {@see \Symfony\Bridge\ProxyManager\LazyProxy\PhpDumper\ProxyDumper}
+ * Tests for {@see \Symfony\Bridge\ProxyManager\LazyProxy\PhpDumper\ProxyDumper}.
  *
  * @author Marco Pivetta <ocramius@gmail.com>
  *

src/Symfony/Bundle/FrameworkBundle/Command/ContainerAwareCommand.php

@@ -29,6 +29,7 @@ abstract class ContainerAwareCommand extends Command implements ContainerAwareIn
 
     /**
      * @return ContainerInterface
+     *
      * @throws \LogicException
      */
     protected function getContainer()

src/Symfony/Bundle/FrameworkBundle/Command/TranslationUpdateCommand.php

@@ -136,6 +136,13 @@ EOF
             ? new DiffOperation($currentCatalogue, $extractedCatalogue)
             : new MergeOperation($currentCatalogue, $extractedCatalogue);
 
+        // Exit if no messages found.
+        if (!count($operation->getDomains())) {
+            $output->writeln("\n<comment>No translation found.</comment>");
+
+            return;
+        }
+
         // show compiled list of messages
         if ($input->getOption('dump-messages') === true) {
             foreach ($operation->getDomains() as $domain) {

src/Symfony/Bundle/FrameworkBundle/Controller/Controller.php

@@ -249,7 +249,7 @@ class Controller extends ContainerAware
     }
 
     /**
-     * Creates and returns a form builder instance
+     * Creates and returns a form builder instance.
      *
      * @param mixed $data    The initial data for the form
      * @param array $options Options for the form
@@ -294,7 +294,7 @@ class Controller extends ContainerAware
     }
 
     /**
-     * Get a user from the Security Context
+     * Get a user from the Security Context.
      *
      * @return mixed
      *

src/Symfony/Bundle/FrameworkBundle/DependencyInjection/Compiler/FormPass.php

@@ -16,7 +16,7 @@ use Symfony\Component\DependencyInjection\Compiler\CompilerPassInterface;
 
 /**
  * Adds all services with the tags "form.type" and "form.type_guesser" as
- * arguments of the "form.extension" service
+ * arguments of the "form.extension" service.
  *
  * @author Bernhard Schussek <bschussek@gmail.com>
  */

src/Symfony/Bundle/FrameworkBundle/DependencyInjection/Compiler/ProfilerPass.php

@@ -16,7 +16,7 @@ use Symfony\Component\DependencyInjection\ContainerBuilder;
 use Symfony\Component\DependencyInjection\Compiler\CompilerPassInterface;
 
 /**
- * Adds tagged data_collector services to profiler service
+ * Adds tagged data_collector services to profiler service.
  *
  * @author Fabien Potencier <fabien@symfony.com>
  */

src/Symfony/Bundle/FrameworkBundle/DependencyInjection/Compiler/RoutingResolverPass.php

@@ -16,7 +16,7 @@ use Symfony\Component\DependencyInjection\ContainerBuilder;
 use Symfony\Component\DependencyInjection\Compiler\CompilerPassInterface;
 
 /**
- * Adds tagged routing.loader services to routing.resolver service
+ * Adds tagged routing.loader services to routing.resolver service.
  *
  * @author Fabien Potencier <fabien@symfony.com>
  */

src/Symfony/Bundle/FrameworkBundle/DependencyInjection/Compiler/TranslationDumperPass.php

@@ -16,7 +16,7 @@ use Symfony\Component\DependencyInjection\ContainerBuilder;
 use Symfony\Component\DependencyInjection\Compiler\CompilerPassInterface;
 
 /**
- * Adds tagged translation.formatter services to translation writer
+ * Adds tagged translation.formatter services to translation writer.
  */
 class TranslationDumperPass implements CompilerPassInterface
 {

src/Symfony/Bundle/FrameworkBundle/DependencyInjection/Compiler/TranslationExtractorPass.php

@@ -16,7 +16,7 @@ use Symfony\Component\DependencyInjection\ContainerBuilder;
 use Symfony\Component\DependencyInjection\Compiler\CompilerPassInterface;
 
 /**
- * Adds tagged translation.extractor services to translation extractor
+ * Adds tagged translation.extractor services to translation extractor.
  */
 class TranslationExtractorPass implements CompilerPassInterface
 {

src/Symfony/Bundle/FrameworkBundle/Templating/Helper/RequestHelper.php

@@ -59,7 +59,7 @@ class RequestHelper extends Helper
     }
 
     /**
-     * Returns the locale
+     * Returns the locale.
      *
      * @return string
      */

src/Symfony/Bundle/FrameworkBundle/Templating/Helper/SessionHelper.php

@@ -44,7 +44,7 @@ class SessionHelper extends Helper
     }
 
     /**
-     * Returns an attribute
+     * Returns an attribute.
      *
      * @param string $name    The attribute name
      * @param mixed  $default The default value

src/Symfony/Bundle/FrameworkBundle/Templating/TemplateReference.php

@@ -34,7 +34,7 @@ class TemplateReference extends BaseTemplateReference
     /**
      * Returns the path to the template
      *  - as a path when the template is not part of a bundle
-     *  - as a resource when the template is part of a bundle
+     *  - as a resource when the template is part of a bundle.
      *
      * @return string A path to the template or a resource
      */

src/Symfony/Bundle/FrameworkBundle/Tests/DependencyInjection/Compiler/ProfilerPassTest.php

@@ -26,8 +26,9 @@ class ProfilerPassTest extends \PHPUnit_Framework_TestCase
 
     /**
      * Tests that collectors that specify a template but no "id" will throw
-     * an exception (both are needed if the template is specified). Thus,
+     * an exception (both are needed if the template is specified).
-     * a fully-valid tag looks something like this:
+     *
+     * Thus, a fully-valid tag looks something like this:
      *
      *     <tag name="data_collector" template="YourBundle:Collector:templatename" id="your_collector_name" />
      */

src/Symfony/Bundle/FrameworkBundle/Tests/DependencyInjection/Compiler/SerializerPassTest.php

@@ -16,7 +16,7 @@ use Symfony\Component\DependencyInjection\Reference;
 use Symfony\Bundle\FrameworkBundle\DependencyInjection\Compiler\SerializerPass;
 
 /**
- * Tests for the SerializerPass class
+ * Tests for the SerializerPass class.
  *
  * @author Javier Lopez <f12loalf@gmail.com>
  */

src/Symfony/Bundle/FrameworkBundle/Tests/Templating/Helper/FormHelperDivLayoutTest.php

@@ -18,8 +18,6 @@ use Symfony\Bundle\FrameworkBundle\Tests\Templating\Helper\Fixtures\StubTemplate
 use Symfony\Bundle\FrameworkBundle\Tests\Templating\Helper\Fixtures\StubTranslator;
 use Symfony\Component\Templating\PhpEngine;
 use Symfony\Component\Templating\Loader\FilesystemLoader;
-
-// should probably be moved to the Translation component
 use Symfony\Bundle\FrameworkBundle\Templating\Helper\TranslatorHelper;
 
 class FormHelperDivLayoutTest extends AbstractDivLayoutTest

src/Symfony/Bundle/FrameworkBundle/Tests/Templating/Helper/FormHelperTableLayoutTest.php

@@ -18,8 +18,6 @@ use Symfony\Bundle\FrameworkBundle\Tests\Templating\Helper\Fixtures\StubTemplate
 use Symfony\Bundle\FrameworkBundle\Tests\Templating\Helper\Fixtures\StubTranslator;
 use Symfony\Component\Templating\PhpEngine;
 use Symfony\Component\Templating\Loader\FilesystemLoader;
-
-// should probably be moved to the Translation component
 use Symfony\Bundle\FrameworkBundle\Templating\Helper\TranslatorHelper;
 
 class FormHelperTableLayoutTest extends AbstractTableLayoutTest

src/Symfony/Bundle/FrameworkBundle/Translation/PhpExtractor.php

@@ -103,7 +103,7 @@ class PhpExtractor implements ExtractorInterface
 
     /**
      * Extracts the message from the iterator while the tokens
-     * match allowed message tokens
+     * match allowed message tokens.
      */
     private function getMessage(\Iterator $tokenIterator)
     {

src/Symfony/Bundle/FrameworkBundle/Translation/PhpStringTokenParser.php

@@ -77,7 +77,7 @@ class PhpStringTokenParser
         if ('\'' === $str[$bLength]) {
             return str_replace(
                 array('\\\\', '\\\''),
-                array(  '\\',   '\''),
+                array(  '\\', '\''),
                 substr($str, $bLength + 1, -1)
             );
         } else {

src/Symfony/Bundle/FrameworkBundle/Translation/TranslationLoader.php

@@ -31,6 +31,7 @@ class TranslationLoader
 
     /**
      * Adds a loader to the translation extractor.
+     *
      * @param string          $format The format of the loader
      * @param LoaderInterface $loader
      */
@@ -47,6 +48,10 @@ class TranslationLoader
      */
     public function loadMessages($directory, MessageCatalogue $catalogue)
     {
+        if (!is_dir($directory)) {
+            return;
+        }
+
         foreach ($this->loaders as $format => $loader) {
             // load any existing translation files
             $finder = new Finder();

src/Symfony/Bundle/SecurityBundle/Command/InitAclCommand.php

@@ -17,7 +17,7 @@ use Symfony\Component\Console\Output\OutputInterface;
 use Doctrine\DBAL\Schema\SchemaException;
 
 /**
- * Installs the tables required by the ACL system
+ * Installs the tables required by the ACL system.
  *
  * @author Johannes M. Schmitt <schmittjoh@gmail.com>
  */

src/Symfony/Bundle/SecurityBundle/DataCollector/SecurityDataCollector.php

@@ -82,7 +82,7 @@ class SecurityDataCollector extends DataCollector
                 'token_class' => get_class($token),
                 'user' => $token->getUsername(),
                 'roles' => array_map(function (RoleInterface $role) { return $role->getRole();}, $assignedRoles),
-                'inherited_roles' => array_map(function (RoleInterface $role) { return $role->getRole();}, $inheritedRoles),
+                'inherited_roles' => array_map(function (RoleInterface $role) { return $role->getRole(); }, $inheritedRoles),
                 'supports_role_hierarchy' => null !== $this->roleHierarchy,
             );
         }

src/Symfony/Bundle/SecurityBundle/DependencyInjection/Compiler/AddSecurityVotersPass.php

@@ -16,7 +16,7 @@ use Symfony\Component\DependencyInjection\ContainerBuilder;
 use Symfony\Component\DependencyInjection\Compiler\CompilerPassInterface;
 
 /**
- * Adds all configured security voters to the access decision manager
+ * Adds all configured security voters to the access decision manager.
  *
  * @author Johannes M. Schmitt <schmittjoh@gmail.com>
  */

src/Symfony/Bundle/SecurityBundle/DependencyInjection/MainConfiguration.php

@@ -18,7 +18,9 @@ use Symfony\Component\Config\Definition\Builder\ArrayNodeDefinition;
 use Symfony\Component\Config\Definition\ConfigurationInterface;
 
 /**
- * This class contains the configuration information for the following tags:
+ * This class contains the configuration information.
+ *
+ * This information is for the following tags:
  *
  *   * security.config
  *   * security.acl

src/Symfony/Bundle/SecurityBundle/DependencyInjection/Security/Factory/AbstractFactory.php

@@ -18,7 +18,7 @@ use Symfony\Component\DependencyInjection\Reference;
 
 /**
  * AbstractFactory is the base class for all classes inheriting from
- * AbstractAuthenticationListener
+ * AbstractAuthenticationListener.
  *
  * @author Fabien Potencier <fabien@symfony.com>
  * @author Lukas Kahwe Smith <smith@pooteeweet.org>

src/Symfony/Bundle/SecurityBundle/Security/FirewallMap.php

@@ -16,7 +16,7 @@ use Symfony\Component\HttpFoundation\Request;
 use Symfony\Component\DependencyInjection\ContainerInterface;
 
 /**
- * This is a lazy-loading firewall map implementation
+ * This is a lazy-loading firewall map implementation.
  *
  * Listeners will only be initialized if we really need them.
  *

src/Symfony/Bundle/SecurityBundle/Tests/DataCollector/SecurityDataCollectorTest.php

@@ -108,6 +108,7 @@ class SecurityDataCollectorTest extends \PHPUnit_Framework_TestCase
             ->disableOriginalConstructor()
             ->getMock();
     }
+
     private function getResponse()
     {
         return $this

src/Symfony/Bundle/SecurityBundle/Tests/DependencyInjection/Fixtures/UserProvider/DummyProvider.php

@@ -10,7 +10,6 @@ class DummyProvider implements UserProviderFactoryInterface
 {
     public function create(ContainerBuilder $container, $id, $config)
     {
-        
     }
 
     public function getKey()
@@ -20,6 +19,5 @@ class DummyProvider implements UserProviderFactoryInterface
 
     public function addConfiguration(NodeDefinition $node)
     {
-        
     }
 }

src/Symfony/Bundle/TwigBundle/DependencyInjection/Compiler/TwigEnvironmentPass.php

@@ -16,7 +16,7 @@ use Symfony\Component\DependencyInjection\ContainerBuilder;
 use Symfony\Component\DependencyInjection\Compiler\CompilerPassInterface;
 
 /**
- * Adds tagged twig.extension services to twig service
+ * Adds tagged twig.extension services to twig service.
  *
  * @author Fabien Potencier <fabien@symfony.com>
  */

src/Symfony/Bundle/TwigBundle/DependencyInjection/Compiler/TwigLoaderPass.php

@@ -17,7 +17,7 @@ use Symfony\Component\DependencyInjection\Compiler\CompilerPassInterface;
 use Symfony\Component\DependencyInjection\Exception\LogicException;
 
 /**
- * Adds services tagged twig.loader as Twig loaders
+ * Adds services tagged twig.loader as Twig loaders.
  *
  * @author Daniel Leech <daniel@dantleech.com>
  */

src/Symfony/Bundle/TwigBundle/Extension/ActionsExtension.php

@@ -15,7 +15,7 @@ use Symfony\Bundle\TwigBundle\TokenParser\RenderTokenParser;
 use Symfony\Component\DependencyInjection\ContainerInterface;
 
 /**
- * Twig extension for Symfony actions helper
+ * Twig extension for Symfony actions helper.
  *
  * @author Fabien Potencier <fabien@symfony.com>
  *

src/Symfony/Bundle/TwigBundle/Extension/AssetsExtension.php

@@ -15,7 +15,7 @@ use Symfony\Component\DependencyInjection\ContainerInterface;
 use Symfony\Component\Routing\RequestContext;
 
 /**
- * Twig extension for Symfony assets helper
+ * Twig extension for Symfony assets helper.
  *
  * @author Fabien Potencier <fabien@symfony.com>
  */

src/Symfony/Bundle/WebProfilerBundle/DependencyInjection/Configuration.php

@@ -15,7 +15,7 @@ use Symfony\Component\Config\Definition\Builder\TreeBuilder;
 use Symfony\Component\Config\Definition\ConfigurationInterface;
 
 /**
- * This class contains the configuration information for the bundle
+ * This class contains the configuration information for the bundle.
  *
  * This information is solely responsible for how the different configuration
  * sections are normalized, and merged.

src/Symfony/Bundle/WebProfilerBundle/Profiler/TemplateManager.php

@@ -16,7 +16,7 @@ use Symfony\Component\HttpKernel\Profiler\Profiler;
 use Symfony\Component\HttpKernel\Profiler\Profile;
 
 /**
- * Profiler Templates Manager
+ * Profiler Templates Manager.
  *
  * @author Fabien Potencier <fabien@symfony.com>
  * @author Artur Wielogórski <wodor@wodor.net>

src/Symfony/Bundle/WebProfilerBundle/Tests/DependencyInjection/WebProfilerExtensionTest.php

@@ -24,7 +24,7 @@ class WebProfilerExtensionTest extends TestCase
 {
     private $kernel;
     /**
-     * @var Symfony\Component\DependencyInjection\Container $container
+     * @var Symfony\Component\DependencyInjection\Container
      */
     private $container;
 

src/Symfony/Bundle/WebProfilerBundle/Tests/Profiler/TemplateManagerTest.php

@@ -66,7 +66,7 @@ class TemplateManagerTest extends TestCase
     }
 
     /**
-     * if template exists in both profile and profiler then its name should be returned
+     * if template exists in both profile and profiler then its name should be returned.
      */
     public function testGetNameValidTemplate()
     {
@@ -85,7 +85,7 @@ class TemplateManagerTest extends TestCase
 
     /**
      * template should be loaded for 'foo' because other collectors are
-     * missing in profile or in profiler
+     * missing in profile or in profiler.
      */
     public function testGetTemplates()
     {

src/Symfony/Component/ClassLoader/ApcClassLoader.php

@@ -44,10 +44,9 @@ class ApcClassLoader
     private $prefix;
 
     /**
-     * The class loader object being decorated.
+     * A class loader object that implements the findFile() method.
      *
      * @var object
-     *   A class loader object that implements the findFile() method.
      */
     protected $decorated;
 

src/Symfony/Component/ClassLoader/ClassLoader.php

@@ -12,7 +12,7 @@
 namespace Symfony\Component\ClassLoader;
 
 /**
- * ClassLoader implements an PSR-0 class loader
+ * ClassLoader implements an PSR-0 class loader.
  *
  * See https://github.com/php-fig/fig-standards/blob/master/accepted/PSR-0.md
  *
@@ -76,7 +76,7 @@ class ClassLoader
     }
 
     /**
-     * Registers a set of classes
+     * Registers a set of classes.
      *
      * @param string       $prefix The classes prefix
      * @param array|string $paths  The location(s) of the classes

src/Symfony/Component/ClassLoader/ClassMapGenerator.php

@@ -20,14 +20,14 @@ if (!defined('SYMFONY_TRAIT')) {
 }
 
 /**
- * ClassMapGenerator
+ * ClassMapGenerator.
  *
  * @author Gyula Sallai <salla016@gmail.com>
  */
 class ClassMapGenerator
 {
     /**
-     * Generate a class map file
+     * Generate a class map file.
      *
      * @param array|string $dirs Directories or a single path to search in
      * @param string       $file The name of the class map file
@@ -45,7 +45,7 @@ class ClassMapGenerator
     }
 
     /**
-     * Iterate over all files in the given directory searching for classes
+     * Iterate over all files in the given directory searching for classes.
      *
      * @param \Iterator|string $dir The directory to search in or an iterator
      *
@@ -81,7 +81,7 @@ class ClassMapGenerator
     }
 
     /**
-     * Extract the classes in the given file
+     * Extract the classes in the given file.
      *
      * @param string $path The file to check
      *

src/Symfony/Component/ClassLoader/DebugClassLoader.php

@@ -82,7 +82,7 @@ class DebugClassLoader
     }
 
     /**
-     * Finds a file by class name
+     * Finds a file by class name.
      *
      * @param string $class A class name to resolve to file
      *

src/Symfony/Component/ClassLoader/Tests/ClassMapGeneratorTest.php

@@ -16,7 +16,7 @@ use Symfony\Component\ClassLoader\ClassMapGenerator;
 class ClassMapGeneratorTest extends \PHPUnit_Framework_TestCase
 {
     /**
-     * @var string $workspace
+     * @var string|null
      */
     private $workspace = null;
 

src/Symfony/Component/ClassLoader/UniversalClassLoader.php

@@ -72,7 +72,7 @@ class UniversalClassLoader
 
     /**
      * Turns on searching the include for class files. Allows easy loading
-     * of installed PEAR packages
+     * of installed PEAR packages.
      *
      * @param bool $useIncludePath
      */
@@ -177,7 +177,7 @@ class UniversalClassLoader
     }
 
     /**
-     * Registers an array of namespaces
+     * Registers an array of namespaces.
      *
      * @param array $namespaces An array of namespaces (namespaces as keys and locations as values)
      *

src/Symfony/Component/ClassLoader/WinCacheClassLoader.php

@@ -43,10 +43,9 @@ class WinCacheClassLoader
     private $prefix;
 
     /**
-     * The class loader object being decorated.
+     * A class loader object that implements the findFile() method.
      *
-     * @var \Symfony\Component\ClassLoader\ClassLoader
+     * @var object
-     *   A class loader object that implements the findFile() method.
      */
     protected $decorated;
 

src/Symfony/Component/ClassLoader/XcacheClassLoader.php

@@ -45,7 +45,9 @@ class XcacheClassLoader
     private $prefix;
 
     /**
-     * @var object A class loader object that implements the findFile() method
+     * A class loader object that implements the findFile() method.
+     *
+     * @var object
      */
     private $decorated;
 

src/Symfony/Component/Config/Definition/BaseNode.php

@@ -17,7 +17,7 @@ use Symfony\Component\Config\Definition\Exception\InvalidConfigurationException;
 use Symfony\Component\Config\Definition\Exception\InvalidTypeException;
 
 /**
- * The base node class
+ * The base node class.
  *
  * @author Johannes M. Schmitt <schmittjoh@gmail.com>
  */
@@ -182,7 +182,7 @@ abstract class BaseNode implements NodeInterface
     }
 
     /**
-     * Returns the name of this node
+     * Returns the name of this node.
      *
      * @return string The Node's name.
      */

src/Symfony/Component/Config/Definition/Builder/ArrayNodeDefinition.php

@@ -329,7 +329,7 @@ class ArrayNodeDefinition extends NodeDefinition implements ParentNodeDefinition
     }
 
     /**
-     * Returns a node builder to be used to add children and prototype
+     * Returns a node builder to be used to add children and prototype.
      *
      * @return NodeBuilder The node builder
      */

src/Symfony/Component/Config/Definition/Builder/BooleanNodeDefinition.php

@@ -31,7 +31,7 @@ class BooleanNodeDefinition extends ScalarNodeDefinition
     }
 
     /**
-     * Instantiate a Node
+     * Instantiate a Node.
      *
      * @return BooleanNode The node
      */

src/Symfony/Component/Config/Definition/Builder/EnumNodeDefinition.php

@@ -36,7 +36,7 @@ class EnumNodeDefinition extends ScalarNodeDefinition
     }
 
     /**
-     * Instantiate a Node
+     * Instantiate a Node.
      *
      * @return EnumNode The node
      *

src/Symfony/Component/Config/Definition/Builder/ExprBuilder.php

@@ -26,7 +26,7 @@ class ExprBuilder
     public $thenPart;
 
     /**
-     * Constructor
+     * Constructor.
      *
      * @param NodeDefinition $node The related node
      */
@@ -196,7 +196,7 @@ class ExprBuilder
     }
 
     /**
-     * Returns the related node
+     * Returns the related node.
      *
      * @return NodeDefinition
      *

src/Symfony/Component/Config/Definition/Builder/MergeBuilder.php

@@ -23,7 +23,7 @@ class MergeBuilder
     public $allowOverwrite = true;
 
     /**
-     * Constructor
+     * Constructor.
      *
      * @param NodeDefinition $node The related node
      */

src/Symfony/Component/Config/Definition/Builder/NodeBuilder.php

@@ -22,8 +22,7 @@ class NodeBuilder implements NodeParentInterface
     protected $nodeMapping;
 
     /**
-     * Constructor
+     * Constructor.
-     *
      */
     public function __construct()
     {

src/Symfony/Component/Config/Definition/Builder/NodeDefinition.php

@@ -40,7 +40,7 @@ abstract class NodeDefinition implements NodeParentInterface
     protected $attributes = array();
 
     /**
-     * Constructor
+     * Constructor.
      *
      * @param string                   $name   The name of the node
      * @param NodeParentInterface|null $parent The parent
@@ -333,7 +333,7 @@ abstract class NodeDefinition implements NodeParentInterface
     }
 
     /**
-     * Instantiate and configure the node according to this definition
+     * Instantiate and configure the node according to this definition.
      *
      * @return NodeInterface $node The node instance
      *

src/Symfony/Component/Config/Definition/Builder/NodeParentInterface.php

@@ -12,7 +12,7 @@
 namespace Symfony\Component\Config\Definition\Builder;
 
 /**
- * An interface that must be implemented by all node parents
+ * An interface that must be implemented by all node parents.
  *
  * @author Victor Berchet <victor@suumit.com>
  */

src/Symfony/Component/Config/Definition/Builder/NormalizationBuilder.php

@@ -23,7 +23,7 @@ class NormalizationBuilder
     public $remappings = array();
 
     /**
-     * Constructor
+     * Constructor.
      *
      * @param NodeDefinition $node The related node
      */

src/Symfony/Component/Config/Definition/Builder/ParentNodeDefinitionInterface.php

@@ -12,7 +12,7 @@
 namespace Symfony\Component\Config\Definition\Builder;
 
 /**
- * An interface that must be implemented by nodes which can have children
+ * An interface that must be implemented by nodes which can have children.
  *
  * @author Victor Berchet <victor@suumit.com>
  */

src/Symfony/Component/Config/Definition/Builder/ScalarNodeDefinition.php

@@ -21,7 +21,7 @@ use Symfony\Component\Config\Definition\ScalarNode;
 class ScalarNodeDefinition extends VariableNodeDefinition
 {
     /**
-     * Instantiate a Node
+     * Instantiate a Node.
      *
      * @return ScalarNode The node
      */

src/Symfony/Component/Config/Definition/Builder/ValidationBuilder.php

@@ -22,7 +22,7 @@ class ValidationBuilder
     public $rules = array();
 
     /**
-     * Constructor
+     * Constructor.
      *
      * @param NodeDefinition $node The related node
      */

src/Symfony/Component/Config/Definition/Builder/VariableNodeDefinition.php

@@ -21,7 +21,7 @@ use Symfony\Component\Config\Definition\VariableNode;
 class VariableNodeDefinition extends NodeDefinition
 {
     /**
-     * Instantiate a Node
+     * Instantiate a Node.
      *
      * @return VariableNode The node
      */

src/Symfony/Component/Config/Definition/ConfigurationInterface.php

@@ -12,7 +12,7 @@
 namespace Symfony\Component\Config\Definition;
 
 /**
- * Configuration interface
+ * Configuration interface.
  *
  * @author Victor Berchet <victor@suumit.com>
  */

src/Symfony/Component/Config/Definition/Exception/Exception.php

@@ -12,7 +12,7 @@
 namespace Symfony\Component\Config\Definition\Exception;
 
 /**
- * Base exception for all configuration exceptions
+ * Base exception for all configuration exceptions.
  *
  * @author Johannes M. Schmitt <schmittjoh@gmail.com>
  */

src/Symfony/Component/Config/Definition/NodeInterface.php

@@ -52,7 +52,8 @@ interface NodeInterface
     /**
      * Returns the default value of the node.
      *
-     * @return mixed             The default value
+     * @return mixed The default value
+     *
      * @throws \RuntimeException if the node has no default value
      */
     public function getDefaultValue();

src/Symfony/Component/Config/Definition/NumericNode.php

@@ -14,7 +14,7 @@ namespace Symfony\Component\Config\Definition;
 use Symfony\Component\Config\Definition\Exception\InvalidConfigurationException;
 
 /**
- * This node represents a numeric value in the config tree
+ * This node represents a numeric value in the config tree.
  *
  * @author David Jeanmonod <david.jeanmonod@gmail.com>
  */

src/Symfony/Component/Config/Definition/PrototypedArrayNode.php

@@ -155,7 +155,7 @@ class PrototypedArrayNode extends ArrayNode
     }
 
     /**
-     * Retrieves the prototype
+     * Retrieves the prototype.
      *
      * @return PrototypeNodeInterface The prototype
      */

src/Symfony/Component/Config/Resource/DirectoryResource.php

@@ -50,7 +50,7 @@ class DirectoryResource implements ResourceInterface, \Serializable
     }
 
     /**
-     * Returns the pattern to restrict monitored files
+     * Returns the pattern to restrict monitored files.
      *
      * @return string|null
      */

src/Symfony/Component/Config/Tests/Definition/Builder/ExprBuilderTest.php

@@ -202,12 +202,11 @@ class ExprBuilderTest extends \PHPUnit_Framework_TestCase
     }
 
     /**
-     * Assert that the given test builder, will return the given value
+     * Assert that the given test builder, will return the given value.
      *
      * @param mixed       $value       The value to test
      * @param TreeBuilder $treeBuilder The tree builder to finalize
-     *
+     * @param mixed       $config      The config values that new to be finalized
-     * @param mixed $config The config values that new to be finalized
      */
     protected function assertFinalizedValueIs($value, $treeBuilder, $config = null)
     {

src/Symfony/Component/Config/Tests/Definition/PrototypedArrayNodeTest.php

@@ -54,7 +54,7 @@ class PrototypedArrayNodeTest extends \PHPUnit_Framework_TestCase
     }
 
     /**
-     * Tests that when a key attribute is mapped, that key is removed from the array:
+     * Tests that when a key attribute is mapped, that key is removed from the array.
      *
      *     <things>
      *         <option id="option1" value="foo">

src/Symfony/Component/Config/Util/XmlUtils.php

@@ -22,7 +22,7 @@ namespace Symfony\Component\Config\Util;
 class XmlUtils
 {
     /**
-     * This class should not be instantiated
+     * This class should not be instantiated.
      */
     private function __construct()
     {

src/Symfony/Component/Console/Application.php

@@ -223,7 +223,7 @@ class Application
     }
 
     /**
-     * Set an input definition set to be used with this application
+     * Set an input definition set to be used with this application.
      *
      * @param InputDefinition $definition The input definition
      *
@@ -736,7 +736,7 @@ class Application
     }
 
     /**
-     * Tries to figure out the terminal width in which this application runs
+     * Tries to figure out the terminal width in which this application runs.
      *
      * @return int|null
      */
@@ -748,7 +748,7 @@ class Application
     }
 
     /**
-     * Tries to figure out the terminal height in which this application runs
+     * Tries to figure out the terminal height in which this application runs.
      *
      * @return int|null
      */
@@ -760,7 +760,7 @@ class Application
     }
 
     /**
-     * Tries to figure out the terminal dimensions based on the current environment
+     * Tries to figure out the terminal dimensions based on the current environment.
      *
      * @return array Array containing width and height
      */
@@ -960,7 +960,7 @@ class Application
     }
 
     /**
-     * Runs and parses stty -a if it's available, suppressing any error output
+     * Runs and parses stty -a if it's available, suppressing any error output.
      *
      * @return string
      */
@@ -983,7 +983,7 @@ class Application
     }
 
     /**
-     * Runs and parses mode CON if it's available, suppressing any error output
+     * Runs and parses mode CON if it's available, suppressing any error output.
      *
      * @return string <width>x<height> or null if it could not be parsed
      */
@@ -1039,7 +1039,7 @@ class Application
 
     /**
      * Finds alternative of $name among $collection,
-     * if nothing is found in $collection, try in $abbrevs
+     * if nothing is found in $collection, try in $abbrevs.
      *
      * @param string             $name       The string
      * @param array|\Traversable $collection The collection

src/Symfony/Component/Console/Command/Command.php

@@ -129,7 +129,7 @@ class Command
     }
 
     /**
-     * Checks whether the command is enabled or not in the current environment
+     * Checks whether the command is enabled or not in the current environment.
      *
      * Override this to check for x or y and return false if the command can not
      * run properly under the current conditions.
@@ -162,6 +162,7 @@ class Command
      * @return null|int null or 0 if everything went fine, or an error code
      *
      * @throws \LogicException When this abstract method is not implemented
+     *
      * @see    setCode()
      */
     protected function execute(InputInterface $input, OutputInterface $output)

src/Symfony/Component/Console/Command/HelpCommand.php

@@ -58,7 +58,7 @@ EOF
     }
 
     /**
-     * Sets the command
+     * Sets the command.
      *
      * @param Command $command The command to set
      */

src/Symfony/Component/Console/Formatter/OutputFormatterStyle.php

@@ -160,7 +160,6 @@ class OutputFormatterStyle implements OutputFormatterStyleInterface
      * @param string $option The option name
      *
      * @throws \InvalidArgumentException When the option name isn't defined
-     *
      */
     public function unsetOption($option)
     {

src/Symfony/Component/Console/Helper/DescriptorHelper.php

@@ -36,10 +36,10 @@ class DescriptorHelper extends Helper
     public function __construct()
     {
         $this
-            ->register('txt',  new TextDescriptor())
+            ->register('txt', new TextDescriptor())
-            ->register('xml',  new XmlDescriptor())
+            ->register('xml', new XmlDescriptor())
             ->register('json', new JsonDescriptor())
-            ->register('md',   new MarkdownDescriptor())
+            ->register('md', new MarkdownDescriptor())
         ;
     }
 

src/Symfony/Component/Console/Helper/DialogHelper.php

@@ -257,7 +257,7 @@ class DialogHelper extends InputAwareHelper
     }
 
     /**
-     * Asks a question to the user, the response is hidden
+     * Asks a question to the user, the response is hidden.
      *
      * @param OutputInterface $output   An Output instance
      * @param string|array    $question The question
@@ -372,7 +372,6 @@ class DialogHelper extends InputAwareHelper
      *
      * @throws \Exception        When any of the validators return an error
      * @throws \RuntimeException In case the fallback is deactivated and the response can not be hidden
-     *
      */
     public function askHiddenResponseAndValidate(OutputInterface $output, $question, $validator, $attempts = false, $fallback = true)
     {
@@ -398,7 +397,7 @@ class DialogHelper extends InputAwareHelper
     }
 
     /**
-     * Returns the helper's input stream
+     * Returns the helper's input stream.
      *
      * @return string
      */
@@ -416,7 +415,7 @@ class DialogHelper extends InputAwareHelper
     }
 
     /**
-     * Return a valid Unix shell
+     * Return a valid Unix shell.
      *
      * @return string|bool The valid shell name, false in case no valid shell is found
      */
@@ -454,7 +453,7 @@ class DialogHelper extends InputAwareHelper
     }
 
     /**
-     * Validate an attempt
+     * Validate an attempt.
      *
      * @param callable        $interviewer A callable that will ask for a question and return the result
      * @param OutputInterface $output      An Output instance