When the C4::Circulation::AddReturn reaches the ILL request update, there are many circumstances under which no return happened, and the ILL code doesn't check that.
Created attachment 107644 [details] [review] Bug 26114: Regression tests Signed-off-by: Tomas Cohen Arazi <tomascohen@theke.io>
Created attachment 107645 [details] [review] Bug 26114: ILL should mark status=RET only if a return happened The original code assumed every codepath should be trated as a successful check-in. Such is not the case and the regression tests verify that the code acknowledges this. To test: 1. Apply the regression tests patch 2. Run: $ kshell k$ prove t/db_dependent/Illrequests.t => FAIL: Tests fail 3. Apply this patch 4. Repeat 2 => SUCCESS: Tests pass! 5. Sign off :-D Signed-off-by: Tomas Cohen Arazi <tomascohen@theke.io>
Created attachment 108144 [details] [review] Bug 26114: ILL should mark status=RET only if a return happened The original code assumed every codepath should be trated as a successful check-in. Such is not the case and the regression tests verify that the code acknowledges this. To test: 1. Apply the regression tests patch 2. Run: $ kshell k$ prove t/db_dependent/Illrequests.t => FAIL: Tests fail 3. Apply this patch 4. Repeat 2 => SUCCESS: Tests pass! 5. Sign off :-D Signed-off-by: Tomas Cohen Arazi <tomascohen@theke.io> Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>
Created attachment 108145 [details] [review] Bug 26114: Regression tests Signed-off-by: Tomas Cohen Arazi <tomascohen@theke.io> Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>
Created attachment 108146 [details] [review] Bug 26114: ILL should mark status=RET only if a return happened The original code assumed every codepath should be trated as a successful check-in. Such is not the case and the regression tests verify that the code acknowledges this. To test: 1. Apply the regression tests patch 2. Run: $ kshell k$ prove t/db_dependent/Illrequests.t => FAIL: Tests fail 3. Apply this patch 4. Repeat 2 => SUCCESS: Tests pass! 5. Sign off :-D Signed-off-by: Tomas Cohen Arazi <tomascohen@theke.io> Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>
Good catch, works as expected. Signing off.
Going for QA on this.. it's well tested and passes QA scripts.. trivial change. Passing QA
Pushed to master for 20.11, thanks to everybody involved!
backported to 20.05.x for 20.05.03
missing dependencies, not backported to 19.11.x