(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 Ubuntu
Bugs, which is subscribed to Ubuntu.
https://bugs.launchpad.net/bugs/748874

Title:
  plaintext domain gets HTML mail

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

-- 
ubuntu-bugs mailing list
ubuntu-bugs@lists.ubuntu.com
https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs

Reply via email to