September 2020 - Koha-bugs - lists.koha-community.org

[Bug 23295] New: Automatically debar patrons if SMS or email notice fail
by bugzilla-daemon＠bugs.koha-community.org 29 Jul '25

29 Jul '25

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=23295 Bug ID: 23295 Summary: Automatically debar patrons if SMS or email notice fail Change sponsored?: --- Product: Koha Version: master Hardware: All OS: All Status: NEW Severity: enhancement Priority: P5 - low Component: Notices Assignee: koha-bugs(a)lists.koha-community.org Reporter: alexbuckley(a)catalyst.net.nz QA Contact: testopia(a)bugs.koha-community.org When the sending of an SMS (or email) notice fails at the Koha level (i.e. before it is passed to the SMS API) then librarians should have the ability to enable automatic restriction (debarment) of patrons prompting them to update their smsalertnumber (or email address) in their library accounts. Adding this ability will tie in nicely with the functionality implemented in bug 13552 and bug 23291. As the enhancement for this bug report would let us restrict when notices failed at the Koha level, whereas 13552 and 23291 would let librarians restrict when notices failed at the SMS API level in batch restriction jobs retrieving patrons by SMS numbers in the SMS gateway report libraries get telling them which notices failed at the SMS API level. So all three enhancements together will mean there will be end-to-end functionality to restrict patrons when SMS notices fail, thereby prompting them to update their smsalertnumber. -- You are receiving this mail because: You are the assignee for the bug. You are watching all bug changes.

1 71

[Bug 23493] New: jquery.dataTables.rowGrouping.js is no longer maintained, but there is an official DataTables version we could switch to
by bugzilla-daemon＠bugs.koha-community.org 28 Jul '25

28 Jul '25

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=23493 Bug ID: 23493 Summary: jquery.dataTables.rowGrouping.js is no longer maintained, but there is an official DataTables version we could switch to Change sponsored?: --- Product: Koha Version: master Hardware: All OS: All Status: NEW Severity: enhancement Priority: P5 - low Component: Templates Assignee: oleonard(a)myacpl.org Reporter: martin.renvoize(a)ptfs-europe.com QA Contact: testopia(a)bugs.koha-community.org We currently sparsely use this plugin on the checkouts table. It would be nice to switch to the equivalent plugin by the datatables authors and bundle it into our package. Doing so would allow other developers to take advantage of the feature as that version is well documented and supported. -- You are receiving this mail because: You are watching all bug changes.

1 45

[Bug 18211] New: Circulation.pl / Moremember.pl has errors in table row id
by bugzilla-daemon＠bugs.koha-community.org 28 Jul '25

28 Jul '25

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18211 Bug ID: 18211 Summary: Circulation.pl / Moremember.pl has errors in table row id Change sponsored?: --- Product: Koha Version: master Hardware: All OS: All Status: NEW Severity: minor Priority: P5 - low Component: Circulation Assignee: koha-bugs(a)lists.koha-community.org Reporter: cbrannon(a)cdalibrary.org QA Contact: testopia(a)bugs.koha-community.org CC: gmcharlt(a)gmail.com, kyle.m.hall(a)gmail.com The table row showing the label "Today's checkouts" shows the id of "group-id-issues-table_-strong-today-s-checkouts-strong-" The id is inaccurate and is taking on values from a child cell. -- You are receiving this mail because: You are the assignee for the bug. You are watching all bug changes.

1 20

[Bug 20601] New: Untranslatable strings in circulation statistics
by bugzilla-daemon＠bugs.koha-community.org 28 Jul '25

28 Jul '25

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=20601 Bug ID: 20601 Summary: Untranslatable strings in circulation statistics Change sponsored?: --- Product: Koha Version: master Hardware: All OS: All Status: NEW Severity: minor Priority: P5 - low Component: I18N/L10N Assignee: koha-bugs(a)lists.koha-community.org Reporter: pasi.kallinen(a)joensuu.fi QA Contact: testopia(a)bugs.koha-community.org CC: frederic(a)tamil.fr Untranslatable strings: reports/issues_stats.pl: push @loopfilter, { crit => "Event", filter => $type }; push @loopfilter, { crit => "Display by", filter => $dsp } if ($dsp); push @loopfilter, { crit => "Select Day", filter => $daysel } if ($daysel); push @loopfilter, { crit => "Select Month", filter => $monthsel } if ($monthsel); -- You are receiving this mail because: You are watching all bug changes. You are the assignee for the bug.

1 14

[Bug 23794] New: Convert OpacMainUserBlock system preference to news block
by bugzilla-daemon＠bugs.koha-community.org 24 Jul '25

24 Jul '25

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=23794 Bug ID: 23794 Summary: Convert OpacMainUserBlock system preference to news block Change sponsored?: --- Product: Koha Version: master Hardware: All OS: All Status: ASSIGNED Severity: enhancement Priority: P5 - low Component: OPAC Assignee: oleonard(a)myacpl.org Reporter: oleonard(a)myacpl.org QA Contact: testopia(a)bugs.koha-community.org CC: hayleymapley(a)catalyst.net.nz, jonathan.druart(a)bugs.koha-community.org, katrin.fischer(a)bsz-bw.de, lucas(a)bywatersolutions.com, m.de.rooy(a)rijksmuseum.nl, testopia(a)bugs.koha-community.org Depends on: 22318 Building on 22318, the OpacMainUserBlock system preference should be converted to a Koha news block. Referenced Bugs: https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=22318 [Bug 22318] Extend Koha news feature to include other content areas -- You are receiving this mail because: You are watching all bug changes.

1 22

[Bug 24403] New: [OMNIBUS] OpacHiddenItems should hide items everywhere in the OPAC
by bugzilla-daemon＠bugs.koha-community.org 24 Jul '25

24 Jul '25

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24403 Bug ID: 24403 Summary: [OMNIBUS] OpacHiddenItems should hide items everywhere in the OPAC Change sponsored?: --- Product: Koha Version: master Hardware: All OS: All Status: NEW Severity: enhancement Priority: P5 - low Component: OPAC Assignee: oleonard(a)myacpl.org Reporter: katrin.fischer(a)bsz-bw.de QA Contact: testopia(a)bugs.koha-community.org -- You are receiving this mail because: You are watching all bug changes.

1 47

[Bug 26186] New: Remove "subroutine redefined" warnings from tests
by bugzilla-daemon＠bugs.koha-community.org 23 Jul '25

23 Jul '25

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=26186 Bug ID: 26186 Summary: Remove "subroutine redefined" warnings from tests Change sponsored?: --- Product: Koha Version: unspecified Hardware: All OS: All Status: NEW Severity: normal Priority: P5 - low Component: Test Suite Assignee: chris(a)bigballofwax.co.nz Reporter: jonathan.druart(a)bugs.koha-community.org QA Contact: testopia(a)bugs.koha-community.org kohadev-koha@kohadevbox:/kohadevbox/koha$ prove t/db_dependent/cronjobs/advance_notices_digest.t t/db_dependent/cronjobs/advance_notices_digest.t .. No circulation PREDUEDGST letter transported by phone at /kohadevbox/koha/C4/Letters.pm line 605. Use of uninitialized value in concatenation (.) or string at (eval 1565) line 587. no letter of type '' found for borrowernumber 58. Please see sample_notices.sql at (eval 1565) line 587. t/db_dependent/cronjobs/advance_notices_digest.t .. 1/2 Subroutine parse_letter redefined at (eval 1572) line 444. Subroutine get_branch_info redefined at (eval 1572) line 481. Subroutine send_digests redefined at (eval 1572) line 532. No circulation PREDUEDGST letter transported by phone at /kohadevbox/koha/C4/Letters.pm line 605. Use of uninitialized value in concatenation (.) or string at (eval 1572) line 587. no letter of type '' found for borrowernumber 59. Please see sample_notices.sql at (eval 1572) line 587. No circulation PREDUEDGST letter transported by phone at /kohadevbox/koha/C4/Letters.pm line 605. Use of uninitialized value in concatenation (.) or string at (eval 1572) line 587. no letter of type '' found for borrowernumber 59. Please see sample_notices.sql at (eval 1572) line 587. t/db_dependent/cronjobs/advance_notices_digest.t .. ok All tests successful. Files=1, Tests=2, 5 wallclock secs ( 0.02 usr 0.01 sys + 3.34 cusr 1.38 csys = 4.75 CPU) Result: PASS kohadev-koha@kohadevbox:/kohadevbox/koha$ prove t/db_dependent/Auth_with_ldap.t t/db_dependent/Auth_with_ldap.t .. 1/4 Subroutine ldapserver_error redefined at /kohadevbox/koha/C4/Auth_with_ldap.pm line 48. Subroutine description redefined at /kohadevbox/koha/C4/Auth_with_ldap.pm line 80. Subroutine search_method redefined at /kohadevbox/koha/C4/Auth_with_ldap.pm line 87. Subroutine checkpw_ldap redefined at /kohadevbox/koha/C4/Auth_with_ldap.pm line 111. Subroutine ldap_entry_2_hash redefined at /kohadevbox/koha/C4/Auth_with_ldap.pm line 261. Subroutine exists_local redefined at /kohadevbox/koha/C4/Auth_with_ldap.pm line 312. Subroutine _do_changepassword redefined at /kohadevbox/koha/C4/Auth_with_ldap.pm line 338. Subroutine update_local redefined at /kohadevbox/koha/C4/Auth_with_ldap.pm line 369. Subroutine ldapserver_error redefined at /kohadevbox/koha/C4/Auth_with_ldap.pm line 48. Subroutine description redefined at /kohadevbox/koha/C4/Auth_with_ldap.pm line 80. Subroutine search_method redefined at /kohadevbox/koha/C4/Auth_with_ldap.pm line 87. Subroutine checkpw_ldap redefined at /kohadevbox/koha/C4/Auth_with_ldap.pm line 111. Subroutine ldap_entry_2_hash redefined at /kohadevbox/koha/C4/Auth_with_ldap.pm line 261. Subroutine exists_local redefined at /kohadevbox/koha/C4/Auth_with_ldap.pm line 312. Subroutine _do_changepassword redefined at /kohadevbox/koha/C4/Auth_with_ldap.pm line 338. Subroutine update_local redefined at /kohadevbox/koha/C4/Auth_with_ldap.pm line 369. -- You are receiving this mail because: You are watching all bug changes.

1 8

[Bug 26157] New: Redirect expected DBI warnings
by bugzilla-daemon＠bugs.koha-community.org 23 Jul '25

23 Jul '25

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=26157 Bug ID: 26157 Summary: Redirect expected DBI warnings Change sponsored?: --- Product: Koha Version: unspecified Hardware: All OS: All Status: NEW Severity: enhancement Priority: P5 - low Component: Test Suite Assignee: chris(a)bigballofwax.co.nz Reporter: jonathan.druart(a)bugs.koha-community.org QA Contact: testopia(a)bugs.koha-community.org We should not display DBI warnings from tests when they are expected. -- You are receiving this mail because: You are watching all bug changes.

1 18

[Bug 26417] New: Remove warn in Koha::Patron is_valid_age
by bugzilla-daemon＠bugs.koha-community.org 23 Jul '25

23 Jul '25

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=26417 Bug ID: 26417 Summary: Remove warn in Koha::Patron is_valid_age Change sponsored?: --- Product: Koha Version: master Hardware: All OS: All Status: NEW Severity: normal Priority: P5 - low Component: Patrons Assignee: koha-bugs(a)lists.koha-community.org Reporter: fridolin.somers(a)biblibre.com QA Contact: testopia(a)bugs.koha-community.org CC: gmcharlt(a)gmail.com, kyle.m.hall(a)gmail.com In Koha::Patron is_valid_age the is a compare : $age < $low But $low may be undef. It generates a warn : Use of uninitialized value $low in numeric lt (<) Needs a test like for $high. -- You are receiving this mail because: You are the assignee for the bug. You are watching all bug changes.

1 23

[Bug 23685] New: Exporting report may consume unlimited memory
by bugzilla-daemon＠bugs.koha-community.org 23 Jul '25

23 Jul '25

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=23685 Bug ID: 23685 Summary: Exporting report may consume unlimited memory Change sponsored?: --- Product: Koha Version: master Hardware: All OS: All Status: NEW Severity: normal Priority: P5 - low Component: Reports Assignee: koha-bugs(a)lists.koha-community.org Reporter: paul(a)flo.org QA Contact: testopia(a)bugs.koha-community.org In guided_reports.pl, when exporting report results ($phase eq 'Export') all rows of data are fetched and then converted to the desired format. This may consume an unlimited amount of memory; when report results are particularly large, it may consume all available memory, leading to timed-out HTTP requests, crashes, and potentially data loss. (We have experienced Zebra index corruption as a result.) In master, reports/guided_reports.pl supports three export types -- tab-delimited, CSV, or *.ods. In each case, all data are loaded into memory and held there before any output is produced: 891 $sql = get_prepped_report( $sql, \@param_names, \@sql_params ); 892 my ($sth, $q_errors) = execute_query($sql); ... 895 if ($format eq 'tab') { 896 $type = 'application/octet-stream'; 897 $content .= join("\t", header_cell_values($sth)) . "\n"; 898 $content = Encode::decode('UTF-8', $content); 899 while (my $row = $sth->fetchrow_arrayref()) { 900 $content .= join("\t", @$row) . "\n"; 901 } 902 } else { 903 my $delimiter = C4::Context->preference('delimiter') || ','; 904 if ( $format eq 'csv' ) { ... 914 while (my $row = $sth->fetchrow_arrayref()) { 915 if ($csv->combine(@$row)) { 916 $content .= $csv->string() . "\n"; 917 } else { 918 push @$q_errors, { combine => $csv->error_diag() } ; 919 } 920 } 921 } 922 elsif ( $format eq 'ods' ) { ... 932 # Other line in Unicode 933 my $sql_rows = $sth->fetchall_arrayref(); 934 foreach my $sql_row ( @$sql_rows ) { 935 my @content_row; 936 foreach my $sql_cell ( @$sql_row ) { 937 push @content_row, Encode::encode( 'UTF8', $sql_cell ); 938 } 939 push @$ods_content, \@content_row; 940 } 941 942 # Process 943 generate_ods($ods_filepath, $ods_content); 944 945 # Output 946 binmode(STDOUT); 947 open $ods_fh, '<', $ods_filepath; 948 $content .= $_ while <$ods_fh>; 949 unlink $ods_filepath; 950 } The *.ods case is particularly problematic, because before any data is sent back to the user's browser, *three* copies of the full results are sitting in memory simultaneously -- @$sql_row, @$ods_content, and $content. -- You are receiving this mail because: You are watching all bug changes. You are the assignee for the bug.

1 78