Sample itemtypes can't load with new itemtypes table structure.
Created attachment 21551 [details] [review] Bug 10965 - Sample itemtypes can't load on new install Bug 10513 added two columns to table itemtypes, but sample_itemtypes can't be loaded because it relies on old structure. This patch rewrites sample_itemtypes.sql, stating explicitly columns, removing backticks and consolidates all in one insertion. To test: 1) On master with current DB structure, delete item types 2) try loading sample_itemtyes.sql It fails with "ERROR 1136 (21S01) at line 1: Column count doesn't match value count at row 1" 3) apply patch 4) try again, this time it will load and itemtypes created
Patch applied cleanly, go forth and signoff
Created attachment 21567 [details] [review] Bug 10965 - Sample itemtypes can't load on new install Bug 10513 added two columns to table itemtypes, but sample_itemtypes can't be loaded because it relies on old structure. This patch rewrites sample_itemtypes.sql, stating explicitly columns, removing backticks and consolidates all in one insertion. To test: 1) On master with current DB structure, delete item types 2) try loading sample_itemtyes.sql It fails with "ERROR 1136 (21S01) at line 1: Column count doesn't match value count at row 1" 3) apply patch 4) try again, this time it will load and itemtypes created Signed-off-by: Chris Cormack <chris@bigballofwax.co.nz>
I think we need follow-ups to fix the translated installers as well.
Created attachment 21597 [details] [review] [PASSED QA] Bug 10965 - Sample itemtypes can't load on new install Bug 10513 added two columns to table itemtypes, but sample_itemtypes can't be loaded because it relies on old structure. This patch rewrites sample_itemtypes.sql, stating explicitly columns, removing backticks and consolidates all in one insertion. To test: 1) On master with current DB structure, delete item types 2) try loading sample_itemtyes.sql It fails with "ERROR 1136 (21S01) at line 1: Column count doesn't match value count at row 1" 3) apply patch 4) try again, this time it will load and itemtypes created Signed-off-by: Chris Cormack <chris@bigballofwax.co.nz> Signed-off-by: Katrin Fischer <Katrin.Fischer.83@web.de> SQL change only. Tested by running the web installer on an empty database. All sample data loads without errors.
Working on a follow-up for translated installers.
(In reply to Katrin Fischer from comment #6) > Working on a follow-up for translated installers. If you like I can do this. Somehow I forget other languages :)
Already done - bug 10969 :)
Pushed to master. Thanks, Bernardo!