Bug #59610

Implement multiple checkbox in CheckboxViewhelper

Added by Max Hellwig about 1 year ago. Updated 9 months ago.

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

duplicates Core - Bug #61529: Cannot ensure checkbox for multiple values Resolved 2014-09-11

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

Also available in: Atom PDF