Project

General

Profile

Actions

Bug #92454

closed

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

Added by Mathias Brodala about 4 years ago. Updated almost 4 years 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 1 (0 open1 closed)

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

Actions
Actions #1

Updated by Mathias Brodala about 4 years ago

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

Updated by Mathias Brodala about 4 years ago

  • Description updated (diff)
Actions #3

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

Actions #4

Updated by Oliver Bartsch about 4 years 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.

Actions #5

Updated by Gerrit Code Review about 4 years 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

Actions #6

Updated by Gerrit Code Review about 4 years 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

Actions #7

Updated by Gerrit Code Review about 4 years 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

Actions #8

Updated by Oliver Bartsch about 4 years ago

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

Updated by Gerrit Code Review about 4 years 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

Actions #10

Updated by Gerrit Code Review about 4 years 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

Actions #11

Updated by Gerrit Code Review about 4 years 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

Actions #12

Updated by Oliver Bartsch about 4 years ago

  • Status changed from Under Review to Resolved
Actions #13

Updated by Gerrit Code Review about 4 years ago

  • Status changed from Resolved to Under Review
Actions #14

Updated by Oliver Hader about 4 years ago

  • Status changed from Under Review to Resolved
Actions #15

Updated by Benni Mack almost 4 years ago

  • Status changed from Resolved to Closed
Actions

Also available in: Atom PDF