https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=43318 Owen Leonard <oleonard@myacpl.org> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|NEW |ASSIGNED -- You are receiving this mail because: You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=43318 --- Comment #1 from Owen Leonard <oleonard@myacpl.org> --- Created attachment 203685 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=203685&action=edit Bug 43318: Remove event attributes from patron_messages.inc This patch removes event attributes from patron_messages.inc and puts the event handling in members-menu.js. members-menu.js, like patron_messages.inc, is included in both circulation.tt and moremember.tt To test, apply the patch and locate a patron in the staff client with overdues. - On both the checkout and patron detail pages, there should be a message, "Overdues: Patron has ITEMS OVERDUE See highlighted items below." Click the link. - The page should scroll to the checkouts tab, which should be activated. - If the AlwaysLoadCheckoutsTable preference says "Don't always load the checkouts table" the loading of the checkouts table should also be triggered by this link. - Add a restriction to the patron's account. - On both the checkout and patron detail pages, confirm that the "View restrictions" button triggers the activation of the "Restrictions" tab and scrolls it into view. Sponsored-by: Athens County Public Libraries -- You are receiving this mail because: You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=43318 Jonathan Druart <jonathan.druart@gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- QA Contact|testopia@bugs.koha-communit |jonathan.druart@gmail.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=43318 David Nind <david@davidnind.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #203685|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=43318 Jonathan Druart <jonathan.druart@gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Signed Off |Failed QA --- Comment #3 from Jonathan Druart <jonathan.druart@gmail.com> --- Hum, there is something fishy. If "Always show checkouts automatically" is not ticked: Before applying the patch it stayed unticked after we clicked "See highlighted items below". With the patch, it is now ticked, and the cookie (issues-table-load-immediately-moremember) is true This is because we had an ugly hack, we called twice `$('#issues-table-load-immediately').click()` I guess it would be acceptable for this patch to restore the 2 calls, and fix this behaviour on a separate bug. -- You are receiving this mail because: You are watching all bug changes.