[BUGFIX] IfViewHelpers use method signature from parent 15/42115/6
authorBenjamin Mack <benni@typo3.org>
Wed, 29 Jul 2015 18:48:33 +0000 (20:48 +0200)
committerSusanne Moog <typo3@susannemoog.de>
Fri, 7 Aug 2015 12:22:12 +0000 (14:22 +0200)
In order to become PHP7 compliant, the IfViewHelpers should
register their arguments via the registerArgument() method
instead of extending the render() method with additional
parameters. This is not allowed in PHP7 and is therefore
changed in order to be compatible with PHP7.

Releases: master
Resolves: #68634
Change-Id: Ia3fa86541d5e112813da3c1d32edfe6180f6f621
Reviewed-on: http://review.typo3.org/42115
Reviewed-by: Markus Klein <markus.klein@typo3.org>
Tested-by: Markus Klein <markus.klein@typo3.org>
Reviewed-by: Susanne Moog <typo3@susannemoog.de>
Tested-by: Susanne Moog <typo3@susannemoog.de>
typo3/sysext/fluid/Classes/ViewHelpers/Be/Security/IfHasRoleViewHelper.php
typo3/sysext/fluid/Classes/ViewHelpers/IfViewHelper.php
typo3/sysext/fluid/Classes/ViewHelpers/Security/IfHasRoleViewHelper.php

index 4f3aefd..fcf99b3 100644 (file)
@@ -63,15 +63,14 @@ namespace TYPO3\CMS\Fluid\ViewHelpers\Be\Security;
 class IfHasRoleViewHelper extends \TYPO3\CMS\Fluid\Core\ViewHelper\AbstractConditionViewHelper {
 
        /**
-        * renders <f:then> child if the current logged in BE user belongs to the specified role (aka usergroup)
+        * Initializes the "role" argument.
+        * Renders <f:then> child if the current logged in BE user belongs to the specified role (aka usergroup)
         * otherwise renders <f:else> child.
         *
-        * @param string $role The usergroup (either the usergroup uid or its title)
-        * @return string the rendered string
-        * @api
+        * @return void
         */
-       public function render($role) {
-               return parent::render();
+       public function initializeArguments() {
+               $this->registerArgument('role', 'string', 'The usergroup (either the usergroup uid or its title).');
        }
 
        /**
index 2db8da4..24ac605 100644 (file)
@@ -84,13 +84,10 @@ namespace TYPO3\CMS\Fluid\ViewHelpers;
 class IfViewHelper extends \TYPO3\CMS\Fluid\Core\ViewHelper\AbstractConditionViewHelper {
 
        /**
-        * renders <f:then> child if $condition is true, otherwise renders <f:else> child.
-        *
-        * @param bool $condition View helper condition
-        * @return string the rendered string
-        * @api
+        * @return void
         */
-       public function render($condition) {
-               return parent::render();
+       public function initializeArguments() {
+               $this->registerArgument('condition', 'boolean', 'Condition to be evaluated.');
        }
+
 }
index 818d963..02fa90e 100644 (file)
@@ -63,15 +63,14 @@ namespace TYPO3\CMS\Fluid\ViewHelpers\Security;
 class IfHasRoleViewHelper extends \TYPO3\CMS\Fluid\Core\ViewHelper\AbstractConditionViewHelper {
 
        /**
-        * renders <f:then> child if the current logged in FE user belongs to the specified role (aka usergroup)
+        * Initializes the "role" argument.
+        * Renders <f:then> child if the current logged in FE user belongs to the specified role (aka usergroup)
         * otherwise renders <f:else> child.
         *
-        * @param string $role The usergroup (either the usergroup uid or its title)
-        * @return string the rendered string
-        * @api
+        * @return void
         */
-       public function render($role) {
-               return parent::render();
+       public function initializeArguments() {
+               $this->registerArgument('role', 'string', 'The usergroup (either the usergroup uid or its title).');
        }
 
        /**