| 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: | --- | 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  |