The DB update in updatedatabase.pl looks like this: 9755 $DBversion = "3.19.00.010"; 9756 if ( CheckVersion($DBversion) ) { 9757 $dbh->do("INSERT INTO `systempreferences` (variable,value,explanation,options,type) VALUES('SessionRestrictionByIP','1','Check for Change in Remote IP address for Session Security. Disable when remote ip address changes frequently.','','YesNo')"); 9758 print "Upgrade to $DBversion done (Bug 5511 - SessionRestrictionByIP)"; 9759 SetVersion ($DBversion); 9760 } The missing newline at the end of line 9758 means that when you run the webinstaller, this update and the next one are displayed on one line/as one bullet point: * Upgrade to 3.19.00.009 done (Bug 13007 - Add new foreign key suggestions.budgetid) * Upgrade to 3.19.00.010 done (Bug 5511 - SessionRestrictionByIP)Upgrade to 3.19.00.011 done (Bug 13417: Add permission to delete public lists) Not a big deal, of course, but it would look better on two lines.
Created attachment 35925 [details] [review] Bug 13712 - Missing newline in DB update 3.19.00.010 To test: - Make sure your database is pre 3.19.00.010 (for example by loading in an older backup) - Run through the web-based database update (web installer, step 3) - Verify that updates 3.19.00.010 and 3.19.00.011 are squashed together on one line - Apply the patch - Reset the database and run the web-based database upgrade again - Verify that the two uupdates are noe displayed on separate lines (as separate bullet points)
Created attachment 35988 [details] [review] [PASSED QA] Bug 13712 - Missing newline in DB update 3.19.00.010 To test: - Make sure your database is pre 3.19.00.010 (for example by loading in an older backup) - Run through the web-based database update (web installer, step 3) - Verify that updates 3.19.00.010 and 3.19.00.011 are squashed together on one line - Apply the patch - Reset the database and run the web-based database upgrade again - Verify that the two uupdates are noe displayed on separate lines (as separate bullet points) Signed-off-by: Tomas Cohen Arazi <tomascohen@gmail.com>
Passing this trivial patch.
Patch pushed to master. Thanks Magnus!