Adamw has submitted this change and it was merged.
Change subject: Failmail now has a log line!
......................................................................
Failmail now has a log line!
It's not as detailed as the whole email but it at least tells us
if a failmail occured and what the error was that caused it.
Change-Id: Ie7df006264905b6f06c2819ccc80bdb9a64aaea9
---
M sites/all/modules/wmf_common/failmail.php
1 file changed, 8 insertions(+), 0 deletions(-)
Approvals:
Adamw: Verified; Looks good to me, approved
diff --git a/sites/all/modules/wmf_common/failmail.php
b/sites/all/modules/wmf_common/failmail.php
index 5363a0e..fb4dcb2 100644
--- a/sites/all/modules/wmf_common/failmail.php
+++ b/sites/all/modules/wmf_common/failmail.php
@@ -12,6 +12,14 @@
} elseif (property_exists($error, 'source')) {
$params['source'][] = $error->source;
}
+
+ watchdog(
+ 'failmail',
+ "What's that? Something wrong: $error",
+ array(),
+ WATCHDOG_ERROR
+ );
+
$params['module'] = $module;
$params['removed'] = (is_callable(array($error, 'isRejectMessage'))) ?
$error->isRejectMessage() : FALSE;
drupal_mail('wmf_common', 'fail', $to, language_default(), $params);
--
To view, visit https://gerrit.wikimedia.org/r/63795
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: Ie7df006264905b6f06c2819ccc80bdb9a64aaea9
Gerrit-PatchSet: 4
Gerrit-Project: wikimedia/fundraising/crm
Gerrit-Branch: master
Gerrit-Owner: Mwalker <mwal...@wikimedia.org>
Gerrit-Reviewer: Adamw <awi...@wikimedia.org>
_______________________________________________
MediaWiki-commits mailing list
MediaWiki-commits@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits
