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

[Bug 26839] New: Add a 'date published' feature to the facets
by bugzilla-daemon＠bugs.koha-community.org 25 Apr '22

25 Apr '22

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=26839 Bug ID: 26839 Summary: Add a 'date published' feature to the facets Change sponsored?: --- Product: Koha Version: 19.11 Hardware: All OS: All Status: NEW Severity: enhancement Priority: P5 - low Component: OPAC Assignee: oleonard(a)myacpl.org Reporter: barbara.johnson(a)bedfordtx.gov QA Contact: testopia(a)bugs.koha-community.org Add a 'date published' facet to the OPAC so that patrons can limit their search results. For example: 2021 (2) 2020 (9) 2019 (14) -- You are receiving this mail because: You are watching all bug changes.

1 8

[Bug 10734] Add Publication Date Facet in Searching
by bugzilla-daemon＠bugs.koha-community.org 25 Apr '22

25 Apr '22

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=10734 Owen Leonard <oleonard(a)myacpl.org> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |barbara.johnson(a)bedfordtx.g | |ov --- Comment #10 from Owen Leonard <oleonard(a)myacpl.org> --- *** Bug 26839 has been marked as a duplicate of this bug. *** -- You are receiving this mail because: You are watching all bug changes.

1 0

[Bug 12029] Patrons should be able to delete their patron messages
by bugzilla-daemon＠bugs.koha-community.org 25 Apr '22

25 Apr '22

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=12029 Katrin Fischer <katrin.fischer(a)bsz-bw.de> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Signed Off |Failed QA --- Comment #27 from Katrin Fischer <katrin.fischer(a)bsz-bw.de> --- Hi Hayley and all, trying to give this another push forward. 1) Unit tests (blocker) We are still missing the unit tests. Is there any update on that? 2) Database update The database update is still a bit too verbose, as noted earlier, using some uncommon output pattern (ALTERATION, NOTICE and INFO are not needed). We said RM could deal with it, so just explaining a bit more what people meant earlier on: +use Modern::Perl; + +return { + bug_number => "12029", + description => "Enable patrons to delete messages", + up => sub { + my ($args) = @_; + my ($dbh, $out) = @$args{qw(dbh out)}; + + # Do you stuffs here + my $alteration = q{ + ALTER TABLE messages + ADD COLUMN `patron_read_date` timestamp NULL DEFAULT NULL + COMMENT 'date and time patron dismissed message' + AFTER `manager_id` + }; + if( column_exists('messages', 'patron_read_date') ) { + say $out "NOTICE: Column 'messages.patron_read_date' already exists"; + } + else { + say $out "ALTERATION: $alteration"; + $dbh->do($alteration); + } + + # Print useful stuff here + say $out "INFO: Bug 12029 migration applied"; + }, +}; 3) Translatability There is some code in opac/dismiss_message.pl that I wondered about. It contains messages that are not translatable, but I am not sure yet if this would show to patrons, in my tests it didn't seem like it. comment => "Patron read date already set", comment => "Set patron_read_date to ($patron_read_date)", Hm, they might not be used after all? >11. What are you doing with $result? $inputs and comment are returned from the dismiss_patron_message sub but never >reused. This script could be 10 lines long IMO. Failing for 1). Testing in the GUI worked well! -- You are receiving this mail because: You are watching all bug changes.

1 0

[Bug 14242] Use ISBN-field to fill out purchase suggestions (using an Ajax-request)
by bugzilla-daemon＠bugs.koha-community.org 24 Apr '22

24 Apr '22

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14242 Amit Gupta <amitddng135(a)gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |amit.gupta@informaticsgloba | |l.com, | |amitddng135(a)gmail.com -- You are receiving this mail because: You are watching all bug changes.

1 0

[Bug 14364] Allow automatically canceled expired waiting holds to fill the next hold
by bugzilla-daemon＠bugs.koha-community.org 24 Apr '22

24 Apr '22

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14364 Katrin Fischer <katrin.fischer(a)bsz-bw.de> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Signed Off |Failed QA --- Comment #58 from Katrin Fischer <katrin.fischer(a)bsz-bw.de> --- 1) Capitalization It's all good, but in the database update statement: Canceled Hold Available for Different Patron', '0', 'Canceled Hold Available for Different Patron', 'The patron picking up <<biblio.title>> (<<items.barcode>>) has changed to <<borrowers.firstname>> <<borrowers.surname>> (<<borrowers.cardnumber 2) Spelling Should this be reply-to address? + - . If no address is defined here, the email will be sent to the library's reply to address. 2) Tests are failing :( prove t/db_dependent/Holds/ExpireReservesAutoFill.t t/db_dependent/Holds/ExpireReservesAutoFill.t .. 1/3 # No tests run! t/db_dependent/Holds/ExpireReservesAutoFill.t .. 2/3 # Failed test 'No tests run for subtest "Test automatically canceled expired waiting holds to fill the next hold, without a transfer"' # at t/db_dependent/Holds/ExpireReservesAutoFill.t line 131. Undefined subroutine &main::CancelExpiredReserves called at t/db_dependent/Holds/ExpireReservesAutoFill.t line 101. # Looks like your test exited with 11 just after 2. t/db_dependent/Holds/ExpireReservesAutoFill.t .. Dubious, test returned 11 (wstat 2816, 0xb00) Failed 2/3 subtests Test Summary Report ------------------- t/db_dependent/Holds/ExpireReservesAutoFill.t (Wstat: 2816 Tests: 2 Failed: 1) Failed test: 2 Non-zero exit status: 11 Parse errors: Bad plan. You planned 3 tests but ran 2. Files=1, Tests=2, 3 wallclock secs ( 0.03 usr 0.00 sys + 2.66 cusr 0.30 csys = 2.99 CPU) Result: FAIL -- You are receiving this mail because: You are watching all bug changes.

1 0

[Bug 29135] New: OAI should not include biblionumbers from deleteditems when determining deletedbiblios
by bugzilla-daemon＠bugs.koha-community.org 23 Apr '22

23 Apr '22

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=29135 Bug ID: 29135 Summary: OAI should not include biblionumbers from deleteditems when determining deletedbiblios Change sponsored?: --- Product: Koha Version: master 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: nick(a)bywatersolutions.com QA Contact: testopia(a)bugs.koha-community.org When including items in OAI we add the biblionumbers of deleted items into the deleted list This is from this line: 93 if ( $include_items ) { 94 $sql = "($sql) UNION (SELECT DISTINCT(biblionumber) FROM deleteditems $criteria)"; 95 push @bind_params, @part_bind_params; This is wrong because deletion of a single item does not indicate a record is deleted To recreate: 1 - Create /var/lib/koha/kohadev/OAI.yaml with: format: marcxml: metadataPrefix: marcxml metadataNamespace: http://www.loc.gov/MARC21/slim http://www.loc.gov/standards/marcxml/schema/MARC21slim schema: http://www.loc.gov/MARC21/slim http://www.loc.gov/standards/marcxml/schema/MARC21slim.xsd include_items: 1 2 - Set OAI-PMH:ConfFile to '/var/lib/koha/kohadev/OAI.yaml' 3 - Delete an item from a record 4 - View the page: http://localhost:8080/cgi-bin/koha/oai.pl?verb=ListRecords&metadataPrefix=m… 5 - Note the record is now listed as deleted -- You are receiving this mail because: You are the assignee for the bug. You are watching all bug changes.

1 36

[Bug 25879] New: Border for guarantor is larger than the container
by bugzilla-daemon＠bugs.koha-community.org 23 Apr '22

23 Apr '22

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=25879 Bug ID: 25879 Summary: Border for guarantor is larger than the container Change sponsored?: --- Product: Koha Version: master Hardware: All OS: All Status: NEW Severity: enhancement Priority: P5 - low Component: Templates Assignee: oleonard(a)myacpl.org Reporter: caroline.cyr-la-rose(a)inlibro.com QA Contact: testopia(a)bugs.koha-community.org Created attachment 106311 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=106311&action=e… Screenshot showing the guarantor section of the patron add form I noticed that the soft blue border that surrounds a guarantor's information in the patron form is slightly larger than the containing box. To reproduce: 1- Add a patron 2- In guarantor information, search for and add a guarantor 3- Notice that the border surrounding the guarantor is larger than the box for guarantor information -- You are receiving this mail because: You are watching all bug changes.

1 18

[Bug 20384] New: Elastic rebuild script improvements - options for indexing
by bugzilla-daemon＠bugs.koha-community.org 22 Apr '22

22 Apr '22

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=20384 Bug ID: 20384 Summary: Elastic rebuild script improvements - options for indexing 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: claire.hernandez(a)biblibre.com I would be glad to use the rebuild script with some more options like: -n 100 index 100 first records -n "100,2" index 2 records after 100th (101 and 102) --from XXX index biblio with biblionumber >= XXX --to YYY index biblio with biblionumber <= YYY bonus: -i gives elastic install information: syspref, numbers of biblios/auth indexed, name of indice used etc. There is maybe an other way to name the options. Advice or opinion or objection on how to do this ? -- You are receiving this mail because: You are watching all bug changes. You are the assignee for the bug.

1 25

[Bug 28747] New: Clarify wording on RestrictionBlockRenewing syspref
by bugzilla-daemon＠bugs.koha-community.org 22 Apr '22

22 Apr '22

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28747 Bug ID: 28747 Summary: Clarify wording on RestrictionBlockRenewing syspref Change sponsored?: --- Product: Koha Version: master Hardware: All OS: All Status: NEW Severity: normal Priority: P5 - low Component: Staff Client Assignee: koha-bugs(a)lists.koha-community.org Reporter: andrew(a)bywatersolutions.com QA Contact: testopia(a)bugs.koha-community.org CC: gmcharlt(a)gmail.com When RestrictionBlockRenewing is set to Allow, it only allows renewal via the intranet. The wording on the syspref does not make this clear. It says: If a patron is restricted, (allow/block) renewing of items. We should amend it to: If a patron is restricted, (allow/block) renewing of items from the staff interface. -- You are receiving this mail because: You are watching all bug changes. You are the assignee for the bug.

1 16

[Bug 28430] New: Typo: ... that are not alreay in your...
by bugzilla-daemon＠bugs.koha-community.org 22 Apr '22

22 Apr '22

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28430 Bug ID: 28430 Summary: Typo: ... that are not alreay in your... Change sponsored?: --- Product: Koha Version: 20.11 Hardware: All OS: All Status: NEW Keywords: Academy, Manual Severity: enhancement Priority: P5 - low Component: Documentation Assignee: koha-bugs(a)lists.koha-community.org Reporter: katrin.fischer(a)bsz-bw.de QA Contact: testopia(a)bugs.koha-community.org ../../source/tools.rst:3096 (Manual for 20.11) Ignore incoming record: choose this if you don't want to import records that are not alreay in your catalogue (for example, if you're only replacing existing records and don't want to add anything) -- You are receiving this mail because: You are the assignee for the bug. You are watching all bug changes.

1 5