Hi,

Great module, especially in combination with the Commerce Recurring Framework (and some elbow grease).

It would be really great if the Delete callback was invoked on hook_user_cancel() as the card on file should never again be used at that stage.

If I get some time myself I will submit a patch.

Cheers,
Deciphered.

Comments

dwkitchen’s picture

dwkitchen commented
Version: 7.x-1.x-dev » 7.x-2.x-dev
jpstrikesback’s picture

jpstrikesback commented

It would be nice to introduce a new status of "Remote deleted" and then upon user cancel optionally delete the card (everywhere) or only delete the remote card and set the status to "Remote deleted" so there is/can be permanent record of what card was used on an order.

dwkitchen’s picture

dwkitchen commented

When a card is deleted (this function exists, there just isn't a link to it on the card overview page) it is set to a "remote deleted" state. When cancelling a user account there are a number of options on what to do with the users content and the action could be configured to follow the same.

In a commerce store it is unlikely that you want to completely delete all records for a user as they will be needed for financial record keeping etc.

jpstrikesback’s picture

jpstrikesback commented

You are totally correct (tho the status is set to "disabled")

So what we'd want are a few options and a default (that disables the card probably) on the card on file settings page for how to react to the various user_cancel methods (and the hook implementation) right?