id,summary,reporter,owner,description,type,status,priority,milestone,component,version,resolution,keywords,cc 5552,Dialog doesn't execute 'commit' and 'setup' method of element when element miss 'id' property.,Minh Nguyen,Minh Nguyen,"=== Steps to reproduce === Create new dialog without set id attribute for element or open one old dialog and remove id attribute of element: {{{ CKEDITOR.dialog.add( 'bulletedListStyle', function( editor ) { return { title : editor.lang.list.numberedTitle, minWidth : 300, minHeight : 50, contents : [ { elements : [ { type : 'hbox', widths : [ '25%', '75%' ], children : [ { label: editor.lang.list.start, type: 'text', setup : function( element ) { var value = element.getAttribute( 'start' ) || 1; value && this.setValue( value ); }, commit : function( element ) { element.setAttribute( 'start', this.getValue() ); } }, { type : 'select', label : editor.lang.list.type, width: '100%', items : [ [ editor.lang.list.style_1 , '1' ], [ editor.lang.list.style_a , 'a' ], [ editor.lang.list.style_A , 'A' ], [ editor.lang.list.style_i , 'i' ], [ editor.lang.list.style_I , 'I' ] ], setup : function( element ) { var value = element.getAttribute( 'type' ) || '1'; this.setValue( value ); }, commit : function( element ) { element.setAttribute( 'type', this.getValue() ); } } ] } ] } ], onShow: function() { var editor = this.getParentEditor(), element = getListElement( editor, 'ol' ); element && this.setupContent( element ); }, onOk: function() { var editor = this.getParentEditor(), element = getListElement( editor, 'ol' ); element && this.commitContent( element ); } } }); }}} * Expected: all 'commit' and 'setup' method of element will execute * Result: only the last element have that behavior",Bug,review_failed,Normal,,General,SVN (CKEditor) - OLD,,,