Project

General

Profile

Actions

Bug #105680

closed

Invalid localStorage treeState prevents dragging elements after a tree-node

Added by Andreas Nedbal about 11 hours ago. Updated about 7 hours ago.

Status:
Resolved
Priority:
Should have
Category:
Backend JavaScript
Start date:
2024-11-22
Due date:
% Done:

100%

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

Description

Example:

You have:

  • Page 1
    • Page 2
      • Page 4
    • Page 3

If you delete "Page 4" and end up with following tree:

  • Page 1
    • Page 2
    • Page 3

Then you can't drag an element at the end of "Page 2", the only drag state that works is at the top of "Page 3". If this element was the last in a tree branch, you would not be able to append new elements at the end of it.

This happens because on opening pages with children, that tree state is persisted, but if the child page doesn't exist anymore (either deleted by myself or someone else) the invalid "expanded" state for the parent is still persisted. The tree fetches this and sets it regardless of it still being the case at the moment.

Actions #1

Updated by Gerrit Code Review about 11 hours ago

  • Status changed from New to Under Review

Patch set 1 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/+/87168

Actions #2

Updated by Gerrit Code Review about 11 hours 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/+/87168

Actions #3

Updated by Gerrit Code Review about 7 hours ago

Patch set 1 for branch 13.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/+/87184

Actions #4

Updated by Andreas Nedbal about 7 hours ago

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

Also available in: Atom PDF