https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=29341
--- Comment #7 from Jonathan Druart <[email protected]> --- (In reply to Joonas Kylmälä from comment #5) > This messes up the statistic table, people are expecting there to be > 'OPACRenew' as the value and not NULL. If you don't want to make this a huge > patch I would suggest to just drop the foreign key for the > 'transaction_branchcode' in pseudonymized_transactions table. I also assume > people don't want there to be foreign key relation in order to be able to > delete branches and still keep the stats in place. > > However, if you want to go the original patch's route, which I think not, > then the change should be done in Koha::Item::renewal_branchcode instead. > The syspref documentation and database upgrade would need to be provided as > well. Oops, my bad! Thanks! I think we want to keep the FK, this patch (can be squashed) is modifying the branchcode AFTER the stat entry has been created. Sonia, Lyon 3, can you confirm the expected behaviour please? -- You are receiving this mail because: You are watching all bug changes. _______________________________________________ Koha-bugs mailing list [email protected] https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
