confirmCheck method of nsIPrompt interface doesnot work properly under certain condition.

VERIFIED FIXED in mozilla0.9.1

Status

()

Core
Embedding: APIs
VERIFIED FIXED
17 years ago
17 years ago

People

(Reporter: dsirnapalli, Assigned: Conrad Carlen (not reading bugmail))

Tracking

Trunk
mozilla0.9.1
x86
Windows NT
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

(Reporter)

Description

17 years ago
i used the below snippet of code in my javascript testcase.

var checkbox = {value:0};
var boo = promptObj.confirmCheck("confirmcheck dialog",
                                 "Do you want to continue",
                                 "check or uncheck",checkbox);
alert("Checkbox checked: " + checkbox.value);
alert("Confirmation accepted: " + boo);

when the confirm dialog came up i selected checkbox checked and pressed cancel.
the above alert Checkbox checked was showing true(should be false since you 
canceled).
(Reporter)

Comment 1

17 years ago
-- Reassigned as per jud's suggestion.
Assignee: pchen → ccarlen
Component: XP Apps → Embedding APIs
(Assignee)

Comment 2

17 years ago
True. If the dialog is canceled, nothing should happen with the check value.
Status: NEW → ASSIGNED
Target Milestone: --- → mozilla0.9.1
(Assignee)

Comment 3

17 years ago
Created attachment 31903 [details] [diff] [review]
patch so check value is not changed on cancel

Updated

17 years ago
QA Contact: sairuh → dsirnapalli
(Assignee)

Comment 4

17 years ago
Dan, can you review?
Whiteboard: review

Comment 5

17 years ago
Only if I get my inbox cleaned out and/or notice the request. Sorry for the 
delay. Sure, r=me. (PS -- the idl says the checkbox parameter in confirmCheck is 
"out", but the code treats it as "inout". Might want to update that as you did 
for confirmEx. Someday.)
(Assignee)

Comment 6

17 years ago
> the idl says the checkbox parameter in confirmCheck is "out"
It should be inout. That was bug 75993 which is checked in.

CC'ing Alec for sr.  

Comment 7

17 years ago
easy enough! sr=alecf
(Assignee)

Comment 8

17 years ago
Fix checked in.
Status: ASSIGNED → RESOLVED
Last Resolved: 17 years ago
Resolution: --- → FIXED
Whiteboard: review
(Reporter)

Comment 9

17 years ago
-- working as expected. marking bug as verified.
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.