Bugzilla – Attachment 33603 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]
[PASSED QA] Bug 12505 - Variable aqorders.listprice in acq claim notice doesn't work
PASSED-QA-Bug-12505---Variable-aqorderslistprice-i.patch (text/plain), 2.16 KB, created by
Katrin Fischer
on 2014-11-16 10:30:56 UTC
(
hide
)
Description:
[PASSED QA] Bug 12505 - Variable aqorders.listprice in acq claim notice doesn't work
Filename:
MIME Type:
Creator:
Katrin Fischer
Created:
2014-11-16 10:30:56 UTC
Size:
2.16 KB
patch
obsolete
>From 0211909a17c6bfd06e071024550f8dd0edd490d5 Mon Sep 17 00:00:00 2001 >From: simith <simith@inlibro.com> >Date: Wed, 20 Aug 2014 15:33:15 -0400 >Subject: [PATCH] [PASSED QA] Bug 12505 - Variable aqorders.listprice in acq > claim notice doesn't work > >Modified: > >C4/Letters.pm - remove aqbooksellers.* from SELECT statement > >In Letters - SendAlerts subrotine, is safe to remove aqbooksellers.* from SELECT statement >for type=claimacquisition or claimissues. Aqbooksellers is passed to GetPreparedLetter subrotine in tables variable. > >Testing: > >I Apply the patch > >Select Tools -> Notices and slips; >Edit ACQCLAIM; >Add : ><order>Ordernumber <<aqorders.ordernumber>> (<<biblio.title>>) (<<aqorders.quantity>> ordered) ($<<aqorders.listprice>> <<aqbooksellers.listprice>> each) has not been received.</order> >Save modifications; >Create a vendor (Acquisition module); >Create an order (Acquisition module); >Click Acquisitions -> Late orders; >Select the order created; >Click Claim order button; >Valide <<aqorders.listprice>>; >Valide <<aqbooksellers.listprice>>. > >Signed-off-by: Paola Rossi <paola.rossi@cineca.it> >Signed-off-by: Katrin Fischer <Katrin.Fischer.83@web.de> >Works as described. It's now possible to output the actual price >in the claim notice. >--- > C4/Letters.pm | 4 +--- > 1 file changed, 1 insertion(+), 3 deletions(-) > >diff --git a/C4/Letters.pm b/C4/Letters.pm >index 5673cbc..a72f310 100644 >--- a/C4/Letters.pm >+++ b/C4/Letters.pm >@@ -306,13 +306,11 @@ sub SendAlerts { > # search the biblionumber > my $strsth = $type eq 'claimacquisition' > ? qq{ >- SELECT aqorders.*,aqbasket.*,biblio.*,biblioitems.*,aqbooksellers.*, >- aqbooksellers.id AS booksellerid >+ SELECT aqorders.*,aqbasket.*,biblio.*,biblioitems.* > FROM aqorders > LEFT JOIN aqbasket ON aqbasket.basketno=aqorders.basketno > LEFT JOIN biblio ON aqorders.biblionumber=biblio.biblionumber > LEFT JOIN biblioitems ON aqorders.biblionumber=biblioitems.biblionumber >- LEFT JOIN aqbooksellers ON aqbasket.booksellerid=aqbooksellers.id > WHERE aqorders.ordernumber IN ( > } > : qq{ >-- >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