﻿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			
