Install Issue - Not compatible with latest Emogrifier Library

I was having a similar problem. I fixed it by editing my copy of emogrifier.php and commenting out the second line:

// namespace Pelago;

Try that and see if it works for you.

Change title, will add patch later...

Here's the patch for this problem, with this patch you don't need to modify any file in the downloaded emogrifier library.

The emogrifier.module strips out all classes and id attributes and just leave the html tag with embedded style attributtes with it, it is because of this code near line 163 - 166 in emogrifier.module

  // Strip unwanted attributes.
  $text = preg_replace(
    array(
      '#(<[^>]*?)\s*class\s*=\s*(["\']).*?\2#si',
      '#(<[^>]*?)\s*id\s*=\s*(["\']).*?\2#si',
    ), '\1', $text);

So if you want to retain the classes and id's in html tag, just comment out these lines, you may want also to read the other similar problem here -> https://drupal.org/node/1842946 and it also have a patch which removes all the regular expression.

I don't now if mine is the same issue.

I don't now where to put the emogrifier.php file, wherver I put it is not seen and i got the message: "The Emogrifier class library was not found."

I've tried this locations and tried to change to lowercase and uppercase letters:
libraries/emogrifier/emogrifier.php
libraries/emogrifier/Emogrifier.php
libraries/Emogrifier/Emogrifier.php

I also tried to downlonad entire master.zip files, unzip and rename folder to emogrifier so the path was:
libraries/emogrifier/Classes/Emogrifier.php

Please help me!

Hi,

• Download the zip file -> https://github.com/jjriv/emogrifier/archive/master.zip inside your sites/all/libraries folder
• Extract the dowloaded zip file and rename the folder "emogrifier-master" to just "emogrifier" of course without a quote
• Apply the patch attached in this issue ->https://drupal.org/files/issues/emogrifier-not-compatible-latest-lib-219...
• That's it

If you don't know how to apply the patch, read on here -> https://drupal.org/patch/apply

Danreb

#11 Worked for me, thanks danreb!

I have reworked this to apply onto #1842946-8: Modify HTML code only by the Emogrifier library (that one is also critical and already rtbc).
Now this has a more verbose message where the library is expected.
Please test and set rtbc so we can get this in!

#13 works fine. Thanks! Setting to rtbc.

Neither of the patches worked on a clean copy of HEAD for me. Attaching a patch that matches the one in #13, but on clean head.

Continue to recieve "The emogrifier class library required by the emogrifier module could not be loaded." message. Have tried all patches including #16 - most have resulted in a WSOD. Emogrifier.php currently exists in /libraries/emogrifier/Classes
Not sure what I am doing wrong. Any help would be very much appreciated.

Hi Andrew,

Emogrifier.php should exists in sites/all/libraries/emogrifier/Classes/Emogrifier.php

Danreb

@axel.rutz:

Please test and set rtbc so we can get this in!

Unfortunately, this module is unmaintained. No activity since 2 years, with critical bugs solved in the issue queue. Are you a new maintainer?

i could commit to do the cleanup but i'm too busy to do maintainership alone.
anyone else to help? (if yes you may also pm me to get the ball rolling as i'm not too often in the issue queue...)

Is there a final resolution for this issue? I don't want to patch anything if it's causing a WSOD, and it's not clear to me from the thread if this has been solved and/or committed to the dev release.

I am getting this error, even though the library is present:

Emogrifier class library      Unavailable
The Emogrifier class library was not found.

Screenshot of the library installed is attached here:

I tried every patch which has been committed here and nothing works.. still the library can't be found. I placed it in /sites/all/libraries/emogrifier/Classes/Emogrifier.php as usual.

The patch in #16 worked perfect for me, the status report is now happy about the installation status of the emogrifier library.

I needed to make one change in the patch at #16 to make this work.

+  @include_once libraries_get_path('emogrifier') . '/Classes/Emogrifier.php';
+  return class_exists('\Pelago\Emogrifier');

It worked when I removed the @ before "include once ..."

I am using PHP 5.3.27. Don't know if that is the reason it didn't work as originally written.

removing the @ symbol will allow that line to throw an error and not ignore it...not sure why that would fix it...

#16 works for me too.

Ok, #16 works for me now too. But the styles are getting removed, and not inlined.

I've rerolled #13 so it applies cleanly to HEAD. Not the right thing to do but until we get a new maintainer...

Nothing works for me.

Why don't you give a "ready-to-use" patched file instead of a patch ?

I'm trying to use "commerce invoice recept" for 2 days, same with "commerce email", they all needs emogrifier and emogrifier is not detected by Drupal...

I tried to apply the patch (#30) with Eclipse and the file just disappear... I'm not a Patch'man and I really need something working.

Is there a solution ?

#16 worked.
But do not mix the two proposals:
#16 does not seem to work if you comment // namespace Pelago

Am I so stupid ?

I tryed to apply #16 and #30 (separately with new module and new database each time) with the command line :
git apply -v '/path-to-the-patch'

My terminal says :

Checking patch emogrifier.install...
Checking patch emogrifier.module...
Applied patch emogrifier.install cleanly.
Applied patch emogrifier.module cleanly.

I checked the files, they look ok.

Then I enable emogrifier module, then I checked my clipboard and :

Emogrifier class library Unavailable
The Emogrifier class library was not found. Usually it is expected in sites/all/libraries/emogrifier/Classes/Emogrifier.php

Could you please give me a working patched zipped module ?

thanks in advance

OK everything is working now !
it was the rights that are inadequate on the folder 'Classes'...

Thanks a lot.

#16 works for me too.

The patch in #16 works for me too. I'd say this needs to be committed but I wonder if this module will ever be updated.

You can report this module as needing a new maintainer and take over maintainership yourself to implement these patches. Pretty easy process really.

Am unable to install 7.x-2.0-beta1.

I am getting error messages for lines 91,96, 101, and 106 in the sites/all/libraries/emogrifier/Classes/Emogrifier.php file.

The errors mostly involve square brackets [] that PHP says are unexpected.

Simply quoting them does not fix the problem.

Wyckham

wdseelig, The error you are experiencing is due to the fact that you have a PHP version before 5.4.0.

Simply copying Emogrifier.php to a new Classes folder as in comment #18 worked for me.