Summary: | Saving edits to a patron - error message clears or resets some fields | ||
---|---|---|---|
Product: | Koha | Reporter: | David Nind <david> |
Component: | Patrons | Assignee: | Bugs List <koha-bugs> |
Status: | NEW --- | QA Contact: | Testopia <testopia> |
Severity: | major | ||
Priority: | P1 - high | CC: | emmi.takkinen, gmcharlt, kyle.m.hall |
Version: | Main | ||
Hardware: | All | ||
OS: | All | ||
GIT URL: | Change sponsored?: | --- | |
Patch complexity: | --- | Documentation contact: | |
Documentation submission: | Text to go in the release notes: | ||
Version(s) released in: | Circulation function: |
Description
David Nind
2024-07-07 09:06:10 UTC
This seems to happen with fields that receive their value from patron object. In surname field we have value="[% patron.surname | html %]", but in firstname field its value="[% borrower_data.firstname | html UNLESS op == 'duplicate' %]". If I change surname field value attribute as value="[% borrower_data.surname | html %]", value retains. This is a tricky one. Template_param patron is used to fill out specific fields with "Add guarantee" functionality (controlled by PrefillGuaranteeField syspref). If we replace it with borrower_data, we lose ability to prefill inputs with guarantors info. I'm not sure why patron param uses %newdata instead of %data and if changing it will break something. Should test. (In reply to Emmi Takkinen from comment #2) > This is a tricky one. Template_param patron is used to fill out specific > fields with "Add guarantee" functionality (controlled by > PrefillGuaranteeField syspref). If we replace it with borrower_data, we lose > ability to prefill inputs with guarantors info. I'm not sure why patron > param uses %newdata instead of %data and if changing it will break > something. Should test. Tested, changing %newdata as %data also breaks "Add guarantee" :D |