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

"Show Version" in Assets app allows to edit and throws error

    XMLWordPrintable

Details

    Description

      If you click on "Show Version" in the Assets app, it is possible to edit it but then an error is thrown when trying to save.

      Steps to reproduce

      1. Go to the Assets app, edit an asset and publish it
      2. Now you have the option to see the versions, select the asset, and click on "Show versions"
      3. Notice you are able to edit the version
      4. Click on save and an error is thrown (show versions save)

      In our docs, we mention that when selecting the "Show version" action it is not possible to edit. If you check other apps, such as the pages app, and click on "Show version" you can see a read-only version of the page. You are not allowed to edit.

      Furthermore, in Magnolia 5.7, it was not possible to edit the version, you could restore it if necessary or even download it in the case of the assets app.

      Expected results

      • Based on our docs and how the versioning usually works, it should not be possible to edit and save a version of an asset. We should instead see a read-only version of the asset.
      • If allowing our customers to edit the version of the asset is intended, then the error should be fixed.

      
Actual results
      It is possible to edit but an error is thrown when saving

       

      Options alternative

      Global solution

      • custom read-only detail view (extend default java class for detail view with all fields disabled)

      Local solution (dropped for a global one)

      • create a view for the asset version with read-only fields 

      Checklists

        Acceptance criteria

        Attachments

          Issue Links

            Activity

              People

                chuong.doan Chuong Doan Huy
                mdamborenea Monica Damborenea
                Votes:
                0 Vote for this issue
                Watchers:
                5 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 - 2d 50m
                      2d 50m