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

Invalid subtree path when assigning role with subtree limitation

    Details

      Description

      When adding a subtree limitation to a role using the "Users and Groups" tab under a role, an invalid subtree path is created missing a trailing /.

      Steps to reproduce:
      • Goto Roles in the admin ui.
      • Click "Editor" under the name column.
      • Select the "Users and Groups using the <Editor> role" tab
      • "Assign to users/groups with subtree limitation"
      • Select subtree and confirm selection.
      • Select "Editors" user group and confirm selection.
      • ( due to cache, Navigate to "Admin/roles" navigation item and back to "Users and Groups using the <Editor> role" tab.)

      You will see a new limitation beside editors "Subtree( /1/2 )" that is missing the trailing slash. Which will throw exceptions in the Subtree criterion constructor.

      Video: https://youtu.be/gNcmraYiKf4

        Issue Links

          Activity

          Hide
          Yannick Roger (Inactive) added a comment -

          Fix on the PlatformUI: https://github.com/ezsystems/PlatformUIBundle/pull/619
          Needed for the "Strict master fix"

          Show
          Yannick Roger (Inactive) added a comment - Fix on the PlatformUI: https://github.com/ezsystems/PlatformUIBundle/pull/619 Needed for the "Strict master fix"
          Hide
          Yannick Roger (Inactive) added a comment - - edited

          Testing this issue you will run on EZP-25073. Please do not report a duplicate

          Show
          Yannick Roger (Inactive) added a comment - - edited Testing this issue you will run on EZP-25073 . Please do not report a duplicate
          Show
          Yannick Roger (Inactive) added a comment - PlatformUI Fix: https://github.com/ezsystems/PlatformUIBundle/commit/b1030fe5041215c641899f220373c307f25db9b4
          Show
          Gunnstein Lye added a comment - https://github.com/ezsystems/ezpublish-kernel/pull/1671 merged to master (6.4.0). https://github.com/ezsystems/ezpublish-kernel/commit/a2084b51d4a0f359059eb0e2176033325d7555a0
          Hide
          Rui Silva (Inactive) added a comment -

          Tested and approved by QA for 6.3 and master.

          Show
          Rui Silva (Inactive) added a comment - Tested and approved by QA for 6.3 and master.

            People

            • Assignee:
              Unassigned
              Reporter:
              Douglas Hammond
            • Votes:
              0 Vote for this issue
              Watchers:
              7 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 - 6 hours, 5 minutes
                6h 5m