hi,

ever since i upgraded to the latest version, hebrew characters come out like this -

תשומת ×œ×‘×›× ×›×™ בשל מרתון תל × ×‘×™×‘ ×œ× ×™×ª×§×™×™× ×¡×ž×™× ×¨ ×‘×™×•× 30.3.2012.

any ideas?

CommentFileSizeAuthor
#7 1508524-non_latin_characters_appear_scrumbled-7.patch1.21 KBParisLiakos

Comments

ParisLiakos’s picture

ParisLiakos commented
Status: Active » Closed (duplicate)

This is fixed in dev version.Try the dev version and if still doesnt work reopen this:)

IdanC’s picture

IdanC commented
Status: Closed (duplicate) » Active

i'm sorry to inform that the bug still very much exist.

i've noticed though that when i send the test email it works fine. Only when i send to the whole mailing list it gets scrumbled.

thanks.

ParisLiakos’s picture

ParisLiakos commented

do you have latest dev installed?

IdanC’s picture

IdanC commented
Version: 7.x-1.0-beta8 » 7.x-1.x-dev

yes. and i do enjoy the rest of the fixes, this is the only "fly in the soup"

ParisLiakos’s picture

ParisLiakos commented

plz give me a chunk of hebrew text to test again

IdanC’s picture

IdanC commented

my pleasure -

רוצים לתת לבן או בת הזוג חיבוק אחרי סרט מדכא? יש לכך סיבה טובה. מחקר חדש מצא שצפייה בסרטים רומנטיים שנגמרים רע גורמת לנו לחשוב על האהובים והיקרים לנו, להעריך אותם ואת הטוב שבחיינו ולהרגיש מאושרים יותר

taken from here

ParisLiakos’s picture

ParisLiakos commented
Status: Active » Needs review
StatusFileSize
new 1508524-non_latin_characters_appear_scrumbled-7.patch1.21 KB

Can you try this patch?

Status: Needs review » Needs work

The last submitted patch, 1508524-non_latin_characters_appear_scrumbled-7.patch, failed testing.

ParisLiakos’s picture

ParisLiakos commented
Status: Needs work » Needs review

#7: 1508524-non_latin_characters_appear_scrumbled-7.patch queued for re-testing.

ParisLiakos’s picture

ParisLiakos commented
Status: Needs review » Fixed

fix accidentally commited..hope it doesnt break anything else

olbion’s picture

olbion commented
Priority: Normal » Critical
Status: Fixed » Active

Just tried this and it's not working. I'm using Chinese characters (like 你好). They all get garbled. This used to work but now it clearly doesn't. Worse, it works when you send out a test email (so I thought it was good to go). But the mass email from cron gets it wrong. So I just sent out a very ugly email to all subscribers.

Tried both beta and dev releases and same result.

ParisLiakos’s picture

ParisLiakos commented

i m sorry olbion:/
which release where you using before the text gets scrumbled?

d.sibaud’s picture

d.sibaud commented

I applied the above patch and nothing happened, after I took a look into the code and I commented out the following line inside includes/newsletter.mail.inc in the function addNewsletterUrlQuery() and now it's working:

      ....
      // Converts back from hmtl entities
      //return html_entity_decode($dom->saveHTML());
    }
    return $message['body'];
  }
ParisLiakos’s picture

ParisLiakos commented

seems DomDocument solution for links is problematic..i consider removing it and replace it with something else..

pio.fernandes’s picture

pio.fernandes commented

Yea, I also did #13, and It works now. ÃãõÕçÇ etc now appear.

ParisLiakos’s picture

ParisLiakos commented

see comment here
http://drupal.org/node/1761148#comment-6550984