Summary: | QuickAdd skips fields from PatronQuickAddFields in some cases | ||
---|---|---|---|
Product: | Koha | Reporter: | Slava Shishkin <slavashishkin> |
Component: | Patrons | Assignee: | Slava Shishkin <slavashishkin> |
Status: | Signed Off --- | QA Contact: | Testopia <testopia> |
Severity: | enhancement | ||
Priority: | P5 - low | CC: | chloe.zermatten, gmcharlt, kyle.m.hall |
Version: | unspecified | ||
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: | ||
Attachments: |
Bug 39828: Fix QuickAdd skipping fields
Bug 39828: Fix QuickAdd skipping fields |
Description
Slava Shishkin
2025-05-05 09:18:46 UTC
Created attachment 181902 [details] [review] Bug 39828: Fix QuickAdd skipping fields Currently, fields like userid are skipped if the last input in the main form is hidden or has no ID. This patch fixes the selection logic to avoid skipping. Steps to reproduce: 1. Add userid (or any other field that matches above structure) to PatronQuickAddFields. 2. Open the Add Patron form and use Quick Add. 3. Observe → the field is missing. 4. Apply the patch. 5. Use Quick Add again and observe that 'Username' field is there. Created attachment 181972 [details] [review] Bug 39828: Fix QuickAdd skipping fields Currently, fields like userid are skipped if the last input in the main form is hidden or has no ID. This patch fixes the selection logic to avoid skipping. Steps to reproduce: 1. Add userid (or any other field that matches above structure) to PatronQuickAddFields. 2. Open the Add Patron form and use Quick Add. 3. Observe → the field is missing. 4. Apply the patch. 5. Use Quick Add again and observe that 'Username' field is there. Signed-off-by: Chloe Zermatten <chloe.zermatten@openfifth.co.uk> |