diff --git a/src/Symfony/Component/OptionsResolver/OptionsResolver.php b/src/Symfony/Component/OptionsResolver/OptionsResolver.php index 96f28cc348..f2756cb147 100644 --- a/src/Symfony/Component/OptionsResolver/OptionsResolver.php +++ b/src/Symfony/Component/OptionsResolver/OptionsResolver.php @@ -409,7 +409,7 @@ class OptionsResolver implements Options, OptionsResolverInterface if (!isset($this->defined[$option])) { throw new UndefinedOptionsException(sprintf( - 'The option "%s" does not exist. Known options are: "%s".', + 'The option "%s" does not exist. Defined options are: "%s".', $option, implode('", "', array_keys($this->defined)) )); @@ -473,7 +473,7 @@ class OptionsResolver implements Options, OptionsResolverInterface if (!isset($this->defined[$option])) { throw new UndefinedOptionsException(sprintf( - 'The option "%s" does not exist. Known options are: "%s".', + 'The option "%s" does not exist. Defined options are: "%s".', $option, implode('", "', array_keys($this->defined)) )); @@ -527,7 +527,7 @@ class OptionsResolver implements Options, OptionsResolverInterface if (!isset($this->defined[$option])) { throw new UndefinedOptionsException(sprintf( - 'The option "%s" does not exist. Known options are: "%s".', + 'The option "%s" does not exist. Defined options are: "%s".', $option, implode('", "', array_keys($this->defined)) )); @@ -579,7 +579,7 @@ class OptionsResolver implements Options, OptionsResolverInterface if (!isset($this->defined[$option])) { throw new UndefinedOptionsException(sprintf( - 'The option "%s" does not exist. Known options are: "%s".', + 'The option "%s" does not exist. Defined options are: "%s".', $option, implode('", "', array_keys($this->defined)) )); @@ -627,7 +627,7 @@ class OptionsResolver implements Options, OptionsResolverInterface if (!isset($this->defined[$option])) { throw new UndefinedOptionsException(sprintf( - 'The option "%s" does not exist. Known options are: "%s".', + 'The option "%s" does not exist. Defined options are: "%s".', $option, implode('", "', array_keys($this->defined)) )); @@ -742,7 +742,7 @@ class OptionsResolver implements Options, OptionsResolverInterface ksort($diff); throw new UndefinedOptionsException(sprintf( - (count($diff) > 1 ? 'The options "%s" do not exist.' : 'The option "%s" does not exist.').' Known options are: "%s".', + (count($diff) > 1 ? 'The options "%s" do not exist.' : 'The option "%s" does not exist.').' Defined options are: "%s".', implode('", "', array_keys($diff)), implode('", "', array_keys($clone->defined)) )); @@ -808,7 +808,7 @@ class OptionsResolver implements Options, OptionsResolverInterface if (!array_key_exists($option, $this->defaults)) { if (!isset($this->defined[$option])) { throw new NoSuchOptionException(sprintf( - 'The option "%s" does not exist. Known options are: "%s".', + 'The option "%s" does not exist. Defined options are: "%s".', $option, implode('", "', array_keys($this->defined)) )); diff --git a/src/Symfony/Component/OptionsResolver/Tests/LegacyOptionsResolverTest.php b/src/Symfony/Component/OptionsResolver/Tests/LegacyOptionsResolverTest.php index da9ee69148..c355333918 100644 --- a/src/Symfony/Component/OptionsResolver/Tests/LegacyOptionsResolverTest.php +++ b/src/Symfony/Component/OptionsResolver/Tests/LegacyOptionsResolverTest.php @@ -206,7 +206,8 @@ class LegacyOptionsResolverTest extends \PHPUnit_Framework_TestCase } /** - * @expectedException \Symfony\Component\OptionsResolver\Exception\UndefinedOptionsException + * @expectedException \Symfony\Component\OptionsResolver\Exception\UndefinedOptionsException + * @expectedExceptionMessage The option "foo" does not exist. Defined options are: "one", "three", "two". */ public function testResolveFailsIfNonExistingOption() {