From 720a44bb0b64e281b52ea1cf8c77344a5bea31f9 Mon Sep 17 00:00:00 2001
From: Matthew Exon <github.mexon@spamgourmet.com>
Date: Fri, 8 Mar 2013 03:08:47 +0100
Subject: [PATCH] The post_remote_end hook was being called with an escaped
 version of the item, not very useful.  This change moves the hook to the end
 of item_store and gets all data fresh from the database.

---
 include/items.php | 9 ++++++++-
 1 file changed, 8 insertions(+), 1 deletion(-)

diff --git a/include/items.php b/include/items.php
index 29f4d7256..8bd28f788 100755
--- a/include/items.php
+++ b/include/items.php
@@ -1149,7 +1149,6 @@ function item_store($arr,$force_parent = false) {
         $arr['deny_gid'] = $deny_gid;
         $arr['private'] = $private;
         $arr['deleted'] = $parent_deleted;
-	call_hooks('post_remote_end',$arr);
 
 	// update the commented timestamp on the parent
 
@@ -1195,6 +1194,14 @@ function item_store($arr,$force_parent = false) {
 		logger('item_store: put item '.$current_post.' into cachefile '.$cachefile);
 	}
 
+        $r = q('SELECT * FROM `item` WHERE id = %d', intval($current_post));
+        if (count($r) == 1) {
+            call_hooks('post_remote_end', $r[0]);
+        }
+        else {
+            logger('item_store: new item not found in DB, id ' . $current_post);
+        }
+
 	return $current_post;
 }