Closed (fixed)
Project:
Webform Validation
Version:
7.x-1.x-dev
Component:
Code
Priority:
Normal
Category:
Bug report
Assigned:
Reporter:
Created:
2 Nov 2016 at 10:23 UTC
Updated:
25 Nov 2016 at 18:34 UTC
Jump to comment: Most recent
I upgraded webform conditional to realase 7.x-1.12 from 7.x-1.11
All the webforms in the site with a conditional statement give a blank page after submit.
I think the problem could be in the webform_validation.module; changing it with the early version (7.x-1.11) all the webforms go back to work
Comments
Comment #2
liam morlandDo you mean that you upgraded webform_validation? Please provide steps to reproduce the problem. Is there any PHP error? Normally, a white screen means there is a PHP error and they are set to be hidden.
Comment #3
karlsheaI'm having the same problem after upgrading. It looks like webform_get_conditional_sorter() is getting called on line 163, and that function isn't defined in Webform 3.x.
Comment #4
liam morlandwebform_get_conditional_sorter() has been part of webform_validation since 7.x-1.10-rc1 (commit 98757f4). Webform 3.x is no longer supported.
@paolo888: What version of Webform are you using?
Comment #5
liam morlandSorry, I see the problem. There is capabilities checking which made things compatible with both 3.x and 4.x. Commit be7a712 moved the call to webform_get_conditional_sorter() outside of this check, thus breaking 3.x compatibility in 1.12-rc1.
I suggest staying on 7.x-1.11. I may remove the 3.x compatibility altogether.
Comment #6
liam morlandComment #7
liam morlandComment #8
paolo888 commented@Liam Morland: sorry for the late answer, the web form version was 7.x-3.25.
Comment #10
liam morlandPlease give it a try in Webform 3.x and confirm here that it is working for you.
Comment #11
karlsheaWorks for me in 3.x, thank you!
Comment #12
liam morlandThis fixes a regression caused by #2630448: Ensure executeConditionals() has been called before evaluating WebformConditionals::componentVisibility.
Comment #13
liam morlandComment #14
paolo888 commentedWorks for me in 3.25, thank you!
Comment #15
afireintheattic commentedPatch worked on 3.25 for me as well -- thanks, Liam!