Attachment 96135 Details for Bug 24191 – Bug 24191: Make objects.search pass to_model to dbic_merge_sorting

[patch] Bug 24191: Make objects.search pass to_model to dbic_merge_sorting

Bug-24191-Make-objectssearch-pass-tomodel-to-dbicm.patch (text/plain), 1.51 KB, created by Josef Moravec on 2019-12-10 08:07:56 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Josef Moravec

Created: 2019-12-10 08:07:56 UTC

Size: 1.51 KB

patch

obsolete

>From b0fc67dac06530178e16b04143eb939a2d209d44 Mon Sep 17 00:00:00 2001
>From: Tomas Cohen Arazi <tomascohen@theke.io>
>Date: Mon, 9 Dec 2019 15:31:36 -0300
>Subject: [PATCH] Bug 24191: Make objects.search pass to_model to
> dbic_merge_sorting
>
>This patch makes objects.search pass the to_mode method reference so it
>can be used to map API attributes to column names when building the
>sorting portion of the query.
>
>To test:
>1. Apply the regression tests on this patchset
>2. Run:
>  $ kshell
> k$ prove t/db_dependent/Koha/REST/Plugin/Objects.t
>=> FAIL: Tests fail!
>3. Apply the rest of the patchset
>4. Run:
> k$ prove t/Koha/REST/Plugin/Query.t
>=> SUCCESS: Tests pass! changes to the methods work as expected!
>5. Run:
> k$ prove t/db_dependent/Koha/REST/Plugin/Objects.t
>=> SUCCESS: Tests pass!
>6. Sign off :-D
>
>Sponsored-by: ByWater Solutions
>
>Signed-off-by: Owen Leonard <oleonard@myacpl.org>
>
>Signed-off-by: Josef Moravec <josef.moravec@gmail.com>
>---
> Koha/REST/Plugin/Objects.pm | 3 ++-
> 1 file changed, 2 insertions(+), 1 deletion(-)
>
>diff --git a/Koha/REST/Plugin/Objects.pm b/Koha/REST/Plugin/Objects.pm
>index 3653f914ef..3d314a784e 100644
>--- a/Koha/REST/Plugin/Objects.pm
>+++ b/Koha/REST/Plugin/Objects.pm
>@@ -60,7 +60,8 @@ sub register {
>             $c->dbic_merge_sorting(
>                 {
>                     attributes => $attributes,
>-                    params     => $reserved_params
>+                    params     => $reserved_params,
>+                    to_model   => $to_model
>                 }
>             );
> 
>-- 
>2.11.0

Actions: View | Diff | Splinter Review

Attachments on bug 24191: 96114 | 96115 | 96116 | 96117 | 96118 | 96119 | 96133 | 96134 | 96135