From 2982de8effa53db99367607e94fcb3025ae76787 Mon Sep 17 00:00:00 2001 From: jitendrapurohit Date: Fri, 22 Jul 2016 12:33:02 +0530 Subject: [PATCH] CRM-19110: Fix delete pledge payments --- CRM/Pledge/BAO/PledgePayment.php | 2 +- tests/phpunit/CRM/Pledge/BAO/PledgePaymentTest.php | 13 +++++++++++++ 2 files changed, 14 insertions(+), 1 deletion(-) diff --git a/CRM/Pledge/BAO/PledgePayment.php b/CRM/Pledge/BAO/PledgePayment.php index 9fbb8518bd..71d238dc6e 100644 --- a/CRM/Pledge/BAO/PledgePayment.php +++ b/CRM/Pledge/BAO/PledgePayment.php @@ -271,7 +271,7 @@ WHERE pledge_id = %1 if ($payment->contribution_id) { CRM_Contribute_BAO_Contribution::deleteContribution($payment->contribution_id); } - $payment->delete(); + self::del($payment->id); } } diff --git a/tests/phpunit/CRM/Pledge/BAO/PledgePaymentTest.php b/tests/phpunit/CRM/Pledge/BAO/PledgePaymentTest.php index 7e654f38c6..311115a569 100644 --- a/tests/phpunit/CRM/Pledge/BAO/PledgePaymentTest.php +++ b/tests/phpunit/CRM/Pledge/BAO/PledgePaymentTest.php @@ -140,6 +140,19 @@ class CRM_Pledge_BAO_PledgePaymentTest extends CiviUnitTestCase { $result = CRM_Pledge_BAO_Pledge::deletePledge($payment->pledge_id); } + /** + * Delete Multiple payments for one pledge. + */ + public function testDeletePledgePayments() { + $contactId = $this->individualCreate(); + $pledgeId = $this->pledgeCreate(array('contact_id' => $contactId)); + CRM_Pledge_BAO_PledgePayment::deletePayments($pledgeId); + + // No payments should be retrieved + $pledgePayment = CRM_Pledge_BAO_PledgePayment::getPledgePayments($pledgeId); + $this->assertEquals(count($pledgePayment), 0, "Checking for empty array"); + } + /** * Pass Null Id for a payment deletion for one pledge. */ -- 2.25.1