D14875: Fix sequence boundary index regression.

[Eric Dejouhanet]

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.

REVISION 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.

REPOSITORY
  R321 KStars

BRANCH
  bugfix__sequence_boundary_regression (branched from master)

REVISION DETAIL
  https://phabricator.kde.org/D14875

AFFECTED FILES
  kstars/ekos/capture/capture.cpp

To: TallFurryMan, mutlaqja, wreissenberger
Cc: kde-edu, narvaez, apol
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-edu/attachments/20180816/c0f5276b/attachment.html>