desktop-packages team mailing list archive

[Ben-bucksch <748874@xxxxxxxxxxxxxxxxxx>]

(In reply to WADA from comment #8)
> (C) If domainpart of mail address(string after @) is substring of
> mailnews.plaintext_domains or mailnews.html_domains, mail&news looks to
> consider that the domainpart is registered as plaintext_domain or
> html_domain.   
> > 4913         PRInt32 atPos = recipient.mEmail.FindChar('@');
> > 4914         if (atPos >= 0)
> > 4915         {
> > 4916           domain = Substring(recipient.mEmail, atPos + 1);
> > 4917           if (CaseInsensitiveFindInReadable(domain, plaintextDomains))
> > 4918             recipient.mPreferFormat = nsIAbPreferMailFormat::plaintext;
> > 4919           else
> > 4920             if (CaseInsensitiveFindInReadable(domain, htmlDomains))
> > 4921               recipient.mPreferFormat = nsIAbPreferMailFormat::html;
> > 4922         }
> 
> Ben Bucksch, is my guess right?

Indeed, that's precisely the bug, nail on head. This comparison is just
wrong.

We need to split the string at ",", then compare the right-most part,
and only get a hit, if the email address domain string is equal or
longer than the pref domain string, not the other way around as now.

Ben

-- 
You received this bug notification because you are a member of Desktop
Packages, which is subscribed to thunderbird in Ubuntu.
https://bugs.launchpad.net/bugs/748874

Title:
  plaintext domain gets HTML mail

Status in Mozilla Thunderbird Mail and News:
  Confirmed
Status in “thunderbird” package in Ubuntu:
  New

Bug description:
  Binary package hint: thunderbird

  Mozilla/5.0 (X11; U; Linux i686; en-US; rv:1.9.2.14) Gecko/20110223
  Thunderbird/3.1.8 on lucid

  I have user_pref("mailnews.plaintext_domains",
  "docomo.ne.jp,debian.org"); in my prefs.js.  Yet, whenever I write a
  mail to @debian.org or @bugs.debian.org it is sent as HTML mail.  Grr.

To manage notifications about this bug go to:
https://bugs.launchpad.net/thunderbird/+bug/748874/+subscriptions