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

Changing Class's attribute order mixes attributes when we use a already existing order number

    Details

      Description

      Hi,
      When we try to change the order number of an attribute, to a order number that already exists, we have a duplicate information (this is correct) and then the all attributes became mixed.

      Example to reproduce:

      - Edit the "Article" class
      we have the following attributes order, including order number:
      Title - 10
      Short Title - 20
      Author - 30
      Intro - 40
      Body - 50
      Enable comments - 60
      image - 70
       
      if now we try to set "Enable comments=70" (repeating the "image" order number) and then press "OK" button, we have the information that the class was not stored because we have a duplicate attribute placement (this is correct).
       
      The problem is that order now is completely different than the one we started with:
      We now have
       
      Title - 10
      Short Title - 20
      Author - 30
      Enable comments - 40 (was 60)
      image - 50 (was 70)
      Intro - 60 (was 40)
      Body - 70 (was 50)
       
      and if we, by mistake, press the "OK" button again, this new order is stored.
      

        Issue Links

          Activity

          Show
          Patrick Allaert (Inactive) added a comment - https://github.com/ezsystems/ezpublish/pull/478
          Show
          Patrick Allaert (Inactive) added a comment - Fixed in: eZ Publish master (merge): https://github.com/ezsystems/ezpublish/commit/069384c61476ca745e5953822b8946147f8d0af0 eZ Publish master: https://github.com/ezsystems/ezpublish/commit/7954e1af22d767d14e30af8c7ef4b0ce053539a7
          Hide
          Patrick Allaert (Inactive) added a comment -

          Cherry picking and resolving the conflicts.

          Show
          Patrick Allaert (Inactive) added a comment - Cherry picking and resolving the conflicts.
          Hide
          Paulo Nunes (Inactive) added a comment -

          QA Approved

          Show
          Paulo Nunes (Inactive) added a comment - QA Approved

            People

            • Assignee:
              Unassigned
              Reporter:
              Paulo Nunes (Inactive)
            • Votes:
              0 Vote for this issue
              Watchers:
              3 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 day, 3 hours, 55 minutes
                1d 3h 55m