Summary: | Barcode column is missing from "Holds to pull" table preferences yaml file | ||
---|---|---|---|
Product: | Koha | Reporter: | Peter Vashchuk <stalkernoid> |
Component: | Circulation | Assignee: | Peter Vashchuk <stalkernoid> |
Status: | CLOSED FIXED | QA Contact: | Testopia <testopia> |
Severity: | minor | ||
Priority: | P5 - low | CC: | caroline.cyr-la-rose, fridolin.somers, gmcharlt, kyle.m.hall, nugged, victor |
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: |
21.05.00,20.11.03
|
|
Circulation function: | |||
Bug Depends on: | 23485 | ||
Bug Blocks: | |||
Attachments: |
Bug 27655: Add barcode "columnname" to "column_settings.yml"
Bug 27655: Add barcode "columnname" to "column_settings.yml" |
Description
Peter Vashchuk
2021-02-08 14:09:12 UTC
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! Pushed to 20.11.x for 20.11.03 Missing dependencies for 19.11.x, it shouldn't be affected, no backport. |