hi,

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

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

any ideas?

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

ParisLiakos’s picture

Status: Active » Closed (duplicate)

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

IdanC’s picture

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

do you have latest dev installed?

IdanC’s picture

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

plz give me a chunk of hebrew text to test again

IdanC’s picture

my pleasure -

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

taken from here

ParisLiakos’s picture

Status: Active » Needs review
FileSize
1.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

Status: Needs work » Needs review
ParisLiakos’s picture

Status: Needs review » Fixed

fix accidentally commited..hope it doesnt break anything else

olbion’s picture

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

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

d.sibaud’s picture

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

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

pio.fernandes’s picture

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

ParisLiakos’s picture