Bug 38604 - ILL: Pre-populate status filter if only one backend in use
Summary: ILL: Pre-populate status filter if only one backend in use
Status: CLOSED INVALID
Alias: None
Product: Koha
Classification: Unclassified
Component: ILL (show other bugs)
Version: Main
Hardware: All All
: P5 - low enhancement
Assignee: Bugs List
QA Contact: Testopia
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2024-12-03 13:29 UTC by Magnus Enger
Modified: 2024-12-12 13:41 UTC (History)
2 users (show)

See Also:
GIT URL:
Initiative type: ---
Sponsorship status: ---
Comma delimited list of Sponsors:
Crowdfunding goal: 0
Patch complexity: ---
Documentation contact:
Documentation submission:
Text to go in the release notes:
Version(s) released in:
Circulation function:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Magnus Enger 2024-12-03 13:29:46 UTC
To populate the "Status" filter dropdown on the left side of the main ILL request table, you have to select the backend first. This seems unnecessary when there is only one backend in use. If this is the case, we should be able to populate the Status dropdown without any interaction from the user.
Comment 1 Pedro Amorim 2024-12-11 11:02:56 UTC
(In reply to Magnus Enger from comment #0)
> To populate the "Status" filter dropdown on the left side of the main ILL
> request table, you have to select the backend first. This seems unnecessary
> when there is only one backend in use. If this is the case, we should be
> able to populate the Status dropdown without any interaction from the user.

Hi Magnus can you please provide more information? Is this on main?

I've just tested this with 4 backends installed (+Standard). The 'status' dropdown is populated with all existing statuses, from all backends, by default.
The backend dropdown is pre-populated to 'All backends'.
This should mean that if you only have one backend you should be able to filter by any existing status from the get go, without having to select the specific backend.
Comment 2 Katrin Fischer 2024-12-11 11:10:13 UTC
I think Magnus means that it offer "status" values without having to select the backend if you only have one, as this means extra clicks. 

I am not sure how possible this is going to be with Standard now always being "in the mix". For our use case most libraries would probably only use 1 backend, but ours and not Standard and might also like to avoid the extra clicks. The list and filters are used a lot in day to day.
Comment 3 Pedro Amorim 2024-12-11 11:12:00 UTC
(In reply to Katrin Fischer from comment #2)
> I think Magnus means that it offer "status" values without having to select
> the backend if you only have one, as this means extra clicks. 

This is what I mean, according to my testing, you don't have to do any extra clicks, as the status dropdown is pre-filled with all statuses, from all backends, by default.
Comment 4 Katrin Fischer 2024-12-11 11:12:53 UTC
(In reply to Pedro Amorim from comment #3)
> (In reply to Katrin Fischer from comment #2)
> > I think Magnus means that it offer "status" values without having to select
> > the backend if you only have one, as this means extra clicks. 
> 
> This is what I mean, according to my testing, you don't have to do any extra
> clicks, as the status dropdown is pre-filled with all statuses, from all
> backends, by default.

Ah! Maybe some version thing? I'll let Magnus check.
Comment 5 Magnus Enger 2024-12-11 14:20:05 UTC
Looks like the behaviour I described is relevant for 23.11.10, at least. I have to choose a backend to enable/populate the status dropdown. 

On main, "All backends" are shown as default, and the status dropdown is populated/active.
Comment 6 Pedro Amorim 2024-12-12 13:41:36 UTC
(In reply to Magnus Enger from comment #5)
> Looks like the behaviour I described is relevant for 23.11.10, at least. I
> have to choose a backend to enable/populate the status dropdown. 
> 
> On main, "All backends" are shown as default, and the status dropdown is
> populated/active.

Yes, I believe this was part of bug 34431 (pushed to 24.05.x).
Closing this.