75055 - search folder picker can fire onChooseFolder() twice

Status: RESOLVED DUPLICATE of bug 68174

(SeaMonkey :: MailNews: Message Display, defect)

Description

[(not reading, please use [e-mail] instead)]

in our search js, we'll fail because folderURI is "".

gCurrentFolder = RDF.GetResource(folderURI).QueryInterface(nsIMsgFolder);


our template code and oncommand handers for the folder picker are calling
onChooseFolder() twice, the second time event.id is ""

I think we're firing our oncommand handler twice.  the plan is to move it out of
the generated content, right?

Comment 1

[(not reading, please use [e-mail] instead)]

I've got a crude patch to fix the problem until the real issue is fixed

Comment 2

[(not reading, please use [e-mail] instead)]

Attachment: fix until the real problem is fixed.

Comment 3

[(not reading, please use [e-mail] instead)]

my safety checking fix is in.

changing subject.  now this bug can be marked as a duplicate of the bug where
you (or gayatrib?) are going to move the oncommand handler out of the generated
xul.

Comment 4

[(not reading, please use [e-mail] instead)]

mass re-assign.

Comment 5

[Wayne Mery (:wsmwk)]

spitzer in comment #3:
> ... now this bug can be marked as a duplicate of the bug where
> you (or gayatrib?) are going to move the oncommand handler out of the generated xul.

Seth, I was not able to find the bug in question.

Comment 6

[(not reading, please use [e-mail] instead)]

> Seth, I was not able to find the bug in question.

hey wayne, I think I meant bug #68174, but at this point (6+ years later), your guess / bugzilla query is as good as mine.