]> git.koha-community.org Git - koha.git/commit
Bug 24191: Add to_model param to _build_order_atom and dbic_merge_sorting
authorTomas Cohen Arazi <tomascohen@theke.io>
Mon, 9 Dec 2019 17:48:48 +0000 (14:48 -0300)
committerJoy Nelson <joy@bywatersolutions.com>
Fri, 20 Dec 2019 15:45:55 +0000 (15:45 +0000)
commit46895911d4668c343446104df41afb151dd2fae3
tree39cd33a4a7296fa71429074e6601fe37f7210b84
parentf38519b0638f620d0faeebdc2f0e0c74085c5838
Bug 24191: Add to_model param to _build_order_atom and dbic_merge_sorting

This patch adds a to_model parameter to dbic_merge_sorting so it is
passed when used (for example from objects.search). The to_model param
is passed along to the _build_order_atom method where it is finally
used.

In the process I wrote tests that reflected some problems in the current
code:
- Mojolicious automatically returns a scalar if a query parameter only
happens once on a request. The code expected an arrayref in every case.
- There's a design issue that forced me to use some hacky code in
_build_order_atom.

The first issue is dealth with, by using Scalar::Util::reftype as the
Perl docs recommend.

The second issue, I don't plan to clean it here, as there's ongoing work
on a Koha::Objects->search_from_api method that will obsolete this code
most probably (see bug 23893 for a better picture of where the mappings
will be living soon).

To test:
1. Apply this patch
2. Run:
  $ kshell
 k$ prove t/Koha/REST/Plugin/Query.t
=> SUCCESS: Tests pass!!
3. Sign off :-D

Sponsored-by: ByWater Solutions
Signed-off-by: Owen Leonard <oleonard@myacpl.org>
Signed-off-by: Josef Moravec <josef.moravec@gmail.com>
Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>
Signed-off-by: Joy Nelson <joy@bywatersolutions.com>
Koha/REST/Plugin/Query.pm
t/Koha/REST/Plugin/Query.t