Cron fails when subscriptions_mail is run in 1.3. Reverted to 1.2 and its ok. Error in logs: mod_fcgid: stderr: PHP Fatal error: Class name must be a valid object or a string in includes/common.inc on line 7999

CommentFileSizeAuthor
#7 subscriptions-no_taxonomy_crash-2800885-7.patch677 bytessalvis
#6 fix_fatal_error_if_taxonomy_disabled.patch824 byteskosar
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

castelar created an issue. See original summary.

salvis’s picture

salvis
he/his
CreditAttribution: salvis commented
Status: Active » Postponed (maintainer needs more info)

We have 700+ sites running 1.3 — this must be an issue that is specific to your site and needs to be debugged locally.

num37’s picture

num37 CreditAttribution: num37 commented

I just ran into this exact error. The solution for me seems to be to make sure the Taxonomy module is enabled.

I haven't had a chance to dig deeper than that yet.

sonvir249’s picture

sonvir249 CreditAttribution: sonvir249 as a volunteer and at QED42 commented
Status: Postponed (maintainer needs more info) » Closed (won't fix)

Hi @num37,

If this is not a bug then this issue be closed and marked as Closed (won't fix). Closing and marking as Closed (won't fix).

salvis’s picture

salvis
he/his
CreditAttribution: salvis commented
Status: Closed (won't fix) » Postponed (maintainer needs more info)

@num37:

Is it just Taxonomy that's needed or Taxonomy Subscriptions?

Is this problem still in the -dev version?

kosar’s picture

kosar CreditAttribution: kosar commented
FileSize
fix_fatal_error_if_taxonomy_disabled.patch824 bytes

The problem occurs when the taxonomy module is off. The problem occurs in the subscriptions_mail.cron.inc file on line 164 when calling entity_load. I do not use taxonomy, so I think it's necessary to add a check for the existence of the taxonomy.

PS the problem is also relevant for the dev version.

salvis’s picture

salvis
he/his
CreditAttribution: salvis commented
Version: 7.x-1.3 » 7.x-1.x-dev
FileSize
subscriptions-no_taxonomy_crash-2800885-7.patch677 bytes

Thank you for your patch, kosar!

Using module_exists() to check for the presence of a module is clearer.

  • salvis committed a655d10 on 7.x-1.x 
    Issue #2800885 by salvis: Cron fails when running Subscriptions Mail...
salvis’s picture

salvis
he/his
CreditAttribution: salvis commented
Status: Postponed (maintainer needs more info) » Fixed

Thanks all for helping to track this down.

Status: Fixed » Closed (fixed)

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