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.
| Comment | File | Size | Author |
|---|---|---|---|
| #6 | 445206.patch | 2.86 KB | tr |
| #5 | 445206_files_forever.patch | 2.75 KB | Island Usurper |
| #3 | 445206_files_forever.patch | 2.75 KB | Island Usurper |
Comments
Comment #1
rszrama commentedSounds 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.
Comment #2
tr commentedRe-opening now that the 7.x-3.x branch exists.
Comment #3
Island Usurper commentedHmm. I think this would actually be good to go into 2.x as well, but we can test it here, too.
Comment #4
tr commentedPatch applies, but I can't test it because of #779036: Legacy calls to db_last_insert_id().
Comment #5
Island Usurper commentedThe return value for hook_uc_add_to_cart() needs nested arrays so that module_invoke_all() can merge the results correctly.
Comment #6
tr commentedRe-rolled patch against current 7.x-3.x-dev. I've tested it and it seem sto work.
Comment #7
longwaveCommitted.
Comment #8
tr commentedLet's get it in 2.x too.
Comment #9
tr commentedPatch ported, tested, and committed to 6.x-2.x.