Good evening

If Cron-Notify is enabled, then I get the following Error if I start Cron manually:

Fatal error: Call to undefined function: comment_num_all() in
drupalt/sites/all/modules/subscriptions/subscriptions.module on line 802

Kind regards,
Thomas

Comments

schittli’s picture

schittli commented

I've found the Problem: I have disabled the comment module...
The tracker.module had the same Problem:
http://drupal.org/node/9031
And maybe you can use the same patch-logic:
http://drupal.org/files/issues/num_comments.patch

???

Kind regards,
Thomas

crmanski’s picture

crmanski commented

I was getting blank pages when submitting any content. The contect would get posted, but the page was blank. I noticed the same type of errors...
Call to undefined function: comment_num_all() in /drupal-5.x-dev/sites/all/modules/subscriptions/subscriptions.module on line 802
Call to undefined function: _comment_get_display_setting() in /drupal-5.x-dev/sites/all/modules/subscriptions/subscriptions.module on line 770

Enabling the comment module stopped the blank pages.

jiangxijay’s picture

jiangxijay commented

I have disabled the comment module and am using nodecomments instead.

I just started getting this error from the subscription module:

Call to undefined function comment_num_all() in
..... modules/subscriptions/subscriptions.module on line 802

It's here:

$comments_num = comment_num_all($nid) + 1; // +1 for comment being added now

I disabled subscriptions.module and the error went away. (I'd previously disabled tracker.)

Can dependency on the comments module be removed or, at least, show up in the modules configuration page, so we can be warned in advance?

bdragon’s picture

bdragon commented
Status: Active » Closed (fixed)

5.x-2.0 is a complete rewrite, therefore any bugs in it are new. All prior releases will be obsolete when 5.x-2.0 final is out. Closing references to bugs in prior releases for this reason. Please help test 5.x-2.0 betas. Thank you! (Please reopen if this is also a problem in the new code, one of the focuses of the rewrite was to split subscriptions out into logical submodules to prevent dependency issues.)