-
Notifications
You must be signed in to change notification settings - Fork 444
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Custom review form possible responses get garbled on upgrade form 3.2.1-x to 3.3.0-x #9953
Comments
Hi @asmecher I think this change introduced a regression in 3.3.0-18 and after. When you upgrade from stable-3.2 to 3.3.0-17 works like a charm (except for the review forms) but when I try to upgrade to *-18 and it shows the following error:
But although the error, the process finishes with the "Successful" message and journal seams to work fine... but the list of Archived articles disappear. Commented initially here: |
@marcbria, can you include the entire output of the upgrade process? |
Sure, but it doesn't explain much more. That's the full output:
|
It's clear that it's not a good idea working at late hours. :-)
An then the full list of changes and the "Succesfully" message. DB dump packaged and ready to send if it's useful to you in some sense. |
Describe the bug
When journals using custom review forms with multiple options upgrade form 3.2.1-x to 3.3.0-x, the possible responses get garbled. See for details:
https://forum.pkp.sfu.ca/t/ojs-review-form-settings-get-deleted-overwritten-on-upgrade-from-3-2-to-3-3/87542
The problem is that there is a generic function that attempts to convert setting values from serialized objects to JSON for ...
_settings
tables, but it does not expect the cardinality of data that thereview_form_element_settings
table has, so multiple rows get clobbered with an update that was supposed to just affect one. This changes thereview_form_element_settings
case to use the right keys.PRs:
stable-3_3_0
: pkp/pkp-lib#9953 Fix garbling of review form responses ojs#4273stable-3_3_0
: pkp/pkp-lib#9953 Fix garbling of review form responses on upgrade omp#1566stable-3_4_0
: pkp/pkp-lib#9953 Fix garbling of review form responses on upgrade (stable-3_4_0 port) #9954main
: pkp/pkp-lib#9953 Fix garbling of review form responses on upgrade (main port) #9955The text was updated successfully, but these errors were encountered: