Reverting r3994: DateViewHelper now supports strftime format strings
authorSebastian Kurfürst <sebastian@typo3.org>
Wed, 26 Jan 2011 08:08:16 +0000 (08:08 +0000)
committerSebastian Kurfürst <sebastian@typo3.org>
Wed, 26 Jan 2011 08:08:16 +0000 (08:08 +0000)
because of non-consensus in discussion.

typo3/sysext/fluid/Classes/ViewHelpers/Format/DateViewHelper.php
typo3/sysext/fluid/Tests/Unit/ViewHelpers/Format/DateViewHelperTest.php

index 4ec37fa..3b27041 100644 (file)
@@ -34,7 +34,7 @@
  * </output>
  *
  * <code title="Custom date format">
- * <f:format.date format="%H:%M">{dateObject}</f:format.date>
+ * <f:format.date format="H:i">{dateObject}</f:format.date>
  * </code>
  * <output>
  * 01:23
@@ -50,7 +50,7 @@
  * </output>
  *
  * <code title="strtotime string">
- * <f:format.date format="%d.%m.%Y - %H:%M:%S">+1 week 2 days 4 hours 2 seconds</f:format.date>
+ * <f:format.date format="d.m.Y - H:i:s">+1 week 2 days 4 hours 2 seconds</f:format.date>
  * </code>
  * <output>
  * 13.12.1980 - 21:03:42
@@ -58,7 +58,7 @@
  * </output>
  *
  * <code title="output date from unix timestamp">
- * <f:format.date format="%d.%m.%Y - %H:%M:%S">@{someTimestamp}</f:format.date>
+ * <f:format.date format="d.m.Y - H:i:s">@{someTimestamp}</f:format.date>
  * </code>
  * <output>
  * 13.12.1980 - 21:03:42
@@ -90,14 +90,13 @@ class Tx_Fluid_ViewHelpers_Format_DateViewHelper extends Tx_Fluid_Core_ViewHelpe
         * Render the supplied DateTime object as a formatted date.
         *
         * @param mixed $date either a DateTime object or a string that is accepted by DateTime constructor
-        * @param string $format Format String which is taken to format the date/time. Note: This expects a format string that is accepted by phps strftime() function
+        * @param string $format Format String which is taken to format the Date/Time
         * @return string Formatted date
         * @author Christopher Hlubek <hlubek@networkteam.com>
         * @author Bastian Waidelich <bastian@typo3.org>
         * @api
-        * @see http://www.php.net/strftime
         */
-       public function render($date = NULL, $format = '%Y-%m-%d') {
+       public function render($date = NULL, $format = 'Y-m-d') {
                if ($date === NULL) {
                        $date = $this->renderChildren();
                        if ($date === NULL) {
@@ -111,12 +110,7 @@ class Tx_Fluid_ViewHelpers_Format_DateViewHelper extends Tx_Fluid_Core_ViewHelpe
                                throw new Tx_Fluid_Core_ViewHelper_Exception('"' . $date . '" could not be parsed by DateTime constructor.', 1241722579);
                        }
                }
-               // fallback to dateTime format for compatibility reasons. To be removed in 1.5
-               if (strpos($format, '%') === FALSE) {
-                       t3lib_div::deprecationLog('ViewHelper format.date: The format "' . $format . '" is in dateTime format. This is deprecated since Fluid version 1.3. Please use strftime() format instead"');
-                       return $date->format($format);
-               }
-               return strftime($format, $date->format('U'));
+               return $date->format($format);
        }
 }
 ?>
\ No newline at end of file
index c1c429a..1cd3c72 100644 (file)
@@ -50,7 +50,7 @@ class Tx_Fluid_Tests_Unit_ViewHelpers_Format_DateViewHelperTest extends Tx_Extba
         */
        public function viewHelperRespectsCustomFormat() {
                $viewHelper = new Tx_Fluid_ViewHelpers_Format_DateViewHelper();
-               $actualResult = $viewHelper->render(new DateTime('1980-02-01'), '%d.%m.%Y');
+               $actualResult = $viewHelper->render(new DateTime('1980-02-01'), 'd.m.Y');
                $this->assertEquals('01.02.1980', $actualResult);
        }
 
@@ -71,7 +71,6 @@ class Tx_Fluid_Tests_Unit_ViewHelpers_Format_DateViewHelperTest extends Tx_Extba
         * @author Bastian Waidelich <bastian@typo3.org>
         */
        public function viewHelperThrowsExceptionIfDateStringCantBeParsed() {
-               $this->markTestSkipped('This test crashes in the current version of PhpUnit..');
                $viewHelper = new Tx_Fluid_ViewHelpers_Format_DateViewHelper();
                $viewHelper->render('foo');
        }
@@ -97,15 +96,5 @@ class Tx_Fluid_Tests_Unit_ViewHelpers_Format_DateViewHelperTest extends Tx_Extba
                $actualResult = $viewHelper->render('1980-12-12');
                $this->assertEquals('1980-12-12', $actualResult);
        }
-
-       /**
-        * @test
-        * @author Bastian Waidelich <bastian@typo3.org>
-        */
-       public function viewHelperRespectsCustomDateTimeFormat() {
-               $viewHelper = new Tx_Fluid_ViewHelpers_Format_DateViewHelper();
-               $actualResult = $viewHelper->render(new DateTime('1980-02-01'), 'd.m.Y');
-               $this->assertEquals('01.02.1980', $actualResult);
-       }
 }
 ?>