]> git.mxchange.org Git - friendica.git/blobdiff - src/Protocol/Diaspora.php
Whitespace removed
[friendica.git] / src / Protocol / Diaspora.php
index 15f9fa3d12109c84addc88365b4f86afa0337630..828333b41f10656119947bf96dbd1aa7a9fc246d 100644 (file)
@@ -1,11 +1,22 @@
 <?php
 /**
- * @file src/Protocol/diaspora.php
- * @brief The implementation of the diaspora protocol
+ * @copyright Copyright (C) 2020, Friendica
+ *
+ * @license GNU AGPL version 3 or any later version
+ *
+ * This program is free software: you can redistribute it and/or modify
+ * it under the terms of the GNU Affero General Public License as
+ * published by the Free Software Foundation, either version 3 of the
+ * License, or (at your option) any later version.
+ *
+ * This program is distributed in the hope that it will be useful,
+ * but WITHOUT ANY WARRANTY; without even the implied warranty of
+ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+ * GNU Affero General Public License for more details.
+ *
+ * You should have received a copy of the GNU Affero General Public License
+ * along with this program.  If not, see <https://www.gnu.org/licenses/>.
  *
- * The new protocol is described here: http://diaspora.github.io/diaspora_federation/index.html
- * This implementation here interprets the old and the new protocol and sends the new one.
- * In the future we will remove most stuff from "validPosting" and interpret only the new protocol.
  */
 
 namespace Friendica\Protocol;
@@ -14,8 +25,6 @@ use Friendica\Content\Feature;
 use Friendica\Content\Text\BBCode;
 use Friendica\Content\Text\Markdown;
 use Friendica\Core\Cache\Duration;
-use Friendica\Core\Config;
-use Friendica\Core\L10n;
 use Friendica\Core\Logger;
 use Friendica\Core\Protocol;
 use Friendica\Core\System;
@@ -41,8 +50,7 @@ use Friendica\Worker\Delivery;
 use SimpleXMLElement;
 
 /**
- * @brief This class contain functions to create and send Diaspora XML files
- *
+ * This class contain functions to create and send Diaspora XML files
  */
 class Diaspora
 {
@@ -77,7 +85,7 @@ class Diaspora
        }
 
        /**
-        * @brief Return a list of relay servers
+        * Return a list of relay servers
         *
         * The list contains not only the official relays but also servers that we serve directly
         *
@@ -92,7 +100,7 @@ class Diaspora
                $serverlist = [];
 
                // Fetching relay servers
-               $serverdata = Config::get("system", "relay_server");
+               $serverdata = DI::config()->get("system", "relay_server");
 
                if (!empty($serverdata)) {
                        $servers = explode(",", $serverdata);
@@ -101,7 +109,7 @@ class Diaspora
                        }
                }
 
-               if (Config::get("system", "relay_directly", false)) {
+               if (DI::config()->get("system", "relay_directly", false)) {
                        // We distribute our stuff based on the parent to ensure that the thread will be complete
                        $parent = Item::selectFirst(['parent'], ['id' => $item_id]);
                        if (!DBA::isResult($parent)) {
@@ -167,7 +175,7 @@ class Diaspora
        }
 
        /**
-        * @brief Return a contact for a given server address or creates a dummy entry
+        * Return a contact for a given server address or creates a dummy entry
         *
         * @param string $server_url The url of the server
         * @param array $fields Fieldlist
@@ -200,7 +208,7 @@ class Diaspora
        }
 
        /**
-        * @brief Update or insert a relay contact
+        * Update or insert a relay contact
         *
         * @param string $server_url     The url of the server
         * @param array  $network_fields Optional network specific fields
@@ -234,7 +242,7 @@ class Diaspora
        }
 
        /**
-        * @brief Return a list of participating contacts for a thread
+        * Return a list of participating contacts for a thread
         *
         * This is used for the participation feature.
         * One of the parameters is a contact array.
@@ -286,7 +294,7 @@ class Diaspora
        }
 
        /**
-        * @brief repairs a signature that was double encoded
+        * repairs a signature that was double encoded
         *
         * The function is unused at the moment. It was copied from the old implementation.
         *
@@ -317,7 +325,7 @@ class Diaspora
        }
 
        /**
-        * @brief verify the envelope and return the verified data
+        * verify the envelope and return the verified data
         *
         * @param string $envelope The magic envelope
         *
@@ -382,7 +390,7 @@ class Diaspora
        }
 
        /**
-        * @brief encrypts data via AES
+        * encrypts data via AES
         *
         * @param string $key  The AES key
         * @param string $iv   The IV (is used for CBC encoding)
@@ -396,7 +404,7 @@ class Diaspora
        }
 
        /**
-        * @brief decrypts data via AES
+        * decrypts data via AES
         *
         * @param string $key       The AES key
         * @param string $iv        The IV (is used for CBC encoding)
@@ -410,7 +418,7 @@ class Diaspora
        }
 
        /**
-        * @brief: Decodes incoming Diaspora message in the new format
+        * Decodes incoming Diaspora message in the new format
         *
         * @param string  $raw      raw post message
         * @param string  $privKey   The private key of the importer
@@ -516,7 +524,7 @@ class Diaspora
        }
 
        /**
-        * @brief: Decodes incoming Diaspora message in the deprecated format
+        * Decodes incoming Diaspora message in the deprecated format
         *
         * @param string $xml      urldecoded Diaspora salmon
         * @param string $privKey  The private key of the importer
@@ -659,7 +667,7 @@ class Diaspora
 
 
        /**
-        * @brief Dispatches public messages and find the fitting receivers
+        * Dispatches public messages and find the fitting receivers
         *
         * @param array $msg The post that will be dispatched
         *
@@ -669,7 +677,7 @@ class Diaspora
         */
        public static function dispatchPublic($msg)
        {
-               $enabled = intval(Config::get("system", "diaspora_enabled"));
+               $enabled = intval(DI::config()->get("system", "diaspora_enabled"));
                if (!$enabled) {
                        Logger::log("diaspora is disabled");
                        return false;
@@ -687,7 +695,7 @@ class Diaspora
        }
 
        /**
-        * @brief Dispatches the different message types to the different functions
+        * Dispatches the different message types to the different functions
         *
         * @param array            $importer Array of the importer user
         * @param array            $msg      The post that will be dispatched
@@ -792,7 +800,7 @@ class Diaspora
        }
 
        /**
-        * @brief Checks if a posting is valid and fetches the data fields.
+        * Checks if a posting is valid and fetches the data fields.
         *
         * This function does not only check the signature.
         * It also does the conversion between the old and the new diaspora format.
@@ -945,7 +953,7 @@ class Diaspora
        }
 
        /**
-        * @brief Fetches the public key for a given handle
+        * Fetches the public key for a given handle
         *
         * @param string $handle The handle
         *
@@ -968,7 +976,7 @@ class Diaspora
        }
 
        /**
-        * @brief Fetches data for a given handle
+        * Fetches data for a given handle
         *
         * @param string $handle The handle
         * @param boolean $update true = always update, false = never update, null = update when not found or outdated
@@ -1022,7 +1030,7 @@ class Diaspora
        }
 
        /**
-        * @brief Updates the fcontact table
+        * Updates the fcontact table
         *
         * @param array $arr The fcontact data
         * @throws \Exception
@@ -1043,7 +1051,7 @@ class Diaspora
        }
 
        /**
-        * @brief get a handle (user@domain.tld) from a given contact id
+        * get a handle (user@domain.tld) from a given contact id
         *
         * @param int $contact_id  The id in the contact table
         * @param int $pcontact_id The id in the contact table (Used for the public contact)
@@ -1090,7 +1098,7 @@ class Diaspora
        }
 
        /**
-        * @brief get a url (scheme://domain.tld/u/user) from a given Diaspora*
+        * get a url (scheme://domain.tld/u/user) from a given Diaspora*
         * fcontact guid
         *
         * @param mixed $fcontact_guid Hexadecimal string guid
@@ -1116,7 +1124,7 @@ class Diaspora
        }
 
        /**
-        * @brief Get a contact id for a given handle
+        * Get a contact id for a given handle
         *
         * @todo  Move to Friendica\Model\Contact
         *
@@ -1160,7 +1168,7 @@ class Diaspora
        }
 
        /**
-        * @brief Check if posting is allowed for this contact
+        * Check if posting is allowed for this contact
         *
         * @param array $importer   Array of the importer user
         * @param array $contact    The contact that is checked
@@ -1213,7 +1221,7 @@ class Diaspora
        }
 
        /**
-        * @brief Fetches the contact id for a handle and checks if posting is allowed
+        * Fetches the contact id for a handle and checks if posting is allowed
         *
         * @param array  $importer   Array of the importer user
         * @param string $handle     The checked handle in the format user@domain.tld
@@ -1245,7 +1253,7 @@ class Diaspora
        }
 
        /**
-        * @brief Does the message already exists on the system?
+        * Does the message already exists on the system?
         *
         * @param int    $uid  The user id
         * @param string $guid The guid of the message
@@ -1265,7 +1273,7 @@ class Diaspora
        }
 
        /**
-        * @brief Checks for links to posts in a message
+        * Checks for links to posts in a message
         *
         * @param array $item The item array
         * @return void
@@ -1291,7 +1299,7 @@ class Diaspora
        }
 
        /**
-        * @brief Checks for relative /people/* links in an item body to match local
+        * Checks for relative /people/* links in an item body to match local
         * contacts or prepends the remote host taken from the author link.
         *
         * @param string $body        The item body to replace links from
@@ -1327,7 +1335,7 @@ class Diaspora
        }
 
        /**
-        * @brief sub function of "fetchGuid" which checks for links in messages
+        * sub function of "fetchGuid" which checks for links in messages
         *
         * @param array $match array containing a link that has to be checked for a message link
         * @param array $item  The item array
@@ -1343,7 +1351,7 @@ class Diaspora
        }
 
        /**
-        * @brief Fetches an item with a given guid from a given server
+        * Fetches an item with a given guid from a given server
         *
         * @param string $guid   the message guid
         * @param string $server The server address
@@ -1378,7 +1386,7 @@ class Diaspora
        }
 
        /**
-        * @brief Fetches a message from a server
+        * Fetches a message from a server
         *
         * @param string $guid   message guid
         * @param string $server The url of the server
@@ -1457,7 +1465,7 @@ class Diaspora
        }
 
        /**
-        * @brief Fetches an item with a given URL
+        * Fetches an item with a given URL
         *
         * @param string $url the message url
         *
@@ -1468,7 +1476,8 @@ class Diaspora
        public static function fetchByURL($url, $uid = 0)
        {
                // Check for Diaspora (and Friendica) typical paths
-               if (!preg_match("=(https?://.+)/(?:posts|display)/([a-zA-Z0-9-_@.:%]+[a-zA-Z0-9])=i", $url, $matches)) {
+               if (!preg_match("=(https?://.+)/(?:posts|display|objects)/([a-zA-Z0-9-_@.:%]+[a-zA-Z0-9])=i", $url, $matches)) {
+                       Logger::info('Invalid url', ['url' => $url]);
                        return false;
                }
 
@@ -1476,21 +1485,26 @@ class Diaspora
 
                $item = Item::selectFirst(['id'], ['guid' => $guid, 'uid' => $uid]);
                if (DBA::isResult($item)) {
+                       Logger::info('Found', ['id' => $item['id']]);
                        return $item['id'];
                }
 
-               self::storeByGuid($guid, $matches[1], $uid);
+               Logger::info('Fetch GUID from origin', ['guid' => $guid, 'server' => $matches[1]]);
+               $ret = self::storeByGuid($guid, $matches[1], $uid);
+               Logger::info('Result', ['ret' => $ret]);
 
                $item = Item::selectFirst(['id'], ['guid' => $guid, 'uid' => $uid]);
                if (DBA::isResult($item)) {
+                       Logger::info('Found', ['id' => $item['id']]);
                        return $item['id'];
                } else {
+                       Logger::info('Not found', ['guid' => $guid, 'uid' => $uid]);
                        return false;
                }
        }
 
        /**
-        * @brief Fetches the item record of a given guid
+        * Fetches the item record of a given guid
         *
         * @param int    $uid     The user id
         * @param string $guid    message guid
@@ -1534,7 +1548,7 @@ class Diaspora
        }
 
        /**
-        * @brief returns contact details
+        * returns contact details
         *
         * @param array $def_contact The default contact if the person isn't found
         * @param array $person      The record of the person
@@ -1561,19 +1575,19 @@ class Diaspora
        }
 
        /**
-        * @brief Is the profile a hubzilla profile?
+        * Is the profile a hubzilla profile?
         *
         * @param string $url The profile link
         *
         * @return bool is it a hubzilla server?
         */
-       public static function isRedmatrix($url)
+       private static function isHubzilla($url)
        {
-               return(strstr($url, "/channel/"));
+               return(strstr($url, '/channel/'));
        }
 
        /**
-        * @brief Generate a post link with a given handle and message guid
+        * Generate a post link with a given handle and message guid
         *
         * @param string $addr        The user handle
         * @param string $guid        message guid
@@ -1586,33 +1600,59 @@ class Diaspora
        private static function plink($addr, $guid, $parent_guid = '')
        {
                $contact = Contact::getDetailsByAddr($addr);
+               if (empty($contact)) {
+                       Logger::info('No contact data for address', ['addr' => $addr]);
+                       return '';
+               }
 
-               // Fallback
-               if (!$contact) {
-                       if ($parent_guid != '') {
-                               return "https://" . substr($addr, strpos($addr, "@") + 1) . "/posts/" . $parent_guid . "#" . $guid;
-                       } else {
-                               return "https://" . substr($addr, strpos($addr, "@") + 1) . "/posts/" . $guid;
+               if (empty($contact['baseurl'])) {
+                       $contact['baseurl'] = 'https://' . substr($addr, strpos($addr, '@') + 1);
+                       Logger::info('Create baseurl from address', ['baseurl' => $contact['baseurl'], 'url' => $contact['url']]);
+               }
+
+               $platform = '';
+               $gserver = DBA::selectFirst('gserver', ['platform'], ['nurl' => Strings::normaliseLink($contact['baseurl'])]);
+               if (!empty($gserver['platform'])) {
+                       $platform = strtolower($gserver['platform']);
+                       Logger::info('Detected platform', ['platform' => $platform, 'url' => $contact['url']]);
+               }
+
+               if (!in_array($platform, ['diaspora', 'friendica', 'hubzilla', 'socialhome'])) {
+                       if (self::isHubzilla($contact['url'])) {
+                               Logger::info('Detected unknown platform as Hubzilla', ['platform' => $platform, 'url' => $contact['url']]);
+                               $platform = 'hubzilla';
+                       } elseif ($contact['network'] == Protocol::DFRN) {
+                               Logger::info('Detected unknown platform as Friendica', ['platform' => $platform, 'url' => $contact['url']]);
+                               $platform = 'friendica';
                        }
                }
 
-               if ($contact["network"] == Protocol::DFRN) {
-                       return str_replace("/profile/" . $contact["nick"] . "/", "/display/" . $guid, $contact["url"] . "/");
+               if ($platform == 'friendica') {
+                       return str_replace('/profile/' . $contact['nick'] . '/', '/display/' . $guid, $contact['url'] . '/');
+               }
+
+               if ($platform == 'hubzilla') {
+                       return $contact['baseurl'] . '/item/' . $guid;
+               }
+
+               if ($platform == 'socialhome') {
+                       return $contact['baseurl'] . '/content/' . $guid;
                }
 
-               if (self::isRedmatrix($contact["url"])) {
-                       return $contact["url"] . "/?mid=" . $guid;
+               if ($platform != 'diaspora') {
+                       Logger::info('Unknown platform', ['platform' => $platform, 'url' => $contact['url']]);
+                       return '';
                }
 
                if ($parent_guid != '') {
-                       return "https://" . substr($addr, strpos($addr, "@") + 1) . "/posts/" . $parent_guid . "#" . $guid;
+                       return $contact['baseurl'] . '/posts/' . $parent_guid . '#' . $guid;
                } else {
-                       return "https://" . substr($addr, strpos($addr, "@") + 1) . "/posts/" . $guid;
+                       return $contact['baseurl'] . '/posts/' . $guid;
                }
        }
 
        /**
-        * @brief Receives account migration
+        * Receives account migration
         *
         * @param array  $importer Array of the importer user
         * @param object $data     The message object
@@ -1675,7 +1715,7 @@ class Diaspora
        }
 
        /**
-        * @brief Processes an account deletion
+        * Processes an account deletion
         *
         * @param object $data The message object
         *
@@ -1699,7 +1739,7 @@ class Diaspora
        }
 
        /**
-        * @brief Fetch the uri from our database if we already have this item (maybe from ourselves)
+        * Fetch the uri from our database if we already have this item (maybe from ourselves)
         *
         * @param string  $author    Author handle
         * @param string  $guid      Message guid
@@ -1728,7 +1768,7 @@ class Diaspora
        }
 
        /**
-        * @brief Fetch the guid from our database with a given uri
+        * Fetch the guid from our database with a given uri
         *
         * @param string $uri Message uri
         * @param string $uid Author handle
@@ -1747,7 +1787,7 @@ class Diaspora
        }
 
        /**
-        * @brief Find the best importer for a comment, like, ...
+        * Find the best importer for a comment, like, ...
         *
         * @param string $guid The guid of the item
         *
@@ -1768,7 +1808,7 @@ class Diaspora
        }
 
        /**
-        * @brief Processes an incoming comment
+        * Processes an incoming comment
         *
         * @param array  $importer Array of the importer user
         * @param string $sender   The sender of the message
@@ -1855,7 +1895,6 @@ class Diaspora
                $datarray["changed"] = $datarray["created"] = $datarray["edited"] = $created_at;
 
                $datarray["plink"] = self::plink($author, $guid, $parent_item['guid']);
-
                $body = Markdown::toBBCode($text);
 
                $datarray["body"] = self::replacePeopleGuid($body, $person["url"]);
@@ -1885,7 +1924,7 @@ class Diaspora
        }
 
        /**
-        * @brief processes and stores private messages
+        * processes and stores private messages
         *
         * @param array  $importer     Array of the importer user
         * @param array  $contact      The contact of the message
@@ -1945,7 +1984,7 @@ class Diaspora
        }
 
        /**
-        * @brief Processes new private messages (answers to private messages are processed elsewhere)
+        * Processes new private messages (answers to private messages are processed elsewhere)
         *
         * @param array  $importer Array of the importer user
         * @param array  $msg      Array of the processed message, author handle and key
@@ -2004,7 +2043,7 @@ class Diaspora
        }
 
        /**
-        * @brief Processes "like" messages
+        * Processes "like" messages
         *
         * @param array  $importer Array of the importer user
         * @param string $sender   The sender of the message
@@ -2116,7 +2155,7 @@ class Diaspora
        }
 
        /**
-        * @brief Processes private messages
+        * Processes private messages
         *
         * @param array  $importer Array of the importer user
         * @param object $data     The message object
@@ -2177,7 +2216,7 @@ class Diaspora
        }
 
        /**
-        * @brief Processes participations - unsupported by now
+        * Processes participations - unsupported by now
         *
         * @param array  $importer Array of the importer user
         * @param object $data     The message object
@@ -2203,7 +2242,7 @@ class Diaspora
                        return false;
                }
 
-               $item = Item::selectFirst(['id'], ['guid' => $parent_guid, 'origin' => true, 'private' => false]);
+               $item = Item::selectFirst(['id'], ['guid' => $parent_guid, 'origin' => true, 'private' => [Item::PUBLIC, Item::UNLISTED]]);
                if (!DBA::isResult($item)) {
                        Logger::log('Item not found, no origin or private: '.$parent_guid);
                        return false;
@@ -2241,7 +2280,7 @@ class Diaspora
        }
 
        /**
-        * @brief Processes photos - unneeded
+        * Processes photos - unneeded
         *
         * @param array  $importer Array of the importer user
         * @param object $data     The message object
@@ -2256,7 +2295,7 @@ class Diaspora
        }
 
        /**
-        * @brief Processes poll participations - unssupported
+        * Processes poll participations - unssupported
         *
         * @param array  $importer Array of the importer user
         * @param object $data     The message object
@@ -2270,7 +2309,7 @@ class Diaspora
        }
 
        /**
-        * @brief Processes incoming profile updates
+        * Processes incoming profile updates
         *
         * @param array  $importer Array of the importer user
         * @param object $data     The message object
@@ -2291,7 +2330,6 @@ class Diaspora
                $name = XML::unescape($data->first_name).((strlen($data->last_name)) ? " ".XML::unescape($data->last_name) : "");
                $image_url = XML::unescape($data->image_url);
                $birthday = XML::unescape($data->birthday);
-               $gender = XML::unescape($data->gender);
                $about = Markdown::toBBCode(XML::unescape($data->bio));
                $location = Markdown::toBBCode(XML::unescape($data->location));
                $searchable = (XML::unescape($data->searchable) == "true");
@@ -2339,8 +2377,7 @@ class Diaspora
                }
 
                $fields = ['name' => $name, 'location' => $location,
-                       'name-date' => DateTimeFormat::utcNow(),
-                       'about' => $about, 'gender' => $gender,
+                       'name-date' => DateTimeFormat::utcNow(), 'about' => $about,
                        'addr' => $author, 'nick' => $nick, 'keywords' => $keywords,
                        'unsearchable' => !$searchable, 'sensitive' => $nsfw];
 
@@ -2354,7 +2391,7 @@ class Diaspora
 
                $gcontact = ["url" => $contact["url"], "network" => Protocol::DIASPORA, "generation" => 2,
                                        "photo" => $image_url, "name" => $name, "location" => $location,
-                                       "about" => $about, "birthday" => $birthday, "gender" => $gender,
+                                       "about" => $about, "birthday" => $birthday,
                                        "addr" => $author, "nick" => $nick, "keywords" => $keywords,
                                        "hide" => !$searchable, "nsfw" => $nsfw];
 
@@ -2368,7 +2405,7 @@ class Diaspora
        }
 
        /**
-        * @brief Processes incoming friend requests
+        * Processes incoming friend requests
         *
         * @param array $importer Array of the importer user
         * @param array $contact  The contact that send the request
@@ -2387,7 +2424,7 @@ class Diaspora
        }
 
        /**
-        * @brief Processes incoming sharing notification
+        * Processes incoming sharing notification
         *
         * @param array  $importer Array of the importer user
         * @param object $data     The message object
@@ -2499,7 +2536,7 @@ class Diaspora
        }
 
        /**
-        * @brief Fetches a message with a given guid
+        * Fetches a message with a given guid
         *
         * @param string $guid        message guid
         * @param string $orig_author handle of the original post
@@ -2517,7 +2554,7 @@ class Diaspora
                // Do we already have this item?
                $fields = ['body', 'title', 'attach', 'tag', 'app', 'created', 'object-type', 'uri', 'guid',
                        'author-name', 'author-link', 'author-avatar'];
-               $condition = ['guid' => $guid, 'visible' => true, 'deleted' => false, 'private' => false];
+               $condition = ['guid' => $guid, 'visible' => true, 'deleted' => false, 'private' => [Item::PUBLIC, Item::UNLISTED]];
                $item = Item::selectFirst($fields, $condition);
 
                if (DBA::isResult($item)) {
@@ -2561,7 +2598,7 @@ class Diaspora
                        if ($stored) {
                                $fields = ['body', 'title', 'attach', 'tag', 'app', 'created', 'object-type', 'uri', 'guid',
                                        'author-name', 'author-link', 'author-avatar'];
-                               $condition = ['guid' => $guid, 'visible' => true, 'deleted' => false, 'private' => false];
+                               $condition = ['guid' => $guid, 'visible' => true, 'deleted' => false, 'private' => [Item::PUBLIC, Item::UNLISTED]];
                                $item = Item::selectFirst($fields, $condition);
 
                                if (DBA::isResult($item)) {
@@ -2579,7 +2616,7 @@ class Diaspora
        }
 
        /**
-        * @brief Stores a reshare activity
+        * Stores a reshare activity
         *
         * @param array   $item              Array of reshare post
         * @param integer $parent_message_id Id of the parent post
@@ -2627,7 +2664,7 @@ class Diaspora
        }
 
        /**
-        * @brief Processes a reshare message
+        * Processes a reshare message
         *
         * @param array  $importer Array of the importer user
         * @param object $data     The message object
@@ -2705,7 +2742,7 @@ class Diaspora
                $datarray["app"]  = $original_item["app"];
 
                $datarray["plink"] = self::plink($author, $guid);
-               $datarray["private"] = (($public == "false") ? 1 : 0);
+               $datarray["private"] = (($public == "false") ? Item::PRIVATE : Item::PUBLIC);
                $datarray["changed"] = $datarray["created"] = $datarray["edited"] = $created_at;
 
                $datarray["object-type"] = $original_item["object-type"];
@@ -2732,7 +2769,7 @@ class Diaspora
        }
 
        /**
-        * @brief Processes retractions
+        * Processes retractions
         *
         * @param array  $importer Array of the importer user
         * @param array  $contact  The contact of the item owner
@@ -2788,7 +2825,7 @@ class Diaspora
                                continue;
                        }
 
-                       Item::delete(['id' => $item['id']]);
+                       Item::markForDeletion(['id' => $item['id']]);
 
                        Logger::log("Deleted target ".$target_guid." (".$item["id"].") from user ".$item["uid"]." parent: ".$item["parent"], Logger::DEBUG);
                }
@@ -2797,7 +2834,7 @@ class Diaspora
        }
 
        /**
-        * @brief Receives retraction messages
+        * Receives retraction messages
         *
         * @param array  $importer Array of the importer user
         * @param string $sender   The sender of the message
@@ -2843,7 +2880,7 @@ class Diaspora
        }
 
        /**
-        * @brief Receives status messages
+        * Receives status messages
         *
         * @param array            $importer Array of the importer user
         * @param SimpleXMLElement $data     The message object
@@ -2895,7 +2932,7 @@ class Diaspora
                        $datarray["object-type"] = Activity\ObjectType::NOTE;
 
                        // Add OEmbed and other information to the body
-                       if (!self::isRedmatrix($contact["url"])) {
+                       if (!self::isHubzilla($contact["url"])) {
                                $body = add_page_info_to_body($body, false, true);
                        }
                }
@@ -2935,7 +2972,7 @@ class Diaspora
                }
 
                $datarray["plink"] = self::plink($author, $guid);
-               $datarray["private"] = (($public == "false") ? 1 : 0);
+               $datarray["private"] = (($public == "false") ? Item::PRIVATE : Item::PUBLIC);
                $datarray["changed"] = $datarray["created"] = $datarray["edited"] = $created_at;
 
                if (isset($address["address"])) {
@@ -2967,7 +3004,7 @@ class Diaspora
         * ************************************************************************************** */
 
        /**
-        * @brief returnes the handle of a contact
+        * returnes the handle of a contact
         *
         * @param array $contact contact array
         *
@@ -2993,7 +3030,7 @@ class Diaspora
 
 
        /**
-        * @brief Creates the data for a private message in the new format
+        * Creates the data for a private message in the new format
         *
         * @param string $msg     The message that is to be transmitted
         * @param array  $user    The record of the sender
@@ -3035,7 +3072,7 @@ class Diaspora
        }
 
        /**
-        * @brief Creates the envelope for the "fetch" endpoint and for the new format
+        * Creates the envelope for the "fetch" endpoint and for the new format
         *
         * @param string $msg  The message that is to be transmitted
         * @param array  $user The record of the sender
@@ -3075,7 +3112,7 @@ class Diaspora
        }
 
        /**
-        * @brief Create the envelope for a message
+        * Create the envelope for a message
         *
         * @param string $msg     The message that is to be transmitted
         * @param array  $user    The record of the sender
@@ -3101,7 +3138,7 @@ class Diaspora
        }
 
        /**
-        * @brief Creates a signature for a message
+        * Creates a signature for a message
         *
         * @param array $owner   the array of the owner of the message
         * @param array $message The message that is to be signed
@@ -3120,7 +3157,7 @@ class Diaspora
        }
 
        /**
-        * @brief Transmit a message to a target server
+        * Transmit a message to a target server
         *
         * @param array  $owner        the array of the item owner
         * @param array  $contact      Target of the communication
@@ -3134,7 +3171,7 @@ class Diaspora
         */
        private static function transmit(array $owner, array $contact, $envelope, $public_batch, $guid = "")
        {
-               $enabled = intval(Config::get("system", "diaspora_enabled"));
+               $enabled = intval(DI::config()->get("system", "diaspora_enabled"));
                if (!$enabled) {
                        return 200;
                }
@@ -3161,7 +3198,7 @@ class Diaspora
 
                Logger::log("transmit: ".$logid."-".$guid." ".$dest_url);
 
-               if (!intval(Config::get("system", "diaspora_test"))) {
+               if (!intval(DI::config()->get("system", "diaspora_test"))) {
                        $content_type = (($public_batch) ? "application/magic-envelope+xml" : "application/json");
 
                        $postResult = Network::post($dest_url."/", $envelope, ["Content-Type: ".$content_type]);
@@ -3178,7 +3215,7 @@ class Diaspora
 
 
        /**
-        * @brief Build the post xml
+        * Build the post xml
         *
         * @param string $type    The message type
         * @param array  $message The message data
@@ -3193,7 +3230,7 @@ class Diaspora
        }
 
        /**
-        * @brief Builds and transmit messages
+        * Builds and transmit messages
         *
         * @param array  $owner        the array of the item owner
         * @param array  $contact      Target of the communication
@@ -3228,7 +3265,7 @@ class Diaspora
        }
 
        /**
-        * @brief sends a participation (Used to get all further updates)
+        * sends a participation (Used to get all further updates)
         *
         * @param array $contact Target of the communication
         * @param array $item    Item array
@@ -3239,7 +3276,7 @@ class Diaspora
        private static function sendParticipation(array $contact, array $item)
        {
                // Don't send notifications for private postings
-               if ($item['private']) {
+               if ($item['private'] == Item::PRIVATE) {
                        return;
                }
 
@@ -3277,7 +3314,7 @@ class Diaspora
        }
 
        /**
-        * @brief sends an account migration
+        * sends an account migration
         *
         * @param array $owner   the array of the item owner
         * @param array $contact Target of the communication
@@ -3305,7 +3342,7 @@ class Diaspora
        }
 
        /**
-        * @brief Sends a "share" message
+        * Sends a "share" message
         *
         * @param array $owner   the array of the item owner
         * @param array $contact Target of the communication
@@ -3349,7 +3386,7 @@ class Diaspora
        }
 
        /**
-        * @brief sends an "unshare"
+        * sends an "unshare"
         *
         * @param array $owner   the array of the item owner
         * @param array $contact Target of the communication
@@ -3370,7 +3407,7 @@ class Diaspora
        }
 
        /**
-        * @brief Checks a message body if it is a reshare
+        * Checks a message body if it is a reshare
         *
         * @param string $body     The message body that is to be check
         * @param bool   $complete Should it be a complete check or a simple check?
@@ -3428,7 +3465,7 @@ class Diaspora
        }
 
        /**
-        * @brief Create an event array
+        * Create an event array
         *
         * @param integer $event_id The id of the event
         *
@@ -3508,7 +3545,7 @@ class Diaspora
        }
 
        /**
-        * @brief Create a post (status message or reshare)
+        * Create a post (status message or reshare)
         *
         * @param array $item  The item that will be exported
         * @param array $owner the array of the item owner
@@ -3530,12 +3567,12 @@ class Diaspora
 
                $myaddr = self::myHandle($owner);
 
-               $public = ($item["private"] ? "false" : "true");
+               $public = ($item["private"] == Item::PRIVATE ? "false" : "true");
                $created = DateTimeFormat::utc($item['received'], DateTimeFormat::ATOM);
                $edited = DateTimeFormat::utc($item["edited"] ?? $item["created"], DateTimeFormat::ATOM);
 
                // Detect a share element and do a reshare
-               if (!$item['private'] && ($ret = self::isReshare($item["body"]))) {
+               if (($item['private'] != Item::PRIVATE) && ($ret = self::isReshare($item["body"]))) {
                        $message = ["author" => $myaddr,
                                        "guid" => $item["guid"],
                                        "created_at" => $created,
@@ -3574,7 +3611,7 @@ class Diaspora
                        if ($item["attach"]) {
                                $cnt = preg_match_all('/href=\"(.*?)\"(.*?)title=\"(.*?)\"/ism', $item["attach"], $matches, PREG_SET_ORDER);
                                if ($cnt) {
-                                       $body .= "\n".L10n::t("Attachments:")."\n";
+                                       $body .= "\n".DI::l10n()->t("Attachments:")."\n";
                                        foreach ($matches as $mtch) {
                                                $body .= "[".$mtch[3]."](".$mtch[1].")\n";
                                        }
@@ -3647,7 +3684,7 @@ class Diaspora
        }
 
        /**
-        * @brief Sends a post
+        * Sends a post
         *
         * @param array $item         The item that will be exported
         * @param array $owner        the array of the item owner
@@ -3666,7 +3703,7 @@ class Diaspora
        }
 
        /**
-        * @brief Creates a "like" object
+        * Creates a "like" object
         *
         * @param array $item  The item that will be exported
         * @param array $owner the array of the item owner
@@ -3698,7 +3735,7 @@ class Diaspora
        }
 
        /**
-        * @brief Creates an "EventParticipation" object
+        * Creates an "EventParticipation" object
         *
         * @param array $item  The item that will be exported
         * @param array $owner the array of the item owner
@@ -3736,7 +3773,7 @@ class Diaspora
        }
 
        /**
-        * @brief Creates the object for a comment
+        * Creates the object for a comment
         *
         * @param array $item  The item that will be exported
         * @param array $owner the array of the item owner
@@ -3773,7 +3810,7 @@ class Diaspora
                if (
                        $item['author-id'] != $thread_parent_item['author-id']
                        && (empty($item['uid']) || !Feature::isEnabled($item['uid'], 'explicit_mentions'))
-                       && !Config::get('system', 'disable_implicit_mentions')
+                       && !DI::config()->get('system', 'disable_implicit_mentions')
                ) {
                        $body = self::prependParentAuthorMention($body, $thread_parent_item['author-link']);
                }
@@ -3803,7 +3840,7 @@ class Diaspora
        }
 
        /**
-        * @brief Send a like or a comment
+        * Send a like or a comment
         *
         * @param array $item         The item that will be exported
         * @param array $owner        the array of the item owner
@@ -3837,7 +3874,7 @@ class Diaspora
        }
 
        /**
-        * @brief Creates a message from a signature record entry
+        * Creates a message from a signature record entry
         *
         * @param array $item The item that will be exported
         * @return array The message
@@ -3882,7 +3919,7 @@ class Diaspora
        }
 
        /**
-        * @brief Relays messages (like, comment, retraction) to other servers if we are the thread owner
+        * Relays messages (like, comment, retraction) to other servers if we are the thread owner
         *
         * @param array $item         The item that will be exported
         * @param array $owner        the array of the item owner
@@ -3938,7 +3975,7 @@ class Diaspora
        }
 
        /**
-        * @brief Sends a retraction (deletion) of a message, like or comment
+        * Sends a retraction (deletion) of a message, like or comment
         *
         * @param array $item         The item that will be exported
         * @param array $owner        the array of the item owner
@@ -3973,7 +4010,7 @@ class Diaspora
        }
 
        /**
-        * @brief Sends a mail
+        * Sends a mail
         *
         * @param array $item    The item that will be exported
         * @param array $owner   The owner
@@ -4024,7 +4061,7 @@ class Diaspora
        }
 
        /**
-        * @brief Split a name into first name and last name
+        * Split a name into first name and last name
         *
         * @param string $name The name
         *
@@ -4080,7 +4117,7 @@ class Diaspora
        }
 
        /**
-        * @brief Create profile data
+        * Create profile data
         *
         * @param int $uid The user id
         *
@@ -4094,7 +4131,7 @@ class Diaspora
                        FROM `profile`
                        INNER JOIN `user` ON `profile`.`uid` = `user`.`uid`
                        INNER JOIN `contact` ON `profile`.`uid` = `contact`.`uid`
-                       WHERE `user`.`uid` = %d AND `profile`.`is-default` AND `contact`.`self` LIMIT 1",
+                       WHERE `user`.`uid` = %d AND `contact`.`self` LIMIT 1",
                        intval($uid)
                );
 
@@ -4112,7 +4149,7 @@ class Diaspora
                $large = DI::baseUrl().'/photo/custom/300/'.$profile['uid'].'.jpg';
                $medium = DI::baseUrl().'/photo/custom/100/'.$profile['uid'].'.jpg';
                $small = DI::baseUrl().'/photo/custom/50/'  .$profile['uid'].'.jpg';
-               $searchable = (($profile['publish'] && $profile['net-publish']) ? 'true' : 'false');
+               $searchable = ($profile['net-publish'] ? 'true' : 'false');
 
                $dob = null;
                $about = null;
@@ -4129,8 +4166,7 @@ class Diaspora
                                $dob = DateTimeFormat::utc($year . '-' . $month . '-'. $day, 'Y-m-d');
                        }
 
-                       $about = $profile['about'];
-                       $about = strip_tags(BBCode::convert($about));
+                       $about = BBCode::toMarkdown($profile['about']);
 
                        $location = Profile::formatLocation($profile);
                        $tags = '';
@@ -4156,7 +4192,6 @@ class Diaspora
                                "image_url_medium" => $medium,
                                "image_url_small" => $small,
                                "birthday" => $dob,
-                               "gender" => $profile['gender'],
                                "bio" => $about,
                                "location" => $location,
                                "searchable" => $searchable,
@@ -4165,7 +4200,7 @@ class Diaspora
        }
 
        /**
-        * @brief Sends profile data
+        * Sends profile data
         *
         * @param int  $uid    The user id
         * @param bool $recips optional, default false
@@ -4207,7 +4242,7 @@ class Diaspora
        }
 
        /**
-        * @brief Creates the signature for likes that are created on our system
+        * Creates the signature for likes that are created on our system
         *
         * @param integer $uid  The user of that comment
         * @param array   $item Item array
@@ -4238,7 +4273,7 @@ class Diaspora
        }
 
        /**
-        * @brief Creates the signature for Comments that are created on our system
+        * Creates the signature for Comments that are created on our system
         *
         * @param integer $uid  The user of that comment
         * @param array   $item Item array