Uploaded image for project: 'Magnolia DAM Module'
  1. Magnolia DAM Module
  2. MGNLDAM-890

writePermissionRequired is not set in actions as in old assets app

    XMLWordPrintable

Details

    • Bug
    • Resolution: Fixed
    • Neutral
    • 3.0.5
    • 3.0.3
    • None

    Description

      writePermissionRequired is used to check writting permissions in a node. In the old assets app, it disabled all the actions that required write permisions. In the new dam app, not all the actions that requires writting permissions in the node are including it, it leads to actions available when it shouldn't and errors when trying to execute this actions.

      This is the status of the action bar when a read-only permissions assigned to a node in new dam app:

      This is how it used to be in 5.7.*:

      Steps to reproduce (bundle with travel-demo)

      1. As superuser, go to the Security App and configure user eric readonly permissions for Dam > /tours (role: travel-demo-editor).
      2. Login as eric
      3. Go to dam app and click on /tours. See that almost all the actions are available. Choose one asset: Edit the asset > Edit the dialog and save.

      Expected result

      When opening dam app and click on /tours(readonly), actions that require write permissions shouldn't be available. Same when clicking on an asset within /tours

      Current result

      You can select an action that requires writting permissions. An error comes up:

      Workaround

      Decorate the dam app and add writePermissionRequired: true to all the actions that need it.

      Acceptance criteria

      • It is just a matter of configuration, however the dam app should be configured in a correct way as it used to be in the old assets app in 5.7.

      Checklists

        Acceptance criteria

        Attachments

          Issue Links

            Activity

              People

                asiska Adam Siska
                miruela Mercedes Iruela
                Votes:
                0 Vote for this issue
                Watchers:
                4 Start watching this issue

                Dates

                  Created:
                  Updated:
                  Resolved:

                  Checklists

                    Bug DoR
                    Task DoD

                    Time Tracking

                      Estimated:
                      Original Estimate - Not Specified
                      Not Specified
                      Remaining:
                      Remaining Estimate - Not Specified
                      Not Specified
                      Logged:
                      Time Spent - 40m
                      40m