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

[Bug 34364] New: Background job - Fix visual progress of progress bar
by bugzilla-daemon＠bugs.koha-community.org 28 Aug '23

28 Aug '23

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=34364 Bug ID: 34364 Summary: Background job - Fix visual progress of progress bar 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: pedro.amorim(a)ptfs-europe.com QA Contact: testopia(a)bugs.koha-community.org Depends on: 31666 Caught during our development of ERM SUSHI, hard to come up with a test plan for current way StageMARCForImport is set up. Referenced Bugs: https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=31666 [Bug 31666] Add job progress bar to stage-marc-import.pl -- You are receiving this mail because: You are watching all bug changes. You are the assignee for the bug.

1 15

[Bug 33556] New: $c->validation->output should be avoided
by bugzilla-daemon＠bugs.koha-community.org 28 Aug '23

28 Aug '23

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33556 Bug ID: 33556 Summary: $c->validation->output should be avoided Change sponsored?: --- Product: Koha Version: master Hardware: All OS: All Status: NEW Severity: normal Priority: P5 - low Component: REST API Assignee: koha-bugs(a)lists.koha-community.org Reporter: tomascohen(a)gmail.com CC: tomascohen(a)gmail.com Talking to the OpenAPI plugin maintainer, he mentioned the use of $c->validation->output should be avoided as the plugin is not designed to have a stable behavior there, and he even thought of just removing the method. That method returns an internal data structure the plugin uses to validate things, and then updates the request itself. Take the following example: GET /patrons x-koha-embed: checkouts,library without the OpenAPI plugin, requesting the header like this: $c->req->headers->header('x-koha-embed') would return a scalar, the string 'checkouts,library'. When using the plugin, and with `x-koha-embed` being defined as collectionFormat: csv, that header is entirely replaced by an arrayref. That's how the plugin works and how it is expected to be used. So we need to replace the uses of $c->validation->format, with normal Mojo usage to avoid future headaches. -- You are receiving this mail because: You are watching all bug changes. You are the assignee for the bug.

1 42

[Bug 34126] New: Tests in t/db_dependent/Koha/Patron.t fail due broken FK constraint
by bugzilla-daemon＠bugs.koha-community.org 28 Aug '23

28 Aug '23

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=34126 Bug ID: 34126 Summary: Tests in t/db_dependent/Koha/Patron.t fail due broken FK constraint Change sponsored?: --- Product: Koha Version: master Hardware: All OS: All Status: NEW Severity: minor Priority: P5 - low Component: Test Suite Assignee: chris(a)bigballofwax.co.nz Reporter: emmi.takkinen(a)koha-suomi.fi QA Contact: testopia(a)bugs.koha-community.org Koha/Patron.t tests fail when apparently when patron should be deleted after paying accountlines: t/db_dependent/Koha/Patron.t .. 10/24 DBIx::Class::Storage::DBI::_dbh_execute(): DBI Exception: DBD::mysql::st execute failed: Cannot add or update a child row: a foreign key constraint fails (`koha_kv`.`accountlines`, CONSTRAINT `accountlines_ibfk_borrowers_2` FOREIGN KEY (`manager_id`) REFERENCES `borrowers` (`borrowernumber`) ON DELETE SET NULL ON UPDATE CASCADE) at /home/emmi/Koha/Koha/Object.pm line 170 # Looks like you planned 14 tests but ran 12. # Failed test 'safe_to_delete() tests' # at t/db_dependent/Koha/Patron.t line 1124. Broken FK constraint# Looks like your test exited with 255 just after 14. t/db_dependent/Koha/Patron.t .. Dubious, test returned 255 (wstat 65280, 0xff00) Failed 11/24 subtests Test Summary Report ------------------- t/db_dependent/Koha/Patron.t (Wstat: 65280 Tests: 14 Failed: 1) Failed test: 14 Non-zero exit status: 255 Parse errors: Bad plan. You planned 24 tests but ran 14. Files=1, Tests=14, 8 wallclock secs ( 0.05 usr 0.00 sys + 7.20 cusr 0.57 csys = 7.82 CPU) Result: FAIL -- You are receiving this mail because: You are watching all bug changes.

1 3

[Bug 15032] [Plack] Scripts that fork (like stage-marc-import.pl) don't work as expected
by bugzilla-daemon＠bugs.koha-community.org 28 Aug '23

28 Aug '23

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=15032 --- Comment #48 from Marcel de Rooy <m.de.rooy(a)rijksmuseum.nl> --- Yes, I think so too. We still have a few forks but they are not in the scope of this report: misc/search_tools/rebuild_elasticsearch.pl: my $pid = fork(); Koha/MetaSearcher.pm: if ( ( $pid = fork ) ) { -- You are receiving this mail because: You are watching all bug changes.

1 0

[Bug 34628] New: Lost item statistics are not copied to pseudonymized transactions
by bugzilla-daemon＠bugs.koha-community.org 28 Aug '23

28 Aug '23

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=34628 Bug ID: 34628 Summary: Lost item statistics are not copied to pseudonymized transactions Change sponsored?: --- Product: Koha Version: master Hardware: All OS: All Status: NEW Severity: enhancement Priority: P5 - low Component: Reports Assignee: koha-bugs(a)lists.koha-community.org Reporter: m.de.rooy(a)rijksmuseum.nl QA Contact: testopia(a)bugs.koha-community.org See bug 33608 comment76 from Andrew. Quote: If we're recording this in statistics, I'd love to see it included in what we also enter in pseudonymized_transactions. Otherwise this data is lost when/if libraries delete their old statistics lines. Currently, only transactions related to a specific borrower are copied to these pseudo tables. The item_lost and item_found type do have a borrowernumber. -- You are receiving this mail because: You are watching all bug changes. You are the assignee for the bug.

1 2

[Bug 15045] URLs should be built using PATH_INFO and SCRIPT_NAME
by bugzilla-daemon＠bugs.koha-community.org 28 Aug '23

28 Aug '23

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=15045 David Cook <dcook(a)prosentient.com.au> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |dcook(a)prosentient.com.au --- Comment #2 from David Cook <dcook(a)prosentient.com.au> --- Do you mean something like Catalyst's "uri_for"? https://metacpan.org/pod/Catalyst#$c-%3Euri_for(-$path?,-@args?,-%5C%25quer… -- You are receiving this mail because: You are watching all bug changes.

1 0

[Bug 15019] Plack testing - Umbrella
by bugzilla-daemon＠bugs.koha-community.org 28 Aug '23

28 Aug '23

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=15019 Bug 15019 depends on bug 15032, which changed state. Bug 15032 Summary: [Plack] Scripts that fork (like stage-marc-import.pl) don't work as expected https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=15032 What |Removed |Added ---------------------------------------------------------------------------- Status|In Discussion |RESOLVED Resolution|--- |WONTFIX -- You are receiving this mail because: You are watching all bug changes.

1 0

[Bug 15032] [Plack] Scripts that fork (like stage-marc-import.pl) don't work as expected
by bugzilla-daemon＠bugs.koha-community.org 28 Aug '23

28 Aug '23

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=15032 David Cook <dcook(a)prosentient.com.au> changed: What |Removed |Added ---------------------------------------------------------------------------- Resolution|--- |WONTFIX Status|In Discussion |RESOLVED --- Comment #47 from David Cook <dcook(a)prosentient.com.au> --- (In reply to Katrin Fischer from comment #46) > I wonder if we could close this now that the staged MARC import has been > moved into a background job (and then close the omnibus bug 15019 as well)? I think so. We still have other scripts that should be run in the background but I don't think they fork so yeah. -- You are receiving this mail because: You are watching all bug changes.

1 0

[Bug 15191] Serials display improvements
by bugzilla-daemon＠bugs.koha-community.org 27 Aug '23

27 Aug '23

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=15191 Katrin Fischer <katrin.fischer(a)bsz-bw.de> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|NEW |RESOLVED Resolution|--- |WONTFIX --- Comment #3 from Katrin Fischer <katrin.fischer(a)bsz-bw.de> --- I am not sure what to do with this one... * The bug description is very general, just hinting on improvements. * There is no test plan. * Patch includes a lot of changes and would need major updating for current codebase and coding style/guidelines. * Original author and later author are no longer active in Koha development. I propose closing this as WONTFIX, but please reopen if you see a way we can still save some of this development. -- You are receiving this mail because: You are watching all bug changes. You are the assignee for the bug.

1 0

[Bug 15174] Allow borrowers to be members of additional categories
by bugzilla-daemon＠bugs.koha-community.org 27 Aug '23

27 Aug '23

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=15174 Katrin Fischer <katrin.fischer(a)bsz-bw.de> changed: What |Removed |Added ---------------------------------------------------------------------------- Resolution|--- |WONTFIX Status|In Discussion |RESOLVED --- Comment #11 from Katrin Fischer <katrin.fischer(a)bsz-bw.de> --- I think the idea of this to be able to apply different circulation rules for a short term loan or other means is still valid, but the proposed implementation left some unresolved questions. I am closing this WONTFIX for now, but please feel free to reopen! -- You are receiving this mail because: You are watching all bug changes.

1 0