From 828b18f4673f09832be1692bd99bfe6de9112689 Mon Sep 17 00:00:00 2001 From: Katsuhiro OGAWA Date: Wed, 5 Oct 2011 17:57:43 +0900 Subject: [PATCH 1/2] [Form] Fixed lacking attributes in DateTimeType --- .../Form/Extension/Core/Type/DateTimeType.php | 4 ++++ .../Extension/Core/Type/DateTimeTypeTest.php | 24 +++++++++++++++++++ 2 files changed, 28 insertions(+) diff --git a/src/Symfony/Component/Form/Extension/Core/Type/DateTimeType.php b/src/Symfony/Component/Form/Extension/Core/Type/DateTimeType.php index e966e26f86..46a2bb1dc0 100644 --- a/src/Symfony/Component/Form/Extension/Core/Type/DateTimeType.php +++ b/src/Symfony/Component/Form/Extension/Core/Type/DateTimeType.php @@ -55,6 +55,8 @@ class DateTimeType extends AbstractType 'days', 'empty_value', 'required', + 'invalid_message', + 'invalid_message_parameters' ))); $timeOptions = array_intersect_key($options, array_flip(array( 'hours', @@ -63,6 +65,8 @@ class DateTimeType extends AbstractType 'with_seconds', 'empty_value', 'required', + 'invalid_message', + 'invalid_message_parameters' ))); // If `widget` is set, overwrite widget options from `date` and `time` diff --git a/tests/Symfony/Tests/Component/Form/Extension/Core/Type/DateTimeTypeTest.php b/tests/Symfony/Tests/Component/Form/Extension/Core/Type/DateTimeTypeTest.php index 422fa36637..c606fbb2da 100644 --- a/tests/Symfony/Tests/Component/Form/Extension/Core/Type/DateTimeTypeTest.php +++ b/tests/Symfony/Tests/Component/Form/Extension/Core/Type/DateTimeTypeTest.php @@ -16,6 +16,7 @@ require_once __DIR__ . '/LocalizedTestCase.php'; use Symfony\Component\Form\DateTimeField; use Symfony\Component\Form\DateField; use Symfony\Component\Form\TimeField; +use Symfony\Component\Form\FormError; class DateTimeTypeTest extends LocalizedTestCase { @@ -234,4 +235,27 @@ class DateTimeTypeTest extends LocalizedTestCase $this->assertDateTimeEquals($dateTime, $form->getData()); } + + public function testSubmit_invalidDateTime() + { + $form = $this->factory->create('datetime', null, array( + 'invalid_message' => 'Customized invalid message', + )); + + $form->bind(array( + 'date' => array( + 'day' => '31', + 'month' => '9', + 'year' => '2010', + ), + 'time' => array( + 'hour' => '25', + 'minute' => '4', + ), + )); + + $this->assertFalse($form->isValid()); + $this->assertEquals(array(new FormError('Customized invalid message', array())), $form['date']->getErrors()); + $this->assertEquals(array(new FormError('Customized invalid message', array())), $form['time']->getErrors()); + } } From 15f19e50fc94384897549d8119b026dc6ed71721 Mon Sep 17 00:00:00 2001 From: Fabien Potencier Date: Thu, 6 Oct 2011 21:40:38 +0200 Subject: [PATCH 2/2] [DoctrineBundle] fixed a unit test (detected thanks to PHP 3.6.0) --- .../Tests/DependencyInjection/AbstractDoctrineExtensionTest.php | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/Symfony/Bundle/DoctrineBundle/Tests/DependencyInjection/AbstractDoctrineExtensionTest.php b/src/Symfony/Bundle/DoctrineBundle/Tests/DependencyInjection/AbstractDoctrineExtensionTest.php index 812a05b95a..f3dfd74387 100644 --- a/src/Symfony/Bundle/DoctrineBundle/Tests/DependencyInjection/AbstractDoctrineExtensionTest.php +++ b/src/Symfony/Bundle/DoctrineBundle/Tests/DependencyInjection/AbstractDoctrineExtensionTest.php @@ -513,7 +513,7 @@ abstract class AbstractDoctrineExtensionTest extends TestCase $this->assertEquals('%doctrine.orm.cache.apc.class%', $cacheDefinition->getClass()); $configDefinition = $container->getDefinition('doctrine.orm.default_configuration'); - $this->assertDICDefinitionMethodCallOnce($configDefinition, 'setAutoGenerateProxyClasses', array(true)); + $this->assertDICDefinitionMethodCallOnce($configDefinition, 'setAutoGenerateProxyClasses', array('%doctrine.orm.auto_generate_proxy_classes%')); } public function testSingleEntityManagerMultipleMappingBundleDefinitions()