bug 19287 added a new option to MarkLostItemsAsReturned it accidentally set an incorrect value for the preference
Created attachment 82640 [details] [review] Bug 21869: Fix incorrect value setting in DB update To test: 1 - Set some value for MarkLostAsReturned syspref 2 - Set your db version to 17.12.00.035 (or load an older db) 3 - updatedatabase 4 - Note your values are lost and db is incorrectly set 5 - Apply patch 6 - Reset values 7 - Reset DB version 8 - Run updatedatabase 9 - Original values are retained
We can't restore any lost values, but this should allow us to prevent any further damage during upgrades
For me now
What about the dbrev .35 in front: UPDATE systempreferences SET type="multiple", options="batchmod|moredetail|cronjob|additem", value="batchmod,moredetail,cronjob,additem" WHERE variable="MarkLostItemsAsReturned" Type is multiple so more values are allowed. But options are separated with | and values are separated with , comma Needs more attention
IRC <kidclamp> marcelr: yeah, the original db update did that, this was a followup adding a new option to the list [14:45] <kidclamp> so the values shouldn't be changed
Created attachment 82647 [details] [review] Bug 21869: Fix incorrect value setting in DB update To test: 1 - Set some value for MarkLostAsReturned syspref 2 - Set your db version to 17.12.00.035 (or load an older db) 3 - updatedatabase 4 - Note your values are lost and db is incorrectly set 5 - Apply patch 6 - Reset values 7 - Reset DB version 8 - Run updatedatabase 9 - Original values are retained Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl> Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl>
oops two SOs: one for signoff, one for qa :)
Awesome work all! Pushed to master for 18.11
Pushed to stable for 18.05.07
Depends on Bug 19287 not in 17.11.x