Uploaded image for project: 'Magnolia UI'
  1. Magnolia UI
  2. MGNLUI-6459

Rename DefaultJcrNodeOrderHandler to IndexAppendingJcrNodeOrderHandler

    XMLWordPrintable

Details

    • Improvement
    • Resolution: Won't Do
    • Neutral
    • None
    • None
    • None
    • None

    Description

      Current situation

      • DefaultJcrNodeOrderHandler is not communicating intent well. It sounds like it's ordering stuff and not renaming stuff, whereas it actually does rename stuff.

      Desired situation

      • Rename to IndexAppendingJcrNodeOrderHandler and create a NaturalJcrNodeOrderHandler, which does just the moving (ordering), but doesn't touch the names, because in some cases this may have destructive consequences. (e.g. in security app there's a multi field which relies on the names of the multi entries as identifiers).

      Checklists

        Acceptance criteria

        Attachments

          Activity

            People

              Unassigned Unassigned
              sdemocko Šimon Demočko
              Votes:
              0 Vote for this issue
              Watchers:
              1 Start watching this issue

              Dates

                Created:
                Updated:
                Resolved:

                Checklists

                  Task DoD