From abf0d662e33ac13475eb03e88e9349df0c21439d Mon Sep 17 00:00:00 2001
From: eileen <emcnaughton@wikimedia.org>
Date: Tue, 5 Nov 2019 11:06:46 +1300
Subject: [PATCH] Re-apply useful cleanups from reverted commits

---
 CRM/Core/Payment.php        | 4 +---
 api/v3/PaymentProcessor.php | 4 +++-
 2 files changed, 4 insertions(+), 4 deletions(-)

diff --git a/CRM/Core/Payment.php b/CRM/Core/Payment.php
index 69b97a342f..91a7c0be8c 100644
--- a/CRM/Core/Payment.php
+++ b/CRM/Core/Payment.php
@@ -81,7 +81,7 @@ abstract class CRM_Core_Payment {
     RECURRING_PAYMENT_END = 'END';
 
   /**
-   * @var object
+   * @var array
    */
   protected $_paymentProcessor;
 
@@ -257,8 +257,6 @@ abstract class CRM_Core_Payment {
    * @todo move to factory class \Civi\Payment\System (or similar)
    *
    * @param array $params
-   *
-   * @return mixed
    */
   public static function logPaymentNotification($params) {
     $message = 'payment_notification ';
diff --git a/api/v3/PaymentProcessor.php b/api/v3/PaymentProcessor.php
index e2406dc5d0..c2e5e4e163 100644
--- a/api/v3/PaymentProcessor.php
+++ b/api/v3/PaymentProcessor.php
@@ -124,8 +124,10 @@ function _civicrm_api3_payment_processor_getlist_defaults(&$request) {
  *   API result array.
  *
  * @throws \API_Exception
+ * @throws \CiviCRM_API3_Exception
  */
 function civicrm_api3_payment_processor_pay($params) {
+  /* @var CRM_Core_Payment $processor */
   $processor = Civi\Payment\System::singleton()->getById($params['payment_processor_id']);
   $processor->setPaymentProcessor(civicrm_api3('PaymentProcessor', 'getsingle', ['id' => $params['payment_processor_id']]));
   try {
@@ -149,7 +151,7 @@ function civicrm_api3_payment_processor_pay($params) {
  */
 function _civicrm_api3_payment_processor_pay_spec(&$params) {
   $params['payment_processor_id'] = [
-    'api.required' => 1,
+    'api.required' => TRUE,
     'title' => ts('Payment processor'),
     'type' => CRM_Utils_Type::T_INT,
   ];
-- 
2.25.1