Bugzilla – Attachment 15169 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 if lost items are removed from a borrowers record. - Followup
Bug-8484---Add-ability-to-choose-if-lost-items-are.patch (text/plain), 4.31 KB, created by
Kyle M Hall
on 2013-02-08 15:02:58 UTC
(
hide
)
Description:
Bug 8484 - Add ability to choose if lost items are removed from a borrowers record. - Followup
Filename:
MIME Type:
Creator:
Kyle M Hall
Created:
2013-02-08 15:02:58 UTC
Size:
4.31 KB
patch
obsolete
>From 1eeb118fbd57b962da6b652829d3187590cea75a Mon Sep 17 00:00:00 2001 >From: Kyle M Hall <kyle@bywatersolutions.com> >Date: Fri, 8 Feb 2013 10:02:26 -0500 >Subject: [PATCH] Bug 8484 - Add ability to choose if lost items are removed from a borrowers record. - Followup > >--- > installer/data/mysql/sysprefs.sql | 2 +- > installer/data/mysql/updatedatabase.pl | 2 +- > misc/cronjobs/longoverdue.pl | 5 +---- > 3 files changed, 3 insertions(+), 6 deletions(-) > >diff --git a/installer/data/mysql/sysprefs.sql b/installer/data/mysql/sysprefs.sql >index 45d0c7d..9aecea3 100644 >--- a/installer/data/mysql/sysprefs.sql >+++ b/installer/data/mysql/sysprefs.sql >@@ -411,4 +411,4 @@ INSERT INTO systempreferences (variable,value,options,explanation,type) VALUES ( > INSERT IGNORE INTO systempreferences (variable,value,explanation,options,type) VALUES('IDreamBooksReviews','0','Display book review snippets from IDreamBooks.com','','YesNo'); > INSERT IGNORE INTO systempreferences (variable,value,explanation,options,type) VALUES('IDreamBooksReadometer','0','Display Readometer from IDreamBooks.com','','YesNo'); > INSERT IGNORE INTO systempreferences (variable,value,explanation,options,type) VALUES('IDreamBooksResults','0','Display IDreamBooks.com rating in search results','','YesNo'); >-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'); >+INSERT INTO systempreferences (variable,value,explanation,options,type) VALUES('ReturnLostItems', '1', 'If enabled, items marked as lost will be automatically returned.','', 'YesNo'); >diff --git a/installer/data/mysql/updatedatabase.pl b/installer/data/mysql/updatedatabase.pl >index 8119c70..2b03bcb 100755 >--- a/installer/data/mysql/updatedatabase.pl >+++ b/installer/data/mysql/updatedatabase.pl >@@ -6401,7 +6401,7 @@ if ( CheckVersion($DBversion) ) { > > $DBversion ="3.11.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')"); >+ $dbh->do("INSERT INTO systempreferences (variable,value,options,explanation,type) VALUES('ReturnLostItems', '1', '','If enabled, items marked as lost will be automatically returned.', 'YesNo')"); > print "Upgrade to $DBversion done (Add system preference ReturnLostItems)\n"; > SetVersion($DBversion); > } >diff --git a/misc/cronjobs/longoverdue.pl b/misc/cronjobs/longoverdue.pl >index 159db51..ce9fd12 100755 >--- a/misc/cronjobs/longoverdue.pl >+++ b/misc/cronjobs/longoverdue.pl >@@ -50,7 +50,6 @@ GetOptions( > 'verbose' => \$verbose, > 'quiet' => \$quiet, > 'maxdays=s' => \$endrange, >- 'mark-returned' => \$mark_returned, > ); > > my $usage = << 'ENDUSAGE'; >@@ -77,8 +76,6 @@ This script takes the following parameters : > --maxdays Specifies the end of the range of overdue days to deal with (defaults to 366). This > value is universal to all lost num days overdue passed. > >- --mark-returned When an item is marked lost, remove it from the borrowers issued items. >- > examples : > $PERL5LIB/misc/cronjobs/longoverdue.pl --lost 30=1 > Would set LOST=1 after 30 days (up to one year), but not charge the account. >@@ -165,7 +162,7 @@ foreach my $startrange (sort keys %$lost) { > printf ("Due %s: item %5s from borrower %5s to lost: %s\n", $row->{date_due}, $row->{itemnumber}, $row->{borrowernumber}, $lostvalue) if($verbose); > if($confirm) { > ModItem({ itemlost => $lostvalue }, $row->{'biblionumber'}, $row->{'itemnumber'}); >- LostItem($row->{'itemnumber'}, $mark_returned, 'CHARGE FEE') if( $charge && $charge eq $lostvalue); >+ LostItem($row->{'itemnumber'}, C4::Context->preference("ReturnLostItems"), 'CHARGE FEE') if( $charge && $charge eq $lostvalue); > } > $count++; > } >-- >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