If I don't have any content types set to be gallery or image content types, I get the following errors:

warning: array_merge() [function.array-merge]: Argument #1 is not an array in /var/www/dev/sites/all/modules/node_gallery/node_gallery.inc on line 61.

As well as all sorts of messages like this in my watchdog:

in_array() [function.in-array]: Wrong datatype for second argument in /var/www/dev/sites/all/modules/node_gallery/node_gallery.module on line 498.

I believe the attached patch takes care of these issues.

CommentFileSizeAuthor
#1 node_gallery.inc_.patch683 bytesshaisachs

Comments

shaisachs’s picture

shaisachs commented
Status: Active » Needs review
StatusFileSize
new node_gallery.inc_.patch683 bytes

Whoops! Somehow that file didn't attach.

justintime’s picture

justintime commented
Status: Needs review » Fixed

Thanks - committed your patch into CVS!

Status: Fixed » Closed (fixed)

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

justintime’s picture

justintime commented

This patch actually broke some contrib modules (i18n specifically), the fix was to move the init of the array down by a line - see #801012: Conflict with i18n: Sorting/Managing Galleries empty - uploads failing (Dev broken?)