]> git.mxchange.org Git - friendica.git/blobdiff - src/Database/PostUpdate.php
Remove deprecated code
[friendica.git] / src / Database / PostUpdate.php
index b7c69668e066012c21542cd03b5ad8fdfc4b533d..7fd28419ea60106eeef11689bb7cb57968a55ed5 100644 (file)
@@ -4,31 +4,31 @@
  */
 namespace Friendica\Database;
 
-use Friendica\Core\Config;
+use Friendica\Core\Logger;
+use Friendica\Core\Protocol;
+use Friendica\DI;
 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';
+use Friendica\Model\UserItem;
 
 /**
- * Post update functions
+ * These database-intensive post update routines are meant to be executed in the background by the cronjob.
+ *
+ * If there is a need for a intensive migration after a database structure change, update this file
+ * by adding a new method at the end with the number of the new DB_UPDATE_VERSION.
  */
 class PostUpdate
 {
        /**
-        * @brief Calls the post update functions
+        * Calls the post update functions
         */
        public static function update()
        {
                if (!self::update1194()) {
                        return false;
                }
-               if (!self::update1198()) {
-                       return false;
-               }
                if (!self::update1206()) {
                        return false;
                }
@@ -38,36 +38,46 @@ class PostUpdate
                if (!self::update1281()) {
                        return false;
                }
+               if (!self::update1297()) {
+                       return false;
+               }
+               if (!self::update1322()) {
+                       return false;
+               }
+               if (!self::update1329()) {
+                       return false;
+               }
 
                return true;
        }
 
        /**
-        * @brief Updates the "global" field in the item table
+        * 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()
        {
                // Was the script completed?
-               if (Config::get("system", "post_update_version") >= 1194) {
+               if (DI::config()->get("system", "post_update_version") >= 1194) {
                        return true;
                }
 
-               logger("Start", LOGGER_DEBUG);
+               Logger::log("Start", Logger::DEBUG);
 
-               $end_id = Config::get("system", "post_update_1194_end");
+               $end_id = DI::config()->get("system", "post_update_1194_end");
                if (!$end_id) {
                        $r = q("SELECT `id` FROM `item` WHERE `uid` != 0 ORDER BY `id` DESC LIMIT 1");
                        if ($r) {
-                               Config::set("system", "post_update_1194_end", $r[0]["id"]);
-                               $end_id = Config::get("system", "post_update_1194_end");
+                               DI::config()->set("system", "post_update_1194_end", $r[0]["id"]);
+                               $end_id = DI::config()->get("system", "post_update_1194_end");
                        }
                }
 
-               logger("End ID: ".$end_id, LOGGER_DEBUG);
+               Logger::log("End ID: ".$end_id, Logger::DEBUG);
 
-               $start_id = Config::get("system", "post_update_1194_start");
+               $start_id = DI::config()->get("system", "post_update_1194_start");
 
                $query1 = "SELECT `item`.`id` FROM `item` ";
 
@@ -81,132 +91,51 @@ 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);
+                       DI::config()->set("system", "post_update_version", 1194);
+                       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");
+                       DI::config()->set("system", "post_update_1194_start", $r[0]["id"]);
+                       $start_id = DI::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);
-
-                       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]);
-               }
+                       DBA::escape(Protocol::DFRN), DBA::escape(Protocol::DIASPORA), DBA::escape(Protocol::OSTATUS));
 
-               logger("Updated items", LOGGER_DEBUG);
-               return false;
+               Logger::log("Done", Logger::DEBUG);
        }
 
        /**
-        * @brief update the "last-item" field in the "self" contact
+        * update the "last-item" field in the "self" contact
         *
         * 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()
        {
                // Was the script completed?
-               if (Config::get("system", "post_update_version") >= 1206) {
+               if (DI::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`
@@ -221,26 +150,28 @@ class PostUpdate
                        }
                }
 
-               Config::set("system", "post_update_version", 1206);
-               logger("Done", LOGGER_DEBUG);
+               DI::config()->set("system", "post_update_version", 1206);
+               Logger::log("Done", Logger::DEBUG);
                return true;
        }
 
        /**
-        * @brief update the item related tables
+        * 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) {
+               if (DI::config()->get("system", "post_update_version") >= 1279) {
                        return true;
                }
 
-               $id = Config::get("system", "post_update_version_1279_id", 0);
+               $id = DI::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 +183,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'];
 
@@ -270,14 +207,20 @@ class PostUpdate
                        }
 
                        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;
+                               $item['psid'] = PermissionSet::getIdFromACL(
+                                       $item['uid'],
+                                       $item['allow_cid'],
+                                       $item['allow_gid'],
+                                       $item['deny_cid'],
+                                       $item['deny_gid']
+                               );
                        }
 
+                       $item['allow_cid'] = null;
+                       $item['allow_gid'] = null;
+                       $item['deny_cid'] = null;
+                       $item['deny_gid'] = null;
+
                        if ($item['post-type'] == 0) {
                                if (!empty($item['type']) && ($item['type'] == 'note')) {
                                        $item['post-type'] = Item::PT_PERSONAL_NOTE;
@@ -301,9 +244,9 @@ class PostUpdate
                }
                DBA::close($items);
 
-               Config::set("system", "post_update_version_1279_id", $id);
+               DI::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 +258,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);
+                       DI::config()->set("system", "post_update_version", 1279);
+                       Logger::log("Done", Logger::DEBUG);
                        return true;
                }
 
@@ -371,20 +314,21 @@ class PostUpdate
        }
 
        /**
-        * @brief update item-uri data. Prerequisite for the next item structure update.
+        * 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()
        {
                // Was the script completed?
-               if (Config::get("system", "post_update_version") >= 1281) {
+               if (DI::config()->get("system", "post_update_version") >= 1281) {
                        return true;
                }
 
-               $id = Config::get("system", "post_update_version_1281_id", 0);
+               $id = DI::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 +337,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'];
 
@@ -428,19 +378,139 @@ class PostUpdate
                }
                DBA::close($items);
 
-               Config::set("system", "post_update_version_1281_id", $id);
+               DI::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);
+                       DI::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 (DI::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());
+
+               DI::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 (DI::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);
+               DI::config()->set('system', 'post_update_version', 1322);
+
+               Logger::info('Done');
+
+               return true;
+       }
+
+       /**
+        * update user-item data with notifications
+        *
+        * @return bool "true" when the job is done
+        * @throws \Friendica\Network\HTTPException\InternalServerErrorException
+        */
+       private static function update1329()
+       {
+               // Was the script completed?
+               if (DI::config()->get('system', 'post_update_version') >= 1329) {
+                       return true;
+               }
+
+               $id = DI::config()->get('system', 'post_update_version_1329_id', 0);
+
+               Logger::info('Start', ['item' => $id]);
+
+               $start_id = $id;
+               $rows = 0;
+               $condition = ["`id` > ?", $id];
+               $params = ['order' => ['id'], 'limit' => 10000];
+               $items = DBA::select('item', ['id'], $condition, $params);
+
+               if (DBA::errorNo() != 0) {
+                       Logger::error('Database error', ['no' => DBA::errorNo(), 'message' => DBA::errorMessage()]);
+                       return false;
+               }
+
+               while ($item = DBA::fetch($items)) {
+                       $id = $item['id'];
+
+                       UserItem::setNotification($item['id']);
+
+                       ++$rows;
+               }
+               DBA::close($items);
+
+               DI::config()->set('system', 'post_update_version_1329_id', $id);
+
+               Logger::info('Processed', ['rows' => $rows, 'last' => $id]);
+
+               if ($start_id == $id) {
+                       DI::config()->set('system', 'post_update_version', 1329);
+                       Logger::info('Done');
                        return true;
                }