]> git.mxchange.org Git - friendica.git/blobdiff - src/Database/PostUpdate.php
Merge pull request #7943 from MrPetovan/bug/7920-api-fix-followers-friends-ids
[friendica.git] / src / Database / PostUpdate.php
index 13da437796d709158d9b863404acd5dcd47df8c3..885c9eea013d3b81a66999dc235fcc74ed52fa2c 100644 (file)
@@ -5,11 +5,12 @@
 namespace Friendica\Database;
 
 use Friendica\Core\Config;
-use Friendica\Database\DBM;
+use Friendica\Core\Logger;
+use Friendica\Core\Protocol;
 use Friendica\Model\Contact;
-use dba;
-
-require_once 'include/dba.php';
+use Friendica\Model\Item;
+use Friendica\Model\ItemURI;
+use Friendica\Model\PermissionSet;
 
 /**
  * Post update functions
@@ -22,20 +23,32 @@ class PostUpdate
        public static function update()
        {
                if (!self::update1194()) {
-                       return;
-               }
-               if (!self::update1198()) {
-                       return;
+                       return false;
                }
                if (!self::update1206()) {
-                       return;
+                       return false;
+               }
+               if (!self::update1279()) {
+                       return false;
+               }
+               if (!self::update1281()) {
+                       return false;
+               }
+               if (!self::update1297()) {
+                       return false;
+               }
+               if (!self::update1322()) {
+                       return false;
                }
+
+               return true;
        }
 
        /**
         * @brief Updates the "global" field in the item table
         *
         * @return bool "true" when the job is done
+        * @throws \Friendica\Network\HTTPException\InternalServerErrorException
         */
        private static function update1194()
        {
@@ -44,7 +57,7 @@ class PostUpdate
                        return true;
                }
 
-               logger("Start", LOGGER_DEBUG);
+               Logger::log("Start", Logger::DEBUG);
 
                $end_id = Config::get("system", "post_update_1194_end");
                if (!$end_id) {
@@ -55,7 +68,7 @@ class PostUpdate
                        }
                }
 
-               logger("End ID: ".$end_id, LOGGER_DEBUG);
+               Logger::log("End ID: ".$end_id, Logger::DEBUG);
 
                $start_id = Config::get("system", "post_update_1194_start");
 
@@ -67,154 +80,377 @@ class PostUpdate
                                AND `item`.`visible` AND NOT `item`.`private`
                                AND NOT `item`.`deleted` AND NOT `item`.`moderated`
                                AND `item`.`network` IN ('%s', '%s', '%s', '')
-                               AND `item`.`allow_cid` = '' AND `item`.`allow_gid` = ''
-                               AND `item`.`deny_cid` = '' AND `item`.`deny_gid` = ''
                                AND NOT `item`.`global`";
 
                $r = q($query1.$query2.$query3."  ORDER BY `item`.`id` LIMIT 1",
                        intval($start_id), intval($end_id),
-                       dbesc(NETWORK_DFRN), dbesc(NETWORK_DIASPORA), dbesc(NETWORK_OSTATUS));
+                       DBA::escape(Protocol::DFRN), DBA::escape(Protocol::DIASPORA), DBA::escape(Protocol::OSTATUS));
                if (!$r) {
                        Config::set("system", "post_update_version", 1194);
-                       logger("Update is done", LOGGER_DEBUG);
+                       Logger::log("Update is done", Logger::DEBUG);
                        return true;
                } else {
                        Config::set("system", "post_update_1194_start", $r[0]["id"]);
                        $start_id = Config::get("system", "post_update_1194_start");
                }
 
-               logger("Start ID: ".$start_id, LOGGER_DEBUG);
+               Logger::log("Start ID: ".$start_id, Logger::DEBUG);
 
                $r = q($query1.$query2.$query3."  ORDER BY `item`.`id` LIMIT 1000,1",
                        intval($start_id), intval($end_id),
-                       dbesc(NETWORK_DFRN), dbesc(NETWORK_DIASPORA), dbesc(NETWORK_OSTATUS));
+                       DBA::escape(Protocol::DFRN), DBA::escape(Protocol::DIASPORA), DBA::escape(Protocol::OSTATUS));
                if ($r) {
                        $pos_id = $r[0]["id"];
                } else {
                        $pos_id = $end_id;
                }
-               logger("Progress: Start: ".$start_id." position: ".$pos_id." end: ".$end_id, LOGGER_DEBUG);
+               Logger::log("Progress: Start: ".$start_id." position: ".$pos_id." end: ".$end_id, Logger::DEBUG);
 
                q("UPDATE `item` ".$query2." SET `item`.`global` = 1 ".$query3,
                        intval($start_id), intval($pos_id),
-                       dbesc(NETWORK_DFRN), dbesc(NETWORK_DIASPORA), dbesc(NETWORK_OSTATUS));
+                       DBA::escape(Protocol::DFRN), DBA::escape(Protocol::DIASPORA), DBA::escape(Protocol::OSTATUS));
 
-               logger("Done", LOGGER_DEBUG);
+               Logger::log("Done", Logger::DEBUG);
        }
 
        /**
-        * @brief set the author-id and owner-id in all item entries
+        * @brief update the "last-item" field in the "self" contact
         *
-        * This job has to be started multiple times until all entries are set.
-        * It isn't started in the update function since it would consume too much time and can be done in the background.
+        * This field avoids cost intensive calls in the admin panel and in "nodeinfo"
         *
         * @return bool "true" when the job is done
+        * @throws \Friendica\Network\HTTPException\InternalServerErrorException
         */
-       private static function update1198()
+       private static function update1206()
        {
                // Was the script completed?
-               if (Config::get("system", "post_update_version") >= 1198) {
+               if (Config::get("system", "post_update_version") >= 1206) {
                        return true;
                }
 
-               logger("Start", LOGGER_DEBUG);
+               Logger::log("Start", Logger::DEBUG);
+               $r = q("SELECT `contact`.`id`, `contact`.`last-item`,
+                       (SELECT MAX(`changed`) FROM `item` USE INDEX (`uid_wall_changed`) WHERE `wall` AND `uid` = `user`.`uid`) AS `lastitem_date`
+                       FROM `user`
+                       INNER JOIN `contact` ON `contact`.`uid` = `user`.`uid` AND `contact`.`self`");
 
-               // Check if the first step is done (Setting "author-id" and "owner-id" in the item table)
-               $fields = ['author-link', 'author-name', 'author-avatar', 'owner-link', 'owner-name', 'owner-avatar', 'network', 'uid'];
-               $r = dba::select('item', $fields, ['author-id' => 0, 'owner-id' => 0], ['limit' => 1000]);
-               if (!$r) {
-                       // Are there unfinished entries in the thread table?
-                       $r = q("SELECT COUNT(*) AS `total` FROM `thread`
-                               INNER JOIN `item` ON `item`.`id` =`thread`.`iid`
-                               WHERE `thread`.`author-id` = 0 AND `thread`.`owner-id` = 0 AND
-                                       (`thread`.`uid` IN (SELECT `uid` from `user`) OR `thread`.`uid` = 0)");
-
-                       if ($r && ($r[0]["total"] == 0)) {
-                               Config::set("system", "post_update_version", 1198);
-                               logger("Done", LOGGER_DEBUG);
-                               return true;
+               if (!DBA::isResult($r)) {
+                       return false;
+               }
+               foreach ($r as $user) {
+                       if (!empty($user["lastitem_date"]) && ($user["lastitem_date"] > $user["last-item"])) {
+                               DBA::update('contact', ['last-item' => $user['lastitem_date']], ['id' => $user['id']]);
                        }
+               }
 
-                       // Update the thread table from the item table
-                       $r = q("UPDATE `thread` INNER JOIN `item` ON `item`.`id`=`thread`.`iid`
-                                       SET `thread`.`author-id` = `item`.`author-id`,
-                                       `thread`.`owner-id` = `item`.`owner-id`
-                               WHERE `thread`.`author-id` = 0 AND `thread`.`owner-id` = 0 AND
-                                       (`thread`.`uid` IN (SELECT `uid` from `user`) OR `thread`.`uid` = 0)");
-
-                       logger("Updated threads", LOGGER_DEBUG);
-                       if (DBM::is_result($r)) {
-                               Config::set("system", "post_update_version", 1198);
-                               logger("Done", LOGGER_DEBUG);
-                               return true;
-                       }
-                       return false;
+               Config::set("system", "post_update_version", 1206);
+               Logger::log("Done", Logger::DEBUG);
+               return true;
+       }
+
+       /**
+        * @brief update the item related tables
+        *
+        * @return bool "true" when the job is done
+        * @throws \Friendica\Network\HTTPException\InternalServerErrorException
+        * @throws \ImagickException
+        */
+       private static function update1279()
+       {
+               // Was the script completed?
+               if (Config::get("system", "post_update_version") >= 1279) {
+                       return true;
                }
 
-               logger("Query done", LOGGER_DEBUG);
+               $id = Config::get("system", "post_update_version_1279_id", 0);
 
-               $item_arr = [];
-               foreach ($r as $item) {
-                       $index = $item["author-link"]."-".$item["owner-link"]."-".$item["uid"];
-                       $item_arr[$index] = ["author-link" => $item["author-link"],
-                                                       "owner-link" => $item["owner-link"],
-                                                       "uid" => $item["uid"]];
+               Logger::log("Start from item " . $id, Logger::DEBUG);
+
+               $fields = array_merge(Item::MIXED_CONTENT_FIELDLIST, ['network', 'author-id', 'owner-id', 'tag', 'file',
+                       'author-name', 'author-avatar', 'author-link', 'owner-name', 'owner-avatar', 'owner-link', 'id',
+                       'uid', 'allow_cid', 'allow_gid', 'deny_cid', 'deny_gid', 'psid', 'post-type', 'bookmark', 'type',
+                       'inform', 'postopts', 'icid']);
+
+               $start_id = $id;
+               $rows = 0;
+               $condition = ["`id` > ?", $id];
+               $params = ['order' => ['id'], 'limit' => 10000];
+               $items = Item::select($fields, $condition, $params);
+
+               if (DBA::errorNo() != 0) {
+                       Logger::log('Database error ' . DBA::errorNo() . ':' . DBA::errorMessage());
+                       return false;
                }
 
-               // Set the "author-id" and "owner-id" in the item table and add a new public contact entry if needed
-               foreach ($item_arr as $item) {
-                       $default = ['url' => $item['author-link'], 'name' => $item['author-name'],
-                               'photo' => $item['author-avatar'], 'network' => $item['network']];
-                       $author_id = Contact::getIdForURL($item["author-link"], 0, false, $default);
+               while ($item = Item::fetch($items)) {
+                       $id = $item['id'];
+
+                       if (empty($item['author-id'])) {
+                               $default = ['url' => $item['author-link'], 'name' => $item['author-name'],
+                                       'photo' => $item['author-avatar'], 'network' => $item['network']];
 
-                       $default = ['url' => $item['owner-link'], 'name' => $item['owner-name'],
-                               'photo' => $item['owner-avatar'], 'network' => $item['network']];
-                       $owner_id = Contact::getIdForURL($item["owner-link"], 0, false, $default);
+                               $item['author-id'] = Contact::getIdForURL($item["author-link"], 0, false, $default);
+                       }
+
+                       if (empty($item['owner-id'])) {
+                               $default = ['url' => $item['owner-link'], 'name' => $item['owner-name'],
+                                       'photo' => $item['owner-avatar'], 'network' => $item['network']];
+
+                               $item['owner-id'] = Contact::getIdForURL($item["owner-link"], 0, false, $default);
+                       }
+
+                       if (empty($item['psid'])) {
+                               $item['psid'] = PermissionSet::fetchIDForPost($item);
+                       } else {
+                               $item['allow_cid'] = null;
+                               $item['allow_gid'] = null;
+                               $item['deny_cid'] = null;
+                               $item['deny_gid'] = null;
+                       }
 
-                       if ($author_id == 0) {
-                               $author_id = -1;
+                       if ($item['post-type'] == 0) {
+                               if (!empty($item['type']) && ($item['type'] == 'note')) {
+                                       $item['post-type'] = Item::PT_PERSONAL_NOTE;
+                               } elseif (!empty($item['type']) && ($item['type'] == 'photo')) {
+                                       $item['post-type'] = Item::PT_IMAGE;
+                               } elseif (!empty($item['bookmark']) && $item['bookmark']) {
+                                       $item['post-type'] = Item::PT_PAGE;
+                               }
                        }
-                       if ($owner_id == 0) {
-                               $owner_id = -1;
+
+                       self::createLanguage($item);
+
+                       if (!empty($item['icid']) && !empty($item['language'])) {
+                               DBA::update('item-content', ['language' => $item['language']], ['id' => $item['icid']]);
                        }
-                       dba::update('item', ['author-id' => $author_id, 'owner-id' => $owner_id], ['uid' => $item['uid'], 'author-link' => $item['author-link'], 'owner-link' => $item['owner-link'], 'author-id' => 0, 'owner-id' => 0]);
+                       unset($item['language']);
+
+                       Item::update($item, ['id' => $id]);
+
+                       ++$rows;
+               }
+               DBA::close($items);
+
+               Config::set("system", "post_update_version_1279_id", $id);
+
+               Logger::log("Processed rows: " . $rows . " - last processed item:  " . $id, Logger::DEBUG);
+
+               if ($start_id == $id) {
+                       // Set all deprecated fields to "null" if they contain an empty string
+                       $nullfields = ['allow_cid', 'allow_gid', 'deny_cid', 'deny_gid', 'postopts', 'inform', 'type',
+                               'bookmark', 'file', 'location', 'coord', 'tag', 'plink', 'title', 'content-warning',
+                               'body', 'app', 'verb', 'object-type', 'object', 'target-type', 'target',
+                               'author-name', 'author-link', 'author-avatar', 'owner-name', 'owner-link', 'owner-avatar',
+                               'rendered-hash', 'rendered-html'];
+                       foreach ($nullfields as $field) {
+                               $fields = [$field => null];
+                               $condition = [$field => ''];
+                               Logger::log("Setting '" . $field . "' to null if empty.", Logger::DEBUG);
+                               // Important: This has to be a "DBA::update", not a "Item::update"
+                               DBA::update('item', $fields, $condition);
+                       }
+
+                       Config::set("system", "post_update_version", 1279);
+                       Logger::log("Done", Logger::DEBUG);
+                       return true;
                }
 
-               logger("Updated items", LOGGER_DEBUG);
                return false;
        }
 
+       private static function createLanguage(&$item)
+       {
+               if (empty($item['postopts'])) {
+                       return;
+               }
+
+               $opts = explode(',', $item['postopts']);
+
+               $postopts = [];
+
+               foreach ($opts as $opt) {
+                       if (strstr($opt, 'lang=')) {
+                               $language = substr($opt, 5);
+                       } else {
+                               $postopts[] = $opt;
+                       }
+               }
+
+               if (empty($language)) {
+                       return;
+               }
+
+               if (!empty($postopts)) {
+                       $item['postopts'] = implode(',', $postopts);
+               } else {
+                       $item['postopts'] = null;
+               }
+
+               $lang_pairs = explode(':', $language);
+
+               $lang_arr = [];
+
+               foreach ($lang_pairs as $pair) {
+                       $lang_pair_arr = explode(';', $pair);
+                       if (count($lang_pair_arr) == 2) {
+                               $lang_arr[$lang_pair_arr[0]] = $lang_pair_arr[1];
+                       }
+               }
+
+               $item['language'] = json_encode($lang_arr);
+       }
+
        /**
-        * @brief update the "last-item" field in the "self" contact
-        *
-        * This field avoids cost intensive calls in the admin panel and in "nodeinfo"
+        * @brief update item-uri data. Prerequisite for the next item structure update.
         *
         * @return bool "true" when the job is done
+        * @throws \Friendica\Network\HTTPException\InternalServerErrorException
         */
-       private static function update1206()
+       private static function update1281()
        {
                // Was the script completed?
-               if (Config::get("system", "post_update_version") >= 1206) {
+               if (Config::get("system", "post_update_version") >= 1281) {
                        return true;
                }
 
-               logger("Start", LOGGER_DEBUG);
-               $r = q("SELECT `contact`.`id`, `contact`.`last-item`,
-                       (SELECT MAX(`changed`) FROM `item` USE INDEX (`uid_wall_changed`) WHERE `wall` AND `uid` = `user`.`uid`) AS `lastitem_date`
-                       FROM `user`
-                       INNER JOIN `contact` ON `contact`.`uid` = `user`.`uid` AND `contact`.`self`");
+               $id = Config::get("system", "post_update_version_1281_id", 0);
+
+               Logger::log("Start from item " . $id, Logger::DEBUG);
+
+               $fields = ['id', 'guid', 'uri', 'uri-id', 'parent-uri', 'parent-uri-id', 'thr-parent', 'thr-parent-id'];
 
-               if (!DBM::is_result($r)) {
+               $start_id = $id;
+               $rows = 0;
+               $condition = ["`id` > ?", $id];
+               $params = ['order' => ['id'], 'limit' => 10000];
+               $items = DBA::select('item', $fields, $condition, $params);
+
+               if (DBA::errorNo() != 0) {
+                       Logger::log('Database error ' . DBA::errorNo() . ':' . DBA::errorMessage());
                        return false;
                }
-               foreach ($r as $user) {
-                       if (!empty($user["lastitem_date"]) && ($user["lastitem_date"] > $user["last-item"])) {
-                               dba::update('contact', ['last-item' => $user['lastitem_date']], ['id' => $user['id']]);
+
+               while ($item = DBA::fetch($items)) {
+                       $id = $item['id'];
+
+                       if (empty($item['uri'])) {
+                               // Should not happen
+                               continue;
+                       } elseif (empty($item['uri-id'])) {
+                               $item['uri-id'] = ItemURI::insert(['uri' => $item['uri'], 'guid' => $item['guid']]);
+                       }
+
+                       if (empty($item['parent-uri'])) {
+                               $item['parent-uri-id'] = $item['uri-id'];
+                       } elseif (empty($item['parent-uri-id'])) {
+                               $item['parent-uri-id'] = ItemURI::getIdByURI($item['parent-uri']);
                        }
+
+                       // Very old items don't have this field
+                       if (empty($item['thr-parent'])) {
+                               $item['thr-parent-id'] = $item['parent-uri-id'];
+                       } elseif (empty($item['thr-parent-id'])) {
+                               $item['thr-parent-id'] = ItemURI::getIdByURI($item['thr-parent']);
+                       }
+
+                       unset($item['id']);
+                       unset($item['guid']);
+                       unset($item['uri']);
+                       unset($item['parent-uri']);
+                       unset($item['thr-parent']);
+
+                       DBA::update('item', $item, ['id' => $id]);
+
+                       ++$rows;
                }
+               DBA::close($items);
+
+               Config::set("system", "post_update_version_1281_id", $id);
+
+               Logger::log("Processed rows: " . $rows . " - last processed item:  " . $id, Logger::DEBUG);
+
+               if ($start_id == $id) {
+                       Logger::log("Updating item-uri in item-activity", Logger::DEBUG);
+                       DBA::e("UPDATE `item-activity` INNER JOIN `item-uri` ON `item-uri`.`uri` = `item-activity`.`uri` SET `item-activity`.`uri-id` = `item-uri`.`id` WHERE `item-activity`.`uri-id` IS NULL");
+
+                       Logger::log("Updating item-uri in item-content", Logger::DEBUG);
+                       DBA::e("UPDATE `item-content` INNER JOIN `item-uri` ON `item-uri`.`uri` = `item-content`.`uri` SET `item-content`.`uri-id` = `item-uri`.`id` WHERE `item-content`.`uri-id` IS NULL");
+
+                       Config::set("system", "post_update_version", 1281);
+                       Logger::log("Done", Logger::DEBUG);
+                       return true;
+               }
+
+               return false;
+       }
+
+       /**
+        * Set the delivery queue count to a negative value for all items preceding the feature.
+        *
+        * @return bool "true" when the job is done
+        * @throws \Friendica\Network\HTTPException\InternalServerErrorException
+        */
+       private static function update1297()
+       {
+               // Was the script completed?
+               if (Config::get('system', 'post_update_version') >= 1297) {
+                       return true;
+               }
+
+               $max_item_delivery_data = DBA::selectFirst('item-delivery-data', ['iid'], ['queue_count > 0 OR queue_done > 0'], ['order' => ['iid']]);
+               $max_iid = $max_item_delivery_data['iid'];
+
+               Logger::info('Start update1297 with max iid: ' . $max_iid);
+
+               $condition = ['`queue_count` = 0 AND `iid` < ?', $max_iid];
+
+               DBA::update('item-delivery-data', ['queue_count' => -1], $condition);
+
+               if (DBA::errorNo() != 0) {
+                       Logger::error('Database error ' . DBA::errorNo() . ':' . DBA::errorMessage());
+                       return false;
+               }
+
+               Logger::info('Processed rows: ' . DBA::affectedRows());
+
+               Config::set('system', 'post_update_version', 1297);
+
+               Logger::info('Done');
+
+               return true;
+       }
+       /**
+        * Remove contact duplicates
+        *
+        * @return bool "true" when the job is done
+        * @throws \Friendica\Network\HTTPException\InternalServerErrorException
+        */
+       private static function update1322()
+       {
+               // Was the script completed?
+               if (Config::get('system', 'post_update_version') >= 1322) {
+                       return true;
+               }
+
+               Logger::info('Start');
+
+               $contacts = DBA::p("SELECT `nurl`, `uid` FROM `contact`
+                       WHERE EXISTS (SELECT `nurl` FROM `contact` AS `c2`
+                               WHERE `c2`.`nurl` = `contact`.`nurl` AND `c2`.`id` != `contact`.`id` AND `c2`.`uid` = `contact`.`uid` AND `c2`.`network` IN (?, ?, ?) AND NOT `deleted`)
+                       AND (`network` IN (?, ?, ?) OR (`uid` = ?)) AND NOT `deleted` GROUP BY `nurl`, `uid`",
+                       Protocol::DIASPORA, Protocol::OSTATUS, Protocol::ACTIVITYPUB,
+                       Protocol::DIASPORA, Protocol::OSTATUS, Protocol::ACTIVITYPUB, 0);
+
+               while ($contact = DBA::fetch($contacts)) {
+                       Logger::info('Remove duplicates', ['nurl' => $contact['nurl'], 'uid' => $contact['uid']]);
+                       Contact::removeDuplicates($contact['nurl'], $contact['uid']);
+               }
+
+               DBA::close($contact);
+               Config::set('system', 'post_update_version', 1322);
+
+               Logger::info('Done');
 
-               Config::set("system", "post_update_version", 1206);
-               logger("Done", LOGGER_DEBUG);
                return true;
        }
 }