fix(frontend,backend): more unread count fixes

Author: kroky

modules/desktop_notifications/site.js

@@ -23,7 +23,6 @@ $(function() {
     setInterval(() => {
         // undefined_undefined: load with no filter and no keyword
         new Hm_MessagesStore('unread', 1, 'undefined_undefined').load(true, true).then((store) => {
-            $('.total_unread_count').html(' '+store.count+' ');
             store.newMessages.forEach((messageRow) => {
                 triggerNewMessageEvent($(messageRow).data('uid'), $(messageRow)[0]);
             });

modules/imap/handler_modules.php

@@ -1282,7 +1282,13 @@ public function process() {
 
         list($sort, $reverse) = process_sort_arg($this->request->get['sort'], $this->user_config->get('default_sort_order_setting', 'arrival'));
 
-        switch ($this->get('list_path')) {
+        if (isset($this->request->post['list_path'])) {
+            $list_path = $this->request->post['list_path'];
+        } else {
+            $list_path = $this->get('list_path');
+        }
+
+        switch ($list_path) {
             case 'email':
                 $filter = 'ALL';
                 $limit = $this->user_config->get('all_email_per_source_setting', DEFAULT_ALL_EMAIL_PER_SOURCE);
@@ -1295,14 +1301,14 @@ public function process() {
                 break;
             case 'flagged':
             case 'unread':
-                $filter = $this->get('list_path') == 'unread' ? 'UNSEEN' : mb_strtoupper($this->get('list_path'));
+                $filter = $list_path == 'unread' ? 'UNSEEN' : mb_strtoupper($list_path);
             default:
                 if (empty($filter)) {
                     $filter = 'ALL';
                 }
-                if ($this->get('list_path')) {
-                    $limit = $this->user_config->get($this->get('list_path').'_per_source_setting', DEFAULT_PER_SOURCE);
-                    $date = process_since_argument($this->user_config->get($this->get('list_path').'_since_setting', DEFAULT_SINCE));
+                if ($list_path) {
+                    $limit = $this->user_config->get($list_path.'_per_source_setting', DEFAULT_PER_SOURCE);
+                    $date = process_since_argument($this->user_config->get($list_path.'_since_setting', DEFAULT_SINCE));
                 } else {
                     $limit = $this->user_config->get('all_per_source_setting', DEFAULT_ALL_PER_SOURCE);
                     $date = process_since_argument($this->user_config->get('all_since_setting', DEFAULT_SINCE));

modules/imap/site.js

@@ -479,12 +479,10 @@ var setup_imap_folder_page = async function(listPath, listPage = 1) {
         await select_imap_folder(listPath, listPage, true)
     }
 
-    new Hm_MessagesStore('unread', 1, 'undefined_undefined').load(true, true);
-
-    // Refresh in the background each 30 seconds and abort any pending request when the page unmounts
+    // Refresh in the background each 60 seconds
     const interval = setInterval(async () => {
         select_imap_folder(listPath, 1, true, true);
-    }, 30000);
+    }, 60000);
 
     // Return cleanup function to clear interval when page unmounts
     return () => {