mirror of
https://we.phorge.it/source/phorge.git
synced 2024-11-30 18:52:42 +01:00
2d240142c8
Summary: Fixes T3803. Turns out my advice was sort of terrible. :/ In strict mode, the `INSERT INTO x (y, z)` raises an error unless `(y, z, ...)` includes //all// columns without default values. Test Plan: Ran `storage upgrade` on a strict-mode install. Verified no inserts were performed. Reviewers: btrahan Reviewed By: btrahan CC: aran Maniphest Tasks: T3803 Differential Revision: https://secure.phabricator.com/D6894
38 lines
1 KiB
PHP
38 lines
1 KiB
PHP
<?php
|
|
|
|
echo "Populating Phabricator files with mail keys xactions...\n";
|
|
|
|
$table = new PhabricatorFile();
|
|
$table_name = $table->getTableName();
|
|
|
|
$conn_w = $table->establishConnection('w');
|
|
$conn_w->openTransaction();
|
|
|
|
$sql = array();
|
|
foreach (new LiskRawMigrationIterator($conn_w, 'file') as $row) {
|
|
// NOTE: MySQL requires that the INSERT specify all columns which don't
|
|
// have default values when configured in strict mode. This query will
|
|
// never actually insert rows, but we need to hand it values anyway.
|
|
|
|
$sql[] = qsprintf(
|
|
$conn_w,
|
|
'(%d, %s, 0, 0, 0, 0, 0, 0, 0, 0)',
|
|
$row['id'],
|
|
Filesystem::readRandomCharacters(20));
|
|
}
|
|
|
|
if ($sql) {
|
|
foreach (PhabricatorLiskDAO::chunkSQL($sql, ', ') as $chunk) {
|
|
queryfx(
|
|
$conn_w,
|
|
'INSERT INTO %T
|
|
(id, mailKey, phid, byteSize, storageEngine, storageFormat,
|
|
storageHandle, dateCreated, dateModified, metadata) VALUES %Q '.
|
|
'ON DUPLICATE KEY UPDATE mailKey = VALUES(mailKey)',
|
|
$table_name,
|
|
$chunk);
|
|
}
|
|
}
|
|
|
|
$table->saveTransaction();
|
|
echo "Done.\n";
|