Bug #55731

Inconsistent group field maxitems validation

Added by Alexander Stehlik over 6 years ago. Updated over 2 years ago.

Status:
Closed
Priority:
Should have
Category:
FormEngine aka TCEforms
Target version:
Start date:
2014-02-06
Due date:
% Done:

100%

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

Description

During the validation when a group field value is stored in the database by the DataHandler the value for maxitems defaults to 1 (as described in the TCA reference).

In the Backend form the maxitems value of a group field defaults to 10000. So if you do not enter a maxitems value you can add 10000 items to the form element but only one will be stored in the database.

testmaxitems_0.0.1_201407111016.zip (1.34 KB) Alexander Stehlik, 2014-07-11 10:20


Related issues

Related to TYPO3 Core - Bug #60263: sys_category relations: unexpected limit of item number in TYPO3 6.2.4 Closed 2014-07-11

Associated revisions

Revision c578d476 (diff)
Added by Alexander Stehlik almost 6 years ago

[BUGFIX] Unified maxitems default value for group fields

The maxitems value for a group field defaults to 1 instead of 100000 so
that it is consistent with the TCA reference and the validation in the
DataHandler.

Resolves: #55731
Releases: 6.2
Change-Id: I517e45a9227ab9d971a9e7bc4410498319192517
Reviewed-on: https://review.typo3.org/27378
Reviewed-by: Xavier Perseguers
Tested-by: Xavier Perseguers
Reviewed-by: Wouter Wolters
Tested-by: Wouter Wolters

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/27378

#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 https://review.typo3.org/27378

#3 Updated by Gerrit Code Review about 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 https://review.typo3.org/27378

#4 Updated by Alexander Stehlik almost 6 years ago

  • Status changed from Under Review to Resolved
  • % Done changed from 0 to 100

#5 Updated by Alexander Stehlik almost 6 years ago

Attached Extension for testing. Test instructions:

  • note that no "maxitems" config is present in the TCA
  • Revert TYPO3 core patch git revert c578d476
  • Create a new test record
  • Choose at least two files in the group field
  • save the record
  • See that only one record is stored in the database
  • Undo revert
  • See that it is not possible to add more than one record in the first place

#6 Updated by Riccardo De Contardi over 2 years ago

  • Status changed from Resolved to Closed

Also available in: Atom PDF