Created attachment 160899 [details] [review]
Bug 35783: Replace TT plugin's method Biblio::RecallsCount

We can use biblio.recalls.search( completed => 0 ).count instead.

Test plan:
1. Go to the biblio detail view, click on the different entries in the menu
on the left. Confirm that the "Recalls" tab always has the correct number
of current recalls display in the parenthesis.

2. Run a search and confirm that the number of current recalls are still displayed
for each result.

I have had a go at testing.

For me, there were no changes on record page - things worked as expected.

However, it didn't show in the search results (it just showed Recalls (0)). It did work before the patches were applied.

Testing notes (using KTD):

1. Apply the patch, run updatedatabase, restart_all

2. Enable the UseRecalls system preference

3. Set the circulation rules. For ALL ALL:
   - Recalls allowed (total): 5
   - Recalls per record (count): 5
   - On shelf recalls allowed: If any available
   - Recall due date interval (day): 5
   - Recall overdue fine amount: 1
   - Recall pickup period (day): 2

4. Checkout two different records to different patrons
   - Programming Perl (262): Mary Burton
   - Perl template toolkit (255): Bobbie Middleton

5. Use the Koha user to place recall for the two records in the OPAC.

6. Look at the record in the staff interface - the tab for recalls should show the correct number of recalls (Recalls (1)).

7. Do a search that should return the record - it should show Recalls (1) as well.

8. Repeat steps 5-6 with a different patron (I used Henry). The number should change to match the number of recalls (Recalls (2)) in the record and search results.

Created attachment 160931 [details] [review]
Bug 35783: Replace TT plugin's method Biblio::RecallsCount

We can use biblio.recalls.search( completed => 0 ).count instead.

Test plan:
1. Go to the biblio detail view, click on the different entries in the menu
on the left. Confirm that the "Recalls" tab always has the correct number
of current recalls display in the parenthesis.

2. Run a search and confirm that the number of current recalls are still displayed
for each result.

Oops sorry, inverted one flag. Should work now!

Created attachment 160934 [details] [review]
Bug 35783: Replace TT plugin's method Biblio::RecallsCount

We can use biblio.recalls.search( completed => 0 ).count instead.

Test plan:
1. Go to the biblio detail view, click on the different entries in the menu
on the left. Confirm that the "Recalls" tab always has the correct number
of current recalls display in the parenthesis.

2. Run a search and confirm that the number of current recalls are still displayed
for each result.

Signed-off-by: David Nind <david@davidnind.com>

Created attachment 160963 [details] [review]
Bug 35783: Replace TT plugin's method Biblio::RecallsCount

We can use biblio.recalls.search( completed => 0 ).count instead.

Test plan:
1. Go to the biblio detail view, click on the different entries in the menu
on the left. Confirm that the "Recalls" tab always has the correct number
of current recalls display in the parenthesis.

2. Run a search and confirm that the number of current recalls are still displayed
for each result.

Signed-off-by: David Nind <david@davidnind.com>

Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com>

We should probably have a follow-up bug to remove the subroutine in 24.11

(In reply to Kyle M Hall from comment #7)
> We should probably have a follow-up bug to remove the subroutine in 24.11

Yes! Done. Bug 35791.

Pushed for 24.05!

Well done everyone, thank you!

Not backported to 23.11.x

Architecture change, nothing to add/edit in the Koha manual.