LP#1858114: Present proper input widget for aggregate parameters
authorMike Rylander <mrylander@gmail.com>
Thu, 13 Aug 2020 17:05:19 +0000 (13:05 -0400)
committerMike Rylander <mrylander@gmail.com>
Thu, 13 Aug 2020 17:05:19 +0000 (13:05 -0400)
commit1c5426fc43907d3df2c3ac6eea36ac7f097ac200
treeadc4d6585fb07d20f0728aa967c3b45c64d14ed7
parent30dcf71b98f4b48fb1e05c7d3c74afa3b31a0ebc
LP#1858114: Present proper input widget for aggregate parameters

There's currently logic to override the filter widget for all columns
that have a selector defined, such as the ID column of many tables. When
we transform the column with an aggregate transform, though, we don't want
to pick a particular selector-labeled value.

While I can imagine using a min or max aggregate to get a representative
value for a group, when other dependent filters would make the value
stable or where the value in particular doesn't matter but isn't needed
in a count (say), all the other aggregate transforms that are currently
supported are specifically numeric (sum, average, count, and
count_distinct). Because of that, and the most likely use case of
matching a count to "0" or "1", we should only present the "remote
object widget" for non-aggregates, and just use the baseline text string
input here.

Signed-off-by: Mike Rylander <mrylander@gmail.com>
Open-ILS/web/reports/oils_rpt_param_editor.js