revi-archive/phorge-phorge
1
0
Fork 0
mirror of https://we.phorge.it/source/phorge.git synced 2025-01-18 18:51:12 +01:00
Code Issues Releases Wiki Activity

Fix unmigrated load() call in Audit inlines

Browse source 
Summary: Fixes T5711. I missed this somehow in grepping. :/

Test Plan: Edited and deleted an inline draft.

Reviewers: chad, btrahan, joshuaspence

Reviewed By: joshuaspence

Subscribers: epriestley

Maniphest Tasks: T5711

Differential Revision: https://secure.phabricator.com/D10051
This commit is contained in:
epriestley 2014-07-25 06:23:44 -07:00
parent 1450c0e8d6
commit 51b5bf1e67
2 changed files with 12 additions and 1 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

11
src/applications/audit/storage/PhabricatorAuditInlineComment.php
View file

@ -28,6 +28,17 @@ final class PhabricatorAuditInlineComment
return $this->proxy;
}
public static function loadID($id) {
$inlines = id(new PhabricatorAuditTransactionComment())->loadAllWhere(
'id = %d',
$id);
if (!$inlines) {
return null;
}
return head(self::buildProxies($inlines));
}
public static function loadDraftComments(
PhabricatorUser $viewer,
$commit_phid) {

2
src/applications/diffusion/controller/DiffusionInlineCommentController.php
View file

@ -40,7 +40,7 @@ final class DiffusionInlineCommentController
}
protected function loadComment($id) {
return id(new PhabricatorAuditInlineComment())->load($id);
return PhabricatorAuditInlineComment::loadID($id);
}
protected function loadCommentForEdit($id) {