996935f...
by
Eric Dejouhanet <email address hidden>
Fix sequence boundary index regression.
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.
Summary:
This fixes the very minor creation of an empty folder when preparing a sequence job.
The empty folder is a side-effect of the addition of the sequence prefix to SequenceJob's signature.
It is created by updateSequencePrefix. This differential removes the prefix from the path before creating the destination folder.
Test Plan:
Create a few sequence jobs in the Capture module, with a prefix or with the default empty prefix.
Create a scheduler job in the Scheduler module using that sequence file.
Run the scheduler job, check the storage where capture files were generated.
Without the differential, an empty folder with the same name as the prefix for capture files is created.
With the differential, this empty folder is not created.