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

Cancel Gateway Subscriptions without having to put orders in "cancelled" status. #41

Closed
strangerstudios opened this issue Jun 7, 2013 · 4 comments

Comments

@strangerstudios
Copy link
Collaborator

Right now, when you cancel a membership, PMPro sets the last order's status to "cancelled" and checks that status to see if a level has been cancelled.

Now that we are showing orders in the admin and using their statuses for other reporting (i.e. to check if an ORDER has really been cancelled), let's have PMPro check the membership_levels_users table RE cancellations instead of the orders table.

@lisaleague
Copy link

Suggest using "expired" or "inactive" as designation in orders table rather than "cancelled" as cancelled may imply cancelled at gateway.

@ideadude
Copy link
Member

First step. Audit all core code and add ons that might be using the order status.

@ideadude
Copy link
Member

Here is one place where we check the status of an order:
https://github.com/strangerstudios/paid-memberships-pro/blob/dev/services/ipnhandler.php#L263

In the PayPal IPN handler, we check if an order in question has already been cancelled before trying to cancel the related subscription. This concept, that a subscription attached to an order has been cancelled, needs to be preserved somehow.

@ideadude ideadude modified the milestones: 2.0, Future Versions May 17, 2019
@dparker1005
Copy link
Member

Resolved in 3.0

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

No branches or pull requests

4 participants