Attachment 89676 Details for Bug 11492 – Bug 11492: (follow-up) Don't pass publisheddatetext param to NewIssue

[patch] Bug 11492: (follow-up) Don't pass publisheddatetext param to NewIssue

Bug-11492-follow-up-Dont-pass-publisheddatetext-pa.patch (text/plain), 2.19 KB, created by Liz Rea on 2019-05-13 17:31:50 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Liz Rea

Created: 2019-05-13 17:31:50 UTC

Size: 2.19 KB

patch

obsolete

>From f638d19171511cb0fc291ff07fe5027c274af359 Mon Sep 17 00:00:00 2001
>From: Aleisha Amohia <aleishaamohia@hotmail.com>
>Date: Fri, 14 Dec 2018 02:05:36 +0000
>Subject: [PATCH] Bug 11492: (follow-up) Don't pass publisheddatetext param to
> NewIssue
>
>I have removed publisheddatetext as a param from my patch, but only in
>the places where my patch added it. Any other instance already existed
>before my patch.
>
>Signed-off-by: Liz Rea <wizzyrea@gmail.com>
>---
> C4/Serials.pm                 | 2 +-
> serials/serials-collection.pl | 2 +-
> 2 files changed, 2 insertions(+), 2 deletions(-)
>
>diff --git a/C4/Serials.pm b/C4/Serials.pm
>index d928c73718..07111282a3 100644
>--- a/C4/Serials.pm
>+++ b/C4/Serials.pm
>@@ -1165,7 +1165,7 @@ sub ModSerialStatus {
>                     WHERE  subscriptionid = ?";
>         $sth = $dbh->prepare($query);
>         $sth->execute( $newlastvalue1, $newlastvalue2, $newlastvalue3, $newinnerloop1, $newinnerloop2, $newinnerloop3, $subscriptionid );
>-        NewIssue( $newserialseq, $subscriptionid, $subscription->{'biblionumber'}, 1, $nextpubdate, $nextpubdate, $publisheddatetext, $notes, $routingnotes );
>+        NewIssue( $newserialseq, $subscriptionid, $subscription->{'biblionumber'}, 1, $nextpubdate, $nextpubdate, undef, $notes, $routingnotes );
>         # check if an alert must be sent... (= a letter is defined & status became "arrived"
>         if ( $subscription->{letter} && $status == ARRIVED && $oldstatus != ARRIVED ) {
>             require C4::Letters;
>diff --git a/serials/serials-collection.pl b/serials/serials-collection.pl
>index 18a46b5ea1..21e6463bbf 100755
>--- a/serials/serials-collection.pl
>+++ b/serials/serials-collection.pl
>@@ -90,7 +90,7 @@ if($op eq 'gennext' && @subscriptionid){
>              my $planneddate = $date_received_today ? dt_from_string : $nextpublisheddate;
>              ## Creating the new issue
>              NewIssue( $newserialseq, $subscriptionid, $subscription->{'biblionumber'},
>-                     1, $planneddate, $nextpublisheddate, $issue->{publisheddatetext},
>+                     1, $planneddate, $nextpublisheddate, undef,
>                      $issue->{notes}, $issue->{routingnotes} );
> 
>              ## Updating the subscription seq status
>-- 
>2.11.0

Actions: View | Diff | Splinter Review

Attachments on bug 11492: 82945 | 82946 | 83036 | 83192 | 89675 | 89676 | 90214 | 90215