revert extra Diaspora disabling changes to try to eliminate Mustard double-posting

1 files changed, 15 insertions, 31 deletions

diff --git a/include/items.php b/include/items.php
index af46eaaa1..7e8b8af4c 100755
--- a/include/items.php
+++ b/include/items.php
@@ -383,21 +383,16 @@ function get_atom_elements($feed,$item) {
 			$res['app'] = 'OStatus';
 	}		   
 
-
 	// base64 encoded json structure representing Diaspora signature
-	$dspr_enabled = intval(get_config('system','diaspora_enabled'));
-
-	if( $dspr_enabled) { 
-		$dsig = $item->get_item_tags(NAMESPACE_DFRN,'diaspora_signature');
-		if($dsig) {
-			$res['dsprsig'] = unxmlify($dsig[0]['data']);
-		}
 
-		$dguid = $item->get_item_tags(NAMESPACE_DFRN,'diaspora_guid');
-		if($dguid)
-			$res['guid'] = unxmlify($dguid[0]['data']);
+	$dsig = $item->get_item_tags(NAMESPACE_DFRN,'diaspora_signature');
+	if($dsig) {
+		$res['dsprsig'] = unxmlify($dsig[0]['data']);
 	}
 
+	$dguid = $item->get_item_tags(NAMESPACE_DFRN,'diaspora_guid');
+	if($dguid)
+		$res['guid'] = unxmlify($dguid[0]['data']);
 
 	$bm = $item->get_item_tags(NAMESPACE_DFRN,'bookmark');
 	if($bm)
@@ -704,17 +699,13 @@ function item_store($arr,$force_parent = false) {
 
 	// If a Diaspora signature structure was passed in, pull it out of the 
 	// item array and set it aside for later storage.
-	$dspr_enabled = intval(get_config('system','diaspora_enabled'));
 
 	$dsprsig = null;
-
 	if(x($arr,'dsprsig')) {
-		if($dspr_enabled)
-			$dsprsig = json_decode(base64_decode($arr['dsprsig']));
+		$dsprsig = json_decode(base64_decode($arr['dsprsig']));
 		unset($arr['dsprsig']);
 	}
 
-
 	if(x($arr, 'gravity'))
 		$arr['gravity'] = intval($arr['gravity']);
 	elseif($arr['parent-uri'] === $arr['uri'])
@@ -943,9 +934,7 @@ function item_store($arr,$force_parent = false) {
 		intval($parent_id)
 	);
 
-
-	// Store the Diaspora signature if there is one
-	if($dspr_enabled && $dsprsig) {
+	if($dsprsig) {
 		q("insert into sign (`iid`,`signed_text`,`signature`,`signer`) values (%d,'%s','%s','%s') ",
 			intval($current_post),
 			dbesc($dsprsig->signed_text),
@@ -1019,7 +1008,6 @@ function tag_deliver($uid,$item_id) {
 
 	$dlink = normalise_link($a->get_baseurl() . '/u/' . $u[0]['nickname']);
 
-
 	$cnt = preg_match_all('/[\@\!]\[url\=(.*?)\](.*?)\[\/url\]/ism',$item['body'],$matches,PREG_SET_ORDER);
 	if($cnt) {
 		foreach($matches as $mtch) {
@@ -2280,6 +2268,7 @@ function local_delivery($importer,$data) {
 
 			$is_a_remote_comment = false;
 
+			// POSSIBLE CLEANUP --> Why select so many fields when only forum_mode and wall are used?
 			$r = q("select `item`.`id`, `item`.`uri`, `item`.`tag`, `item`.`forum_mode`,`item`.`origin`,`item`.`wall`, 
 				`contact`.`name`, `contact`.`url`, `contact`.`thumb` from `item` 
 				LEFT JOIN `contact` ON `contact`.`id` = `item`.`contact-id` 
@@ -2985,15 +2974,12 @@ function atom_entry($item,$type,$author,$owner,$comment = false,$cid = 0) {
 	if($item['app'])
 		$o .= '<statusnet:notice_info local_id="' . $item['id'] . '" source="' . xmlify($item['app']) . '" ></statusnet:notice_info>' . "\r\n";
 
-	$dspr_enabled = intval(get_config('system','diaspora_enabled'));
-	if( $dspr_enabled) {
-		if($item['guid'])
-			$o .= '<dfrn:diaspora_guid>' . $item['guid'] . '</dfrn:diaspora_guid>' . "\r\n";
+	if($item['guid'])
+		$o .= '<dfrn:diaspora_guid>' . $item['guid'] . '</dfrn:diaspora_guid>' . "\r\n";
 
-		if($item['signed_text']) {
-			$sign = base64_encode(json_encode(array('signed_text' => $item['signed_text'],'signature' => $item['signature'],'signer' => $item['signer'])));
-			$o .= '<dfrn:diaspora_signature>' . xmlify($sign) . '</dfrn:diaspora_signature>' . "\r\n";
-		}
+	if($item['signed_text']) {
+		$sign = base64_encode(json_encode(array('signed_text' => $item['signed_text'],'signature' => $item['signature'],'signer' => $item['signer'])));
+		$o .= '<dfrn:diaspora_signature>' . xmlify($sign) . '</dfrn:diaspora_signature>' . "\r\n";
 	}
 
 	$verb = construct_verb($item);
@@ -3332,9 +3318,7 @@ function drop_item($id,$interactive = true) {
 			// ignore the result
 		}
 
-		// clean up item_id and sign (Diaspora signature) meta-data tables
-		// Clean up the sign table even if Diaspora support is disabled. We may still need to
-		// clean it up if Diaspora support had been enabled in the past
+		// clean up item_id and sign meta-data tables
 
 		$r = q("DELETE FROM item_id where iid in (select id from item where parent = %d and uid = %d)",
 			intval($item['id']),