Created attachment 116501 [details] [review] Bug 27655: Add barcode "columnname" to "column_settings.yml" Preferences of the "Holds to pull" table missing the "Barcode" column. This patch appends that column name to the "column_settings.yml" in correct order. This also led to checked out columns in this settings page that were not those which were hidden when the table displayed (i.e. columns were shifted). To reproduce: 1) Head over to /cgi-bin/koha/admin/columns_settings.pl 2) Open the collapsed "Circulation" list, go to the "Holds" page and find preferences of the "Holds to pull" table there. 3) Check that the "Barcodes" column is missing from it. 4) Mark the last few columns for not to be displayed (for example, three) and save the preferences. Remember the column names you checked out. 5) Open the circulations section, then open "Holds to pull" report when you have there some elements displayed, you should notice that hidden columns shifted by one, as well if you press "gear" icon to quickly enable/disable the columns, you will see those unchecked in list last three will be not those you marked out. 6) Apply the patch. 7) Reload the "table settings" page, check that the "Barcodes" column is present. 8) Check that on the "Circulation -> Holds to Pull" page all columns hidden accordingly and properly (as explained in step 5.
Created attachment 116527 [details] [review] Bug 27655: Add barcode "columnname" to "column_settings.yml" Preferences of the "Holds to pull" table missing the "Barcode" column. This patch appends that column name to the "column_settings.yml" in correct order. This also led to checked out columns in this settings page that were not those which were hidden when the table displayed (i.e. columns were shifted). To reproduce: 1) Head over to /cgi-bin/koha/admin/columns_settings.pl 2) Open the collapsed "Circulation" list, go to the "Holds" page and find preferences of the "Holds to pull" table there. 3) Check that the "Barcodes" column is missing from it. 4) Mark the last few columns for not to be displayed (for example, three) and save the preferences. Remember the column names you checked out. 5) Open the circulations section, then open "Holds to pull" report when you have there some elements displayed, you should notice that hidden columns shifted by one, as well if you press "gear" icon to quickly enable/disable the columns, you will see those unchecked in list last three will be not those you marked out. 6) Apply the patch. 7) Reload the "table settings" page, check that the "Barcodes" column is present. 8) Check that on the "Circulation -> Holds to Pull" page all columns hidden accordingly and properly (as explained in step 5. Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>
Obvious fix, going straight to PQA.
Pushed to master for 21.05, thanks to everybody involved!
Surely depends on Bug 23485
Pushed to 20.11.x for 20.11.03
Missing dependencies for 19.11.x, it shouldn't be affected, no backport.