Attachment 165153 Details for Bug 36018 – Bug 36018: (follow-up) Disambiguate quantity

[patch] Bug 36018: (follow-up) Disambiguate quantity

Bug-36018-follow-up-Disambiguate-quantity.patch (text/plain), 1.15 KB, created by Marcel de Rooy on 2024-04-19 07:35:14 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Marcel de Rooy

Created: 2024-04-19 07:35:14 UTC

Size: 1.15 KB

patch

obsolete

>From e166741d1cabd8a0fa4a8d40e8cd056aff4a6d3d Mon Sep 17 00:00:00 2001
>From: Marcel de Rooy <m.de.rooy@rijksmuseum.nl>
>Date: Fri, 19 Apr 2024 07:21:57 +0000
>Subject: [PATCH] Bug 36018: (follow-up) Disambiguate quantity
>Content-Type: text/plain; charset=utf-8
>
>One of the embeds refers to suggestions. This table also has
>a quantity column.
>
>Test plan:
>Verify that Pending Orders table does load now instead of
>triggering an internal server error on ambiguous column.
>
>Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl>
>---
> Koha/Acquisition/Orders.pm | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
>diff --git a/Koha/Acquisition/Orders.pm b/Koha/Acquisition/Orders.pm
>index 43e0832277..a71b1f2e0d 100644
>--- a/Koha/Acquisition/Orders.pm
>+++ b/Koha/Acquisition/Orders.pm
>@@ -166,7 +166,7 @@ sub filter_by_active {
>                   'orderstatus' => [ 'new', 'ordered', 'partial' ] },
>                 { 'orderstatus' => [ 'ordered', 'partial' ] }
>             ],
>-            quantityreceived => { '<', \['COALESCE(quantity,0)'] },
>+            quantityreceived => { '<', \['COALESCE(me.quantity,0)'] },
>         },
>         { join => 'basket' }
>     );
>-- 
>2.30.2

Actions: View | Diff | Splinter Review

Attachments on bug 36018: 162318 | 162320 | 162321 | 163769 | 163770 | 163771 | 163772 | 164953 | 164954 | 164955 | 164956 | 165153 | 166503 | 166504 | 166505 | 166506 | 166507 | 166677