Bug 38738

Summary: additional-fields-entry.inc always closing select element even when it doesn't exist
Product: Koha Reporter: Pedro Amorim <pedro.amorim>
Component: System AdministrationAssignee: Pedro Amorim <pedro.amorim>
Status: Pushed to main --- QA Contact: Jonathan Druart <jonathan.druart>
Severity: minor    
Priority: P5 - low CC: gmcharlt, pedro.amorim
Version: Main   
Hardware: All   
OS: All   
Change sponsored?: --- Patch complexity: Trivial patch
Documentation contact: Documentation submission:
Text to go in the release notes:
Version(s) released in:
25.05.00
Circulation function:
Bug Depends on: 35044    
Bug Blocks:    
Attachments: Bug 38738: Only close select element if needed
Bug 38738: [TEST ONLY] Add UI message for test
Bug 38738: [TEST ONLY] Add UI message for test
Bug 38738: Only close select element if needed

Description Pedro Amorim 2024-12-18 11:08:00 UTC

    
Comment 1 Pedro Amorim 2024-12-18 11:08:47 UTC
Created attachment 175641 [details] [review]
Bug 38738: Only close select element if needed

Preparation: Create additional fields for table 'subscription', visit:
/cgi-bin/koha/admin/additional-fields.pl?tablename=subscription

2 text fields, one repeatable, one not-repeatable
2 AV fields, one repeatable, one not-repeatable
2 MARC fields, one 'get' and one 'set', both non-repeatable, MARC field 942$c

Test plan, apply both patches:
1) Add a new serial subscription, visit:
/cgi-bin/koha/serials/subscription-add.pl
2) Set the mandatory "Record" input (e.g. '112'). Click the 'Next' and press 'Ok' on the alert box.
3) Notice the message 'This message should only show if a select input is present' only shows for select inputs, i.e. it's only closing the element when it needs to
Comment 2 Pedro Amorim 2024-12-18 11:08:50 UTC
Created attachment 175642 [details] [review]
Bug 38738: [TEST ONLY] Add UI message for test
Comment 3 David Nind 2024-12-18 21:04:35 UTC
Created attachment 175680 [details] [review]
Bug 38738: [TEST ONLY] Add UI message for test

Signed-off-by: David Nind <david@davidnind.com>
Comment 4 Jonathan Druart 2025-01-06 10:28:38 UTC
Created attachment 176148 [details] [review]
Bug 38738: Only close select element if needed

Preparation: Create additional fields for table 'subscription', visit:
/cgi-bin/koha/admin/additional-fields.pl?tablename=subscription

2 text fields, one repeatable, one not-repeatable
2 AV fields, one repeatable, one not-repeatable
2 MARC fields, one 'get' and one 'set', both non-repeatable, MARC field 942$c

Test plan, apply both patches:
1) Add a new serial subscription, visit:
/cgi-bin/koha/serials/subscription-add.pl
2) Set the mandatory "Record" input (e.g. '112'). Click the 'Next' and press 'Ok' on the alert box.
3) Notice the message 'This message should only show if a select input is present' only shows for select inputs, i.e. it's only closing the element when it needs to

Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org>
Comment 5 Katrin Fischer 2025-01-07 14:49:19 UTC
Pushed for 25.05!

Well done everyone, thank you!