Bug 18651: [QA Follow-up] Fix the MAX(issue_id) calculation
authorMarcel de Rooy <m.de.rooy@rijksmuseum.nl>
Tue, 20 Jun 2017 13:03:35 +0000 (15:03 +0200)
committerJonathan Druart <jonathan.druart@bugs.koha-community.org>
Tue, 20 Jun 2017 17:29:22 +0000 (14:29 -0300)
commitee7969d3461e6ce08b4fbfd982c90f263f19f693
treed886e9ece389e62f3d076f6d09053f8590da8528
parent4906d58f7c507613c9ad20c4524afd278fca467c
Bug 18651: [QA Follow-up] Fix the MAX(issue_id) calculation

Found this by inserting the same issue_id in old_issues before checkin:
The call to ->search( )->get_column is in scalar context and will
return the number of results, i.e. always 1.
If you have an issue_id 2 in old_issues, it will crash:
    DBIx::Class::Storage::DBI::_dbh_execute(): Duplicate entry '2' for key 'PRIMARY'

The fix is fairly simple: Put get_column in list context and pick the first
array entry.
NOTE: Using DBIx's get_column()->max here might look simpler here.

Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl>
C4/Circulation.pm