Bug 4942 - When a new patron is created and the expiry date is left blank, it comes up with an invalid date
Summary: When a new patron is created and the expiry date is left blank, it comes up w...
Status: CLOSED FIXED
Alias: None
Product: Koha
Classification: Unclassified
Component: Patrons (show other bugs)
Version: Main
Hardware: All All
: PATCH-Sent (DO NOT USE) normal (vote)
Assignee: Robin Sheat
QA Contact: Bugs List
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2010-07-06 06:38 UTC by Robin Sheat
Modified: 2012-10-25 23:04 UTC (History)
3 users (show)

See Also:
Change sponsored?: ---
Patch complexity: ---
Documentation contact:
Documentation submission:
Text to go in the release notes:
Version(s) released in:


Attachments
proposed patch (1.55 KB, patch)
2010-07-07 04:44 UTC, Robin Sheat
Details | Diff | Splinter Review

Note You need to log in before you can comment on or make changes to this bug.
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