Bugzilla – Attachment 33527 Details for
Bug 12505
Make variable aqorders.listprice in acq claim notice work and improve error handling
Home
|
New
|
Browse
|
Search
|
[?]
|
Reports
|
Help
|
New Account
|
Log In
[x]
|
Forgot Password
Login:
[x]
[patch]
Bug 12505 - Variable aqorders.listprice from 'Tools' - 'Notices and Slips'
Bug-12505---Variable-aqorderslistprice-from-Tools-.patch (text/plain), 2.26 KB, created by
simith.doliveira
on 2014-11-13 18:26:45 UTC
(
hide
)
Description:
Bug 12505 - Variable aqorders.listprice from 'Tools' - 'Notices and Slips'
Filename:
MIME Type:
Creator:
simith.doliveira
Created:
2014-11-13 18:26:45 UTC
Size:
2.26 KB
patch
obsolete
>From 038555bbd666d905c7856b2c971f6046912475e7 Mon Sep 17 00:00:00 2001 >From: simith <simith@inlibro.com> >Date: Thu, 13 Nov 2014 13:25:38 -0500 >Subject: [PATCH] Bug 12505 - Variable aqorders.listprice from 'Tools' - > 'Notices and Slips' > >--- > C4/Letters.pm | 6 ++++++ > acqui/lateorders.pl | 2 ++ > koha-tmpl/intranet-tmpl/prog/en/modules/acqui/lateorders.tt | 2 ++ > 3 files changed, 10 insertions(+) > >diff --git a/C4/Letters.pm b/C4/Letters.pm >index 7f2087c..d11bba4 100644 >--- a/C4/Letters.pm >+++ b/C4/Letters.pm >@@ -322,6 +322,12 @@ sub SendAlerts { > LEFT JOIN aqbooksellers ON subscription.aqbooksellerid=aqbooksellers.id > WHERE serial.serialid IN ( > }; >+ >+ if (!@$externalid){ >+ carp "No Order seleted"; >+ return { error => "no_order_seleted" }; >+ } >+ > $strsth .= join( ",", @$externalid ) . ")"; > my $sthorders = $dbh->prepare($strsth); > $sthorders->execute; >diff --git a/acqui/lateorders.pl b/acqui/lateorders.pl >index d43dbbe..b68673f 100755 >--- a/acqui/lateorders.pl >+++ b/acqui/lateorders.pl >@@ -116,6 +116,8 @@ if ($op and $op eq "send_alert"){ > $template->param(error_claim => $@); > } elsif ( ref $err and exists $err->{error} and $err->{error} eq "no_email" ) { > $template->{VARS}->{'error_claim'} = "no_email"; >+ } elsif ( ref $err and exists $err->{error} and $err->{error} eq "no_order_seleted"){ >+ $template->{VARS}->{'error_claim'} = "no_order_seleted"; > } else { > $template->{VARS}->{'info_claim'} = 1; > } >diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/acqui/lateorders.tt b/koha-tmpl/intranet-tmpl/prog/en/modules/acqui/lateorders.tt >index efa0db6..7fc7f1a 100644 >--- a/koha-tmpl/intranet-tmpl/prog/en/modules/acqui/lateorders.tt >+++ b/koha-tmpl/intranet-tmpl/prog/en/modules/acqui/lateorders.tt >@@ -83,6 +83,8 @@ $(document).ready(function() { > [% IF error_claim %] > [% IF ( error_claim == "no_email" ) %] > <div class="error">This vendor has no email</div> >+ [% ELSIF ( error_claim == "no_order_seleted" ) %] >+ <div class="error">No order selected</div> > [% ELSE %] > <div class="error">[% error_claim %]</div> > [% END %] >-- >1.9.1
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 12505
:
29382
|
31034
|
33527
|
33562
|
33602
|
33603