Attachment 159290 Details for Bug 35418 – Bug 35418: Fix _get_usage_months

[patch] Bug 35418: Fix _get_usage_months

Bug-35418-Fix-getusagemonths.patch (text/plain), 1.22 KB, created by Pedro Amorim on 2023-11-28 13:28:06 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Pedro Amorim

Created: 2023-11-28 13:28:06 UTC

Size: 1.22 KB

patch

obsolete

>From 02fcd7d057a0f656a1a82770cea6a8ca59c2deee Mon Sep 17 00:00:00 2001
>From: Pedro Amorim <pedro.amorim@ptfs-europe.com>
>Date: Tue, 28 Nov 2023 13:24:49 +0000
>Subject: [PATCH] Bug 35418: Fix _get_usage_months
>
>No longer falls in an endless loop if the last month in the report period is december.
>
>Test plan:
>1) Apply just the tests patch, run EUsage tests
>prove t/db_dependent/Koha/ERM/EUsage/
>2) Wait for a few seconds. Notice SushiCounter.t it hangs. CTRL+C (cancel it)
>3) Apply the fix patch.
>4) Run the tests again. Notice all EUsage tests pass with no issues.
>---
> Koha/ERM/EUsage/SushiCounter.pm | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
>diff --git a/Koha/ERM/EUsage/SushiCounter.pm b/Koha/ERM/EUsage/SushiCounter.pm
>index d128b868b7..4e33f8988f 100644
>--- a/Koha/ERM/EUsage/SushiCounter.pm
>+++ b/Koha/ERM/EUsage/SushiCounter.pm
>@@ -681,7 +681,7 @@ sub _get_usage_months {
>     my $end_year  = substr( $end_date[0], 0, 4 );
> 
>     my @month_headings = ();
>-    while ( $begin_month <= $end_month || $begin_year < $end_year ) {
>+    while ( $begin_year <= $end_year && ( $begin_year != $end_year || $begin_month <= $end_month ) ) {
>         push(
>             @month_headings,
>             $column_headings_formatting
>-- 
>2.30.2

Actions: View | Diff | Splinter Review

Attachments on bug 35418: 159289 | 159290 | 159323 | 159324 | 159347 | 159348 | 159373 | 159374