Bug 33156: Fix JS error on batch patron modification
authorKatrin Fischer <katrin.fischer.83@web.de>
Sun, 19 Mar 2023 22:06:12 +0000 (22:06 +0000)
committerTomas Cohen Arazi <tomascohen@theke.io>
Fri, 5 May 2023 12:13:18 +0000 (09:13 -0300)
commit7ca83d4b8c5581646af50d8378003b4233ad5fa2
tree2c7f598ac1b571aca8dc34f08204fe770222a3a5
parenta104722b9d4150c48eb4793c123d10e66144a7c7
Bug 33156: Fix JS error on batch patron modification

The error currently breaks some of the functionality for
updating patron attributes linked to AV values. With the
bug, the input field will not change to the AV list when selected
from the pull down of patron attributes.

It was probably bug 29648 that changed the data structure
without adapting for the additional data that is pushed
to the object for the patron attribute columns.

To test:
* Create several extended patron attributes
  * Make sure at least one is linked to an AV
  * Repeatable and unique patron attributes can't be
  updated in batch patron modification, so you need some
  that can be modified.
* Go to tools > batch patron modification
* Enter one or more cardnumbers to update
* Verify the table is missing columns, export and configure
* Verify when selecting the AV tied patron attribute, the input
  remains a text one
* Apply patch
* Both should work now as expected

Note: I believe there are some more errors on this page,
but they should better be handled separately. So with this
patch we push configuration again for each column to table
columns, but the very last column keeps missing from the
columns settings.

Signed-off-by: David Nind <david@davidnind.com>
Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org>
Signed-off-by: Tomas Cohen Arazi <tomascohen@theke.io>
koha-tmpl/intranet-tmpl/prog/en/modules/tools/modborrowers.tt