HomePhabricator

Clean up a couple more URI alter() calls

Description

Clean up a couple more URI alter() calls

Summary:
See https://discourse.phabricator-community.org/t/create-new-phriction-document-fails-with-unhandled-exception-invalidargumentexception/2406.

These weren't obviously nullable from a cursory grep, but are sometimes nullable in practice.

Test Plan: Created, then saved a new Phriction document.

Reviewers: amckinley

Reviewed By: amckinley

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

Details

Provenance
epriestleyAuthored on Feb 15 2019, 9:57 PM
epriestleyPushed on Feb 15 2019, 10:07 PM
Reviewer
amckinley
Differential Revision
D20184: Clean up a couple more URI alter() calls
Parents
rPc5772f51dea7: Fix Content-Security-Policy headers on "Email Login" page
Branches
Unknown
Tags
Unknown
Build Status
Buildable 22025
Build 30091: Run Core Tests