Closed Bug 334053 Opened 15 years ago Closed 7 years ago

In prefs about reply headers, some sequences beginning with % give anomalous results

Categories

(MailNews Core :: Composition, defect)

defect
Not set
minor

Tracking

(Not tracked)

RESOLVED FIXED
Thunderbird 31.0

People

(Reporter: intendentedelleacque, Assigned: aceman)

References

Details

(Whiteboard: [fixed by bug 995797])

User-Agent:       Mozilla/5.0 (X11; U; Linux i686; it; rv:1.8.0.1) Gecko/20060124 Firefox/1.5.0.1
Build Identifier: Mozilla/5.0 (X11; U; Linux i686; it; rv:1.8.0.1) Gecko/20060124 Firefox/1.5.0.1

If you set the pref, for example, "mailnews.reply_header_authorwrote" equal to "%d", when you reply you'll have in the header a random (?) hexadecimal (?) number, that seems to change everytime. 
I found many sequences that give similar results, surely %d %e %f %g %o, while other sequences have a regoular behaviour (for ex. %a %y).
A particular case is the sequence %S that gives a long series of asian chars.
The same happens with "mailnews.reply_header_ondate".


Reproducible: Always

Steps to Reproduce:
1. set the pref "mailnews.reply_header_authorwrote" equal to "%d"
2. reply a message

Actual Results:  
1. you'll have in the reply header something like 165262100

Expected Results:  
1. you should have "%d"

Tested on TB 1.5, on Linux and win XP
OS: Linux → All
Version: unspecified → 1.5
This may not be ideal, but it makes sense.  Those two headers normally use %s, which indicates that the %s should be subsitituted by the authorname or date, *as a string* -- this is a standard format in programming.  By the same token, those other sequences also work as substitutions, but as numerical values.

If you really want to see  %d   in the string, set the pref to  %%d   -- double-% is substituted by a single %, always.
Status: UNCONFIRMED → RESOLVED
Closed: 14 years ago
Resolution: --- → WONTFIX
Yes, those % sequences are special for nsTextFormatter::smprintf which processes those strings.
I plan to remove nsTextFormatter::smprintf processing in bug 995797 so %d (etc) should stay untouched.

Please close this as FIXED when bug 995797 is fixed with removal of smprintf.
Status: RESOLVED → REOPENED
Depends on: 995797
Ever confirmed: true
Resolution: WONTFIX → ---
Fixed by bug 995797.
Status: REOPENED → RESOLVED
Closed: 14 years ago7 years ago
Resolution: --- → FIXED
Assignee: mscott → acelists
Component: General → Composition
Product: Thunderbird → MailNews Core
Hardware: x86 → All
Target Milestone: --- → Thunderbird 31.0
Version: 1.5 → Trunk
Whiteboard: [fixed by bug 995797]
You need to log in before you can comment on or make changes to this bug.