Ejegg has uploaded a new change for review.
https://gerrit.wikimedia.org/r/280580
Change subject: Send failmail when TY job can't find contribution
......................................................................
Send failmail when TY job can't find contribution
This should come to our attention.
Bug: T131200
Change-Id: If2493a62b432fa7c1c1af2f47f2630fcc497066c
---
M sites/all/modules/thank_you/thank_you.module
1 file changed, 2 insertions(+), 1 deletion(-)
git pull ssh://gerrit.wikimedia.org:29418/wikimedia/fundraising/crm
refs/changes/80/280580/1
diff --git a/sites/all/modules/thank_you/thank_you.module
b/sites/all/modules/thank_you/thank_you.module
index 8bb36eb..582777a 100644
--- a/sites/all/modules/thank_you/thank_you.module
+++ b/sites/all/modules/thank_you/thank_you.module
@@ -241,7 +241,8 @@
// check that the API result is a valid contribution result
if( !WMFCiviAPICheck::check_api_contribution( $contribution,
$contribution_id ) ){
// the API result is bad
- watchdog('thank_you', 'Could not retrieve contribution record
for: ' . $contribution_id . '<pre>' . print_r( $contribution, true ) .
'</pre>', array(), WATCHDOG_ERROR);
+ $msg = 'Could not retrieve contribution record for: ' .
$contribution_id . '<pre>' . print_r( $contribution, true ) . '</pre>';
+ wmf_common_failmail( 'thank_you', $msg );
return false;
}
// go ahead and remove the extra layer of indirection to make it easier
to use
--
To view, visit https://gerrit.wikimedia.org/r/280580
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings
Gerrit-MessageType: newchange
Gerrit-Change-Id: If2493a62b432fa7c1c1af2f47f2630fcc497066c
Gerrit-PatchSet: 1
Gerrit-Project: wikimedia/fundraising/crm
Gerrit-Branch: master
Gerrit-Owner: Ejegg <[email protected]>
_______________________________________________
MediaWiki-commits mailing list
[email protected]
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits
