Bug 31138: Don't ignore DataTables errors and raise them to the end-user
authorJonathan Druart <jonathan.druart@bugs.koha-community.org>
Tue, 12 Jul 2022 08:09:25 +0000 (10:09 +0200)
committerTomas Cohen Arazi <tomascohen@theke.io>
Fri, 22 Jul 2022 17:07:13 +0000 (14:07 -0300)
commit76b984bb6ccee55ba1ddfdb5b08c6ee2e75eaa37
treea02db8379b4e807c91f237bec72a86b015fe6fa6
parent5ec4dced13a57ea4e1cd443a1898ee5e3e914f43
Bug 31138: Don't ignore DataTables errors and raise them to the end-user

DataTables is not raising error to the end user, it's only displaying a generic message in the console.
This leads to lot of questions on IRC or the mailing list ("the table is empty").
We can do better and display what went wrong.

We could also imagine add a link to the wiki for the more common problems (won't be implemented here however).

Test plan:
Revert "Bug 31104: Put each link on a separate header" and search for a
patron using a very long string
You should get an alert with the reponse of the server.
In my case:
"""
Something went wrong when loading the table.
414: Request-URI Too Long
"""

You can also make the REST API returns a 500 (bad specs or simply with a
die statement).

Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>
Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com>
Signed-off-by: Tomas Cohen Arazi <tomascohen@theke.io>
koha-tmpl/intranet-tmpl/prog/js/datatables.js