Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

CRM-19697: No recycling the static _links object when you change it! #9471

Merged
merged 1 commit into from
Apr 23, 2017

Conversation

adixon
Copy link
Contributor

@adixon adixon commented Nov 30, 2016

Do not make use of the static _links object when customizing per payment processor for recurring contributions.


}

if (is_object($paymentProcessorObj) && $paymentProcessorObj->isSupported('updateSubscriptionBillingInfo')) {
self::$_links[CRM_Core_Action::RENEW] = array(
CRM_Core_Error::debug_var('ppobj', $paymentProcessorObj);
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ppobj!

@eileenmcnaughton
Copy link
Contributor

Can you add the JIRA into the commit message?

…g per payment processor for recurring contributions
@adixon
Copy link
Contributor Author

adixon commented Nov 30, 2016

Cleaned up my messes.

@monishdeb monishdeb changed the title No recycling the static _links object when you change it! CRM-19697: No recycling the static _links object when you change it! Dec 5, 2016
@seamuslee001
Copy link
Contributor

Jenkins re test this please

@seamuslee001
Copy link
Contributor

This i think makes sense to me, @eileenmcnaughton does it make sense to you?

@seamuslee001
Copy link
Contributor

I tested this and found the links correctly load, This looks good

@eileenmcnaughton eileenmcnaughton merged commit 43a6933 into civicrm:master Apr 23, 2017
@adixon
Copy link
Contributor Author

adixon commented Apr 24, 2017

Thanks seamus! I'd forgotten about this one.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants