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

David Cook <[email protected]> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|Needs Signoff               |Failed QA

--- Comment #31 from David Cook <[email protected]> ---
But... we do need a solution for handling date range searching throughout Koha
with Elastic! 

I'm going to use my workaround for now locally, but we do need to keep working
on this.

--

Note that this is a "Failed QA" for a few reasons.

1. "Bug 30745 : Fix pagination error" isn't how we've chosen to fix this in
Koha.

Since it's a stateless operation, it should be a GET and not a POST. That's
reported on bug 36931

I think a number of these patches probably should be on separate bugs... 

2. It doesn't add any unit tests.

(And actually now that I look at
t/db_dependent/Koha/SearchEngine/Elasticsearch/QueryBuilder.t it looks like we
already have some interesting date handling for ranges. Something to look at
further there...)

-- 
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