use data attribute for accessing serverId instead of picking it from class name

Author: mercihabam

modules/advanced_search/js_modules/route_handlers.js

@@ -72,7 +72,7 @@ function applyAdvancedSearchPageHandlers() {
             size: 'lg',
         });
 
-        const serverId = $(this).closest('li').attr("class").split('_')[1];
+        const serverId = $(this).closest('li').data('serverId');
         const specialFolders = [];
         ['archive', 'draft', 'junk', 'sent', 'trash'].forEach(folderType => {
             const folderName = hm_special_folders()[serverId][folderType];

modules/imap/output_modules.php

@@ -880,7 +880,7 @@ protected function output() {
         $res = '';
         if ($this->get('imap_folders')) {
             foreach ($this->get('imap_folders', array()) as $id => $folder) {
-                $res .= '<li class="imap_'.$id.'_"><a href="#" class="imap_folder_link" data-target="imap_'.$id.'_">';
+                $res .= '<li class="imap_'.$id.'_" data-server-id="' . $id . '"><a href="#" class="imap_folder_link" data-target="imap_'.$id.'_">';
                 if (!$this->get('hide_folder_icons')) {
                     $res .= '<i class="bi bi-folder me-2"></i>';
                 }