clean up changes for batch 18

[civicrm-core.git] / CRM / Member / BAO / MembershipPayment.php

diff --git a/CRM/Member/BAO/MembershipPayment.php b/CRM/Member/BAO/MembershipPayment.php
index ec04cacc55bb6825b436d7c6faf76a9f81525f4a..aeef774c4a53588f6911258483fbad4089121cde 100644 (file)
--- a/CRM/Member/BAO/MembershipPayment.php
+++ b/CRM/Member/BAO/MembershipPayment.php
@@ -1,7 +1,7 @@
 <?php
 /*
  +--------------------------------------------------------------------+
- | CiviCRM version 4.5                                                |
+ | CiviCRM version 4.6                                                |
  +--------------------------------------------------------------------+
  | Copyright CiviCRM LLC (c) 2004-2014                                |
  +--------------------------------------------------------------------+
@@ -38,20 +38,20 @@ class CRM_Member_BAO_MembershipPayment extends CRM_Member_DAO_MembershipPayment
   /**
    * Class constructor
    */
-  function __construct() {
+  public function __construct() {
     parent::__construct();
   }
+
   /**
    * Add the membership Payments
    *
-   * @param array $params reference array contains the values submitted by the form
+   * @param array $params
+   *   Reference array contains the values submitted by the form.
    *
-   * @access public
-   * @static
    *
    * @return object
    */
-  static function create($params) {
+  public static function create($params) {
     $hook = empty($params['id']) ? 'create' : 'edit';
     CRM_Utils_Hook::pre($hook, 'MembershipPayment', CRM_Utils_Array::value('id', $params), $params);
     $dao = new CRM_Member_DAO_MembershipPayment();
@@ -67,13 +67,20 @@ class CRM_Member_BAO_MembershipPayment extends CRM_Member_DAO_MembershipPayment
     // table. However, at this stage we have both - there is still quite a bit of refactoring to do to set the line_iten entity_id right the first time
     // however, we can assume at this stage that any contribution id will have only one line item with that membership type in the line item table
     // OR the caller will have taken responsibility for updating the line items themselves so we will update using SQL here
-    $membershipTypeID = civicrm_api3('membership', 'getvalue', array('id' => $dao->membership_id, 'return' => 'membership_type_id'));
+    $membershipTypeID = civicrm_api3('membership', 'getvalue', array(
+        'id' => $dao->membership_id,
+        'return' => 'membership_type_id',
+      ));
     $sql = "UPDATE civicrm_line_item li
       LEFT JOIN civicrm_price_field_value pv ON pv.id = li.price_field_value_id
       SET entity_table = 'civicrm_membership', entity_id = %1
       WHERE pv.membership_type_id = %2
       AND contribution_id = %3";
-    CRM_Core_DAO::executeQuery($sql, array(1 => array($dao->membership_id, 'Integer'), 2 => array($membershipTypeID, 'Integer'), 3 => array($dao->contribution_id, 'Integer')));
+    CRM_Core_DAO::executeQuery($sql, array(
+        1 => array($dao->membership_id, 'Integer'),
+        2 => array($membershipTypeID, 'Integer'),
+        3 => array($dao->contribution_id, 'Integer'),
+      ));
     return $dao;
   }
 
@@ -83,9 +90,8 @@ class CRM_Member_BAO_MembershipPayment extends CRM_Member_DAO_MembershipPayment
    * @param int $id
    *
    * @return bool
-   * @static
    */
-  static function del($id) {
+  public static function del($id) {
     $dao = new CRM_Member_DAO_MembershipPayment();
     $dao->id = $id;
     $result = FALSE;
@@ -98,4 +104,3 @@ class CRM_Member_BAO_MembershipPayment extends CRM_Member_DAO_MembershipPayment
 
 
 }
-