2020.09 CHANGELOG
[friendica.git/.git] / update.php
index 314b166..ec6a94f 100644 (file)
@@ -45,10 +45,11 @@ use Friendica\Core\Logger;
 use Friendica\Core\Update;
 use Friendica\Core\Worker;
 use Friendica\Database\DBA;
+use Friendica\Database\DBStructure;
 use Friendica\DI;
 use Friendica\Model\Contact;
-use Friendica\Model\GContact;
 use Friendica\Model\Item;
+use Friendica\Model\Photo;
 use Friendica\Model\User;
 use Friendica\Model\Storage;
 use Friendica\Util\DateTimeFormat;
@@ -70,7 +71,8 @@ function update_1181()
 {
 
        // Fill the new fields in the term table.
-       Worker::add(PRIORITY_LOW, "TagUpdate");
+       // deactivated, the "term" table is deprecated
+       // Worker::add(PRIORITY_LOW, "TagUpdate");
 
        return Update::SUCCESS;
 }
@@ -92,23 +94,7 @@ function update_1191()
        DI::config()->set('system', 'maintenance', 1);
 
        if (Addon::isEnabled('forumlist')) {
-               $addon = 'forumlist';
-               $addons = DI::config()->get('system', 'addon');
-               $addons_arr = [];
-
-               if ($addons) {
-                       $addons_arr = explode(",", str_replace(" ", "", $addons));
-
-                       $idx = array_search($addon, $addons_arr);
-                       if ($idx !== false) {
-                               unset($addons_arr[$idx]);
-                               //delete forumlist manually from addon and hook table
-                               // since Addon::uninstall() don't work here
-                               q("DELETE FROM `addon` WHERE `name` = 'forumlist' ");
-                               q("DELETE FROM `hook` WHERE `file` = 'addon/forumlist/forumlist.php' ");
-                               DI::config()->set('system', 'addon', implode(", ", $addons_arr));
-                       }
-               }
+               Addon::uninstall('forumlist');
        }
 
        // select old formlist addon entries
@@ -217,7 +203,7 @@ function update_1260()
        while ($item = DBA::fetch($items)) {
                $contact = ['url' => $item['owner-link'], 'name' => $item['owner-name'],
                        'photo' => $item['owner-avatar'], 'network' => $item['network']];
-               $cid = Contact::getIdForURL($item['owner-link'], 0, false, $contact);
+               $cid = Contact::getIdForURL($item['owner-link'], 0, null, $contact);
                if (empty($cid)) {
                        continue;
                }
@@ -233,7 +219,7 @@ function update_1260()
        while ($item = DBA::fetch($items)) {
                $contact = ['url' => $item['author-link'], 'name' => $item['author-name'],
                        'photo' => $item['author-avatar'], 'network' => $item['network']];
-               $cid = Contact::getIdForURL($item['author-link'], 0, false, $contact);
+               $cid = Contact::getIdForURL($item['author-link'], 0, null, $contact);
                if (empty($cid)) {
                        continue;
                }
@@ -329,7 +315,6 @@ function update_1298()
                                                'was' => $data[$translateKey]]);
                                        Worker::add(PRIORITY_LOW, 'ProfileUpdate', $data['id']);
                                        Contact::updateSelfFromUserID($data['id']);
-                                       GContact::updateForUser($data['id']);
                                        $success++;
                                }
                        }
@@ -364,7 +349,9 @@ function update_1309()
 
 function update_1315()
 {
-       DBA::delete('item-delivery-data', ['postopts' => '', 'inform' => '', 'queue_count' => 0, 'queue_done' => 0]);
+       if (DBStructure::existsTable('item-delivery-data')) {
+               DBA::delete('item-delivery-data', ['postopts' => '', 'inform' => '', 'queue_count' => 0, 'queue_done' => 0]);
+       }
        return Update::SUCCESS;
 }
 
@@ -392,8 +379,8 @@ function update_1327()
 {
        $contacts = DBA::select('contact', ['uid', 'id', 'blocked', 'readonly'], ["`uid` != ? AND (`blocked` OR `readonly`) AND NOT `pending`", 0]);
        while ($contact = DBA::fetch($contacts)) {
-               Contact::setBlockedForUser($contact['id'], $contact['uid'], $contact['blocked']);
-               Contact::setIgnoredForUser($contact['id'], $contact['uid'], $contact['readonly']);
+               Contact\User::setBlocked($contact['id'], $contact['uid'], $contact['blocked']);
+               Contact\User::setIgnored($contact['id'], $contact['uid'], $contact['readonly']);
        }
        DBA::close($contacts);
 
@@ -438,3 +425,326 @@ function update_1332()
 
        return Update::SUCCESS;
 }
+
+function update_1347()
+{
+       foreach (Item::ACTIVITIES as $index => $activity) {
+               DBA::insert('verb', ['id' => $index + 1, 'name' => $activity], true);
+       }
+
+       return Update::SUCCESS;
+}
+
+function pre_update_1348()
+{
+       if (!DBA::exists('contact', ['id' => 0])) {
+               DBA::insert('contact', ['nurl' => '']);
+               $lastid = DBA::lastInsertId();
+               if ($lastid != 0) {
+                       DBA::update('contact', ['id' => 0], ['id' => $lastid]);
+               }
+       }
+
+       // The tables "permissionset" and "tag" could or could not exist during the update.
+       // This depends upon the previous version. Depending upon this situation we have to add
+       // the "0" values before adding the foreign keys - or after would be sufficient.
+
+       update_1348();
+
+       DBA::e("DELETE FROM `auth_codes` WHERE NOT `client_id` IN (SELECT `client_id` FROM `clients`)");
+       DBA::e("DELETE FROM `tokens` WHERE NOT `client_id` IN (SELECT `client_id` FROM `clients`)");
+
+       return Update::SUCCESS;
+}
+
+function update_1348()
+{
+       // Insert a permissionset with id=0
+       // Inserting it without an ID and then changing the value to 0 tricks the auto increment
+       if (!DBA::exists('permissionset', ['id' => 0])) {
+               DBA::insert('permissionset', ['allow_cid' => '', 'allow_gid' => '', 'deny_cid' => '', 'deny_gid' => '']);       
+               $lastid = DBA::lastInsertId();
+               if ($lastid != 0) {
+                       DBA::update('permissionset', ['id' => 0], ['id' => $lastid]);
+               }
+       }
+
+       if (!DBA::exists('tag', ['id' => 0])) {
+               DBA::insert('tag', ['name' => '']);
+               $lastid = DBA::lastInsertId();
+               if ($lastid != 0) {
+                       DBA::update('tag', ['id' => 0], ['id' => $lastid]);
+               }
+       }
+
+       return Update::SUCCESS;
+}
+
+function update_1349()
+{
+       $correct = true;
+       foreach (Item::ACTIVITIES as $index => $activity) {
+               if (!DBA::exists('verb', ['id' => $index + 1, 'name' => $activity])) {
+                       $correct = false;
+               }
+       }
+
+       if (!$correct) {
+               // The update failed - but it cannot be recovered, since the data doesn't match our expectation
+               // This means that we can't use this "shortcut" to fill the "vid" field and we have to rely upon
+               // the postupdate. This is not fatal, but means that it will take some longer time for the system
+               // to fill all data.
+               return Update::SUCCESS;
+       }
+
+       if (!DBA::e("UPDATE `item` INNER JOIN `item-activity` ON `item`.`uri-id` = `item-activity`.`uri-id`
+               SET `vid` = `item-activity`.`activity` + 1 WHERE `gravity` = ? AND (`vid` IS NULL OR `vid` = 0)", GRAVITY_ACTIVITY)) {
+               return Update::FAILED;
+       }
+
+       return Update::SUCCESS;
+}
+
+function update_1351()
+{
+       if (!DBA::e("UPDATE `thread` INNER JOIN `item` ON `thread`.`iid` = `item`.`id` SET `thread`.`uri-id` = `item`.`uri-id`")) {
+               return Update::FAILED;
+       }
+
+       return Update::SUCCESS;
+}
+
+function pre_update_1354()
+{
+       if (DBStructure::existsColumn('contact', ['ffi_keyword_blacklist'])
+               && !DBStructure::existsColumn('contact', ['ffi_keyword_denylist'])
+               && !DBA::e("ALTER TABLE `contact` CHANGE `ffi_keyword_blacklist` `ffi_keyword_denylist` text null")) {
+               return Update::FAILED;
+       }
+       return Update::SUCCESS;
+}
+
+function update_1354()
+{
+       if (DBStructure::existsColumn('contact', ['ffi_keyword_blacklist'])
+               && DBStructure::existsColumn('contact', ['ffi_keyword_denylist'])) {
+               if (!DBA::e("UPDATE `contact` SET `ffi_keyword_denylist` = `ffi_keyword_blacklist`")) {
+                       return Update::FAILED;
+               }
+
+               // When the data had been copied then the main task is done.
+               // Having the old field removed is only beauty but not crucial.
+               // So we don't care if this was successful or not.
+               DBA::e("ALTER TABLE `contact` DROP `ffi_keyword_blacklist`");
+       }
+       return Update::SUCCESS;
+}
+
+function update_1357()
+{
+       if (!DBA::e("UPDATE `contact` SET `failed` = true WHERE `success_update` < `failure_update` AND `failed` IS NULL")) {
+               return Update::FAILED;
+       }
+
+       if (!DBA::e("UPDATE `contact` SET `failed` = false WHERE `success_update` > `failure_update` AND `failed` IS NULL")) {
+               return Update::FAILED;
+       }
+
+       if (!DBA::e("UPDATE `contact` SET `failed` = false WHERE `updated` > `failure_update` AND `failed` IS NULL")) {
+               return Update::FAILED;
+       }
+
+       if (!DBA::e("UPDATE `contact` SET `failed` = false WHERE `last-item` > `failure_update` AND `failed` IS NULL")) {
+               return Update::FAILED;
+       }
+
+       if (!DBA::e("UPDATE `gserver` SET `failed` = true WHERE `last_contact` < `last_failure` AND `failed` IS NULL")) {
+               return Update::FAILED;
+       }
+
+       if (!DBA::e("UPDATE `gserver` SET `failed` = false WHERE `last_contact` > `last_failure` AND `failed` IS NULL")) {
+               return Update::FAILED;
+       }
+
+       return Update::SUCCESS;
+}
+
+function pre_update_1358()
+{
+       if (!DBA::e("DELETE FROM `contact-relation` WHERE NOT `relation-cid` IN (SELECT `id` FROM `contact`) OR NOT `cid` IN (SELECT `id` FROM `contact`)")) {
+               return Update::FAILED;
+       }
+
+       return Update::SUCCESS;
+}
+
+function pre_update_1363()
+{
+       Photo::delete(["`contact-id` != ? AND NOT `contact-id` IN (SELECT `id` FROM `contact`)", 0]);
+       return Update::SUCCESS;
+}
+
+function pre_update_1364()
+{
+       if (!DBA::e("DELETE FROM `2fa_recovery_codes` WHERE NOT `uid` IN (SELECT `uid` FROM `user`)")) {
+               return Update::FAILED;
+       }
+
+       if (!DBA::e("DELETE FROM `2fa_app_specific_password` WHERE NOT `uid` IN (SELECT `uid` FROM `user`)")) {
+               return Update::FAILED;
+       }
+
+       if (!DBA::e("DELETE FROM `attach` WHERE NOT `uid` IN (SELECT `uid` FROM `user`)")) {
+               return Update::FAILED;
+       }
+
+       if (!DBA::e("DELETE FROM `clients` WHERE NOT `uid` IN (SELECT `uid` FROM `user`)")) {
+               return Update::FAILED;
+       }
+
+       if (!DBA::e("DELETE FROM `conv` WHERE NOT `uid` IN (SELECT `uid` FROM `user`)")) {
+               return Update::FAILED;
+       }
+
+       if (!DBA::e("DELETE FROM `fsuggest` WHERE NOT `uid` IN (SELECT `uid` FROM `user`)")) {
+               return Update::FAILED;
+       }
+
+       if (!DBA::e("DELETE FROM `group` WHERE NOT `uid` IN (SELECT `uid` FROM `user`)")) {
+               return Update::FAILED;
+       }
+
+       if (!DBA::e("DELETE FROM `intro` WHERE NOT `uid` IN (SELECT `uid` FROM `user`)")) {
+               return Update::FAILED;
+       }
+
+       if (!DBA::e("DELETE FROM `manage` WHERE NOT `uid` IN (SELECT `uid` FROM `user`)")) {
+               return Update::FAILED;
+       }
+
+       if (!DBA::e("DELETE FROM `manage` WHERE NOT `mid` IN (SELECT `uid` FROM `user`)")) {
+               return Update::FAILED;
+       }
+
+       if (!DBA::e("DELETE FROM `mail` WHERE NOT `uid` IN (SELECT `uid` FROM `user`)")) {
+               return Update::FAILED;
+       }
+
+       if (!DBA::e("DELETE FROM `mailacct` WHERE NOT `uid` IN (SELECT `uid` FROM `user`)")) {
+               return Update::FAILED;
+       }
+
+       if (!DBA::e("DELETE FROM `notify` WHERE NOT `uid` IN (SELECT `uid` FROM `user`)")) {
+               return Update::FAILED;
+       }
+
+       if (!DBA::e("DELETE FROM `openwebauth-token` WHERE NOT `uid` IN (SELECT `uid` FROM `user`)")) {
+               return Update::FAILED;
+       }
+
+       if (!DBA::e("DELETE FROM `pconfig` WHERE NOT `uid` IN (SELECT `uid` FROM `user`)")) {
+               return Update::FAILED;
+       }
+
+       if (!DBA::e("DELETE FROM `profile` WHERE NOT `uid` IN (SELECT `uid` FROM `user`)")) {
+               return Update::FAILED;
+       }
+
+       if (!DBA::e("DELETE FROM `profile_check` WHERE NOT `uid` IN (SELECT `uid` FROM `user`)")) {
+               return Update::FAILED;
+       }
+
+       if (!DBA::e("DELETE FROM `profile_field` WHERE NOT `uid` IN (SELECT `uid` FROM `user`)")) {
+               return Update::FAILED;
+       }
+
+       if (!DBA::e("DELETE FROM `push_subscriber` WHERE NOT `uid` IN (SELECT `uid` FROM `user`)")) {
+               return Update::FAILED;
+       }
+
+       if (!DBA::e("DELETE FROM `register` WHERE NOT `uid` IN (SELECT `uid` FROM `user`)")) {
+               return Update::FAILED;
+       }
+
+       if (!DBA::e("DELETE FROM `search` WHERE NOT `uid` IN (SELECT `uid` FROM `user`)")) {
+               return Update::FAILED;
+       }
+
+       if (!DBA::e("DELETE FROM `tokens` WHERE NOT `uid` IN (SELECT `uid` FROM `user`)")) {
+               return Update::FAILED;
+       }
+
+       if (!DBA::e("DELETE FROM `user-contact` WHERE NOT `uid` IN (SELECT `uid` FROM `user`)")) {
+               return Update::FAILED;
+       }
+
+       if (!DBA::e("DELETE FROM `user-item` WHERE NOT `uid` IN (SELECT `uid` FROM `user`)")) {
+               return Update::FAILED;
+       }
+
+       if (!DBA::e("DELETE FROM `notify-threads` WHERE NOT `receiver-uid` IN (SELECT `uid` FROM `user`)")) {
+               return Update::FAILED;
+       }
+
+       if (!DBA::e("DELETE FROM `event` WHERE NOT `cid` IN (SELECT `id` FROM `contact`)")) {
+               return Update::FAILED;
+       }
+
+       if (!DBA::e("DELETE FROM `fsuggest` WHERE NOT `cid` IN (SELECT `id` FROM `contact`)")) {
+               return Update::FAILED;
+       }
+
+       if (!DBA::e("DELETE FROM `group_member` WHERE NOT `contact-id` IN (SELECT `id` FROM `contact`)")) {
+               return Update::FAILED;
+       }
+
+       if (!DBA::e("DELETE FROM `intro` WHERE NOT `contact-id` IN (SELECT `id` FROM `contact`)")) {
+               return Update::FAILED;
+       }
+
+       if (!DBA::e("DELETE FROM `participation` WHERE NOT `cid` IN (SELECT `id` FROM `contact`)")) {
+               return Update::FAILED;
+       }
+
+       if (!DBA::e("DELETE FROM `profile_check` WHERE NOT `cid` IN (SELECT `id` FROM `contact`)")) {
+               return Update::FAILED;
+       }
+
+       if (!DBA::e("DELETE FROM `user-contact` WHERE NOT `cid` IN (SELECT `id` FROM `contact`)")) {
+               return Update::FAILED;
+       }
+
+       if (!DBA::e("DELETE FROM `participation` WHERE NOT `fid` IN (SELECT `id` FROM `fcontact`)")) {
+               return Update::FAILED;
+       }
+
+       if (!DBA::e("DELETE FROM `group_member` WHERE NOT `gid` IN (SELECT `id` FROM `group`)")) {
+               return Update::FAILED;
+       }
+
+       if (!DBA::e("DELETE FROM `gserver-tag` WHERE NOT `gserver-id` IN (SELECT `id` FROM `gserver`)")) {
+               return Update::FAILED;
+       }
+
+       if (!DBA::e("DELETE FROM `participation` WHERE NOT `iid` IN (SELECT `id` FROM `item`)")) {
+               return Update::FAILED;
+       }
+
+       if (!DBA::e("DELETE FROM `user-item` WHERE NOT `iid` IN (SELECT `id` FROM `item`)")) {
+               return Update::FAILED;
+       }
+
+       return Update::SUCCESS;
+}
+
+function pre_update_1365()
+{
+       if (!DBA::e("DELETE FROM `notify-threads` WHERE NOT `notify-id` IN (SELECT `id` FROM `notify`)")) {
+               return Update::FAILED;
+       }
+
+       if (!DBA::e("DELETE FROM `thread` WHERE NOT `iid` IN (SELECT `id` FROM `item`)")) {
+               return Update::FAILED;
+       }
+
+}