| 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 |
| Version: | Main | ||
| Hardware: | All | ||
| OS: | All | ||
| GIT URL: | Initiative type: | --- | |
| Sponsorship status: | --- | Comma delimited list of Sponsors: | |
| Crowdfunding goal: | 0 | 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 |