From a5d24609748c449efcf23a8296ca70391f481e5b Mon Sep 17 00:00:00 2001 From: epriestley Date: Wed, 13 Aug 2014 10:06:41 -0700 Subject: [PATCH] Probably fix bad method call in Diffusion Summary: Fixes T5869. Ref T4896. This `setID()` method no longer exists. Test Plan: (WARNING) This is a pain to reproduce locally so I'm just winging it. I'm 99% sure this ID is only used to generate an anchor link. This is a hack to start with, and T4896 will eventualy clean it up properly. Reviewers: btrahan Reviewed By: btrahan Subscribers: epriestley Maniphest Tasks: T4896, T5869 Differential Revision: https://secure.phabricator.com/D10254 --- .../diffusion/controller/DiffusionBrowseFileController.php | 1 - 1 file changed, 1 deletion(-) diff --git a/src/applications/diffusion/controller/DiffusionBrowseFileController.php b/src/applications/diffusion/controller/DiffusionBrowseFileController.php index 2b3c879577..cd744ef05f 100644 --- a/src/applications/diffusion/controller/DiffusionBrowseFileController.php +++ b/src/applications/diffusion/controller/DiffusionBrowseFileController.php @@ -613,7 +613,6 @@ final class DiffusionBrowseFileController extends DiffusionBrowseController { foreach ($this->lintMessages as $message) { $inline = id(new PhabricatorAuditInlineComment()) - ->setID($message['id']) ->setSyntheticAuthor( ArcanistLintSeverity::getStringForSeverity($message['severity']). ' '.$message['code'].' ('.$message['name'].')')