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

[Bug 29018] New: Deleting patrons from REST API doesn't do any checks or move to deletedborrowers
by bugzilla-daemon＠bugs.koha-community.org 21 Feb '22

21 Feb '22

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=29018 Bug ID: 29018 Summary: Deleting patrons from REST API doesn't do any checks or move to deletedborrowers Change sponsored?: --- Product: Koha Version: master Hardware: All OS: All Status: NEW Severity: major Priority: P5 - low Component: REST API Assignee: koha-bugs(a)lists.koha-community.org Reporter: katrin.fischer(a)bsz-bw.de When deleting a patron via the REST API it doesn't block on - has a guarantee - has checkouts - has pending fines and it doesn't move the patron to deletedborrowers :( It looks like the code around deleting patrons could do with a little work: Patrons batch deletes appear to use: https://git.koha-community.org/Koha-community/Koha/src/commit/42ce8c29960dc… Deleting a single patron from the account in staff has a lot of additional checks in the .pl https://git.koha-community.org/Koha-community/Koha/src/commit/9d6d641d1f8b7… And then there are delete and move_to_deleted in Koha/Patron* -- You are receiving this mail because: You are the assignee for the bug. You are watching all bug changes.

1 33

[Bug 29696] New: "Suggest for purchase" missing biblio link
by bugzilla-daemon＠bugs.koha-community.org 21 Feb '22

21 Feb '22

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=29696 Bug ID: 29696 Summary: "Suggest for purchase" missing biblio link Change sponsored?: --- Product: Koha Version: unspecified Hardware: All OS: All Status: ASSIGNED Severity: major Priority: P5 - low Component: OPAC Assignee: jonathan.druart+koha(a)gmail.com Reporter: jonathan.druart+koha(a)gmail.com QA Contact: testopia(a)bugs.koha-community.org Caused by commit 586bed1319592e05f5dc3acf64a1dba8cae69d6b Bug 28941: Filter suggestion inputs at the OPAC We are loosing the link with the biblio (suggestion.biblionumber) -- You are receiving this mail because: You are watching all bug changes.

1 25

[Bug 28932] New: Backend overriding status_graph element causes duplicate actions
by bugzilla-daemon＠bugs.koha-community.org 21 Feb '22

21 Feb '22

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28932 Bug ID: 28932 Summary: Backend overriding status_graph element causes duplicate actions Change sponsored?: --- Product: Koha Version: unspecified Hardware: All OS: All Status: NEW Severity: normal Priority: P5 - low Component: ILL Assignee: koha-bugs(a)lists.koha-community.org Reporter: andrew.isherwood(a)ptfs-europe.com Koha::Illrequest::_status_graph_union merges the core status graph and one provided by a backend. Although not officially supported... > Whilst it is not intended that this works, you can override entries in $origin > with entries with the same key in $new_graph. ...a backend should be able to override a core method. This was necessary when implementing the RapidILL backend. However, this revealed a bug in which any next_actions and prev_actions declared in both would end up being duplicated, leading to duplicate buttons in the toolbar. -- You are receiving this mail because: You are the assignee for the bug. You are watching all bug changes.

1 21

[Bug 29758] New: CGI::param in list context in boraccount.pl warning
by bugzilla-daemon＠bugs.koha-community.org 20 Feb '22

20 Feb '22

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=29758 Bug ID: 29758 Summary: CGI::param in list context in boraccount.pl warning Change sponsored?: --- Product: Koha Version: 21.05 Hardware: All OS: All Status: NEW Severity: minor Priority: P5 - low Component: Architecture, internals, and plumbing Assignee: tomascohen(a)gmail.com Reporter: evelyn(a)bywatersolutions.com QA Contact: testopia(a)bugs.koha-community.org Found the following warning in plack intranet error log: CGI::param called in list context from boraccount.pl line 58, this can lead to vulnerabilities. See the warning in "Fetching the value or values of a single named parameter" at /usr/share/perl5/CGI.pm line 412. -- You are receiving this mail because: You are watching all bug changes.

1 15

[Bug 18540] New: koha-indexdefs-to-zebra.xsl introduces MARC21 stuff into UNIMARC xslts
by bugzilla-daemon＠bugs.koha-community.org 20 Feb '22

20 Feb '22

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18540 Bug ID: 18540 Summary: koha-indexdefs-to-zebra.xsl introduces MARC21 stuff into UNIMARC xslts Change sponsored?: --- Product: Koha Version: master Hardware: All OS: All Status: NEW Severity: normal Priority: P5 - low Component: Architecture, internals, and plumbing Assignee: gmcharlt(a)gmail.com Reporter: tomascohen(a)gmail.com QA Contact: testopia(a)bugs.koha-community.org Bug 14332 introduced troubles because the koha-indexdefs-to-zebra.xsl is by no means MARC21-only or even biblio-only. To reproduce: - On current master, run: $ xsltproc etc/zebradb/xsl/koha-indexdefs-to-zebra.xsl etc/zebradb/marc_defs/unimarc/biblios/biblio-koha-indexdefs.xml > etc/zebradb/marc_defs/unimarc/biblios/biblio-zebra-indexdefs.xsl - Run: $ git diff => FAIL: The following is added to etc/zebradb/marc_defs/unimarc/biblios/biblio-zebra-indexdefs.xsl, thus UNIMARC field 245 is being sent to the Title index @@ -42,6 +43,7 @@ definition file (probably something like {biblio,authority}-koha-indexdefs.xml) <xslo:apply-templates mode="index_match_heading"/> <xslo:apply-templates mode="index_subject_thesaurus"/> <xslo:apply-templates mode="index_all"/> + <xslo:apply-templates mode="index_sort_tit"/> </z:record> </xslo:template> <xslo:template match="marc:controlfield[@tag='001']"> @@ -2681,6 +2683,19 @@ definition file (probably something like {biblio,authority}-koha-indexdefs.xml) <xslo:value-of select="."/> </z:index> </xslo:template> + <xslo:template mode="index_sort_tit" match="marc:datafield[@tag='245']"> + <xslo:variable name="chop"> + <xslo:choose> + <xslo:when test="not(number(@ind2))">0</xslo:when> + <xslo:otherwise> + <xslo:value-of select="number(@ind2)"/> + </xslo:otherwise> + </xslo:choose> + </xslo:variable> + <z:index name="Title:s"> + <xslo:value-of select="substring(marc:subfield[@code='a'], $chop+1)"/> + </z:index> + </xslo:template> -- You are receiving this mail because: You are watching all bug changes.

1 19

[Bug 29812] New: C4::Context not included, but used in Koha::Token
by bugzilla-daemon＠bugs.koha-community.org 20 Feb '22

20 Feb '22

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=29812 Bug ID: 29812 Summary: C4::Context not included, but used in Koha::Token Change sponsored?: --- Product: Koha Version: master Hardware: All OS: All Status: NEW Severity: normal Priority: P5 - low Component: Architecture, internals, and plumbing Assignee: koha-bugs(a)lists.koha-community.org Reporter: nick(a)bywatersolutions.com QA Contact: testopia(a)bugs.koha-community.org -- You are receiving this mail because: You are watching all bug changes. You are the assignee for the bug.

1 12

[Bug 29884] New: Missing test in api/v1/patrons.t
by bugzilla-daemon＠bugs.koha-community.org 20 Feb '22

20 Feb '22

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=29884 Bug ID: 29884 Summary: Missing test in api/v1/patrons.t 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 Depends on: 28632 t::lib::Dates::compare( $updated_on_got, $updated_on_expected, 'updated_on values matched' ); compare is not calling is() or ok(), it's caller job to do it. Referenced Bugs: https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28632 [Bug 28632] patrons.t fragile on slow boxes -- You are receiving this mail because: You are watching all bug changes.

1 14

[Bug 29808] New: Stock rotation fails to advance when an item is checked out from the branch that is the next stage
by bugzilla-daemon＠bugs.koha-community.org 20 Feb '22

20 Feb '22

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=29808 Bug ID: 29808 Summary: Stock rotation fails to advance when an item is checked out from the branch that is the next stage Change sponsored?: --- Product: Koha Version: master Hardware: All OS: All Status: NEW Severity: major Priority: P5 - low Component: Tools Assignee: koha-bugs(a)lists.koha-community.org Reporter: nick(a)bywatersolutions.com QA Contact: testopia(a)bugs.koha-community.org When an item is advanced we generate a transfer to the next branch. If the item is already at that branch we receive the transfer If an item is issued, an exception is thrown when receiving a transfer To recreate: 1 - Enable StockRotation system preference 2 - Browse to Tools->Stock rotation->New Rota and create a new rota, set as cyclical 3 - Manage->stages and add two stages, each with 0 day duration 4 - Manage->items - add an item 5 - Checkout the item from the first branch of rotation 6 - Tools->Stock rotation->Manage->items->move to next stage 7 - Error: [Transfer item is currently checked out] at /usr/share/perl5/Exception/Class/Base.pm line 88 -- You are receiving this mail because: You are watching all bug changes. You are the assignee for the bug.

1 14

[Bug 14210] Manage staged MARC records deletes replaced item when undoing import into catalog
by bugzilla-daemon＠bugs.koha-community.org 20 Feb '22

20 Feb '22

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14210 Katrin Fischer <katrin.fischer(a)bsz-bw.de> changed: What |Removed |Added ---------------------------------------------------------------------------- Severity|normal |major --- Comment #6 from Katrin Fischer <katrin.fischer(a)bsz-bw.de> --- I am raising severity as this is about data loss. -- You are receiving this mail because: You are watching all bug changes.

1 0

[Bug 14210] Manage staged MARC records deletes replaced item when undoing import into catalog
by bugzilla-daemon＠bugs.koha-community.org 20 Feb '22

20 Feb '22

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14210 Katrin Fischer <katrin.fischer(a)bsz-bw.de> changed: What |Removed |Added ---------------------------------------------------------------------------- Summary|Manage staged marc records |Manage staged MARC records |deletes replaced item when |deletes replaced item when |undoing import into catalog |undoing import into catalog -- You are receiving this mail because: You are watching all bug changes.

1 0