Bug 4942

Summary: When a new patron is created and the expiry date is left blank, it comes up with an invalid date
Product: Koha Reporter: Robin Sheat <robin>
Component: PatronsAssignee: Robin Sheat <robin>
Status: CLOSED FIXED QA Contact: Bugs List <koha-bugs>
Severity: normal    
Priority: PATCH-Sent (DO NOT USE) CC: chris, gmcharlt, katrin.fischer
Version: Main   
Hardware: All   
OS: All   
Change sponsored?: --- Patch complexity: ---
Documentation contact: Documentation submission:
Text to go in the release notes:
Version(s) released in:
Attachments: proposed patch

Description Robin Sheat 2010-07-06 06:38:14 UTC
To reproduce:
1. Create a new patron, leave the expiry date blank (which the text claims will auto calculate)
2. Note that the expiry date for the new patron is something like '00/02/0'.

Expected result:
1. The expiry date will be a valid date.

Note:
* Need to decide on what a good default date is.
Comment 1 Owen Leonard 2010-07-06 12:58:26 UTC
> * Need to decide on what a good default date is.

Each patron category has an enrollment period. The expiration date is supposed to be getting populated based on that number (today + XX months).
Comment 2 Katrin Fischer 2010-07-06 14:14:43 UTC
As alternative to the enrollment period there is a new feature for an 'until date' in 3.2. If there is an 'until date', it is the expiry date.
Comment 3 Robin Sheat 2010-07-06 22:50:54 UTC
Thanks, yeah, I found that after a bit more poking around.
Comment 4 Robin Sheat 2010-07-07 04:06:45 UTC
This seems to actually trace back to the default when you create a new patron category. If you don't set it, it starts with a broken default.
Comment 5 Robin Sheat 2010-07-07 04:44:00 UTC
Created attachment 2317 [details] [review]
proposed patch
Comment 6 Chris Cormack 2011-01-01 20:32:05 UTC
Pushed please test