mirror of
https://we.phorge.it/source/phorge.git
synced 2024-11-09 16:32:39 +01:00
Update a very old Phriction migration which incorrectly uses "save()"
Summary: See <https://discourse.phabricator-community.org/t/storage-upgrade-error/3748>. It is broadly unsafe for migrations to use "save()". If the object gains new fields later, the query will include "SET newField = X", which will fail against the old schema which is in the process of being upgraded. Instead, migrations must issue raw SQL against the schema as it is expected to exist at the time the migration executes. Migrations have followed this rule for a long time, but this ~6 year old migration was overlooked. Update it to issue a raw query to perform the policy update. Test Plan: This is somewhat flimsy since rebuilding a genuine reproduction case is messy, but used "bin/storage --apply ..." to at least get the new query to execute against modern Phabricator without issues. Subscribers: PHID-OPKG-gm6ozazyms6q6i22gyam Differential Revision: https://secure.phabricator.com/D21124
This commit is contained in:
parent
45665dd3b4
commit
d86506052c
1 changed files with 15 additions and 14 deletions
|
@ -16,6 +16,9 @@ foreach (new LiskMigrationIterator($table) as $doc) {
|
|||
continue;
|
||||
}
|
||||
|
||||
$new_view_policy = $default_view_policy;
|
||||
$new_edit_policy = $default_edit_policy;
|
||||
|
||||
// If this was previously a magical project wiki page (under projects/, but
|
||||
// not projects/ itself) we need to apply the project policies. Otherwise,
|
||||
// apply the default policies.
|
||||
|
@ -35,26 +38,24 @@ foreach (new LiskMigrationIterator($table) as $doc) {
|
|||
->executeOne();
|
||||
|
||||
if ($project) {
|
||||
|
||||
$view_policy = nonempty($project->getViewPolicy(), $default_view_policy);
|
||||
$edit_policy = nonempty($project->getEditPolicy(), $default_edit_policy);
|
||||
|
||||
$project_name = $project->getName();
|
||||
echo pht(
|
||||
"Migrating document %d to project policy %s...\n",
|
||||
$id,
|
||||
$project_name);
|
||||
$doc->setViewPolicy($view_policy);
|
||||
$doc->setEditPolicy($edit_policy);
|
||||
$doc->save();
|
||||
continue;
|
||||
$new_view_policy = $view_policy;
|
||||
$new_edit_policy = $edit_policy;
|
||||
}
|
||||
}
|
||||
|
||||
echo pht('Migrating document %d to default install policy...', $id)."\n";
|
||||
$doc->setViewPolicy($default_view_policy);
|
||||
$doc->setEditPolicy($default_edit_policy);
|
||||
$doc->save();
|
||||
echo pht('Migrating document %d to new policy...', $id)."\n";
|
||||
|
||||
queryfx(
|
||||
$conn_w,
|
||||
'UPDATE %R SET viewPolicy = %s, editPolicy = %s
|
||||
WHERE id = %d',
|
||||
$table,
|
||||
$new_view_policy,
|
||||
$new_edit_policy,
|
||||
$id);
|
||||
}
|
||||
|
||||
echo pht('Done.')."\n";
|
||||
|
|
Loading…
Reference in a new issue