Closed Bug 289223 Opened 19 years ago Closed 19 years ago

nsMsgStatusFeedback prototype uses myDefaultStatus incorrectly

Categories

(SeaMonkey :: MailNews: Message Display, defect)

defect
Not set
normal

Tracking

(Not tracked)

VERIFIED FIXED

People

(Reporter: iannbugzilla, Assigned: iannbugzilla)

References

()

Details

Attachments

(1 file)

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)
Attachment #179787 - Flags: review?(neil.parkwaycc.co.uk) → review+
Attachment #179787 - Flags: superreview?(mscott) → superreview?(bienvenu)
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 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
Verified FIXED via code inspection through LXR.
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: