Comment 7 for bug 1667080
Revision history for this message
| Josh Stompro (u-launchpad-stompro-org) wrote | #7 |
© 2004
Canonical Ltd.
•
Terms of use
•
Data privacy
•
Contact Launchpad Support
•
Blog
•
Careers
•
System status
•
0f7b58d
(Get the code!)
A couple things,
1. There is a newer version of the Flowroute API that this code should be updated for.
2. There is a bug with how the event output id is grabbed when events are not grouped. When used with the Test SMS notice for example, that is a single event fired that doesn't go through the grouping logic. So the code that tries to grab the event output id fails, and the event gets marked as erroring out even though the message gets sent.
3. The Error checking needs to directly set the state to error when a fatal error is detected. Returning 0 doesn't take care of that. It should also create an error_output with error details in it.