See http://fab.wmflabs.org/T88. While this issue is on MW's side, these links are kind of ugly and have more readable alternate forms now. Update them to use proper modern forms.
Details
Details
- Reviewers
btrahan - Commits
- Restricted Diffusion Commit
rPb64415c9b138: Use bracket-free, human-readable query URIs everywhere
See inlines.
Diff Detail
Diff Detail
- Repository
- rP Phabricator
- Branch
- blinks
- Lint
Lint Passed - Unit
Tests Passed - Build Status
Buildable 345 Build 345: [Placeholder Plan] Wait for 30 Seconds
Event Timeline
| src/applications/differential/event/DifferentialActionMenuEventListener.php | ||
|---|---|---|
| 36–40 | Clicked the link on my profile. We don't need to make this a form anymore, I think that was a leftover from a very long time ago before ApplicationSearch. | |
| src/applications/home/controller/PhabricatorHomeMainController.php | ||
| 151–154 | This was buggy anyway on installs with more than "open" status option, since statuses[]=a,b is not a valid construction. | |
| 199 | This should be urisprintf(), I'll fix that locally... | |
| 200 | Same as above, buggy with more than one open status. | |
| src/applications/maniphest/event/ManiphestActionMenuEventListener.php | ||
| 47 | This has no callsite, and wasn't updated for configurable statuses. | |