r100064 MediaWiki - Code Review archive

Repository:MediaWiki
Revision:r100063‎ | r100064 | r100065 >
Date:18:05, 17 October 2011
Author:khorn
Status:ok (Comments)
Tags:fundraising 
Comment:
Prevents the donation Thank You page from displaying only a stack trace in the event that the activeMQ server is unavailable.
Modified paths:
  • /branches/fundraising/extensions/DonationInterface/gateway_common/gateway.adapter.php (modified) (history)

Diff [purge]

Index: branches/fundraising/extensions/DonationInterface/gateway_common/gateway.adapter.php
@@ -956,8 +956,13 @@
957957 $transaction += $this->getData();
958958
959959 self::log( "Intended STOMP transaction: " . print_r( $transaction, true ) );
960 -
961 - wfRunHooks( $hook, array( $transaction ) );
 960+
 961+ try {
 962+ wfRunHooks( $hook, array( $transaction ) );
 963+ } catch ( Exception $e ) {
 964+ self::log( "STOMP ERROR. Could not add message. " . $e->getMessage() , LOG_CRIT );
 965+ }
 966+
962967 }
963968
964969 function smooshVarsForStaging() {

Past revisions this follows-up on

RevisionCommit summaryAuthorDate
r97780Card #282, and a significant redefinition of what it means to stage gateway d...khorn02:02, 22 September 2011

Comments

#Comment by Khorn (WMF) (talk | contribs)   18:10, 17 October 2011

This is related to r97780.

Status & tagging log