# warning: Invalid argument supplied for foreach() in /home/www/sites/all/modules/annotationfield/annotationfield.module on line 137.

should be nested in a check that $field['referenceable_types'] is an array (since it's presumably NULL when first created).

CommentFileSizeAuthor
#5 fix_field_creation.patch1.34 KBschuyler1d

Comments

aaron’s picture

aaron commented

I have only annotationfield and videoannotation installed, which might be a reason. (especially as it seems to not catch my emvideo fields, leading me to believe that part is still in initial development.)

aaron’s picture

aaron commented

when adding content with this field, i subsequently get:

* warning: array_merge() [function.array-merge]: Argument #1 is not an array in /home/www/sites/all/modules/annotationfield/annotationfield.module on line 477.
* warning: array_merge() [function.array-merge]: Argument #2 is not an array in /home/www/sites/all/modules/annotationfield/annotationfield.module on line 477.
* warning: array_merge() [function.array-merge]: Argument #1 is not an array in /home/www/sites/all/modules/annotationfield/annotationfield.module on line 477.
* warning: array_merge() [function.array-merge]: Argument #2 is not an array in /home/www/sites/all/modules/annotationfield/annotationfield.module on line 477.

schuyler1d’s picture

schuyler1d commented
Status: Active » Fixed

this is now fixed in CVS. thanks for taking a look.

schuyler1d’s picture

schuyler1d commented

oh wait. having commit problems. just a sec :-/

schuyler1d’s picture

schuyler1d commented
Status: Fixed » Needs review
StatusFileSize
new fix_field_creation.patch1.34 KB

ugh, while i sort out the CVS issue, here's a patch.

schuyler1d’s picture

schuyler1d commented

well, pretty much everything is in 'initial development' What video format, etc are you using? the download only supports youtube atm--most video sites/etc will not be clippable unless they have JS apis like youtube does. Quicktime will be one that we can migrate, and i'd like to put my jw_player support in the download once i figure out how that would work under GPL.

schuyler1d’s picture

schuyler1d commented
Status: Needs review » Fixed

ok, cvs.drupal.org seems to be up again. It's now committed.

aaron’s picture

aaron commented

thanks, i'll take a look soon

Anonymous’s picture

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

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