Ejegg has uploaded a new change for review.
https://gerrit.wikimedia.org/r/317189
Change subject: Don't delete pending data for failed:review
......................................................................
Don't delete pending data for failed:review
'Failed' is correct as far as the front-end goes, but when the
payment is in review, Donor Services might push it through manually.
We should leave donor details unless it's been rejected with finality.
Bug: T148859
Change-Id: Ic415217fd1fe4d2ecfbb5c53f93a1ce619cfd856
---
M sites/all/modules/queue2civicrm/fredge/PaymentsInitQueueConsumer.php
1 file changed, 11 insertions(+), 2 deletions(-)
git pull ssh://gerrit.wikimedia.org:29418/wikimedia/fundraising/crm
refs/changes/89/317189/1
diff --git
a/sites/all/modules/queue2civicrm/fredge/PaymentsInitQueueConsumer.php
b/sites/all/modules/queue2civicrm/fredge/PaymentsInitQueueConsumer.php
index 9590deb..334120d 100644
--- a/sites/all/modules/queue2civicrm/fredge/PaymentsInitQueueConsumer.php
+++ b/sites/all/modules/queue2civicrm/fredge/PaymentsInitQueueConsumer.php
@@ -28,8 +28,17 @@
// Delete corresponding pending rows if this contribution
failed.
// The DonationQueueConsumer will delete pending rows for
successful
// contributions, and we don't want to be too hasty.
- if ( $message['payments_final_status'] === 'failed' ) {
- watchdog( 'fredge', "Deleting pending row for failed
message {$logId}", array(), WATCHDOG_INFO );
+ // Leave details for payments still open for manual review.
+ if (
+ $message['payments_final_status'] === 'failed' &&
+ $message['validation_action'] === 'reject'
+ ) {
+ watchdog(
+ 'fredge',
+ "Deleting pending row for failed and rejected
payment {$logId}",
+ array(),
+ WATCHDOG_INFO
+ );
PendingDatabase::get()->deleteMessage( $message );
}
--
To view, visit https://gerrit.wikimedia.org/r/317189
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings
Gerrit-MessageType: newchange
Gerrit-Change-Id: Ic415217fd1fe4d2ecfbb5c53f93a1ce619cfd856
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
