When a limited number of download attempts is used, it may make sense (to some people) to allow a user to purchase additional download chances (with a warning).

But if a user has already purchased a file, and download limit is set to infinite (which I think is most fair), they should not be allowed to accidentally re-purchase a file. They'd be getting nothing for their money, which is unfair, and could create legal and ethical problems for the site owner and customer.

If a file download limit is set to infinite, when a customer attempts to accidentally re-purchase the file product, they should either be told that they already have the product and can download it whenever they like, or should be charged $0 for it - something that keeps them from spending money for nothing.

This would also reduce the number of superfluous refund and customer service interactions that would necessarily result when someone accidentally pays for the same non-physical item twice.

Comments

rszrama’s picture

Status: Active » Postponed

Sounds like a worthy feature request and a good way to generate extra revenue, but I'm going to mark this postponed as we're not adding new features to 2.x any more.

tr’s picture

Version: 6.x-2.x-dev » 7.x-3.x-dev
Status: Postponed » Active

Re-opening now that the 7.x-3.x branch exists.

Island Usurper’s picture

Status: Active » Needs review
StatusFileSize
new2.75 KB

Hmm. I think this would actually be good to go into 2.x as well, but we can test it here, too.

tr’s picture

Patch applies, but I can't test it because of #779036: Legacy calls to db_last_insert_id().

Island Usurper’s picture

StatusFileSize
new2.75 KB

The return value for hook_uc_add_to_cart() needs nested arrays so that module_invoke_all() can merge the results correctly.

tr’s picture

Status: Needs review » Reviewed & tested by the community
StatusFileSize
new2.86 KB

Re-rolled patch against current 7.x-3.x-dev. I've tested it and it seem sto work.

longwave’s picture

Status: Reviewed & tested by the community » Fixed

Committed.

tr’s picture

Version: 7.x-3.x-dev » 6.x-2.x-dev
Status: Fixed » Patch (to be ported)

Let's get it in 2.x too.

tr’s picture

Status: Patch (to be ported) » Fixed

Patch ported, tested, and committed to 6.x-2.x.

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.