Confirm before associate/dissociate forum to LP item - refs BT#10629

main/newscorm/learnpath.class.php

@@ -5642,9 +5642,9 @@ class learnpath
                             )
                             ) {
                                 $forumIcon = Display::url(
-                                    Display::return_icon('forum.png', get_lang('CreateForum'), [], ICON_SIZE_TINY),
+                                    Display::return_icon('forum.png', get_lang('DissociateForumToLPItem'), [], ICON_SIZE_TINY),
                                     '#',
-                                    ['class' => 'btn btn-default disabled']
+                                    ['class' => 'btn btn-default disabled lp-btn-dissociate-forum']
                                 );
                             } else {
                                 $forumIconUrl = api_get_self() . '?' . api_get_cidreq() . '&' . http_build_query([
@@ -5653,9 +5653,9 @@ class learnpath
                                         'lp_id' => $this->lp_id
                                     ]);
                                 $forumIcon = Display::url(
-                                    Display::return_icon('forum.png', get_lang('CreateForum'), [], ICON_SIZE_TINY),
+                                    Display::return_icon('forum.png', get_lang('AssociateForumToLPItem'), [], ICON_SIZE_TINY),
                                     $forumIconUrl,
-                                    ['class' => "btn btn-default"]
+                                    ['class' => "btn btn-default lp-btn-associate-forum"]
                                 );
                             }
                         }

main/newscorm/lp_add_item.php

@@ -172,7 +172,22 @@ $(document).ready(function() {
        
         $("#hide_bar_template").toggleClass("hide_bar_template_not_hide");
     });
-    
+
+    $('.lp-btn-associate-forum').on('click', function (e) {
+        var associate = confirm('<?php echo get_lang('ConfirmAssociateForumToLPItem') ?>');
+
+        if (!associate) {
+            e.preventDefault();
+        }
+    });
+
+    $('.lp-btn-dissociate-forum').on('click', function (e) {
+        var dissociate = confirm('<?php echo get_lang('ConfirmDissociateForumToLPItem') ?>');
+
+        if (!dissociate) {
+            e.preventDefault();
+        }
+    });
 });
 </script>
 <?php

main/newscorm/lp_edit_item.php

@@ -125,6 +125,24 @@ function confirmation(name) {
         return false;
     }
 }
+
+$(document).ready(function() {
+    $('.lp-btn-associate-forum').on('click', function (e) {
+        var associate = confirm('<?php echo get_lang('ConfirmAssociateForumToLPItem') ?>');
+
+        if (!associate) {
+            e.preventDefault();
+        }
+    });
+
+    $('.lp-btn-dissociate-forum').on('click', function (e) {
+        var dissociate = confirm('<?php echo get_lang('ConfirmDissociateForumToLPItem') ?>');
+
+        if (!dissociate) {
+            e.preventDefault();
+        }
+    });
+});
 </script>
 <?php