Page MenuHomePhabricator

Fix a couple more "URI->alter()" callsites in paging code
ClosedPublicDraft

Authored by epriestley on Feb 16 2019, 3:27 PM.
Tags
None
Referenced Files
F15463405: D20186.diff
Tue, Apr 1, 11:07 PM
F15433908: D20186.diff
Tue, Mar 25, 1:08 AM
F15425377: D20186.id.diff
Sun, Mar 23, 4:25 AM
Unknown Object (File)
Mar 3 2025, 9:21 PM
Unknown Object (File)
Mar 3 2025, 1:59 AM
Unknown Object (File)
Feb 4 2025, 12:35 PM
Unknown Object (File)
Jan 28 2025, 5:15 PM
Unknown Object (File)
Jan 26 2025, 7:29 PM
Subscribers
None

Details

Summary

grep had a hard time finding these.

Test Plan

Will just hotfix this since I'm still reasonably in the deploy window, this currently fatals: https://secure.phabricator.com/search/query/_dgatshiRBSy/#R

Diff Detail

Repository
rP Phabricator
Branch
pager1
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 22040
Build 30108: Run Core Tests
Build 30107: arc lint + arc unit

Unit TestsFailed

TimeTest
50 msDifferentialParseRenderTestCase::Unknown Unit Message ("")
Assertion failed, expected values to be equal (at DifferentialParseRenderTestCase.php:56): generated.diff.one.unshielded Expected vs Actual Output Diff --- Old Value
1 msAlmanacNamesTestCase::Unknown Unit Message ("")
30 assertions passed.
0 msAlmanacServiceTypeTestCase::Unknown Unit Message ("")
1 assertion passed.
0 msAphrontHTTPSinkTestCase::Unknown Unit Message ("")
6 assertions passed.
0 msAphrontHTTPSinkTestCase::Unknown Unit Message ("")
4 assertions passed.
View Full Test Results (1 Failed · 374 Passed)

Event Timeline

This revision was not accepted when it landed; it landed in state Draft.Feb 16 2019, 3:28 PM
This revision was automatically updated to reflect the committed changes.