https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=38268

Phil Ringnalda <[email protected]> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
          Component|Lists                       |Templates
            Summary|Cancelling 'Delete list'    |Callers of confirmModal
                   |then using 'Delete selected |need to remove the modal as
                   |items' deletes list         |the first step in their
                   |                            |callback function
           Assignee|[email protected] |[email protected]
                   |ity.org                     |

--- Comment #2 from Phil Ringnalda <[email protected]> ---
Le sigh.

I was starting to suspect that just hiding the modal was the problem, though
why is beyond my pay-grade.

Then I went looking for other places that should be affected (by having more
than one caller of confirmModal), and saw that opac_user.tt should confuse
cancel and suspend holds and cancel article requests. But it doesn't, because
in bug 27947 three years ago Agustín apparently realized this was a problem and
how to fix it, silently fixed it in opac_user.tt by making the first step in
every callback be '$("#bootstrap-confirm-box-modal").remove()', and nobody
reviewing the fix noticed it and considered other places that might need the
same fix.

-- 
You are receiving this mail because:
You are watching all bug changes.
You are the assignee for the bug.
_______________________________________________
Koha-bugs mailing list
[email protected]
https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/

Reply via email to