Bugzilla – Attachment 165172 Details for
Bug 35394
Correct the message displayed when attempting to checkout an item during it's booking period
Home
|
New
|
Browse
|
Search
|
[?]
|
Reports
|
Help
|
New Account
|
Log In
[x]
|
Forgot Password
Login:
[x]
[patch]
Bug 35394: Correction to booked items logic
Bug-35394-Correction-to-booked-items-logic.patch (text/plain), 2.82 KB, created by
Nick Clemens (kidclamp)
on 2024-04-19 13:24:44 UTC
(
hide
)
Description:
Bug 35394: Correction to booked items logic
Filename:
MIME Type:
Creator:
Nick Clemens (kidclamp)
Created:
2024-04-19 13:24:44 UTC
Size:
2.82 KB
patch
obsolete
>From 54ab8f9e6275454539f49e487d9ac776d18636ce Mon Sep 17 00:00:00 2001 >From: Martin Renvoize <martin.renvoize@ptfs-europe.com> >Date: Fri, 8 Mar 2024 15:25:10 +0000 >Subject: [PATCH] Bug 35394: Correction to booked items logic > >This patch fixes the overly extreme message that was designed to appeared >when someone tried to check out a booked item to the wrong user. >However, the message never appeared due to a logic error within >CanBookBeIssued which this patch also resolved. > >Test plan >1) Make an item bookable >2) Add a booking for the item to patron A from tomorrow >3) In the database, update the booking start_date to today (or wait a > day before the next step) >4) Attempt to check the book out to patron B. > Note that you are displayed with the new message 'The item is booked > for another patron' and you are unable to check the item out. > >Signed-off-by: David Nind <david@davidnind.com> >Signed-off-by: Nick Clemens <nick@bywatersolutions.com> >--- > C4/Circulation.pm | 15 ++++++++------- > .../prog/en/modules/circ/circulation.tt | 2 +- > 2 files changed, 9 insertions(+), 8 deletions(-) > >diff --git a/C4/Circulation.pm b/C4/Circulation.pm >index b454ff241f8..a62a1acf636 100644 >--- a/C4/Circulation.pm >+++ b/C4/Circulation.pm >@@ -1248,17 +1248,18 @@ sub CanBookBeIssued { > } else { > $alerts{'BOOKED'} = $booking; > } >+ } > >- } else { >- >- # Loan falls inside booking >- if ( $now > $booking_start ) { >- $issuingimpossible{'BOOKED_TO_ANOTHER'} = $booking; >+ # Booked to another patron >+ else { >+ # Booking starts before due date, reduce loan? >+ if ( $now < $booking_start && $duedate > $booking_start ) { >+ $needsconfirmation{'BOOKED_TO_ANOTHER'} = $booking; > } > >- # Booking starts before due date, reduce loan? >+ # Loan falls inside booking > else { >- $needsconfirmation{'BOOKED_TO_ANOTHER'} = $booking; >+ $issuingimpossible{'BOOKED_TO_ANOTHER'} = $booking; > } > } > } >diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/circ/circulation.tt b/koha-tmpl/intranet-tmpl/prog/en/modules/circ/circulation.tt >index e94e5554d04..f51c70c1d27 100644 >--- a/koha-tmpl/intranet-tmpl/prog/en/modules/circ/circulation.tt >+++ b/koha-tmpl/intranet-tmpl/prog/en/modules/circ/circulation.tt >@@ -612,7 +612,7 @@ > [% END %] > > [% IF ( BOOKED_TO_ANOTHER ) %] >- <li>!!A booking exists!!</li> >+ <li>The item is booked for another patron</li> > [% END %] > </ul> > </div> <!-- /#circ_impossible --> >-- >2.39.2
You cannot view the attachment while viewing its details because your browser does not support IFRAMEs.
View the attachment on a separate page
.
View Attachment As Diff
View Attachment As Raw
Actions:
View
|
Diff
|
Splinter Review
Attachments on
bug 35394
:
162973
|
162979
|
164998
| 165172