https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=41604 Caroline Cyr La Rose <caroline.cyr-la-rose@inlibro.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Keywords| |regression Depends on| |41314 Referenced Bugs: https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=41314 [Bug 41314] Column visibility broken on the checkouts table -- You are receiving this mail because: You are watching all bug changes. You are the assignee for the bug.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=41604 Hammat wele <hammat.wele@inlibro.com> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |hammat.wele@inlibro.com Assignee|koha-bugs@lists.koha-commun |hammat.wele@inlibro.com |ity.org | -- You are receiving this mail because: You are watching all bug changes. You are the assignee for the bug.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=41604 Hammat wele <hammat.wele@inlibro.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=41604 David Nind <david@davidnind.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Needs Signoff |Signed Off -- You are receiving this mail because: You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=41604 Jonathan Druart <jonathan.druart@gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Signed Off |In Discussion CC| |jonathan.druart@gmail.com --- Comment #3 from Jonathan Druart <jonathan.druart@gmail.com> --- I have decided to force the visibility of columns depending on syspref's setting. Otherwise the column visibility state is broken when the syspref's value is modified. However here it depends on permission, but the problem is the same. Try to: add superlibrarian permission to edna login with edna check an item out, and on "Details" tab (moremember, not circulation!) you can see the "check in" column. Now restrict the permission to only catalogue+borrowers Refresh moremember => With this patch the checkin column is displayed So this can be considered a bug, and that's why I forced the visibility. In discussion to get an answer from Caroline, we need to discuss if we are ready for those bugs. Also if we go with this solution I don't see a good reason to not apply the same change to the "Renew" column. -- You are receiving this mail because: You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=41604 --- Comment #5 from Jonathan Druart <jonathan.druart@gmail.com> --- (In reply to Caroline Cyr La Rose from comment #4)

(In reply to Jonathan Druart from comment #3)

...

I have decided to force the visibility of columns depending on syspref's setting.

Which syspref controls the checkin column?

"However here it depends on permission, but the problem is the same." I meant that in case of the "Checkin" column the visibility does not depend on a syspref, but on a permission (circulate:circulate_remaining_permissions). -- You are receiving this mail because: You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=41604 --- Comment #7 from Caroline Cyr La Rose <caroline.cyr-la-rose@inlibro.com> --- I think if the column is hidden in the table settings, that should trump the permissions. -- You are receiving this mail because: You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=41604 --- Comment #9 from Jonathan Druart <jonathan.druart@gmail.com> --- Created attachment 192316 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=192316&action=edit Bug 41604: Centralize visibility condition in datatables.js We want to hide column if: * the feature is not accessible (pref disabled or insufficient permissions) or * the column is hidden in the settings or * the column has been hidden by the user -- You are receiving this mail because: You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=41604 Jason Robb <jrobb@sekls.org> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |jrobb@sekls.org --- Comment #11 from Jason Robb <jrobb@sekls.org> --- This is causing major issues for us. We have the check in column hidden, as we do not want staff to check anything in unless they are physically holding an item in their hand and scan the barcode. Otherwise, bad things happen. Removing the circulate_remaining_permissions appears to remove the staff member's ability to access a patron's check out screen completely, so the only option presently is to keep the check in column visible if I want a staff account to be able to conduct checkouts. I tested the patch and it appears to restore the desired behavior. Table settings to hide the check in column are respected even when the account has the circulate_remaining_permissions ticked or the account is a superlibrarian. -- You are receiving this mail because: You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=41604 --- Comment #12 from Jonathan Druart <jonathan.druart@gmail.com> --- Created attachment 194641 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=194641&action=edit Bug 41604: Use visibility_condition in other places There are 3 other places where we force the visibility of columns. They must als use this new visibility_condition flag, so they are all using it. Test plan: The 3 tables are: 1. basket with the columns edifact, modify, cancel 2. hold history with the column itemtype 3. batch edit patron with the columns checkbox, password expiration, protected -- You are receiving this mail because: You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=41604 Jonathan Druart <jonathan.druart@gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|In Discussion |Needs Signoff -- You are receiving this mail because: You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=41604 --- Comment #15 from Jason Robb <jrobb@sekls.org> --- There's a lot to look at here. I'm not getting through it all today so leaving some notes as reference. issues-table at circulation & moremember - checkin_column: AllowCirculate requires edit_borrowers and circulate_remaining_permissions to visit the proper pages - claims_returned_column: requires ClaimReturnedLostValue syspref enabled - export_column: requires ExportCircHistory syspref enabled orders table at basket - supplier_report_column: requires EDIFACT syspref enabled and an EDI account to exist - modify_column: basket must be open - cancel_column: basket must be open, or CancelOrdersInClosedBaskets syspref is enabled holdhistory-table at holdhistory - item_type_column: AllowHoldItemTypeSelection must be enabled borrowerst at batch_patron_modifications - selections_column (checkboxes): shown while editing, hidden on results - password_expiration_date_column: only visible to superlibrarians - protected_column: only visible to superlibrarians Table locations: - issues-table: Search a patron and click on "Check out" and/or "Details" tab on the left - orders table: Acquisitions > Search for "My vendor" > Click 1 basket > click "My basket" > Add an order if it doesn't have one - holdhistory-table: Search a patron and click on "Holds history" tab on the left - borrowerst (batch patron mod): Tools > Batch patron modification > enter a cardnumber (i.e. 42) To be able to see and test visibility of the various columns: * A superlibrarian account * A limited permission account with: - catalogue - edit_borrowers - circulate_remaining_permissions - all acquisitions permissions - view_holds_history - edit_patrons * These system preferences set: - ClaimReturnedLostValue: Lost (or other non-null value) - ExportCircHistory: Enable - EDIFACT: Enable - CancelOrdersInClosedBaskets: Enable - AllowHoldItemTypeSelection: Enable * An EDI account -- You are receiving this mail because: You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=41604 --- Comment #16 from Jonathan Druart <jonathan.druart@gmail.com> --- Created attachment 194998 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=194998&action=edit Bug 41604: Use visibility_condition in other places There are 3 other places where we force the visibility of columns. They must als use this new visibility_condition flag, so they are all using it. Test plan: The 3 tables are: 1. basket with the columns edifact, modify, cancel 2. hold history with the column itemtype 3. batch edit patron with the columns checkbox, password expiration, protected -- You are receiving this mail because: You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=41604 ryanb@dubcolib.org changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |ryanb@dubcolib.org -- You are receiving this mail because: You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=41604 Kristi Krueger <kkrueger@cuyahogalibrary.org> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |kkrueger@cuyahogalibrary.or | |g -- You are receiving this mail because: You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=41604 --- Comment #20 from Jason Robb <jrobb@sekls.org> --- These are the steps I took to verify table column settings were taking priority when hiding columns, while also demonstrating that syspref-controlled columns that aren't configurable in table settings continue to display correctly. This likely doesn't cover everything. Everything was done as the superlibrarian unless otherwise noted. Setup before testing: 1.) Set permissions & login for Henry - catalogue (staff access) - edit_patrons (in tools) 2.) Set system preferences so columns display in various places - ClaimReturnedLostValue: Lost (or other non-null value) - ExportCircHistory: Enable - EDIFACT: Enable - CancelOrdersInClosedBaskets: Enable - AllowHoldItemTypeSelection: Enable Check current behaviors: --Checkouts table (circulation & moremember)-- 1.) Check an item out to Henry 2.) Note the "Check in" and "Return claims" columns are visible 3.) Click "Configure" in the upper right of the table to get to table settings 4.) Check "Is hidden by default" for "checkin" and "claims_returned" 5.) Uncheck "Save configuration state on page change" (this can cause inconsistent/confusing behavior when testing) 6.) Save table settings 7.) Return to Henry's checkout screen, note those two columns are still visible 8.) Repeat steps 2-6 on Henry's Details page 9.) Navigate back to Henry's Details page, note those two columns are still visible --Holds history table-- 1.) Place a hold on a record for Henry 2.) Go to Henry's "Holds history" tab 3.) Note the "Requested item type" column 4.) Click "Configure" in the upper right of the table to get to table settings 5.) Check "Is hidden by default" for "itemtype" 6.) Uncheck "Save configuration state on page change" 7.) Save table settings 8.) Return to Henry's Holds history page, note the column is still visible --Orders table in baskets-- 1.) Add an EDI account a.) Administration > EDI accounts > New account > Submit 2.) Add an item to the existing basket a.) Acquisitions b.) Search "My vendor" c.) Click "1 basket(s)" > "My Basket" d.) Add to basket e.) Choose "From new (empty) record" > Add item > Save 3.) Note the "Supplier report" column 4.) Close the basket 5.) Note the "Supplier report" column 6.) Click "Configure" in the upper right of the table to get to table settings 7.) Check "Is hidden by default" for "supplier_report" 8.) Uncheck "Save configuration state on page change" 9.) Save 10.) Go back to the basket, note the "Supplier report" column 11.) Reopen the basket, note the "Supplier report" column --Patron batch modification table-- Note: The table settings seem to overtake permissions before the patch is applied. Logged in as Henry: 1.) Go to Tools > Batch patron modification 2.) Enter a card number (i.e. 23529000035676) and Continue 3.) Note the absence of the "Password expiration date" and "Protected" columns 4.) Note the checkbox column on the left 5.) Click Save 6.) Note the absence of the checkbox column on the left Logged in as superlibrarian: 1.) Go to Tools > Batch patron modification 2.) Enter a card number (i.e. 23529000035676) and Continue 3.) Note the "Password expiration date" and "Protected" columns 4.) Click "Configure" in the upper right of the table to get to table settings 5.) Check "Is hidden by default" for "batch_patron_edit_password_expiration_date" and "batch_patron_edit_protected" 6.) Uncheck "Save configuration state on page change" 7.) Repeat 1-2, columns are hidden 8.) Note the checkbox column on the left 9.) Click Save 10.) Note the absence of the checkbox column on the left ---> APPLY THE PATCH <--- Check new behaviors: --Checkouts table (circulation & moremember)-- 1.) Pull up Henry's circulation page 2.) "Check in" and "Return claims" columns should be hidden 3.) Pull up Henry's Details page 4.) "Check in" and "Return claims" columns should be hidden 5.) Unhide columns in the table settings and verify they display again --Holds history table-- 1.) Pull up Henry's Holds history page 2.) "Requested item type" column should be hidden 3.) Unhide column in the table settings and verify it displays again --Orders table in baskets-- 1.) Pull up the basket 2.) "Supplier report" column should be hidden 3.) Close the basket 4.) "Supplier report" column should be hidden 5.) Unhide column in the table settings and verify it displays again --Patron batch modification table-- Logged in as Henry: 1.) Go to Tools > Batch patron modification 2.) Enter a card number (i.e. 23529000035676) and Continue 3.) "Password expiration date" and "Protected" should still be hidden 4.) Note the checkbox column on the left 5.) Click Save 6.) Note the absence of the checkbox column on the left <-- currently not working, see comment 18 (https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=41604#c18) 7.) Unhide columns in the table settings and verify they are still hidden (Henry doesn't have permission to see them) <-- currently not working, see comment 19 (https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=41604#c19) Log in as superlibrarian: 1.) Go to Tools > Batch patron modification 2.) Enter a card number (i.e. 23529000035676) and Continue 3.) "Password expiration date" and "Protected" should still be hidden 4.) Note the checkbox column on the left 5.) Click Save 6.) Note the absence of the checkbox column on the left <-- currently not working, see comment 18 (https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=41604#c18) 7.) Unhide columns in the table settings and verify they display again -- You are receiving this mail because: You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=41604 David Nind <david@davidnind.com> changed: What |Removed |Added ---------------------------------------------------------------------------- See Also| |https://bugs.koha-community | |.org/bugzilla3/show_bug.cgi | |?id=42063 -- You are receiving this mail because: You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=41604 Jonathan Druart <jonathan.druart@gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |stalkernoid@gmail.com --- Comment #22 from Jonathan Druart <jonathan.druart@gmail.com> --- *** Bug 41468 has been marked as a duplicate of this bug. *** -- You are receiving this mail because: You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=41604 Jonathan Druart <jonathan.druart@gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #194998|0 |1 is obsolete| | -- You are receiving this mail because: You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=41604 --- Comment #24 from Jonathan Druart <jonathan.druart@gmail.com> --- Created attachment 195314 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=195314&action=edit Bug 41604: take visibility_condition when save state is disabled Patch from commit 35c8292 -- You are receiving this mail because: You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=41604 Raina <raina@flo.org> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |raina@flo.org -- You are receiving this mail because: You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=41604 Arthur Suzuki <arthur.suzuki@biblibre.com> changed: What |Removed |Added ---------------------------------------------------------------------------- See Also| |https://bugs.koha-community | |.org/bugzilla3/show_bug.cgi | |?id=30221 -- You are receiving this mail because: You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=41604 Jason Robb <jrobb@sekls.org> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Needs Signoff |Signed Off -- You are receiving this mail because: You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=41604 Jason Robb <jrobb@sekls.org> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #192316|0 |1 is obsolete| | -- You are receiving this mail because: You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=41604 Jason Robb <jrobb@sekls.org> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #195314|0 |1 is obsolete| | -- You are receiving this mail because: You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=41604 --- Comment #28 from Jason Robb <jrobb@sekls.org> --- Created attachment 196276 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=196276&action=edit Bug 41604: Centralize visibility condition in datatables.js We want to hide column if: * the feature is not accessible (pref disabled or insufficient permissions) or * the column is hidden in the settings or * the column has been hidden by the user Signed-off-by: Jason Robb <jrobb@sekls.org> -- You are receiving this mail because: You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=41604 --- Comment #30 from Jason Robb <jrobb@sekls.org> --- Created attachment 196278 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=196278&action=edit Bug 41604: take visibility_condition when save state is disabled Signed-off-by: Jason Robb <jrobb@sekls.org> -- You are receiving this mail because: You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=41604 --- Comment #32 from Lucas Gass (lukeg) <lucas@bywatersolutions.com> --- *** Bug 42183 has been marked as a duplicate of this bug. *** -- You are receiving this mail because: You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=41604 Jonathan Druart <jonathan.druart@gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Severity|enhancement |major --- Comment #33 from Jonathan Druart <jonathan.druart@gmail.com> --- Thanks a lot, Jason! -- You are receiving this mail because: You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=41604 Victor Grousset/tuxayo <victor@tuxayo.net> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Signed Off |Failed QA CC| |victor@tuxayo.net --- Comment #34 from Victor Grousset/tuxayo <victor@tuxayo.net> --- QA script fails with [FAIL] koha-tmpl/intranet-tmpl/prog/en/modules/acqui/basket.tt FAIL filters missing_filter at line 1427 ( const cancel_column_available = [% !closedate || Koha.Preference('CancelOrdersInClosedBaskets') ? 1 : 0 %]) [FAIL] koha-tmpl/intranet-tmpl/prog/en/modules/members/holdshistory.tt FAIL tidiness File is not tidy, please run `perl misc/devel/tidy.pl koha-tmpl/intranet-tmpl/prog/en/modules/members/holdshistory.tt` -- You are receiving this mail because: You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=41604 Jonathan Druart <jonathan.druart@gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Failed QA |Signed Off -- You are receiving this mail because: You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=41604 Victor Grousset/tuxayo <victor@tuxayo.net> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Signed Off |Failed QA --- Comment #37 from Victor Grousset/tuxayo <victor@tuxayo.net> --- 1. With the patches 2. go the existing acquisitions basket 3. add an order 4. close the basket 5. the Modify and Cancel columns are still visible even if empty (bug) 6. supplier report also (bug) 7. go back to main, restart_all 8. the 3 columns are not shown anymore -- You are receiving this mail because: You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=41604 Jonathan Druart <jonathan.druart@gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Failed QA |Signed Off --- Comment #39 from Jonathan Druart <jonathan.druart@gmail.com> --- (In reply to Victor Grousset/tuxayo from comment #37)

1. With the patches 2. go the existing acquisitions basket 3. add an order 4. close the basket 5. the Modify and Cancel columns are still visible even if empty (bug) 6. supplier report also (bug) 7. go back to main, restart_all 8. the 3 columns are not shown anymore

Oops, there was a typo. Should be fixed now. -- You are receiving this mail because: You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=41604 Victor Grousset/tuxayo <victor@tuxayo.net> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Signed Off |Failed QA -- You are receiving this mail because: You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=41604 --- Comment #41 from Jonathan Druart <jonathan.druart@gmail.com> --- Created attachment 199223 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=199223&action=edit Bug 41604: Fix force display of a column If a column must be displayed but was hidden before -- You are receiving this mail because: You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=41604 --- Comment #43 from Jonathan Druart <jonathan.druart@gmail.com> --- Can we move forward here please? -- You are receiving this mail because: You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=41604 Lucas Gass (lukeg) <lucas@bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Blocks| |39592 Referenced Bugs: https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=39592 [Bug 39592] requestspecific table on reserve/request.pl should be column configurable -- You are receiving this mail because: You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=41604 Lucas Gass (lukeg) <lucas@bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- QA Contact|testopia@bugs.koha-communit |lucas@bywatersolutions.com |y.org | -- You are receiving this mail because: You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=41604 Lucas Gass (lukeg) <lucas@bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #196275|0 |1 is obsolete| | Attachment #196276|0 |1 is obsolete| | Attachment #196277|0 |1 is obsolete| | Attachment #196278|0 |1 is obsolete| | Attachment #198514|0 |1 is obsolete| | Attachment #198925|0 |1 is obsolete| | Attachment #199223|0 |1 is obsolete| | -- You are receiving this mail because: You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=41604 --- Comment #46 from Lucas Gass (lukeg) <lucas@bywatersolutions.com> --- Created attachment 199944 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=199944&action=edit Bug 41604: Centralize visibility condition in datatables.js We want to hide column if: * the feature is not accessible (pref disabled or insufficient permissions) or * the column is hidden in the settings or * the column has been hidden by the user Signed-off-by: Jason Robb <jrobb@sekls.org> Signed-off-by: Lucas Gass <lucas@bywatersolutions.com> -- You are receiving this mail because: You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=41604 --- Comment #48 from Lucas Gass (lukeg) <lucas@bywatersolutions.com> --- Created attachment 199946 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=199946&action=edit Bug 41604: take visibility_condition when save state is disabled Signed-off-by: Jason Robb <jrobb@sekls.org> Signed-off-by: Lucas Gass <lucas@bywatersolutions.com> -- You are receiving this mail because: You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=41604 --- Comment #50 from Lucas Gass (lukeg) <lucas@bywatersolutions.com> --- Created attachment 199948 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=199948&action=edit Bug 41604: Fix typo Signed-off-by: Lucas Gass <lucas@bywatersolutions.com> -- You are receiving this mail because: You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=41604 --- Comment #52 from Lucas Gass (lukeg) <lucas@bywatersolutions.com> --- Created attachment 199995 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=199995&action=edit Bug 41604: (follow-up) Fix the visibility when visibility_condition is 0 or null, instead of false Patch from commit d6ed804 -- You are receiving this mail because: You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=41604 Pedro Amorim (ammopt) <pedro.amorim@openfifth.co.uk> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Passed QA |Failed QA CC| |pedro.amorim@openfifth.co.u | |k --- Comment #53 from Pedro Amorim (ammopt) <pedro.amorim@openfifth.co.uk> --- Hi guys, if I refresh the page, the Check in column comes back for me: To recreate: 1. Hide the checkin column by default in circulation issues-table 1.1. Go to Administration > Table settings > Circulation tab > Jump to page : circulation > Table id: issues-table 1.2. Check 'Is hidden by default' for 'checkin' 1.3. Click 'Save' 2. logout and login 3. Checkout an item to a patron 3.1. In the search bar, set to Checkout, search for Henry 3.2. In the checkout field, enter the barcode 39999000010114 and press enter --> In the table of issues, Checkin column is visible, it should be hidden by default 4. Apply the patch 5. Repeat step 2, 3 --> In the table of issues, Checkin column is not visible 6. Refresh the page, access: http://localhost:8081/cgi-bin/koha/circ/circulation.pl?borrowernumber=19 7. Click 'Show checkouts': The 'Check in' column is back again -- You are receiving this mail because: You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=41604 Jonathan Druart <jonathan.druart@gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Failed QA |ASSIGNED -- You are receiving this mail because: You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=41604 --- Comment #56 from Jonathan Druart <jonathan.druart@gmail.com> --- Created attachment 200504 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=200504&action=edit Bug 41604: Add one more Cypress test Patch from commit 4e37f26 -- You are receiving this mail because: You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=41604 --- Comment #58 from Jonathan Druart <jonathan.druart@gmail.com> --- Created attachment 200506 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=200506&action=edit Bug 41604: Hide column if the feature is disabled Example: supplier_report on the basket page -- You are receiving this mail because: You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=41604 Sally <sally.healey@cheshiresharedservices.gov.uk> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |sally.healey@cheshireshared | |services.gov.uk -- You are receiving this mail because: You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=41604 Jason Robb <jrobb@sekls.org> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #199943|0 |1 is obsolete| | Attachment #199944|0 |1 is obsolete| | Attachment #199945|0 |1 is obsolete| | Attachment #199946|0 |1 is obsolete| | Attachment #199947|0 |1 is obsolete| | Attachment #199948|0 |1 is obsolete| | Attachment #199949|0 |1 is obsolete| | Attachment #199995|0 |1 is obsolete| | Attachment #200503|0 |1 is obsolete| | Attachment #200504|0 |1 is obsolete| | Attachment #200505|0 |1 is obsolete| | Attachment #200506|0 |1 is obsolete| | -- You are receiving this mail because: You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=41604 --- Comment #61 from Jason Robb <jrobb@sekls.org> --- Created attachment 200686 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=200686&action=edit Bug 41604: Centralize visibility condition in datatables.js We want to hide column if: * the feature is not accessible (pref disabled or insufficient permissions) or * the column is hidden in the settings or * the column has been hidden by the user Signed-off-by: Jason Robb <jrobb@sekls.org> Signed-off-by: Lucas Gass <lucas@bywatersolutions.com> Signed-off-by: Jason Robb <jrobb@sekls.org> -- You are receiving this mail because: You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=41604 --- Comment #63 from Jason Robb <jrobb@sekls.org> --- Created attachment 200688 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=200688&action=edit Bug 41604: take visibility_condition when save state is disabled Signed-off-by: Jason Robb <jrobb@sekls.org> Signed-off-by: Lucas Gass <lucas@bywatersolutions.com> Signed-off-by: Jason Robb <jrobb@sekls.org> -- You are receiving this mail because: You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=41604 --- Comment #65 from Jason Robb <jrobb@sekls.org> --- Created attachment 200690 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=200690&action=edit Bug 41604: Fix typo Signed-off-by: Lucas Gass <lucas@bywatersolutions.com> Signed-off-by: Jason Robb <jrobb@sekls.org> -- You are receiving this mail because: You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=41604 --- Comment #67 from Jason Robb <jrobb@sekls.org> --- Created attachment 200692 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=200692&action=edit Bug 41604: (follow-up) Fix the visibility when visibility_condition is 0 or null, instead of false Signed-off-by: Jason Robb <jrobb@sekls.org> -- You are receiving this mail because: You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=41604 --- Comment #69 from Jason Robb <jrobb@sekls.org> --- Created attachment 200694 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=200694&action=edit Bug 41604: Add one more Cypress test Signed-off-by: Jason Robb <jrobb@sekls.org> -- You are receiving this mail because: You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=41604 --- Comment #71 from Jason Robb <jrobb@sekls.org> --- Created attachment 200696 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=200696&action=edit Bug 41604: Hide column if the feature is disabled Example: supplier_report on the basket page Signed-off-by: Jason Robb <jrobb@sekls.org> -- You are receiving this mail because: You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=41604 Jonathan Druart <jonathan.druart@gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Blocks| |42910 Referenced Bugs: https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=42910 [Bug 42910] Restore ability to save state for holdings table -- You are receiving this mail because: You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=41604 --- Comment #73 from Jonathan Druart <jonathan.druart@gmail.com> --- Created attachment 201150 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=201150&action=edit Bug 41604: Impossible to hide Checkin column in issues-table in circ/circulation.pl Bug 41314 caused a regression, making it impossible to hide the Checkin column in the issues table in the patron's circulation page. To recreate: 1. Hide the checkin column by default in circulation issues-table 1.1. Go to Administration > Table settings > Circulation tab > Jump to page : circulation > Table id: issues-table 1.2. Check 'Is hidden by default' for 'checkin' 1.3. Click 'Save' 2. logout and login 3. Checkout an item to a patron 3.1. In the search bar, set to Checkout, search for Henry 3.2. In the checkout field, enter the barcode 39999000010114 and press enter --> In the table of issues, Checkin column is visible, it should be hidden by default 4. Apply the patch 5. Repeat step 2, 3 --> In the table of issues, Checkin column is not visible Signed-off-by: David Nind <david@davidnind.com> Signed-off-by: Jason Robb <jrobb@sekls.org> Signed-off-by: Lucas Gass <lucas@bywatersolutions.com> Signed-off-by: Jason Robb <jrobb@sekls.org> -- You are receiving this mail because: You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=41604 --- Comment #75 from Jonathan Druart <jonathan.druart@gmail.com> --- Created attachment 201152 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=201152&action=edit Bug 41604: Use visibility_condition in other places There are 3 other places where we force the visibility of columns. They must als use this new visibility_condition flag, so they are all using it. Test plan: The 3 tables are: 1. basket with the columns edifact, modify, cancel 2. hold history with the column itemtype 3. batch edit patron with the columns checkbox, password expiration, protected Signed-off-by: Jason Robb <jrobb@sekls.org> Signed-off-by: Lucas Gass <lucas@bywatersolutions.com> Signed-off-by: Jason Robb <jrobb@sekls.org> -- You are receiving this mail because: You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=41604 --- Comment #77 from Jonathan Druart <jonathan.druart@gmail.com> --- Created attachment 201154 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=201154&action=edit Bug 41604: QA fixes Signed-off-by: Lucas Gass <lucas@bywatersolutions.com> Signed-off-by: Jason Robb <jrobb@sekls.org> -- You are receiving this mail because: You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=41604 --- Comment #79 from Jonathan Druart <jonathan.druart@gmail.com> --- Created attachment 201156 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=201156&action=edit Bug 41604: Fix force display of a column If a column must be displayed but was hidden before Signed-off-by: Lucas Gass <lucas@bywatersolutions.com> Signed-off-by: Jason Robb <jrobb@sekls.org> -- You are receiving this mail because: You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=41604 --- Comment #81 from Jonathan Druart <jonathan.druart@gmail.com> --- Created attachment 201158 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=201158&action=edit Bug 41604: Add Cypress tests Signed-off-by: Jason Robb <jrobb@sekls.org> -- You are receiving this mail because: You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=41604 --- Comment #83 from Jonathan Druart <jonathan.druart@gmail.com> --- Created attachment 201160 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=201160&action=edit Bug 41604: Save user decision per column when visibility changes - Modify column-visibility.dt event handler to store the visibility preference of the user - Improve _dt_force_visibility() to correctly pick the correctly visibility value depending on visibility_condition, is_hidden, and the user preference - Add _dt_get_expected_column_visibility() to guess what's the expected visibility - Force boolean true/false (vs 1/0) using !! - Rely on a global variable kohaTableUserChoices to store the user choice. It will be part of the state and store in localStorage. Signed-off-by: Jason Robb <jrobb@sekls.org> -- You are receiving this mail because: You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=41604 --- Comment #85 from Jonathan Druart <jonathan.druart@gmail.com> --- Yet another rebase. -- You are receiving this mail because: You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=41604 --- Comment #87 from Lucas Gass (lukeg) <lucas@bywatersolutions.com> --- Looks good, test coverage is OK, everything seems to work when I test in the UI. One minor thing I found: function _dt_force_visibility() sets the variable "var columns_settings" but it is not used in the function. Do we need it? -- You are receiving this mail because: You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=41604 Lucas Gass (lukeg) <lucas@bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #201150|0 |1 is obsolete| | Attachment #201151|0 |1 is obsolete| | Attachment #201152|0 |1 is obsolete| | Attachment #201153|0 |1 is obsolete| | Attachment #201154|0 |1 is obsolete| | Attachment #201155|0 |1 is obsolete| | Attachment #201156|0 |1 is obsolete| | Attachment #201157|0 |1 is obsolete| | Attachment #201158|0 |1 is obsolete| | Attachment #201159|0 |1 is obsolete| | Attachment #201160|0 |1 is obsolete| | Attachment #201161|0 |1 is obsolete| | Attachment #201170|0 |1 is obsolete| | Attachment #201444|0 |1 is obsolete| | -- You are receiving this mail because: You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=41604 --- Comment #90 from Lucas Gass (lukeg) <lucas@bywatersolutions.com> --- Created attachment 201450 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=201450&action=edit Bug 41604: Centralize visibility condition in datatables.js We want to hide column if: * the feature is not accessible (pref disabled or insufficient permissions) or * the column is hidden in the settings or * the column has been hidden by the user Signed-off-by: Jason Robb <jrobb@sekls.org> Signed-off-by: Lucas Gass <lucas@bywatersolutions.com> Signed-off-by: Jason Robb <jrobb@sekls.org> Signed-off-by: Lucas Gass <lucas@bywatersolutions.com> -- You are receiving this mail because: You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=41604 --- Comment #92 from Lucas Gass (lukeg) <lucas@bywatersolutions.com> --- Created attachment 201452 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=201452&action=edit Bug 41604: take visibility_condition when save state is disabled Signed-off-by: Jason Robb <jrobb@sekls.org> Signed-off-by: Lucas Gass <lucas@bywatersolutions.com> Signed-off-by: Jason Robb <jrobb@sekls.org> Signed-off-by: Lucas Gass <lucas@bywatersolutions.com> -- You are receiving this mail because: You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=41604 --- Comment #94 from Lucas Gass (lukeg) <lucas@bywatersolutions.com> --- Created attachment 201454 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=201454&action=edit Bug 41604: Fix typo Signed-off-by: Lucas Gass <lucas@bywatersolutions.com> Signed-off-by: Jason Robb <jrobb@sekls.org> Signed-off-by: Lucas Gass <lucas@bywatersolutions.com> -- You are receiving this mail because: You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=41604 --- Comment #96 from Lucas Gass (lukeg) <lucas@bywatersolutions.com> --- Created attachment 201456 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=201456&action=edit Bug 41604: (follow-up) Fix the visibility when visibility_condition is 0 or null, instead of false Signed-off-by: Jason Robb <jrobb@sekls.org> Signed-off-by: Lucas Gass <lucas@bywatersolutions.com> -- You are receiving this mail because: You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=41604 --- Comment #98 from Lucas Gass (lukeg) <lucas@bywatersolutions.com> --- Created attachment 201458 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=201458&action=edit Bug 41604: Add one more Cypress test Signed-off-by: Jason Robb <jrobb@sekls.org> Signed-off-by: Lucas Gass <lucas@bywatersolutions.com> -- You are receiving this mail because: You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=41604 --- Comment #100 from Lucas Gass (lukeg) <lucas@bywatersolutions.com> --- Created attachment 201460 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=201460&action=edit Bug 41604: Hide column if the feature is disabled Example: supplier_report on the basket page Signed-off-by: Jason Robb <jrobb@sekls.org> Signed-off-by: Lucas Gass <lucas@bywatersolutions.com> -- You are receiving this mail because: You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=41604 --- Comment #102 from Lucas Gass (lukeg) <lucas@bywatersolutions.com> --- Created attachment 201462 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=201462&action=edit Bug 41604: (QA follow-up) Add a semicolon Signed-off-by: Lucas Gass <lucas@bywatersolutions.com> -- You are receiving this mail because: You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=41604 Lucas Gass (lukeg) <lucas@bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Signed Off |Passed QA -- You are receiving this mail because: You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=41604 Victor Grousset/tuxayo <victor@tuxayo.net> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #201450|0 |1 is obsolete| | -- You are receiving this mail because: You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=41604 Victor Grousset/tuxayo <victor@tuxayo.net> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #201452|0 |1 is obsolete| | -- You are receiving this mail because: You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=41604 Victor Grousset/tuxayo <victor@tuxayo.net> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #201454|0 |1 is obsolete| | -- You are receiving this mail because: You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=41604 Victor Grousset/tuxayo <victor@tuxayo.net> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #201456|0 |1 is obsolete| | -- You are receiving this mail because: You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=41604 Victor Grousset/tuxayo <victor@tuxayo.net> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #201458|0 |1 is obsolete| | -- You are receiving this mail because: You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=41604 Victor Grousset/tuxayo <victor@tuxayo.net> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #201460|0 |1 is obsolete| | -- You are receiving this mail because: You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=41604 Victor Grousset/tuxayo <victor@tuxayo.net> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #201462|0 |1 is obsolete| | -- You are receiving this mail because: You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=41604 --- Comment #104 from Victor Grousset/tuxayo <victor@tuxayo.net> --- Created attachment 201515 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=201515&action=edit Bug 41604: Impossible to hide Checkin column in issues-table in circ/circulation.pl Bug 41314 caused a regression, making it impossible to hide the Checkin column in the issues table in the patron's circulation page. To recreate: 1. Hide the checkin column by default in circulation issues-table 1.1. Go to Administration > Table settings > Circulation tab > Jump to page : circulation > Table id: issues-table 1.2. Check 'Is hidden by default' for 'checkin' 1.3. Click 'Save' 2. logout and login 3. Checkout an item to a patron 3.1. In the search bar, set to Checkout, search for Henry 3.2. In the checkout field, enter the barcode 39999000010114 and press enter --> In the table of issues, Checkin column is visible, it should be hidden by default 4. Apply the patch 5. Repeat step 2, 3 --> In the table of issues, Checkin column is not visible Signed-off-by: David Nind <david@davidnind.com> Signed-off-by: Jason Robb <jrobb@sekls.org> Signed-off-by: Lucas Gass <lucas@bywatersolutions.com> Signed-off-by: Jason Robb <jrobb@sekls.org> Signed-off-by: Victor Grousset/tuxayo <victor@tuxayo.net> Signed-off-by: Lucas Gass <lucas@bywatersolutions.com> -- You are receiving this mail because: You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=41604 --- Comment #106 from Victor Grousset/tuxayo <victor@tuxayo.net> --- Created attachment 201517 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=201517&action=edit Bug 41604: Use visibility_condition in other places There are 3 other places where we force the visibility of columns. They must als use this new visibility_condition flag, so they are all using it. Test plan: The 3 tables are: 1. basket with the columns edifact, modify, cancel 2. hold history with the column itemtype 3. batch edit patron with the columns checkbox, password expiration, protected Signed-off-by: Jason Robb <jrobb@sekls.org> Signed-off-by: Lucas Gass <lucas@bywatersolutions.com> Signed-off-by: Jason Robb <jrobb@sekls.org> Signed-off-by: Victor Grousset/tuxayo <victor@tuxayo.net> Signed-off-by: Lucas Gass <lucas@bywatersolutions.com> -- You are receiving this mail because: You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=41604 --- Comment #108 from Victor Grousset/tuxayo <victor@tuxayo.net> --- Created attachment 201519 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=201519&action=edit Bug 41604: QA fixes Signed-off-by: Lucas Gass <lucas@bywatersolutions.com> Signed-off-by: Jason Robb <jrobb@sekls.org> Signed-off-by: Victor Grousset/tuxayo <victor@tuxayo.net> Signed-off-by: Lucas Gass <lucas@bywatersolutions.com> -- You are receiving this mail because: You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=41604 --- Comment #110 from Victor Grousset/tuxayo <victor@tuxayo.net> --- Created attachment 201521 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=201521&action=edit Bug 41604: Fix force display of a column If a column must be displayed but was hidden before Signed-off-by: Lucas Gass <lucas@bywatersolutions.com> Signed-off-by: Jason Robb <jrobb@sekls.org> Signed-off-by: Victor Grousset/tuxayo <victor@tuxayo.net> Signed-off-by: Lucas Gass <lucas@bywatersolutions.com> -- You are receiving this mail because: You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=41604 --- Comment #112 from Victor Grousset/tuxayo <victor@tuxayo.net> --- Created attachment 201523 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=201523&action=edit Bug 41604: Add Cypress tests Signed-off-by: Jason Robb <jrobb@sekls.org> Signed-off-by: Victor Grousset/tuxayo <victor@tuxayo.net> Signed-off-by: Lucas Gass <lucas@bywatersolutions.com> -- You are receiving this mail because: You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=41604 --- Comment #114 from Victor Grousset/tuxayo <victor@tuxayo.net> --- Created attachment 201525 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=201525&action=edit Bug 41604: Save user decision per column when visibility changes - Modify column-visibility.dt event handler to store the visibility preference of the user - Improve _dt_force_visibility() to correctly pick the correctly visibility value depending on visibility_condition, is_hidden, and the user preference - Add _dt_get_expected_column_visibility() to guess what's the expected visibility - Force boolean true/false (vs 1/0) using !! - Rely on a global variable kohaTableUserChoices to store the user choice. It will be part of the state and store in localStorage. Signed-off-by: Jason Robb <jrobb@sekls.org> Signed-off-by: Victor Grousset/tuxayo <victor@tuxayo.net> Signed-off-by: Lucas Gass <lucas@bywatersolutions.com> -- You are receiving this mail because: You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=41604 --- Comment #116 from Victor Grousset/tuxayo <victor@tuxayo.net> --- Created attachment 201527 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=201527&action=edit Bug 41604: Correctly restore alert when bKohaColumnsUseNames is off If we define visibility_condition we must have implemented bKohaColumnsUseNames correctly. Signed-off-by: Victor Grousset/tuxayo <victor@tuxayo.net> Signed-off-by: Lucas Gass <lucas@bywatersolutions.com> -- You are receiving this mail because: You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=41604 --- Comment #118 from Victor Grousset/tuxayo <victor@tuxayo.net> --- Created attachment 201529 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=201529&action=edit Bug 41604: (QA follow-up) Remove unused JS variable Signed-off-by: Victor Grousset/tuxayo <victor@tuxayo.net> Signed-off-by: Lucas Gass <lucas@bywatersolutions.com> -- You are receiving this mail because: You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=41604 Pedro Amorim (ammopt) <pedro.amorim@openfifth.co.uk> changed: What |Removed |Added ---------------------------------------------------------------------------- Version(s)| |26.11.00 released in| | Status|Passed QA |Pushed to main -- You are receiving this mail because: You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=41604 --- Comment #121 from Pedro Amorim (ammopt) <pedro.amorim@openfifth.co.uk> --- Created attachment 202208 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=202208&action=edit Bug 41604: (follow-up) Wait for the page to be ready before searching in patrons_search.t Bug 41604 made every page do a bit more work when it loads. This Selenium test didn't wait for that work to finish before typing into the search box, so sometimes it searched too early and got the wrong results. This adds a short wait after each page load so the test only searches once the page is actually ready. Introduced by: Bug 41604: Save user decision per column when visibility changes Test plan: prove t/db_dependent/selenium/patrons_search.t Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com> -- You are receiving this mail because: You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=41604 Pedro Amorim (ammopt) <pedro.amorim@openfifth.co.uk> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #202208|0 |1 is obsolete| | -- You are receiving this mail because: You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=41604 Pedro Amorim (ammopt) <pedro.amorim@openfifth.co.uk> changed: What |Removed |Added ---------------------------------------------------------------------------- Keywords|additional_work_needed | --- Comment #123 from Pedro Amorim (ammopt) <pedro.amorim@openfifth.co.uk> --- Follow up pushed -- You are receiving this mail because: You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=41604 --- Comment #124 from Jonathan Druart <jonathan.druart@gmail.com> --- (In reply to Pedro Amorim (ammopt) from comment #122)

Created attachment 202211 [details] [review] Bug 41604: (follow-up) Wait for the page to be ready before searching in patrons_search.t

Something else was hidden behind this failure, see bug 43174. -- You are receiving this mail because: You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=41604 Martin Renvoize (ashimema) <martin.renvoize@openfifth.co.uk> changed: What |Removed |Added ---------------------------------------------------------------------------- Blocks| |30144 Referenced Bugs: https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=30144 [Bug 30144] Add support for EDI GIR:LVT and/or GIR:LVC, Servicing instructions, segment -- You are receiving this mail because: You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=41604 --- Comment #125 from Lucas Gass (lukeg) <lucas@bywatersolutions.com> --- Nice work everyone! Pushed to 26.05.x for the upcoming 26.05.03 release. -- You are receiving this mail because: You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=41604 --- Comment #126 from Baptiste Wojtkowski (bwoj) <baptiste.wojtkowski@biblibre.com> --- Created attachment 205592 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=205592&action=edit Bug 41604: [25.11] Impossible to hide Checkin column in issues-table in circ/circulation.pl Bug 41314 caused a regression, making it impossible to hide the Checkin column in the issues table in the patron's circulation page. To recreate: 1. Hide the checkin column by default in circulation issues-table 1.1. Go to Administration > Table settings > Circulation tab > Jump to page : circulation > Table id: issues-table 1.2. Check 'Is hidden by default' for 'checkin' 1.3. Click 'Save' 2. logout and login 3. Checkout an item to a patron 3.1. In the search bar, set to Checkout, search for Henry 3.2. In the checkout field, enter the barcode 39999000010114 and press enter --> In the table of issues, Checkin column is visible, it should be hidden by default 4. Apply the patch 5. Repeat step 2, 3 --> In the table of issues, Checkin column is not visible Signed-off-by: David Nind <david@davidnind.com> Signed-off-by: Jason Robb <jrobb@sekls.org> Signed-off-by: Lucas Gass <lucas@bywatersolutions.com> Signed-off-by: Jason Robb <jrobb@sekls.org> Signed-off-by: Victor Grousset/tuxayo <victor@tuxayo.net> Signed-off-by: Lucas Gass <lucas@bywatersolutions.com> -- You are receiving this mail because: You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=41604 Arthur Suzuki <arthur.suzuki@biblibre.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Blocks| |30221 Referenced Bugs: https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=30221 [Bug 30221] Make the tables on opac-user.tt column configurable -- You are receiving this mail because: You are watching all bug changes.