Testing with DRUPAL-4-5. I'm no sure whether I should create a bug report for 4.5.0 or CVS

Looks like 'array_key_exists' should be used instead of '$loaded[$module]' or '$list[$module]'

Patch included

CommentFileSizeAuthor
#3 module_inc_undefined_index_02.patch490 bytesjose reyero
module_inc_undefined_index.patch805 bytesjose reyero

Comments

dries’s picture

dries commented

Committed to the DRUPAL-4-5 branch.

Anonymous’s picture

(not verified) commented

this patch seems to cause:

Warning: array_key_exists(): The second argument should be either an array or an object in /includes/module.inc on line 90

manual reverting the patch make the problem disappear.

jose reyero’s picture

jose reyero commented
StatusFileSize
new module_inc_undefined_index_02.patch490 bytes

Opps! We got rid of a 'Notice' and produced a 'Warning' :-(

Yes, I finally got this Warning, when disabling most of the modules.

automat_svet’s picture

automat_svet commented

got the warning...
Warning: array_key_exists()
module_inc_undefined_index_02.patch won't fix in my case

strangely everything seems ok if I use old version of module.inc v 1.58.2.1 instead of v 1.58.2.5

jose reyero’s picture

jose reyero commented

Got the warning in the same line of module.inc?

killes@www.drop.org’s picture

killes@www.drop.org commented

not needed anymore.

Anonymous’s picture

(not verified) commented