Bug 4254

Summary: adding new patron advanced notice doesn't save
Product: Koha Reporter: Nicole C. Engard <nengard>
Component: NoticesAssignee: Chris Nighswonger <cnighswonger>
Status: CLOSED FIXED QA Contact: Bugs List <koha-bugs>
Severity: critical    
Priority: PATCH-Sent (DO NOT USE) CC: chris, cnighswonger
Version: Main   
Hardware: PC   
OS: All   
URL: cgi-bin/koha/members/memberentry.pl?op=add&amp;categorycode=PT
Change sponsored?: --- Patch complexity: ---
Documentation contact: Documentation submission:
Text to go in the release notes:
Version(s) released in:
Attachments: adding patron with advanced notice set
new patron saved without advanced notice selected
Patch

Description Chris Cormack 2010-05-21 01:24:59 UTC


---- Reported by nengard@gmail.com 2010-02-23 17:27:16 ----

Created an attachment
adding patron with advanced notice set

When adding a new patron if I choose to have that patron receive an Advance notice (no matter what comboo I use) that doesn't save when the patron is created.  I have to edit the patron to get it to actually save.



---- Additional Comments From nengard@gmail.com 2010-02-23 17:29:54 ----

Created an attachment
new patron saved without advanced notice selected 





--- Bug imported by chris@bigballofwax.co.nz 2010-05-21 01:24 UTC  ---

This bug was previously known as _bug_ 4254 at http://bugs.koha.org/cgi-bin/bugzilla3/show_bug.cgi?id=4254
Imported an attachment (id=1954)
Imported an attachment (id=1955)

Actual time not defined. Setting to 0.0
Setting qa contact to the default for this product.
   This bug either had no qa contact or an invalid one.
The original submitter of attachment 1954 [details] is unknown.
   Reassigning to the person who moved it here: chris@bigballofwax.co.nz.
The original submitter of attachment 1955 [details] is unknown.
   Reassigning to the person who moved it here: chris@bigballofwax.co.nz.

Comment 1 Chris Nighswonger 2010-10-30 06:29:03 UTC
Created attachment 2717 [details] [review]
Patch
Comment 2 Chris Cormack 2010-11-04 08:02:55 UTC
Patch pushed, please test and close
Comment 3 Chris Nighswonger 2010-11-18 03:15:43 UTC
This bug has a patch presently committed to HEAD and 3.2.x.

Please take the appropriate action to update the status of this bug.

Remember, bugs which have been resolved should be tested and marked closed, preferably by the original reporter.
Comment 4 Chris Nighswonger 2010-11-30 15:54:55 UTC
A fix for this bug has been committed to the current development HEAD as well as released in 3.2.1.