443035 - Some Address Book functions are disabled when the abResultsTree has focus

Status: RESOLVED FIXED

(MailNews Core :: Address Book, defect)

Description

[Josh Geenen (:pi)]

User-Agent:       Mozilla/5.0 (X11; U; Linux i686 (x86_64); en-US; rv:1.9) Gecko/2008052912 Firefox/3.0
Build Identifier: version 3.0a2pre (2008070114)

I just noticed this today after checking out and rebuilding Thunderbird.

When I click on a card in the abResultsTree, New Card List and Delete are grayed out and I am unable to delete a card through the GUI or by using the delete button.  I can delete cards through a script.  Additionally, Properties in the context menu for cards does not work.

There were no errors in the Error Console.

Reproducible: Always

Steps to Reproduce:
1. Open Thunderbird (the lastest nightly) and the Address Book
2. Select a card
Actual Results:  
New Card List and Delete are grayed out and Properties in the context menu doesn't work.

Expected Results:  
The three commands should work as normal.

In addressbook.js:
function CommandUpdate_AddressBook()
{
  goUpdateCommand('cmd_delete');
  goUpdateCommand('button_delete');
  goUpdateCommand('cmd_newlist');
}
The problems I noticed were with these three commands.

Comment 1

[Mark Banner (:standard8)]

Attachment: [checked in] The fix

The problem is the command controller for the results pane is never getting registered. In bug 437619 I moved the gAbResultsTree initialisation from OnLoad functions to SetAbView(). This caused gAbResultsTree to be null at the SetupAbCommandUpdateHandlers stage.

As we never clear gAbResultsTree, we can just do the addition of the command handler once we get it in SetAbView.

Comment 2

[David :Bienvenu]

Comment on attachment 327689 [details] [diff] [review]
[checked in] The fix

one nit - SetupAbCommandUpdateHandlers only really initializes the DirPaneController, i.e., a single handler, so its name might be changed to SetupDirPaneController or you could even inline that code in the three places it's used, across TB and SM.

Comment 3

[Mark Banner (:standard8)]

Comment on attachment 327689 [details] [diff] [review]
[checked in] The fix

I've checked this in without addressing the nits - I'll do those in a follow up patch.

Comment 4

[Mark Banner (:standard8)]

Attachment: [checked in] Follow-up - remove SetupAbCommandUpdateHandlers

I took a look at the SetupAbCommandUpdateHandlers function calls. The addressbook-panel one isn't actually required (as the panel doesn't have a dir tree), and the other two calls can just be replaced in-line.

I guess at some stage, it'll be a good idea to centralise the dir pane functions into one file as well, but that's definitely another bug ;-)

Comment 5

[David :Bienvenu]

Comment on attachment 327788 [details] [diff] [review]
[checked in] Follow-up - remove SetupAbCommandUpdateHandlers

thx, looks good.