]> git.evergreen-ils.org Git - Evergreen.git/commit
LP#1796945 Match new path_label/alias standard
authorDan Wells <dbw2@calvin.edu>
Thu, 15 Aug 2019 14:20:17 +0000 (10:20 -0400)
committerJason Boyer <jboyer@library.in.gov>
Thu, 22 Aug 2019 18:20:36 +0000 (14:20 -0400)
commit5357f139e97d3d8f4d8f9c80bb9e315339d1e2d7
tree77faaaa364fa4f1144918f530be569135cd4ca54
parentcc444ec0f6a201ccda27d58c56a0bb7e07bd9261
LP#1796945 Match new path_label/alias standard

The old reporter used '::' as a path separator in some labels, while the
new version uses '->' in its place.  This would normally be just a
curiosity, except that the hashes used as join aliases are generated
from these path labels.  The end result is that while a report can be
cloned, if you add a field anywhere other than the top level, it
references a join alias which does not exist, and the report breaks.

Now, the method by which report templates are upgraded from v4 to v5
does not happen all in one pass, but rather a first pass is made
populate the interface, then the rest is done when the user saves the
clone.  Because of this, it actually seems adequate to only fix the
label, then allow the other steps to fill in the other bits upon saving
of the report.

Thank you to J. Boyer for helping track down the culprit.

Signed-off-by: Dan Wells <dbw2@calvin.edu>
Signed-off-by: Jason Boyer <jboyer@library.in.gov>
Open-ILS/web/js/ui/default/staff/reporter/template/app.js