Task #59113

Epic #58282: Workspaces Workpackage #2

Story #58284: Bug fixes & Behaviour

Failing cascaded delete action of localized elements

Added by Oliver Hader about 1 year ago. Updated 12 months ago.

Status:Resolved Start date:2014-05-26
Priority:Should have Due date:
Assigned To:Oliver Hader % Done:

100%
Category:Workspaces Spent time: -
Target version:-
TYPO3 Version:6.2 Complexity:medium
PHP Version:5.3 Sprint Focus:

Description

Scenario (live):
  • content element in default language with localization in a different language
Actions (workspace):
  • in list view, delete localized content element
  • in list view, delete content element in default language
Experience:
  • deletion of localized element is discarded

Associated revisions

Revision ab25817a
Added by Oliver Hader 12 months ago

[BUGFIX] Failing cascaded delete action of localized elements

Having a record in live workspace with an accordant localization
leads to the fact that actions to the default language also have
an impact to all localizations.

In a workspace, first deleting the localization, then deleting
the record of the default language, ends up in actually having
reverted the previous deletion of the localization.

The solution is to check for delete placeholders in
DataHandler::deleteL10nOverlayRecords().

Resolves: #59113
Releases: master, 6.2
Change-Id: I888c83a58d591f57bc2ea75a49fa603a56a7f722
Reviewed-on: http://review.typo3.org/31142
Reviewed-by: Oliver Hader <>
Tested-by: Oliver Hader <>

Revision f184ddc1
Added by Oliver Hader 12 months ago

[BUGFIX] Failing cascaded delete action of localized elements

Having a record in live workspace with an accordant localization
leads to the fact that actions to the default language also have
an impact to all localizations.

In a workspace, first deleting the localization, then deleting
the record of the default language, ends up in actually having
reverted the previous deletion of the localization.

The solution is to check for delete placeholders in
DataHandler::deleteL10nOverlayRecords().

Resolves: #59113
Releases: master, 6.2
Change-Id: I888c83a58d591f57bc2ea75a49fa603a56a7f722
Reviewed-on: http://review.typo3.org/32137
Reviewed-by: Oliver Hader <>
Tested-by: Oliver Hader <>

History

#1 Updated by Oliver Hader about 1 year ago

  • Status changed from New to Accepted

#2 Updated by Oliver Hader about 1 year ago

  • Assigned To set to Oliver Hader

#3 Updated by Gerrit Code Review about 1 year ago

  • Status changed from Accepted 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/31142

#4 Updated by Gerrit Code Review about 1 year 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/31142

#5 Updated by Gerrit Code Review 12 months ago

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

#6 Updated by Gerrit Code Review 12 months ago

Patch set 1 for branch TYPO3_6-2 of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at http://review.typo3.org/32137

#7 Updated by Oliver Hader 12 months ago

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

Also available in: Atom PDF