580143 - customDBHeaders has no effect on nsIMsgDBHdr with NNTP

Status: RESOLVED FIXED

(MailNews Core :: Backend, defect)

Description

[Marcin Szewczyk (:wodny)]

User-Agent:       Mozilla/5.0 (X11; U; Linux i686; en-US; rv:1.9.1.10) Gecko/20100623 Iceweasel/3.5.10 (like Firefox/3.5.10)
Build Identifier: Mozilla/5.0 (X11; U; Linux i686; en-US; rv:1.9.1.10) Gecko/20100619 Lightning/1.0b1 Icedove/3.0.5

customDBHeaders has no effect on nsIMsgDBHdr with NNTP and I suppose this causes for example gMessageDisplay.displayedMessage.getStringProperty('face') to fail, even if there is the "face" header in a USENET message.

Reproducible: Always

Steps to Reproduce:
1. Install the Display Contact Photo add-on
2. Turn on Face header support in the add-on's preferences
3. Watch it not work with NNTP folders
Actual Results:  
No Face header detected in a NNTP folder.

Expected Results:  
Detected Face header.

Quick and dirty debug method:

var data = "Winners:\n";
var en = gMessageDisplay.displayedMessage.propertyEnumerator;
while(en.hasMore())
  data += en.getNext()+"\n";
alert(data);  

I suggest adding it to getPhoto() in Display Contact Photo add-on

Comment 1

[Samuel Müller]

(In reply to comment #0)
> User-Agent:       Mozilla/5.0 (X11; U; Linux i686; en-US; rv:1.9.1.10)
> Gecko/20100623 Iceweasel/3.5.10 (like Firefox/3.5.10)
> Build Identifier: Mozilla/5.0 (X11; U; Linux i686; en-US; rv:1.9.1.10)
> Gecko/20100619 Lightning/1.0b1 Icedove/3.0.5
> 
> customDBHeaders has no effect on nsIMsgDBHdr with NNTP and I suppose this
> causes for example gMessageDisplay.displayedMessage.getStringProperty('face')
> to fail, even if there is the "face" header in a USENET message.
> 
> Reproducible: Always
> 
> Steps to Reproduce:
> 1. Install the Display Contact Photo add-on
> 2. Turn on Face header support in the add-on's preferences
> 3. Watch it not work with NNTP folders
> Actual Results:  
> No Face header detected in a NNTP folder.

This bug has been fixed in version 1.2b2 of the Display Contact Photo add-on thanks to the patch of Marcin.

Comment 2

[Joshua Cranmer [:jcranmer]]

RESO FIX per comment 1