Closed (fixed)
Project:
Commerce Email
Version:
7.x-2.x-dev
Component:
Code
Priority:
Normal
Category:
Bug report
Assigned:
Unassigned
Reporter:
Created:
14 Jun 2013 at 15:15 UTC
Updated:
16 Jun 2020 at 18:54 UTC
Jump to comment: Most recent, Most recent file
Comments
Comment #1
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 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 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 commentedPrevious patch failed to apply so providing new one for 2.x-dev version.
Comment #5
tijsdeboeckPatch #4 works perfectly!
Comment #6
nicjasno commentedPatch #4 also worked for me. Thanks a lot!
Comment #7
bluecobalt commentedPatch #4 works. Thanks!
Comment #8
vood002 commented#4 worked for me as well, thank you!
Comment #9
rszrama commentedReviewing now.
Comment #10
rszrama commentedHad to use patch but got it to apply. Normally would use a !empty() check on a string, but oh well ... committing as is.