Uploaded image for project: 'eZ Publish / Platform'
  1. eZ Publish / Platform
  2. EZP-20540

(Public API) Moving a folder to its subtree doesnt raise an exception

    Details

      Description

      Steps to reproduce:
      1 - Create "Folder1" in "eZ Publish"
      2 - Create "Folder1.1" in "eZ Publish/Folder1"
      3 - Move subtree "Folder1" to "eZ Publish/Folder1/Folder1.1"

      And this is allowed.
      We lose the "Folder1" on "eZ Publish" root.

        Activity

        Hide
        Jérôme Vieilledent (Inactive) added a comment -

        Marcos Loureiro: Why is this a blocker ? Can you post somewhere your script please (like in a gist) ? Thanks

        Show
        Jérôme Vieilledent (Inactive) added a comment - Marcos Loureiro : Why is this a blocker ? Can you post somewhere your script please (like in a gist) ? Thanks
        Hide
        Marcos Loureiro (Inactive) added a comment -

        Here is a gist link: https://gist.github.com/MLoureiro/40d411cbdf6d013eafa0
        After the move all the subtree will have a weird behavior, and the main folder wont be there, so only possible to access the subtree through the URL using the node IDs.

        Show
        Marcos Loureiro (Inactive) added a comment - Here is a gist link: https://gist.github.com/MLoureiro/40d411cbdf6d013eafa0 After the move all the subtree will have a weird behavior, and the main folder wont be there, so only possible to access the subtree through the URL using the node IDs.
        Hide
        Jérôme Vieilledent (Inactive) added a comment -

        OK Thanks for the link.
        However I'm still wondering why this is a blocker...

        Show
        Jérôme Vieilledent (Inactive) added a comment - OK Thanks for the link. However I'm still wondering why this is a blocker...
        Hide
        Vidar Langseid added a comment -

        Marked as blocker because user actually looses content here...

        Show
        Vidar Langseid added a comment - Marked as blocker because user actually looses content here...
        Show
        Petar Spanja (Inactive) added a comment - Pull request: https://github.com/ezsystems/ezpublish-kernel/pull/279
        Show
        Petar Spanja (Inactive) added a comment - Merged in https://github.com/ezsystems/ezpublish-kernel/commit/7e53450fc668d44461853280d6bd2ac623bcf4a2
        Hide
        Marcos Loureiro (Inactive) added a comment -

        QA Approved

        Show
        Marcos Loureiro (Inactive) added a comment - QA Approved

          People

          • Assignee:
            Unassigned
            Reporter:
            Marcos Loureiro (Inactive)
          • Votes:
            0 Vote for this issue
            Watchers:
            4 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Time Tracking

              Estimated:
              Original Estimate - Not Specified
              Not Specified
              Remaining:
              Remaining Estimate - 0 minutes
              0m
              Logged:
              Time Spent - 1 hour, 50 minutes
              1h 50m