I just created a new Drupal site (drupal-6.10) added cck, draft, flexifield, ahah_response

enabled these modules and went to ?q=admin/content/types/add

Where I get the error message :

* warning: Invalid argument supplied for foreach() in draft.module on line 564.
* warning: Invalid argument supplied for foreach() in draft.module on line 573.

(I've edited the path there)

CommentFileSizeAuthor
#3 draft-add-check-for-array-patch.txt1.35 KBseanburlington

Comments

darren.ferguson’s picture

darren.ferguson commented
Assigned: Unassigned » darren.ferguson
Status: Active » Fixed

Thank you for reporting the issue, this issue is now fixed in the CVS version for the module and will be available on the development branch tonight once the drupal package manager runs. http://drupal.org/cvs?commit=197638 is the revison for this fix

Will make it into the next 6.x-1.6 release for the module.

darren.ferguson’s picture

darren.ferguson commented
Status: Fixed » Closed (fixed)
seanburlington’s picture

seanburlington commented
Assigned: darren.ferguson » Unassigned
Status: Closed (fixed) » Needs work
StatusFileSize
new draft-add-check-for-array-patch.txt1.35 KB

It looks to me like the draft module needs to check if the arrays it want to iterate over are valid - patch to this effect attached

I've only just started looking at this module - but it isn't clear to me why draft is being triggered on the add content page.

seanburlington’s picture

seanburlington commented
Status: Needs work » Closed (fixed)

I didn't mean to re-open this

I hadn't expected such a lightening fast fix and hadn't reloaded the page between initial report and follow up (the status got retained and then saved over your changes)

Thanks :-)

darren.ferguson’s picture

darren.ferguson commented

Not a problem, it popped up on my screen so i quickly looked on my machine, same issue same line numbers, knew what the problem was and fixed :-)

Thank you for the report, helps me make this module as stable as it can be.

Respectfully
Darren Ferguson