Summary: | Acquisitions actions on suggestions don't generate email | ||
---|---|---|---|
Product: | Koha | Reporter: | Jane Wagner <jwagner> |
Component: | Acquisitions | Assignee: | Chris Cormack <chris> |
Status: | CLOSED FIXED | QA Contact: | Bugs List <koha-bugs> |
Severity: | major | ||
Priority: | P1 - high | CC: | chris, jcamins, magnus, nengard |
Version: | Main | ||
Hardware: | PC | ||
OS: | All | ||
See Also: | http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=5539 | ||
GIT URL: | Change sponsored?: | --- | |
Patch complexity: | --- | Documentation contact: | |
Documentation submission: | Text to go in the release notes: | ||
Version(s) released in: | Circulation function: | ||
Bug Depends on: | 5126, 5127, 7093, 7094, 7096 | ||
Bug Blocks: | 1886 | ||
Attachments: |
Proposed fix for HEAD
Followup fix, (includes first patch) |
Description
Chris Cormack
2010-05-21 01:24:27 UTC
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. |