aboutsummaryrefslogtreecommitdiffstats
path: root/include
diff options
context:
space:
mode:
authorzotlabs <mike@macgirvin.com>2018-03-23 21:24:24 -0700
committerzotlabs <mike@macgirvin.com>2018-03-23 21:24:24 -0700
commitabe6ab477531916158c04e3d387cb02e839d28b6 (patch)
tree1d1e6f9867923046d9a0c65024dc1dfb0a80a55e /include
parentaf8cbf000fb1433b7c7d2c6c4543980f01ba69a9 (diff)
downloadvolse-hubzilla-abe6ab477531916158c04e3d387cb02e839d28b6.tar.gz
volse-hubzilla-abe6ab477531916158c04e3d387cb02e839d28b6.tar.bz2
volse-hubzilla-abe6ab477531916158c04e3d387cb02e839d28b6.zip
logic fallacy when updating parent timestamp of moderated comment
Diffstat (limited to 'include')
-rwxr-xr-xinclude/items.php10
1 files changed, 9 insertions, 1 deletions
diff --git a/include/items.php b/include/items.php
index 53d1a3c02..a2068868a 100755
--- a/include/items.php
+++ b/include/items.php
@@ -1977,11 +1977,19 @@ function item_store($arr, $allow_exec = false, $deliver = true) {
*/
call_hooks('post_remote_end', $arr);
+ $update_parent = true;
+
// update the commented timestamp on the parent - unless this is a moderated comment or a potential clone of an older item
// which we don't wish to bring to the surface. As the queue only holds deliveries for 3 days, it's
// suspected of being an older cloned item if the creation time is older than that.
- if((intval($arr['item_blocked']) != ITEM_MODERATED) || ($arr['created'] > datetime_convert('','','now - 4 days'))) {
+ if(intval($arr['item_blocked']) === ITEM_MODERATED)
+ $update_parent = false;
+
+ if($arr['created'] < datetime_convert('','','now - 4 days'))
+ $update_parent = false;
+
+ if($update_parent) {
$z = q("select max(created) as commented from item where parent_mid = '%s' and uid = %d and item_delayed = 0 ",
dbesc($arr['parent_mid']),
intval($arr['uid'])