]> git.mxchange.org Git - friendica.git/blobdiff - src/Protocol/DFRN.php
Merge pull request #11519 from MrPetovan/task/11511-console-domain-move
[friendica.git] / src / Protocol / DFRN.php
index 6c692b21fd24fcca4f122e9b3149f8384a63cd73..0fb7394579f7513d4667d600f5b598aef9306f75 100644 (file)
@@ -1,6 +1,6 @@
 <?php
 /**
- * @copyright Copyright (C) 2010-2021, the Friendica project
+ * @copyright Copyright (C) 2010-2022, the Friendica project
  *
  * @license GNU AGPL version 3 or any later version
  *
@@ -413,6 +413,48 @@ class DFRN
                return $root;
        }
 
+       /**
+        * Determine the next birthday, but only if the birthday is published
+        * in the default profile. We _could_ also look for a private profile that the
+        * recipient can see, but somebody could get mad at us if they start getting
+        * public birthday greetings when they haven't made this info public.
+        *
+        * Assuming we are able to publish this info, we are then going to convert
+        * the start time from the owner's timezone to UTC.
+        *
+        * This will potentially solve the problem found with some social networks
+        * where birthdays are converted to the viewer's timezone and salutations from
+        * elsewhere in the world show up on the wrong day. We will convert it to the
+        * viewer's timezone also, but first we are going to convert it from the birthday
+        * person's timezone to GMT - so the viewer may find the birthday starting at
+        * 6:00PM the day before, but that will correspond to midnight to the birthday person.
+        */
+       private static function determineNextBirthday($uid, $tz)
+       {
+               $birthday = '';
+
+               if (!strlen($tz)) {
+                       $tz = 'UTC';
+               }
+
+               $profile = DBA::selectFirst('profile', ['dob'], ['uid' => $uid]);
+               if (DBA::isResult($profile)) {
+                       $tmp_dob = substr($profile['dob'], 5);
+                       if (intval($tmp_dob)) {
+                               $y = DateTimeFormat::timezoneNow($tz, 'Y');
+                               $bd = $y . '-' . $tmp_dob . ' 00:00';
+                               $t_dob = strtotime($bd);
+                               $now = strtotime(DateTimeFormat::timezoneNow($tz));
+                               if ($t_dob < $now) {
+                                       $bd = $y + 1 . '-' . $tmp_dob . ' 00:00';
+                               }
+                               $birthday = DateTimeFormat::convert($bd, 'UTC', $tz, DateTimeFormat::ATOM);
+                       }
+               }
+
+               return $birthday;
+       }
+
        /**
         * Adds the author element in the header for the DFRN protocol
         *
@@ -467,7 +509,7 @@ class DFRN
                        return $author;
                }
 
-               $birthday = feed_birthday($owner['uid'], $owner['timezone']);
+               $birthday = self::determineNextBirthday($owner['uid'], $owner['timezone']);
 
                if ($birthday) {
                        XML::addElement($doc, $author, "dfrn:birthday", $birthday);
@@ -488,10 +530,7 @@ class DFRN
                        XML::addElement($doc, $author, "poco:note", $profile["about"]);
                        XML::addElement($doc, $author, "poco:preferredUsername", $profile["nickname"]);
 
-                       $savetz = date_default_timezone_get();
-                       date_default_timezone_set($profile["timezone"]);
-                       XML::addElement($doc, $author, "poco:utcOffset", date("P"));
-                       date_default_timezone_set($savetz);
+                       XML::addElement($doc, $author, "poco:utcOffset", DateTimeFormat::timezoneNow($profile["timezone"], "P"));
 
                        if (trim($profile["homepage"]) != "") {
                                $urls = $doc->createElement("poco:urls");
@@ -886,9 +925,9 @@ class DFRN
 
                foreach ($mentioned as $mention) {
                        $condition = ['uid' => $owner["uid"], 'nurl' => Strings::normaliseLink($mention)];
-                       $contact = DBA::selectFirst('contact', ['forum', 'prv'], $condition);
+                       $contact = DBA::selectFirst('contact', ['contact-type'], $condition);
 
-                       if (DBA::isResult($contact) && ($contact["forum"] || $contact["prv"])) {
+                       if (DBA::isResult($contact) && ($contact['contact-type'] == Contact::TYPE_COMMUNITY)) {
                                XML::addElement(
                                        $doc,
                                        $entry,
@@ -932,21 +971,21 @@ class DFRN
        {
                if (!$public_batch) {
                        if (empty($contact['addr'])) {
-                               Logger::log('Empty contact handle for ' . $contact['id'] . ' - ' . $contact['url'] . ' - trying to update it.');
+                               Logger::notice('Empty contact handle for ' . $contact['id'] . ' - ' . $contact['url'] . ' - trying to update it.');
                                if (Contact::updateFromProbe($contact['id'])) {
                                        $new_contact = DBA::selectFirst('contact', ['addr'], ['id' => $contact['id']]);
                                        $contact['addr'] = $new_contact['addr'];
                                }
 
                                if (empty($contact['addr'])) {
-                                       Logger::log('Unable to find contact handle for ' . $contact['id'] . ' - ' . $contact['url']);
+                                       Logger::notice('Unable to find contact handle for ' . $contact['id'] . ' - ' . $contact['url']);
                                        return -21;
                                }
                        }
 
                        $fcontact = FContact::getByURL($contact['addr']);
                        if (empty($fcontact)) {
-                               Logger::log('Unable to find contact details for ' . $contact['id'] . ' - ' . $contact['addr']);
+                               Logger::notice('Unable to find contact details for ' . $contact['id'] . ' - ' . $contact['addr']);
                                return -22;
                        }
                        $pubkey = $fcontact['pubkey'];
@@ -979,7 +1018,7 @@ class DFRN
 
                $curl_stat = $postResult->getReturnCode();
                if (empty($curl_stat) || empty($xml)) {
-                       Logger::log('Empty answer from ' . $contact['id'] . ' - ' . $dest_url);
+                       Logger::notice('Empty answer from ' . $contact['id'] . ' - ' . $dest_url);
                        return -9; // timed out
                }
 
@@ -988,8 +1027,8 @@ class DFRN
                }
 
                if (strpos($xml, '<?xml') === false) {
-                       Logger::log('No valid XML returned from ' . $contact['id'] . ' - ' . $dest_url);
-                       Logger::log('Returned XML: ' . $xml, Logger::DATA);
+                       Logger::notice('No valid XML returned from ' . $contact['id'] . ' - ' . $dest_url);
+                       Logger::debug('Returned XML: ' . $xml);
                        return 3;
                }
 
@@ -1000,7 +1039,7 @@ class DFRN
                }
 
                if (!empty($res->message)) {
-                       Logger::log('Transmit to ' . $dest_url . ' returned status '.$res->status.' - '.$res->message, Logger::DEBUG);
+                       Logger::info('Transmit to ' . $dest_url . ' returned status '.$res->status.' - '.$res->message);
                }
 
                return intval($res->status);
@@ -1091,12 +1130,12 @@ class DFRN
                }
 
                if (empty($author['avatar'])) {
-                       Logger::log('Empty author: ' . $xml);
+                       Logger::notice('Empty author: ' . $xml);
                        $author['avatar'] = '';
                }
 
                if (DBA::isResult($contact_old) && !$onlyfetch) {
-                       Logger::log("Check if contact details for contact " . $contact_old["id"] . " (" . $contact_old["nick"] . ") have to be updated.", Logger::DEBUG);
+                       Logger::info("Check if contact details for contact " . $contact_old["id"] . " (" . $contact_old["nick"] . ") have to be updated.");
 
                        $poco = ["url" => $contact_old["url"], "network" => $contact_old["network"]];
 
@@ -1157,7 +1196,7 @@ class DFRN
                        // If the "hide" element is present then the profile isn't searchable.
                        $hide = intval(XML::getFirstNodeValue($xpath, $element . "/dfrn:hide/text()", $context) == "true");
 
-                       Logger::log("Hidden status for contact " . $contact_old["url"] . ": " . $hide, Logger::DEBUG);
+                       Logger::info("Hidden status for contact " . $contact_old["url"] . ": " . $hide);
 
                        // If the contact isn't searchable then set the contact to "hidden".
                        // Problem: This can be manually overridden by the user.
@@ -1298,7 +1337,7 @@ class DFRN
         */
        private static function processMail($xpath, $mail, $importer)
        {
-               Logger::log("Processing mails");
+               Logger::notice("Processing mails");
 
                $msg = [];
                $msg["uid"] = $importer["importer_uid"];
@@ -1333,7 +1372,61 @@ class DFRN
                $cid = Contact::getIdForURL($url);
                $note = $xpath->evaluate('string(dfrn:note[1]/text())', $suggestion);
 
-               return FContact::addSuggestion($importer['importer_uid'], $cid, $importer['id'], $note);
+               return self::addSuggestion($importer['importer_uid'], $cid, $importer['id'], $note);
+       }
+
+       /**
+        * Suggest a given contact to a given user from a given contact
+        *
+        * @param integer $uid
+        * @param integer $cid
+        * @param integer $from_cid
+        * @return bool   Was the adding successful?
+        */
+       private static function addSuggestion(int $uid, int $cid, int $from_cid, string $note = '')
+       {
+               $owner = User::getOwnerDataById($uid);
+               $contact = Contact::getById($cid);
+               $from_contact = Contact::getById($from_cid);
+
+               if (DBA::exists('contact', ['nurl' => Strings::normaliseLink($contact['url']), 'uid' => $uid])) {
+                       return false;
+               }
+
+               // Quit if we already have an introduction for this person
+               if (DI::intro()->suggestionExistsForUser($cid, $uid)) {
+                       return false;
+               }
+
+               $suggest = [];
+               $suggest['uid'] = $uid;
+               $suggest['cid'] = $from_cid;
+               $suggest['url'] = $contact['url'];
+               $suggest['name'] = $contact['name'];
+               $suggest['photo'] = $contact['photo'];
+               $suggest['request'] = $contact['request'];
+               $suggest['title'] = '';
+               $suggest['body'] = $note;
+
+               DI::intro()->save(DI::introFactory()->createNew(
+                       $suggest['uid'],
+                       $suggest['cid'],
+                       $suggest['body'],
+                       null,
+                       $cid
+               ));
+
+               DI::notify()->createFromArray([
+                       'type'  => Notification\Type::SUGGEST,
+                       'otype' => Notification\ObjectType::INTRO,
+                       'verb'  => Activity::REQ_FRIEND,
+                       'uid'   => $owner['uid'],
+                       'cid'   => $from_contact['uid'],
+                       'item'  => $suggest,
+                       'link'  => DI::baseUrl().'/notifications/intros',
+               ]);
+
+               return true;
        }
 
        /**
@@ -1349,7 +1442,7 @@ class DFRN
         */
        private static function processRelocation($xpath, $relocation, $importer)
        {
-               Logger::log("Processing relocations");
+               Logger::notice("Processing relocations");
 
                /// @TODO Rewrite this to one statement
                $relocate = [];
@@ -1396,7 +1489,7 @@ class DFRN
 
                Contact::updateAvatar($importer["id"], $relocate["avatar"], true);
 
-               Logger::log('Contacts are updated.');
+               Logger::notice('Contacts are updated.');
 
                /// @TODO
                /// merge with current record, current contents have priority
@@ -1454,32 +1547,21 @@ class DFRN
                if ($item["thr-parent"] != $item["uri"]) {
                        $community = false;
 
-                       if ($importer["page-flags"] == User::PAGE_FLAGS_COMMUNITY || $importer["page-flags"] == User::PAGE_FLAGS_PRVGROUP) {
+                       if ($importer['account-type'] == User::ACCOUNT_TYPE_COMMUNITY) {
                                $sql_extra = "";
                                $community = true;
-                               Logger::log("possible community action");
+                               Logger::notice("possible community action");
                        } else {
                                $sql_extra = " AND `self` AND `wall`";
                        }
 
                        // was the top-level post for this action written by somebody on this site?
                        // Specifically, the recipient?
-                       $parent = Post::selectFirst(['forum_mode', 'wall'],
+                       $parent = Post::selectFirst(['wall'],
                                ["`uri` = ? AND `uid` = ?" . $sql_extra, $item["thr-parent"], $importer["importer_uid"]]);
 
                        $is_a_remote_action = DBA::isResult($parent);
 
-                       /*
-                        * Does this have the characteristics of a community or private group action?
-                        * If it's an action to a wall post on a community/prvgroup page it's a
-                        * valid community action. Also forum_mode makes it valid for sure.
-                        * If neither, it's not.
-                        */
-                       if ($is_a_remote_action && $community && (!$parent["forum_mode"]) && (!$parent["wall"])) {
-                               $is_a_remote_action = false;
-                               Logger::log("not a community action");
-                       }
-
                        if ($is_a_remote_action) {
                                return DFRN::REPLY_RC;
                        } else {
@@ -1528,7 +1610,7 @@ class DFRN
                                $item['parent'] = $parent['id'];
 
                                // send a notification
-                               notification(
+                               DI::notify()->createFromArray(
                                        [
                                        "type"     => Notification\Type::POKE,
                                        "otype"    => Notification\ObjectType::PERSON,
@@ -1558,7 +1640,7 @@ class DFRN
         */
        private static function processVerbs($entrytype, $importer, &$item, &$is_like)
        {
-               Logger::log("Process verb ".$item["verb"]." and object-type ".$item["object-type"]." for entrytype ".$entrytype, Logger::DEBUG);
+               Logger::info("Process verb ".$item["verb"]." and object-type ".$item["object-type"]." for entrytype ".$entrytype);
 
                if (($entrytype == DFRN::TOP_LEVEL) && !empty($importer['id'])) {
                        // The filling of the the "contact" variable is done for legcy reasons
@@ -1570,23 +1652,23 @@ class DFRN
                        // Big question: Do we need these functions? They were part of the "consume_feed" function.
                        // This function once was responsible for DFRN and OStatus.
                        if ($activity->match($item["verb"], Activity::FOLLOW)) {
-                               Logger::log("New follower");
+                               Logger::notice("New follower");
                                Contact::addRelationship($importer, $contact, $item);
                                return false;
                        }
                        if ($activity->match($item["verb"], Activity::UNFOLLOW)) {
-                               Logger::log("Lost follower");
+                               Logger::notice("Lost follower");
                                Contact::removeFollower($contact);
                                return false;
                        }
                        if ($activity->match($item["verb"], Activity::REQ_FRIEND)) {
-                               Logger::log("New friend request");
+                               Logger::notice("New friend request");
                                Contact::addRelationship($importer, $contact, $item, true);
                                return false;
                        }
                        if ($activity->match($item["verb"], Activity::UNFRIEND)) {
-                               Logger::log("Lost sharer");
-                               Contact::removeSharer($importer, $contact, $item);
+                               Logger::notice("Lost sharer");
+                               Contact::removeSharer($contact);
                                return false;
                        }
                } else {
@@ -1630,7 +1712,7 @@ class DFRN
                                        $item_tag = Post::selectFirst(['id', 'uri-id'], ['uri' => $xt->id, 'uid' => $importer["importer_uid"]]);
 
                                        if (!DBA::isResult($item_tag)) {
-                                               Logger::log("Query failed to execute, no result returned in " . __FUNCTION__);
+                                               Logger::notice("Query failed to execute, no result returned in " . __FUNCTION__);
                                                return false;
                                        }
 
@@ -1687,19 +1769,34 @@ class DFRN
         * Checks if an incoming message is wanted
         *
         * @param array $item
+        * @param array $imporer
         * @return boolean Is the message wanted?
         */
-       private static function isSolicitedMessage(array $item)
+       private static function isSolicitedMessage(array $item, array $importer)
        {
                if (DBA::exists('contact', ["`nurl` = ? AND `uid` != ? AND `rel` IN (?, ?)",
                        Strings::normaliseLink($item["author-link"]), 0, Contact::FRIEND, Contact::SHARING])) {
-                       Logger::info('Author has got followers - accepted', ['uri' => $item['uri'], 'author' => $item["author-link"]]);
+                       Logger::debug('Author has got followers - accepted', ['uri-id' => $item['uri-id'], 'guid' => $item['guid'], 'url' => $item['uri'], 'author' => $item["author-link"]]);
+                       return true;
+               }
+
+               if ($importer['importer_uid'] != 0) {
+                       Logger::debug('Message is directed to a user - accepted', ['uri-id' => $item['uri-id'], 'guid' => $item['guid'], 'url' => $item['uri'], 'importer' => $importer['importer_uid']]);
+                       return true;
+               }
+
+               if ($item['uri'] != $item['thr-parent']) {
+                       Logger::debug('Message is no parent - accepted', ['uri-id' => $item['uri-id'], 'guid' => $item['guid'], 'url' => $item['uri']]);
                        return true;
                }
 
-               $taglist = Tag::getByURIId($item['uri-id'], [Tag::HASHTAG]);
-               $tags = array_column($taglist, 'name');
-               return Relay::isSolicitedPost($tags, $item['body'], $item['author-id'], $item['uri'], Protocol::DFRN);
+               $tags = array_column(Tag::getByURIId($item['uri-id'], [Tag::HASHTAG]), 'name');
+               if (Relay::isSolicitedPost($tags, $item['body'], $item['author-id'], $item['uri'], Protocol::DFRN)) {
+                       Logger::debug('Post is accepted because of the relay settings', ['uri-id' => $item['uri-id'], 'guid' => $item['guid'], 'url' => $item['uri'], 'author' => $item["author-link"]]);
+                       return true;
+               } else {
+                       return false;
+               }
        }
 
        /**
@@ -1717,7 +1814,7 @@ class DFRN
         */
        private static function processEntry($header, $xpath, $entry, $importer, $xml, $protocol)
        {
-               Logger::log("Processing entries");
+               Logger::notice("Processing entries");
 
                $item = $header;
 
@@ -1735,7 +1832,7 @@ class DFRN
                );
                // Is there an existing item?
                if (DBA::isResult($current) && !self::isEditedTimestampNewer($current, $item)) {
-                       Logger::log("Item ".$item["uri"]." (".$item['edited'].") already existed.", Logger::DEBUG);
+                       Logger::info("Item ".$item["uri"]." (".$item['edited'].") already existed.");
                        return;
                }
 
@@ -1900,11 +1997,9 @@ class DFRN
                }
 
                // Check if the message is wanted
-               if (($importer['importer_uid'] == 0) && ($item['uri'] == $item['thr-parent'])) {
-                       if (!self::isSolicitedMessage($item)) {
-                               DBA::delete('item-uri', ['uri' => $item['uri']]);
-                               return 403;
-                       }
+               if (!self::isSolicitedMessage($item, $importer)) {
+                       DBA::delete('item-uri', ['uri' => $item['uri']]);
+                       return 403;
                }
 
                // Get the type of the item (Top level post, reply or remote reply)
@@ -1945,10 +2040,10 @@ class DFRN
 
                        // Is it an event?
                        if (($item["object-type"] == Activity\ObjectType::EVENT) && !$owner_unknown) {
-                               Logger::log("Item ".$item["uri"]." seems to contain an event.", Logger::DEBUG);
+                               Logger::info("Item ".$item["uri"]." seems to contain an event.");
                                $ev = Event::fromBBCode($item["body"]);
                                if ((!empty($ev['desc']) || !empty($ev['summary'])) && !empty($ev['start'])) {
-                                       Logger::log("Event in item ".$item["uri"]." was found.", Logger::DEBUG);
+                                       Logger::info("Event in item ".$item["uri"]." was found.");
                                        $ev["cid"]       = $importer["id"];
                                        $ev["uid"]       = $importer["importer_uid"];
                                        $ev["uri"]       = $item["uri"];
@@ -1976,13 +2071,13 @@ class DFRN
                }
 
                if (!self::processVerbs($entrytype, $importer, $item, $is_like)) {
-                       Logger::log("Exiting because 'processVerbs' told us so", Logger::DEBUG);
+                       Logger::info("Exiting because 'processVerbs' told us so");
                        return;
                }
 
                // This check is done here to be able to receive connection requests in "processVerbs"
                if (($entrytype == DFRN::TOP_LEVEL) && $owner_unknown) {
-                       Logger::log("Item won't be stored because user " . $importer["importer_uid"] . " doesn't follow " . $item["owner-link"] . ".", Logger::DEBUG);
+                       Logger::info("Item won't be stored because user " . $importer["importer_uid"] . " doesn't follow " . $item["owner-link"] . ".");
                        return;
                }
 
@@ -1990,9 +2085,9 @@ class DFRN
                // Update content if 'updated' changes
                if (DBA::isResult($current)) {
                        if (self::updateContent($current, $item, $importer, $entrytype)) {
-                               Logger::log("Item ".$item["uri"]." was updated.", Logger::DEBUG);
+                               Logger::info("Item ".$item["uri"]." was updated.");
                        } else {
-                               Logger::log("Item " . $item["uri"] . " already existed.", Logger::DEBUG);
+                               Logger::info("Item " . $item["uri"] . " already existed.");
                        }
                        return;
                }
@@ -2005,7 +2100,7 @@ class DFRN
 
                        $posted_id = Item::insert($item);
                        if ($posted_id) {
-                               Logger::log("Reply from contact ".$item["contact-id"]." was stored with id ".$posted_id, Logger::DEBUG);
+                               Logger::info("Reply from contact ".$item["contact-id"]." was stored with id ".$posted_id);
 
                                if ($item['uid'] == 0) {
                                        Item::distribute($posted_id);
@@ -2015,7 +2110,7 @@ class DFRN
                        }
                } else { // $entrytype == DFRN::TOP_LEVEL
                        if (($importer["uid"] == 0) && ($importer["importer_uid"] != 0)) {
-                               Logger::log("Contact ".$importer["id"]." isn't known to user ".$importer["importer_uid"].". The post will be ignored.", Logger::DEBUG);
+                               Logger::info("Contact ".$importer["id"]." isn't known to user ".$importer["importer_uid"].". The post will be ignored.");
                                return;
                        }
                        if (!Strings::compareLink($item["owner-link"], $importer["url"])) {
@@ -2025,13 +2120,13 @@ class DFRN
                                 * the tgroup delivery code called from Item::insert will correct it if it's a forum,
                                 * but we're going to unconditionally correct it here so that the post will always be owned by our contact.
                                 */
-                               Logger::log('Correcting item owner.', Logger::DEBUG);
+                               Logger::info('Correcting item owner.');
                                $item["owner-link"] = $importer["url"];
                                $item["owner-id"] = Contact::getIdForURL($importer["url"], 0);
                        }
 
                        if (($importer["rel"] == Contact::FOLLOWER) && (!self::tgroupCheck($importer["importer_uid"], $item))) {
-                               Logger::log("Contact ".$importer["id"]." is only follower and tgroup check was negative.", Logger::DEBUG);
+                               Logger::info("Contact ".$importer["id"]." is only follower and tgroup check was negative.");
                                return;
                        }
 
@@ -2045,7 +2140,7 @@ class DFRN
                                $posted_id = $notify;
                        }
 
-                       Logger::log("Item was stored with id ".$posted_id, Logger::DEBUG);
+                       Logger::info("Item was stored with id ".$posted_id);
 
                        if ($item['uid'] == 0) {
                                Item::distribute($posted_id);
@@ -2070,7 +2165,7 @@ class DFRN
         */
        private static function processDeletion($xpath, $deletion, $importer)
        {
-               Logger::log("Processing deletions");
+               Logger::notice("Processing deletions");
                $uri = null;
 
                foreach ($deletion->attributes as $attributes) {
@@ -2086,7 +2181,7 @@ class DFRN
                $condition = ['uri' => $uri, 'uid' => $importer["importer_uid"]];
                $item = Post::selectFirst(['id', 'parent', 'contact-id', 'uri-id', 'deleted', 'gravity'], $condition);
                if (!DBA::isResult($item)) {
-                       Logger::log("Item with uri " . $uri . " for user " . $importer["importer_uid"] . " wasn't found.", Logger::DEBUG);
+                       Logger::info("Item with uri " . $uri . " for user " . $importer["importer_uid"] . " wasn't found.");
                        return;
                }
 
@@ -2097,7 +2192,7 @@ class DFRN
 
                // When it is a starting post it has to belong to the person that wants to delete it
                if (($item['gravity'] == GRAVITY_PARENT) && ($item['contact-id'] != $importer["id"])) {
-                       Logger::log("Item with uri " . $uri . " don't belong to contact " . $importer["id"] . " - ignoring deletion.", Logger::DEBUG);
+                       Logger::info("Item with uri " . $uri . " don't belong to contact " . $importer["id"] . " - ignoring deletion.");
                        return;
                }
 
@@ -2105,7 +2200,7 @@ class DFRN
                if (($item['gravity'] != GRAVITY_PARENT) && ($item['contact-id'] != $importer["id"])) {
                        $condition = ['id' => $item['parent'], 'contact-id' => $importer["id"]];
                        if (!Post::exists($condition)) {
-                               Logger::log("Item with uri " . $uri . " wasn't found or mustn't be deleted by contact " . $importer["id"] . " - ignoring deletion.", Logger::DEBUG);
+                               Logger::info("Item with uri " . $uri . " wasn't found or mustn't be deleted by contact " . $importer["id"] . " - ignoring deletion.");
                                return;
                        }
                }
@@ -2114,7 +2209,7 @@ class DFRN
                        return;
                }
 
-               Logger::log('deleting item '.$item['id'].' uri='.$uri, Logger::DEBUG);
+               Logger::info('deleting item '.$item['id'].' uri='.$uri);
 
                Item::markForDeletion(['id' => $item['id']]);
        }
@@ -2176,7 +2271,7 @@ class DFRN
                        self::fetchauthor($xpath, $doc->firstChild, $importer, "dfrn:owner", false, $xml);
                }
 
-               Logger::log("Import DFRN message for user " . $importer["importer_uid"] . " from contact " . $importer["id"], Logger::DEBUG);
+               Logger::info("Import DFRN message for user " . $importer["importer_uid"] . " from contact " . $importer["id"]);
 
                if (!empty($importer['gsid']) && ($protocol == Conversation::PARCEL_DIASPORA_DFRN)) {
                        GServer::setProtocol($importer['gsid'], Post\DeliveryData::DFRN);
@@ -2259,7 +2354,7 @@ class DFRN
                        self::processEntry($header, $xpath, $entry, $importer, $xml, $protocol);
                }
 
-               Logger::log("Import done for user " . $importer["importer_uid"] . " from contact " . $importer["id"], Logger::DEBUG);
+               Logger::info("Import done for user " . $importer["importer_uid"] . " from contact " . $importer["id"]);
                return 200;
        }
 
@@ -2288,14 +2383,11 @@ class DFRN
                        return false;
                }
 
-               $user = DBA::selectFirst('user', ['page-flags', 'nickname'], ['uid' => $uid]);
+               $user = DBA::selectFirst('user', ['account-type', 'nickname'], ['uid' => $uid]);
                if (!DBA::isResult($user)) {
                        return false;
                }
 
-               $community_page = ($user['page-flags'] == User::PAGE_FLAGS_COMMUNITY);
-               $prvgroup = ($user['page-flags'] == User::PAGE_FLAGS_PRVGROUP);
-
                $link = Strings::normaliseLink(DI::baseUrl() . '/profile/' . $user['nickname']);
 
                /*
@@ -2309,7 +2401,7 @@ class DFRN
                        foreach ($matches as $mtch) {
                                if (Strings::compareLink($link, $mtch[1]) || Strings::compareLink($dlink, $mtch[1])) {
                                        $mention = true;
-                                       Logger::log('mention found: ' . $mtch[2]);
+                                       Logger::notice('mention found: ' . $mtch[2]);
                                }
                        }
                }
@@ -2318,7 +2410,7 @@ class DFRN
                        return false;
                }
 
-               return $community_page || $prvgroup;
+               return ($user['account-type'] == User::ACCOUNT_TYPE_COMMUNITY);
        }
 
        /**