Is there a best practice recommendation for the use of
error() in contrib nodes? I notice that the email node calls
error() when it is finished (as picked up by Complete and Catch nodes) but never both, whereas the http request node calls
done() or both
error(). It may even call just
error() under some circumstances, I am not certain.
[Edit] I realise I have been a bit sloppy in my wording above as now
done() is used both for normal and error conditions. This may be a better way of phrasing the question:
Is there a best practice recommendation for the triggering of Complete and Catch in contrib nodes? I notice that the email node triggers either Complete or Catch but never both (as far as I can see). The http request node sometimes triggers Complete and sometimes triggers both Catch and Complete. It may even trigger just Catch under some circumstances, I don't know.