Make Storage testable & add tests

[friendica.git] / src / Database / PostUpdate.php

diff --git a/src/Database/PostUpdate.php b/src/Database/PostUpdate.php
index a5bdcdeadd6c84653c03341d18cdb9533efae9ec..f0050e0ef353f3b0298c87d2d896cdf0876b2169 100644 (file)
--- a/src/Database/PostUpdate.php
+++ b/src/Database/PostUpdate.php
@@ -10,8 +10,8 @@ use Friendica\Core\Protocol;
 use Friendica\Model\Contact;
 use Friendica\Model\Item;
 use Friendica\Model\ItemURI;
+use Friendica\Model\UserItem;
 use Friendica\Model\PermissionSet;
-use Friendica\Database\DBA;
 
 /**
  * Post update functions
@@ -35,6 +35,15 @@ class PostUpdate
                if (!self::update1281()) {
                        return false;
                }
+               if (!self::update1297()) {
+                       return false;
+               }
+               if (!self::update1322()) {
+                       return false;
+               }
+               if (!self::update1329()) {
+                       return false;
+               }
 
                return true;
        }
@@ -378,4 +387,124 @@ class PostUpdate
 
                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;
+       }
+
+       /**
+        * @brief 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 (Config::get('system', 'post_update_version') >= 1329) {
+                       return true;
+               }
+
+               $id = 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);
+
+               Config::set('system', 'post_update_version_1329_id', $id);
+
+               Logger::info('Processed', ['rows' => $rows, 'last' => $id]);
+
+               if ($start_id == $id) {
+                       Config::set('system', 'post_update_version', 1329);
+                       Logger::info('Done');
+                       return true;
+               }
+
+               return false;
+       }
 }