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

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

21 Sep '20

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14242 Katrin Fischer <katrin.fischer(a)bsz-bw.de> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #110141|0 |1 is obsolete| | --- Comment #87 from Katrin Fischer <katrin.fischer(a)bsz-bw.de> --- Created attachment 110493 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=110493&action=e… Bug 14242: Use ISBN-field to automatically fill out purchase suggestions Add new jQuery plugin "autofill" which transforms selected element(s) into search fields for Google Books API and automatically fills requested fields with search result. Use in OPAC purchase suggestions to automatically fill out fields when entering ISBN-number. Test plan: 1. Run updatedatabase.pl 2. Enable system precference "OPACAutoFill" 3. Log into OPAC and go to purchase suggetions page 4. Write a valid ISBN into the ISBN field and press the edit icon next to the field, or unfocus the field. 5. Title, Author, Publisher and Item Type fields should now be filled automaticly. 6. Press "undo" to undo - should restore all changed fields to original values Tested all patches together, works as expected. Signed-off-by: Marc <veron(a)veron.ch> Signed-off-by: Maryse Simard <maryse.simard(a)inlibro.com> Signed-off-by: Lisette Scheer <lisetteslatah(a)gmail.com> -- 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 21 Sep '20

21 Sep '20

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14242 Katrin Fischer <katrin.fischer(a)bsz-bw.de> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Signed Off |Passed QA -- You are receiving this mail because: You are watching all bug changes.

1 0

[Bug 14367] History for MARC records. Roll back changes on a timeline or per field.
by bugzilla-daemon＠bugs.koha-community.org 21 Sep '20

21 Sep '20

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14367 David Nind <david(a)davidnind.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #110444|0 |1 is obsolete| | --- Comment #48 from David Nind <david(a)davidnind.com> --- Created attachment 110491 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=110491&action=e… Bug 14367: (follow-up) Rebasing to make patch work on master Run through test plan and confirm everything still works as expected. Signed-off-by: David Nind <david(a)davidnind.com> Signed-off-by: David Nind <david(a)davidnind.com> Signed-off-by: David Nind <david(a)davidnind.com> -- You are receiving this mail because: You are watching all bug changes.

1 0

[Bug 14367] History for MARC records. Roll back changes on a timeline or per field.
by bugzilla-daemon＠bugs.koha-community.org 21 Sep '20

21 Sep '20

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14367 David Nind <david(a)davidnind.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #110443|0 |1 is obsolete| | --- Comment #47 from David Nind <david(a)davidnind.com> --- Created attachment 110490 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=110490&action=e… Bug 14367: (follow-up) Fixing merge conflicts... ...and other fixes to make patch work. Signed-off-by: David Nind <david(a)davidnind.com> Signed-off-by: David Nind <david(a)davidnind.com> Signed-off-by: David Nind <david(a)davidnind.com> -- You are receiving this mail because: You are watching all bug changes.

1 0

[Bug 14367] History for MARC records. Roll back changes on a timeline or per field.
by bugzilla-daemon＠bugs.koha-community.org 21 Sep '20

21 Sep '20

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14367 David Nind <david(a)davidnind.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #110442|0 |1 is obsolete| | --- Comment #46 from David Nind <david(a)davidnind.com> --- Created attachment 110489 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=110489&action=e… Bug 14367: Add MARC record history This is a proof-of-concept implementation for adding history support to MARC records. Every time a change is made a complete copy of the record is stored along with the date/time and user identity. The changes are listed under each field in the MARC record editor and can be reverted with a click. The changes are stored as a JSON array in a new column named 'history' in the database. The array is re-read from the database before updating the record to prevent old data lingering in the session from overwriting newer changes made by other users. If we decide to implement this feature it might be better to simply create a new table altogether and link it rather than the clumsy JSON solution. That would eliminate a lot of bulky code that transforms MARC-KOHA-JSON and back while ensuring data integrity. Also, there are plans to add permissions to the MARC records; this likely requires more complex interactions that will scale badly with the current JSON solution. At present, the history is hardcoded to 10 entries. This can easily be made into a syspref. The current implementation should probably be refactored into a 'BiblioHistory' class before deploying. Documentation of the functions/methods are also needed. Icon is ugly and needs to be improved. Sponsored-By: Halland County Library & Catalyst IT Test plan: * Run updatedatabase.pl * Log into Staff interface, search for a title, choose to edit it's MARC record. * Chose a MARC field and modify it, press Save. * Open the MARC editor again for the same title. * Next to the edited field a new icon should appear, looking like a clock face with a encircling arrow; the history icon. * Clicking the icon should open a table showing all changes done to the record, including value change, date/time and user name. * Clicking a record in the history log should revert that field to the clicked value (a previous value) * Run prove t/db_dependent/BiblioHistory.t and confirm all tests pass Signed-off-by: David Nind <david(a)davidnind.com> -- You are receiving this mail because: You are watching all bug changes.

1 0

[Bug 14367] History for MARC records. Roll back changes on a timeline or per field.
by bugzilla-daemon＠bugs.koha-community.org 21 Sep '20

21 Sep '20

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14367 David Nind <david(a)davidnind.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Needs Signoff |Signed Off -- You are receiving this mail because: You are watching all bug changes.

1 0

[Bug 14784] Missing checkin message for debarred patrons when issuing rules 'fine days = 0'
by bugzilla-daemon＠bugs.koha-community.org 21 Sep '20

21 Sep '20

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14784 Alexis Ripetti <alexis.ripetti(a)inLibro.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Signed Off |Needs Signoff -- You are receiving this mail because: You are watching all bug changes.

1 0

[Bug 14784] Missing checkin message for debarred patrons when issuing rules 'fine days = 0'
by bugzilla-daemon＠bugs.koha-community.org 21 Sep '20

21 Sep '20

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14784 Alexis Ripetti <alexis.ripetti(a)inLibro.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Failed QA |Signed Off CC| |alexis.ripetti(a)inLibro.com --- Comment #46 from Alexis Ripetti <alexis.ripetti(a)inLibro.com> --- This patch adds tests for PrevDebarred message of AddReturn when finesdays = 0. I also changed one thing that seemed weird to me in the C4/Circulation.pm file. Test plan: Before 1) Select a user with active restrictions 2) Make sure finesday=0 for the user category 3) Checkout and return an item (not overdue) : a previous restriction reminder will appear 4) Checkout and return an overdue item : no previous restriction reminder will appear After applying patch: Same steps, but a reminder should appear for step 4) And finally: 5) prove -v t/db_dependent/Circulation.t All the tests should now pass. Waiting for your comments. -- You are receiving this mail because: You are watching all bug changes.

1 0

[Bug 14784] Missing checkin message for debarred patrons when issuing rules 'fine days = 0'
by bugzilla-daemon＠bugs.koha-community.org 21 Sep '20

21 Sep '20

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14784 Alexis Ripetti <alexis.ripetti(a)inLibro.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #106809|0 |1 is obsolete| | --- Comment #45 from Alexis Ripetti <alexis.ripetti(a)inLibro.com> --- Created attachment 110485 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=110485&action=e… Bug 14784: Add tests This patch adds tests for PrevDebarred message of AddReturn when finesdays = 0. To test: 1. Apply patches 2. prove -v t/db_dependent/Circulation.t When applying only this patch, one test should fail. -- You are receiving this mail because: You are watching all bug changes.

1 0

[Bug 15436] MARC21: Use semicolon between series name and volume information
by bugzilla-daemon＠bugs.koha-community.org 21 Sep '20

21 Sep '20

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=15436 David Nind <david(a)davidnind.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #110439|0 |1 is obsolete| | --- Comment #18 from David Nind <david(a)davidnind.com> --- Created attachment 110481 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=110481&action=e… Bug 15436: (follow-up) Fix separator for a single 440 in staff Signed-off-by: David Nind <david(a)davidnind.com> -- You are receiving this mail because: You are watching all bug changes.

1 0