From 25195de742b222433a9a8def2ca412106cb07f3d Mon Sep 17 00:00:00 2001
From: Owen Leonard <oleonard@myacpl.org>
Date: Fri, 3 Jan 2020 17:40:34 +0000
Subject: [PATCH] Bug 24341: Allow for more complex markup in OPAC confirmation
 dialogs

This patch changes the globally-available confirmModal() function in the
OPAC so that it uses jQuery's html() function for adding information to
the confirmation modal's body. This will allow for more than just plain
text to be added.

This patch also removes some obsolete lines from the JavaScript in
opac-user.tt and renames some variable for clarity.

To test, apply the patch log in to the OPAC as a user who has holds and
article requests.

 - Test the following interactions, both the confirmation and cancel
   options for each:
     - Cancel a hold
     - Suspend all holds
     - Resume all holds
     - Cancel an article request
---
 koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-user.tt | 6 ++----
 koha-tmpl/opac-tmpl/bootstrap/js/global.js            | 2 +-
 2 files changed, 3 insertions(+), 5 deletions(-)

diff --git a/koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-user.tt b/koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-user.tt
index 96c494c9499..879867af0a1 100644
--- a/koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-user.tt
+++ b/koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-user.tt
@@ -750,7 +750,6 @@
                 e.preventDefault();
                 var hold_title = $(this).data("title");
                 var reserve_id = $(this).data("reserve_id");
-                var body = "<p><strong>" + hold_title + "</strong></p>";
                 confirmModal( hold_title, _("Are you sure you want to cancel this hold?"), _("Yes, cancel hold"), _("No, do not cancel hold"), function( result ){
                         if( result ){
                             $("#delete_hold_" + reserve_id ).submit();
@@ -761,10 +760,9 @@
 
             $(".btn-delete-article-request").on("click", function(e){
                 e.preventDefault();
-                var hold_title = $(this).data("title");
+                var article_request = $(this).data("title");
                 var article_request_id = $(this).data("article-request_id");
-                var body = "<p><strong>" + hold_title + "</strong></p>";
-                confirmModal( hold_title, _("Are you sure you want to cancel this article request?"), _("Yes, cancel article request"), _("No, do not cancel article request"), function( result ){
+                confirmModal( article_request, _("Are you sure you want to cancel this article request?"), _("Yes, cancel article request"), _("No, do not cancel article request"), function( result ){
                         if( result ){
                             $("#delete_article_request_" + article_request_id ).submit();
                         }
diff --git a/koha-tmpl/opac-tmpl/bootstrap/js/global.js b/koha-tmpl/opac-tmpl/bootstrap/js/global.js
index 7f95b324ff9..84ee7e844b3 100644
--- a/koha-tmpl/opac-tmpl/bootstrap/js/global.js
+++ b/koha-tmpl/opac-tmpl/bootstrap/js/global.js
@@ -82,7 +82,7 @@ function confirmModal(message, title, yes_label, no_label, callback) {
     }
 
     $("#bootstrap-confirm-box-modal .modal-header h4").text( title || "" );
-    $("#bootstrap-confirm-box-modal .modal-body p").text( message || "" );
+    $("#bootstrap-confirm-box-modal .modal-body").html( message || "" );
     $("#bootstrap-confirm-box-modal-submit").text( yes_label || 'Confirm' );
     $("#bootstrap-confirm-box-modal-cancel").text( no_label || 'Cancel' );
     $("#bootstrap-confirm-box-modal").modal('show');
-- 
2.11.0