[BUGFIX] PageBrowsing ViewHelper defines unused method argument 76/26676/4
authorBenjamin Rau <rau@codearts.at>
Tue, 7 Jan 2014 17:08:37 +0000 (18:08 +0100)
committerAnja Leichsenring <aleichsenring@ab-softlab.de>
Tue, 7 Jan 2014 17:39:46 +0000 (18:39 +0100)
The PHP-Doc comments for the render-method in PageBrowsing
Viewhelper on Indexed Search define a $details argument
for that method with is then not part of the method arguments
itself.

As it seems to be unused the argument definition should simply
be removed from doc comments to dont mess up the defintions
of the other arguments.

Right now this results in maximumNumberOfResultPages be defined
as array and causes a fatal error like:
The argument "maximumNumberOfResultPages" was registered with
type "array", but is of type "string" in view helper
"TYPO3\CMS\IndexedSearch\ViewHelpers\PageBrowsingViewHelper"

Resolves: #54807
Releases: 6.2, 6.1
Change-Id: I58376880ef64a1e1088561fbdb3ff25f9c1fbdc2
Reviewed-on: https://review.typo3.org/26676
Reviewed-by: Anja Leichsenring
Tested-by: Anja Leichsenring
typo3/sysext/indexed_search/Classes/ViewHelpers/PageBrowsingViewHelper.php

index a0047ea..2a72977 100644 (file)
@@ -40,7 +40,6 @@ class PageBrowsingViewHelper extends \TYPO3\CMS\Fluid\Core\ViewHelper\AbstractVi
        /**
         * main render function
         *
        /**
         * main render function
         *
-        * @param array $details        an array with the browser settings
         * @param integer $maximumNumberOfResultPages
         * @param integer $numberOfResults
         * @param integer $resultsPerPage
         * @param integer $maximumNumberOfResultPages
         * @param integer $numberOfResults
         * @param integer $resultsPerPage