293190 - [MAS v1.8b2-xxxx] In <AccountManager.js>, "Warning: reference to undefined property formElement.selectedItem.value"

Status: VERIFIED FIXED

(SeaMonkey :: MailNews: Account Configuration, defect)

Description

[Serge Gautherie (:sgautherie)]

[Mozilla/5.0 (Windows; U; Win98; en-US; rv:1.7.7) Gecko/20050414] (release) (W98SE)
[Mozilla/5.0 (Windows; U; Win98; en-US; rv:1.8b) Gecko/20050217] (<-- 1.8b1 !)
(W98SE)

No bug.

[Mozilla/5.0 (Windows; U; Win98; en-US; rv:1.8b2) Gecko/20050506] (nightly) (W98SE)

You have to have multiple accounts,
and it seems not to happen for the first of its kind (mail OR news).

1. Start MailNews
2. Edit > M. & N. Account Settings...
3. Select almost any root account panel (the one with '-' on the left)
4. Click on any other panel in the list

{{
Warning: reference to undefined property formElement.selectedItem.value
Source File: chrome://messenger/content/AccountManager.js
Line: 990
}}

Comment 1

[[e-mail]]

Looks like a funky XBL issue. As a workaround we don't actually need that block
any more; menulists have always had a value and radiogroups now have one too.

Comment 2

[Serge Gautherie (:sgautherie)]

[Mozilla/5.0 (Windows; U; Win98; en-US; rv:1.8b2) Gecko/20050705 SeaMonkey/1.0a]
(nightly) (W98SE)

Warning still there.

Comment 3

[Ian Neal]

Attachment: Simple patch v0.1 (Checked in)

This patch:
* Removes block as suggested by Neil so it falls through to returning
formElement.value

Could you try this and see if fixes your problem?

Comment 4

[Serge Gautherie (:sgautherie)]

(In reply to comment #3)
> Created an attachment (id=188765) [edit]
> Simple patch v0.1
> 
> Could you try this and see if fixes your problem?

[Mozilla/5.0 (Windows; U; Win98; en-US; rv:1.8b3) Gecko/20050708 SeaMonkey/1.0a]
(nightly) (W98SE)

It does :-)

Comment 5

[Serge Gautherie (:sgautherie)]

Comment on attachment 188765 [details] [diff] [review]
Simple patch v0.1 (Checked in)

'approval1.8b4=?': (SeaMonkey only !?)
Trivial U.I. code cleanup, no risk.

Comment 6

[Ian Neal]

Comment on attachment 188765 [details] [diff] [review]
Simple patch v0.1 (Checked in)

Checking in
AccountManager.js;
new revision: 1.114; previous revision: 1.113
done

Comment 7

[Serge Gautherie (:sgautherie)]

[Mozilla/5.0 (Windows; U; Win98; en-US; rv:1.8b4) Gecko/20050729 SeaMonkey/1.0a]
(nightly) (W98SE)

V.Fixed.