I just tried out the new 4.7 release. What can I say? It's fantastic!

I do get a warning when viewing a diff:

warning: Invalid argument supplied for foreach() in /Library/WebServer/Documents/modules/diff/cck.inc on line 17.

I do have CCK enabled, but this warning appears on a regular "page" type node.

thanks,
Dylan

CommentFileSizeAuthor
#1 cck_null_check.patch3.74 KBrötzi

Comments

rötzi’s picture

rötzi commented
Assigned: Unassigned » rötzi
Status: Active » Needs review
StatusFileSize
new cck_null_check.patch3.74 KB

Try this patch.

It adds a check if cck fields are present in the node before trying to iterate through them.

grendzy’s picture

grendzy commented

Your patch fixed the warning, thank you!

rötzi’s picture

rötzi commented
Status: Needs review » Fixed

patch commited to DRUPAL-5 and DRUPAL-4-7 branch in cvs

moshe weitzman’s picture

moshe weitzman commented

the way i do development is to edit HEAD and then when i am satisfied i copy to DRUPAL-5 and commit there. So please remember to commit to HEAD. I have been considering using DRUPAL-5 for development and forgetting about HEAD but not quite used to that yet.

Anonymous’s picture

(not verified) commented
Status: Fixed » Closed (fixed)