When duplicating the patron in the top section only the Surname should carry over - right now the other name and initials carry over. Along the same lines at the bottom the expiration date is coming over and that could be different based on when you're doing the duplication and your patron category rules.
I see that gender, OPAC & Staff notes, and restrictions also carry over. I have confirmed all the Nicole has mentioned, and agree that copying the expiration is a problem. Christopher Brannon
Created attachment 31903 [details] [review] Bug 7878 - When duplicating a patron too many fields carrying over When duplicating a patron the resulting entry form contains the data from the original patron in the following fields: Other names, Initials, Gender, OPAC note, Staff note, and expiration date. These should be blank for a new duplicated patron. This patch corrects this. To test, locate or create a patron record which contains data in the fields listed above. Duplicate that patron and confirm that those fields are blank in the resulting patron entry form. Confirm also that editing an existing patron with data in those fields works correctly.
Created attachment 32001 [details] [review] [SIGNED OFF] Bug 7878 - When duplicating a patron too many fields carrying over When duplicating a patron the resulting entry form contains the data from the original patron in the following fields: Other names, Initials, Gender, OPAC note, Staff note, and expiration date. These should be blank for a new duplicated patron. This patch corrects this. To test, locate or create a patron record which contains data in the fields listed above. Duplicate that patron and confirm that those fields are blank in the resulting patron entry form. Confirm also that editing an existing patron with data in those fields works correctly. Signed-off-by: Nick Clemens <nick@quecheelibrary.org>
Is there a reason not to use UNLESS instead of "IF do nothing ELSE do something"?
(In reply to Jonathan Druart from comment #4) > Is there a reason not to use UNLESS instead of "IF do nothing ELSE do > something"? Probably not, just following the pattern already established in the template.
Created attachment 32234 [details] [review] Bug 7878 - When duplicating a patron too many fields carrying over When duplicating a patron the resulting entry form contains the data from the original patron in the following fields: Other names, Initials, Gender, OPAC note, Staff note, and expiration date. These should be blank for a new duplicated patron. This patch corrects this. To test, locate or create a patron record which contains data in the fields listed above. Duplicate that patron and confirm that those fields are blank in the resulting patron entry form. Confirm also that editing an existing patron with data in those fields works correctly. Signed-off-by: Nick Clemens <nick@quecheelibrary.org> Signed-off-by: Jonathan Druart <jonathan.druart@biblibre.com>
Created attachment 32235 [details] [review] Bug 7878: (qa follow-up) Fix the sex default value Without the previous patch, when you edited a patron, the "N/A" value was selected for sex. With, no checkboxes were selected. This patch also replaces the IF opduplicate do nothing ELSE do something END statement with UNLESS opduplicate do something END which is more readable. Signed-off-by: Jonathan Druart <jonathan.druart@biblibre.com>
Patches pushed to master. Thanks Owen!