id	summary	reporter	owner	description	type	status	priority	milestone	component	version	resolution	keywords	cc
8771	getDialogValue in docprops plugin failing to release event handlers	NicHolt		"Steps to recreate: a page in the docprops dialog with two elements that invoke a child dialog - e.g. colorpicker (in my case a treeview dialog.

1. Click to open the child dialog for one element. 

2. Cancel.

3. Click to open the child dialog for the other element. 

4. Make a selection in the child dialog.

5. Click OK.

Expected results:

The first element should be unchanged, and the second should be updated with the value from the child dialog.

Actual result:

BOTH elements are updated with the value from the second invocation of the child dialog.

Explanation: releaseHandlers is not being called in the correct context, so when OK is clicked, there are TWO OK event handlers bound to the dialog, one for the field for which OK was clicked, one ofr the field which was canceled.

The following code explicitly establishes the context both for the event handlers and for releaseHandlers, rather than relying on ""this"".


{{{
		var onOk = function()
		{
			var dlg = editor._.storedDialogs[dialogName];  // Changed by Nic to fix failure to releaseHandlers correctly
			releaseHandlers();
			callback( dlg, dlg._.parentDialog );
		};

		var releaseHandlers = function()
		{
			var dlg = editor._.storedDialogs[dialogName];  // Changed by Nic to fix failure to releaseHandlers correctly
			dlg.removeListener( 'ok', onOk );
			dlg.removeListener( 'cancel', releaseHandlers );
		};



}}}
"	Bug	closed	Normal		General	3.6.1	expired