[MGNLUI-2754] Restore previous version including subtree Created: 18/Mar/14  Updated: 11/Nov/16  Resolved: 29/Sep/14

Status: Closed
Project: Magnolia UI
Component/s: pages app
Affects Version/s: None
Fix Version/s: 5.3.4

Type: Improvement Priority: Major
Reporter: Lars Fischer Assignee: Federico Grilli
Resolution: Fixed Votes: 0
Labels: support, ux, uzh
Remaining Estimate: Not Specified
Time Spent: Not Specified
Original Estimate: Not Specified

Issue Links:
Relates
relates to MGNLDAM-515 Provide a way to download a versioned... Closed
dependency
depends upon MAGNOLIA-5933 Create new RestorePreviousCommand Closed
relation
is related to MGNLUI-1619 Create general RestorePreviousVersion... Closed
is related to MGNLDAM-557 Restore including subnodes must be po... Closed
Template:
Acceptance criteria:
Empty
Task DoD:
[ ]* Doc/release notes changes? Comment present?
[ ]* Downstream builds green?
[ ]* Solution information and context easily available?
[ ]* Tests
[ ]* FixVersion filled and not yet released
[ ]  Architecture Decision Record (ADR)
Release notes required:
Yes
Date of First Response:

 Description   

It's possible to delete a page including subtrees with a single command.

To restore this tree you have to restore every single page in the tree. It would be convenient if a user would have the possibility to restore a page including the subtree.

This issue is based on an actual customer request!



 Comments   
Comment by Daniel Lipp [ 25/Sep/14 ]

As discussed the benefit of having this fixed outweighs the "potential" damage of not being 100% backwards compatible. Hence we want to implement as follow:

The action to restore previously deleted trees ALWAYS restores the whole tree (no config option) as a long running action (async command)

Comment by Federico Grilli [ 26/Sep/14 ]

For the release notes:

  • Created a base command-based action at info.magnolia.ui.contentapp.browser.action.RestoreItemPreviousVersionAction with related definition
  • info.magnolia.ui.contentapp.detail.action.RestorePreviousVersionAction and def are deprecated (they are in the wrong package and basically are a duplicate of the same name classes in pages app)
  • RestoreItemPreviousVersionActionDefinition has an parentNodeTypeOnly boolean option which by default is false.
    • When false it will restore all descendants of a parent whose node type is in the mgnl: namespace. This should be the most common use case, e.g. in the dam-assets apps when you delete a folder containing assets.
    • When true, it will restore all descendants whose node type is the same as the parent. This is used, e.g., by the pages app. In that case we can restore only pages and not their areas and components.
  • info.magnolia.pages.app.action.RestorePreviousVersionAction breaks the previous API being now a subclass of info.magnolia.ui.contentapp.browser.action.RestoreItemPreviousVersionAction instead of AbstractAction
Generated at Mon Feb 12 08:59:49 CET 2024 using Jira 9.4.2#940002-sha1:46d1a51de284217efdcb32434eab47a99af2938b.