merged branch bschussek/datetimeimpr (PR symfony#5896)

fabpot · fabpot · commit c589982df69c · 2012-11-02T18:59:59.000+01:00
This PR was merged into the 2.1 branch.

Commits
-------

a0af8bf [Form] Adapted HTML5 format in DateTimeType as response to a closed ICU ticket

Discussion
----------

[Form] Adapted HTML5 format in DateTimeType as response to a closed ICU ticket

Bug fix: no
Feature addition: no
Backwards compatibility break: no
Symfony2 tests pass: yes
Fixes the following tickets: -
Todo: -
License of the code: MIT
Documentation PR: -
diff --git a/src/Symfony/Component/Form/Extension/Core/Type/DateTimeType.php b/src/Symfony/Component/Form/Extension/Core/Type/DateTimeType.php
@@ -42,20 +42,20 @@ class DateTimeType extends AbstractType
      *  * "Z" for UTC
      *  * "(-|+)HH:mm" for other timezones (note the colon!)
      *
+     * For more information see:
+     *
      * http://userguide.icu-project.org/formatparse/datetime#TOC-Date-Time-Format-Syntax
      * http://www.w3.org/TR/html-markup/input.datetime.html
      * http://tools.ietf.org/html/rfc3339
      *
      * An ICU ticket was created:
      * http://icu-project.org/trac/ticket/9421
      *
-     * To temporarily circumvent this issue, DateTimeToRfc3339Transformer is used
-     * when the format matches this constant.
-     *
-     * ("ZZZZZZ" is not recognized by ICU and used here to differentiate this
-     * pattern from custom patterns).
+     * It was supposedly fixed, but is not available in all PHP installations
+     * yet. To temporarily circumvent this issue, DateTimeToRfc3339Transformer
+     * is used when the format matches this constant.
      */
-    const HTML5_FORMAT = "yyyy-MM-dd'T'HH:mm:ssZZZZZZ";
+    const HTML5_FORMAT = "yyyy-MM-dd'T'HH:mm:ssZZZZZ";
 
     private static $acceptedFormats = array(
         \IntlDateFormatter::FULL,
