If you install this module with Drush, you'll see that your call to system_modules() in _nodecomment_install_type_create isn't defined yet, because you need to load the 'system' module (in addition to the other ones you're already loading).

Very small patch attached :)

CommentFileSizeAuthor
#3 nodecomment_804912.patch390 bytescrea
nodecomment_install.patch723 bytessemperos

Comments

crea’s picture

crea commented
Status: Active » Needs review
crea’s picture

crea commented

How is this supposed to work ? system_modules() is inside system.admin.inc so it won't be available when you load just the system module. Did you try your own patch before submitting ?

crea’s picture

crea commented
StatusFileSize
new nodecomment_804912.patch390 bytes

This one should work.

crea’s picture

crea commented
Status: Needs review » Fixed

Committed

Status: Fixed » Closed (fixed)

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

semperos’s picture

semperos commented

Was fairly sure I tested it before submitting, but your comment makes absolute sense. Thanks for taking the time to correct it and submit it.