LP#1721807: fix webstaff report templates that have might_have and has_many joins user/csharp/lp1721807_webstaff_reporter_joins
authorGalen Charlton <gmc@equinoxinitiative.org>
Tue, 6 Mar 2018 23:00:23 +0000 (18:00 -0500)
committerChris Sharp <csharp@georgialibraries.org>
Wed, 14 Mar 2018 18:48:33 +0000 (14:48 -0400)
commitd7d8e108c28eedf3a1fb13ab5cbaa601bbbae7cc
tree57395f987e27fa628a3846e5c355c4dc38a8643c
parent73bc91dc58de13f7c77242f0ce36f8f8ba93a7df
LP#1721807: fix webstaff report templates that have might_have and has_many joins

This patch fixes a bug where using a virtual field in the web staff
report template editor to join another table would result in a syntax
error in the generated SQL.

To test
-------
[1] Apply the patch and verify that the regression test in the previous
    commit passes.
[2] Create new report template in the web staff client that include
    virtual fields and joins and verify that they work. One example
    would be a report that uses ILS User as the base source and joins
    in the patron notes virtual field.
[3] If available, clone a report template originally created in the web
    staff client that had previously failed, then save it. The new
    report should work.

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