rpc/sendEmail: use JSON

modules/backend-rpc.php

@@ -668,23 +668,20 @@
 		}
 
 		if ($subop == "catchupFeed") {
-
 			$feed_id = db_escape_string($_REQUEST['feed_id']);
 			$is_cat = db_escape_string($_REQUEST['is_cat']);
 
-			print "<rpc-reply>";
-
 			catchup_feed($link, $feed_id, $is_cat);
 
-			print "</rpc-reply>";
-
 			return;
 		}
 
 		if ($subop == "sendEmail") {
+			header("Content-Type: text/plain");
+
 			$secretkey = $_REQUEST['secretkey'];
 
-			print "<rpc-reply>";
+			$reply = array();
 
 			if (DIGEST_ENABLE && $_SESSION['email_secretkey'] && 
 						$secretkey == $_SESSION['email_secretkey']) {
@@ -724,18 +721,17 @@
 				$rc = $mail->Send();
 
 				if (!$rc) {
-					print "<error><![CDATA[" . $mail->ErrorInfo . "]]></error>";
+					$reply['error'] =  $mail->ErrorInfo;
 				} else {
 					save_email_address($link, db_escape_string($destination));
-					print "<message>UPDATE_COUNTERS</message>";
+					$reply['message'] = "UPDATE_COUNTERS";
 				}
 
 			} else {
-				print "<error>Not authorized.</error>";
+				$reply['error'] = "Not authorized.";
 			}
 
-			print "</rpc-reply>";
+			print json_encode($reply);
-
 			return;
 		}
 

viewfeed.js

@@ -1843,10 +1843,12 @@ function emailArticle(id) {
 						parameters: dojo.objectToQuery(this.attr('value')),
 						onComplete: function(transport) { 
 
-							var error = transport.responseXML.getElementsByTagName('error')[0];
+							var reply = JSON.parse(transport.responseText);
+		
+							var error = reply['error'];
 			
 							if (error) {
-								alert(__('Error sending email:') + ' ' + error.firstChild.nodeValue);
+								alert(__('Error sending email:') + ' ' + error);
 							} else {
 								notify_info('Your message has been sent.');
 								dialog.hide();