X-Git-Url: https://git.mxchange.org/?a=blobdiff_plain;f=update.php;h=83011108b539b6dcaafcd7b724e01d33bd9e4584;hb=7f515cdaa4074c2919e5117bc41c0b23ac74de27;hp=d889e241e22061f56b549857a82995ce023c8957;hpb=a19f8d642d6078b2a9f43770fcc863b5c006c026;p=friendica.git diff --git a/update.php b/update.php index d889e241e2..83011108b5 100644 --- a/update.php +++ b/update.php @@ -1,28 +1,31 @@ . * - * update.php - automatic system update + * Automatic post-databse structure change updates * - * This function is responsible for doing post update changes to the data - * (not the structure) in the database. + * These functions are responsible for doing critical post update changes to the data (not the structure) in the database. * - * Database structure changes are done in src/Database/DBStructure.php + * Database structure changes are done in static/dbstructure.config.php * - * If there is a need for a post process to a structure change, update this file + * For non-critical database migrations, please add a method in the Database\PostUpdate class + * + * If there is a need for a post update to a structure change, update this file * by adding a new function at the end with the number of the new DB_UPDATE_VERSION. * * The numbered script in this file has to be exactly like the DB_UPDATE_VERSION @@ -31,83 +34,67 @@ require_once 'include/dba.php'; * You are currently on version 4711 and you are preparing changes that demand an update script. * * 1. Create a function "update_4712()" here in the update.php - * 2. Apply the needed structural changes in src/Database/DBStructure.php - * 3. Set DB_UPDATE_VERSION in boot.php to 4712. + * 2. Apply the needed structural changes in static/dbStructure.php + * 3. Set DB_UPDATE_VERSION in static/dbstructure.config.php to 4712. * * If you need to run a script before the database update, name the function "pre_update_4712()" */ -function update_1178() { - require_once 'mod/profiles.php'; - - $profiles = q("SELECT `uid`, `about`, `locality`, `pub_keywords`, `gender` FROM `profile` WHERE `is-default`"); - - foreach ($profiles AS $profile) { - if ($profile["about"].$profile["locality"].$profile["pub_keywords"].$profile["gender"] == "") - continue; - - $profile["pub_keywords"] = profile_clean_keywords($profile["pub_keywords"]); +use Friendica\Core\Addon; +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\User; +use Friendica\Model\Storage; +use Friendica\Util\DateTimeFormat; +use Friendica\Worker\Delivery; - $r = q("UPDATE `contact` SET `about` = '%s', `location` = '%s', `keywords` = '%s', `gender` = '%s' WHERE `self` AND `uid` = %d", - DBA::escape($profile["about"]), - DBA::escape($profile["locality"]), - DBA::escape($profile["pub_keywords"]), - DBA::escape($profile["gender"]), - intval($profile["uid"]) - ); +function update_1179() +{ + if (DI::config()->get('system', 'no_community_page')) { + DI::config()->set('system', 'community_page_style', CP_NO_COMMUNITY_PAGE); } -} - -function update_1179() { - if (Config::get('system','no_community_page')) - Config::set('system','community_page_style', CP_NO_COMMUNITY_PAGE); // Update the central item storage with uid=0 Worker::add(PRIORITY_LOW, "threadupdate"); - return UPDATE_SUCCESS; + return Update::SUCCESS; } -function update_1181() { +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; + return Update::SUCCESS; } -function update_1189() { +function update_1189() +{ - if (strlen(Config::get('system','directory_submit_url')) && - !strlen(Config::get('system','directory'))) { - Config::set('system','directory', dirname(Config::get('system','directory_submit_url'))); - Config::delete('system','directory_submit_url'); + if (strlen(DI::config()->get('system', 'directory_submit_url')) && + !strlen(DI::config()->get('system', 'directory'))) { + DI::config()->set('system', 'directory', dirname(DI::config()->get('system', 'directory_submit_url'))); + DI::config()->delete('system', 'directory_submit_url'); } - return UPDATE_SUCCESS; + return Update::SUCCESS; } -function update_1191() { - Config::set('system', 'maintenance', 1); +function update_1191() +{ + DI::config()->set('system', 'maintenance', 1); if (Addon::isEnabled('forumlist')) { - $addon = 'forumlist'; - $addons = 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' "); - Config::set('system','addon', implode(", ", $addons_arr)); - } - } + Addon::uninstall('forumlist'); } // select old formlist addon entries @@ -123,64 +110,74 @@ function update_1191() { $key = $rr['k']; $value = $rr['v']; - if ($key === 'randomise') - PConfig::delete($uid,$family,$key); + if ($key === 'randomise') { + DI::pConfig()->delete($uid, $family, $key); + } if ($key === 'show_on_profile') { - if ($value) - PConfig::set($uid,feature,forumlist_profile,$value); + if ($value) { + DI::pConfig()->set($uid, 'feature', 'forumlist_profile', $value); + } - PConfig::delete($uid,$family,$key); + DI::pConfig()->delete($uid, $family, $key); } if ($key === 'show_on_network') { - if ($value) - PConfig::set($uid,feature,forumlist_widget,$value); + if ($value) { + DI::pConfig()->set($uid, 'feature', 'forumlist_widget', $value); + } - PConfig::delete($uid,$family,$key); + DI::pConfig()->delete($uid, $family, $key); } } } - Config::set('system', 'maintenance', 0); + DI::config()->set('system', 'maintenance', 0); - return UPDATE_SUCCESS; + return Update::SUCCESS; } -function update_1203() { +function update_1203() +{ $r = q("UPDATE `user` SET `account-type` = %d WHERE `page-flags` IN (%d, %d)", - DBA::escape(Contact::ACCOUNT_TYPE_COMMUNITY), DBA::escape(Contact::PAGE_COMMUNITY), DBA::escape(Contact::PAGE_PRVGROUP)); + DBA::escape(User::ACCOUNT_TYPE_COMMUNITY), + DBA::escape(User::PAGE_FLAGS_COMMUNITY), + DBA::escape(User::PAGE_FLAGS_PRVGROUP) + ); } -function update_1244() { +function update_1244() +{ // Sets legacy_password for all legacy hashes DBA::update('user', ['legacy_password' => true], ['SUBSTR(password, 1, 4) != "$2y$"']); // All legacy hashes are re-hashed using the new secure hashing function $stmt = DBA::select('user', ['uid', 'password'], ['legacy_password' => true]); - while($user = DBA::fetch($stmt)) { + while ($user = DBA::fetch($stmt)) { DBA::update('user', ['password' => User::hashPassword($user['password'])], ['uid' => $user['uid']]); } // Logged in users are forcibly logged out DBA::delete('session', ['1 = 1']); - return UPDATE_SUCCESS; + return Update::SUCCESS; } -function update_1245() { - $rino = Config::get('system', 'rino_encrypt'); +function update_1245() +{ + $rino = DI::config()->get('system', 'rino_encrypt'); if (!$rino) { - return UPDATE_SUCCESS; + return Update::SUCCESS; } - Config::set('system', 'rino_encrypt', 1); + DI::config()->set('system', 'rino_encrypt', 1); - return UPDATE_SUCCESS; + return Update::SUCCESS; } -function update_1247() { +function update_1247() +{ // Removing hooks with the old name DBA::e("DELETE FROM `hook` WHERE `hook` LIKE 'plugin_%'"); @@ -189,9 +186,17 @@ WHERE `hook` LIKE 'plugin_%'"); Addon::reload(); } -function update_1260() { - Config::set('system', 'maintenance', 1); - Config::set('system', 'maintenance_reason', L10n::t('%s: Updating author-id and owner-id in item and thread table. ', DateTimeFormat::utcNow().' '.date('e'))); +function update_1260() +{ + DI::config()->set('system', 'maintenance', 1); + DI::config()->set( + 'system', + 'maintenance_reason', + DI::l10n()->t( + '%s: Updating author-id and owner-id in item and thread table. ', + DateTimeFormat::utcNow().' '.date('e') + ) + ); $items = DBA::p("SELECT `id`, `owner-link`, `owner-name`, `owner-avatar`, `network` FROM `item` WHERE `owner-id` = 0 AND `owner-link` != ''"); @@ -225,33 +230,292 @@ function update_1260() { DBA::e("UPDATE `thread` INNER JOIN `item` ON `thread`.`iid` = `item`.`id` SET `thread`.`author-id` = `item`.`author-id` WHERE `thread`.`author-id` = 0"); - Config::set('system', 'maintenance', 0); - return UPDATE_SUCCESS; + DI::config()->set('system', 'maintenance', 0); + return Update::SUCCESS; } -function update_1261() { +function update_1261() +{ // This fixes the results of an issue in the develop branch of 2018-05. DBA::update('contact', ['blocked' => false, 'pending' => false], ['uid' => 0, 'blocked' => true, 'pending' => true]); - return UPDATE_SUCCESS; + return Update::SUCCESS; } -function update_1278() { - Config::set('system', 'maintenance', 1); - Config::set('system', 'maintenance_reason', L10n::t('%s: Updating post-type.', DateTimeFormat::utcNow().' '.date('e'))); +function update_1278() +{ + DI::config()->set('system', 'maintenance', 1); + DI::config()->set( + 'system', + 'maintenance_reason', + DI::l10n()->t( + '%s: Updating post-type.', + DateTimeFormat::utcNow().' '.date('e') + ) + ); Item::update(['post-type' => Item::PT_PAGE], ['bookmark' => true]); Item::update(['post-type' => Item::PT_PERSONAL_NOTE], ['type' => 'note']); - Config::set('system', 'maintenance', 0); + DI::config()->set('system', 'maintenance', 0); - return UPDATE_SUCCESS; + return Update::SUCCESS; } -function update_1288() { +function update_1288() +{ // Updates missing `uri-id` values DBA::e("UPDATE `item-activity` INNER JOIN `item` ON `item`.`iaid` = `item-activity`.`id` SET `item-activity`.`uri-id` = `item`.`uri-id` WHERE `item-activity`.`uri-id` IS NULL OR `item-activity`.`uri-id` = 0"); DBA::e("UPDATE `item-content` INNER JOIN `item` ON `item`.`icid` = `item-content`.`id` SET `item-content`.`uri-id` = `item`.`uri-id` WHERE `item-content`.`uri-id` IS NULL OR `item-content`.`uri-id` = 0"); - return UPDATE_SUCCESS; + return Update::SUCCESS; +} + +// Post-update script of PR 5751 +function update_1298() +{ + $keys = ['gender', 'marital', 'sexual']; + foreach ($keys as $translateKey) { + $allData = DBA::select('profile', ['id', $translateKey]); + $allLangs = DI::l10n()->getAvailableLanguages(); + $success = 0; + $fail = 0; + foreach ($allData as $key => $data) { + $toTranslate = $data[$translateKey]; + if ($toTranslate != '') { + foreach ($allLangs as $key => $lang) { + $a = new \stdClass(); + $a->strings = []; + + // First we get the the localizations + if (file_exists("view/lang/$lang/strings.php")) { + include "view/lang/$lang/strings.php"; + } + if (file_exists("addon/morechoice/lang/$lang/strings.php")) { + include "addon/morechoice/lang/$lang/strings.php"; + } + + $localizedStrings = $a->strings; + unset($a); + + $key = array_search($toTranslate, $localizedStrings); + if ($key !== false) { + break; + } + + // defaulting to empty string + $key = ''; + } + + if ($key == '') { + $fail++; + } else { + DBA::update('profile', [$translateKey => $key], ['id' => $data['id']]); + Logger::notice('Updated contact', ['action' => 'update', 'contact' => $data['id'], "$translateKey" => $key, + 'was' => $data[$translateKey]]); + Worker::add(PRIORITY_LOW, 'ProfileUpdate', $data['id']); + Contact::updateSelfFromUserID($data['id']); + GContact::updateForUser($data['id']); + $success++; + } + } + } + + Logger::notice($translateKey . " fix completed", ['action' => 'update', 'translateKey' => $translateKey, 'Success' => $success, 'Fail' => $fail ]); + } + return Update::SUCCESS; +} + +function update_1309() +{ + $queue = DBA::select('queue', ['id', 'cid', 'guid']); + while ($entry = DBA::fetch($queue)) { + $contact = DBA::selectFirst('contact', ['uid'], ['id' => $entry['cid']]); + if (!DBA::isResult($contact)) { + continue; + } + + $item = Item::selectFirst(['id', 'gravity'], ['uid' => $contact['uid'], 'guid' => $entry['guid']]); + if (!DBA::isResult($item)) { + continue; + } + + $deliver_options = ['priority' => PRIORITY_MEDIUM, 'dont_fork' => true]; + Worker::add($deliver_options, 'Delivery', Delivery::POST, $item['id'], $entry['cid']); + Logger::info('Added delivery worker', ['item' => $item['id'], 'contact' => $entry['cid']]); + DBA::delete('queue', ['id' => $entry['id']]); + } + return Update::SUCCESS; +} + +function update_1315() +{ + DBA::delete('item-delivery-data', ['postopts' => '', 'inform' => '', 'queue_count' => 0, 'queue_done' => 0]); + return Update::SUCCESS; +} + +function update_1318() +{ + DBA::update('profile', ['marital' => "In a relation"], ['marital' => "Unavailable"]); + DBA::update('profile', ['marital' => "Single"], ['marital' => "Available"]); + + Worker::add(PRIORITY_LOW, 'ProfileUpdate'); + return Update::SUCCESS; +} + +function update_1323() +{ + $users = DBA::select('user', ['uid']); + while ($user = DBA::fetch($users)) { + Contact::updateSelfFromUserID($user['uid']); + } + DBA::close($users); + + return Update::SUCCESS; +} + +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']); + } + DBA::close($contacts); + + return Update::SUCCESS; +} + +function update_1330() +{ + $currStorage = DI::config()->get('storage', 'class', ''); + + // set the name of the storage instead of the classpath as config + if (!empty($currStorage)) { + /** @var Storage\IStorage $currStorage */ + if (!DI::config()->set('storage', 'name', $currStorage::getName())) { + return Update::FAILED; + } + + // try to delete the class since it isn't needed. This won't work with config files + DI::config()->delete('storage', 'class'); + } + + // Update attachments and photos + if (!DBA::p("UPDATE `photo` SET `photo`.`backend-class` = SUBSTR(`photo`.`backend-class`, 25) WHERE `photo`.`backend-class` LIKE 'Friendica\\\Model\\\Storage\\\%' ESCAPE '|'") || + !DBA::p("UPDATE `attach` SET `attach`.`backend-class` = SUBSTR(`attach`.`backend-class`, 25) WHERE `attach`.`backend-class` LIKE 'Friendica\\\Model\\\Storage\\\%' ESCAPE '|'")) { + return Update::FAILED; + }; + + return Update::SUCCESS; +} + +function update_1332() +{ + $condition = ["`is-default` IS NOT NULL"]; + $profiles = DBA::select('profile', [], $condition); + + while ($profile = DBA::fetch($profiles)) { + DI::profileField()->migrateFromLegacyProfile($profile); + } + DBA::close($profiles); + + DBA::update('contact', ['profile-id' => null], ['`profile-id` IS NOT NULL']); + + 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(); + + 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']) + && !DBA::e("ALTER TABLE `contact` CHANGE `ffi_keyword_blacklist` `ffi_keyword_denylist` text null")) { + return Update::FAILED; + } + + return Update::SUCCESS; }