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

Martin Renvoize (ashimema) <[email protected]> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|Patch doesn't apply         |In Discussion

--- Comment #7 from Martin Renvoize (ashimema) 
<[email protected]> ---
Moving this to In Discussion rather than leaving it as 'Patch doesn't apply': a
straight rebase wouldn't get us there, this needs reworking against how main
looks now. What I found:

- Main still has no flatpickr in the DataTables column filters: every
searchable column gets a plain text input, and a column with type 'date' only
has the typed value converted with $date_to_rfc3339 when the query is built.
- The second patch (allowyear) was an unfinished experiment, as its own commit
message says. It also has a bug: new RegExp('^\d{4}$') in a JS string loses the
backslash, so it never matches a four digit year, and it leaves a console.log
behind.
- The first patch only added calendar.inc to patron-search.inc. Since bug
33568, type 'date' is used by more DataTables on the staff side (holdshistory,
the items table in catalogue detail, batch_modify_holds, transferstoreceive,
pendingbookings as well as patron search), and apply_flatpickr lives in
calendar.inc, so each of those would need it loaded. The patch also runs
apply_flatpickr over every .flatpickr on the page rather than only the filter
inputs.
- calendar.inc's flatpickr defaults now add an altInput and a maskito date
input mask. That mask is in conflict with the original wish to filter on a year
only (or a month/day), so that needs an explicit decision.

I'd still like to do this, but it isn't a priority right now and it would need
rebuilding rather than rebasing the 2023 patches. If someone wants to pick it
up (or has an opinion on full dates vs partial dates in these filters), please
do say.

-- 
You are receiving this mail because:
You are watching all bug changes.
_______________________________________________
Koha-bugs mailing list -- [email protected]
To unsubscribe send an email to [email protected]
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/

Reply via email to