Bug #92454

Invalid colPos/language UID used in "Languages" view with defLangBinding

Added by Mathias Brodala 10 months ago. Updated 7 months ago.

Status:
Closed
Priority:
Should have
Assignee:
-
Category:
-
Target version:
-
Start date:
2020-09-30
Due date:
% Done:

100%

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

Description

Given the "Languages" view is used in the Page module with mod.web_layout.defLangBinding enabled, when reordering content elements in a custom section (colPos > 0) via drag and drop, invalid values for colPos and sys_language_uid are sent to the backend and eventually the DataHandler:

cmd[tt_content][13][move]: -10
data[tt_content][13][colPos]: false
data[tt_content][13][sys_language_uid]: NaN

Here 13 is the UID of the dragged content element and 10 is the UID of the content element after which the dragged element should be sorted.

This affects sorting to positions anywhere else but the beginning of the section. The "Columns" view is fine however.

This make it impossible to move the record:

2: SQL error: 'Incorrect integer value: 'false' for column 'colPos' at row 1' (tt_content:13)

This can be observed in TYPO3v8 and TYPO3v9. Probably also TYPO3v10 with the classic Page module but this cannot be checked ATM due to #90617.


Related issues

Blocked by TYPO3 Core - Bug #90617: Page Module: No content elements displayed with mod.web_layout.defLangBindingClosed2020-03-02

Actions
#1

Updated by Mathias Brodala 10 months ago

  • Blocked by Bug #90617: Page Module: No content elements displayed with mod.web_layout.defLangBinding added
#2

Updated by Mathias Brodala 10 months ago

  • Description updated (diff)
#3

Updated by Gerrit Code Review 10 months 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/c/Packages/TYPO3.CMS/+/66015

#4

Updated by Oliver Bartsch 10 months ago

As Mathias already mentioned, this is currently blocked by #90617 and can therefore only be tested by applying https://review.typo3.org/c/Packages/TYPO3.CMS/+/66014.

#5

Updated by Gerrit Code Review 9 months ago

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

#6

Updated by Gerrit Code Review 9 months 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/+/66394

#7

Updated by Gerrit Code Review 9 months 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/+/66394

#8

Updated by Oliver Bartsch 9 months ago

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

Updated by Gerrit Code Review 9 months ago

  • Status changed from Resolved to Under Review

Patch set 3 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/+/66394

#10

Updated by Gerrit Code Review 9 months ago

Patch set 4 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/+/66394

#11

Updated by Gerrit Code Review 9 months ago

Patch set 5 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/+/66394

#12

Updated by Oliver Bartsch 9 months ago

  • Status changed from Under Review to Resolved
#13

Updated by Gerrit Code Review 9 months ago

  • Status changed from Resolved to Under Review
#14

Updated by Oliver Hader 9 months ago

  • Status changed from Under Review to Resolved
#15

Updated by Benni Mack 7 months ago

  • Status changed from Resolved to Closed

Also available in: Atom PDF