Just a to-do added

diff --git a/include/ostatus.php b/include/ostatus.php
index 74196b56a3be2479021d42410c023b942a40ce78..86d0e36dbe2f2f6b5794096ad7f22c747b5c3005 100644 (file)
--- a/include/ostatus.php
+++ b/include/ostatus.php
@@ -376,6 +376,8 @@ function ostatus_import($xml,$importer,&$contact, &$hub) {
                } else
                        $item["parent-uri"] = $item["uri"];
 
+               // We risk the chance of getting orphan items, we correct it some lines later
+               // To-Do: See To-Do line below.
                $item_id = item_store($item, true);
                //echo $xml;
                //print_r($item);
@@ -416,6 +418,9 @@ function ostatus_import($xml,$importer,&$contact, &$hub) {
                if ($conversation != "") {
                        // Check for duplicates. We really don't need to check the same conversation twice.
                        if (!in_array($conversation, $conversationlist)) {
+                               // To-Do:
+                               // Call this before item_store is called to avoid posts with orphans
+                               // The routine then needs to get the item array.
                                complete_conversation($item_id, $conversation);
                                $conversationlist[] = $conversation;
                        }