Remove references to item.parent-uri in Worker\OnePoll

[friendica.git] / src / Worker / OnePoll.php

diff --git a/src/Worker/OnePoll.php b/src/Worker/OnePoll.php
index 00d5694db374cdb0ece4b580ddfbcb67457ae5c3..c5dfb70ab78eafd54c704f5d98cb4f2a6abdf1ed 100644 (file)
--- a/src/Worker/OnePoll.php
+++ b/src/Worker/OnePoll.php
@@ -60,7 +60,7 @@ class OnePoll
                        return;
                }
 
-               if (($contact['network'] != Protocol::MAIL) || $force) {
+               if (($contact['network'] != Protocol::MAIL) && $force) {
                        Contact::updateFromProbe($contact_id);
                }
 
@@ -103,6 +103,15 @@ class OnePoll
                        return;
                }
 
+               // Don't poll local contacts
+               if (User::getIdForURL($contact['url'])) {
+                       Logger::info('Local contacts are not polled', ['id' => $contact['id']]);
+
+                       // set the last-update so we don't keep polling
+                       DBA::update('contact', ['last-update' => $updated], ['id' => $contact['id']]);
+                       return;
+               }               
+
                // We don't poll AP contacts by now
                if ($protocol === Protocol::ACTIVITYPUB) {
                        Logger::log("Don't poll AP contact");
@@ -435,7 +444,7 @@ class OnePoll
                }
 
                $cookiejar = tempnam(get_temppath(), 'cookiejar-onepoll-');
-               $curlResult = DI::httpRequest()->get($contact['poll'], false, ['cookiejar' => $cookiejar]);
+               $curlResult = DI::httpRequest()->get($contact['poll'], ['cookiejar' => $cookiejar]);
                unlink($cookiejar);
 
                if ($curlResult->isTimeout()) {
@@ -574,9 +583,9 @@ class OnePoll
                                                        }
                                                }
                                                $condition = ['uri' => $refs_arr, 'uid' => $importer_uid];
-                                               $parent = Item::selectFirst(['parent-uri'], $condition);
+                                               $parent = Item::selectFirst(['uri'], $condition);
                                                if (DBA::isResult($parent)) {
-                                                       $datarray['parent-uri'] = $parent['parent-uri'];  // Set the parent as the top-level item
+                                                       $datarray['thr-parent'] = $parent['uri'];
                                                }
                                        }
 
@@ -604,19 +613,15 @@ class OnePoll
                                        $datarray['title'] = self::RemoveReply($datarray['title']);
 
                                        // If it seems to be a reply but a header couldn't be found take the last message with matching subject
-                                       if (empty($datarray['parent-uri']) && $reply) {
+                                       if (empty($datarray['thr-parent']) && $reply) {
                                                $condition = ['title' => $datarray['title'], 'uid' => $importer_uid, 'network' => Protocol::MAIL];
                                                $params = ['order' => ['created' => true]];
-                                               $parent = Item::selectFirst(['parent-uri'], $condition, $params);
+                                               $parent = Item::selectFirst(['uri'], $condition, $params);
                                                if (DBA::isResult($parent)) {
-                                                       $datarray['parent-uri'] = $parent['parent-uri'];
+                                                       $datarray['thr-parent'] = $parent['uri'];
                                                }
                                        }
 
-                                       if (empty($datarray['parent-uri'])) {
-                                               $datarray['parent-uri'] = $datarray['uri'];
-                                       }
-
                                        $headers = imap_headerinfo($mbox, $meta->msgno);
 
                                        $object = [];
@@ -654,7 +659,7 @@ class OnePoll
                                        $datarray['owner-link'] = "mailto:".$contact['addr'];
                                        $datarray['owner-avatar'] = $contact['photo'];
 
-                                       if ($datarray['parent-uri'] === $datarray['uri']) {
+                                       if ($datarray['thr-parent'] === $datarray['uri']) {
                                                $datarray['private'] = Item::PRIVATE;
                                        }
 
@@ -699,6 +704,9 @@ class OnePoll
                        Logger::log("Mail: no mails for ".$mailconf['user']);
                }
 
+               self::updateContact($contact, ['failed' => false, 'last-update' => $updated, 'success_update' => $updated]);
+               Contact::unmarkForArchival($contact);
+
                Logger::log("Mail: closing connection for ".$mailconf['user']);
                imap_close($mbox);
        }