diff --git a/src/Symfony/Component/Form/Tests/Extension/Validator/Constraints/FormValidatorTest.php b/src/Symfony/Component/Form/Tests/Extension/Validator/Constraints/FormValidatorTest.php index 41c7439ba850..4e3f3a47187e 100644 --- a/src/Symfony/Component/Form/Tests/Extension/Validator/Constraints/FormValidatorTest.php +++ b/src/Symfony/Component/Form/Tests/Extension/Validator/Constraints/FormValidatorTest.php @@ -361,7 +361,7 @@ public function testHandleClosureValidationGroups() $object = $this->getMock('\stdClass'); $options = array('validation_groups' => function (FormInterface $form) { return array('group1', 'group2'); - },); + }); $form = $this->getBuilder('name', '\stdClass', $options) ->setData($object) ->getForm(); diff --git a/src/Symfony/Component/Intl/ResourceBundle/CurrencyBundle.php b/src/Symfony/Component/Intl/ResourceBundle/CurrencyBundle.php index f36a05fa54dc..38254fa03618 100644 --- a/src/Symfony/Component/Intl/ResourceBundle/CurrencyBundle.php +++ b/src/Symfony/Component/Intl/ResourceBundle/CurrencyBundle.php @@ -52,7 +52,7 @@ public function getCurrencySymbol($currency, $displayLocale = null) try { return $this->getSymbol($currency, $displayLocale); } catch (MissingResourceException $e) { - return null; + return; } } @@ -64,7 +64,7 @@ public function getCurrencyName($currency, $displayLocale = null) try { return $this->getName($currency, $displayLocale); } catch (MissingResourceException $e) { - return null; + return; } } @@ -76,7 +76,7 @@ public function getCurrencyNames($displayLocale = null) try { return $this->getNames($displayLocale); } catch (MissingResourceException $e) { - return null; + return; } } @@ -88,7 +88,7 @@ public function getFractionDigits($currency) try { return parent::getFractionDigits($currency); } catch (MissingResourceException $e) { - return null; + return; } } @@ -100,7 +100,7 @@ public function getRoundingIncrement($currency) try { return parent::getRoundingIncrement($currency); } catch (MissingResourceException $e) { - return null; + return; } } @@ -112,7 +112,7 @@ public function getLocales() try { return $this->localeProvider->getLocales(); } catch (MissingResourceException $e) { - return null; + return; } } } diff --git a/src/Symfony/Component/Intl/ResourceBundle/LanguageBundle.php b/src/Symfony/Component/Intl/ResourceBundle/LanguageBundle.php index d87910fadc8f..bb393d2063b7 100644 --- a/src/Symfony/Component/Intl/ResourceBundle/LanguageBundle.php +++ b/src/Symfony/Component/Intl/ResourceBundle/LanguageBundle.php @@ -68,7 +68,7 @@ public function getLanguageName($language, $region = null, $displayLocale = null try { return $this->getName($language, $displayLocale); } catch (MissingResourceException $e) { - return null; + return; } } @@ -80,7 +80,7 @@ public function getLanguageNames($displayLocale = null) try { return $this->getNames($displayLocale); } catch (MissingResourceException $e) { - return null; + return; } } @@ -92,7 +92,7 @@ public function getScriptName($script, $language = null, $displayLocale = null) try { return $this->scriptProvider->getName($script, $displayLocale); } catch (MissingResourceException $e) { - return null; + return; } } @@ -104,7 +104,7 @@ public function getScriptNames($displayLocale = null) try { return $this->scriptProvider->getNames($displayLocale); } catch (MissingResourceException $e) { - return null; + return; } } @@ -116,7 +116,7 @@ public function getLocales() try { return $this->localeProvider->getLocales(); } catch (MissingResourceException $e) { - return null; + return; } } } diff --git a/src/Symfony/Component/Intl/ResourceBundle/LocaleBundle.php b/src/Symfony/Component/Intl/ResourceBundle/LocaleBundle.php index f45e9efdc460..22c9aecfb369 100644 --- a/src/Symfony/Component/Intl/ResourceBundle/LocaleBundle.php +++ b/src/Symfony/Component/Intl/ResourceBundle/LocaleBundle.php @@ -31,7 +31,7 @@ public function getLocales() try { return parent::getLocales(); } catch (MissingResourceException $e) { - return null; + return; } } @@ -43,7 +43,7 @@ public function getLocaleName($locale, $displayLocale = null) try { return $this->getName($locale, $displayLocale); } catch (MissingResourceException $e) { - return null; + return; } } @@ -55,7 +55,7 @@ public function getLocaleNames($displayLocale = null) try { return $this->getNames($displayLocale); } catch (MissingResourceException $e) { - return null; + return; } } } diff --git a/src/Symfony/Component/Intl/ResourceBundle/RegionBundle.php b/src/Symfony/Component/Intl/ResourceBundle/RegionBundle.php index a84f7896bb3e..126d62ac04a4 100644 --- a/src/Symfony/Component/Intl/ResourceBundle/RegionBundle.php +++ b/src/Symfony/Component/Intl/ResourceBundle/RegionBundle.php @@ -52,7 +52,7 @@ public function getCountryName($country, $displayLocale = null) try { return $this->getName($country, $displayLocale); } catch (MissingResourceException $e) { - return null; + return; } } @@ -64,7 +64,7 @@ public function getCountryNames($displayLocale = null) try { return $this->getNames($displayLocale); } catch (MissingResourceException $e) { - return null; + return; } } @@ -76,7 +76,7 @@ public function getLocales() try { return $this->localeProvider->getLocales(); } catch (MissingResourceException $e) { - return null; + return; } } } diff --git a/src/Symfony/Component/Intl/Resources/bin/update-data.php b/src/Symfony/Component/Intl/Resources/bin/update-data.php index 5fa5b6a62298..1d92bd851eb0 100644 --- a/src/Symfony/Component/Intl/Resources/bin/update-data.php +++ b/src/Symfony/Component/Intl/Resources/bin/update-data.php @@ -76,7 +76,6 @@ ? $urlVersion : $maxVersion; - echo " $urlVersion\n"; } diff --git a/src/Symfony/Component/Stopwatch/Tests/StopwatchTest.php b/src/Symfony/Component/Stopwatch/Tests/StopwatchTest.php index db24aed2b375..4edc4a706c1a 100644 --- a/src/Symfony/Component/Stopwatch/Tests/StopwatchTest.php +++ b/src/Symfony/Component/Stopwatch/Tests/StopwatchTest.php @@ -56,13 +56,13 @@ public function testIsNotStartedEvent() $events = new \ReflectionProperty('Symfony\Component\Stopwatch\Section', 'events'); $events->setAccessible(true); - $events->setValue( - end($section), - array( - 'foo' => $this->getMockBuilder('Symfony\Component\Stopwatch\StopwatchEvent') - ->setConstructorArgs(array(microtime(true) * 1000)) - ->getMock()) - ); + + $stopwatchMockEvent = $this->getMockBuilder('Symfony\Component\Stopwatch\StopwatchEvent') + ->setConstructorArgs(array(microtime(true) * 1000)) + ->getMock() + ; + + $events->setValue(end($section), array('foo' => $stopwatchMockEvent)); $this->assertFalse($stopwatch->isStarted('foo')); } diff --git a/src/Symfony/Component/Validator/Tests/Constraints/ChoiceValidatorTest.php b/src/Symfony/Component/Validator/Tests/Constraints/ChoiceValidatorTest.php index 7394ef850ac2..59b9e03f4172 100644 --- a/src/Symfony/Component/Validator/Tests/Constraints/ChoiceValidatorTest.php +++ b/src/Symfony/Component/Validator/Tests/Constraints/ChoiceValidatorTest.php @@ -89,7 +89,7 @@ public function testValidChoiceCallbackClosure() { $constraint = new Choice(array('callback' => function () { return array('foo', 'bar'); - }, )); + })); $this->validator->validate('bar', $constraint);