Page MenuHomePhabricator

Replace "getRequestURI()->setQueryParams(array())" with "getPath()"
ClosedPublic

Authored by epriestley on Feb 12 2019, 4:09 PM.
Tags
None
Referenced Files
Unknown Object (File)
Thu, Jul 3, 10:45 AM
Unknown Object (File)
Mon, Jun 30, 2:41 PM
Unknown Object (File)
Fri, Jun 27, 11:47 AM
Unknown Object (File)
Fri, Jun 27, 4:07 AM
Unknown Object (File)
Thu, Jun 26, 12:04 PM
Unknown Object (File)
May 27 2025, 12:08 PM
Unknown Object (File)
May 26 2025, 5:02 AM
Unknown Object (File)
May 25 2025, 3:06 PM
Subscribers
None

Details

Summary

Ref T13250. A handful of callsites are doing getRequestURI() + setQueryParams(array()) to get a bare request path.

They can just use getPath() instead.

Test Plan

See inlines.

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

src/aphront/response/AphrontAjaxResponse.php
34–51

Test plan: viewed a revision in Differential, opened DarkConsole, saw the Ajax "request" tabs populate on the left.

src/applications/chatlog/controller/PhabricatorChatLogChannelLogController.php
14

Test plan: viewed /chatlog/channel/1/, which hits this code before 404'ing.

src/applications/metamta/applicationpanel/PhabricatorMetaMTAApplicationEmailPanel.php
57

Test plan: added and removed application email addresses for Maniphest.

src/applications/settings/panel/PhabricatorEmailAddressesSettingsPanel.php
34

Test plan: added and removed account email addresses.

src/applications/transactions/editengine/PhabricatorEditEngine.php
1282

Test plan: edited a task.

This revision is now accepted and ready to land.Feb 12 2019, 7:29 PM
This revision was automatically updated to reflect the committed changes.