Bug #68247
closed
Story #69712: Further FormEngine development
group maxitems woes
100%
Description
The maxitems setting for TCA group fields is somehow unclear.
tca docu states there is a default of 1 if no maxitems is set. but this only seems to fire in case there is no mm relation (?).
for "csv" relations, the maxitems=1 is active. Current behavior is that if no maxitems is set in csv relation and default 1 kicks in, the formengine still allows selecting more than one element, but datahandler throws all of them away except the first one on saving. this is a bug in formengine validation and should be fixed.
Additionally, we should find out when, where and why the maxitem default kicks in and think about if that makes sense, or if a different default should be chosen consistently (0 for "no restriction, or 9999 or similar?). This implies thought about b/w compatibility and maybe a tca migration, then.
it might be a good idea to enrich ext:styleguide with additional scenarios to nail the current and the expected situation.
Updated by Christian Kuhn about 9 years ago
- Category set to FormEngine aka TCEforms
Updated by Frank Nägler almost 9 years ago
- Status changed from New to Needs Feedback
I can't reproduce the problem.
Can you provide example(s) in EXT:styleguide?
Updated by Alexander Opitz almost 9 years ago
- PHP Version deleted (
7.0)
It seems this haven't todo with PHP version, so unsetting.
Updated by Alexander Opitz over 8 years ago
- Status changed from Needs Feedback to Closed
- Target version deleted (
7 LTS)
No feedback within the last 90 days => closing this issue.
If you think that this is the wrong decision or experience this issue again, then please write to the mailing list typo3.teams.bugs with issue number and an explanation or open a new ticket and add a relation to this ticket number.
Updated by Christian Kuhn over 8 years ago
- Status changed from Closed to New
I think this is still the case. at least the whole maxitem area in combination with the different renderTypes is a bit weird and could be simplified.
I re-opened the issues and set #69712 as parent. That should make it more obvious that this issue is still an area to improve / adapt TCA syntax, simplify and document in a better way.
Updated by Gerrit Code Review almost 8 years ago
- Status changed from New to Under Review
Patch set 39 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/50879
Updated by Gerrit Code Review almost 8 years ago
Patch set 40 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/50879
Updated by Gerrit Code Review almost 8 years ago
Patch set 41 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/50879
Updated by Gerrit Code Review almost 8 years ago
Patch set 42 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/50879
Updated by Gerrit Code Review almost 8 years ago
Patch set 43 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/50879
Updated by Gerrit Code Review almost 8 years ago
Patch set 44 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/50879
Updated by Gerrit Code Review almost 8 years ago
Patch set 45 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/50879
Updated by Gerrit Code Review almost 8 years ago
Patch set 46 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/50879
Updated by Gerrit Code Review almost 8 years ago
Patch set 47 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/50879
Updated by Gerrit Code Review almost 8 years ago
Patch set 48 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/50879
Updated by Christian Kuhn almost 8 years ago
- Status changed from Under Review to Resolved
- % Done changed from 0 to 100
Applied in changeset 38a1bc5dd7990b63e45c0419065cd02fe91187e9.
Updated by