Bug #65345

Cannot set select multiple conditionally

Added by Mathias Brodala about 6 years ago. Updated over 2 years ago.

Status:
Closed
Priority:
Must have
Category:
Fluid
Target version:
-
Start date:
2015-02-25
Due date:
% Done:

100%

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

Description

Currently one cannot set the multiple attribute of the SelectViewHelper conditionally like this:

<f:form.select property="foo" options="{options}" multiple="{multiple}"/>

As soon as the multiple attribute is used, it is rendered in any case with whatever value was passed. (Thus if you pass true as suggested currently, you get an invalid multiple="true".

This should be fixed by converting multiple into a boolean argument.

#1

Updated by Gerrit Code Review about 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 http://review.typo3.org/37238

#2

Updated by Gerrit Code Review about 6 years ago

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

#3

Updated by Gerrit Code Review almost 6 years ago

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

#4

Updated by Gerrit Code Review almost 6 years ago

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

#5

Updated by Gerrit Code Review over 5 years ago

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

#6

Updated by Mathias Schreiber over 5 years ago

  • Target version deleted (next-patchlevel)
#7

Updated by Gerrit Code Review over 5 years ago

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

#8

Updated by Mathias Brodala over 5 years ago

  • Status changed from Under Review to Resolved
  • % Done changed from 50 to 100
#9

Updated by Benni Mack over 2 years ago

  • Status changed from Resolved to Closed

Also available in: Atom PDF