Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
Always end up with WSOD and the following error meassage in log when trying to install the module.
PHP Fatal error: Call to undefined method stdClass::save() in .../sites/all/modules/commerce_email/commerce_email.install on line 12
Version 1 seems to install ok.
Comment | File | Size | Author |
---|---|---|---|
#4 | commerce-email-rule-validation-2019773-4.patch | 1.67 KB | seemas |
#3 | commerce-email-rule-validation-2019773-3.patch | 2.03 KB | generalconsensus |
Comments
Comment #1
checker CreditAttribution: checker commentedThis happen if commerce checkout module is not ready during installation of commerce email.
I get the same error if i try to install this module as a dependency of another module.
If i install only this module and commerce checkout is already installed i have no problems.
Comment #2
ippy CreditAttribution: ippy commentedSame issue when combined with other modules (dependencies) to be enabled:
The following extensions will be enabled: commerce_invoice_email, htmlmail, variable_email, commerce_email
Do you really want to continue? (y/n): y
Drush command terminated abnormally due to an unrecoverable error. [error]
Error: Call to undefined method stdClass::save() in
/var/www/vhosts/******************/httpdocs_**/sites/all/modules/commerce_email/commerce_email.install,line 12
Line 12 is the last line of this function:
function commerce_email_enable() {
However, visiting the module page, enabling and saving and then there is no error. Sounds suspiciously similar to this historic issue: https://drupal.org/node/1651034 ?
No time to dig just now (entity/empty or rule naming issue?), but may have a chance to investigate further at some point - and certainly happy to test any patches etc. I can see that this module is not being heavily maintained (and looking for co-maintainer), so expect to DIY :)
Comment #3
generalconsensus CreditAttribution: generalconsensus commentedI've created a basic patch which validates the rules before it tries to set it inactive on enable or disable. Please review
Comment #4
seemas CreditAttribution: seemas at Adapt commentedPrevious patch failed to apply so providing new one for 2.x-dev version.
Comment #5
tijsdeboeckPatch #4 works perfectly!
Comment #6
nicjasno CreditAttribution: nicjasno commentedPatch #4 also worked for me. Thanks a lot!
Comment #7
bluecobalt CreditAttribution: bluecobalt commentedPatch #4 works. Thanks!
Comment #8
vood002 CreditAttribution: vood002 commented#4 worked for me as well, thank you!
Comment #9
rszrama CreditAttribution: rszrama at Centarro commentedReviewing now.
Comment #10
rszrama CreditAttribution: rszrama at Centarro commentedHad to use patch but got it to apply. Normally would use a !empty() check on a string, but oh well ... committing as is.