Task #60249

Epic #58282: Workspaces Workpackage #2

Story #58284: Bug fixes & Behaviour

Publishing delete placeholder affects different workspaces as well

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

Status:Resolved Start date:2014-07-10
Priority:Must have Due date:
Assigned To:Oliver Hader % Done:

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

Description

Scenario:
  • having two workspaces (1 and 2)
  • having a live record with uid 10
  • having a delete placeholder for uid 10 in workspace 1
  • having a delete placeholder for uid 10 in workspace 2
Action:
  • publish the delete placholder for uid 10 in workspace 1
Experience:
  • published delete placeholder for uid 10 in workspace 2 (which was not the active workspace)

Associated revisions

Revision bbbd8146
Added by Oliver Hader 12 months ago

[BUGFIX] Publishing delete placeholder affects other workspaces

On publishing a delete placeholder all other versions shall be
deleted as well, since the original record will be deleted after
that action. However, this is only valid for the live workspace
and using the legacy version behavior there.

DataHandler::deleteVersionsForRecord() calls
BackendUtility::selectVersionsOfRecord(), but without the
additional, but required, $workspace argument. Thus, all other
versions in other workspaces are deleted as well. The SQL query
to determine versions needs to be limited to either the live
workspace (for legacy versions) and the current workspace.

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

Revision 6f945001
Added by Oliver Hader 12 months ago

[BUGFIX] Publishing delete placeholder affects other workspaces

On publishing a delete placeholder all other versions shall be
deleted as well, since the original record will be deleted after
that action. However, this is only valid for the live workspace
and using the legacy version behavior there.

DataHandler::deleteVersionsForRecord() calls
BackendUtility::selectVersionsOfRecord(), but without the
additional, but required, $workspace argument. Thus, all other
versions in other workspaces are deleted as well. The SQL query
to determine versions needs to be limited to either the live
workspace (for legacy versions) and the current workspace.

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

History

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

#2 Updated by Gerrit Code Review 12 months ago

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

#3 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/32139

#4 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