]> 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 b7c69668e066012c21542cd03b5ad8fdfc4b533d..885c9eea013d3b81a66999dc235fcc74ed52fa2c 100644 (file)
@@ -5,13 +5,12 @@
 namespace Friendica\Database;
 
 use Friendica\Core\Config;
+use Friendica\Core\Logger;
+use Friendica\Core\Protocol;
 use Friendica\Model\Contact;
 use Friendica\Model\Item;
 use Friendica\Model\ItemURI;
 use Friendica\Model\PermissionSet;
-use Friendica\Database\DBA;
-
-require_once 'include/dba.php';
 
 /**
  * Post update functions
@@ -26,9 +25,6 @@ class PostUpdate
                if (!self::update1194()) {
                        return false;
                }
-               if (!self::update1198()) {
-                       return false;
-               }
                if (!self::update1206()) {
                        return false;
                }
@@ -38,6 +34,12 @@ class PostUpdate
                if (!self::update1281()) {
                        return false;
                }
+               if (!self::update1297()) {
+                       return false;
+               }
+               if (!self::update1322()) {
+                       return false;
+               }
 
                return true;
        }
@@ -46,6 +48,7 @@ class PostUpdate
         * @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()
        {
@@ -54,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) {
@@ -65,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");
 
@@ -81,115 +84,33 @@ class PostUpdate
 
                $r = q($query1.$query2.$query3."  ORDER BY `item`.`id` LIMIT 1",
                        intval($start_id), intval($end_id),
-                       DBA::escape(NETWORK_DFRN), DBA::escape(NETWORK_DIASPORA), DBA::escape(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),
-                       DBA::escape(NETWORK_DFRN), DBA::escape(NETWORK_DIASPORA), DBA::escape(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),
-                       DBA::escape(NETWORK_DFRN), DBA::escape(NETWORK_DIASPORA), DBA::escape(NETWORK_OSTATUS));
-
-               logger("Done", LOGGER_DEBUG);
-       }
-
-       /**
-        * @brief set the author-id and owner-id in all item entries
-        *
-        * 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.
-        *
-        * @return bool "true" when the job is done
-        */
-       private static function update1198()
-       {
-               // Was the script completed?
-               if (Config::get("system", "post_update_version") >= 1198) {
-                       return true;
-               }
-
-               logger("Start", LOGGER_DEBUG);
-
-               // 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;
-                       }
-
-                       // 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 (DBA::isResult($r)) {
-                               Config::set("system", "post_update_version", 1198);
-                               logger("Done", LOGGER_DEBUG);
-                               return true;
-                       }
-                       return false;
-               }
-
-               logger("Query done", LOGGER_DEBUG);
-
-               $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"]];
-               }
-
-               // 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);
-
-                       $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);
+                       DBA::escape(Protocol::DFRN), DBA::escape(Protocol::DIASPORA), DBA::escape(Protocol::OSTATUS));
 
-                       if ($author_id == 0) {
-                               $author_id = -1;
-                       }
-                       if ($owner_id == 0) {
-                               $owner_id = -1;
-                       }
-                       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]);
-               }
-
-               logger("Updated items", LOGGER_DEBUG);
-               return false;
+               Logger::log("Done", Logger::DEBUG);
        }
 
        /**
@@ -198,6 +119,7 @@ class PostUpdate
         * 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 update1206()
        {
@@ -206,7 +128,7 @@ class PostUpdate
                        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`
@@ -222,7 +144,7 @@ class PostUpdate
                }
 
                Config::set("system", "post_update_version", 1206);
-               logger("Done", LOGGER_DEBUG);
+               Logger::log("Done", Logger::DEBUG);
                return true;
        }
 
@@ -230,6 +152,8 @@ class PostUpdate
         * @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()
        {
@@ -240,7 +164,7 @@ class PostUpdate
 
                $id = Config::get("system", "post_update_version_1279_id", 0);
 
-               logger("Start from item " . $id, LOGGER_DEBUG);
+               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',
@@ -252,6 +176,12 @@ class PostUpdate
                $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;
+               }
+
                while ($item = Item::fetch($items)) {
                        $id = $item['id'];
 
@@ -303,7 +233,7 @@ class PostUpdate
 
                Config::set("system", "post_update_version_1279_id", $id);
 
-               logger("Processed rows: " . $rows . " - last processed item:  " . $id, LOGGER_DEBUG);
+               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
@@ -315,13 +245,13 @@ class PostUpdate
                        foreach ($nullfields as $field) {
                                $fields = [$field => null];
                                $condition = [$field => ''];
-                               logger("Setting '" . $field . "' to null if empty.", LOGGER_DEBUG);
+                               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("Done", LOGGER_DEBUG);
+                       Logger::log("Done", Logger::DEBUG);
                        return true;
                }
 
@@ -374,6 +304,7 @@ class PostUpdate
         * @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 update1281()
        {
@@ -384,7 +315,7 @@ class PostUpdate
 
                $id = Config::get("system", "post_update_version_1281_id", 0);
 
-               logger("Start from item " . $id, LOGGER_DEBUG);
+               Logger::log("Start from item " . $id, Logger::DEBUG);
 
                $fields = ['id', 'guid', 'uri', 'uri-id', 'parent-uri', 'parent-uri-id', 'thr-parent', 'thr-parent-id'];
 
@@ -393,6 +324,12 @@ class PostUpdate
                $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;
+               }
+
                while ($item = DBA::fetch($items)) {
                        $id = $item['id'];
 
@@ -430,20 +367,90 @@ class PostUpdate
 
                Config::set("system", "post_update_version_1281_id", $id);
 
-               logger("Processed rows: " . $rows . " - last processed item:  " . $id, LOGGER_DEBUG);
+               Logger::log("Processed rows: " . $rows . " - last processed item:  " . $id, Logger::DEBUG);
 
                if ($start_id == $id) {
-                       logger("Updating item-uri in item-activity", LOGGER_DEBUG);
+                       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("Updating item-uri in item-content", LOGGER_DEBUG);
+                       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("Done", LOGGER_DEBUG);
+                       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');
+
+               return true;
+       }
 }