Bug 19317

Summary: Move of checkouts - Remove leftover
Product: Koha Reporter: Jonathan Druart <jonathan.druart>
Component: Architecture, internals, and plumbingAssignee: Jonathan Druart <jonathan.druart>
Status: CLOSED FIXED QA Contact: Kyle M Hall <kyle>
Severity: normal    
Priority: P5 - low CC: colin.campbell, fridolin.somers, katrin.fischer, kyle
Version: unspecified   
Hardware: All   
OS: All   
Change sponsored?: --- Patch complexity: ---
Documentation contact: Documentation submission:
Text to go in the release notes:
Version(s) released in:
Bug Depends on: 18966    
Bug Blocks:    
Attachments: Bug 19317: (bug 18966 follow-up) Remove leftover
Signed off removal of redundant line
Bug 19317: (bug 18966 follow-up) Remove leftover

Description Jonathan Druart 2017-09-14 13:57:02 UTC
Nothing important here, but this line should have been removed by bug 18966:

2177         # Update the fines
2178         $dbh->do(q|UPDATE accountlines SET issue_id = ? WHERE issue_id = ?|, undef, $old_checkout->issue_id, $issue->issue_id);

The issue_id is now the same when moved from issues to old_issues. We do not need to update the accountlines table
Comment 1 Jonathan Druart 2017-09-14 14:00:01 UTC
Created attachment 67136 [details] [review]
Bug 19317: (bug 18966 follow-up) Remove leftover

Nothing important here, but this line should have been removed by bug
18966:

2177         # Update the fines
2178         $dbh->do(q|UPDATE accountlines SET issue_id = ? WHERE
issue_id = ?|, undef, $old_checkout->issue_id, $issue->issue_id);

The issue_id is now the same when moved from issues to old_issues. We do
not need to update the accountlines table.

No test plan here, you need to understand previous changes to validate
this patch.
Comment 2 Colin Campbell 2017-09-14 14:52:06 UTC
Created attachment 67139 [details] [review]
Signed off removal of redundant line
Comment 3 Colin Campbell 2017-09-14 14:54:11 UTC
Signed off - not only is it now redundant if you profile checkins it can be a bit wasteful  of resources
Comment 4 Kyle M Hall 2017-09-29 13:26:28 UTC
Created attachment 67450 [details] [review]
Bug 19317: (bug 18966 follow-up) Remove leftover

Nothing important here, but this line should have been removed by bug
18966:

2177         # Update the fines
2178         $dbh->do(q|UPDATE accountlines SET issue_id = ? WHERE
issue_id = ?|, undef, $old_checkout->issue_id, $issue->issue_id);

The issue_id is now the same when moved from issues to old_issues. We do
not need to update the accountlines table.

No test plan here, you need to understand previous changes to validate
this patch.

Signed-off-by: Colin Campbell <colin.campbell@ptfs-europe.com>

Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com>
Comment 5 Jonathan Druart 2017-09-29 20:18:48 UTC
Pushed to master for 17.11, thanks to everybody involved!
Comment 6 Fridolin Somers 2017-10-27 12:03:36 UTC
Pushed to 17.05.x, will be in 17.05.06.
Comment 7 Katrin Fischer 2017-10-29 18:23:00 UTC
The removed lines don't appear to exist in 16.11.x. Closing.