Changeset View
Changeset View
Standalone View
Standalone View
resources/sql/autopatches/20180910.audit.04.xactions.php
- This file was added.
<?php | |||||
$table = new PhabricatorAuditTransaction(); | |||||
$conn = $table->establishConnection('w'); | |||||
$status_map = array( | |||||
0 => 'none', | |||||
1 => 'needs-audit', | |||||
2 => 'concern-raised', | |||||
3 => 'partially-audited', | |||||
4 => 'audited', | |||||
5 => 'needs-verification', | |||||
); | |||||
$state_type = DiffusionCommitStateTransaction::TRANSACTIONTYPE; | |||||
foreach (new LiskMigrationIterator($table) as $xaction) { | |||||
if ($xaction->getTransactionType() !== $state_type) { | |||||
continue; | |||||
} | |||||
$old_value = $xaction->getOldValue(); | |||||
$new_value = $xaction->getNewValue(); | |||||
amckinley: No need to update `old_value`? | |||||
Done Inline ActionsOh! Nothing ever uses the old value so I thought it wasn't populated, but I went back and double checked and it actually is. So we don't need to, but I'll update this so we do just for completeness. epriestley: Oh! Nothing ever uses the old value so I thought it wasn't populated, but I went back and… | |||||
$any_change = false; | |||||
if (isset($status_map[$old_value])) { | |||||
$old_value = $status_map[$old_value]; | |||||
$any_change = true; | |||||
} | |||||
if (isset($status_map[$new_value])) { | |||||
$new_value = $status_map[$new_value]; | |||||
$any_change = true; | |||||
} | |||||
if (!$any_change) { | |||||
continue; | |||||
} | |||||
queryfx( | |||||
$conn, | |||||
'UPDATE %T SET oldValue = %s, newValue = %s WHERE id = %d', | |||||
$table->getTableName(), | |||||
phutil_json_encode($old_value), | |||||
phutil_json_encode($new_value), | |||||
$xaction->getID()); | |||||
} |
No need to update old_value?