Don't show a notification if engine_result is not tesSUCCESS.

We do need to show notifications for certain types of failures, but right now we
are showing a notification as if the transaction succeeded which is very bad
indeed.
Minor margin fix.