Page MenuHomePhabricator

Clean up a couple more URI alter() calls
ClosedPublic

Authored by epriestley on Feb 15 2019, 9:58 PM.
Tags
None
Referenced Files
F13128427: D20184.diff
Tue, Apr 30, 11:55 PM
Unknown Object (File)
Thu, Apr 25, 12:43 AM
Unknown Object (File)
Mar 29 2024, 5:02 AM
Unknown Object (File)
Mar 29 2024, 12:51 AM
Unknown Object (File)
Mar 23 2024, 11:27 AM
Unknown Object (File)
Mar 15 2024, 10:56 AM
Unknown Object (File)
Feb 11 2024, 7:44 PM
Unknown Object (File)
Jan 15 2024, 3:17 PM
Subscribers
None

Details

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.

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

This revision is now accepted and ready to land.Feb 15 2019, 10:01 PM
This revision was automatically updated to reflect the committed changes.