Bugzilla – Attachment 109565 Details for
Bug 25560
Define itemtype specific rules in the UpdateNotForLoanStatusOnCheckin system preference
Home
|
New
|
Browse
|
Search
|
[?]
|
Reports
|
Help
|
New Account
|
Log In
[x]
|
Forgot Password
Login:
[x]
[patch]
Bug 25560: Exclude itemtypes from UpdateNotForLoanStatusOnCheckin
Bug-25560-Exclude-itemtypes-from-UpdateNotForLoanS.patch (text/plain), 4.44 KB, created by
Alex Buckley
on 2020-09-02 22:39:05 UTC
(
hide
)
Description:
Bug 25560: Exclude itemtypes from UpdateNotForLoanStatusOnCheckin
Filename:
MIME Type:
Creator:
Alex Buckley
Created:
2020-09-02 22:39:05 UTC
Size:
4.44 KB
patch
obsolete
>From 5f3eb1ba73e095209f9e1175e49a3ad403892368 Mon Sep 17 00:00:00 2001 >From: Alex Buckley <alexbuckley@catalyst.net.nz> >Date: Thu, 21 May 2020 23:56:15 +0000 >Subject: [PATCH] Bug 25560: Exclude itemtypes from > UpdateNotForLoanStatusOnCheckin > >Adds a boolean flag itemtypes.updatenotforloanstatusoncheckin, enabled by default. >When disabled (if the UpdateNoForLoanStatusOnCheckin syspref is >configured) when an item of the type if checked in the notforloan status >does not update. > >This enhancement respects the 'item-level_itypes' syspref, so will look >at the authoritative item type when determining to update the notforloan >status. > >Test plan: >1. Paste into UpdateNotForLoanStatusOnCheckin syspref > 0 : 1 >2. Issue and return an item and observe the item's notforloan status updates >3. Apply patch & update database >cd installer/data/mysql >sudo koha-shell <instancename> >./updatedatabase.pl >4. Restart plack >5. Go to BK (Book) item type page (Administration > Item types > BK). >6. Observe the 'Update not for loan status on check-in' checkbox is >selected by default. Untick it. >7. Make sure the 'item-level_itypes' syspref is set to 'specific item' >8. Issue and return a BK item (item level itype='BK') and observe item's >notforloan status does not update >9. Issue and return a CR (Continuing Resources) item (item level >itype='CR') and observe the item's notforloan status updates > >10. Change 'item-level_itypes' syspref to 'bibliographic record' >11. Issue and return an item (with biblio-level itemtype='BK' & >item-level itype='CR') and observe the item's notforloan status does not >update >12. Issue and return an item (with biblio-level itemtype='CR' & >item-level itype='BK') and observe the item's notforloan status >updates > >Sponsored-By: Waikato Institute of Technology, NZ >--- > C4/Circulation.pm | 4 ++++ > admin/itemtypes.pl | 3 +++ > 2 files changed, 7 insertions(+) > >diff --git a/C4/Circulation.pm b/C4/Circulation.pm >index 3571051e84f..b3e9d60182d 100644 >--- a/C4/Circulation.pm >+++ b/C4/Circulation.pm >@@ -1943,6 +1943,10 @@ sub AddReturn { > } > else { > foreach my $key ( keys %$rules ) { >+ # If item type in use has a disabled updatenotforloanstatusoncheckin flag then don't update notforloan >+ my $updatenotforloanstatusoncheckin = C4::Context->preference("item-level_itypes") ? Koha::ItemTypes->find($item->itype)->updatenotforloanstatusoncheckin : Koha::ItemTypes->find($item->biblioitem->itemtype)->updatenotforloanstatusoncheckin; >+ last if !$updatenotforloanstatusoncheckin; >+ > if ( $item->notforloan eq $key ) { > $messages->{'NotForLoanStatusUpdated'} = { from => $item->notforloan, to => $rules->{$key} }; > $item->notforloan($rules->{$key})->store({ log_action => 0 }); >diff --git a/admin/itemtypes.pl b/admin/itemtypes.pl >index ae98300cfc2..b6bd166d46d 100755 >--- a/admin/itemtypes.pl >+++ b/admin/itemtypes.pl >@@ -110,6 +110,7 @@ if ( $op eq 'add_form' ) { > my $checkinmsg = $input->param('checkinmsg'); > my $checkinmsgtype = $input->param('checkinmsgtype'); > my $hideinopac = $input->param('hideinopac') // 0; >+ my $updatenotforloanstatusoncheckin = $input->param('updatenotforloanstatusoncheckin') // 0; > my $searchcategory = $input->param('searchcategory'); > my $rentalcharge_daily_calendar = $input->param('rentalcharge_daily_calendar') // 0; > my $rentalcharge_hourly_calendar = $input->param('rentalcharge_hourly_calendar') // 0; >@@ -129,6 +130,7 @@ if ( $op eq 'add_form' ) { > $itemtype->checkinmsgtype($checkinmsgtype); > $itemtype->sip_media_type($sip_media_type); > $itemtype->hideinopac($hideinopac); >+ $itemtype->updatenotforloanstatusoncheckin($updatenotforloanstatusoncheckin); > $itemtype->searchcategory($searchcategory); > $itemtype->rentalcharge_daily_calendar($rentalcharge_daily_calendar); > $itemtype->rentalcharge_hourly_calendar($rentalcharge_hourly_calendar); >@@ -161,6 +163,7 @@ if ( $op eq 'add_form' ) { > checkinmsgtype => $checkinmsgtype, > sip_media_type => $sip_media_type, > hideinopac => $hideinopac, >+ updatenotforloanstatusoncheckin => $updatenotforloanstatusoncheckin, > searchcategory => $searchcategory, > rentalcharge_daily_calendar => $rentalcharge_daily_calendar, > rentalcharge_hourly_calendar => $rentalcharge_hourly_calendar, >-- >2.11.0
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 25560
:
105236
|
105237
|
105238
|
105239
|
105240
|
105241
|
105242
|
105243
|
105244
|
105245
|
109562
|
109563
|
109564
|
109565
|
109566
|
109567
|
112492
|
112887
|
113161
|
113212
|
113213
|
113214
|
113215
|
113216
|
113217
|
131816
|
131817
|
131818
|
131819
|
131820
|
135742
|
135743
|
136079
|
136080
|
136081
|
136082
|
136083
|
136084
|
138494
|
138495
|
138496
|
138497
|
138498
|
138499
|
146556
|
146557
|
146558
|
146559
|
146993
|
148679
|
148680
|
148681
|
148682
|
148683