| Summary: | Don't queue check-in/checkout messages if patron has no e-mail | ||
|---|---|---|---|
| Product: | Koha | Reporter: | Katrin Fischer <katrin.fischer> |
| Component: | Notices | Assignee: | Bugs List <koha-bugs> |
| Status: | CLOSED WONTFIX | QA Contact: | Testopia <testopia> |
| Severity: | normal | ||
| Priority: | P5 - low | CC: | aude.charillon, caroline.cyr-la-rose, hayleygpelham, jonathan.druart |
| Version: | Main | ||
| Hardware: | All | ||
| OS: | All | ||
| GIT URL: | Initiative type: | --- | |
| Sponsorship status: | --- | Comma delimited list of Sponsors: | |
| Crowdfunding goal: | 0 | Patch complexity: | --- |
| Documentation contact: | Documentation submission: | ||
| Text to go in the release notes: | Version(s) released in: | ||
| Circulation function: | |||
|
Description
Katrin Fischer
2017-08-08 10:49:36 UTC
Status is supposed to be set to 'failed' and so not resent later. How could they be sent by accident? Is this resolved? They are still added as pending, being moved to 'failed' when the message queue is processed. Still wondering if this makes sense? This is the expected behaviour in my understanding. You could add an email later and resend the message for instance. (In reply to Jonathan Druart from comment #4) > This is the expected behaviour in my understanding. You could add an email > later and resend the message for instance. Would be good to get some more opinions here, I am actually not sure what the expected behavior is. At least we are not consistent as other messages don't create a mail entry and fall back to print and others are not generated at all. Would still be interesting to figure out if we consider this a bug or a feature :) My understanding is the same as Jonathan's. If the patron has no address, when process_message_queue runs, it puts it as "failed", and adds a comment in the db ("NO_EMAIL").
After that, it is not treated again, unless we click on the resend button, or change it manually in the db back to "pending".
I am closing this WONTFIX then. |