revi-archive/phorge-phorge
1
0
Fork 0
mirror of https://we.phorge.it/source/phorge.git synced 2024-11-23 15:22:41 +01:00
Code Issues Releases Wiki Activity

Make "bin/audit synchronize" actually save changes

Browse source 
Summary: Ref T10978. Although this script prints out some very good changes, it does not currently persist them to the database.

Test Plan: Ran `bin/audit synchronize`, saw the change appear both on the CLI and in the database.

Reviewers: chad

Reviewed By: chad

Maniphest Tasks: T10978

Differential Revision: https://secure.phabricator.com/D17276
This commit is contained in:
epriestley 2017-01-31 12:03:15 -08:00
parent 2a527a51f2
commit a3417ccd78
1 changed files with 3 additions and 1 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

4
src/applications/audit/management/PhabricatorAuditSynchronizeManagementWorkflow.php
View file

@ -44,12 +44,14 @@ final class PhabricatorAuditSynchronizeManagementWorkflow
echo tsprintf( echo tsprintf(
"%s\n", "%s\n",
pht( pht(
'Updated "%s": "%s" -> "%s".', 'Updating "%s": "%s" -> "%s".',
$commit->getDisplayName(), $commit->getDisplayName(),
PhabricatorAuditCommitStatusConstants::getStatusName( PhabricatorAuditCommitStatusConstants::getStatusName(
$old_status), $old_status),
PhabricatorAuditCommitStatusConstants::getStatusName( PhabricatorAuditCommitStatusConstants::getStatusName(
$new_status))); $new_status)));
$commit->save();
} }
} }
} }