| Summary: | Double-clicking the 'renew' button on circulation.pl will double-charge account management fee | ||
|---|---|---|---|
| Product: | Koha | Reporter: | Barton Chittenden <barton> |
| Component: | Circulation | Assignee: | Bugs List <koha-bugs> |
| Status: | NEW --- | QA Contact: | Testopia <testopia> |
| Severity: | normal | ||
| Priority: | P5 - low | CC: | bdaeuber, clackman, gmcharlt, kyle, veron |
| Version: | Main | ||
| Hardware: | All | ||
| OS: | All | ||
| GIT URL: | Initiative type: | --- | |
| Sponsorship status: | --- | Comma delimited list of Sponsors: | |
| Crowdfunding goal: | 0 | Patch complexity: | --- |
| Documentation contact: | Documentation submission: | ||
| Text to go in the release notes: | Version(s) released in: | ||
| Circulation function: | |||
|
Description
Barton Chittenden
2016-02-10 21:43:37 UTC
Confirmed. Rather than try to address this in the template I wonder if it would make sense to add a check to the renewal process to abort the process if the patron's expiration date is already set to the date calculated by the renew_account? (In reply to Owen Leonard from comment #2) > Rather than try to address this in the template I wonder if it would make > sense to add a check to the renewal process to abort the process if the > patron's expiration date is already set to the date calculated by the > renew_account? I agree with this. There's really no reason I can think of the renew to the same day. |