Thanks. Good detective work.
We should use the method "send_robust instead" instead of "send" in
"Order.order_success".
Django docs:
"send_robust() catches all errors derived from Python's Exception
class, and ensures all receivers are notified of the signal. If an
error occurs, the error instance is returned in the tuple pair for the
receiver that raised the error."
ALL developers:
Please Do you know more signals which need "send_robust" instead of
"send"?
Please, not too much signals to be changed, only the most important
because
a) It is more complicated - it requires to solve error handling for
them.
b) For some signals it is better to normally raise exception if one
listener fails,
e.g. for satchmo_price_query.
(It could be more expensive to sell one hour with bad price than to
stop it for one hour.)
Shaun,
Good detective work, but how it is possible that you did not note any
html 500 errors in web server log at the time of payment?
You probably mean "order_success". Hm, I do not know any
"order_completed".
-- Hynek
On 4 lis, 14:07, Shaun Dishman <[email protected]> wrote:
> Well, the problem disappeared shortly after I posted, and then
> reappeared three weeks later. After investigating the second
> occurrence, I discovered that the problem was related to an exception
> being caused in a listener we had registered against the
> "order_completed" signal. So the candidate would finish their order,
> get successfully charged by Auth.NET, the order_completed signal would
> fire, and our code would throw an error, which apparently was
> preventing the Order status from being updated.
>
> I figured I would post that just in case someone comes along behind
> someday with the same issue. Thanks for the help!
>
> -- Shaun
>
--
You received this message because you are subscribed to the Google Groups
"Satchmo users" group.
To post to this group, send email to [email protected].
To unsubscribe from this group, send email to
[email protected].
For more options, visit this group at
http://groups.google.com/group/satchmo-users?hl=en.
