Page MenuHomePhabricator

Fix an export bug where queries specified in the URI ("?param=value") were ignored when filtering the result set
ClosedPublic

Authored by epriestley on Jan 30 2018, 4:06 PM.
Tags
None
Referenced Files
Unknown Object (File)
Wed, Apr 24, 11:21 PM
Unknown Object (File)
Fri, Apr 19, 7:13 PM
Unknown Object (File)
Tue, Apr 16, 10:01 AM
Unknown Object (File)
Thu, Apr 11, 7:52 AM
Unknown Object (File)
Wed, Apr 10, 5:23 AM
Unknown Object (File)
Mon, Apr 8, 11:39 AM
Unknown Object (File)
Sun, Apr 7, 6:34 AM
Unknown Object (File)
Mar 28 2024, 3:05 PM
Subscribers
None

Details

Summary

Depends on D18968. Ref T13049. Currently, if you visit /query/?param=value, there is no queryKey for the page but we build a query later on.

Right now, we incorrectly link to /query/all/export/ in this case (and export too many results), but we should actually link to /query/<constructed query key>/export/ to export only the desired/previewed results.

Swap the logic around a little bit so we look at the query we're actually executing, not the original URI, to figure out the query key we should use when building the link.

Test Plan

Visited a /?param=value page, exported data, got a subset of the full data instead of everything.

Diff Detail

Repository
rP Phabricator
Branch
export5
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 19279
Build 26051: Run Core Tests
Build 26050: arc lint + arc unit