When an item from another branch is checked in, the holdingbranch is updated, and this change is logged. We record the checkin in statistics, we do not need the cataloguing update logged To recreate: 1 - Enable cataloguing log 2 - Check in an item with a different holding branch 3 - Check the action logs 4 - There is a cataloguing log for the holdingbranch update
Created attachment 134763 [details] [review] Bug 30714: Unit test
Created attachment 134764 [details] [review] Bug 30714: Do no log update to holdingbranch When an item from another branch is checked in, the holdingbranch is updated, and this change is logged. We record the checkin in statistics, we do not need the cataloguing update logged To recreate: 1 - Enable cataloguing log 2 - Check in an item with a different holding branch 3 - Check the action logs 4 - There is a cataloguing log for the holdingbranch update 5 - Apply patch 6 - Repeate with another item 7 - No log added!
Created attachment 134863 [details] [review] Bug 30714: Unit test Signed-off-by: Owen Leonard <oleonard@myacpl.org>
Created attachment 134864 [details] [review] Bug 30714: Do no log update to holdingbranch When an item from another branch is checked in, the holdingbranch is updated, and this change is logged. We record the checkin in statistics, we do not need the cataloguing update logged To recreate: 1 - Enable cataloguing log 2 - Check in an item with a different holding branch 3 - Check the action logs 4 - There is a cataloguing log for the holdingbranch update 5 - Apply patch 6 - Repeate with another item 7 - No log added! Signed-off-by: Owen Leonard <oleonard@myacpl.org>
I'm a bit unsure about this proposal, as the info in statistics table might be deleted at some point and the CataloguingLog syspref promises to log all item record changes. Why do you propose this change?
We had decided not to add checkins/checkouts to the cataloguing log (they update the onloan column). So I think it seems consequential to say that changes that happen due to circulation actions should not update. Having changes to holding library and onloan not log would personally be ok for me as a rule. But if this is controversial maybe we could have something like a "MarkLostItemsAsReturned" for this log where we list what is logged more clearly?
To me CataloguingLog implies changes made during cataloging, that is changes to a record by a person - not changes made second-hand as in circulation, and I don't think this is the correct place to maintain this information I could be convinced that 'ReturnLog' should record the return, hence the branch update, even if item not issued, but I think that would be a separate bug
Created attachment 135100 [details] [review] Bug 30714: Do no log update to holdingbranch When an item from another branch is checked in, the holdingbranch is updated, and this change is logged. We record the checkin in statistics, we do not need the cataloguing update logged To recreate: 1 - Enable cataloguing log 2 - Check in an item with a different holding branch 3 - Check the action logs 4 - There is a cataloguing log for the holdingbranch update 5 - Apply patch 6 - Repeate with another item 7 - No log added! Signed-off-by: Owen Leonard <oleonard@myacpl.org> Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl>
This also rises again the need to separate biblio cataloguing logs from items cataloguing logs
Pushed to master for 22.05, thanks to everybody involved [U+1F984]