[Koha-bugs] [Bug 33490] Agreements - Filter by expired results in error

bugzilla-daemon at bugs.koha-community.org bugzilla-daemon at bugs.koha-community.org
Tue Apr 25 14:47:26 CEST 2023


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

--- Comment #40 from Jonathan Druart <jonathan.druart+koha at gmail.com> ---
Created attachment 150197
  -->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=150197&action=edit
Bug 33490: Improve max date handling

It fixes:
1. /agreements?by_expired=true do no longer explode
2. Fill date with today's date when we setup the component and when we
hit "filter"

However there are a couple of things totally wrong:
1. We should not need is_fp_disabled, we could be able to use
  :disabled="!filters.by_expired"
in the template, but it does not work
2. We should not need to deal with filters.max_expiration_date the "old
JS way", but should use a Vue computed value instead. I've failed at
that

I think those 2 problems have the same root, the way we deal with
filters in data() is wrong, and we may be loosing the ref at some point.

Signed-off-by: Jonathan Druart <jonathan.druart at bugs.koha-community.org>

-- 
You are receiving this mail because:
You are watching all bug changes.


More information about the Koha-bugs mailing list