]> git.evergreen-ils.org Git - working/Evergreen.git/commit
LP#1785061: move the filter value munging to the template service user/dbwells/lp1785061_signoff_rebase
authorGalen Charlton <gmc@equinoxinitiative.org>
Wed, 22 May 2019 15:12:37 +0000 (11:12 -0400)
committerDan Wells <dbw2@calvin.edu>
Fri, 24 May 2019 22:46:35 +0000 (18:46 -0400)
commit97bcf0729b8001dd57e5d688775cc49f15695452
treea98289c6a8912e8f057ed1c5eb07a9e0047f8f30
parente2e9b972bc15db64621938d8d47e4722cf4e5161
LP#1785061: move the filter value munging to the template service

This allows the reporter app to stay a bit more focused on
display concerns.

Here's a test plan for the patch series
---------------------------------------
[1] Create a reporter template that has a filter field
    and a filter operator of "In list", "Not in list", "Between",
    or "Not between" and hard-code a value for that filter
    in the template, separating values with a comma.
[2] Try to create a report from that template; note that it
    fails with an error.
[3] Apply the patch and repeat steps 1 and 2. This time, the
    report should succeed.

Note that this fix applies only to hardcoding filter values in the
template; it doesn't change any behavior when attempting to set
a filter value for any of the four operators above at the point of
creating a report.

Signed-off-by: Galen Charlton <gmc@equinoxinitiative.org>
Conflicts:
Open-ILS/web/js/ui/default/staff/reporter/template/app.js

Signed-off-by: Dan Wells <dbw2@calvin.edu>
Open-ILS/web/js/ui/default/staff/reporter/services/template.js
Open-ILS/web/js/ui/default/staff/reporter/template/app.js