symfony · fabpot · Oct 29, 2012 · Oct 28, 2012
diff --git a/src/Symfony/Component/Console/Helper/DialogHelper.php b/src/Symfony/Component/Console/Helper/DialogHelper.php
@@ -264,9 +264,9 @@ private function hasSttyAvailable()
             return self::$stty;
         }
 
-        exec('/usr/bin/env stty', $output, $exicode);
+        exec('/usr/bin/env stty', $output, $exitcode);
 
-        return self::$stty = $exicode === 0;
+        return self::$stty = $exitcode === 0;
     }
 
     /**

diff --git a/src/Symfony/Component/Console/Tests/ApplicationTest.php b/src/Symfony/Component/Console/Tests/ApplicationTest.php
@@ -598,7 +598,7 @@ public function testOverwritingDefaultInputDefinitionOverwritesDefaultValues()
 
         $inputDefinition = $application->getDefinition();
 
-        // check wether the default arguments and options are not returned any more
+        // check whether the default arguments and options are not returned any more
         $this->assertFalse($inputDefinition->hasArgument('command'));
 
         $this->assertFalse($inputDefinition->hasOption('help'));
@@ -622,7 +622,7 @@ public function testSettingCustomInputDefinitionOverwritesDefaultValues()
 
         $inputDefinition = $application->getDefinition();
 
-        // check wether the default arguments and options are not returned any more
+        // check whether the default arguments and options are not returned any more
         $this->assertFalse($inputDefinition->hasArgument('command'));
 
         $this->assertFalse($inputDefinition->hasOption('help'));

diff --git a/src/Symfony/Component/DomCrawler/Tests/Field/ChoiceFormFieldTest.php b/src/Symfony/Component/DomCrawler/Tests/Field/ChoiceFormFieldTest.php
@@ -258,7 +258,7 @@ public function testUntick()
         $node = $this->createNode('input', '', array('type' => 'checkbox', 'name' => 'name', 'checked' => 'checked'));
         $field = new ChoiceFormField($node);
         $field->untick();
-        $this->assertNull($field->getValue(), '->untick() unticks checkoxes');
+        $this->assertNull($field->getValue(), '->untick() unticks checkboxes');
     }
 
     public function testSelect()

diff --git a/src/Symfony/Component/DomCrawler/Tests/FormTest.php b/src/Symfony/Component/DomCrawler/Tests/FormTest.php
@@ -282,7 +282,7 @@ public function testMultiselectSetValues()
     {
         $form = $this->createForm('<form><select multiple="multiple" name="multi"><option value="foo">foo</option><option value="bar">bar</option></select><input type="submit" /></form>');
         $form->setValues(array('multi' => array("foo", "bar")));
-        $this->assertEquals(array('multi' => array('foo', 'bar')), $form->getValues(), '->setValue() sets tehe values of select');
+        $this->assertEquals(array('multi' => array('foo', 'bar')), $form->getValues(), '->setValue() sets the values of select');
     }
 
     public function testGetPhpValues()

diff --git a/src/Symfony/Component/Finder/Adapter/GnuFindAdapter.php b/src/Symfony/Component/Finder/Adapter/GnuFindAdapter.php
@@ -230,7 +230,7 @@ private function buildDatesFiltering(Command $command, array $dates)
             if (0 > $mins) {
                 // mtime is in the future
                 $command->add(' -mmin -0');
-                // we will have no result so we dont need to continue
+                // we will have no result so we don't need to continue
                 return;
             }
 

diff --git a/src/Symfony/Component/Finder/Tests/FinderTest.php b/src/Symfony/Component/Finder/Tests/FinderTest.php
@@ -528,7 +528,7 @@ public function testNotContainsOnDirectory(Adapter\AdapterInterface $adapter)
 
     /**
      * Searching in multiple locations involves AppendIterator which does an unnecessary rewind which leaves FilterIterator
-     * with inner FilesystemIterator in an ivalid state.
+     * with inner FilesystemIterator in an invalid state.
      *
      * @see https://bugs.php.net/bug.php?id=49104
      *

diff --git a/src/Symfony/Component/Form/Tests/Extension/Core/Type/CollectionTypeTest.php b/src/Symfony/Component/Form/Tests/Extension/Core/Type/CollectionTypeTest.php
@@ -141,7 +141,7 @@ public function testPrototypeMultipartPropagation()
         $this->assertTrue($form->createView()->vars['multipart']);
     }
 
-    public function testGetDataDoesNotContainsProtypeNameBeforeDataAreSet()
+    public function testGetDataDoesNotContainsPrototypeNameBeforeDataAreSet()
     {
         $form = $this->factory->create('collection', array(), array(
             'type'      => 'file',

diff --git a/src/Symfony/Component/HttpFoundation/Response.php b/src/Symfony/Component/HttpFoundation/Response.php
@@ -1097,7 +1097,7 @@ public function isOk()
     }
 
     /**
-     * Is the reponse forbidden?
+     * Is the response forbidden?
      *
      * @return Boolean
      *

diff --git a/src/Symfony/Component/Routing/Tests/Matcher/UrlMatcherTest.php b/src/Symfony/Component/Routing/Tests/Matcher/UrlMatcherTest.php
@@ -240,7 +240,7 @@ public function testAdjacentVariables()
         $matcher = new UrlMatcher($coll, new RequestContext());
         // 'w' eagerly matches as much as possible and the other variables match the remaining chars.
         // This also shows that the variables w-z must all exclude the separating char (the dot '.' in this case) by default requirement.
-        // Otherwise they would also comsume '.xml' and _format would never match as it's an optional variable.
+        // Otherwise they would also consume '.xml' and _format would never match as it's an optional variable.
         $this->assertEquals(array('w' => 'wwwww', 'x' => 'x', 'y' => 'Y', 'z' => 'Z','_format' => 'xml', '_route' => 'test'), $matcher->match('/wwwwwxYZ.xml'));
         // As 'y' has custom requirement and can only be of value 'y|Y', it will leave  'ZZZ' to variable z.
         // So with carefully chosen requirements adjacent variables, can be useful.

diff --git a/src/Symfony/Component/Validator/Tests/Constraints/RegexValidatorTest.php b/src/Symfony/Component/Validator/Tests/Constraints/RegexValidatorTest.php
@@ -151,13 +151,13 @@ public function testHtmlPattern()
         ));
         $this->assertNull($constraint->getHtmlPattern());
 
-        // Automaticaly converted
+        // Automatically converted
         $constraint = new Regex(array(
             'pattern' => '/^[a-z]+$/',
         ));
         $this->assertEquals('[a-z]+', $constraint->getHtmlPattern());
 
-        // Automaticaly converted, adds .*
+        // Automatically converted, adds .*
         $constraint = new Regex(array(
             'pattern' => '/[a-z]+/',
         ));
-Original file line number
+Diff line change
@@ Expand Up / @@ -1097,7 +1097,7 @@ public function isOk() @@
         }
         /**
-         * Is the reponse forbidden?
+         * Is the response forbidden?
          *
          * @return Boolean
          *
@@ Expand Down @@