After fixing all Emogrifier issues and using it properly, I believe that the Emogrifier dependency should be re-introduced in the 7.x-2.x branch.

The module isn't really usable if you want to send the order information in an email - inline styles will be ignored by some email clients, and there is no point in providing broken functionality.

I'll think about this a bit more, and then commit the changes.

If anyone has any comments, please let me know.

Thanks!

Comments

aramboyajyan created an issue. See original summary.

aramboyajyan’s picture

Note: this would also stay consistent with what's written on the project page.

rooby’s picture

So far I have only used the 1.x branch (is the 2.x branch stable enough to use yet?) but I would plead that you don't make emogrifier a dependency this time around.

The reason being is that I use the mimemail module on all my sites and it provides the submodule mimemail_compress, which removes the need to also have the emogrifier module.
With almost 90000 installs I think that mimemail users need to be considered.

If there is emogrifier integration I think it should be optional.

aramboyajyan’s picture

Thanks for pointing that out Reuben; I'll have a look at it when I get some time. One possibility would be to allow both modules to be used. The usage of Emogrifier is anyway limited to one single function for processing the HTML, so we can easily have an alternative callback if the mimemail is installed instead.

Just for reference: this might require hook_requirements() in the .install file.

Will keep you posted.

rooby’s picture

Awesome, thanks.

aramboyajyan’s picture

Just tested this multiple times and it works perfectly. Integration with mimemail_compress is actually pretty easy.
Updates will be up shortly on the 7.x-2.x-dev branch.

I didn't add a strict dependency for either of them, because the module still works, though with limited success across mail hosts.

I might also implement hook_requirements() to add some messages for administrators about Emogrifier and Mimail Compress modules.

Thanks again for bringing this up.

  • aramboyajyan committed c2dd06b on 7.x-2.x
    Issue #2694105 by aramboyajyan: Return the dependency on Emogrifier?
    
aramboyajyan’s picture

Status: Active » Fixed
aramboyajyan’s picture

Title: Return the dependency on Emogrifier? » Emogrifier and alternative ways of inlining the CSS

Updating the title.

Status: Fixed » Closed (fixed)

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