547452 - No event fires at completion of folder tree construction

Status: RESOLVED WONTFIX

(Thunderbird :: Mail Window Front End, enhancement)

Description

[Eyal Rozenberg]

Currently, the gFolderTreeView fires a mapRebuild event after creating the top-level folders in the tree. An event is also necessary after _restoreOpenStates() is run and all folders are in place. An example of the need for this is the restoration to the _rowMap array, after reconstruction, of additional fields in various elements. 

(I do this with my remove duplicate messages extension, which marks some folders as containing the dupe originals; the markings are in _mapRow, to avoid long lookups whenever folder properties need to be generated.)

Comment 1

[Ludovic Hirlimann [:Usul]]

Marking wanted as this seems to be needed by this add-on author (other add-ons might want this too).

Comment 2

[Dan Mosedale (:dmosedale, :dmose)]

I don't think this would block if it were the last bug standing, but it would be great to get.  I'm guessing it's a very small amount of work.  asuth/sid0, is my guess likely to be true?

Comment 3

[:aceman]

I was trying to add exactly such event in bug 1490639 :)
But it seems to us the "mapRebuild" event is fired only after the function runs to the end (and tree is fully built) as if the execution does not yield to other JS code (like addons) while the mode() is running so nobody can consume the "mapRebuild" before _restoreOpenStates().
Do you have a different experience?

Comment 4

[Eyal Rozenberg]

Well, I did have a different experience, but that was 9 years ago. I frankly have not looked at this for a long time. I'll check and get back to you within a couple of weeks I hope.

Comment 5

[Wayne Mery (:wsmwk)]

Is this still an issue with version 115?

Comment 6

[John Bieling (:TbSync)]

I have to forward this to Geoff. I can see that even the mapRebuild event does not exist anymore after the 3pane rework.

I do not know if this is still needed at all. It is very easy to keep the additional properties the OP needs to attach to folders in a local data structure and do a single lookup based on the folder path/url.

Comment 7

[Geoff Lankow (:darktrojan)]

I don't think there's any need for this either. If an add-on has a need they can file a new bug.