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
F14081605: D18969.diff
Fri, Nov 22, 8:47 PM
Unknown Object (File)
Wed, Nov 20, 7:26 PM
Unknown Object (File)
Sun, Nov 17, 4:19 AM
Unknown Object (File)
Fri, Nov 15, 7:00 PM
Unknown Object (File)
Sun, Nov 3, 5:08 PM
Unknown Object (File)
Fri, Oct 25, 4:19 PM
Unknown Object (File)
Oct 14 2024, 5:23 PM
Unknown Object (File)
Oct 2 2024, 10:02 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
Lint
Lint Not Applicable
Unit
Tests Not Applicable