From 0c340fdf22815f5b5ee35d6e95fb6861a935af3a Mon Sep 17 00:00:00 2001 From: Ian Walls Date: Wed, 7 Dec 2011 11:43:20 -0500 Subject: [PATCH] Bug 7093: placeholds for suggestion table not working in notices In parseletter_sth, the suggestions table was defined twice. The first time (which is the only definition to get called, as it matches first then exists), defined the primary search keys as borrowernumber and biblionumber. This is incorrect; the only usage of the suggestions table tokens is with suggestionid as the key. This is defined further down the if/then chain. This patch removes the erroneous sth query definition. To test: 1. Place a suggestion with a patron account with a configured email you can access 2. Approve or reject it 3. Verify the message you receive contains suggestion tokens (like title, author and reason) Signed-off-by: Katrin Fischer Suggestion mails for accepted, rejected and ordered look good now. I couldn't test the suggestion mail AVAILABLE (bug 7096), but assuming this would work too. Signed-off-by: Paul Poulain Signed-off-by: Chris Nighswonger --- C4/Letters.pm | 1 + 1 file changed, 1 insertion(+) diff --git a/C4/Letters.pm b/C4/Letters.pm index c894a57d62..b409c9bdaa 100644 --- a/C4/Letters.pm +++ b/C4/Letters.pm @@ -485,6 +485,7 @@ sub parseletter_sth { ($table eq 'biblio' ) ? "SELECT * FROM $table WHERE biblionumber = ?" : ($table eq 'biblioitems' ) ? "SELECT * FROM $table WHERE biblionumber = ?" : ($table eq 'items' ) ? "SELECT * FROM $table WHERE itemnumber = ?" : + ($table eq 'issues' ) ? "SELECT * FROM $table WHERE itemnumber = ?" : ($table eq 'reserves' ) ? "SELECT * FROM $table WHERE borrowernumber = ? and biblionumber = ?" : ($table eq 'borrowers' ) ? "SELECT * FROM $table WHERE borrowernumber = ?" : ($table eq 'branches' ) ? "SELECT * FROM $table WHERE branchcode = ?" : -- 2.39.5