Bug #87038

Unique evaluation does not work with l10n_mode=exclude after editing original record again

Added by Alexander Bohndorf almost 2 years ago. Updated 8 months ago.

Status:
Closed
Priority:
Must have
Assignee:
Category:
DataHandler aka TCEmain
Target version:
-
Start date:
2018-11-29
Due date:
% Done:

100%

Estimated time:
TYPO3 Version:
9
PHP Version:
7.2
Tags:
Complexity:
easy
Is Regression:
Sprint Focus:

Description

Translation handling with l10n_mode=exclude:
the field is copied from default language to language children. In combination with eval=unique the language children get a unique value too like “fieldvalue”.
If you edit and save the original record afterwards then "fieldvalue" turns into "fieldvalue0" because the uniqueness is also checked in translated records.
My expected behaviour is: uniqueness should be ignored for fields with l10n_mode=exclude in translated records.

Related to #84267
Related to #83572


Files

test_extension.zip (4.67 KB) test_extension.zip Jan Helke, 2018-12-07 11:33

Related issues

Related to TYPO3 Core - Bug #90653: TCA eval with uniqueInPid: allow identical value in localized contentNeeds Feedback2020-03-05

Actions
Related to TYPO3 Core - Bug #84267: Unique evaluation does not work with l10n_mode=excludeClosedOliver Hader2018-03-14

Actions
Related to TYPO3 Core - Bug #83572: Changed behaviour for uniqueInPid/unique in translated recordsClosed2018-01-15

Actions
#1

Updated by Alexander Bohndorf almost 2 years ago

  • Description updated (diff)
#2

Updated by Gerrit Code Review almost 2 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/58979

#3

Updated by Gerrit Code Review almost 2 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/58979

#4

Updated by Gerrit Code Review almost 2 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/58979

#5

Updated by Gerrit Code Review almost 2 years ago

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

#6

Updated by Jan Helke almost 2 years ago

I hacked a little extension to test the behaviour as an editor.

Install
Create new language (if needed)
Create new Test element from extension
Create new translation of the page (if needed)
Translate element
Open default language element
Hit save button.

Wrong behaviour: The value in the field will switch from "fieldvalue" to "fieldvalue0"
Correct behaviour: The value stays the same.

#7

Updated by Gerrit Code Review almost 2 years ago

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

#8

Updated by Alexander Bohndorf almost 2 years ago

  • Assignee changed from Alexander Bohndorf to Benni Mack
#9

Updated by Gerrit Code Review almost 2 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/58979

#10

Updated by Gerrit Code Review almost 2 years ago

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

#11

Updated by Gerrit Code Review almost 2 years ago

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

#12

Updated by Gerrit Code Review about 1 year ago

Patch set 1 for branch 9.5 of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/c/Packages/TYPO3.CMS/+/61851

#13

Updated by Gerrit Code Review about 1 year ago

Patch set 2 for branch 9.5 of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/c/Packages/TYPO3.CMS/+/61851

#14

Updated by Gerrit Code Review about 1 year ago

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

#15

Updated by Gerrit Code Review about 1 year ago

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

#16

Updated by Gerrit Code Review about 1 year ago

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

#17

Updated by Gerrit Code Review about 1 year ago

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

#18

Updated by Gerrit Code Review about 1 year ago

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

#19

Updated by Susanne Moog 11 months ago

  • Sprint Focus deleted (On Location Sprint)
#20

Updated by Alexander Bohndorf 9 months ago

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

Updated by Georg Ringer 9 months ago

  • Related to Bug #90653: TCA eval with uniqueInPid: allow identical value in localized content added
#22

Updated by Benni Mack 8 months ago

  • Status changed from Resolved to Closed
#23

Updated by Christian Kuhn 3 months ago

  • Related to Bug #84267: Unique evaluation does not work with l10n_mode=exclude added
#24

Updated by Christian Kuhn 3 months ago

  • Related to Bug #83572: Changed behaviour for uniqueInPid/unique in translated records added

Also available in: Atom PDF