diff options
author | friendica <info@friendica.com> | 2014-10-19 17:31:10 -0700 |
---|---|---|
committer | friendica <info@friendica.com> | 2014-10-19 17:31:10 -0700 |
commit | fd492386bf13152750c4db7306950d52b1105af1 (patch) | |
tree | ffee288e320053675f5a496d686fee0c67cf0591 /include/zot.php | |
parent | 26d626efe539d3ceabc4a682f512a05e3faaedf0 (diff) | |
download | volse-hubzilla-fd492386bf13152750c4db7306950d52b1105af1.tar.gz volse-hubzilla-fd492386bf13152750c4db7306950d52b1105af1.tar.bz2 volse-hubzilla-fd492386bf13152750c4db7306950d52b1105af1.zip |
Tried this once before but had to revert. Basically prevent a comment from being relayed more than once - as it's a huge drain on resources. But last time I tried this, wall-to-wall comments stopped getting relayed. This checkin should do the right thing in both conditions.
Diffstat (limited to 'include/zot.php')
-rw-r--r-- | include/zot.php | 8 |
1 files changed, 7 insertions, 1 deletions
diff --git a/include/zot.php b/include/zot.php index 869bca668..e96eae10d 100644 --- a/include/zot.php +++ b/include/zot.php @@ -1476,11 +1476,13 @@ function process_delivery($sender,$arr,$deliveries,$relay,$public = false) { continue; } - $r = q("select id, edited from item where mid = '%s' and uid = %d limit 1", + $r = q("select id, edited, item_flags, mid, parent_mid from item where mid = '%s' and uid = %d limit 1", dbesc($arr['mid']), intval($channel['channel_id']) ); if($r) { + // We already have this post. + // Maybe it has been edited? $item_id = $r[0]['id']; if($arr['edited'] > $r[0]['edited']) { $arr['id'] = $r[0]['id']; @@ -1492,6 +1494,10 @@ function process_delivery($sender,$arr,$deliveries,$relay,$public = false) { } else { $result[] = array($d['hash'],'update ignored',$channel['channel_name'] . ' <' . $channel['channel_address'] . '@' . get_app()->get_hostname() . '>',$arr['mid']); + // We need this line to ensure wall-to-wall comments are relayed (by falling through to the relay bit), + // and at the same time not relay any other relayable posts more than once, because to do so is very wasteful. + if(! ($r[0]['item_flags'] & ITEM_ORIGIN)) + continue; } } else { |