Fixed a few bugs regarding sending empty responses. If no file is selected for upload an alert will appeart to notify the user.

scripts/ckeditor/plugins/fileupload/dialogs/fileupload.js

@@ -9,9 +9,7 @@
 CKEDITOR.dialog.add( 'fileuploadDialog', function( editor ) {
 
 	var lang = editor.lang.image2;
-	console.log(editor.lang);
+4
-	console.log(lang.btnUpload);
-
 	return {
 
 		// Basic properties of the dialog window: title, minimum size.
@@ -46,6 +44,13 @@ CKEDITOR.dialog.add( 'fileuploadDialog', function( editor ) {
 					        	// grab the file(s) from the file input tag and upload it using AJAX
 				            	var dialog = this.getDialog();
             					var files = dialog.getContentElement("Upload", "upload").getInputElement().$.files;
+
+            					// no files were selected
+            					if(files.length == 0) {
+            						alert("Please select a file!");
+            						return false;
+            					}
+
             					var formData = new FormData();
             					formData.append('drop-count', files.length);		// number of files being uploaded
             					formData.append('acmd', "Upload");			// Command for the server to recognize this as an ajax upload
@@ -70,7 +75,7 @@ CKEDITOR.dialog.add( 'fileuploadDialog', function( editor ) {
 								      if (evt.lengthComputable) {
 								        var percentComplete = evt.loaded / evt.total;
 								        //Do something with upload progress
-								        // console.log(percentComplete);
+								        console.log(percentComplete);
 								      }
 								    }, false);
 
@@ -82,7 +87,11 @@ CKEDITOR.dialog.add( 'fileuploadDialog', function( editor ) {
 								  url: URL,
 								  data: formData,
 								  success: function(data) {
+								  	if(data.attachments[0]) {	// check if we have the correct response
 								  		dialog.getContentElement( 'Upload', 'src' ).setValue( data.attachments[0].fullName );
+								  	} else {
+								  		console.log("Data returned is not json...");
+								  	}
 								  },
 								  fail: function() {
 								  	console.log("error");
@@ -119,9 +128,6 @@ CKEDITOR.dialog.add( 'fileuploadDialog', function( editor ) {
 		onOk: function() {
             var dialog = this;
 
-            console.log(dialog.getValueOf( 'Upload', 'src' ));
-            console.log(dialog.getValueOf( 'Upload', 'name' ));
-
             var file = editor.document.createElement( 'a' );
             file.setAttribute( 'href', dialog.getValueOf( 'Upload', 'src' ) );
             file.setText( dialog.getValueOf( 'Upload', 'name' ) );

scripts/ckeditor/plugins/image2/dialogs/image2.js

@@ -545,6 +545,13 @@ CKEDITOR.dialog.add( 'image2', function( editor ) {
 					        	// grab the file(s) from the file input tag and upload it using AJAX
 				            	var dialog = this.getDialog();
             					var files = dialog.getContentElement("Upload", "upload").getInputElement().$.files;
+
+            					// no files were selected
+            					if(files.length == 0) {
+            						alert("Please select a file!");
+            						return false;
+            					}
+
             					var formData = new FormData();
             					formData.append('drop-count', files.length);		// number of files being uploaded
 							    for (var i = 0; i < files.length; i++) {
@@ -578,7 +585,11 @@ CKEDITOR.dialog.add( 'image2', function( editor ) {
 								  url: URL,
 								  data: formData,
 								  success: function(data) {
+								  	if(data.attachments) {	// check if we have the correct response
             							dialog.getContentElement( 'info', 'src' ).setValue( data.attachments[0].fullName );
+            						} else {
+            							console.log("Data returned is not json...");
+            						}
 								  },
 								  fail: function() {
 								  	console.log("error");

scripts/load-ckeditor.js

@@ -30,8 +30,8 @@ $(document).ready(function() {
         var dialogDefinition = ev.data.definition;
 
         // Check if the definition is from the dialog we're
-        // interested in (the 'image2' and 'fileupload' dialog).
+        // interested in (the 'image2' and 'fileuploadDialog' dialog).
-        if ( dialogName == 'image2' || dialogName == 'fileupload') {
+        if ( dialogName == 'image2' || dialogName == 'fileuploadDialog') {
 
             var dialogObj = dialogDefinition.dialog;
             dialogObj.on("show", function() {