Closed
Bug 289223
Opened 19 years ago
Closed 19 years ago
nsMsgStatusFeedback prototype uses myDefaultStatus incorrectly
Categories
(SeaMonkey :: MailNews: Message Display, defect)
SeaMonkey
MailNews: Message Display
Tracking
(Not tracked)
VERIFIED
FIXED
People
(Reporter: iannbugzilla, Assigned: iannbugzilla)
References
()
Details
Attachments
(1 file)
2.93 KB,
patch
|
neil
:
review+
Bienvenu
:
superreview+
asa
:
approval1.8b2+
|
Details | Diff | Splinter Review |
nsMsgStatusFeedback.prototype defines myDefaultStatus as null but in its function uses myDefaultStatus instead of this.myDefaultStatus causing JS errors to appear in the JS console
This patch: * changes myDefaultStatus to this.myDefaultStatus in the relevant pages in both mailnews and TB
Assignee: sspitzer → bugzilla
Status: NEW → ASSIGNED
Attachment #179787 -
Flags: superreview?(mscott)
Attachment #179787 -
Flags: review?(mscott)
Attachment #179787 -
Flags: review?(mscott) → review?(neil.parkwaycc.co.uk)
Updated•19 years ago
|
Attachment #179787 -
Flags: review?(neil.parkwaycc.co.uk) → review+
Attachment #179787 -
Flags: superreview?(mscott) → superreview?(bienvenu)
Updated•19 years ago
|
Attachment #179787 -
Flags: superreview?(bienvenu) → superreview+
Comment on attachment 179787 [details] [diff] [review] Patch v0.1 (Checked in) Requesting approval to checkin in this low risk fix
Attachment #179787 -
Flags: approval1.8b2?
Attachment #179787 -
Flags: approval-aviary1.1a?
Comment 3•19 years ago
|
||
Comment on attachment 179787 [details] [diff] [review] Patch v0.1 (Checked in) a=asa for checkin.
Attachment #179787 -
Flags: approval1.8b2?
Attachment #179787 -
Flags: approval1.8b2+
Attachment #179787 -
Flags: approval-aviary1.1a?
Comment on attachment 179787 [details] [diff] [review] Patch v0.1 (Checked in) Checking in mailnews/base/resources/content/mailWindow.js; /cvsroot/mozilla/mailnews/base/resources/content/mailWindow.js,v <-- mailWindow.js new revision: 1.97; previous revision: 1.96 done Checking in mail/base/content/mailWindow.js; /cvsroot/mozilla/mail/base/content/mailWindow.js,v <-- mailWindow.js new revision: 1.28; previous revision: 1.27 done
Attachment #179787 -
Attachment description: Patch v0.1 → Patch v0.1 (Checked in)
Status: ASSIGNED → RESOLVED
Closed: 19 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•