Bug 28744: Better handling of undefined to_api_mapping
authorTomas Cohen Arazi <tomascohen@theke.io>
Fri, 23 Jul 2021 03:39:16 +0000 (00:39 -0300)
committerJonathan Druart <jonathan.druart@bugs.koha-community.org>
Mon, 30 Aug 2021 15:02:07 +0000 (17:02 +0200)
commitd41b197ff6f5e7b80cb47f37375568536b045bf5
tree3ca4fa53510939d80283ab1f43757831dd9bbbde
parentbcc586ef93eea0258cf1032a8deca3c7ade6af11
Bug 28744: Better handling of undefined to_api_mapping

We always had some mapping because of the DB vs. API object definition discrepancies. But bug 25260 revamps the reserves table, and it is a perfect match. It highlights this edge case: if no mapping defined, then and undef from_api_mapping is generated (as opposed to an empty hashref) and this leads to errors in the query translation from the API.

This patch makes a small change so this method always returns an empty
hashref.

To test:
1. Apply the regression tests
2. Run:
   $ kshell
  k$ prove t/db_dependent/Koha/Object.t
=> FAIL: Test 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: David Nind <david@davidnind.com>
Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl>
JD amended patch: remove uneeded commented lines

Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org>
Koha/Object.pm