Merge pull request #1208 from Shadow243/update-fixes-before-release

Update fixes before release
cypht-org · Aug 29, 2024 · ef76d24 · ef76d24
2 parents c607012 + 1bd46f3
commit ef76d24
Show file tree

Hide file tree

Showing 32 changed files with 319 additions and 34 deletions.
diff --git a/.github/tests/selenium/creds.py b/.github/tests/selenium/creds.py
@@ -8,7 +8,6 @@
 chrome_options.add_argument('--headless')
 chrome_options.add_argument('--disable-gpu')
 chrome_options.BinaryLocation = "/usr/bin/google-chrome"
-chrome_options.add_argument("--window-size=3200,3800")
 
 chrome_options.headless = False
 chrome_options.add_argument("start-maximized")

diff --git a/composer.lock b/composer.lock
diff --git a/language/az.php b/language/az.php
@@ -635,4 +635,5 @@
     'No tags available yet.' => false,
     'Server capabilities' => false,
     'Capabilities' => false,
+    'Screen %s first emails' => false,
 );
diff --git a/language/de.php b/language/de.php
@@ -632,4 +632,5 @@
     'No tags available yet.' => false,
     'Server capabilities' => false,
     'Capabilities' => false,
+    'Screen %s first emails' => false,
 );
diff --git a/language/en.php b/language/en.php
@@ -650,4 +650,5 @@
     'No tags available yet.' => false,
     'Server capabilities' => false,
     'Capabilities' => false,
+    'Screen %s first emails' => false,
 );
diff --git a/language/es.php b/language/es.php
@@ -632,4 +632,5 @@
     'No tags available yet.' => false,
     'Server capabilities' => false,
     'Capabilities' => false,
+    'Screen %s first emails' => false,
 );
diff --git a/language/et.php b/language/et.php
@@ -640,4 +640,5 @@
     'No tags available yet.' => false,
     'Server capabilities' => false,
     'Capabilities' => false,
+    'Screen %s first emails' => false,
 );
diff --git a/language/fa.php b/language/fa.php
@@ -684,4 +684,5 @@
     'No tags available yet.' => false,
     'Server capabilities' => false,
     'Capabilities' => false,
+    'Screen %s first emails' => false,
 );
diff --git a/language/fr.php b/language/fr.php
@@ -631,4 +631,5 @@
     'No tags available yet.' => false,
     'Server capabilities' => false,
     'Capabilities' => false,
+    'Screen %s first emails' => false,
 );
diff --git a/language/hu.php b/language/hu.php
@@ -632,4 +632,5 @@
     'No tags available yet.' => false,
     'Server capabilities' => false,
     'Capabilities' => false,
+    'Screen %s first emails' => false,
 );
diff --git a/language/id.php b/language/id.php
@@ -639,4 +639,5 @@
     'No tags available yet.' => false,
     'Server capabilities' => false,
     'Capabilities' => false,
+    'Screen %s first emails' => false,
 );
diff --git a/language/it.php b/language/it.php
@@ -632,4 +632,5 @@
     'No tags available yet.' => false,
     'Server capabilities' => false,
     'Capabilities' => false,
+    'Screen %s first emails' => false,
 );
diff --git a/language/ja.php b/language/ja.php
@@ -632,4 +632,5 @@
     'No tags available yet.' => false,
     'Server capabilities' => false,
     'Capabilities' => false,
+    'Screen %s first emails' => false,
 );
diff --git a/language/nl.php b/language/nl.php
@@ -632,4 +632,5 @@
     'No tags available yet.' => false,
     'Server capabilities' => false,
     'Capabilities' => false,
+    'Screen %s first emails' => false,
 );
diff --git a/language/pt-BR.php b/language/pt-BR.php
@@ -631,4 +631,5 @@
     'No tags available yet.' => false,
     'Server capabilities' => false,
     'Capabilities' => false,
+    'Screen %s first emails' => false,
 );
diff --git a/language/ro.php b/language/ro.php
@@ -631,4 +631,5 @@
     'No tags available yet.' => false,
     'Server capabilities' => false,
     'Capabilities' => false,
+    'Screen %s first emails' => false,
 );
diff --git a/language/ru.php b/language/ru.php
@@ -633,4 +633,5 @@
     'No tags available yet.' => false,
     'Server capabilities' => false,
     'Capabilities' => false,
+    'Screen %s first emails' => false,
 );
diff --git a/language/zh-Hans.php b/language/zh-Hans.php
@@ -653,4 +653,5 @@
     'No tags available yet.' => false,
     'Server capabilities' => false,
     'Capabilities' => false,
+    'Screen %s first emails' => false,
 );
diff --git a/lib/repository.php b/lib/repository.php
@@ -55,7 +55,11 @@ public static function add($entity, $save = true) {
 
     public static function edit($id, $entity) {
         if (array_key_exists($id, self::$entities)) {
-            self::$entities[$id] = array_merge(self::$entities[$id], $entity);
+            if (is_array($entity)) {
+                self::$entities[$id] = array_merge(self::$entities[$id], $entity);
+            } else {
+                self::$entities[$id] = $entity;
+            }
             self::save();
             return true;
         }

diff --git a/modules/contacts/modules.php b/modules/contacts/modules.php
@@ -399,6 +399,56 @@ protected function output() {
     }
 }
 
+/**
+ * @subpackage contacts/handler
+ */
+class Hm_Handler_save_contact extends Hm_Handler_Module
+{
+    public function process()
+    {
+        list($success, $form ) = $this->process_form(array('email_address'));
+        if ($success) {
+            $contacts = $this->get('contact_store');
+            $contact_list = $contacts->getAll();
+            $existingEmails = array_map(function($contact){
+                return $contact->value('email_address');
+            },$contact_list);
+
+            $list_mails = array_unique(explode(",", $form['email_address']));
+
+            foreach ($list_mails as $addr) {
+                $addresses = process_address_fld($addr);
+                $newEmails = array_column($addresses, 'email');
+                if (!empty($newEmails)) {
+                    $newContacts = array_filter($newEmails, function ($email) use ($existingEmails) {
+                        return !in_array($email, $existingEmails);
+                    });
+                    $existingContacts = array_filter($existingEmails, function ($email) use ($newEmails) {
+                        return in_array($email, $newEmails);
+                    });
+                    if (!empty($newContacts)) {
+                        $newContacts = array_map(function ($email) {
+                            return ['source' => 'local', 'email_address' => $email, 'display_name' => $email, 'group' => 'Trusted Senders'];
+                        }, $newContacts);
+                        $contacts->add_contact($newContacts[0]);
+                    }
+                    if (!empty($existingContacts)) {
+                        $existingContacts = array_map(function ($email) {
+                            return ['source' => 'local', 'email_address' => $email, 'group' => 'Trusted Senders'];
+                        }, $existingContacts);
+                        foreach ($existingContacts as $key => $contact) {
+                            $contacts->update_contact($key, $contact);
+                        }
+                    }
+                }
+            }
+            $this->session->record_unsaved('Contacts added to  Trusted Contacts list');
+            Hm_Msgs::add('Contacts added to  Trusted Contacts list');
+        }
+
+    }
+}
+
 /**
  * @subpackage contacts/output
  */

diff --git a/modules/contacts/setup.php b/modules/contacts/setup.php
@@ -23,6 +23,8 @@
 add_handler('compose', 'load_contacts', true, 'contacts', 'process_compose_form_submit', 'after');
 add_handler('compose', 'store_contact_message', true, 'contacts', 'load_contacts', 'after');
 
+add_handler('ajax_imap_folder_display', 'load_contacts', true, 'contacts', 'load_user_data', 'after');
+
 add_handler('ajax_imap_message_content', 'load_contacts', true, 'contacts', 'load_user_data', 'after');
 add_handler('ajax_imap_message_content', 'find_message_contacts', true, 'contacts', 'imap_message_content', 'after');
 add_output('ajax_imap_message_content', 'add_message_contacts', true, 'contacts', 'filter_message_headers', 'after');
@@ -31,6 +33,7 @@
 setup_base_ajax_page('ajax_add_contact', 'core');
 add_handler('ajax_add_contact', 'load_contacts', true, 'contacts', 'load_user_data', 'after');
 add_handler('ajax_add_contact', 'save_user_data', true, 'core', 'language', 'after');
+add_handler('ajax_add_contact', 'save_contact',  true);
 
 
 setup_base_ajax_page('ajax_autocomplete_contact', 'core');
@@ -81,8 +84,8 @@
         'contact_source' => FILTER_SANITIZE_FULL_SPECIAL_CHARS,
         'contact_type' => FILTER_SANITIZE_FULL_SPECIAL_CHARS,
         'contact_auto_collect' => FILTER_VALIDATE_BOOLEAN,
-        'enable_warn_contacts_cc_not_exist_in_list_contact' => FILTER_VALIDATE_INT
-
+        'enable_warn_contacts_cc_not_exist_in_list_contact' => FILTER_VALIDATE_INT,
+        'email_address' => FILTER_SANITIZE_FULL_SPECIAL_CHARS
     ),
     'allowed_get' => array(
         'contact_id' => FILTER_SANITIZE_FULL_SPECIAL_CHARS,

diff --git a/modules/core/message_list_functions.php b/modules/core/message_list_functions.php
@@ -206,7 +206,18 @@ function message_list_row($values, $id, $style, $output_mod, $row_class='') {
     if ($row_class) {
         $res .= ' '.$output_mod->html_safe($row_class);
     }
-    $res .= '">';
+    $data_uid = "";
+    if ($uids = explode("_", $id)) {
+        if (isset($uids[2])) {
+            $data_uid = 'data-uid="'. $uids[2] .'"';
+        }
+    }
+    if (!empty($data_uid)) {
+        $res .= '" '.$data_uid.'>';
+    } else {
+        $res .= '">';
+    }
+
     if ($style == 'news') {
         $res .= '<td class="news_cell checkbox_cell">';
     }

diff --git a/modules/core/setup.php b/modules/core/setup.php
@@ -264,6 +264,7 @@
         'filter' => FILTER_DEFAULT,
         'sort' => FILTER_DEFAULT,
         'keyword' => FILTER_DEFAULT,
+        'screen_emails' => FILTER_DEFAULT,
     ),
 
     'allowed_post' => array(

diff --git a/modules/feeds/hm-feed.php b/modules/feeds/hm-feed.php
@@ -75,6 +75,8 @@ class Hm_Feed {
     var $init_cache;
     var $cache_limit;
     var $sort;
+    var $status_code;
+    var $feed_type;
 
     /**
      * Setup defaults

diff --git a/modules/imap/functions.php b/modules/imap/functions.php
@@ -857,19 +857,33 @@ function imap_refresh_oauth2_token($server, $config) {
  * @return int count of messages moved
  */
 if (!hm_exists('imap_move_same_server')) {
-function imap_move_same_server($ids, $action, $hm_cache, $dest_path) {
+function imap_move_same_server($ids, $action, $hm_cache, $dest_path, $screen_emails=false) {
     $moved = array();
     $keys = array_keys($ids);
     $server_id = array_pop($keys);
     $cache = Hm_IMAP_List::get_cache($hm_cache, $server_id);
     $imap = Hm_IMAP_List::connect($server_id, $cache);
     foreach ($ids[$server_id] as $folder => $msgs) {
         if (imap_authed($imap) && $imap->select_mailbox(hex2bin($folder))) {
-            if ($imap->message_action(mb_strtoupper($action), $msgs, hex2bin($dest_path[2]))) {
+            if ($screen_emails) {
                 foreach ($msgs as $msg) {
                     $moved[]  = sprintf('imap_%s_%s_%s', $server_id, $msg, $folder);
+                    $email = current(array_column(process_address_fld($imap->get_message_headers($msg)['From']), "email"));
+                    $uids = $imap->search('ALL', false, array(array('FROM', $email)));
+                    foreach ($uids as $uid) {
+                        if ($imap->message_action(mb_strtoupper($action), $uid, hex2bin($dest_path[2]))) {
+                            $moved[]  = sprintf('imap_%s_%s_%s', $server_id, $uid, $folder);
+                        }
+                    }
+                }
+            } else {
+                if ($imap->message_action(mb_strtoupper($action), $msgs, hex2bin($dest_path[2]))) {
+                    foreach ($msgs as $msg) {
+                        $moved[]  = sprintf('imap_%s_%s_%s', $server_id, $msg, $folder);
+                    }
                 }
             }
+
         }
     }
     return $moved;