FCKDialog.OpenDialog() - wrong variable mapping (parentWindow)

[Ivan Tcholakov]

FCKEditor 2.6.4 This is not a "theoretical" note, I found this bug during development of something.

See .../_source/internals/fckdialog.js

The parameter *parentWindow* in the function OpenDialog() declaration and and the variable *topWindow* that sets a property of the dialogInfo object are the same thing actually. Only one should stay - parentWindow or topWindow.

Here is an example for a correction:

var FCKDialog = ( function()
{

  ...

	return {
		/**
		 * Opens a dialog window using the standard dialog template.
		 */
		OpenDialog : function( dialogName, dialogTitle, dialogPage, width, height, customValue, parentWindow, resizable )
		{

			...

      // Setup the dialog info to be passed to the dialog.
			var dialogInfo =
			{
				Title : dialogTitle,
				Page : dialogPage,
				Editor : window,
				CustomValue : customValue,		// Optional
				// -------------- A correction starts here ---------------------
				//TopWindow : topWindow       // Wrong
				TopWindow : parentWindow      // Correct
				// -------------------------------------------------------------
			}

			...

		},

  ...

[Ivan Tcholakov]

My Previous correction was wrong, topWindow and parentWindow are not the same. But anyway I need to access parrentWindow when I close a dialog. Here is another modification that works fine for me:

var FCKDialog = ( function()
{

  ...

	return {
		/**
		 * Opens a dialog window using the standard dialog template.
		 */
		OpenDialog : function( dialogName, dialogTitle, dialogPage, width, height, customValue, parentWindow, resizable )
		{

			...

      // Setup the dialog info to be passed to the dialog.
			var dialogInfo =
			{
				Title : dialogTitle,
				Page : dialogPage,
				Editor : window,
				CustomValue : customValue,		// Optional
				TopWindow : topWindow,
				ParentWindow : parentWindow		// <----------- Added property
			}

			...

		},

  ...

[Martin Kou]

The parentWindow argument is actually useless - if you need access to the parent dialog's window, use

ParentDialog().contentWindow

instead.

[Martin Kou]

Proposed patch to remove parentWindow reference to reduce code size and avoid confusion.

[Ivan Tcholakov]

Thank you, martinkou. Your suggestion works.

I had a look at your patch. I think, there is one place that was missed to be updated:

File: fckeditor/editor/_source/commandclasses/fck_othercommands.js
Line 189: FCKDialog.OpenDialog( 'FCKDialog_Source', FCKLang.Source, 'dialog/fck_source.html', iWidth, iHeight, null, null, true ) ;

[Martin Kou]

Thanks for the correction, updated the patch.

[Martin Kou]

Fixed with [3975].

Click here for more info about our SVN system.