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

[Bug 12561] Omnibus: Deprecate non-XSLT detail and result views
by bugzilla-daemon＠bugs.koha-community.org 18 May '21

18 May '21

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=12561 Fridolin Somers <fridolin.somers(a)biblibre.com> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |fridolin.somers(a)biblibre.co | |m -- You are receiving this mail because: You are watching all bug changes.

1 0

[Bug 8132] Batch delete tool deletes items with holds on them
by bugzilla-daemon＠bugs.koha-community.org 18 May '21

18 May '21

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=8132 Jonathan Druart <jonathan.druart+koha(a)gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Blocks| |28353 Referenced Bugs: https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28353 [Bug 28353] regression: Batch Item Deletion no longer shows which items were not removed -- You are receiving this mail because: You are watching all bug changes.

1 0

[Bug 6473] Test bug for Git-bz
by bugzilla-daemon＠bugs.koha-community.org 18 May '21

18 May '21

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=6473 Victor Grousset/tuxayo <victor(a)tuxayo.net> changed: What |Removed |Added ---------------------------------------------------------------------------- Depends on|28294 | Referenced Bugs: https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28294 [Bug 28294] C4::Circulation::updateWrongTransfer should be moved into Koha:: -- You are receiving this mail because: You are watching all bug changes.

1 0

[Bug 6473] Test bug for Git-bz
by bugzilla-daemon＠bugs.koha-community.org 18 May '21

18 May '21

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=6473 Victor Grousset/tuxayo <victor(a)tuxayo.net> changed: What |Removed |Added ---------------------------------------------------------------------------- Depends on|27064 |28294 Referenced Bugs: https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=27064 [Bug 27064] Transferring an item with a hold allows the user to set a hold waiting without transferring to the correct branch https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28294 [Bug 28294] C4::Circulation::updateWrongTransfer should be moved into Koha:: -- You are receiving this mail because: You are watching all bug changes.

1 0

[Bug 6473] Test bug for Git-bz
by bugzilla-daemon＠bugs.koha-community.org 18 May '21

18 May '21

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=6473 Victor Grousset/tuxayo <victor(a)tuxayo.net> changed: What |Removed |Added ---------------------------------------------------------------------------- Depends on| |27064 Referenced Bugs: https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=27064 [Bug 27064] Transferring an item with a hold allows the user to set a hold waiting without transferring to the correct branch -- You are receiving this mail because: You are watching all bug changes.

1 0

[Bug 6473] Test bug for Git-bz
by bugzilla-daemon＠bugs.koha-community.org 18 May '21

18 May '21

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=6473 Victor Grousset/tuxayo <victor(a)tuxayo.net> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |victor(a)tuxayo.net --- Comment #251 from Victor Grousset/tuxayo <victor(a)tuxayo.net> --- tested dependency following -- 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 18 May '21

18 May '21

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14367 Aleisha Amohia <aleisha(a)catalyst.net.nz> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #121025|0 |1 is obsolete| | --- Comment #57 from Aleisha Amohia <aleisha(a)catalyst.net.nz> --- Created attachment 121076 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=121076&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: * Update the database, update the schema files, restart services * 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> 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 18 May '21

18 May '21

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14367 Aleisha Amohia <aleisha(a)catalyst.net.nz> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Failed QA |Needs Signoff -- 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 18 May '21

18 May '21

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14367 --- Comment #56 from Aleisha Amohia <aleisha(a)catalyst.net.nz> --- (In reply to David Nind from comment #55) > Thanks Aleisha. > > I'm getting an error when clicking save after making a change to a record: > > No property history for Koha::Biblio::Metadata at > /usr/share/perl5/Exception/Class/Base.pm line 88 > > in Exception::Class::Base::throw at > /usr/share/perl5/Exception/Class/Base.pm line 88 > > 85: > 86: $proto->rethrow if ref $proto; > 87: > 88: die $proto->new(@_); > 89: } > 90: > 91: sub rethrow { > > Show function arguments > in Koha::Object::set at /kohadevbox/koha/Koha/Object.pm line 270 > > 267: my @columns = @{$self->_columns()}; > 268: > 269: foreach my $p ( keys %$properties ) { > 270: unless ( grep { $_ eq $p } @columns ) { > 271: Koha::Exceptions::Object::PropertyNotFound->throw( > "No property $p for " . ref($self) ); > 272: } > 273: } > > Show function arguments > in Koha::Object::update at /kohadevbox/koha/Koha/Object.pm line 217 > > 214: sub update { > 215: my ($self, $values) = @_; > 216: Koha::Exceptions::Object::NotInStorage->throw unless > $self->in_storage; > 217: $self->set($values)->store(); > 218: } > 219: > 220: =head3 $object->delete(); > > Show function arguments > > .... Probably just my test plan missing the requirement of updating the schema files! Can you please test again? -- You are receiving this mail because: You are watching all bug changes.

1 0

[Bug 12362] Branch transfer records orphaned on cancelled holds
by bugzilla-daemon＠bugs.koha-community.org 17 May '21

17 May '21

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=12362 Martin Renvoize <martin.renvoize(a)ptfs-europe.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Blocks| |28294 Referenced Bugs: https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28294 [Bug 28294] C4::Circulation::updateWrongTransfer should be moved into Koha:: -- You are receiving this mail because: You are watching all bug changes.

1 0