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

[Bug 36441] New: Improve performance of Item::is_bundle
by bugzilla-daemon＠bugs.koha-community.org 18 Feb '25

18 Feb '25

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=36441 Bug ID: 36441 Summary: Improve performance of Item::is_bundle 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: glasklas(a)gmail.com QA Contact: testopia(a)bugs.koha-community.org For a biblio with a large number of holdings (which could be the case for serials for example) Item::is_bundle is one of the main culprits in terms of performance. I know it's usually frowned upon substituting DBIx queries with the DBI-equivalent, but think there are few other option in this case and should be very unlikely to cause any future regressions. Attached before and after flamegraphs to illustrate the difference in performance. -- You are receiving this mail because: You are the assignee for the bug. You are watching all bug changes.

1 11

[Bug 26884] New: overdue_notices.pl
by bugzilla-daemon＠bugs.koha-community.org 18 Feb '25

18 Feb '25

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=26884 Bug ID: 26884 Summary: overdue_notices.pl Change sponsored?: --- Product: Koha Version: unspecified Hardware: All OS: All Status: NEW Severity: enhancement Priority: P5 - low Component: Command-line Utilities Assignee: koha-bugs(a)lists.koha-community.org Reporter: koha(a)trust-box.at QA Contact: testopia(a)bugs.koha-community.org CC: robin(a)catalyst.net.nz command line script overdue_notices.pl should 1. moved to a module (see https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16063) 2. refactored/cleaned up most importantly 3. made idempotent this could be achived via a checksum/signature including the "defining" fields for the message ie borrowernumber subject content metadata letter_code message_transport_type to_address from_address reply_address content_type so before a message is created a checksum is calculated and a look-up in the database is done whether the message already exists (maybe be ignored via a flag) so no matter how often or "how/too late" the script is executed, no message is "forgotten"/send multiple times -- You are receiving this mail because: You are the assignee for the bug. You are watching all bug changes.

1 9

[Bug 35213] New: Standardize country information sent to Hea
by bugzilla-daemon＠bugs.koha-community.org 17 Feb '25

17 Feb '25

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=35213 Bug ID: 35213 Summary: Standardize country information sent to Hea Change sponsored?: --- Product: Project Infrastructure Version: unspecified Hardware: All OS: All Status: NEW Severity: enhancement Priority: P5 - low Component: Hea Assignee: chris(a)bigballofwax.co.nz Reporter: katrin.fischer(a)bsz-bw.de QA Contact: jonathan.druart+koha(a)gmail.com At the moment we store the translated name of the country in UsageStatsCountry which leads to a confusing display in https://hea.koha-community.org/libraries-by-country. We should use standardized language codes in the database instead of names. -- You are receiving this mail because: You are watching all bug changes.

1 3

[Bug 31242] New: Add rate-limiting to the REST API
by bugzilla-daemon＠bugs.koha-community.org 17 Feb '25

17 Feb '25

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=31242 Bug ID: 31242 Summary: Add rate-limiting to the REST API Change sponsored?: --- Product: Koha Version: master Hardware: All OS: All Status: NEW Severity: enhancement Priority: P5 - low Component: REST API Assignee: koha-bugs(a)lists.koha-community.org Reporter: dcook(a)prosentient.com.au CC: tomascohen(a)gmail.com As has been noted on bug 25870, there are times where we want to rate limit the REST API. -- You are receiving this mail because: You are watching all bug changes. You are the assignee for the bug.

1 11

[Bug 35902] New: Add English country and language codes as authorised values to be using in cataloguing MARC21 008 builder plugin
by bugzilla-daemon＠bugs.koha-community.org 16 Feb '25

16 Feb '25

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=35902 Bug ID: 35902 Summary: Add English country and language codes as authorised values to be using in cataloguing MARC21 008 builder plugin Change sponsored?: --- Product: Koha Version: master Hardware: All OS: All Status: NEW Severity: enhancement Priority: P5 - low Component: Cataloging Assignee: koha-bugs(a)lists.koha-community.org Reporter: aleisha(a)catalyst.net.nz QA Contact: testopia(a)bugs.koha-community.org CC: m.de.rooy(a)rijksmuseum.nl When editing a biblio record, you can launch a builder plugin for 008. Positions 15 to 17 hold a two- or three-character alphabetic code that indicates the place of publication, production, or execution. There is a link to a MARC Code List for Countries Positions 35 to 37 hold a three-character alphabetic code for language. There is a link to a MARC Code List for Languages. It would be helpful if these codes were available as authorised value categories, to prevent user typing error. -- You are receiving this mail because: You are the assignee for the bug. You are watching all bug changes.

1 8

[Bug 22831] New: Elasticsearch - add a maintenance script for checking DB vs index counts
by bugzilla-daemon＠bugs.koha-community.org 14 Feb '25

14 Feb '25

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=22831 Bug ID: 22831 Summary: Elasticsearch - add a maintenance script for checking DB vs index counts Change sponsored?: --- Product: Koha Version: master Hardware: All OS: All Status: NEW Severity: enhancement Priority: P5 - low Component: Searching - Elasticsearch Assignee: koha-bugs(a)lists.koha-community.org Reporter: nick(a)bywatersolutions.com Target Milestone: --- -- You are receiving this mail because: You are the assignee for the bug. You are watching all bug changes.

1 52

[Bug 32499] New: HOLDPLACED email goes to patron branch rather than item branch
by bugzilla-daemon＠bugs.koha-community.org 13 Feb '25

13 Feb '25

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=32499 Bug ID: 32499 Summary: HOLDPLACED email goes to patron branch rather than item branch 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: dcook(a)prosentient.com.au QA Contact: testopia(a)bugs.koha-community.org CC: gmcharlt(a)gmail.com, kyle.m.hall(a)gmail.com The HOLDPLACED email is sent out if "emailLibrarianWhenHoldIsPlaced" is enabled. Currently, it is sent to the patron's branch, but it seems to me (and my librarians) that it would make more sense to be emailed to the item's branch, since that's the branch where the librarian is going to be able to do something about it. -- You are receiving this mail because: You are the assignee for the bug. You are watching all bug changes.

1 15

[Bug 35314] New: yarn css:build generates deprecation warnings
by bugzilla-daemon＠bugs.koha-community.org 13 Feb '25

13 Feb '25

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=35314 Bug ID: 35314 Summary: yarn css:build generates deprecation warnings Change sponsored?: --- Product: Koha Version: unspecified Hardware: All OS: All Status: NEW Severity: major 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 DEPRECATION WARNING: Using / for division outside of calc() is deprecated and will be removed in Dart Sass 2.0.0. Recommendation: math.div($spacer, 2) or calc($spacer / 2) More info and automated migrator: https://sass-lang.com/d/slash-div ╷ 302 │ $headings-margin-bottom: $spacer / 2 !default; │ ^^^^^^^^^^^ ╵ node_modules/bootstrap/scss/_variables.scss 302:31 @import koha-tmpl/opac-tmpl/bootstrap/css/src/_common.scss 38:9 @import koha-tmpl/opac-tmpl/bootstrap/css/src/opac.scss 1:9 root stylesheet DEPRECATION WARNING: Using / for division outside of calc() is deprecated and will be removed in Dart Sass 2.0.0. Recommendation: math.div($input-padding-y, 2) or calc($input-padding-y / 2) More info and automated migrator: https://sass-lang.com/d/slash-div ╷ 498 │ $input-height-inner-quarter: add($input-line-height * .25em, $input-padding-y / 2) !default; │ ^^^^^^^^^^^^^^^^^^^^ ╵ node_modules/bootstrap/scss/_variables.scss 498:73 @import koha-tmpl/opac-tmpl/bootstrap/css/src/_common.scss 38:9 @import koha-tmpl/opac-tmpl/bootstrap/css/src/opac.scss 1:9 root stylesheet DEPRECATION WARNING: Using / for division outside of calc() is deprecated and will be removed in Dart Sass 2.0.0. Recommendation: math.div($custom-control-indicator-size, 2) or calc($custom-control-indicator-size / 2) More info and automated migrator: https://sass-lang.com/d/slash-div ╷ 568 │ $custom-switch-indicator-border-radius: $custom-control-indicator-size / 2 !default; │ ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ ╵ node_modules/bootstrap/scss/_variables.scss 568:49 @import koha-tmpl/opac-tmpl/bootstrap/css/src/_common.scss 38:9 @import koha-tmpl/opac-tmpl/bootstrap/css/src/opac.scss 1:9 root stylesheet DEPRECATION WARNING: Using / for division outside of calc() is deprecated and will be removed in Dart Sass 2.0.0. Recommendation: math.div($spacer, 2) or calc($spacer / 2) More info and automated migrator: https://sass-lang.com/d/slash-div ╷ 713 │ $nav-divider-margin-y: $spacer / 2 !default; │ ^^^^^^^^^^^ ╵ node_modules/bootstrap/scss/_variables.scss 713:37 @import koha-tmpl/opac-tmpl/bootstrap/css/src/_common.scss 38:9 @import koha-tmpl/opac-tmpl/bootstrap/css/src/opac.scss 1:9 root stylesheet DEPRECATION WARNING: Using / for division outside of calc() is deprecated and will be removed in Dart Sass 2.0.0. Recommendation: math.div($spacer, 2) or calc($spacer / 2) More info and automated migrator: https://sass-lang.com/d/slash-div ╷ 718 │ $navbar-padding-y: $spacer / 2 !default; │ ^^^^^^^^^^^ ╵ node_modules/bootstrap/scss/_variables.scss 718:37 @import koha-tmpl/opac-tmpl/bootstrap/css/src/_common.scss 38:9 @import koha-tmpl/opac-tmpl/bootstrap/css/src/opac.scss 1:9 root stylesheet WARNING: 45 repetitive deprecation warnings omitted. DEPRECATION WARNING: Using / for division outside of calc() is deprecated and will be removed in Dart Sass 2.0.0. Recommendation: math.div($spacer, 2) or calc($spacer / 2) More info and automated migrator: https://sass-lang.com/d/slash-div ╷ 302 │ $headings-margin-bottom: $spacer / 2 !default; │ ^^^^^^^^^^^ ╵ node_modules/bootstrap/scss/_variables.scss 302:31 @import koha-tmpl/opac-tmpl/bootstrap/css/src/_common.scss 38:9 @import koha-tmpl/opac-tmpl/bootstrap/css/src/sco.scss 1:9 root stylesheet DEPRECATION WARNING: Using / for division outside of calc() is deprecated and will be removed in Dart Sass 2.0.0. Recommendation: math.div($input-padding-y, 2) or calc($input-padding-y / 2) More info and automated migrator: https://sass-lang.com/d/slash-div ╷ 498 │ $input-height-inner-quarter: add($input-line-height * .25em, $input-padding-y / 2) !default; │ ^^^^^^^^^^^^^^^^^^^^ ╵ node_modules/bootstrap/scss/_variables.scss 498:73 @import koha-tmpl/opac-tmpl/bootstrap/css/src/_common.scss 38:9 @import koha-tmpl/opac-tmpl/bootstrap/css/src/sco.scss 1:9 root stylesheet DEPRECATION WARNING: Using / for division outside of calc() is deprecated and will be removed in Dart Sass 2.0.0. Recommendation: math.div($custom-control-indicator-size, 2) or calc($custom-control-indicator-size / 2) More info and automated migrator: https://sass-lang.com/d/slash-div ╷ 568 │ $custom-switch-indicator-border-radius: $custom-control-indicator-size / 2 !default; │ ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ ╵ node_modules/bootstrap/scss/_variables.scss 568:49 @import koha-tmpl/opac-tmpl/bootstrap/css/src/_common.scss 38:9 @import koha-tmpl/opac-tmpl/bootstrap/css/src/sco.scss 1:9 root stylesheet DEPRECATION WARNING: Using / for division outside of calc() is deprecated and will be removed in Dart Sass 2.0.0. Recommendation: math.div($spacer, 2) or calc($spacer / 2) More info and automated migrator: https://sass-lang.com/d/slash-div ╷ 713 │ $nav-divider-margin-y: $spacer / 2 !default; │ ^^^^^^^^^^^ ╵ node_modules/bootstrap/scss/_variables.scss 713:37 @import koha-tmpl/opac-tmpl/bootstrap/css/src/_common.scss 38:9 @import koha-tmpl/opac-tmpl/bootstrap/css/src/sco.scss 1:9 root stylesheet DEPRECATION WARNING: Using / for division outside of calc() is deprecated and will be removed in Dart Sass 2.0.0. Recommendation: math.div($spacer, 2) or calc($spacer / 2) More info and automated migrator: https://sass-lang.com/d/slash-div ╷ 718 │ $navbar-padding-y: $spacer / 2 !default; │ ^^^^^^^^^^^ ╵ node_modules/bootstrap/scss/_variables.scss 718:37 @import koha-tmpl/opac-tmpl/bootstrap/css/src/_common.scss 38:9 @import koha-tmpl/opac-tmpl/bootstrap/css/src/sco.scss 1:9 root stylesheet WARNING: 45 repetitive deprecation warnings omitted. -- You are receiving this mail because: You are watching all bug changes. You are the assignee for the bug.

1 3

[Bug 29924] New: Introduce password expiration to patron categories
by bugzilla-daemon＠bugs.koha-community.org 13 Feb '25

13 Feb '25

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=29924 Bug ID: 29924 Summary: Introduce password expiration to patron categories Change sponsored?: --- Product: Koha Version: master Hardware: All OS: All Status: NEW Severity: enhancement Priority: P5 - low Component: Authentication Assignee: koha-bugs(a)lists.koha-community.org Reporter: nick(a)bywatersolutions.com QA Contact: testopia(a)bugs.koha-community.org CC: dpavlin(a)rot13.org -- You are receiving this mail because: You are the assignee for the bug. You are watching all bug changes.

1 119

[Bug 34477] New: Further reduce memory usage for idle background job worker
by bugzilla-daemon＠bugs.koha-community.org 13 Feb '25

13 Feb '25

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=34477 Bug ID: 34477 Summary: Further reduce memory usage for idle background job worker 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 While bug 28413 reduced the amount of memory used by background job worker, I think that we can reduce it further yet. I'm seeing a lot of workers around 56MB RES and 104MB VIRT at the moment (although there are some smaller ones). Multiple that for multiple workers across multiple instances, and it adds up pretty quickly. Contrast that with bug 27267 where I've got an idle Zebra indexer that uses 7MB RES and 12MB VIRT. That scales up much nicer. Anyway, not a burning issue, but it would be useful to do sometime down the track. It could be an easy win for server resource usage. -- You are receiving this mail because: You are watching all bug changes. You are the assignee for the bug.

1 3