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
Unknown Object (File)
Tue, Jan 28, 5:15 PM
Unknown Object (File)
Sun, Jan 26, 7:29 PM
Unknown Object (File)
Sun, Jan 26, 7:29 PM
Unknown Object (File)
Sun, Jan 26, 7:28 PM
Unknown Object (File)
Sat, Jan 18, 4:44 AM
Unknown Object (File)
Sun, Jan 12, 6:47 AM
Unknown Object (File)
Dec 28 2024, 3:19 AM
Unknown Object (File)
Dec 25 2024, 1:56 AM
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.