[Bug 32504] New: Empty column name for basket/orders in Table settings
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=32504 Bug ID: 32504 Summary: Empty column name for basket/orders in Table settings Change sponsored?: --- Product: Koha Version: master Hardware: All OS: All Status: NEW Severity: minor Priority: P5 - low Component: Staff interface Assignee: slavashishkin@gmail.com Reporter: slavashishkin@gmail.com QA Contact: testopia@bugs.koha-community.org CC: gmcharlt@gmail.com -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=32504 Slava Shishkin <slavashishkin@gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|NEW |Needs Signoff -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=32504 --- Comment #1 from Slava Shishkin <slavashishkin@gmail.com> --- Created attachment 144739 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=144739&action=edit Bug 32504: Fix typo columnname in columns_settings.yml Trivial fix. To reproduce: 1. Go to /cgi-bin/koha/admin/columns_settings.pl 2. Observe in the section Acquisition > Basket > Orders an empty field in the Column name, the row between fund and supplier_report rows: it was expected estimated_delivery_date value. 3. Apply the patch. 4. Restart Memcached (console: service memcached restart) 5. Reload cgi-bin/koha/admin/columns_settings.pl: that empty row now will have estimated_delivery_date value. -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=32504 Lucas Gass <lucas@bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |lucas@bywatersolutions.com --- Comment #2 from Lucas Gass <lucas@bywatersolutions.com> --- This does correct the empty row. However when I try hiding estimated_delivery_date value via Table settings I cannot. The last few couples seem mis-configured. -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=32504 Slava Shishkin <slavashishkin@gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Summary|Empty column name for |Empty column name, |basket/orders in Table |misaligning visibility, and |settings |export for basket/orders in | |Table settings -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=32504 --- Comment #3 from Slava Shishkin <slavashishkin@gmail.com> --- Created attachment 144767 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=144767&action=edit Bug 32504: Fix misalignment of visibility of basket orders table columns and missing export columns To reproduce: 1. Go to /cgi-bin/koha/admin/columns_settings.pl 2. In the section Acquisition > Basket > Orders uncheck all checkboxes in the table settings 3. Go to a separate browser window to /cgi-bin/koha/acqui/booksellers.pl and open/create a new basket and add one book to it so the basket table will have rows in the "Order" table which we will configure below: 4. Click on the ["Columns" settings] for the table. 5. In the dropdown menu, observe that all Columns are marked as visible and correspond to the current table display. 6. Also check the export table feature: Click on "Export" and choose print, you will see that Columns up to GST (goods_and_services_tax) are displayed, and Fund and Estimated delivery date are missing. 7. Go back to table settings (like it was in steps.1-2) 8. Make the estimated_delivery_date column 'invisible by default' by checking out the appropriate checkbox and saving the changes. 9. Refresh the basket window/page (like in step 3). 10. Observe that in the table the whole columns "Estimated delivery date" is visible but "Fund" is gone. 11. Click on the ["Columns" settings] for the table. 12. In the dropdown menu observe that the wrong column is disabled ("Fund" instead of "Estimated delivery date"). 13. Apply the patch. 14. Go through steps 3-12 and make sure that everything mentioned in those steps now works correctly: when you hide any columns in columns_settings.pl they are correctly hidden for the "Order" table, as well when exporting the table, the Fund, and Estimated delivery date columns are now exported. Note: this patch also respects Koha.Preference('EDIFACT') setting and when it's enabled it also correctly hides/shows and exports the 'supplier_report' column. -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=32504 --- Comment #4 from Slava Shishkin <slavashishkin@gmail.com> --- (In reply to Lucas Gass from comment #2) Right, this is fixed. And also export was broken: missed two columns. -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=32504 Laura <Laura.escamilla@bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |Laura.escamilla@bywatersolu | |tions.com --- Comment #5 from Laura <Laura.escamilla@bywatersolutions.com> --- I am able to hide estimated_delivery_date correctly using both the Columns Settings and Table Settings. However, even if I have the column hidden, it still shows up when the table is being printed. (In reply to Slava Shishkin from comment #4)
(In reply to Lucas Gass from comment #2)
Right, this is fixed. And also export was broken: missed two columns.
-- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=32504 --- Comment #6 from Slava Shishkin <slavashishkin@gmail.com> --- Especially for this table, the code logic is hardcoded for exporting all columns regardless of view settings. This behavior is not related to this bug. (In reply to Laura from comment #5) -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=32504 Laura <Laura.escamilla@bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Needs Signoff |Signed Off --- Comment #7 from Laura <Laura.escamilla@bywatersolutions.com> --- Thank you for the clarification. All else is working well. Signed-off-by: Laura Escamilla <laura.escamilla@bywatersolutions.com> (In reply to Slava Shishkin from comment #6)
Especially for this table, the code logic is hardcoded for exporting all columns regardless of view settings. This behavior is not related to this bug.
-- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=32504 --- Comment #8 from Lucas Gass <lucas@bywatersolutions.com> --- Slava, Shouldn't we use bKohaColumnsUseNames here? Like we do on detail.tt. -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=32504 Slava Shishkin <slavashishkin@gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |slavashishkin@gmail.com -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=32504 --- Comment #9 from Slava Shishkin <slavashishkin@gmail.com> --- Lucas, thank you for your comment. In this ticket, this is a fix for an existing bug. To improve the code for basket.tt, it's better to create a new ticket depending on this ticket. (In reply to Lucas Gass from comment #8)
Slava,
Shouldn't we use bKohaColumnsUseNames here? Like we do on detail.tt.
-- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=32504 Sarah Cornell <sbcornell@cityofportsmouth.com> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |sbcornell@cityofportsmouth. | |com -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=32504 Katrin Fischer <katrin.fischer@bsz-bw.de> changed: What |Removed |Added ---------------------------------------------------------------------------- Summary|Empty column name, |Empty column name, |misaligning visibility, and |misaligning visibility, and |export for basket/orders in |export for basket/orders in |Table settings |table settings QA Contact|testopia@bugs.koha-communit |katrin.fischer@bsz-bw.de |y.org | -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=32504 Katrin Fischer <katrin.fischer@bsz-bw.de> changed: What |Removed |Added ---------------------------------------------------------------------------- Depends on| |15348 Referenced Bugs: https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=15348 [Bug 15348] Change/Edit estimated delivery date per order line -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=32504 Katrin Fischer <katrin.fischer@bsz-bw.de> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Signed Off |Passed QA Patch complexity|--- |Trivial patch -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=32504 Katrin Fischer <katrin.fischer@bsz-bw.de> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #144739|0 |1 is obsolete| | --- Comment #10 from Katrin Fischer <katrin.fischer@bsz-bw.de> --- Created attachment 145827 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=145827&action=edit Bug 32504: Fix typo columnname in columns_settings.yml Trivial fix. To reproduce: 1. Go to /cgi-bin/koha/admin/columns_settings.pl 2. Observe in the section Acquisition > Basket > Orders an empty field in the Column name, the row between fund and supplier_report rows: it was expected estimated_delivery_date value. 3. Apply the patch. 4. Restart Memcached (console: service memcached restart) 5. Reload cgi-bin/koha/admin/columns_settings.pl: that empty row now will have estimated_delivery_date value. Signed-off-by: Laura Escamilla <laura.escamilla@bywatersolutions.com> Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de> -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=32504 Katrin Fischer <katrin.fischer@bsz-bw.de> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #144767|0 |1 is obsolete| | --- Comment #11 from Katrin Fischer <katrin.fischer@bsz-bw.de> --- Created attachment 145828 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=145828&action=edit Bug 32504: Fix misalignment of visibility of basket orders table columns and missing export columns To reproduce: 1. Go to /cgi-bin/koha/admin/columns_settings.pl 2. In the section Acquisition > Basket > Orders uncheck all checkboxes in the table settings 3. Go to a separate browser window to /cgi-bin/koha/acqui/booksellers.pl and open/create a new basket and add one book to it so the basket table will have rows in the "Order" table which we will configure below: 4. Click on the ["Columns" settings] for the table. 5. In the dropdown menu, observe that all Columns are marked as visible and correspond to the current table display. 6. Also check the export table feature: Click on "Export" and choose print, you will see that Columns up to GST (goods_and_services_tax) are displayed, and Fund and Estimated delivery date are missing. 7. Go back to table settings (like it was in steps.1-2) 8. Make the estimated_delivery_date column 'invisible by default' by checking out the appropriate checkbox and saving the changes. 9. Refresh the basket window/page (like in step 3). 10. Observe that in the table the whole columns "Estimated delivery date" is visible but "Fund" is gone. 11. Click on the ["Columns" settings] for the table. 12. In the dropdown menu observe that the wrong column is disabled ("Fund" instead of "Estimated delivery date"). 13. Apply the patch. 14. Go through steps 3-12 and make sure that everything mentioned in those steps now works correctly: when you hide any columns in columns_settings.pl they are correctly hidden for the "Order" table, as well when exporting the table, the Fund, and Estimated delivery date columns are now exported. Note: this patch also respects Koha.Preference('EDIFACT') setting and when it's enabled it also correctly hides/shows and exports the 'supplier_report' column. Signed-off-by: Laura Escamilla <laura.escamilla@bywatersolutions.com> Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de> -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=32504 --- Comment #12 from Katrin Fischer <katrin.fischer@bsz-bw.de> --- Thx for the fixes Slava, I think that typo was on me. -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=32504 Tomás Cohen Arazi <tomascohen@gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Passed QA |Pushed to master Version(s)| |23.05.00 released in| | -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=32504 --- Comment #13 from Tomás Cohen Arazi <tomascohen@gmail.com> --- Pushed to master for 23.05. Nice work everyone, thanks! -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=32504 Jacob O'Mara <jacob.omara@ptfs-europe.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Pushed to master |Pushed to stable Version(s)|23.05.00 |23.05.00,22.11.03 released in| | -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=32504 --- Comment #14 from Jacob O'Mara <jacob.omara@ptfs-europe.com> --- Nice work, thanks everyone! Pushed to 22.11.x for the next release. -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=32504 Lucas Gass <lucas@bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Pushed to stable |RESOLVED Resolution|--- |FIXED --- Comment #15 from Lucas Gass <lucas@bywatersolutions.com> --- Missing dependencies for 22.05.x, no backport. -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=32504 Bug 32504 depends on bug 15348, which changed state. Bug 15348 Summary: Change/Edit estimated delivery date per order line https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=15348 What |Removed |Added ---------------------------------------------------------------------------- Status|Needs documenting |RESOLVED Resolution|--- |FIXED -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=32504 --- Comment #16 from Aleisha Amohia <aleisha@catalyst.net.nz> --- Created attachment 152704 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=152704&action=edit Bug 32504: [22.05] Fix misalignment of visibility of basket orders table columns and missing export columns To reproduce: 1. Go to /cgi-bin/koha/admin/columns_settings.pl 2. In the section Acquisition > Basket > Orders uncheck all checkboxes in the table settings 3. Go to a separate browser window to /cgi-bin/koha/acqui/booksellers.pl and open/create a new basket and add one book to it so the basket table will have rows in the "Order" table which we will configure below: 4. Click on the ["Columns" settings] for the table. 5. In the dropdown menu, observe that all Columns are marked as visible and correspond to the current table display. 6. Also check the export table feature: Click on "Export" and choose print, you will see that Columns up to GST (goods_and_services_tax) are displayed, and Fund and Estimated delivery date are missing. 7. Go back to table settings (like it was in steps.1-2) 8. Make the estimated_delivery_date column 'invisible by default' by checking out the appropriate checkbox and saving the changes. 9. Refresh the basket window/page (like in step 3). 10. Observe that in the table the whole columns "Estimated delivery date" is visible but "Fund" is gone. 11. Click on the ["Columns" settings] for the table. 12. In the dropdown menu observe that the wrong column is disabled ("Fund" instead of "Estimated delivery date"). 13. Apply the patch. 14. Go through steps 3-12 and make sure that everything mentioned in those steps now works correctly: when you hide any columns in columns_settings.pl they are correctly hidden for the "Order" table, as well when exporting the table, the Fund, and Estimated delivery date columns are now exported. Note: this patch also respects Koha.Preference('EDIFACT') setting and when it's enabled it also correctly hides/shows and exports the 'supplier_report' column. Sponsored-by: Horowhenua Libraries Trust -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=32504 Aleisha Amohia <aleisha@catalyst.net.nz> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |aleisha@catalyst.net.nz --- Comment #17 from Aleisha Amohia <aleisha@catalyst.net.nz> --- Have attached a patch for 22.05.x, not sure how the bug status should change sorry but it should get tested before it's backported! -- You are receiving this mail because: You are watching all bug changes.
participants (1)
-
bugzilla-daemon＠bugs.koha-community.org