[Bug 748874]

Ben-bucksch 748874 at bugs.launchpad.net
Mon Oct 31 11:04:07 UTC 2011


(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 Mozilla
Bugs, which is subscribed to Mozilla.
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




More information about the Ubuntu-mozillateam-bugs mailing list