Page MenuHomePhabricator

Reduce reliance on `getRevisionID()` on DifferentialComment
ClosedPublic

Authored by epriestley on Feb 12 2014, 6:47 PM.
Tags
None
Referenced Files
F18820293: D8209.id18569.diff
Wed, Oct 22, 1:58 PM
F18761430: D8209.diff
Mon, Oct 6, 2:18 PM
F18678090: D8209.diff
Sep 25 2025, 10:18 PM
F18670491: D8209.id18573.diff
Sep 25 2025, 12:29 AM
F18629655: D8209.diff
Sep 16 2025, 9:32 AM
F18510728: D8209.id18573.diff
Sep 5 2025, 5:07 AM
F18510440: D8209.id18569.diff
Sep 5 2025, 4:28 AM
F18508845: D8209.id.diff
Sep 5 2025, 3:03 AM
Subscribers

Details

Summary

Ref T2222. A few rendering interfaces rely on fishing the revision ID out of a DifferentialComment, but it will only have the PHID soon. Pass in the revision and use it to determine the ID instead.

Test Plan

Browsed, previewed, examined comments. Clicked anchors.

Diff Detail

Lint
Lint Skipped
Unit
Tests Skipped

Event Timeline

lamest inline feedback in awhile

src/applications/differential/controller/DifferentialCommentPreviewController.php
51

these could all be $viewer I s'pose