https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=38255

--- Comment #6 from Jonathan Druart <[email protected]> ---
We are removing the occurrences of instantiations of dataTable(), and replace
them with our own kohaTable.

To make this possible we:
1. Set the default ajax options (_dt_default_ajax from datatables.js) only if
an 'ajax' option is passed already
2. Simply replace 'dataTable' with 'kohaTable', and remove the merge with
dataTablesDefaults (we do it already in datatables.js)
3. Introduce kohaTable at the OPAC (a shorter version of what we have
staff-side as we provide less features)
4. Deal with the svc script, see the "Exceptions" patch: we pass a flag
"bKohaAjaxSVC" to tell to not inject the default ajax options even if we using
server-side processing (should be remove in the future, and not be used by new
patches).

Test plan:
Test some of the tables modified by this patchset and search for
errors/regressions

QA note: this patch introduces way more changes than needed, I wanted to indent
correctly the kohaTable block to catch potential syntax error. Happened when I
forgot to remove the extra parenthesis:
-            }));
+            });

-- 
You are receiving this mail because:
You are watching all bug changes.
_______________________________________________
Koha-bugs mailing list
[email protected]
https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/

Reply via email to