HomePhabricator

Clean up a PhutilURI "alter()" callsite in Diffusion blame

Description

Clean up a PhutilURI "alter()" callsite in Diffusion blame

Summary: See https://discourse.phabricator-community.org/t/exception-when-viewing-previous-revision-from-blame/2454/2.

Test Plan: Viewed blame, clicked "Skip Past This Commit". Got jumped to the right place instead of a URI exception.

Reviewers: amckinley

Reviewed By: amckinley

Differential Revision: https://secure.phabricator.com/D20222

Details

Provenance
epriestleyAuthored on Feb 28 2019, 1:43 PM
epriestleyPushed on Mar 1 2019, 3:47 AM
Reviewer
amckinley
Differential Revision
D20222: Clean up a PhutilURI "alter()" callsite in Diffusion blame
Parents
rP620047bcfe3f: Add a "Recent Builds" element to the Build Plan UI and tighten up a few odds…
Branches
Unknown
Tags
Unknown
Build Status
Buildable 22158
Build 30284: Run Core Tests