Project

General

Profile

Actions

Bug #54807

closed

Indexed Search PageBrowsing ViewHelper defines unused method argument in phpdoc comments

Added by Benjamin Rau over 10 years ago. Updated over 5 years ago.

Status:
Closed
Priority:
Must have
Assignee:
Category:
Indexed Search
Target version:
-
Start date:
2014-01-07
Due date:
% Done:

100%

Estimated time:
TYPO3 Version:
6.1
PHP Version:
Tags:
Complexity:
no-brainer
Is Regression:
No
Sprint Focus:

Description

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"

Actions #1

Updated by Gerrit Code Review over 10 years ago

  • Status changed from New to Under Review

Patch set 1 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/26676

Actions #2

Updated by Gerrit Code Review over 10 years ago

Patch set 2 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/26676

Actions #3

Updated by Gerrit Code Review over 10 years ago

Patch set 3 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/26676

Actions #4

Updated by Gerrit Code Review over 10 years ago

Patch set 1 for branch TYPO3_6-1 of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/26679

Actions #5

Updated by Benjamin Rau over 10 years ago

  • Status changed from Under Review to Resolved
  • % Done changed from 0 to 100
Actions #6

Updated by Benni Mack over 5 years ago

  • Status changed from Resolved to Closed
Actions

Also available in: Atom PDF