Attachment 99359 Details for Bug 23064 – [18.11.x] Bug 23064: Use Koha::Subscription in ModSubscription

[patch] [18.11.x] Bug 23064: Use Koha::Subscription in ModSubscription

Bug-23064-Use-KohaSubscription-in-ModSubscription.patch (text/plain), 4.87 KB, created by Jonathan Druart on 2020-02-21 09:09:39 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Jonathan Druart

Created: 2020-02-21 09:09:39 UTC

Size: 4.87 KB

patch

obsolete

>From 7f10b876f59f114a4c4ee3f1575e925f3579000e Mon Sep 17 00:00:00 2001
>From: Jonathan Druart <jonathan.druart@bugs.koha-community.org>
>Date: Wed, 20 Nov 2019 12:22:58 +0100
>Subject: [PATCH] Bug 23064: Use Koha::Subscription in ModSubscription
>
>We must use Koha::Subscription instead of raw SQL.
>It will fix issue with default and integer values.
>
>Test plan:
>Edit a subscription and set number of issues = "f"
>Save
>=> Without this patch there is a SQL error in the log:
>  Incorrect integer value: 'f' for column 'numberlength'
>=> With this patch the other changes are effective.
>
>Note: We also could change the type attribute of the input to "number",
>to have a client-side check
>
>Also, the return value of ModSuggestion is never used, so we are safe
>with that.
>
>Signed-off-by: Hayley Mapley <hayleymapley@catalyst.net.nz>
>The <strict_sql_modes> flag must be present and set to 1 in the <config> section of koha-conf.xml
>
>Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>
>---
> C4/Serials.pm | 73 +++++++++++++++++++++++++++++++++++------------------------
> 1 file changed, 44 insertions(+), 29 deletions(-)
>
>diff --git a/C4/Serials.pm b/C4/Serials.pm
>index c7cbd70a91..3f6da866ad 100644
>--- a/C4/Serials.pm
>+++ b/C4/Serials.pm
>@@ -1305,37 +1305,52 @@ sub ModSubscription {
>     $itemtype, $previousitemtype
>     ) = @_;
> 
>-    my $dbh   = C4::Context->dbh;
>-    my $query = "UPDATE subscription
>-        SET librarian=?, branchcode=?, aqbooksellerid=?, cost=?, aqbudgetid=?,
>-            startdate=?, periodicity=?, firstacquidate=?, irregularity=?,
>-            numberpattern=?, locale=?, numberlength=?, weeklength=?, monthlength=?,
>-            lastvalue1=?, innerloop1=?, lastvalue2=?, innerloop2=?,
>-            lastvalue3=?, innerloop3=?, status=?, biblionumber=?,
>-            callnumber=?, notes=?, letter=?, manualhistory=?,
>-            internalnotes=?, serialsadditems=?, staffdisplaycount=?,
>-            opacdisplaycount=?, graceperiod=?, location = ?, enddate=?,
>-            skip_serialseq=?, itemtype=?, previousitemtype=?
>-        WHERE subscriptionid = ?";
>-
>-    my $sth = $dbh->prepare($query);
>-    $sth->execute(
>-        $auser,           $branchcode,     $aqbooksellerid, $cost,
>-        $aqbudgetid,      $startdate,      $periodicity,    $firstacquidate,
>-        $irregularity,    $numberpattern,  $locale,         $numberlength,
>-        $weeklength,      $monthlength,    $lastvalue1,     $innerloop1,
>-        $lastvalue2,      $innerloop2,     $lastvalue3,     $innerloop3,
>-        $status,          $biblionumber,   $callnumber,     $notes,
>-        $letter,          ($manualhistory ? $manualhistory : 0),
>-        $internalnotes, $serialsadditems, $staffdisplaycount, $opacdisplaycount,
>-        $graceperiod,     $location,       $enddate,        $skip_serialseq,
>-        $itemtype,        $previousitemtype,
>-        $subscriptionid
>-    );
>-    my $rows = $sth->rows;
>+    my $subscription = Koha::Subscriptions->find($subscriptionid);
>+    $subscription->set(
>+        {
>+            librarian         => $auser,
>+            branchcode        => $branchcode,
>+            aqbooksellerid    => $aqbooksellerid,
>+            cost              => $cost,
>+            aqbudgetid        => $aqbudgetid,
>+            biblionumber      => $biblionumber,
>+            startdate         => $startdate,
>+            periodicity       => $periodicity,
>+            numberlength      => $numberlength,
>+            weeklength        => $weeklength,
>+            monthlength       => $monthlength,
>+            lastvalue1        => $lastvalue1,
>+            innerloop1        => $innerloop1,
>+            lastvalue2        => $lastvalue2,
>+            innerloop2        => $innerloop2,
>+            lastvalue3        => $lastvalue3,
>+            innerloop3        => $innerloop3,
>+            status            => $status,
>+            notes             => $notes,
>+            letter            => $letter,
>+            firstacquidate    => $firstacquidate,
>+            irregularity      => $irregularity,
>+            numberpattern     => $numberpattern,
>+            locale            => $locale,
>+            callnumber        => $callnumber,
>+            manualhistory     => $manualhistory,
>+            internalnotes     => $internalnotes,
>+            serialsadditems   => $serialsadditems,
>+            staffdisplaycount => $staffdisplaycount,
>+            opacdisplaycount  => $opacdisplaycount,
>+            graceperiod       => $graceperiod,
>+            location          => $location,
>+            enddate           => $enddate,
>+            skip_serialseq    => $skip_serialseq,
>+            itemtype          => $itemtype,
>+            previousitemtype  => $previousitemtype,
>+        }
>+    )->store;
> 
>     logaction( "SERIAL", "MODIFY", $subscriptionid, "" ) if C4::Context->preference("SubscriptionLog");
>-    return $rows;
>+
>+    $subscription->discard_changes;
>+    return $subscription;
> }
> 
> =head2 NewSubscription
>-- 
>2.11.0

Actions: View | Diff | Splinter Review

Attachments on bug 23064: 90386 | 90387 | 95611 | 97753 | 97754 | 99359