1 files changed, 53 insertions, 8 deletions

diff --git a/js/news.js b/js/news.js
index 912717137..a589ee644 100644
--- a/js/news.js
+++ b/js/news.js
@@ -106,6 +106,7 @@ News={
 	Feed: {
 		id:'',
 		submit:function(button){
+			
 			var feedurl = $("#feed_add_url").val().trim();
 
 			if(feedurl.length == 0) {
@@ -115,11 +116,15 @@ News={
 
 			$(button).attr("disabled", true);
 			$(button).prop('value', t('news', 'Adding...'));
-
+			
 			var folderid = $('#inputfolderid:input[name="folderid"]').val();
-
-			$.post(OC.filePath('news', 'ajax', 'createfeed.php'), { feedurl: feedurl, folderid: folderid },
-				function(jsondata){
+						
+			$.ajax({
+				type: "POST",
+				url: OC.filePath('news', 'ajax', 'createfeed.php'),
+				data: { 'feedurl': feedurl, 'folderid': folderid },
+				dataType: "json",
+				success: function(jsondata){
 					if(jsondata.status == 'success'){
 						$('div[data-id="' + folderid + '"] > ul').append(jsondata.data.listfeed);
 						setupFeedList();
@@ -138,10 +143,50 @@ News={
 					} else {
 						OC.dialogs.alert(jsondata.data.message, t('news', 'Error'));
 					}
-				$("#feed_add_url").val('');
-				$(button).attr("disabled", false);
-				$(button).prop('value', t('news', 'Add feed'));
+					$("#feed_add_url").val('');
+					$(button).attr("disabled", false);
+					$(button).prop('value', t('news', 'Add feed'));
+				},
+				error: function(xhr) {
+					OC.dialogs.alert(t('news', 'Error while parsing the feed'), t('news', 'Fatal Error'));
+					$("#feed_add_url").val('');
+					$(button).attr("disabled", false);
+					$(button).prop('value', t('news', 'Add feed'));
+				},
+				statusCode: {
+					500: function() {
+						OC.dialogs.alert(t('news', 'Error while parsing the feed'), t('news', 'Fatal Error'));
+						$("#feed_add_url").val('');
+						$(button).attr("disabled", false);
+						$(button).prop('value', t('news', 'Add feed'));
+					}
+				}
 			});
+			
+// 			$.post(OC.filePath('news', 'ajax', 'createfeed.php'), { feedurl: feedurl, folderid: folderid },
+// 				function(jsondata){
+// 					if(jsondata.status == 'success'){
+// 						$('div[data-id="' + folderid + '"] > ul').append(jsondata.data.listfeed);
+// 						setupFeedList();
+// 						OC.dialogs.confirm(t('news', 'Do you want to add another feed?'), t('news', 'Feed added!'), function(answer) {
+// 							if(!answer) {
+// 								$('#addfeed_dialog').dialog('destroy').remove();
+// 								var rightcontent = $('div.rightcontent');
+// 								rightcontent.empty();
+// 								rightcontent.attr('data-id', jsondata.data.feedid);
+// 								rightcontent.html(jsondata.data.part_items);
+// 								rightcontent.find('ul.accordion').before(jsondata.data.part_newfeed);
+// 								setupRightContent();
+// 								transformCollapsableTrigger();
+// 							}
+// 						});
+// 					} else {
+// 						OC.dialogs.alert(jsondata.data.message, t('news', 'Error'));
+// 					}
+// 				$("#feed_add_url").val('');
+// 				$(button).attr("disabled", false);
+// 				$(button).prop('value', t('news', 'Add feed'));
+// 			});
 
 		},
 		'delete':function(feedid) {
@@ -325,9 +370,9 @@ $(document).ready(function(){
 	News.Feed.updateAll();
 	var updateInterval = 200000; //how often the feeds should update (in msec)
 	setInterval('News.Feed.updateAll()', updateInterval);
+ 
 });
 
 $(document).click(function(event) {
 	$('#feedfoldermenu').hide();
 });
-