Mark scheduler job dirty when succesfully changing its target.
ClosedPublic

Authored by TallFurryMan on Aug 3 2018, 8:35 PM.

Details

Summary

When editing a scheduler job, call setDirty when target is selected from the pop-up list.
The target and RA/DEC boxes were only triggering the job update when editing the text and leaving the field.
When the pop-up list was used, the programmatic update of those fields wasn't triggering that update.

This fixes the use case where the end-user is looking for a target to start in ASAP mode.
In that use case, the end-user will select targets and observe the starting date/time the Scheduler computed.

Test Plan

Create a Scheduler job.
Click its line in the job table once to edit it, and click the search button.
Choose another target, the target and RA/DEC fields reflect the update.
Click the add/save-job button.
Prior to this fix, the job would not be updated in the queue table.
With this fix, the job is properly updated.

Diff Detail

Repository
R321 KStars
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.
TallFurryMan created this revision.Aug 3 2018, 8:35 PM
Restricted Application added a project: KDE Edu. · View Herald TranscriptAug 3 2018, 8:35 PM
Restricted Application added a subscriber: kde-edu. · View Herald Transcript
TallFurryMan requested review of this revision.Aug 3 2018, 8:35 PM
mutlaqja accepted this revision.Aug 3 2018, 9:08 PM
This revision is now accepted and ready to land.Aug 3 2018, 9:08 PM
This revision was automatically updated to reflect the committed changes.