From b31be9cac20797319e8770712ce5ad72c7320e0d Mon Sep 17 00:00:00 2001
From: AntonLV <vlnotna@gmail.com>
Date: Fri, 15 Mar 2024 12:54:44 +0300
Subject: [PATCH] Ticket #4641 - Use HTMX to load page content without genaral
 page structure. Profile/Context submenu.

---
 inc/classes/BxDolInformer.php                 |  8 ++++--
 inc/classes/BxDolPage.php                     |  2 +-
 .../groups/classes/BxBaseModGroupsModule.php  |  6 ++---
 .../classes/BxBaseModGroupsSearchResult.php   |  4 +--
 .../system/menu_main_submenu_more_auto.html   |  2 +-
 .../template/system/scripts/BxTemplMenu.php   |  2 +-
 modules/boonex/events/install/langs/en.xml    |  4 +--
 .../russian/data/langs/bx_events/ru.xml       |  4 +--
 template/calendar.html                        | 14 ++++++++---
 template/scripts/BxBaseMenu.php               |  2 +-
 template/scripts/BxBasePage.php               | 25 +++++++++++++++++--
 11 files changed, 51 insertions(+), 22 deletions(-)

diff --git a/inc/classes/BxDolInformer.php b/inc/classes/BxDolInformer.php
index 06fa95e09b..6fea93030d 100644
--- a/inc/classes/BxDolInformer.php
+++ b/inc/classes/BxDolInformer.php
@@ -81,11 +81,15 @@ public function setEnabled($bEnabled)
      */
     public function add ($sId, $sMsg, $iType = BX_INFORMER_INFO)
     {
-        $this->_aMessages[$sId] = array (
+        if(!$this->_bEnabled)
+            return;
+
+        $this->_addJsCss();
+        $this->_aMessages[$sId] = [
             'id' => $sId,
             'msg' => $sMsg,
             'type' => $iType,
-        );
+        ];
     }
 
     /**
diff --git a/inc/classes/BxDolPage.php b/inc/classes/BxDolPage.php
index 1bd39e5015..33d1a9b6ee 100644
--- a/inc/classes/BxDolPage.php
+++ b/inc/classes/BxDolPage.php
@@ -697,7 +697,7 @@ public function displayPage ($oTemplate = null)
         $oTemplate->setPageType($this->getType());
         $oTemplate->setPageInjections($this->getInjections());
         if($oTemplate->getPageNameIndex() == BX_PAGE_CONTENT_PRELOAD)
-            $oTemplate->setPageContent('page_main_code', $this->getCodeDynamic(true));
+            $oTemplate->setPageContent('page_main_code', $this->getIncludes());
         else
             $oTemplate->setPageContent('page_main_code', $this->getCode());
         $oTemplate->getPageCode();
diff --git a/modules/base/groups/classes/BxBaseModGroupsModule.php b/modules/base/groups/classes/BxBaseModGroupsModule.php
index e83ec75aa9..4e42a8ffd6 100644
--- a/modules/base/groups/classes/BxBaseModGroupsModule.php
+++ b/modules/base/groups/classes/BxBaseModGroupsModule.php
@@ -128,11 +128,9 @@ public function actionGetQuestionnaire()
 
     public function serviceManageTools($sType = 'common')
     {
-        $sResult = parent::serviceManageTools($sType);
-        if(!empty($sResult))
-            $this->_oTemplate->addJsSystem(['modules/base/groups/js/|manage_tools.js']);
+        $this->_oTemplate->addJs(['modules/base/groups/js/|manage_tools.js']);
 
-        return $sResult;
+        return parent::serviceManageTools($sType);
     }
     
     public function decodeDataAPI($aData, $aParams = [])
diff --git a/modules/base/groups/classes/BxBaseModGroupsSearchResult.php b/modules/base/groups/classes/BxBaseModGroupsSearchResult.php
index 582079b172..170c7f3bed 100644
--- a/modules/base/groups/classes/BxBaseModGroupsSearchResult.php
+++ b/modules/base/groups/classes/BxBaseModGroupsSearchResult.php
@@ -24,8 +24,8 @@ public function __construct($sMode = '', $aParams = array())
     public function displaySearchBox ($sContent, $sPaginate = '')
     {
         $aResult = parent::displaySearchBox($sContent, $sPaginate);
-        
-        if(!empty($aResult['content'])) {
+
+        if(!empty($aResult['content']) && $this->oModule->_oConfig->getJsClass('main') != '') {
             $this->oModule->_oTemplate->addJs(['modules/base/groups/js/|main.js', 'main.js']);
 
             $aResult['content'] = $this->oModule->_oTemplate->getJsCode('main', [
diff --git a/modules/boonex/artificer/data/template/system/menu_main_submenu_more_auto.html b/modules/boonex/artificer/data/template/system/menu_main_submenu_more_auto.html
index 216063aacc..0e03f1b2f2 100644
--- a/modules/boonex/artificer/data/template/system/menu_main_submenu_more_auto.html
+++ b/modules/boonex/artificer/data/template/system/menu_main_submenu_more_auto.html
@@ -1,6 +1,6 @@
 <ul id="__html_id__" class="bx-menu-main-submenu<bx_if:show_more_auto_class> bx-menu-more-auto</bx_if:show_more_auto_class> bx-menu-object-__object__ flex items-stretch">
     <bx_repeat:menu_items>
-        <li class="bx-menu-item bx-menu-item-__name__ __class_add__ flex">
+        <li class="bx-menu-item bx-menu-item-__name__ __class_add__ flex" __attrs_wrp__>
             <a href="__link__" <bx_if:onclick>onclick="__onclick__"</bx_if:onclick> class="group block relative py-4 px-4 text-gray-500 hover:bg-gray-100 dark:hover:bg-gray-700 hover:text-gray-700 dark:hover:text-gray-200 whitespace-nowrap text-sm font-medium text-center hover:bg-transparent focus:z-10 box-sizing"__attrs__>__title__<span class="bx-mi-underline absolute inset-x-0 bottom-0 h-0.5 group-hover:bg-gray-500 rounded-md"></span></a></a>__popup__
         </li>
     </bx_repeat:menu_items>
diff --git a/modules/boonex/artificer/data/template/system/scripts/BxTemplMenu.php b/modules/boonex/artificer/data/template/system/scripts/BxTemplMenu.php
index 5232dbc1ec..72baca6351 100644
--- a/modules/boonex/artificer/data/template/system/scripts/BxTemplMenu.php
+++ b/modules/boonex/artificer/data/template/system/scripts/BxTemplMenu.php
@@ -18,7 +18,7 @@ public function __construct ($aObject, $oTemplate = false)
 
         if(isset($aObject['object'])) {
             //--- For page submenus ---//
-            if(strpos($aObject['object'], '_submenu') !== false && !($this instanceof BxBaseModProfileMenuView)) {
+            if(strpos($aObject['object'], '_submenu') !== false) {
                 $sClass = 'bx-menu-tab-active';
 
                 $this->_bHx = true;
diff --git a/modules/boonex/events/install/langs/en.xml b/modules/boonex/events/install/langs/en.xml
index 002f543786..51937a36dd 100644
--- a/modules/boonex/events/install/langs/en.xml
+++ b/modules/boonex/events/install/langs/en.xml
@@ -193,8 +193,8 @@
     <string name="_bx_events_page_title_sys_manage_profile"><![CDATA[Manage Event]]></string>
     <string name="_bx_events_page_title_sys_delete_profile"><![CDATA[Delete Event]]></string>
     <string name="_bx_events_page_title_delete_profile"><![CDATA[Delete Event]]></string>
-    <string name="_bx_events_page_title_sys_join_profile"><![CDATA[Join Course]]></string>
-    <string name="_bx_events_page_title_join_profile"><![CDATA[Join Course]]></string>
+    <string name="_bx_events_page_title_sys_join_profile"><![CDATA[Join Event]]></string>
+    <string name="_bx_events_page_title_join_profile"><![CDATA[Join Event]]></string>
     <string name="_bx_events_page_title_sys_view_profile_closed"><![CDATA[View Closed Event]]></string>
     <string name="_bx_events_page_title_sys_view_profile"><![CDATA[View Event]]></string>
     <string name="_bx_events_page_title_view_profile"><![CDATA[{display_name}]]></string>
diff --git a/modules/boonex/russian/data/langs/bx_events/ru.xml b/modules/boonex/russian/data/langs/bx_events/ru.xml
index 374d771a4b..3dae0c90d0 100644
--- a/modules/boonex/russian/data/langs/bx_events/ru.xml
+++ b/modules/boonex/russian/data/langs/bx_events/ru.xml
@@ -576,8 +576,8 @@
     <string name="_bx_events_form_price_input_sys_do_cancel"><![CDATA[Отменить]]></string>
     <string name="_bx_events_form_price_input_do_cancel"><![CDATA[Отменить]]></string>
     <string name="_bx_events_acl_action_use_paid_join"><![CDATA[Использовать платное подключение]]></string>
-    <string name="_bx_events_page_title_sys_join_profile"><![CDATA[Присоединиться к курсу]]></string>
-    <string name="_bx_events_page_title_join_profile"><![CDATA[Присоединиться к курсу]]></string>
+    <string name="_bx_events_page_title_sys_join_profile"><![CDATA[Присоединиться к событию]]></string>
+    <string name="_bx_events_page_title_join_profile"><![CDATA[Присоединиться к событию]]></string>
     <string name="_bx_events_page_title_browse_upcoming"><![CDATA[Предстоящие События]]></string>
     <string name="_bx_events_page_title_sys_profile_sessions"><![CDATA[Сессии]]></string>
     <string name="_bx_events_page_title_profile_sessions"><![CDATA[Сессии]]></string>
diff --git a/template/calendar.html b/template/calendar.html
index 2c7cc1a702..50d39ae627 100644
--- a/template/calendar.html
+++ b/template/calendar.html
@@ -23,10 +23,16 @@
         }
     };
     var oOptions = __options__;
-    
+
     var oBxFullCalendar = null;
-    document.addEventListener('DOMContentLoaded', function() {
-        oBxFullCalendar = new FullCalendar.Calendar(document.getElementById('__id__'), $.extend({}, oDefaults, oOptions));
+    var fBxFullCalendarInit = function() {
+        oBxFullCalendar = new FullCalendar.Calendar(oElement, $.extend({}, oDefaults, oOptions));
         oBxFullCalendar.render();
-    });
+    };
+
+    var oElement = document.getElementById('__id__');
+    if(oElement)
+        fBxFullCalendarInit();
+    else
+        document.addEventListener('DOMContentLoaded', fBxFullCalendarInit);
 </script>
diff --git a/template/scripts/BxBaseMenu.php b/template/scripts/BxBaseMenu.php
index 5e165460fd..6a173ccd59 100644
--- a/template/scripts/BxBaseMenu.php
+++ b/template/scripts/BxBaseMenu.php
@@ -236,7 +236,7 @@ protected function _getMenuItem ($a)
         $a['attrs'] = $this->_getMenuAttrs($a);
         $a['attrs_wrp'] = '';
 
-        if($this->_bHx /*&& strpos($a['name'], '-manage') === false*/) {
+        if($this->_bHx && !empty($a['link']) && strpos($a['link'], 'javascript:') === false) {
             $this->_aHx['get'] = $a['link'];
             $a['attrs'] .= bx_get_htmx_attrs($this->_aHx);
 
diff --git a/template/scripts/BxBasePage.php b/template/scripts/BxBasePage.php
index 764631b19b..dfe8305179 100644
--- a/template/scripts/BxBasePage.php
+++ b/template/scripts/BxBasePage.php
@@ -124,7 +124,7 @@ public function performActionEmbed ()
      * Very similar to BxBasePage::getCode
      * but adds css and js files which are needed for the corect page display
      */ 
-    public function getCodeDynamic ($bIncludesOnly = false)
+    public function getCodeDynamic ()
     {
         // TODO: remake to use use collect* template methods
 
@@ -155,7 +155,28 @@ public function getCodeDynamic ($bIncludesOnly = false)
         foreach ($aJsNew as $a)
             $sJs .= $oTemplate->addJs($a['url'], true);
 
-        return $sJs . $sCss . (!$bIncludesOnly ? $sContent : '');
+        return $sJs . $sCss . $sContent;
+    }
+
+    public function getIncludes ()
+    {
+        $oTemplate = BxDolTemplate::getInstance();
+
+        // generate page code
+        $this->getCode();
+
+        // get js&css after the page code is generated
+        $aCss = $oTemplate->getCss();
+        $aJs = $oTemplate->getJs();
+
+        // add js&css files in static mode
+        $sResult = '';
+        foreach ($aCss as $a)
+            $sResult .= $oTemplate->addCss($a['url'], true);
+        foreach ($aJs as $a)
+            $sResult .= $oTemplate->addJs($a['url'], true);
+
+        return $sResult;
     }
     
     /**