Attachment 174917 Details for Bug 38515 – Bug 38515: Keep serial notes when generating next serial.

[patch] Bug 38515: Keep serial notes when generating next serial.

Bug-38515-Keep-serial-notes-when-generating-next-s.patch (text/plain), 1.67 KB, created by Adolfo Rodríguez Taboada on 2024-11-22 09:48:36 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Adolfo Rodríguez Taboada

Created: 2024-11-22 09:48:36 UTC

Size: 1.67 KB

patch

obsolete

>From 7022cba765c6339df88e9ef0dc33f82cc36e7fb4 Mon Sep 17 00:00:00 2001
>From: =?UTF-8?q?Adolfo=20Rodr=C3=ADguez?= <adolfo.rodriguez@xercode.es>
>Date: Fri, 22 Nov 2024 10:47:29 +0100
>Subject: [PATCH] Bug 38515: Keep serial notes when generating next serial.
>
>This patch prevents the loss of notes in serials when generating next serial
>
>Test plan:
>1 Create a subscription, fill the notes field for the serial with Expected status and activate the system preference PreserveSerialNotes
>2 Click Generate next
>3 Check that the notes from the now Late serial are gone and that the new serial doesn't have notes.
>4 Apply patch, restart services
>5 Edit the serial with Expected status to fill the notes field.
>6 Click Generate next
>7 Check that the notes from the now Late serial are kepts and that they have been copied to the new serial.
>---
> serials/serials-collection.pl | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
>diff --git a/serials/serials-collection.pl b/serials/serials-collection.pl
>index 025bde2179..c6651ecef1 100755
>--- a/serials/serials-collection.pl
>+++ b/serials/serials-collection.pl
>@@ -71,7 +71,7 @@ if($op eq 'cud-gennext' && @subscriptionid){
>             my $planneddate = $date_received_today ? dt_from_string : $issue->{planneddate};
>             ModSerialStatus( $issue->{serialid}, $issue->{serialseq},
>                     $planneddate, $issue->{publisheddate},
>-                    $issue->{publisheddatetext}, $status, "", $count_forward );
>+                    $issue->{publisheddatetext}, $status, $issue->{notes}, $count_forward );
>         } else {
>             require C4::Serials::Numberpattern;
>             my $subscription = GetSubscription($subscriptionid);
>-- 
>2.30.2

Actions: View | Diff | Splinter Review

Attachments on bug 38515: 174917 | 175479