---- Reported by jwagner@ptfs.com 2010-02-17 20:03:01 ---- In previous versions, when a staff person went into Acquisitions, Manage Suggestions and took some action on pending suggestions (accept, reject, etc.), the action generated an email to the patron based on the templates in kohaclone/koha-tmpl/intranet-tmpl/prog/en/modules/suggestion. Under current head (3.2 alpha, roughly), those emails aren't being generated. There are no entries in the error log, and these messages don't go through the message queue, so there's nothing to track them by. The system is generating and sending other types of emails. --- Bug imported by chris@bigballofwax.co.nz 2010-05-21 01:24 UTC --- This bug was previously known as _bug_ 4211 at http://bugs.koha.org/cgi-bin/bugzilla3/show_bug.cgi?id=4211 Unknown operating system Windows 2000. Setting to default OS "All". Actual time not defined. Setting to 0.0 The original reporter of this bug does not have an account here. Reassigning to the person who moved it here: chris@bigballofwax.co.nz. Previous reporter was jwagner@ptfs.com. Setting qa contact to the default for this product. This bug either had no qa contact or an invalid one.
On a default installation, there will be no notices defined for suggestions handling (accepted, rejected, etc.)--Bug 5127. In addition, there is no way to add a suggestions notice because that category doesn't exist in the "add notice" form--Bug 5126. I've tested a fix for Bug 5126, and tried adding a suggestions notice, but the notice still failed: suggestion.pl: can't enqueue letter HASH(0xa5073c8) at /kohaclone/C4/Suggestions.pm line 394
Created attachment 2512 [details] [review] Proposed fix for HEAD Assuming 1) The patch for Bug 5126 is approved and 2) Suggestions notices are added by hand (or from default sql--see Bug 5127) this correction should get suggestions notices properly enqueued.
After applying that patch and testing I find that suggestions messages are getting added to the queue, but it doesn't look like the information is being processed correctly. Some data is missing from the entry in the message_queue table: borrowernumber, to_address, and from_address are NULL. The content column is populated, but the message includes the placeholders instead of the actual values: <<borrowers.firstname>> <<borrowers.surname>> etc. I'm not sure where to look next.
Created attachment 2514 [details] [review] Followup fix, (includes first patch)
After applying this patch and testing I'm getting the same results: placeholders in the enqueued message and NULL values for to_address and from_address. We also have a problem with the options available for building the suggestions notices: /cgi-bin/koha/tools/letter.pl?op=add_form&module=suggestions&code=ACCEPTED The original template for suggestions notifications included the suggested title, the title's author, and the suggester's first and last name. In the "add notice" interface I've got the option to add data from branches, biblio, biblioitems, and items. Is it possible to pull fields from suggestions as placeholders in the notice?
Yup, <<suggestions.title>> etc were working for me. So we just need to update the notices page to allow people to add those to the template. None of the biblio ones will work for suggestions, except when a suggestion is filled. Otherwise the biblionumber is always empty.
Patches pushed please test
Setting status to "PATCH-Sent" and "Patch Pushed", see Comment 7.
*** Bug 4273 has been marked as a duplicate of this bug. ***
*** Bug 1886 has been marked as a duplicate of this bug. ***
Ok, I tested this on 3.6/current master today and the mail. This is a summary, I will file additional bugs for the separate issues: There are mails generated when you mark something as available or when you mark it as rejected. Every time you change the status between those too, the patron will get a notification email. BUG 7093: The placeholders for fields from suggestions do not work. The patron name showed up correctly. BUG 7094: I found no way to trigger the ORDERED notice.. BUG 7095: I found no way to trigger the AVAILABLE notice.
The missing placeholders and some of the other bugs this patch addresses have already been fixed in master (see list of related bugs in comment 11) Not sure this patch is still needed.
I just checked. On current master mails for ordering, accepting and rejecting suggestions are generated nicely and are sent to the patron. Closing this RESOLVED FIXED.