Bug #59610
Implement multiple checkbox in CheckboxViewhelper
Status: | Resolved | Start date: | 2014-06-16 | |
---|---|---|---|---|
Priority: | Should have | Due date: | ||
Assigned To: | - | % Done: | 0% |
|
Category: | Fluid | Spent time: | - | |
Target version: | next-patchlevel | |||
TYPO3 Version: | 6.2 | Is Regression: | No | |
PHP Version: | Sprint Focus: | |||
Complexity: |
Description
To cover the case of multivalue-checkboxes in TYPO3 Core Fluid CheckboxViewhelper I have applied implementation of the FLOW.Fluid CheckboxViewhelper.
I don't know whether it is enough but it looks like all to do is to change the method signature from
public function render($checked = NULL)
to
public function render($checked = NULL, $multiple = NULL)
and remove the @todo annotation in the representing if condition:
elseif (($multiple = FALSE) === TRUE) { // @todo: implement correct as in Flow.Fluid $nameAttribute .= '[]'; }
to
elseif ($multiple === TRUE) { $nameAttribute .= '[]'; }
Related issues
History
#1 Updated by Gerrit Code Review about 1 year 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/30756
#2 Updated by Stefan Froemken 9 months ago
- Status changed from Under Review to Resolved
Applied in changeset ad2cfb7d909b7b7d0d84de3630047c2f6d7833e0.