Skip to content

Commit f8711b7

Browse files
authored
Merge pull request #32243 from mattwire/contwhite
NFC: Formatting only from d246fa0
2 parents dbfef2d + 2bbcc3d commit f8711b7

File tree

1 file changed

+8
-6
lines changed

1 file changed

+8
-6
lines changed

CRM/Contribute/BAO/Contribution.php

+8-6
Original file line numberDiff line numberDiff line change
@@ -4001,9 +4001,11 @@ public static function updateMembershipBasedOnCompletionOfContribution($contribu
40014001
'is_test' => $membership['is_test'],
40024002
'membership_type_id' => $membership['membership_type_id'],
40034003
'membership_activity_status' => 'Completed',
4004+
'skipStatusCal' => 1,
40044005
];
40054006

4006-
$currentMembership = CRM_Member_BAO_Membership::getContactMembership($membershipParams['contact_id'],
4007+
$currentMembership = CRM_Member_BAO_Membership::getContactMembership(
4008+
$membershipParams['contact_id'],
40074009
$membershipParams['membership_type_id'],
40084010
$membershipParams['is_test'],
40094011
$membershipParams['id']
@@ -4050,8 +4052,8 @@ public static function updateMembershipBasedOnCompletionOfContribution($contribu
40504052

40514053
// @todo - we should pass membership_type_id instead of null here but not
40524054
// adding as not sure of testing
4053-
$dates = CRM_Member_BAO_MembershipType::getRenewalDatesForMembershipType($membershipParams['id'],
4054-
$changeDate, NULL, $membershipParams['num_terms']
4055+
$dates = CRM_Member_BAO_MembershipType::getRenewalDatesForMembershipType(
4056+
$membershipParams['id'], $changeDate, NULL, $membershipParams['num_terms']
40554057
);
40564058
$dates['join_date'] = $currentMembership['join_date'];
40574059
}
@@ -4060,7 +4062,8 @@ public static function updateMembershipBasedOnCompletionOfContribution($contribu
40604062
}
40614063
else {
40624064
//get the status for membership.
4063-
$calcStatus = CRM_Member_BAO_MembershipStatus::getMembershipStatusByDate($dates['start_date'] ?? NULL,
4065+
$calcStatus = CRM_Member_BAO_MembershipStatus::getMembershipStatusByDate(
4066+
$dates['start_date'] ?? NULL,
40644067
$dates['end_date'] ?? NULL,
40654068
$dates['join_date'] ?? NULL,
40664069
'now',
@@ -4072,8 +4075,7 @@ public static function updateMembershipBasedOnCompletionOfContribution($contribu
40724075
unset($dates['end_date']);
40734076
$membershipParams['status_id'] = $calcStatus['id'] ?? 'New';
40744077
}
4075-
//we might be renewing membership,
4076-
//so make status override false.
4078+
// We might be renewing membership so make status override false.
40774079
$membershipParams['is_override'] = FALSE;
40784080
$membershipParams['status_override_end_date'] = 'null';
40794081
$membership = civicrm_api3('Membership', 'create', $membershipParams);

0 commit comments

Comments
 (0)