[FOLLOWUP][TASK] Remove render arguments in BytesVH 39/49339/2
authorAnja Leichsenring <aleichsenring@ab-softlab.de>
Wed, 3 Aug 2016 08:35:38 +0000 (10:35 +0200)
committerAndreas Fernandez <typo3@scripting-base.de>
Wed, 3 Aug 2016 11:57:12 +0000 (13:57 +0200)
We missed the default value for the decimals argument.
It is added now.

Resolves: #77046
Releases: master
Change-Id: If5c0e2b223b13921c97fd50b501f18cbc64b8be4
Reviewed-on: https://review.typo3.org/49339
Tested-by: Bamboo TYPO3com <info@typo3.com>
Reviewed-by: Wouter Wolters <typo3@wouterwolters.nl>
Reviewed-by: Nicole Cordes <typo3@cordes.co>
Tested-by: Nicole Cordes <typo3@cordes.co>
Reviewed-by: Andreas Fernandez <typo3@scripting-base.de>
Tested-by: Andreas Fernandez <typo3@scripting-base.de>
typo3/sysext/fluid/Classes/ViewHelpers/Format/BytesViewHelper.php

index c7bd8e0..d4769a1 100644 (file)
@@ -69,15 +69,11 @@ class BytesViewHelper extends AbstractViewHelper
     public function initializeArguments()
     {
         parent::initializeArguments();
-        $this->registerArgument('value', 'int',
-            'The incoming data to convert, or NULL if VH children should be used');
-        $this->registerArgument('decimals', 'int', 'The number of digits after the decimal point');
-        $this->registerArgument('decimalSeparator', 'string', 'The decimal point character', false,
-            '.');
-        $this->registerArgument('thousandsSeparator', 'string',
-            'The character for grouping the thousand digits', false, ',');
-        $this->registerArgument('units', 'string',
-            'comma separated list of available units, default is LocalizationUtility::translate(\'viewhelper.format.bytes.units\', \'fluid\')');
+        $this->registerArgument('value', 'int', 'The incoming data to convert, or NULL if VH children should be used');
+        $this->registerArgument('decimals', 'int', 'The number of digits after the decimal point', false, 0);
+        $this->registerArgument('decimalSeparator', 'string', 'The decimal point character', false, '.');
+        $this->registerArgument('thousandsSeparator', 'string', 'The character for grouping the thousand digits', false, ',');
+        $this->registerArgument('units', 'string', 'comma separated list of available units, default is LocalizationUtility::translate(\'viewhelper.format.bytes.units\', \'fluid\')');
     }
 
     /**
@@ -104,11 +100,8 @@ class BytesViewHelper extends AbstractViewHelper
      *
      * @return string
      */
-    public static function renderStatic(
-        array $arguments,
-        \Closure $renderChildrenClosure,
-        RenderingContextInterface $renderingContext
-    ) {
+    public static function renderStatic(array $arguments, \Closure $renderChildrenClosure, RenderingContextInterface $renderingContext)
+    {
         if ($arguments['units'] !== null) {
             $units = $arguments['units'];
         } else {