id	summary	reporter	owner	description	type	status	priority	milestone	component	version	resolution	keywords	cc
8364	Error when opening Link-dialog in IE 9	Henrik Helmø Larsen		"I am currently using version CKEditor 3.6.1 and have found a bug in the plugin.js file of the dialog folder.

I have modified the link dialog - (code added at the end). The dialog displays without problems in both FF and Chrome. In IE 9.0.8112 an error is displayed:

''SCRIPT5007: Unable to get value of the property 'align': object is null or undefined 
plugin.js, line 2539 character 7
''

The problem is the following line:

''if ( CKEDITOR.env.ie && CKEDITOR.env.quirks && children[ i ].align )
''

in the plugin.js in the dialog folder. The children[ i ] is null or undefined. A quick solution is to add a check for children[ i ]:


''if ( CKEDITOR.env.ie && CKEDITOR.env.quirks '''&& children[ i ]''' && children[ i ].align )
''

'''My code:'''[[BR]]
As it is inside a vbox the bug occurs (I believe it is when the radio buttons are defined) I have copied the part of my code where i define the UI of the dialog (and tried to remove unrelevant things - but let me know if you need more code...):


{{{
var infoTab = dialogDefinition.getContents( 'info' );
infoTab.add( {
  type : 'hbox',
  padding : 1,
  widths : ['100px', '100%'],
  id: 'outer',
  align: 'center',
  children : [
    // Insert Link to:
    {
       type : 'vbox',
       padding : 1,
       width : '100px',
       id: 'LinkType',
       align: 'left',
       children : [
         {
           type : 'radio',
           id : 'radioLink',
           label : 'Insert link to:',
           labelLayout: 'vertical',
           items : radioButtons,
           setup : function( data ) {
             debug(""Setup..."");
             //a lot of setup code removed... 
           },
           onChange : function( api ) {
              debug(""onChange..."");
              //code removed
           }
         }
       ]
     },
     {
       type : 'vbox',
       padding : 1,
       width : '100%',
       id: 'LinkCont',
       align: 'right',
       children : [
         {
           type : 'vbox',
           padding : '0px 0px 20px 0px',
           id: 'innerLinkList',
           align: 'right',
           children : [
             {
               type : 'select',
               id : 'link',
               items : [],
               size : 11,
               multiple : true
             }
           ]
         },
         {
           type : 'text',
           id : 'linkurl',
           label : 'Address (URL):',
           'default' : '',
           labelLayout : 'horizontal',
           validate : function(api) {
             //Code removed
           }
         },
         {
           type : 'text',
           id : 'linktext',
           label : 'Displayed Text:',
           'default' : '',
           labelLayout : 'horizontal',
           validate : function() {
             //Code removed
           }
         }
       ]
     }
  ]
});

// Remove the ""Link Type"" combo and the ""Browser
// Server"" button from the ""info"" tab.
infoTab.remove( 'linkType' );
infoTab.remove( 'browse' );
infoTab.remove( 'protocol' );
infoTab.remove( 'url' );

// Remove the ""Target"" tab from the ""Link"" dialog.
dialogDefinition.removeContents( 'target' );

//Remove the ""Advanced"" tab from the ""Link"" dialog
dialogDefinition.removeContents( 'advanced' );

}}}
"	Bug	closed	Normal		UI : Dialogs	3.6.1	invalid