PKP Bugzilla – Full Text Bug Listing
|Summary:||Upgrade fails during post-install with email templates error|
|Product:||OJS||Reporter:||Michael Felczak <michael.pkp>|
|Component:||Installer||Assignee:||Michael Felczak <michael.pkp>|
|Version Reported In:||2.3.4||Also Affects:|
Patch against OJS 2.3.4
Patch #2 against OJS 2.3.4
Patch against OJS 2.3.4 (needs to be applied against lib/pkp)
Description Michael Felczak 2011-05-24 12:20:15 PDT
All upgrade steps complete successfully, but post-install fails with the following: <snip> [code: Installer Installer::addPluginVersions] [post-install] ERROR: Upgrade failed: DB: ##installer.installParseEmailTemplatesFileError##
Comment 1 Michael Felczak 2011-05-24 12:21:59 PDT
This was a side effect of Bug 5837 (wrong upgrade of plugins emailTemplates). After checking for installed email templates, error is raised rather than continuing with the upgrade.
Comment 2 Michael Felczak 2011-05-24 12:26:47 PDT
Created attachment 3516 [details] Patch against OJS 2.3.4 This patch needs to be applied against lib/pkp
Comment 3 Alec Smecher 2011-05-24 12:33:34 PDT
Are you sure, Michael? That fix looks odd to me. I suspect you're running into trouble parsing a plugin's email templates file and I don't think that change will correct it. Were you able to narrow down the problem e.g. to a particular plugin?
Comment 4 Michael Felczak 2011-05-24 12:42:20 PDT
Hi Alec, Yes, fix has been tested. I ended up debugging it and found that the problem applied to all plugins with email templates. Turns out this was correctly handled in master but not stable. Will update fix to match master.
Comment 5 Michael Felczak 2011-05-24 12:49:41 PDT
Created attachment 3517 [details] Patch #2 against OJS 2.3.4 Subsequent patch based on master branch This patch needs to be applied against lib/pkp
Comment 6 Michael Felczak 2011-05-24 13:37:04 PDT
Created attachment 3518 [details] Patch against OJS 2.3.4 (needs to be applied against lib/pkp) This patch needs to be applied against lib/pkp
Comment 7 Alec Smecher 2011-05-24 13:38:12 PDT
Sounds good, Michael -- thanks.
Comment 8 Michael Felczak 2011-05-24 13:41:42 PDT