Bugzilla – Attachment 11053 Details for
Bug 8484
Add ability to choose if lost items are removed from a borrowers record.
Home
|
New
|
Browse
|
Search
|
[?]
|
Reports
|
Help
|
New Account
|
Log In
[x]
|
Forgot Password
Login:
[x]
[patch]
Bug 8484 - Add ability to choose of lost items are removed from a borrowers record.
Bug-8484---Add-ability-to-choose-of-lost-items-are.patch (text/plain), 5.05 KB, created by
Kyle M Hall (khall)
on 2012-07-20 15:57:22 UTC
(
hide
)
Description:
Bug 8484 - Add ability to choose of lost items are removed from a borrowers record.
Filename:
MIME Type:
Creator:
Kyle M Hall (khall)
Created:
2012-07-20 15:57:22 UTC
Size:
5.05 KB
patch
obsolete
>From a724c9378490d1bd9f549d2aa08ca1a5ffdabdbb Mon Sep 17 00:00:00 2001 >From: Kyle M Hall <kyle@bywatersolutions.com> >Date: Fri, 20 Jul 2012 11:43:34 -0400 >Subject: [PATCH] Bug 8484 - Add ability to choose of lost items are removed from a borrowers record. > >Adds the system preference ReturnLostItems to control whether setting >an item as lost will remove the item from the borrower's record. > >Note, this system preference does not affect misc/cronjobs/longoverdue.pl >which has its own command line switch to choose this behavior. >--- > catalogue/updateitem.pl | 2 +- > installer/data/mysql/sysprefs.sql | 1 + > installer/data/mysql/updatedatabase.pl | 7 +++++++ > .../en/modules/admin/preferences/circulation.pref | 8 ++++++++ > tools/batchMod.pl | 2 +- > 5 files changed, 18 insertions(+), 2 deletions(-) > >diff --git a/catalogue/updateitem.pl b/catalogue/updateitem.pl >index abc809d..6d317b0 100755 >--- a/catalogue/updateitem.pl >+++ b/catalogue/updateitem.pl >@@ -74,6 +74,6 @@ if (defined $itemnotes) { # i.e., itemnotes parameter passed from form > > ModItem($item_changes, $biblionumber, $itemnumber); > >-LostItem($itemnumber, 'MARK RETURNED', 'CHARGE FEE') if $itemlost; >+LostItem($itemnumber, C4::Context->preference("ReturnLostItems"), 'CHARGE FEE') if $itemlost; > > print $cgi->redirect("moredetail.pl?biblionumber=$biblionumber&itemnumber=$itemnumber#item$itemnumber"); >diff --git a/installer/data/mysql/sysprefs.sql b/installer/data/mysql/sysprefs.sql >index 7009155..ad0700c 100644 >--- a/installer/data/mysql/sysprefs.sql >+++ b/installer/data/mysql/sysprefs.sql >@@ -375,3 +375,4 @@ INSERT INTO systempreferences (variable,value,options,explanation,type) VALUES ( > INSERT INTO systempreferences (variable,value,explanation,type) VALUES('EnableBorrowerFiles','0','If enabled, allows librarians to upload and attach arbitrary files to a borrower record.','YesNo'); > INSERT INTO systempreferences (variable,value,explanation,options,type) VALUES('UpdateTotalIssuesOnCirc','0','Whether to update the totalissues field in the biblio on each circ.',NULL,'YesNo'); > INSERT INTO systempreferences (variable,value,explanation,options,type) VALUES('IntranetSlipPrinterJS','','Use this JavaScript for printing slips. Define at least function printThenClose(). For use e.g. with Firefox PlugIn jsPrintSetup, see http://jsprintsetup.mozdev.org/','','Free'); >+INSERT INTO systempreferences (variable,value,explanation,options,type) VALUES('ReturnLostItems', '1', 'If enabled, items marked as lost via the librarian intranet will be automatically returned. Note, the cronjob for marking items as lost has its own command line switch and is not affected by this system preference.','', 'YesNo'); >diff --git a/installer/data/mysql/updatedatabase.pl b/installer/data/mysql/updatedatabase.pl >index bb96644..74705f4 100755 >--- a/installer/data/mysql/updatedatabase.pl >+++ b/installer/data/mysql/updatedatabase.pl >@@ -5536,6 +5536,13 @@ if ( C4::Context->preference("Version") < TransformToNum($DBversion) ) { > SetVersion($DBversion); > } > >+$DBversion ="3.09.00.XXX"; >+if ( C4::Context->preference("Version") < TransformToNum($DBversion) ) { >+ $dbh->do("INSERT INTO systempreferences (variable,value,options,explanation,type) VALUES('ReturnLostItems', '1', '','If enabled, items marked as lost via the librarian intranet will be automatically returned. Note, the cronjob for marking items as lost has its own command line switch and is not affected by this system preference.', 'YesNo')"); >+ print "Upgrade to $DBversion done (Add system preference ReturnLostItems)\n"; >+ SetVersion($DBversion); >+} >+ > =head1 FUNCTIONS > > =head2 TableExists($table) >diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/admin/preferences/circulation.pref b/koha-tmpl/intranet-tmpl/prog/en/modules/admin/preferences/circulation.pref >index e07fc0d..5ea1d49 100644 >--- a/koha-tmpl/intranet-tmpl/prog/en/modules/admin/preferences/circulation.pref >+++ b/koha-tmpl/intranet-tmpl/prog/en/modules/admin/preferences/circulation.pref >@@ -438,3 +438,11 @@ Circulation: > - and this password > - pref: AutoSelfCheckPass > - . >+ Lost Items Policy: >+ - >+ - pref: ReturnLostItems >+ type: choice >+ choices: >+ yes: "Check in" >+ no: "Don't check in" >+ - items marked as lost that are checked out to a patron. >diff --git a/tools/batchMod.pl b/tools/batchMod.pl >index 3ba93a8..122d83f 100755 >--- a/tools/batchMod.pl >+++ b/tools/batchMod.pl >@@ -188,7 +188,7 @@ if ($op eq "action") { > UpdateMarcWith( $marcitem, $localmarcitem ); > eval{ > if ( my $item = ModItemFromMarc( $localmarcitem, $itemdata->{biblionumber}, $itemnumber ) ) { >- LostItem($itemnumber, 'MARK RETURNED', 'CHARGE FEE') if $item->{itemlost}; >+ LostItem($itemnumber, C4::Context->preference("ReturnLostItems"), 'CHARGE FEE') if $item->{itemlost}; > } > }; > } >-- >1.7.2.5
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 8484
:
11053
|
15147
|
15169
|
15170
|
17619
|
17620
|
20702
|
20703
|
21083