When an order is canceled we need a conditional action for revoking any file access that order got from uc_file. Currently when an order is canceled the file access is not removed and there is no available action for setting up a ca to do that.

CommentFileSizeAuthor
#3 uc-file-revoke.patch6.3 KBJody Lynn
#2 revoke_access.patch6.29 KBkleinmp
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

rszrama’s picture

Issue tags: +CA

Tagging.

kleinmp’s picture

FileSize
6.29 KB

I made this patch for uc_file that revokes file access that was received from an order, if that order is cancelled. If the order is reinstated, then it renews file access. IT works nicely for the site we're working on, but it hasn't been tested for different file access configurations (such as file directories).

Jody Lynn’s picture

FileSize
6.3 KB

Slight bugfix for the above patch, to fix the drupal_set_message.

I tested and reviewed the patch but it is still not tested for file directory purchase, so leaving as 'needs review' for now.

Jody Lynn’s picture

Status: Active » Needs review
Island Usurper’s picture

Why is $order_args removed, but not replaced in the notify_grant predicate?

rszrama’s picture

Issue tags: +ubercamp sprint

Tagging.

itsmahitha’s picture

Assigned: Unassigned » itsmahitha
Status: Needs review » Reviewed & tested by the community

Patch is working and tested.

TR’s picture

Assigned: itsmahitha » Unassigned
Status: Reviewed & tested by the community » Needs review
TR’s picture

#676336: uc_file should revoke purchased files when an order is Canceled has some discussion of this issue, but is marked as a duplicate of this thread.

jtbayly’s picture

subscribe

longwave’s picture

Component: Code » File downloads

Changing component.

Status: Needs review » Needs work

The last submitted patch, uc-file-revoke.patch, failed testing.

TR’s picture

Version: 6.x-2.0-beta5 » 6.x-2.x-dev

Patch is not in -p0 or -p1 format.

TR’s picture

Title: Conditional Action for removing File Access » Rule for removing File Access
Version: 6.x-2.x-dev » 8.x-4.x-dev
Issue summary: View changes
Issue tags: -CA, -ubercamp sprint