Project

General

Profile

Actions

Bug #85013

open

Drag & Drop, Copy/Cut/Paste don't change colPos in localized content elements if the target is the first position

Added by Attila Glück over 6 years ago. Updated about 1 month ago.

Status:
Under Review
Priority:
Should have
Assignee:
-
Category:
Backend User Interface
Start date:
2018-05-15
Due date:
% Done:

0%

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

Description

See: https://youtu.be/mNCrBRyCes8

DataHandler.php Line 1015
The colPos value don't exist in $incomingFieldArray

Actions #1

Updated by Attila Glück over 6 years ago

If you move the element after an other element in an other colpos, it works!
Because $destPid exist in moveRecord_raw() (DataHandler.php line 4460)

Actions #2

Updated by Attila Glück over 6 years ago

  • Priority changed from Should have to Must have

Steps to reproduce the problem:
-Create and localize 2 content elements
-Move a content element - whether drag and drop or cut/paste - to the first position in other colPos

What is the expected behaviour:
-Localized content element should move too

At the first position no destPid value exist and in localized element no colPos value provided.
I think we need always provide the colPos value to moveRecord() in DataHandler.php

Actions #3

Updated by Attila Glück over 6 years ago

Version 6, 7 and 8 is affected

Actions #4

Updated by Attila Glück over 6 years ago

  • Subject changed from Drag and Drop don't change colpos in localized content elements to Drag & Drop, Copy/Cut/Paste don't change colPos in localized content elements if the target is the first position
  • Assignee set to Mathias Schreiber
Actions #5

Updated by Simon Gilli over 6 years ago

  • Assignee deleted (Mathias Schreiber)

Can reproduce this on master too

Actions #6

Updated by Simon Gilli over 6 years ago

  • Status changed from New to Accepted
  • Target version set to Candidate for patchlevel
  • TYPO3 Version changed from 8 to 9
  • Is Regression set to No
Actions #7

Updated by Simon Gilli over 6 years ago

  • Priority changed from Must have to Should have
Actions #8

Updated by Riccardo De Contardi over 5 years ago

This issue is still reproducible on latest master; see the video in the description to reproduce it.

Actions #9

Updated by Gerrit Code Review over 4 years ago

  • Status changed from Accepted to Under Review

Patch set 18 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/+/63354

Actions #10

Updated by Gerrit Code Review over 4 years ago

Patch set 19 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/+/63354

Actions #11

Updated by Gerrit Code Review over 4 years ago

Patch set 20 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/+/63354

Actions #12

Updated by Gerrit Code Review over 4 years ago

Patch set 21 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/+/63354

Actions #13

Updated by Gerrit Code Review over 4 years ago

Patch set 22 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/+/63354

Actions #14

Updated by Gerrit Code Review over 4 years ago

Patch set 23 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/+/63354

Actions #15

Updated by Gerrit Code Review over 4 years ago

Patch set 24 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/+/63354

Actions #16

Updated by Gerrit Code Review over 4 years ago

Patch set 25 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/+/63354

Actions #17

Updated by Gerrit Code Review over 4 years ago

Patch set 26 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/+/63354

Actions #18

Updated by Gerrit Code Review over 4 years ago

Patch set 27 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/+/63354

Actions #19

Updated by Gerrit Code Review over 4 years ago

Patch set 28 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/+/63354

Actions #20

Updated by Gerrit Code Review over 4 years ago

Patch set 29 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/+/63354

Actions #21

Updated by Gerrit Code Review over 4 years ago

Patch set 30 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/+/63354

Actions #22

Updated by Gerrit Code Review over 4 years ago

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/+/63622

Actions #23

Updated by Gerrit Code Review almost 3 years ago

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

Actions #24

Updated by Gerrit Code Review over 1 year ago

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

Actions #25

Updated by Claus Due over 1 year ago

  • Status changed from Under Review to Closed

Associated patch was abandoned. This is not a problem in TYPO3v10 and I'm pretty sure we don't care about it in TYPO3 <= 9.5. Closing the issue.

Actions #26

Updated by Gerrit Code Review about 1 month ago

  • Status changed from Closed to Under Review

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

Actions

Also available in: Atom PDF