March 2025 - Koha-bugs - lists.koha-community.org

[Bug 29403] New: dt_from_string should fail if passed and rfc3339 formatted string when 'iso' is selected as dateformat
by bugzilla-daemon＠bugs.koha-community.org 04 Jul '25

04 Jul '25

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=29403 Bug ID: 29403 Summary: dt_from_string should fail if passed and rfc3339 formatted string when 'iso' is selected as dateformat Change sponsored?: --- Product: Koha Version: master Hardware: All OS: All Status: NEW Severity: enhancement Priority: P5 - low Component: Architecture, internals, and plumbing Assignee: koha-bugs(a)lists.koha-community.org Reporter: martin.renvoize(a)ptfs-europe.com QA Contact: testopia(a)bugs.koha-community.org Our dt_from_string regex for time is optional.. as such we allow for non-matches.. but we don't check that the string as a whole matches a valid date/datetime string.. thus we don't fail when we should. -- You are receiving this mail because: You are watching all bug changes. You are the assignee for the bug.

1 106

[Bug 29718] New: DateTime - our 'iso' is not ISO 8601
by bugzilla-daemon＠bugs.koha-community.org 04 Jul '25

04 Jul '25

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=29718 Bug ID: 29718 Summary: DateTime - our 'iso' is not ISO 8601 Change sponsored?: --- Product: Koha Version: unspecified Hardware: All OS: All Status: In Discussion Severity: enhancement Priority: P5 - low Component: Architecture, internals, and plumbing Assignee: koha-bugs(a)lists.koha-community.org Reporter: jonathan.druart+koha(a)gmail.com QA Contact: testopia(a)bugs.koha-community.org CC: martin.renvoize(a)ptfs-europe.com, tomascohen(a)gmail.com In some scripts we are doing unnecessary conversion from string to DT and vice-versa, sometimes several times. For instance, see bug 29711. We have epoch => DT => string => sending to template in 'iso' YYYY-MM-DD HH:MM => DT => string. This is because the stringified version of DT (the real ISO 8601 YYYY-MM-DDTHH:MM:SS) is not supported by our dt_from_string method. There are (at least) two options: * Use DateTime::Format::Strptime In dt_from_string: my $formatter = DateTime::Format::Strptime->new(pattern => '%Y-%m-%d %H:%M:%S'); return DateTime->now( time_zone => $tz, formatter => $formatter, ) unless $date_string; Then later $dt_params{formatter} = $formatter; But that's an overhead for all dt_from_string calls. * add "T?" to $time_re, easy and efficient. -- You are receiving this mail because: You are the assignee for the bug. You are watching all bug changes.

1 60

[Bug 18986] New: High Hold Items with Decreased Loan Period Can' t Be Checked-Out with Self-Check
by bugzilla-daemon＠bugs.koha-community.org 04 Jul '25

04 Jul '25

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18986 Bug ID: 18986 Summary: High Hold Items with Decreased Loan Period Can't Be Checked-Out with Self-Check Change sponsored?: --- Product: Koha Version: 16.11 Hardware: All OS: All Status: NEW Severity: normal Priority: P5 - low Component: Self checkout Assignee: koha-bugs(a)lists.koha-community.org Reporter: cslone(a)camdencountylibrary.org QA Contact: testopia(a)bugs.koha-community.org Scenario: A patron has an item with a high number of holds ready for pickup. They come to their library get the item and use the SCO to try to borrow it. When they scan the item they are given the message: Item cannot be checked out. Sorry, this item cannot be checked out at this station. Please see a member of the library staff. Additionally, if they don’t notice the message and scan another item, they are returned their account summary/the check-out screen with neither item checked out to the patron. At best this creates a scenario where there's an inconvenient extra step that the patron has to take in order to get their item. At worst, it can lead to the patron inadvertently taking two items with them that have not been checked-out. For the item that is on hold this can lead to further confusion when the it is returned (Koha will indicate that the item must go back on the hold shelf for the patron that just returned it) or when holds are cleared from the shelf (the item will be listed there but won’t be found). To replicate: 1.) In system preferences, go to Circulation preferences->Holds Policy. 2.) There are five preferences grouped together for decreaseLoanHighHolds, make sure to set the first drop down to Enable and enter the reduced loan period and the number of holds that will trigger the reduction (you can configure the other options as desired). 3.) Place a hold on an item with a patron you can use at self-check and place enough additional holds on the item with other accounts in order to trigger the decreased loan period according to your preferences. 4.) Check-in an item from the record with the holds and confirm the hold for the patron. 5.) Log into self-check with this patron and attempt to check-out the item. 6.) You will be taken to the error message. 7.) Scan the barcode again, you'll be returned to the SCO screen. -- You are receiving this mail because: You are the assignee for the bug. You are watching all bug changes.

1 13

[Bug 38107] New: Allow custom sort order of patron_holds_table
by bugzilla-daemon＠bugs.koha-community.org 03 Jul '25

03 Jul '25

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=38107 Bug ID: 38107 Summary: Allow custom sort order of patron_holds_table Change sponsored?: --- Product: Koha Version: Main Hardware: All OS: All Status: NEW Severity: enhancement Priority: P5 - low Component: Hold requests Assignee: koha-bugs(a)lists.koha-community.org Reporter: lucas(a)bywatersolutions.com QA Contact: testopia(a)bugs.koha-community.org CC: gmcharlt(a)gmail.com The patron_hold_table on reserve/request.pl should allow libraries to choose a Default sort order. -- You are receiving this mail because: You are watching all bug changes. You are the assignee for the bug.

1 13

[Bug 32879] New: Item search: Display additional 245 subfields in CSV export
by bugzilla-daemon＠bugs.koha-community.org 03 Jul '25

03 Jul '25

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=32879 Bug ID: 32879 Summary: Item search: Display additional 245 subfields in CSV export Change sponsored?: --- Product: Koha Version: master Hardware: All OS: All Status: NEW Severity: enhancement Priority: P5 - low Component: Searching Assignee: koha-bugs(a)lists.koha-community.org Reporter: katrin.fischer(a)bsz-bw.de QA Contact: testopia(a)bugs.koha-community.org Bug 14911 fixes this for the result list of the item search. It would be grat if the CSV export file could also include the additional information. It's not as simple as using the biblio-title include there too, as that adds spans that you don't want in the CSV file. -- You are receiving this mail because: You are watching all bug changes. You are the assignee for the bug.

1 4

[Bug 20905] New: Warn from Search.t
by bugzilla-daemon＠bugs.koha-community.org 03 Jul '25

03 Jul '25

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=20905 Bug ID: 20905 Summary: Warn from Search.t Change sponsored?: --- Product: Koha Version: master Hardware: All OS: All Status: NEW Severity: enhancement Priority: P5 - low Component: Test Suite Assignee: chris(a)bigballofwax.co.nz Reporter: m.de.rooy(a)rijksmuseum.nl QA Contact: testopia(a)bugs.koha-community.org Test passes but look at this warn from getRecords: ok 67 - presence of a transit does not block hold request action (bug 10741) ERROR DECODING RECORD - Tag "50%" is not a valid tag. at /usr/share/perl5/MARC /File/USMARC.pm line 223. : 00295nam a22001217a 45000030004000000050017000040080041000211000018000622450 0570008050%001100137942001200148999001300160OSt20111207151138.0111207t xxu||||| |||| 00| 0 eng d aGanser, Helen aA record containing a punctuation character as a tag aa note 2ddccBK c214d214 at /usr/share/koha/devclone/C4/ Search.pm line 683. ok 68 - search for ti:punctuation returned expected number of records A few lines later in the test, a warning_like also looks for this error. -- You are receiving this mail because: You are watching all bug changes.

1 2

[Bug 38526] New: Auth_with_* tests fail randomly
by bugzilla-daemon＠bugs.koha-community.org 03 Jul '25

03 Jul '25

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=38526 Bug ID: 38526 Summary: Auth_with_* tests fail randomly 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)gmail.com QA Contact: testopia(a)bugs.koha-community.org -- You are receiving this mail because: You are watching all bug changes.

1 23

[Bug 35506] New: selenium/regressions.t is failing randomly
by bugzilla-daemon＠bugs.koha-community.org 03 Jul '25

03 Jul '25

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=35506 Bug ID: 35506 Summary: selenium/regressions.t is failing randomly 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+koha(a)gmail.com QA Contact: testopia(a)bugs.koha-community.org koha_1 | koha_1 | # Failed test 'Checkouts table must be correctly aligned' koha_1 | # at t/db_dependent/selenium/regressions.t line 204. koha_1 | # got: '' koha_1 | # expected: '1' koha_1 | # thead: 0 ; tfoot: 18 ; tbody: 0 koha_1 | # Looks like you failed 1 test of 1. koha_1 | koha_1 | # Failed test 'Display circulation table correctly' koha_1 | # at t/db_dependent/selenium/regressions.t line 211. selenium_1 | JavaScript error: http://kohadev.mydnsname.org:8080/cgi-bin/koha/opac-user.pl, line 859: ReferenceError: ssGPCYcY is not defined selenium_1 | [Child 2137, MediaDecoderStateMachine #1] WARNING: 7f7ec97e4d60 OpenCubeb() failed to init cubeb: file /builds/worker/checkouts/gecko/dom/media/AudioStream.cpp:324 selenium_1 | [Child 2137, MediaDecoderStateMachine #1] WARNING: Decoder=7f7ecb817400 [OnMediaSinkAudioError]: file /builds/worker/checkouts/gecko/dom/media/MediaDecoderStateMachine.cpp:3980 selenium_1 | [Child 2137, MediaDecoderStateMachine #1] WARNING: Decoder=7f7ecb817400 Decode error: NS_ERROR_DOM_MEDIA_MEDIASINK_ERR (0x806e000b) - OnMediaSinkAudioError: file /builds/worker/checkouts/gecko/dom/media/MediaDecoderStateMachine.cpp:3541 koha_1 | [13:36:29] t/db_dependent/00-strict.t .................................................. ok 87 ms ( 0.08 usr 0.00 sys + 496.36 cusr 72.29 csys = 568.73 CPU) koha_1 | # Looks like you planned 18 tests but ran 1. koha_1 | koha_1 | # Failed test 'Encoding in session variables' koha_1 | # at t/db_dependent/selenium/regressions.t line 340. koha_1 | Can't call method "get_text" on an undefined value at t/db_dependent/selenium/regressions.t line 325. koha_1 | # Looks like your test exited with 11 just after 7. koha_1 | [13:36:34] t/db_dependent/selenium/regressions.t ....................................... koha_1 | Dubious, test returned 11 (wstat 2816, 0xb00) koha_1 | Failed 3/8 subtests This was failing on Koha_Master/2760 (Docker_11) but fixed on 2761 (Docker_15). Nothing has been pushed in between. -- You are receiving this mail because: You are watching all bug changes.

1 78

[Bug 25362] New: Code to create CGISESSID shoudl be in function
by bugzilla-daemon＠bugs.koha-community.org 03 Jul '25

03 Jul '25

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=25362 Bug ID: 25362 Summary: Code to create CGISESSID shoudl be in function Change sponsored?: --- Product: Koha Version: master Hardware: All OS: All Status: NEW Severity: enhancement Priority: P5 - low Component: Architecture, internals, and plumbing Assignee: koha-bugs(a)lists.koha-community.org Reporter: dcook(a)prosentient.com.au QA Contact: testopia(a)bugs.koha-community.org I found 12 difference instances where we create the CGISESSID cookie, and each one has its own invocation that needs to be updated when you want to change that cookie. It would be better if we wrapped the cookie creation in a function. -- You are receiving this mail because: You are watching all bug changes. You are the assignee for the bug.

1 3

[Bug 36987] New: Add preference for which fields to prefill when copying a patron
by bugzilla-daemon＠bugs.koha-community.org 02 Jul '25

02 Jul '25

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=36987 Bug ID: 36987 Summary: Add preference for which fields to prefill when copying a patron Change sponsored?: --- Product: Koha Version: unspecified Hardware: All OS: All Status: NEW Severity: enhancement Priority: P5 - low Component: Patrons Assignee: koha-bugs(a)lists.koha-community.org Reporter: lisette.scheer(a)bywatersolutions.com QA Contact: testopia(a)bugs.koha-community.org CC: gmcharlt(a)gmail.com, kyle.m.hall(a)gmail.com Similar to PrefillGuaranteeField it would be great to be able to set which fields prefill when duplicating patrons. -- You are receiving this mail because: You are watching all bug changes. You are the assignee for the bug.

1 4