Bugzilla – Attachment 93671 Details for
Bug 15422
Number of items to order on holds ratio report will not fulfill the holds ratio
Home
|
New
|
Browse
|
Search
|
[?]
|
Reports
|
Help
|
New Account
|
Log In
[x]
|
Forgot Password
Login:
[x]
[patch]
Bug 15422: Consider partial copies needed as whole numbers
Bug-15422-Consider-partial-copies-needed-as-whole-.patch (text/plain), 1.19 KB, created by
Katrin Fischer
on 2019-10-04 07:54:04 UTC
(
hide
)
Description:
Bug 15422: Consider partial copies needed as whole numbers
Filename:
MIME Type:
Creator:
Katrin Fischer
Created:
2019-10-04 07:54:04 UTC
Size:
1.19 KB
patch
obsolete
>From 5ea0cc0fb67614cfb33347980c9a2fef5c7547d4 Mon Sep 17 00:00:00 2001 >From: Nick Clemens <nick@bywatersolutions.com> >Date: Tue, 4 Dec 2018 14:58:35 +0000 >Subject: [PATCH] Bug 15422: Consider partial copies needed as whole numbers > >Signed-off-by: Owen Leonard <oleonard@myacpl.org> > >Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de> >--- > circ/reserveratios.pl | 3 ++- > 1 file changed, 2 insertions(+), 1 deletion(-) > >diff --git a/circ/reserveratios.pl b/circ/reserveratios.pl >index 1beb559217..b4b0b81229 100755 >--- a/circ/reserveratios.pl >+++ b/circ/reserveratios.pl >@@ -22,6 +22,7 @@ use Modern::Perl; > > use CGI qw ( -utf8 ); > use Date::Calc qw/Today Add_Delta_YM/; >+use POSIX qw( ceil ); > > use C4::Context; > use C4::Output; >@@ -154,7 +155,7 @@ $sth->execute(@query_params); > my @reservedata; > while ( my $data = $sth->fetchrow_hashref ) { > my $thisratio = $data->{reservecount} / $data->{itemcount}; >- my $ratiocalc = $data->{reservecount}/$ratio - $data->{itemcount}; >+ my $ratiocalc = ceil($data->{reservecount}/$ratio - $data->{itemcount}); > $ratiocalc >= 1 or next; # TODO: tighter targeting -- get ratio limit into SQL using HAVING clause > push( > @reservedata, >-- >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 15422
:
82718
|
82842
|
82873
|
82874
|
92145
|
92146
|
93670
| 93671