Bug #59610

Implement multiple checkbox in CheckboxViewhelper

Added by Max Hellwig over 6 years ago. Updated about 2 years ago.

Status:
Closed
Priority:
Should have
Assignee:
-
Category:
Fluid
Target version:
Start date:
2014-06-16
Due date:
% Done:

0%

TYPO3 Version:
6.2
PHP Version:
Tags:
Complexity:
Is Regression:
No
Sprint Focus:

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 TYPO3 Core - Bug #61529: Cannot ensure checkbox for multiple values Closed 2014-09-11

History

#1 Updated by Gerrit Code Review over 6 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/30756

#2 Updated by Stefan Froemken about 6 years ago

  • Status changed from Under Review to Resolved

#3 Updated by Benni Mack about 2 years ago

  • Status changed from Resolved to Closed

Also available in: Atom PDF