[Koha-bugs] [Bug 14711] Make C4::Reserves::AddReserves take a hashref in parameters

bugzilla-daemon at bugs.koha-community.org bugzilla-daemon at bugs.koha-community.org
Tue Feb 11 15:03:14 CET 2020


https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14711

Jonathan Druart <jonathan.druart at bugs.koha-community.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
  Attachment #77940|0                           |1
        is obsolete|                            |

--- Comment #8 from Jonathan Druart <jonathan.druart at bugs.koha-community.org> ---
Created attachment 98708
  -->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=98708&action=edit
Bug 14711: Change prototype for AddReserve - pass a hashref

The number of parameters of AddReserve makes it hard to read and
maintain.
This patch replace it with a hashref, which will make the calls more
readable.

Moreover the bibitems has been removed as it was not used by the
subroutine.

Test plan:
- Make sure the tests pass
- Read the diff and search for typos
- Place a hold on few items

Note for QA: reservation_date and expiration_date do not match the DB column's
names,
should we?

-- 
You are receiving this mail because:
You are watching all bug changes.


More information about the Koha-bugs mailing list