I just did a trial upgrade of a site and it resulted in 2769 rules in the circulation_rules table.. many of them duplicates and generally a corrupt set of data. (We had 246 circulation_rules and just 87 issuingrules prior to the upgrade being run). It appears to be an error in the update query of bug 18936
Hmm.. the table looks right.. but the display is wrong... Investigating further.
Reducing priority because although I can replicate it regularly, with the exact same dataset Jonathan cannot. I believe it's something to do with DB version but as yet haven't managed to narrow it down further.
Right.. this isn't data related at all.. it's Module version related. I have been running my dev environment with the latest dependencies from CPAN.. it seems Template 3.008 broke support for the [% SET variable UNLESS condition %] syntax which we are using in the smart rules template. See https://github.com/abw/Template2/issues/263 for the bug report upstream
Created attachment 106241 [details] [review] Bug 25807: Add Template 3.008 to the exclude list This patch adds version 3.008 of Template to the excludes list in our cpanfile. It also adds support for displaying this version on the about page.
Test plan 1/ Ensure you have a version of Template installed that is supported (i.e >= 2.22 but not 3.008) 2/ Look at the about page and note `2.22, !3.008` is displayed for the 'Template' module in 'grey' 3/ Update your version of Template to 3.008 (cpanm Template@3.008) 4/ Refresh the about screen and note that 'Template' now displays in 'yellow' 5/ Signoff 6/ Revert your version of Template (so you can work properly again, cpanm Template@2.22)
Created attachment 106242 [details] [review] Bug 25807: Add Template 3.008 to the exclude list This patch adds version 3.008 of Template to the excludes list in our cpanfile. It also adds support for displaying this version on the about page. Signed-off-by: Tomas Cohen Arazi <tomascohen@theke.io>
Created attachment 106325 [details] [review] Bug 25807: Add Template 3.008 to the exclude list This patch adds version 3.008 of Template to the excludes list in our cpanfile. It also adds support for displaying this version on the about page. Signed-off-by: Tomas Cohen Arazi <tomascohen@theke.io> Signed-off-by: Julian Maurice <julian.maurice@biblibre.com>
Pushed to master for 20.11, thanks to everybody involved!
backported to 20.05.x for 20.05.02
Hey all, this does not apply cleanly on 19.11.x, please rebase if you'd like me to backport.