Fixed typos

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

@@ -44,7 +44,7 @@ class RegisterKernelListenersPass implements CompilerPassInterface
         }
 
         foreach ($container->findTaggedServiceIds('kernel.event_subscriber') as $id => $attributes) {
-            // We must assume that the class value has been correcly filled, even if the service is created by a factory
+            // We must assume that the class value has been correctly filled, even if the service is created by a factory
             $class = $container->getDefinition($id)->getClass();
 
             $refClass = new \ReflectionClass($class);

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

@@ -69,7 +69,7 @@ class ArrayNodeDefinitionTest extends \PHPUnit_Framework_TestCase
     /**
      * @expectedException Symfony\Component\Config\Definition\Exception\InvalidDefinitionException
      */
-    public function testPrototypeNodesCantHaveADefaultValueWhenUsingDefaulChildren()
+    public function testPrototypeNodesCantHaveADefaultValueWhenUsingDefaultChildren()
     {
         $node = new ArrayNodeDefinition('root');
         $node

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

@@ -90,7 +90,7 @@ class TreeBuilderTest extends \PHPUnit_Framework_TestCase
         ->end();
     }
 
-    public function testDefinitionInfoGetsTransferedToNode()
+    public function testDefinitionInfoGetsTransferredToNode()
     {
         $builder = new TreeBuilder();
 
@@ -107,7 +107,7 @@ class TreeBuilderTest extends \PHPUnit_Framework_TestCase
         $this->assertEquals('child info', $children['child']->getInfo());
     }
 
-    public function testDefinitionExampleGetsTransferedToNode()
+    public function testDefinitionExampleGetsTransferredToNode()
     {
         $builder = new TreeBuilder();
 

src/Symfony/Component/Console/Application.php

@@ -1041,7 +1041,7 @@ class Application
      * if nothing is found in $collection, try in $abbrevs
      *
      * @param string               $name       The string
-     * @param array|Traversable    $collection The collecion
+     * @param array|Traversable    $collection The collection
      * @param array                $abbrevs    The abbreviations
      * @param Closure|string|array $callback   The callable to transform collection item before comparison
      *

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

@@ -50,7 +50,7 @@ interface OutputFormatterStyleInterface
     /**
      * Unsets some specific style option.
      *
-     * @param string $option Theoption name
+     * @param string $option The option name
      */
     public function unsetOption($option);
 

src/Symfony/Component/Console/Tests/ApplicationTest.php

@@ -224,7 +224,7 @@ class ApplicationTest extends \PHPUnit_Framework_TestCase
         $application->add(new \Foo2Command());
 
         try {
-            $application->find($commandName = 'Unknow command');
+            $application->find($commandName = 'Unknown command');
             $this->fail('->find() throws an \InvalidArgumentException if command does not exist');
         } catch (\Exception $e) {
             $this->assertInstanceOf('\InvalidArgumentException', $e, '->find() throws an \InvalidArgumentException if command does not exist');
@@ -264,11 +264,11 @@ class ApplicationTest extends \PHPUnit_Framework_TestCase
         $application->add(new \foo3Command());
 
         try {
-            $application->find('Unknow-namespace:Unknow-command');
+            $application->find('Unknown-namespace:Unknown-command');
             $this->fail('->find() throws an \InvalidArgumentException if namespace does not exist');
         } catch (\Exception $e) {
             $this->assertInstanceOf('\InvalidArgumentException', $e, '->find() throws an \InvalidArgumentException if namespace does not exist');
-            $this->assertEquals('There are no commands defined in the "Unknow-namespace" namespace.', $e->getMessage(), '->find() throws an \InvalidArgumentException if namespace does not exist, without alternatives');
+            $this->assertEquals('There are no commands defined in the "Unknown-namespace" namespace.', $e->getMessage(), '->find() throws an \InvalidArgumentException if namespace does not exist, without alternatives');
         }
 
         try {

src/Symfony/Component/DependencyInjection/Compiler/InlineServiceDefinitionsPass.php

@@ -81,7 +81,7 @@ class InlineServiceDefinitionsPass implements RepeatablePassInterface
                     continue;
                 }
 
-                if ($this->isInlinableDefinition($container, $id, $definition = $container->getDefinition($id))) {
+                if ($this->isInlineableDefinition($container, $id, $definition = $container->getDefinition($id))) {
                     $this->compiler->addLogMessage($this->formatter->formatInlineService($this, $id, $this->currentId));
 
                     if (ContainerInterface::SCOPE_PROTOTYPE !== $definition->getScope()) {
@@ -109,7 +109,7 @@ class InlineServiceDefinitionsPass implements RepeatablePassInterface
      *
      * @return Boolean If the definition is inlineable
      */
-    private function isInlinableDefinition(ContainerBuilder $container, $id, Definition $definition)
+    private function isInlineableDefinition(ContainerBuilder $container, $id, Definition $definition)
     {
         if (ContainerInterface::SCOPE_PROTOTYPE === $definition->getScope()) {
             return true;

src/Symfony/Component/Finder/Tests/Iterator/FilecontentFilterIteratorTest.php

@@ -165,12 +165,10 @@ class MockSplFileInfo extends \SplFileInfo
         if (is_string($type)) {
             switch ($type) {
                 case 'directory':
-                    $this->type = self::TYPE_DIRECTORY;
                 case 'd':
                     $this->type = self::TYPE_DIRECTORY;
                     break;
                 case 'file':
-                    $this->type = self::TYPE_FILE;
                 case 'f':
                     $this->type = self::TYPE_FILE;
                     break;

src/Symfony/Component/Form/Tests/AbstractDivLayoutTest.php

@@ -465,7 +465,7 @@ abstract class AbstractDivLayoutTest extends AbstractLayoutTest
     public function testRepeatedWithCustomOptions()
     {
         $form = $this->factory->createNamed('name', 'repeated', null, array(
-            // the global required value cannot be overriden
+            // the global required value cannot be overridden
             'first_options'  => array('label' => 'Test', 'required' => false),
             'second_options' => array('label' => 'Test2')
         ));

src/Symfony/Component/Form/Tests/AbstractLayoutTest.php

@@ -66,7 +66,7 @@ abstract class AbstractLayoutTest extends FormIntegrationTestCase
             // the top level
             $dom->loadXml('<root>'.$html.'</root>');
         } catch (\Exception $e) {
-            return $this->fail(sprintf(
+            $this->fail(sprintf(
                 "Failed loading HTML:\n\n%s\n\nError: %s",
                 $html,
                 $e->getMessage()

src/Symfony/Component/Form/Tests/AbstractTableLayoutTest.php

@@ -116,7 +116,7 @@ abstract class AbstractTableLayoutTest extends AbstractLayoutTest
             ->getForm()
             ->createView();
 
-        // Render field2 row -> does not implicitely call renderWidget because
+        // Render field2 row -> does not implicitly call renderWidget because
         // it is a repeated field!
         $this->renderRow($view['field2']);
 

src/Symfony/Component/Form/Tests/Extension/Core/Type/FormTypeTest.php

@@ -206,7 +206,7 @@ class FormTypeTest extends TypeTestCase
         $this->assertEquals('test', $view['child']->vars['translation_domain']);
     }
 
-    public function testNonTranlsationDomainFormWithNonTranslationDomainParentBeingTranslationDomainDefault()
+    public function testNonTranslationDomainFormWithNonTranslationDomainParentBeingTranslationDomainDefault()
     {
         $parent = $this->factory->createNamed('parent', 'form');
         $child  = $this->factory->createNamed('child', 'form');

src/Symfony/Component/Form/Tests/Extension/Core/Type/RepeatedTypeTest.php

@@ -49,7 +49,7 @@ class RepeatedTypeTest extends TypeTestCase
     public function testSetOptionsPerChild()
     {
         $form = $this->factory->create('repeated', null, array(
-            // the global required value cannot be overriden
+            // the global required value cannot be overridden
             'type'           => 'text',
             'first_options'  => array('label' => 'Test', 'required' => false),
             'second_options' => array('label' => 'Test2')

src/Symfony/Component/Form/Tests/FormFactoryTest.php

@@ -141,7 +141,7 @@ class FormFactoryTest extends \PHPUnit_Framework_TestCase
             ->with($type)
             ->will($this->returnValue($resolvedType));
 
-        // The type is also implicitely added to the registry
+        // The type is also implicitly added to the registry
         $this->registry->expects($this->once())
             ->method('addType')
             ->with($resolvedType);
@@ -159,7 +159,7 @@ class FormFactoryTest extends \PHPUnit_Framework_TestCase
         $options = array('a' => '1', 'b' => '2');
         $resolvedType = $this->getMockResolvedType();
 
-        // The type is also implicitely added to the registry
+        // The type is also implicitly added to the registry
         $this->registry->expects($this->once())
             ->method('addType')
             ->with($resolvedType);

src/Symfony/Component/HttpFoundation/Request.php

@@ -515,7 +515,7 @@ class Request
      *  * slow
      *  * prefer to get from a "named" source
      *
-     * It is better to explicity get request parameters from the appropriate
+     * It is better to explicitly get request parameters from the appropriate
      * public property instead (query, request, attributes, ...).
      *
      * @param string  $key     the key

src/Symfony/Component/HttpFoundation/Session/Storage/NativeSessionStorage.php

@@ -173,7 +173,7 @@ class NativeSessionStorage implements SessionStorageInterface
      */
     public function setId($id)
     {
-        return $this->saveHandler->setId($id);
+        $this->saveHandler->setId($id);
     }
 
     /**

src/Symfony/Component/HttpKernel/Event/PostResponseEvent.php

@@ -61,7 +61,7 @@ class PostResponseEvent extends Event
     }
 
     /**
-     * Returns the reponse for which this event was thrown.
+     * Returns the response for which this event was thrown.
      *
      * @return Response
      */

src/Symfony/Component/HttpKernel/KernelEvents.php

@@ -93,7 +93,7 @@ final class KernelEvents
     const RESPONSE = 'kernel.response';
 
     /**
-     * The TERMINATE event occurs once a reponse was sent
+     * The TERMINATE event occurs once a response was sent
      *
      * This event allows you to run expensive post-response jobs.
      * The event listener method receives a

src/Symfony/Component/HttpKernel/Profiler/FileProfilerStorage.php

@@ -205,7 +205,7 @@ class FileProfilerStorage implements ProfilerStorageInterface
      *
      * @param resource $file The file resource, with the pointer placed at the end of the line to read
      *
-     * @return mixed A string representating the line or FALSE if beginning of file is reached
+     * @return mixed A string representing the line or FALSE if beginning of file is reached
      */
     protected function readLineFromFile($file)
     {

src/Symfony/Component/OptionsResolver/Tests/OptionsTest.php

@@ -430,7 +430,7 @@ class OptionsTest extends \PHPUnit_Framework_TestCase
         $this->options->clear();
     }
 
-    public function testOverloadCannotBeEvaluatedLazilyWithoutExpectedClousureParams()
+    public function testOverloadCannotBeEvaluatedLazilyWithoutExpectedClosureParams()
     {
         $this->options->set('foo', 'bar');
 

src/Symfony/Component/Process/Process.php

@@ -180,7 +180,7 @@ class Process
      * the output in real-time while writing the standard input to the process.
      * It allows to have feedback from the independent process during execution.
      * If there is no callback passed, the wait() method can be called
-     * with true as a second parameter then the callback will get all data occured
+     * with true as a second parameter then the callback will get all data occurred
      * in (and since) the start call.
      *
      * @param Closure|string|array $callback A PHP callback to run whenever there is some
@@ -662,8 +662,8 @@ class Process
     /**
      * Builds up the callback used by wait().
      *
-     * The callbacks adds all occured output to the specific buffer and calls
-     * the usercallback (if present) with the received output.
+     * The callbacks adds all occurred output to the specific buffer and calls
+     * the user callback (if present) with the received output.
      *
      * @param mixed $callback The user defined PHP callback
      *

src/Symfony/Component/Process/Tests/ProcessTest.php

@@ -146,7 +146,7 @@ class ProcessTest extends \PHPUnit_Framework_TestCase
     {
         $this->markTestSkipped('Can course php to hang');
 
-        // Sleep dont work as it will allow the process to handle signals and close
+        // Sleep doesn't work as it will allow the process to handle signals and close
         // file handles from the other end.
         $process = new Process('php -r "while (true) {}"');
         $process->start();

src/Symfony/Component/Routing/RouteCollection.php

@@ -308,7 +308,7 @@ class RouteCollection implements \IteratorAggregate, \Countable
     {
         // It is ensured by the adders (->add and ->addCollection) that there can
         // only be one route per name in all connected collections. So we can stop
-        // interating recursively on the first hit.
+        // iterating recursively on the first hit.
         if (isset($this->routes[$name])) {
             unset($this->routes[$name]);
 

src/Symfony/Component/Routing/Tests/Fixtures/dumper/url_matcher1.php

@@ -139,9 +139,9 @@ class ProjectUrlMatcher extends Symfony\Component\Routing\Matcher\UrlMatcher
 
             }
 
-            // overriden
+            // overridden
             if (preg_match('#^/a/(?<var>.*)$#s', $pathinfo, $matches)) {
-                $matches['_route'] = 'overriden';
+                $matches['_route'] = 'overridden';
 
                 return $matches;
             }
@@ -171,9 +171,9 @@ class ProjectUrlMatcher extends Symfony\Component\Routing\Matcher\UrlMatcher
                 return array_merge($this->mergeDefaults($matches, array (  'who' => 'World!',)), array('_route' => 'helloWorld'));
             }
 
-            // overriden2
+            // overridden2
             if ($pathinfo === '/multi/new') {
-                return array('_route' => 'overriden2');
+                return array('_route' => 'overridden2');
             }
 
             // hey

src/Symfony/Component/Routing/Tests/Fixtures/dumper/url_matcher2.php

@@ -147,9 +147,9 @@ class ProjectUrlMatcher extends Symfony\Component\Routing\Tests\Fixtures\Redirec
 
             }
 
-            // overriden
+            // overridden
             if (preg_match('#^/a/(?<var>.*)$#s', $pathinfo, $matches)) {
-                $matches['_route'] = 'overriden';
+                $matches['_route'] = 'overridden';
 
                 return $matches;
             }
@@ -179,9 +179,9 @@ class ProjectUrlMatcher extends Symfony\Component\Routing\Tests\Fixtures\Redirec
                 return array_merge($this->mergeDefaults($matches, array (  'who' => 'World!',)), array('_route' => 'helloWorld'));
             }
 
-            // overriden2
+            // overridden2
             if ($pathinfo === '/multi/new') {
-                return array('_route' => 'overriden2');
+                return array('_route' => 'overridden2');
             }
 
             // hey

src/Symfony/Component/Routing/Tests/Generator/UrlGeneratorTest.php

@@ -191,7 +191,7 @@ class UrlGeneratorTest extends \PHPUnit_Framework_TestCase
     /**
      * @expectedException Symfony\Component\Routing\Exception\InvalidParameterException
      */
-    public function testGenerateForRouteWithInvalidManditoryParameter()
+    public function testGenerateForRouteWithInvalidMandatoryParameter()
     {
         $routes = $this->getRoutes('test', new Route('/testing/{foo}', array(), array('foo' => 'd+')));
         $this->getGenerator($routes)->generate('test', array('foo' => 'bar'), true);

src/Symfony/Component/Routing/Tests/Matcher/Dumper/PhpMatcherDumperTest.php

@@ -50,7 +50,7 @@ class PhpMatcherDumperTest extends \PHPUnit_Framework_TestCase
 
         $collection = new RouteCollection();
 
-        $collection->add('overriden', new Route('/overriden'));
+        $collection->add('overridden', new Route('/overridden'));
 
         // defaults and requirements
         $collection->add('foo', new Route(
@@ -116,25 +116,25 @@ class PhpMatcherDumperTest extends \PHPUnit_Framework_TestCase
 
         // prefixes
         $collection1 = new RouteCollection();
-        $collection1->add('overriden', new Route('/overriden1'));
+        $collection1->add('overridden', new Route('/overridden1'));
         $collection1->add('foo1', new Route('/{foo}'));
         $collection1->add('bar1', new Route('/{bar}'));
         $collection2 = new RouteCollection();
         $collection2->addCollection($collection1, '/b\'b');
-        $collection2->add('overriden', new Route('/{var}', array(), array('var' => '.*')));
+        $collection2->add('overridden', new Route('/{var}', array(), array('var' => '.*')));
         $collection1 = new RouteCollection();
         $collection1->add('foo2', new Route('/{foo1}'));
         $collection1->add('bar2', new Route('/{bar1}'));
         $collection2->addCollection($collection1, '/b\'b');
         $collection->addCollection($collection2, '/a');
 
-        // overriden through addCollection() and multiple sub-collections with no own prefix
+        // overridden through addCollection() and multiple sub-collections with no own prefix
         $collection1 = new RouteCollection();
-        $collection1->add('overriden2', new Route('/old'));
+        $collection1->add('overridden2', new Route('/old'));
         $collection1->add('helloWorld', new Route('/hello/{who}', array('who' => 'World!')));
         $collection2 = new RouteCollection();
         $collection3 = new RouteCollection();
-        $collection3->add('overriden2', new Route('/new'));
+        $collection3->add('overridden2', new Route('/new'));
         $collection3->add('hey', new Route('/hey/'));
         $collection1->addCollection($collection2);
         $collection2->addCollection($collection3);

src/Symfony/Component/Routing/Tests/Matcher/UrlMatcherTest.php

@@ -178,7 +178,7 @@ class UrlMatcherTest extends \PHPUnit_Framework_TestCase
         $this->assertEquals(array('_route' => 'foo', 'foo' => "\n"), $matcher->match('/'.urlencode("\n").'/bar'), 'linefeed character is matched');
     }
 
-    public function testMatchOverridenRoute()
+    public function testMatchOverriddenRoute()
     {
         $collection = new RouteCollection();
         $collection->add('foo', new Route('/foo'));

src/Symfony/Component/Routing/Tests/RouteCollectionTest.php

@@ -37,7 +37,7 @@ class RouteCollectionTest extends \PHPUnit_Framework_TestCase
         $collection->add('f o o', $route);
     }
 
-    public function testOverridenRoute()
+    public function testOverriddenRoute()
     {
         $collection = new RouteCollection();
         $collection->add('foo', new Route('/foo'));
@@ -46,7 +46,7 @@ class RouteCollectionTest extends \PHPUnit_Framework_TestCase
         $this->assertEquals('/foo1', $collection->get('foo')->getPattern());
     }
 
-    public function testDeepOverridenRoute()
+    public function testDeepOverriddenRoute()
     {
         $collection = new RouteCollection();
         $collection->add('foo', new Route('/foo'));
@@ -64,7 +64,7 @@ class RouteCollectionTest extends \PHPUnit_Framework_TestCase
         $this->assertEquals('/foo2', $collection->get('foo')->getPattern());
     }
 
-    public function testIteratorWithOverridenRoutes()
+    public function testIteratorWithOverriddenRoutes()
     {
         $collection = new RouteCollection();
         $collection->add('foo', new Route('/foo'));

src/Symfony/Component/Security/CHANGELOG.md

@@ -21,7 +21,7 @@ CHANGELOG
    `UserSecurityIdentity::fromToken` now return correct identities for proxies
    objects (e.g. Doctrine proxies)
  * [BC BREAK] moved the default authentication success and failure handling to
-   seperate classes. The order of arguments in the constructor of the
+   separate classes. The order of arguments in the constructor of the
    `AbstractAuthenticationListener` has changed.
- * [BC BREAK] moved the default logout sucess handling to a seperate class. The
+ * [BC BREAK] moved the default logout success handling to a separate class. The
    order of arguments in the constructor of `LogoutListener` has changed.

src/Symfony/Component/Security/Tests/Acl/Dbal/MutableAclProviderTest.php

@@ -267,7 +267,7 @@ class MutableAclProviderTest extends \PHPUnit_Framework_TestCase
         $provider->updateAcl($acl);
     }
 
-    public function testUpdateAclThrowsExceptionOnConcurrentModifcationOfSharedProperties()
+    public function testUpdateAclThrowsExceptionOnConcurrentModificationOfSharedProperties()
     {
         $provider = $this->getProvider();
         $acl1 = $provider->createAcl(new ObjectIdentity(1, 'Foo'));

src/Symfony/Component/Security/Tests/Acl/Domain/AclTest.php

@@ -433,7 +433,7 @@ class AclTest extends \PHPUnit_Framework_TestCase
      * @expectedException \InvalidArgumentException
      * @dataProvider getUpdateFieldAuditingTests
      */
-    public function testUpdateFieldAuditingthrowsExceptionOnInvalidField($type)
+    public function testUpdateFieldAuditingThrowsExceptionOnInvalidField($type)
     {
         $acl = $this->getAcl();
         $acl->{'update'.$type.'Auditing'}(0, 'foo', true, true);

src/Symfony/Component/Security/Tests/Acl/Domain/PermissionGrantingStrategyTest.php

@@ -31,7 +31,7 @@ class PermissionGrantingStrategyTest extends \PHPUnit_Framework_TestCase
         $this->assertFalse($strategy->isGranted($acl, array(1), array($sid)));
     }
 
-    public function testIsGrantedFallsbackToClassAcesIfNoApplicableObjectAceWasFound()
+    public function testIsGrantedFallsBackToClassAcesIfNoApplicableObjectAceWasFound()
     {
         $strategy = new PermissionGrantingStrategy();
         $acl = $this->getAcl($strategy);

src/Symfony/Component/Security/Tests/Acl/Domain/UserSecurityIdentityTest.php

@@ -25,7 +25,7 @@ class UserSecurityIdentityTest extends \PHPUnit_Framework_TestCase
     }
 
     // Test that constructor never changes the type, even for proxies
-    public function testContructorWithProxy()
+    public function testConstructorWithProxy()
     {
         $id = new UserSecurityIdentity('foo', 'Acme\DemoBundle\Proxy\__CG__\Symfony\Component\Security\Tests\Acl\Domain\Foo');
 

src/Symfony/Component/Translation/Tests/Dumper/CsvFileDumperTest.php

@@ -24,7 +24,7 @@ foo', 'foo;foo' => 'bar'));
 
         $tempDir = sys_get_temp_dir();
         $dumper = new CsvFileDumper();
-        $dumperString = $dumper->dump($catalogue, array('path' => $tempDir));
+        $dumper->dump($catalogue, array('path' => $tempDir));
 
         $this->assertEquals(file_get_contents(__DIR__.'/../fixtures/valid.csv'), file_get_contents($tempDir.'/messages.en.csv'));
 

src/Symfony/Component/Translation/Tests/Dumper/IcuResFileDumperTest.php

@@ -27,7 +27,7 @@ class IcuResFileDumperTest extends \PHPUnit_Framework_TestCase
 
         $tempDir = sys_get_temp_dir();
         $dumper = new IcuResFileDumper();
-        $dumperString = $dumper->dump($catalogue, array('path' => $tempDir));
+        $dumper->dump($catalogue, array('path' => $tempDir));
 
         $this->assertEquals(file_get_contents(__DIR__.'/../fixtures/resourcebundle/res/en.res'), file_get_contents($tempDir.'/messages/en.res'));
 

src/Symfony/Component/Translation/Tests/Dumper/IniFileDumperTest.php

@@ -23,7 +23,7 @@ class IniFileDumperTest extends \PHPUnit_Framework_TestCase
 
         $tempDir = sys_get_temp_dir();
         $dumper = new IniFileDumper();
-        $dumperString = $dumper->dump($catalogue, array('path' => $tempDir));
+        $dumper->dump($catalogue, array('path' => $tempDir));
 
         $this->assertEquals(file_get_contents(__DIR__.'/../fixtures/resources.ini'), file_get_contents($tempDir.'/messages.en.ini'));
 

src/Symfony/Component/Translation/Tests/Dumper/MoFileDumperTest.php

@@ -23,7 +23,7 @@ class MoFileDumperTest extends \PHPUnit_Framework_TestCase
 
         $tempDir = sys_get_temp_dir();
         $dumper = new MoFileDumper();
-        $dumperString = $dumper->dump($catalogue, array('path' => $tempDir));
+        $dumper->dump($catalogue, array('path' => $tempDir));
         $this->assertEquals(file_get_contents(__DIR__.'/../fixtures/resources.mo'), file_get_contents($tempDir.'/messages.en.mo'));
 
         unlink($tempDir.'/messages.en.mo');

src/Symfony/Component/Translation/Tests/Dumper/PhpFileDumperTest.php

@@ -23,7 +23,7 @@ class PhpFileDumperTest extends \PHPUnit_Framework_TestCase
 
         $tempDir = sys_get_temp_dir();
         $dumper = new PhpFileDumper();
-        $dumperString = $dumper->dump($catalogue, array('path' => $tempDir));
+        $dumper->dump($catalogue, array('path' => $tempDir));
 
         $this->assertEquals(file_get_contents(__DIR__.'/../fixtures/resources.php'), file_get_contents($tempDir.'/messages.en.php'));
 

src/Symfony/Component/Translation/Tests/Dumper/PoFileDumperTest.php

@@ -23,7 +23,7 @@ class PoFileDumperTest extends \PHPUnit_Framework_TestCase
 
         $tempDir = sys_get_temp_dir();
         $dumper = new PoFileDumper();
-        $dumperString = $dumper->dump($catalogue, array('path' => $tempDir));
+        $dumper->dump($catalogue, array('path' => $tempDir));
         $this->assertEquals(file_get_contents(__DIR__.'/../fixtures/resources.po'), file_get_contents($tempDir.'/messages.en.po'));
 
         unlink($tempDir.'/messages.en.po');

src/Symfony/Component/Translation/Tests/Dumper/QtFileDumperTest.php

@@ -23,7 +23,7 @@ class QtFileDumperTest extends \PHPUnit_Framework_TestCase
 
         $tempDir = sys_get_temp_dir();
         $dumper = new QtFileDumper();
-        $dumperString = $dumper->dump($catalogue, array('path' => $tempDir));
+        $dumper->dump($catalogue, array('path' => $tempDir));
 
         $this->assertEquals(file_get_contents(__DIR__.'/../fixtures/resources.ts'), file_get_contents($tempDir.'/resources.en.ts'));
 

src/Symfony/Component/Translation/Tests/Dumper/XliffFileDumperTest.php

@@ -23,7 +23,7 @@ class XliffFileDumperTest extends \PHPUnit_Framework_TestCase
 
         $tempDir = sys_get_temp_dir();
         $dumper = new XliffFileDumper();
-        $dumperString = $dumper->dump($catalogue, array('path' => $tempDir));
+        $dumper->dump($catalogue, array('path' => $tempDir));
 
         $this->assertEquals(file_get_contents(__DIR__.'/../fixtures/resources-clean.xlf'), file_get_contents($tempDir.'/messages.en.xlf'));
 

src/Symfony/Component/Translation/Tests/Dumper/YamlFileDumperTest.php

@@ -30,7 +30,7 @@ class YamlFileDumperTest extends \PHPUnit_Framework_TestCase
 
         $tempDir = sys_get_temp_dir();
         $dumper = new YamlFileDumper();
-        $dumperString = $dumper->dump($catalogue, array('path' => $tempDir));
+        $dumper->dump($catalogue, array('path' => $tempDir));
 
         $this->assertEquals(file_get_contents(__DIR__.'/../fixtures/resources.yml'), file_get_contents($tempDir.'/messages.en.yml'));
 

src/Symfony/Component/Translation/Tests/PluralizationRulesTest.php

@@ -22,7 +22,7 @@ use Symfony\Component\Translation\PluralizationRules;
  *
  * As mentioned by chx http://drupal.org/node/1273968 we can cover all by testing number from 0 to 199
  *
- * The goal to cover all languagues is to far fetched so this test case is smaller.
+ * The goal to cover all languages is to far fetched so this test case is smaller.
  *
  * @author Clemens Tolboom clemens@build2be.nl
  */

src/Symfony/Component/Translation/Writer/TranslationWriter.php

@@ -53,7 +53,7 @@ class TranslationWriter
      * Writes translation from the catalogue according to the selected format.
      *
      * @param MessageCatalogue $catalogue The message catalogue to dump
-     * @param type             $format    The format to use to dump the messages
+     * @param string           $format    The format to use to dump the messages
      * @param array            $options   Options that are passed to the dumper
      */
     public function writeTranslations(MessageCatalogue $catalogue, $format, $options = array())

src/Symfony/Component/Validator/Tests/Constraints/LengthValidatorTest.php

@@ -108,7 +108,7 @@ class LengthValidatorTest extends \PHPUnit_Framework_TestCase
     public function testValidValuesMin($value, $mbOnly = false)
     {
         if ($mbOnly && !function_exists('mb_strlen')) {
-            return $this->markTestSkipped('mb_strlen does not exist');
+            $this->markTestSkipped('mb_strlen does not exist');
         }
 
         $this->context->expects($this->never())
@@ -124,7 +124,7 @@ class LengthValidatorTest extends \PHPUnit_Framework_TestCase
     public function testValidValuesMax($value, $mbOnly = false)
     {
         if ($mbOnly && !function_exists('mb_strlen')) {
-            return $this->markTestSkipped('mb_strlen does not exist');
+            $this->markTestSkipped('mb_strlen does not exist');
         }
 
         $this->context->expects($this->never())
@@ -140,7 +140,7 @@ class LengthValidatorTest extends \PHPUnit_Framework_TestCase
     public function testValidValuesExact($value, $mbOnly = false)
     {
         if ($mbOnly && !function_exists('mb_strlen')) {
-            return $this->markTestSkipped('mb_strlen does not exist');
+            $this->markTestSkipped('mb_strlen does not exist');
         }
 
         $this->context->expects($this->never())
@@ -156,7 +156,7 @@ class LengthValidatorTest extends \PHPUnit_Framework_TestCase
     public function testInvalidValuesMin($value, $mbOnly = false)
     {
         if ($mbOnly && !function_exists('mb_strlen')) {
-            return $this->markTestSkipped('mb_strlen does not exist');
+            $this->markTestSkipped('mb_strlen does not exist');
         }
 
         $constraint = new Length(array(
@@ -180,7 +180,7 @@ class LengthValidatorTest extends \PHPUnit_Framework_TestCase
     public function testInvalidValuesMax($value, $mbOnly = false)
     {
         if ($mbOnly && !function_exists('mb_strlen')) {
-            return $this->markTestSkipped('mb_strlen does not exist');
+            $this->markTestSkipped('mb_strlen does not exist');
         }
 
         $constraint = new Length(array(
@@ -204,7 +204,7 @@ class LengthValidatorTest extends \PHPUnit_Framework_TestCase
     public function testInvalidValuesExact($value, $mbOnly = false)
     {
         if ($mbOnly && !function_exists('mb_strlen')) {
-            return $this->markTestSkipped('mb_strlen does not exist');
+            $this->markTestSkipped('mb_strlen does not exist');
         }
 
         $constraint = new Length(array(

src/Symfony/Component/Validator/Tests/Constraints/MaxLengthValidatorTest.php

@@ -62,7 +62,7 @@ class MaxLengthValidatorTest extends \PHPUnit_Framework_TestCase
     public function testValidValues($value, $mbOnly = false)
     {
         if ($mbOnly && !function_exists('mb_strlen')) {
-            return $this->markTestSkipped('mb_strlen does not exist');
+            $this->markTestSkipped('mb_strlen does not exist');
         }
 
         $this->context->expects($this->never())
@@ -88,7 +88,7 @@ class MaxLengthValidatorTest extends \PHPUnit_Framework_TestCase
     public function testInvalidValues($value, $mbOnly = false)
     {
         if ($mbOnly && !function_exists('mb_strlen')) {
-            return $this->markTestSkipped('mb_strlen does not exist');
+            $this->markTestSkipped('mb_strlen does not exist');
         }
 
         $constraint = new MaxLength(array(

src/Symfony/Component/Validator/Tests/Constraints/MinLengthValidatorTest.php

@@ -62,7 +62,7 @@ class MinLengthValidatorTest extends \PHPUnit_Framework_TestCase
     public function testValidValues($value, $mbOnly = false)
     {
         if ($mbOnly && !function_exists('mb_strlen')) {
-            return $this->markTestSkipped('mb_strlen does not exist');
+            $this->markTestSkipped('mb_strlen does not exist');
         }
 
         $this->context->expects($this->never())
@@ -88,7 +88,7 @@ class MinLengthValidatorTest extends \PHPUnit_Framework_TestCase
     public function testInvalidValues($value, $mbOnly = false)
     {
         if ($mbOnly && !function_exists('mb_strlen')) {
-            return $this->markTestSkipped('mb_strlen does not exist');
+            $this->markTestSkipped('mb_strlen does not exist');
         }
 
         $constraint = new MinLength(array(