To recreate: 1 - Enable Pseudonymization in system preferences NOTE: See bug 28911 to setup bcrypt value in koha conf 2 - Checkout an item to a patron 3 - From the checkouts page, check the box for the checkout under renewals 4 - Click the 'Renew selected items' button 5 - Internal server error: ==> /var/log/koha/kohadev/plack-error.log <== {UNKNOWN}: encountered object '2024-07-31T16:56:36', but neither allow_blessed, convert_blessed nor allow_tags settings are enabled (or TO_JSON/FREEZE method missing) at /kohadevbox/koha/Koha/BackgroundJob.pm line 108. at /kohadevbox/koha/C4/Circulation.pm line 3411
Created attachment 169907 [details] [review] Bug 37540: Ensure datetime is a string before enqueueing pseudonymization To test: 1 - Enable Pseudonymization in system preferences NOTE: See bug 28911 for bcrypt setup 2 - Issue an item to a patron 3 - View the patrons checkouts 4 - Check the box under 'Renew' 5 - Renew selected items 6 - Internal server error 7 - Apply patch 8 - Restart all 9 - Repeat 4&5 10 - Success!
Libraries report this is actually blocking all circulations
Created attachment 169911 [details] [review] Bug 37540: Ensure datetime is a string before enqueueing pseudonymization To test: 1 - Enable Pseudonymization in system preferences NOTE: See bug 28911 for bcrypt setup 2 - Issue an item to a patron 3 - View the patrons checkouts 4 - Check the box under 'Renew' 5 - Renew selected items 6 - Internal server error 7 - Apply patch 8 - Restart all 9 - Repeat 4&5 10 - Success! Signed-off-by: Roman Dolny <roman.dolny@jezuici.pl>
Created attachment 169931 [details] [review] Bug 37540: Ensure datetime is a string before enqueueing pseudonymization To test: 1 - Enable Pseudonymization in system preferences NOTE: See bug 28911 for bcrypt setup 2 - Issue an item to a patron 3 - View the patrons checkouts 4 - Check the box under 'Renew' 5 - Renew selected items 6 - Internal server error 7 - Apply patch 8 - Restart all 9 - Repeat 4&5 10 - Success! Signed-off-by: Roman Dolny <roman.dolny@jezuici.pl> Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>
Happy to push as is, but should we have a bug for adding a primary key to statistics?
Pushed for 24.11! Well done everyone, thank you!
(In reply to Katrin Fischer from comment #5) > Happy to push as is, but should we have a bug for adding a primary key to > statistics? I didn't consider it largely because of the cost of the update for systems with lots of statistics, but it might be worth filing for discussion
(In reply to Nick Clemens (kidclamp) from comment #7) > (In reply to Katrin Fischer from comment #5) > > Happy to push as is, but should we have a bug for adding a primary key to > > statistics? > > I didn't consider it largely because of the cost of the update for systems > with lots of statistics, but it might be worth filing for discussion Yes, I think let's discuss it at least. I believe it probably also makes other things harder.
Backported to 24.05.x for upcoming 24.05.06