[MGNLUI-6459] Rename DefaultJcrNodeOrderHandler to IndexAppendingJcrNodeOrderHandler Created: 14/Dec/20  Updated: 12/Jan/23  Resolved: 12/Jan/23

Status: Closed
Project: Magnolia UI
Component/s: None
Affects Version/s: None
Fix Version/s: None

Type: Improvement Priority: Neutral
Reporter: Šimon Demočko Assignee: Unassigned
Resolution: Won't Do Votes: 0
Labels: None
Remaining Estimate: Not Specified
Time Spent: Not Specified
Original Estimate: Not Specified

Template:
Acceptance criteria:
Empty
Task DoD:
[ ]* Doc/release notes changes? Comment present?
[ ]* Downstream builds green?
[ ]* Solution information and context easily available?
[ ]* Tests
[ ]* FixVersion filled and not yet released
[ ]  Architecture Decision Record (ADR)

 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).

Generated at Mon Feb 12 09:36:40 CET 2024 using Jira 9.4.2#940002-sha1:46d1a51de284217efdcb32434eab47a99af2938b.