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
F13061311: D18969.diff
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)
Thu, Mar 28, 3:05 PM
Unknown Object (File)
Sat, Mar 23, 4:59 AM
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
Lint
Lint Not Applicable
Unit
Tests Not Applicable