diff --git a/CRM/Member/Form/MembershipRenewal.php b/CRM/Member/Form/MembershipRenewal.php index 894f70123098..6a6d41cb6cf2 100644 --- a/CRM/Member/Form/MembershipRenewal.php +++ b/CRM/Member/Form/MembershipRenewal.php @@ -748,7 +748,6 @@ protected function sendReceipt($membership) { * @throws \CiviCRM_API3_Exception */ public function processMembership($contactID, $membershipTypeID, $is_test, $changeToday, $customFieldsFormatted, $numRenewTerms, $membershipID, $pending, $contributionRecurID, $isPayLater) { - $updateStatusId = FALSE; $allStatus = CRM_Member_PseudoConstant::membershipStatus(); $format = '%Y%m%d'; $membershipTypeDetails = CRM_Member_BAO_MembershipType::getMembershipTypeDetails($membershipTypeID); @@ -855,23 +854,12 @@ public function processMembership($contactID, $membershipTypeID, $is_test, $chan } $memParams['membership_activity_status'] = ($pending || $isPayLater) ? 'Scheduled' : 'Completed'; } - //CRM-4555 - if ($pending) { - $updateStatusId = array_search('Pending', $allStatus); - } // Putting this in an IF is precautionary as it seems likely that it would be ignored if empty, but // perhaps shouldn't be? if ($contributionRecurID) { $memParams['contribution_recur_id'] = $contributionRecurID; } - //CRM-4555 - //if we decided status here and want to skip status - //calculation in create( ); then need to pass 'skipStatusCal'. - if ($updateStatusId) { - $memParams['status_id'] = $updateStatusId; - $memParams['skipStatusCal'] = TRUE; - } //since we are renewing, //make status override false.