441270 - IMAP Server Settings pane broken in SeaMonkey: js error on console

Status: RESOLVED DUPLICATE of bug 438647

(SeaMonkey :: MailNews: Account Configuration, defect)

Description

[Boying Lu]

Can't login IMAP server thourgh SSL

Comment 1

[Boying Lu]

Attachment: patch

Comment 2

[David :Bienvenu]

Is that the right patch? And why isn't there a trashPopup element?

Comment 3

[Boying Lu]

(In reply to comment #2)
> Is that the right patch? And why isn't there a trashPopup element?
> 
Yes.  the trashPopup is there but the "_teardown" and "_ensureInitialized" is not defined in seamonkey.  These two methods are defined in TB ( mail/base/content/mail-folder-bindings.xml)

Comment 4

[David :Bienvenu]

so is this bug really that you can't pick SSL in the server settings, not that LOGON with SSL doesn't work?

Comment 5

[Boying Lu]

(In reply to comment #4)
> so is this bug really that you can't pick SSL in the server settings, not that
> LOGON with SSL doesn't work?
> 
Yes.

Comment 6

[David :Bienvenu]

changing summary - in future, it might also help to put the warning that appears on the console in the bug. Thx for fixing this.

Comment 7

[Boying Lu]

Can we check in the patch now or wait for "sr"?

Comment 8

[Karsten Düsterloh]

I don't think that this is the right fix.
Why does a shared file in /mailnews have a dependency on /mail?
Joey?

Comment 9

[Joey Minta]

The mail-folder-bindings.xml file is moving to mailnews shortly.  There's an open bug on it, though I don't have the number available.  Once that file moves, these functions will work (so the patch should be unnecessary), and this bug should disappear.

Comment 10

[[e-mail]]

Does the patch in bug 438647 fix this? (Sorry for not reviewing it yet.)

Comment 11

[Tobias Fischer]

(In reply to comment #10)
> Does the patch in bug 438647 fix this? (Sorry for not reviewing it yet.)
> 
Well, I hope so, according to the Messages in Error-Console. 

But when I noticed today, that it was not possible to apply any changes in the MN Account-Manager in SeaMonkey Trunk (have tested to change settig for "Check for new Messages at Startup"), I can cut down the regression window to 2008-05-31 02 (works) and 2008-06-01 01 (works not), so i suspected the checkin for Bug 436692 too for breaking the AM, but I am not sure about that.

Comment 12

[Boying Lu]

(In reply to comment #10)
> Does the patch in bug 438647 fix this? (Sorry for not reviewing it yet.)
> 

I tried the last patch post at bug 438647. It doesn't work because mail-folder-bindings.xml hasn't been moved to mailnews/ yet.

Does anyone know when it will be moved to mailnews?

Comment 13

[Serge Gautherie (:sgautherie)]

[Mozilla/5.0 (Windows; U; Windows NT 5.0; en-US; rv:1.9.0.1pre) Gecko/2008062515 SeaMonkey/2.0a1pre] (SEA-WIN32-TBOX-trunk) (W2Ksp4)

The error, on opening the pane:
{{
Error: trashPopup._teardown is not a function
Source File: chrome://messenger/content/am-server.js
Line: 282
}}

Comment 14

[Serge Gautherie (:sgautherie)]

[Mozilla/5.0 (Windows; U; Windows NT 5.0; en-US; rv:1.9.0.2pre) Gecko/2008071202 SeaMonkey/2.0a1pre] (nightly) (W2Ksp4)

R.Duplicate

Comment 15

[Mark Banner (:standard8)]

Comment on attachment 326273 [details] [diff] [review]
patch

This bug was marked as a duplicate of another. Therefore clearing request. Boying - if you feel this is still necessary please reopen and re-request.