Created attachment 105490 [details] differences With the recent change of the Guarantors in Koha, the Contact section of the Patron Detail page (when editing) is confusing. Prior to 19.11, the Contact section was created for the Contact details of the Patron, phone, email, fax. Now in 19.11, the Guarantor (Non-Koha) has been added to the contact section, causing questions if this is the Guarantor's contact information or the primary patron's contact information. It would be nice if there could be a separation of the two areas in the Koha patrons form (memberentry.pl) Attaching an image of the before 19.11 and 19.11 contact section.
+1, this is confusing
+1 I agree that the non-patron Guarantor information should go in the 'Guarantor information' section rather than the 'Contact' section. We could hide it behind a button. I would put a button next to the 'Search to add' button that says something like 'Guarantor is not a patron'. Or have it like pre-19.11 where the non-patron fields are always showing and you can click on 'Search to add' to fill them, but I admit that I didn't find that it was very intuitive...
+1 it seems like the non-patron Guarantor should be with patron Guarantors
Is the implication if we move those fields that the non-patron guarantors will not have contact information?
(In reply to Benjamin Daeuber from comment #4) > Is the implication if we move those fields that the non-patron guarantors > will not have contact information? The contact information is that of the patron, not the guarantor. That is the main confusion reported by Kelly. I also find it confusing to have a section named 'Guarantor information', and then other fields named 'Guarantor something' that are not in that section.
I guess my point is that (In reply to Caroline Cyr La Rose from comment #5) > The contact information is that of the patron, not the guarantor. That is > the main confusion reported by Kelly. > I think we need additional contact fields if this is the case. A whole separate section for non-patron guarantor contact information labeled "Non-Patron Guarantor". Right now there is essentially one set of contact info and it appears like it could belong to the non-patron guarantor who doesn't actually have any contact info.
Pre-19.11, the alternate contact section would be used for that. But I agree that yours is a good idea. That would mean moving the Alternate contact section to the 'Guarantor information' section and only making it appear if we choose 'Non-patron guarantor', I guess?
(In reply to Caroline Cyr La Rose from comment #7) > Pre-19.11, the alternate contact section would be used for that. > > But I agree that yours is a good idea. That would mean moving the Alternate > contact section to the 'Guarantor information' section and only making it > appear if we choose 'Non-patron guarantor', I guess? We use the alternate contact for adults that have guarantors, but which we can't actually add guarantors to (which is a whole other issue). So I think we need a whole new set of contact information. And I'm not sure of the original intent of the development, but as it stands now, I think this is a serious issue if we are indeed distinguishing between alternate contact and non-patron guarantor contact. And if we're not, why add the non-patron guarantor feature at all?
> And I'm not sure of the original intent of the development, but as it stands > now, I think this is a serious issue if we are indeed distinguishing between > alternate contact and non-patron guarantor contact. And if we're not, why > add the non-patron guarantor feature at all? Pre-19.11 we could add non-patron guarantors in the guarantor information (only first and last names). I think that's why we want to keep it. I suggest you go give a +1 to bug 12446 which is to allow adults (or any patron category in fact) to have guarantors.
Is there a sense of who's using the alternate contact fields other than for guarantor information? We use the alternate address for college students and people who might own property in town but actually live out of town. We mostly use the alternate contact field as described above, but I would hate to shoehorn guarantor information into those fields if they are in use otherwise.
Guarantors are usually assumed to be parents or guardians. That's why there is the option to show the guarantee's checkouts to the guarantor. Having a separate alternate contact allows there to be a greater separation between the patron and the contact.
Created attachment 106351 [details] [review] Bug 25654: Make the contact and non-patron guarantor sections separate on patron entry form This patch reverses what I think was an unintended change made by Bug 14570 in which the non-patron guarantor fields are shown for all patron types (not just child patrons) and grouped with the patron contact fields. This patch creates a new "Non-patron guarantor" fieldset to show along with "Patron guarantor" when adding or editing child patrons. To test, apply the patch check that the BorrowerUnwantedField system preference does not contain contactname, contactfirstname, or relationship. - Open an adult patron for editing. You should see no guarantor-related fields. - Open a child patron for editing. There should be two guarantor sections: "Patron guarantor" and "Non-patron guarantor." - Test that adding patron guarantors and non-patron guarantors works correctly. - Add "contactname|contactfirstname|relationship" to the BorrowerUnwantedField system preference. - Reload the child patron entry form. The non-patron guarantor section should now be hidden.
*** Bug 25742 has been marked as a duplicate of this bug. ***
Created attachment 106355 [details] [review] Bug 25654: Make the contact and non-patron guarantor sections separate on patron entry form This patch reverses what I think was an unintended change made by Bug 14570 in which the non-patron guarantor fields are shown for all patron types (not just child patrons) and grouped with the patron contact fields. This patch creates a new "Non-patron guarantor" fieldset to show along with "Patron guarantor" when adding or editing child patrons. To test, apply the patch check that the BorrowerUnwantedField system preference does not contain contactname, contactfirstname, or relationship. - Open an adult patron for editing. You should see no guarantor-related fields. - Open a child patron for editing. There should be two guarantor sections: "Patron guarantor" and "Non-patron guarantor." - Test that adding patron guarantors and non-patron guarantors works correctly. - Add "contactname|contactfirstname|relationship" to the BorrowerUnwantedField system preference. - Reload the child patron entry form. The non-patron guarantor section should now be hidden.
Created attachment 106456 [details] [review] Bug 25654: Make the contact and non-patron guarantor sections separate on patron entry form This patch reverses what I think was an unintended change made by Bug 14570 in which the non-patron guarantor fields are shown for all patron types (not just child patrons) and grouped with the patron contact fields. This patch creates a new "Non-patron guarantor" fieldset to show along with "Patron guarantor" when adding or editing child patrons. To test, apply the patch check that the BorrowerUnwantedField system preference does not contain contactname, contactfirstname, or relationship. - Open an adult patron for editing. You should see no guarantor-related fields. - Open a child patron for editing. There should be two guarantor sections: "Patron guarantor" and "Non-patron guarantor." - Test that adding patron guarantors and non-patron guarantors works correctly. - Add "contactname|contactfirstname|relationship" to the BorrowerUnwantedField system preference. - Reload the child patron entry form. The non-patron guarantor section should now be hidden. Signed-off-by: Lucas Gass <lucas@bywatersolutions.com>
Created attachment 106457 [details] [review] Bug 25654: Make the contact and non-patron guarantor sections separate on patron entry form This patch reverses what I think was an unintended change made by Bug 14570 in which the non-patron guarantor fields are shown for all patron types (not just child patrons) and grouped with the patron contact fields. This patch creates a new "Non-patron guarantor" fieldset to show along with "Patron guarantor" when adding or editing child patrons. To test, apply the patch check that the BorrowerUnwantedField system preference does not contain contactname, contactfirstname, or relationship. - Open an adult patron for editing. You should see no guarantor-related fields. - Open a child patron for editing. There should be two guarantor sections: "Patron guarantor" and "Non-patron guarantor." - Test that adding patron guarantors and non-patron guarantors works correctly. - Add "contactname|contactfirstname|relationship" to the BorrowerUnwantedField system preference. - Reload the child patron entry form. The non-patron guarantor section should now be hidden. Signed-off-by: Lucas Gass <lucas@bywatersolutions.com> Signed-off-by: Christopher Brannon <cbrannon@cdalibrary.org>
Clean and straight forward. Thanks for fixing this!
Created attachment 108363 [details] [review] Bug 25654: Make the contact and non-patron guarantor sections separate on patron entry form This patch reverses what I think was an unintended change made by Bug 14570 in which the non-patron guarantor fields are shown for all patron types (not just child patrons) and grouped with the patron contact fields. This patch creates a new "Non-patron guarantor" fieldset to show along with "Patron guarantor" when adding or editing child patrons. To test, apply the patch check that the BorrowerUnwantedField system preference does not contain contactname, contactfirstname, or relationship. - Open an adult patron for editing. You should see no guarantor-related fields. - Open a child patron for editing. There should be two guarantor sections: "Patron guarantor" and "Non-patron guarantor." - Test that adding patron guarantors and non-patron guarantors works correctly. - Add "contactname|contactfirstname|relationship" to the BorrowerUnwantedField system preference. - Reload the child patron entry form. The non-patron guarantor section should now be hidden. Signed-off-by: Lucas Gass <lucas@bywatersolutions.com> Signed-off-by: Christopher Brannon <cbrannon@cdalibrary.org> Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>
I really like this change, thx Owen!
Pushed to master for 20.11, thanks to everybody involved!
enhancement will not be backported to 20.05.x
Hi, Is it possible to backport this one to 19.11? It's really confusing staff members and I think I have a working patch