Bug #60263

sys_category relations: unexpected limit of item number in TYPO3 6.2.4

Added by Christian Reiter over 5 years ago. Updated about 1 year ago.

Status:
Closed
Priority:
Should have
Assignee:
-
Category:
-
Target version:
-
Start date:
2014-07-11
Due date:
% Done:

100%

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

Description

In a TYPO3 6.2.4 project, when more than one FAL metadata record is associated with a sys_category, it is no longer possible to save any changes to the sys_category record when it is edited through the Web>List module on the root page. The tab "Items" which lists the relations is always labelled as "not correctly filled in" as long as there is more than one item.
This affects both existing categories as well as categories newly created and filled with relations after upgrading to 6.2.4
As such it is not possible anymore to do maintenance of categories like changing its name, its parent etc while keeping the relations.

This problem did not exist in 6.2.3 and it disappears when downgrading from 6.2.4 to 6.2.3

The TCA in typo3\sysext\core\Configuration\TCA\sys_category.php doesn't set a value for 'maxitems' in the 'type' => 'group' definition for "items".

The change https://review.typo3.org/#/c/27378/4/typo3/sysext/backend/Classes/Form/FormEngine.php which is inclued in 6.2.4 appears to change the default behaviour of getSingleField_typeGroup when maxitems is not defined in TCA.

Keeping the 6.2.4 codebase but reverting this single change makes the problem with editing sys_category go away.

Screenshot-2014-07-11-10.17.png View (19.5 KB) Christian Reiter, 2014-07-11 10:22


Related issues

Related to TYPO3 Core - Bug #55731: Inconsistent group field maxitems validation Closed 2014-02-06

Associated revisions

Revision 85004e58 (diff)
Added by Wouter Wolters over 5 years ago

Revert "[BUGFIX] Unified maxitems default value for group fields"

Unexpected behaviour. Default of maxitems in now 1 by default
which doesn't work if you need more items. Previously this
was possible. Revert this to have the old working
behaviour again.

Releases: 6.3,6.2
Resolves: #60263
Reverts: #55731
This reverts commit c578d47617f803c9ea167d88000e63a92c13fa93.

Change-Id: Ice5357aecb1969f11c6821cb81824d2d68200717
Reviewed-on: http://review.typo3.org/31818
Reviewed-by: Helmut Hummel <>
Reviewed-by: Wouter Wolters <>
Tested-by: Wouter Wolters <>

Revision 36d1be99 (diff)
Added by Wouter Wolters over 5 years ago

Revert "[BUGFIX] Unified maxitems default value for group fields"

Unexpected behaviour. Default of maxitems in now 1 by default
which doesn't work if you need more items. Previously this
was possible. Revert this to have the old working
behaviour again.

Releases: 6.3,6.2
Resolves: #60263
Reverts: #55731
This reverts commit c578d47617f803c9ea167d88000e63a92c13fa93.

Change-Id: Ice5357aecb1969f11c6821cb81824d2d68200717
Reviewed-on: http://review.typo3.org/32284
Reviewed-by: Wouter Wolters <>
Tested-by: Wouter Wolters <>

History

#1 Updated by Tobias Pierschel over 5 years ago

Have the same issue.

#2 Updated by Alexander Opitz over 5 years ago

  • Status changed from New to Accepted
  • Is Regression changed from No to Yes

#3 Updated by Gerrit Code Review over 5 years ago

  • Status changed from Accepted 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/31818

#4 Updated by Gerrit Code Review over 5 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/31818

#5 Updated by Gerrit Code Review over 5 years ago

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

#6 Updated by Wouter Wolters over 5 years ago

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

#7 Updated by Benni Mack about 1 year ago

  • Status changed from Resolved to Closed

Also available in: Atom PDF