Closed (fixed)
Project:
Webform Conditional (Same Page Conditionals)
Version:
7.x-1.x-dev
Component:
Code
Priority:
Normal
Category:
Bug report
Assigned:
Unassigned
Reporter:
Created:
9 Apr 2011 at 08:33 UTC
Updated:
28 Jun 2011 at 18:10 UTC
Jump to comment: Most recent file
Comments
Comment #1
rumblewand commentedHoly wow the list of errors on viewing the actual form is long mainly involving the cid. Did not hide the field either. I will try a few things and see what I can find.
Comment #2
rumblewand commentedOK. I uninstalled and reinstalled the dev 7 module and did not receive the first batch of errors. However I do still receive errors on all my fields in the existing webform along the lines of
Comment #3
rumblewand commentedSpoke too soon. The first set of errors still appears when editing a particular field. I'll hold off on anymore investigating for now, my eyes are bleeding.
Comment #4
tedbowI am thinking that you may have your php error_reporting set wrong. I think generally you should not be showing the error type "Notice".
Can somebody confirm this?
Comment #5
rumblewand commentedHi Ted! I don't mind disabling the output of the Notice errors but I'm inclined to believe they are somehow responsible for me not being able to hide any fields conditionally however I can't say for sure since the dev release is new.
Comment #6
rumblewand commentedSOLVED - the following change to my template.php file disrupts the webform conditional somehow and you may need to be aware. I used it to ensure that field descriptions would appear above the input for user instructions.
Once I removed this and cleaned my caches the webform conditionals module works as expected! I also noticed a patch in the drupal 7 thread that fixes various errors!
Comment #7
rumblewand commentedI'll open a different thread regarding getting the module to work with the above enhancement to webform.
Comment #8
tedbow@rumblewand, Ok I am closing this issue.
Comment #9
teranex commentedIn PHP 5.3 notices are enabled by default. This notice can be avoided by always defining the $currPage variable. The attached patch does this.
Comment #10
tedbowteranex
thanks, I have commited this to 6 & 7 dev branches