[Koha-bugs] [Bug 25279] Make the cities list use the API
bugzilla-daemon at bugs.koha-community.org
bugzilla-daemon at bugs.koha-community.org
Tue May 5 20:34:18 CEST 2020
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=25279
--- Comment #20 from Tomás Cohen Arazi <tomascohen at gmail.com> ---
(In reply to Jonathan Druart from comment #19)
> (In reply to Tomás Cohen Arazi from comment #18)
> > (In reply to Jonathan Druart from comment #17)
> > > 1. Small UI changes (not advertised)
> > >
> > > Before: https://snipboard.io/cRdEWA.jpg
> > > After : https://snipboard.io/qep6rS.jpg
> > >
> > > a) No more pagination (should be restored imo)
> >
> > Pagination works, it just doesn't have pages.
>
> Yes, I did not say it did not work. The pages should be displayed imo.
I will dig a bit about it and fix it.
> > > d) Table is displayed even if empty
> >
> > I don't think this is a problem. The only solution is to count them in the
> > controller .pl before rendering the page. Would you agree with such approach?
>
> Maybe something DT-side to handle that?
The only thing I found DataTables provided was a way to put a string on the
rows place, stating there are no rows. But maybe we can overload the rendering
code so it hides the table. Will take a look as well.
My next submission will just use a count for now, so there's no behaviour
change and this is not blocked until we find a better solution.
> > > e) Default number of entries was 10, not 20 (better as it)
> >
> > Will fix, this is configurable.
>
> No, 20 is correct.
Ok!
> > > 3. Why the indentation changes in cities.json?
> >
> > I think my editor did that when I intended to re-indent the portion I added.
> > Is it a problem?
>
> No, I wanted to know if the existing syntax was a problem for something.
> We should prevent such unnecessary changes (no blocker here).
To be honest, I always hated the original indentation, and the cities one has
diverted from newer ones when it comes to arrays. (single line vs. multi line).
> > > 4. We reached a point where we don't have XSS issue in our templates, all
> > > variables are correctly escaped.
> > > With this adding more JS code we should continue to enforce the rule, all
> > > the variables must be correctly escaped.
> >
> > You mean data that comes from the API????
>
> I did not investigate it, I don't know where it's best to escape them.
We will take a look, ideally we would do something DT-side, so this doesn't get
borked with calls to URI/HTML encoding methods all over the place.
Thanks Jonathan for your valuable feedback.
--
You are receiving this mail because:
You are watching all bug changes.
More information about the Koha-bugs
mailing list