[Bug 38045] New: Patron messaging preference table says it is loading but never does if fieldset is collapsed
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=38045 Bug ID: 38045 Summary: Patron messaging preference table says it is loading but never does if fieldset is collapsed Change sponsored?: --- Product: Koha Version: Main Hardware: All OS: All Status: NEW Severity: normal Priority: P5 - low Component: Patrons Assignee: koha-bugs@lists.koha-community.org Reporter: lucas@bywatersolutions.com QA Contact: testopia@bugs.koha-community.org CC: gmcharlt@gmail.com, kyle.m.hall@gmail.com To recreate: 1. Add Patron messaging preferences to `CollapseFieldsPatronAddForm` system preference. 2. Go to a patron record, new or existing. 3. Scroll down to the message pref section and see `Loading new messaging defaults `. 4. It never loads If the fieldset is collapsed we shouldn't load the table but we also shouldn't show a message saying `Loading new messaging defaults...` -- You are receiving this mail because: You are the assignee for the bug. You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=38045 Phil Ringnalda <phil@chetcolibrary.org> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |david@davidnind.com --- Comment #1 from Phil Ringnalda <phil@chetcolibrary.org> --- *** Bug 39047 has been marked as a duplicate of this bug. *** -- 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=38045 Nick Clemens (kidclamp) <nick@bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|NEW |Needs Signoff -- 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=38045 --- Comment #2 from Nick Clemens (kidclamp) <nick@bywatersolutions.com> --- Created attachment 178100 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=178100&action=edit Bug 38045: Don't show loading message when hiding messaging preferences This spinner is meant to show when the category has changed and we are loading the new defaults for a category. There is no reason to toggle visibility when the form is initially hidden To test: 1 - Mark 'Patron messaging preference' in CollapseFieldsPatronAddForm 2 - Edit an existing patron or add a new one 3 - Note the spinner 'Loading new messaging preferences' 4 - Click the toggle to hide/show messaging preferences a few times 5 - Spinner disappears and table can be shown 6 - Apply patch 7 - Edit existing or add new patron 8 - No spinner 9 - Change patorn category and say yes to laod new prefs 10 - Note spinner shows and then hides very fast It is easier to see if you collapse all the fields between category and messaing prefs 11 - Confirm messaging prefs can be edited/saved correctly still -- 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=38045 Nick Clemens (kidclamp) <nick@bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Assignee|koha-bugs@lists.koha-commun |nick@bywatersolutions.com |ity.org | CC| |nick@bywatersolutions.com -- You are receiving this mail because: You are the assignee for the bug. You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=38045 David Nind <david@davidnind.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Needs Signoff |Patch doesn't apply --- Comment #3 from David Nind <david@davidnind.com> --- I'm getting the sha1/fake ancestor error when trying to apply the patch 8-(... git bz apply 38045 Bug 38045 - Patron messaging preference table says it is loading but never does if fieldset is collapsed 178100 - Bug 38045: Don't show loading message when hiding messaging preferences Apply? [(y)es, (n)o, (i)nteractive] y Applying: Bug 38045: Don't show loading message when hiding messaging preferences error: sha1 information is lacking or useless (koha-tmpl/intranet-tmpl/prog/en/modules/members/memberentrygen.tt). error: could not build fake ancestor Patch failed at 0001 Bug 38045: Don't show loading message when hiding messaging preferences -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=38045 Nick Clemens (kidclamp) <nick@bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Resolution|--- |WORKSFORME Status|Patch doesn't apply |RESOLVED --- Comment #4 from Nick Clemens (kidclamp) <nick@bywatersolutions.com> --- Maybe bug 39189 fixed this, but I can no longer recreate -- You are receiving this mail because: You are watching all bug changes.
participants (1)
-
bugzilla-daemon@bugs.koha-community.org