Learnpath: Fix order menu tree when it has final item - refs BT#20362

chamilo · Jan 31, 2023 · e750540 · e750540
1 parent 9635771
commit e750540
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 2 deletions.
diff --git a/main/lp/learnpath.class.php b/main/lp/learnpath.class.php
@@ -6775,6 +6775,9 @@ public function print_recursive($elements, $default_data, $default_content)
     {
         $return = '';
         foreach ($elements as $key => $item) {
+            if (TOOL_LP_FINAL_ITEM === $item['type']) {
+                $key = 'final_item';
+            }
             if (isset($item['load_data']) || empty($item['data'])) {
                 $item['data'] = $default_data[$item['load_data']];
                 $item['type'] = $default_content[$item['load_data']]['item_type'];
@@ -13507,7 +13510,7 @@ public function getFinalItemForm()
         );
         $form->addHtml(
             Display::return_message(
-                get_lang('LPEndStepAddTagsToShowCertificateOrSkillAutomatically') . '</br></br> <b>((certificate))</b> </br> <b>((skill))</b>',
+                get_lang('LPEndStepAddTagsToShowCertificateOrSkillAutomatically').'</br></br> <b>((certificate))</b> </br> <b>((skill))</b>',
                 'normal',
                 false
             )

diff --git a/main/lp/lp_controller.php b/main/lp/lp_controller.php
@@ -102,7 +102,7 @@ function get_UL_integer_id(in_ul_id) {
         $(".li_container .order_items").click(function(e) {
             var dir = $(this).data("dir");
             var itemId = $(this).data("id");
-            var jItems = $("#lp_item_list li.li_container");
+            var jItems = $("#lp_item_list li.li_container:not(#final_item)");
             var jItem = $("#"+ itemId);
             var index = jItems.index(jItem);
             var total = jItems.length;