Skip to content
Snippets Groups Projects
Commit d7b0e446 authored by mattwire's avatar mattwire
Browse files

Add some FIXME comments

parent 1b903715
No related branches found
No related tags found
No related merge requests found
...@@ -404,6 +404,7 @@ class CRM_Core_Payment_StripeIPN extends CRM_Core_Payment_BaseIPN { ...@@ -404,6 +404,7 @@ class CRM_Core_Payment_StripeIPN extends CRM_Core_Payment_BaseIPN {
$query_params $query_params
); );
// FIXME: MJW Do we need this custom handling for memberships here? Core should do all we need
if ($this->membership_id) { if ($this->membership_id) {
$plan_elements = explode("-", $this->plan_id); $plan_elements = explode("-", $this->plan_id);
$plan_name_elements = explode("-", $this->plan_name); $plan_name_elements = explode("-", $this->plan_name);
...@@ -581,6 +582,7 @@ class CRM_Core_Payment_StripeIPN extends CRM_Core_Payment_BaseIPN { ...@@ -581,6 +582,7 @@ class CRM_Core_Payment_StripeIPN extends CRM_Core_Payment_BaseIPN {
} }
// Check for membership id. // Check for membership id.
// FIXME: MJW Not sure why we assign membership_id here
$membership = civicrm_api3('Membership', 'get', array( $membership = civicrm_api3('Membership', 'get', array(
'contribution_recur_id' => $this->contribution_recur_id, 'contribution_recur_id' => $this->contribution_recur_id,
)); ));
......
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment