506510 - Disable filters menu item when no accounts are set up

Status: RESOLVED FIXED

(SeaMonkey :: MailNews: General, defect)

Description

[Magnus Melin [:mkmelin] (away, back Aug 4)]

+++ This bug was initially created as a clone of Bug #67219 +++

When no account are set up, the message filters menu is not disabled as it should be. (Fixed for tb in bug 67219.)

Comment 1

[Jens Hatlak (:InvisibleSmiley)]

Attachment: proposed patch

The "no account" case is probably not much of an issue for SM since you're prompted to create an account each time you open MailNews until an account exists. OTOH the solution is simple so we may just as well fix this.

Comment 2

[Jens Hatlak (:InvisibleSmiley)]

Hmm, saw Good First Bug only after submitting the patch. Now it's too late. Anyway, there have been several calls for participation without much effect in the past so it probably doesn't make much of a difference. Still I'll try to wait a little longer next time... if I notice. ;-)

Comment 3

[Karsten Düsterloh]

Comment on attachment 391643 [details] [diff] [review]
proposed patch

>diff --git a/suite/mailnews/mailWindowOverlay.xul b/suite/mailnews/mailWindowOverlay.xul
>+  <command id="cmd_displayMsgFilters"
>+           oncommand="goDoCommand('cmd_displayMsgFilters');" disabled="true"/>

Move disabled on its own line before the oncommand.

r=me with that.

Comment 4

[Jens Hatlak (:InvisibleSmiley)]

Attachment: patch v2

Comment 5

[[e-mail]]

Comment on attachment 391648 [details] [diff] [review]
patch v2

>+      case "cmd_displayMsgFilters":
>+        return gDBView;
Unfortunately this disables the menuitem when Account Central is showing... this is the Account Central that proudly displays "Manage message filters" ...

Comment 6

[Jens Hatlak (:InvisibleSmiley)]

(In reply to comment #5)
> (From update of attachment 391648 [details] [diff] [review])
> >+      case "cmd_displayMsgFilters":
> >+        return gDBView;
> Unfortunately this disables the menuitem when Account Central is showing...
> this is the Account Central that proudly displays "Manage message filters" ...

You're right, and that also applies to TB.

There's gAccountCentralLoaded (SM only) but it's useless since it's also TRUE if there are no accounts. :-(

The only reliable thing I could find is this:

return gDBView || window.frames["accountCentralPane"].location
                                .href == pref.getComplexValue("mailnews.account_central_page.url",
                                  Components.interfaces.nsIPrefLocalizedString).data;

window.frames["accountCentralPane"].location.href is equal to about:blank in case there are no accounts but just checking against that feels like a bad idea to me.

Magnus, how would you fix this for TB?

Comment 7

[Magnus Melin [:mkmelin] (away, back Aug 4)]

Added a followup patch to bug 67219. 
Re good first bug, no worries, we could add that notation a lot more, and a i think a lot of smaller porting bugs are that almost automatically.

Comment 8

[Jens Hatlak (:InvisibleSmiley)]

Attachment: patch v3 [Checkin: Comment 10]

Thanks Neil and Magnus. This is obviously better. :-)

Comment 9

[[e-mail]]

Comment on attachment 391903 [details] [diff] [review]
patch v3
[Checkin: Comment 10]

You know me, I always like finding Thunderbird bugs ;-)

Comment 10

[Serge Gautherie (:sgautherie)]

Comment on attachment 391903 [details] [diff] [review]
patch v3
[Checkin: Comment 10]


http://hg.mozilla.org/comm-central/rev/8662b1b4e546