In <mailWindow.js>, "Error: ClearMessagePane is not defined"

NEW
Unassigned

Status

15 years ago
10 years ago

People

(Reporter: sgautherie, Unassigned)

Tracking

({helpwanted})

Trunk
x86
Windows 98
helpwanted

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

(Reporter)

Description

15 years ago
[Mozilla/5.0 (Windows; U; Win98; en-US; rv:1.7b) Gecko/20040316] (W98SE)

Reproducible: 100%

Steps:
1. In 3-pane, open a news message
2. Go offline
3. Open the message in a new window

Result: (window opens with an "offline warning" instead the message, and:)
{{
Error: ClearMessagePane is not defined
Source File: chrome://messenger/content/mailWindow.js
Line: 496
}}

Code is
{{
nsMessagePaneController.prototype =
{
  clearMsgPane: function()
  {
    if (gDBView)
      setTitleFromFolder(gDBView.msgFolder,null);
    else
      setTitleFromFolder(null,null);
    ClearMessagePane();     <-- 496 !
  }
}
}}

Note:
*No error when online !
 *This reminds me of bug 108107 comment 13 !!


[Mozilla/5.0 (Windows; U; Win98; en-US; rv:1.7a) Gecko/20040219] (W98SE)

Already there.
(Reporter)

Updated

15 years ago
Depends on: 108107
(Reporter)

Comment 1

15 years ago
[Mozilla/5.0 (Windows; U; Win98; en-US; rv:1.8a1) Gecko/20040520] (W98SE)

Line number has changed:
{{
Error: ClearMessagePane is not defined
Source File: chrome://messenger/content/mailWindow.js
Line: 512
}}

Another case:
Keep steps 1 and 3, stay online at step 2;
This happens (only) for expired messages too:
{{ (those that display as:)
Error!
newsgroup server responded:No such article number in this group
Perhaps the article has expired
<x@y> (nnn)
Click here to remove all expired articles 
}}

This function is defined at
{{
/mailnews/base/resources/content/msgMail3PaneWindow.js, line 1069 -- function
ClearMessagePane()
}}

It seems like the full-window display tries to call a 3Pane-only function !?
(Reporter)

Updated

15 years ago
Keywords: helpwanted

Comment 2

15 years ago
I think messageWindow.js needs its own version, it already has separate versions
of many functions e.g. folderListener, nsMsgDBViewCommandUpdater.
(Reporter)

Comment 3

15 years ago
Created attachment 151289 [details] [diff] [review]
(Av1-r) <messageWindow.js> (WorkInProgress)

(In reply to comment #2)
> I think messageWindow.js needs its own version,

"Done": fixes the warning.
Then, I need hints about the needed code.

ThunderBird patch will come after Mozilla one...
(Reporter)

Updated

15 years ago
No longer depends on: 108107

Comment 4

15 years ago
Some of the variables and functions are declared in msgMail3PaneWindow.js which
obviously aren't available, although one is set in threadPane.js as part of
nsMsgDBViewCommandUpdater which has its own equivalent in messageWindow.js
Product: Browser → Seamonkey

Updated

14 years ago
Assignee: sspitzer → mail
(Reporter)

Comment 5

13 years ago
Sorry, but your previous comment didn't make it much clearer for me :-/

[Mozilla/5.0 (Windows; U; Win98; en-US; rv:1.8.0.2) Gecko/20060407 SeaMonkey/1.0.1] (nightly) (W98SE)
[Mozilla/5.0 (Windows; U; Win98; en-US; rv:1.8) Gecko/20060406 SeaMonkey/1.1a] (nightly) (W98SE)

{{
Error: ClearMessagePane is not defined
Source File: chrome://messenger/content/mailWindow.js
Line: 525
}}

[Mozilla/5.0 (Windows; U; Win98; en-US; rv:1.9a1) Gecko/20060403 SeaMonkey/1.5a] (nightly) (W98SE)

{{
Error: ClearMessagePane is not defined
Source File: chrome://messenger/content/mailWindow.js
Line: 520
}}

Comment 6

13 years ago
Comment on attachment 151289 [details] [diff] [review]
(Av1-r) <messageWindow.js> (WorkInProgress)

>+  if(gHaveLoadedMessage)
Defined in msgMail3PaneWindow.js but you might be able to use gCurrentMessageUri instead.

>+    gHaveLoadedMessage = false;
Might be able to use ClearMessageSelection() instead.

>+    gCurrentDisplayedMessage = null;
Not used, as far as I can tell.
(Reporter)

Updated

10 years ago
Assignee: mail → nobody
QA Contact: esther → message-display
You need to log in before you can comment on or make changes to this bug.