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

[Bug 37600] New: OpacSuppression does not work in Unimarc
by bugzilla-daemon＠bugs.koha-community.org 23 Sep '25

23 Sep '25

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=37600 Bug ID: 37600 Summary: OpacSuppression does not work in Unimarc Change sponsored?: --- Product: Koha Version: unspecified Hardware: All OS: All Status: NEW Severity: enhancement Priority: P5 - low Component: MARC Bibliographic data support Assignee: koha-bugs(a)lists.koha-community.org Reporter: Igor.A.Sychev(a)mail.ru QA Contact: testopia(a)bugs.koha-community.org OpacSuppression does not work in Unimarc. If I understood correctly, need to add <index_subfields tag="955" subfields="n"> <target_index>Suppress:w</target_index> <target_index>Suppress:n</target_index> </index_subfields> in https://git.koha-community.org/Koha-community/Koha/src/branch/main/etc/zebr… -- You are receiving this mail because: You are watching all bug changes. You are the assignee for the bug.

1 19

[Bug 6796] Overnight checkouts taking into account opening and closing hours
by bugzilla-daemon＠bugs.koha-community.org 23 Sep '25

23 Sep '25

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=6796 --- Comment #159 from Marcel de Rooy <m.de.rooy(a)rijksmuseum.nl> --- (In reply to Emily Lamancusa (emlam) from comment #158) > Hmmm...I'm seeing a few errors in this part of the dbrev: > > + my $branches = > + $dbh->selectall_arrayref( "SELECT branchcode FROM > branches", { Slice => {} } ); > + for my $branch (@$branches) { > + my $branchcode = $branch->{branchcode}; > + > + my @indexes = ( 0 .. 6 ); > + for my $i (@indexes) { > + $dbh->do( > + q{ INSERT INTO library_hours (library_id, day) > VALUES ( ?, ? ) }, undef, $branchcode, > + $i > + ); > + } > > > 1. The parameter bindings in the INSERT statement are reversed - the order > in the SQL is (library_id, day), but the order of the parameter bindings is > undef, $branchcode. > > 2. The INSERT statement isn't idempotent - should be INSERT IGNORE > > 3. The dbrev is using a Koha object, which goes against the coding guidelines > > The section of the dbrev that adds the syspref comes after the above > section. I would think #1 would crash the dbrev, which would explain why the > syspref didn't get added...but I'm surprised someone didn't notice it sooner > if that was happening consistently. (Unless someone did notice and there's > already a follow-up that isn't linked - I didn't dig for one.) Great that you had a closer look ! I think that the binding is correct btw. There is always an undef in the middle in those statements (room for attributes hash). The $i is on the next line. Agree with 2 and 3. -- You are receiving this mail because: You are watching all bug changes.

1 0

[Bug 40001] New: edit_items permission allows deletion of individual items. Should be separate.
by bugzilla-daemon＠bugs.koha-community.org 23 Sep '25

23 Sep '25

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=40001 Bug ID: 40001 Summary: edit_items permission allows deletion of individual items. Should be separate. Change sponsored?: --- Product: Koha Version: 24.11 Hardware: All OS: All Status: NEW Severity: enhancement Priority: P5 - low Component: Cataloging Assignee: koha-bugs(a)lists.koha-community.org Reporter: hananmustafaawan(a)gmail.com QA Contact: testopia(a)bugs.koha-community.org CC: m.de.rooy(a)rijksmuseum.nl Currently, the edit_items permission allows a user to delete individual items using the "Action" button on the edit item page. This behavior persists even when the user does not have the delete_all_items permission, which is intended (according to documentation) to manage bulk deletion. This creates a permissions flaw: staff with rights to edit items can also delete items, which may not always be desirable from a library’s operational or security perspective. Steps to Reproduce: Create a staff account with only the edit_items permission (without delete_all_items) Go to: catalogue/itemsearch.pl Edit an item (via cataloguing/additem.pl?op=edititem&...) Click the Action button, then Delete the item The item is deleted successfully — despite lacking delete_all_items permission Expected Behavior: There should be a separate delete_items permission for deleting individual items OR Deleting any item (individual or bulk) should require the delete_all_items permission. -- You are receiving this mail because: You are watching all bug changes. You are the assignee for the bug.

1 4

[Bug 6796] Overnight checkouts taking into account opening and closing hours
by bugzilla-daemon＠bugs.koha-community.org 23 Sep '25

23 Sep '25

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=6796 --- Comment #158 from Emily Lamancusa (emlam) <emily.lamancusa(a)montgomerycountymd.gov> --- Hmmm...I'm seeing a few errors in this part of the dbrev: + my $branches = + $dbh->selectall_arrayref( "SELECT branchcode FROM branches", { Slice => {} } ); + for my $branch (@$branches) { + my $branchcode = $branch->{branchcode}; + + my @indexes = ( 0 .. 6 ); + for my $i (@indexes) { + $dbh->do( + q{ INSERT INTO library_hours (library_id, day) VALUES ( ?, ? ) }, undef, $branchcode, + $i + ); + } 1. The parameter bindings in the INSERT statement are reversed - the order in the SQL is (library_id, day), but the order of the parameter bindings is undef, $branchcode. 2. The INSERT statement isn't idempotent - should be INSERT IGNORE 3. The dbrev is using a Koha object, which goes against the coding guidelines The section of the dbrev that adds the syspref comes after the above section. I would think #1 would crash the dbrev, which would explain why the syspref didn't get added...but I'm surprised someone didn't notice it sooner if that was happening consistently. (Unless someone did notice and there's already a follow-up that isn't linked - I didn't dig for one.) -- You are receiving this mail because: You are watching all bug changes.

1 0

[Bug 26779] New: Make desk selection consistent in auth.tt and set-library.tt
by bugzilla-daemon＠bugs.koha-community.org 23 Sep '25

23 Sep '25

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=26779 Bug ID: 26779 Summary: Make desk selection consistent in auth.tt and set-library.tt Change sponsored?: --- Product: Koha Version: master Hardware: All OS: All Status: NEW Severity: enhancement Priority: P5 - low Component: Circulation Assignee: koha-bugs(a)lists.koha-community.org Reporter: nicolas.legrand(a)bulac.fr QA Contact: testopia(a)bugs.koha-community.org CC: gmcharlt(a)gmail.com, kyle.m.hall(a)gmail.com, severine.queune(a)bulac.fr When changing libraries in set-library.tt, the desks selection list print only desk. The desk list is printed with a different code from auth.pp but modified by the same javascript. Make the template code consistent and work for both pages. -- You are receiving this mail because: You are watching all bug changes. You are the assignee for the bug.

1 4

[Bug 40148] New: Add ability to set desks as hold pickup locations or non-hold pickup locations
by bugzilla-daemon＠bugs.koha-community.org 23 Sep '25

23 Sep '25

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=40148 Bug ID: 40148 Summary: Add ability to set desks as hold pickup locations or non-hold pickup locations 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: lisette(a)bywatersolutions.com QA Contact: testopia(a)bugs.koha-community.org CC: gmcharlt(a)gmail.com For some libraries being able to capture a hold (thus removing it from the holds queue/preventing other branches from filling it) without sending the notice because it's not ready for pickup yet. The hold could then be checked in again at a desk that does allow hold pickups. -- You are receiving this mail because: You are watching all bug changes. You are the assignee for the bug.

1 3

[Bug 36127] New: Port default HOLDPLACED and HOLD_CHANGED notices to Template Toolkit syntax
by bugzilla-daemon＠bugs.koha-community.org 23 Sep '25

23 Sep '25

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=36127 Bug ID: 36127 Summary: Port default HOLDPLACED and HOLD_CHANGED notices to Template Toolkit syntax Change sponsored?: --- Product: Koha Version: master Hardware: All OS: All Status: NEW Keywords: roadmap_24_05 Severity: enhancement Priority: P5 - low Component: Notices Assignee: koha-bugs(a)lists.koha-community.org Reporter: emily.lamancusa(a)montgomerycountymd.gov QA Contact: testopia(a)bugs.koha-community.org Blocks: 15278 Referenced Bugs: https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=15278 [Bug 15278] [Omnibus] Port default slips and notices from bespoke syntax to Template Toolkit -- You are receiving this mail because: You are watching all bug changes. You are the assignee for the bug.

1 15

[Bug 6796] Overnight checkouts taking into account opening and closing hours
by bugzilla-daemon＠bugs.koha-community.org 23 Sep '25

23 Sep '25

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=6796 --- Comment #157 from Marcel de Rooy <m.de.rooy(a)rijksmuseum.nl> --- Leading to warnings on $considerlibraryhours ne 'ignore' -- You are receiving this mail because: You are watching all bug changes.

1 0

[Bug 6796] Overnight checkouts taking into account opening and closing hours
by bugzilla-daemon＠bugs.koha-community.org 23 Sep '25

23 Sep '25

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=6796 Marcel de Rooy <m.de.rooy(a)rijksmuseum.nl> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |m.de.rooy(a)rijksmuseum.nl --- Comment #156 from Marcel de Rooy <m.de.rooy(a)rijksmuseum.nl> --- Testing a 24.11 instance with production data where the table library_hours is created but the pref does not exist? Any idea? -- You are receiving this mail because: You are watching all bug changes.

1 0

[Bug 9762] Log circulation overrides
by bugzilla-daemon＠bugs.koha-community.org 23 Sep '25

23 Sep '25

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=9762 --- Comment #117 from Jonathan Druart <jonathan.druart(a)gmail.com> --- + $message = "Restriction overridden temporarily"; + @message = ("Restriction overridden temporarily"); 793 <h4>No patron matched <span class="ex">[% message | html %]</span></h4> Then the message is not translatable? -- You are receiving this mail because: You are watching all bug changes.

1 0