D16173: Fix regression on job deletion when sorting jobs based on alitude and priority is enabled.
noreply at phabricator.kde.org
Sat Oct 13 09:00:10 BST 2018
TallFurryMan created this revision.
TallFurryMan added reviewers: mutlaqja, wreissenberger.
Herald added a project: KDE Edu.
Herald added a subscriber: kde-edu.
TallFurryMan requested review of this revision.
Refactoring job manipulation did include both arrows for job reordering and button for job deletion.
When using that refactored function to prevent reordering of jobs sorted by altitude and priority, the job deletion was mistakenly impacted too.
This differential fixes this issue.
Disable Ekos option "Sort Jobs by Altitude and Priority", and load or create a schedule.
Observe as jobs can be reordered and deleted without issue.
Enable Ekos option "Sort Jobs by Altitude and Priority".
BUG: Observe as the button bar does not change when saving settings, but updates as soon as a job is clicked.
Without this differential, clicking on a job disables the reorder buttons and the delete button.
With this differential, clicking on a job disables the reorder buttons, but not the delete button.
Check that starting the scheduler still disables both reorder and delete buttons, and that stopping resets them based on the sort option.
Check that both reorder and delete buttons are disabled when there is no job in the list.
Check that the reorder buttons are disabled, but the delete button is kept enabled when there is one single job in the list.
Note: this will need a signal to indicate options were updated, so that UI may react to changes immediately.
bugfix__regression_job_deletion (branched from master)
To: TallFurryMan, mutlaqja, wreissenberger
Cc: kde-edu, narvaez, apol
-------------- next part --------------
An HTML attachment was scrubbed...
More information about the kde-edu