https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=35451 --- Comment #26 from David Nind <david@davidnind.com> --- Testing notes (using KTD) - main aim was to make sure everything works as expected: 1. Apply the patch. 2. Update the database: updatedatabase 3. Restart everything: restart_all 4. Run the tests - these should pass: prove t/db_dependent/Koha/Objects/Mixin/AdditionalFields.t 5. Make sure additional fields for acquisition order baskets continue to work as expected: 5.1 Go to Administration > Additional parameters > Additional fields 5.2 Select the table to add additional fields to (in this case, Order baskets (aqbasket)) 5.3 Add a new field: 5.3.1 + New field 5.3.2 Name: TESTFIELD 5.3.3 Repeatable: select 5.3.4 Searchable: select 5.4 Add a new basket to "My Vendor" using the additional field 5.4.1 Go to Acquisitions 5.4.2 In the "Search vendors" section, click "Search" 5.4.3 Add a new order basket: + New > Basket . Basket name: bz35451 . TESTFIELD (add two entries): This is a test + This is a second test 5.4.4 Check that you can edit and delete the entries for your additional field 5.5 Bonus (optional): add different types of additional fields (such as authorized values, or not repeatable) and test that they work SQL === Before the patch, table for additional_field_values - no record_table column: MariaDB [koha_kohadev]> select * from additional_field_values; +----+----------+-----------+-----------------------+ | id | field_id | record_id | value | +----+----------+-----------+-----------------------+ | 1 | 1 | 2 | This is a test | | 2 | 1 | 2 | This is a second test | +----+----------+-----------+-----------------------+ 2 rows in set (0.000 sec) After patch, table for additional_field_values - now has a record_table column: MariaDB [koha_kohadev]> select * from additional_field_values;; +----+----------+--------------+-----------+-----------------------+ | id | field_id | record_table | record_id | value | +----+----------+--------------+-----------+-----------------------+ | 5 | 1 | aqbasket | 2 | This is a test | | 6 | 1 | aqbasket | 2 | This is a second test | +----+----------+--------------+-----------+-----------------------+ 2 rows in set (0.000 sec) -- You are receiving this mail because: You are watching all bug changes.