id	summary	reporter	owner	description	type	status	priority	milestone	component	version	resolution	keywords	cc
4251	Adding wrong plugin doesn't give proper warning	Alfonso Martínez de Lizarrondo		"Modify a sample so it tries to load a non-existing plugin:
{{{
var editor = CKEDITOR.replace( 'editor1',
	{
		extraPlugins : 'World'
	});
}}}

Now loading the page in IE gives an error message without any clue about the problem.[[BR]]
It can be improved this way:
{{{
#!patch
Index: _source/core/plugins.js
===================================================================
--- _source/core/plugins.js	(revision 4127)
+++ _source/core/plugins.js	(working copy)
@@ -56,6 +56,11 @@
 							for ( pluginName in allPlugins )
 							{
 								plugin = allPlugins[ pluginName ];
+								if ( !plugin )
+								{
+									alert( 'Plugin ""' + pluginName + '"" not loaded.' ) ;
+									continue ;
+								}
 								if ( plugin.onLoad && !plugin.onLoad._called )
 								{
 									plugin.onLoad();

}}}

For other browsers, it throws an exception in core\scriptmanager.js:
{{{
				if ( failed.length )
				{
					throw '[CKEDITOR.resourceManager.load] Resource name ""' + urlsNames[ failed[ 0 ] ].join( ',' )
						+ '"" was not found at ""' + failed[ 0 ] + '"".';
				}
}}}
But that's shown only in the error console, it will be more useful if the message is shown in an alert."	Bug	confirmed	Normal		General	3.0