Page MenuHomePhabricator

Replace all "setQueryParam()" calls with "remove/replaceQueryParam()"
ClosedPublic

Authored by epriestley on Feb 12 2019, 9:38 PM.
Tags
None
Referenced Files
F15422048: D20154.id48120.diff
Sat, Mar 22, 5:00 AM
F15420535: D20154.id48161.diff
Fri, Mar 21, 2:44 PM
F15411829: D20154.id.diff
Wed, Mar 19, 9:57 AM
F15408013: D20154.diff
Tue, Mar 18, 8:51 PM
F15407025: D20154.id.diff
Tue, Mar 18, 3:14 PM
Unknown Object (File)
Feb 21 2025, 6:29 PM
Unknown Object (File)
Feb 9 2025, 7:11 AM
Unknown Object (File)
Feb 9 2025, 7:11 AM
Subscribers

Details

Summary

Ref T13250. See D20149. Mostly: clarify semantics. Partly: remove magic "null" behavior.

Test Plan

Poked around, but mostly just inspection since these are pretty much one-for-one.

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

Harbormaster returned this revision to the author for changes because remote builds failed.Feb 12 2019, 9:39 PM
Harbormaster failed remote builds in B21964: Diff 48120!

Depends on D20149. In the cases where the second parameter is a variable, there's some possibility that it's sometimes null in some tricky case, although I've tried to look through all the callsites and catch any of this. If this is the case, we'll get an explicit exception shortly ("second parameter to replace must not be null").

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