Closed (fixed)
Project:
Signup
Version:
5.x-1.x-dev
Component:
Code
Priority:
Normal
Category:
Bug report
Assigned:
Reporter:
Created:
22 Dec 2006 at 14:15 UTC
Updated:
11 Apr 2007 at 15:46 UTC
Jump to comment: Most recent file
The module chokes on signup forms with fieldsets in them which contain form elements, causing errors to be thrown when viewing pages that list individual sign ups:
htmlspecialchars() expects parameter 1 to be string, array given in /home/unionsites/public_html/includes/bootstrap.inc on line 597.
And the data is not shown on the page.
This is because the signup module is not set up to traverse a from with a tree hierarchy more than one level deep when outputting the data.
| Comment | File | Size | Author |
|---|---|---|---|
| #5 | signup_form_fieldsets_47x.patch.txt | 4.35 KB | dww |
| #4 | signup_11.patch | 4.3 KB | hunmonk |
| #2 | signup_10.patch | 3.1 KB | hunmonk |
Comments
Comment #1
dwwyup, sounds like an accurate assessment of the problem. if you can, please provide a patch to fix it. ;)
thanks,
-derek
Comment #2
hunmonk commentedattached patch fixes the issue on the DRUPAL-5 branch of the module -- should be ported to other versions.
since the stored data doesn't really contain enough information to provide meaningful fieldset titles on output, i've simply enclosed the relevant fieldset data in divs with an id equal to the fieldset key. this should at least allow themers some control over the sections.
Comment #3
dwwyay, this is great (chad and i were sitting next to eachother when we wrote it).
however, we forgot about the emails... ;) that will need cleanup to work right, too.
Comment #4
hunmonk commentedthis version handles emails as well -- tested as working on my local install.
Comment #5
dwwhere's the DRUPAL-4-7 backport patch. down with 4.6.x. ;) reviewed, tested -- RTBC. thanks!
Comment #6
hunmonk commentedcommitted to 4.7, 5.0 and HEAD. the code should now handle nested values regardless of depth.
Comment #7
(not verified) commented