Bug 34533: jsdiff library missing from guided reports page
authorOwen Leonard <oleonard@myacpl.org>
Tue, 15 Aug 2023 08:38:20 +0000 (08:38 +0000)
committerTomas Cohen Arazi <tomascohen@theke.io>
Tue, 29 Aug 2023 17:37:13 +0000 (14:37 -0300)
commitf2be2cedc5e721e715a98b18299c55a0fdfa61d1
treede62dcde6472962a29cb4cf6667171c9073de386
parentfb48155cc4bdb17935c7d3cd4bdd4e74695d3948
Bug 34533: jsdiff library missing from guided reports page

This patch restores the inclusion of the jsdiff JS library on the guided
reports page when displaying saved SQL reports. This fixes a JS error in
the browser console when the user tries to use our automatic update on
SQL reports which use an older syntax.

The patch also corrects a problem with svc/convert_report which was
causing an error when trying to display the diff information.

To test, apply the patch and restart services.

- Go to Reports -> Create from SQL.
- Add a report which is based on an older db structure. The example I
  found was:

  SELECT biblionumber,
  ExtractValue(marcxml,'//datafield/@tag/text()[substring(.,1,1) = "5"]') as 'notes'
  FROM biblioitems
  HAVING notes <> ''

- Save the report and go to the list of saved reports. There should be
  an "Update" column in the table, with a message about your newly-added
  report:

  "This report seems obsolete, it uses biblioitems.marcxml field."

- Clicking the "Update SQL" button should trigger a modal window that
  shows the before an after versions of the report. There should be
  no error in the browser console. Unfortunately the diff will not
  display because of a separate bug in svc/convert_report

Signed.off-by: Jessie Zairo <jzairo@bywatersolutions.com>
Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>
Signed-off-by: Tomas Cohen Arazi <tomascohen@theke.io>
koha-tmpl/intranet-tmpl/prog/en/modules/reports/guided_reports_start.tt
svc/convert_report