Attachment 165090 Details for Bug 35646 – Bug 35646: (follow-up) Correct firstaquidate type

[patch] Bug 35646: (follow-up) Correct firstaquidate type

Bug-35646-follow-up-Correct-firstaquidate-type.patch (text/plain), 2.12 KB, created by Nick Clemens (kidclamp) on 2024-04-18 13:12:52 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Nick Clemens (kidclamp)

Created: 2024-04-18 13:12:52 UTC

Size: 2.12 KB

patch

obsolete

>From 6cd0aa9164bc16a14155c1198f2995bc781f5f9b Mon Sep 17 00:00:00 2001
>From: Nick Clemens <nick@bywatersolutions.com>
>Date: Thu, 18 Apr 2024 13:10:16 +0000
>Subject: [PATCH] Bug 35646: (follow-up) Correct firstaquidate type
>
>---
> C4/Serials.pm            | 2 +-
> t/db_dependent/Serials.t | 5 ++---
> 2 files changed, 3 insertions(+), 4 deletions(-)
>
>diff --git a/C4/Serials.pm b/C4/Serials.pm
>index 6211779d43d..dccfa6c1558 100644
>--- a/C4/Serials.pm
>+++ b/C4/Serials.pm
>@@ -956,7 +956,7 @@ sub GetSeq {
>     $newlastvalue3 = _numeration($newlastvalue3, $pattern->{numbering3}, $locale) if ($pattern->{numbering3}); # reset counter if needed.
>     $calculated =~ s/\{Z\}/$newlastvalue3/g;
> 
>-    my $dt = dt_from_string( $subscription->{firstaquidate} );
>+    my $dt = dt_from_string( $subscription->{firstacquidate} );
>     $calculated =~ s/\{Month\}/$dt->month/eg;
>     $calculated =~ s/\{MonthName\}/$dt->month_name/eg;
>     $calculated =~ s/\{Year\}/$dt->year/eg;
>diff --git a/t/db_dependent/Serials.t b/t/db_dependent/Serials.t
>index 8f91362696b..1b748883bbd 100755
>--- a/t/db_dependent/Serials.t
>+++ b/t/db_dependent/Serials.t
>@@ -560,7 +560,7 @@ subtest "test numbering pattern with dates in GetSeq GetNextSeq" => sub {
>         skip_serialseq => 0,
>         irregularity   => '',
>         locale         => 'C',            # locale set to 'C' to ensure we'll have english strings
>-        firstaquidate  => '1970-11-01',
>+        firstacquidate => '1970-11-01',
>     };
>     $pattern = {
>         numberingmethod => '{Year} {Day} {DayName} {Month} {MonthName}',
>@@ -568,7 +568,7 @@ subtest "test numbering pattern with dates in GetSeq GetNextSeq" => sub {
> 
>     my $numbering = GetSeq( $subscription, $pattern );
>     is( $numbering, '1970 1 Sunday 11 November', 'GetSeq correctly calculates numbering from first aqui date' );
>-    $subscription->{firstaquidate} = '2024-02-29';
>+    $subscription->{firstacquidate} = '2024-02-29';
> 
>     $numbering = GetSeq( $subscription, $pattern );
>     is(
>@@ -588,7 +588,6 @@ subtest "test numbering pattern with dates in GetSeq GetNextSeq" => sub {
> 
> };
> 
>-
> subtest "_numeration" => sub {
> 
>     plan tests => 6;
>-- 
>2.39.2

Actions: View | Diff | Splinter Review

Attachments on bug 35646: 160275 | 160276 | 160277 | 160297 | 160298 | 165088 | 165089 | 165090 | 165093 | 165094 | 165095