[Options Providers] 2.6.9 - display of sorted options does not respect "preserveOrder" or definition settings #1998

Closed
katanafleet opened this Issue Aug 10, 2016 · 1 comment

Projects

None yet

2 participants

@katanafleet

We were waiting for 2.6.9 options provider improvements thanks a lot.
Unfortunately, it seems that options are not always sorted in defined order that has been set in GUI or XML "preserveOrder" tag ?

We have the issue when there's dependencies between option A => B so B is displayed before A even if we set A before B in the GUI.

Simple options can also be mixed between definition and display ...

cf. job sample zip and attached and screenshots associated.

2016-08-10 14_56_18-rundeck - edit job
2016-08-10 14_53_10-rundeck - prepare
Prepare.zip

@katanafleet katanafleet changed the title from [Options Providers] display of sorted options does not respect "presverOrder" or editing settings to [Options Providers] display of sorted options does not respect "presverOrder" or definition settings Aug 10, 2016
@katanafleet katanafleet changed the title from [Options Providers] display of sorted options does not respect "presverOrder" or definition settings to [Options Providers] display of sorted options does not respect "preserveOrder" or definition settings Aug 10, 2016
@katanafleet katanafleet changed the title from [Options Providers] display of sorted options does not respect "preserveOrder" or definition settings to [Options Providers] 2.6.9 - display of sorted options does not respect "preserveOrder" or definition settings Aug 10, 2016
@gschueler gschueler added this to the 2.6.x milestone Aug 10, 2016
@gschueler
Contributor

Since the GUI manual ordering is now available, I'm going to change the behavior to this:

  • always show options in the order defined

The sorting was intended to automatically put the options in the natural order based on the cascading order, however that was when there was no way to manually order them in the GUI. Now that you can do that it makes more sense to just use that ordering always.

@gschueler gschueler self-assigned this Nov 3, 2016
@gschueler gschueler closed this in 6ac6670 Nov 3, 2016
@gschueler gschueler removed the in progress label Nov 3, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment