i have created a fieldset and added child items to it. I am not making any of the field conditional on another. When I go to save a new field (whether I add it to the fieldset or not) I get the notice above.

CommentFileSizeAuthor
#11 fixed_notice_d6-1218118-10.patch1.79 KBtedbow
#10 fixed_notice_d7-1218118-10.patch1.65 KBtedbow
#6 1218118.patch1.35 KBstella

Comments

laroccadahouse’s picture

laroccadahouse commented

when i remove the fieldsets from the form the error goes away.

tedbow’s picture

tedbow
he/him
Primary language English
Location Ithaca, NY, USA
commented
Status: Active » Postponed (maintainer needs more info)

@laroccadahouse can you make sure you are using the latest version of 7.x-1.x-dev. There is no array index used on line 529 in the latest version. Please update and let me know the line number of the error(if any).

Thanks

laroccadahouse’s picture

laroccadahouse commented

I downloaded the latest dev version on july 25th. I no longer get the error.

tedbow’s picture

tedbow
he/him
Primary language English
Location Ithaca, NY, USA
commented
Status: Postponed (maintainer needs more info) » Closed (fixed)

cool. closing

brightbold’s picture

brightbold
she/her
Primary language English
Location Boston, MA
commented
Status: Closed (fixed) » Active

I'm using the latest dev and am getting this error for line 534. I also am using fieldsets in my webform (and I notice that section of the code deals with fieldsets.)

Also FYI there's a typo on line 529 where it says "childern" instead of "children," but I don't think that's causing any problems because you're defining a variable that doesn't seem to be called again. But I figured you'd want to fix it anyway.

stella’s picture

stella commented
Status: Active » Needs review
StatusFileSize
new 1218118.patch1.35 KB

Here's a patch for the error reported in comment #5

brightbold’s picture

brightbold
she/her
Primary language English
Location Boston, MA
commented

Thanks stella, that patch solves the problem for me, thanks!

allio_froggio’s picture

allio_froggio commented

Do you think you could tell me where exactly I am supposed to insert this patch? (I apologize for my ignorance and very much appreciate the help!)

allio_froggio’s picture

allio_froggio commented

Ohhhh...it says exactly that in the patch. Duh. Thanks!

tedbow’s picture

tedbow
he/him
Primary language English
Location Ithaca, NY, USA
commented
StatusFileSize
new fixed_notice_d7-1218118-10.patch1.65 KB

Stella, thank for this patch. I changed it slightly to remove unused $children var.
Thanks for any help testing.

tedbow’s picture

tedbow
he/him
Primary language English
Location Ithaca, NY, USA
commented
StatusFileSize
new fixed_notice_d6-1218118-10.patch1.79 KB

Here is the Drupal 6 version of the same patch

tedbow’s picture

tedbow
he/him
Primary language English
Location Ithaca, NY, USA
commented

Just committed these patches to dev 6 & 7 branches. Thanks again stella.

tedbow’s picture

tedbow
he/him
Primary language English
Location Ithaca, NY, USA
commented
Status: Needs review » Fixed

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.