<table><tr><td style="">wreissenberger created this revision.<br />wreissenberger added reviewers: mutlaqja, TallFurryMan.<br />Herald added a project: KDE Edu.<br />Herald added a subscriber: kde-edu.<br />wreissenberger requested review of this revision.
</td><a style="text-decoration: none; padding: 4px 8px; margin: 0 8px 8px; float: right; color: #464C5C; font-weight: bold; border-radius: 3px; background-color: #F7F7F9; background-image: linear-gradient(to bottom,#fff,#f1f0f1); display: inline-block; border: 1px solid rgba(71,87,120,.2);" href="https://phabricator.kde.org/D19840">View Revision</a></tr></table><br /><div><strong>REVISION SUMMARY</strong><div><p>Before diff <a href="https://phabricator.kde.org/D19528" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: line-through;">D19528</a>, abort events in the capture module where not always communicated to the scheduler. As a result, an aborted guiding was never restarted and brought the scheduler and the capture module out of sync. The scheduler thought that the capture was running while it already had been aborted.</p>

<p>Unfortunately, the Capture::abort() function is also used for properly terminating a capture session. If a capture session has been completed, stop() is called and then abort(), resulting in an unexpected abort event in the scheduler.</p>

<p>If a scheduler restarts a capture session, this abort event from the previous run leads to the situation, that Capture::start() is called twice, which leads to the situation that the first entry of the capture sequence is skipped.</p></div></div><br /><div><strong>TEST PLAN</strong><div><ul class="remarkup-list">
<li class="remarkup-list-item">Create a capture session with at least two entries.</li>
<li class="remarkup-list-item">Create a schedule that uses this session and runs at least twice.</li>
<li class="remarkup-list-item">Start the schedule and check, whether in the second run of the capture session all session entries are executed properly. Without this diff, the first session entry is skipped except for the first run. With this diff, everything should work properly.</li>
</ul></div></div><br /><div><strong>REPOSITORY</strong><div><div>R321 KStars</div></div></div><br /><div><strong>BRANCH</strong><div><div>bugfix_capture_completion</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D19840">https://phabricator.kde.org/D19840</a></div></div><br /><div><strong>AFFECTED FILES</strong><div><div>kstars/ekos/capture/capture.cpp</div></div></div><br /><div><strong>To: </strong>wreissenberger, mutlaqja, TallFurryMan<br /><strong>Cc: </strong>kde-edu, narvaez, apol<br /></div>