Fix sequence boundary index regression.
ClosedPublic

Authored by TallFurryMan on Aug 16 2018, 6:48 AM.

Details

Summary

This differential fixes the regression on sequence boundary indices.
Following the signature change fixing the issue with sequence prefixes, the snippet in the Scheduler module was updated, but there was a duplicate in the Capture module, which was not.

The fixing code was duplicated so that it is easier to spot for refactoring later. Variable "sig_file" is unneeded by the way.

Test Plan

Create a sequence with several captures, with prefix defined and with prefix left to default.
Without this differential, all capture files get index 001. This goes unnoticed if the sequence job has "TS" checked, because all files will have a different naming.
With this differential, capture files get indexed properly based on the number of files and sequence number in the storage folder.

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 16 2018, 6:48 AM
Restricted Application added a project: KDE Edu. · View Herald TranscriptAug 16 2018, 6:48 AM
Restricted Application added a subscriber: kde-edu. · View Herald Transcript
TallFurryMan requested review of this revision.Aug 16 2018, 6:48 AM
mutlaqja accepted this revision.Aug 16 2018, 7:37 AM
This revision is now accepted and ready to land.Aug 16 2018, 7:37 AM
This revision was automatically updated to reflect the committed changes.