Model editor bindings: Clean up assignment of priorities
IModelEditorBinding::getPriority()
returns the priority of the editor, where values have lower priority.
While ModelEditorBindingBase
provides a default implementation that returns 0, most concrete bindings override this method (and return 0, too). When there are more than editor of a specific model element, some of the implementations provide higher values (e.g., 10).
In this ticket, the assignment of priority values should be cleaned up:
- Remove unneeded code
- Avoid magic constants (without breaking modularity)
A possible approach could be the definition of constants / an enumn in IModelEditorBinding
, e.g. PRIO_1, PRIO_2, PRIO_3, ... PRIO_10 that prevents subclasses to directly use integer constants.