Request 3114 (accepted)

You're not reviewing the full diff of request 3114 , but the diff to the superseded request 3109 (Show full diff)

Overview

Request 3114 (accepted)

New release

Created by mollekopf over 1 year ago
In state accepted
Supersedes 3109
Open review for Admin
Open review for vanmeeuwen
Open review for mollekopf
Open review for kolab-developers

Submit kolab-syncroton

Submit package Kolab:16:Testing / kolab-syncroton to package Kolab:16 / kolab-syncroton

kolab-syncroton-2.3.22.tar.gz/lib/kolab_sync.php Changed

@@ -45,8 +45,6 @@
     public $username;
     public $password;
 
-    public $task = null;
-
     const CHARSET = 'UTF-8';
     const VERSION = "2.3.22";
 
@@ -209,7 +207,7 @@
             }
 
             // LDAP server failure... send 503 error
-            if ($auth'kolab_ldap_error' ?? null) {
+            if ($auth'kolab_ldap_error') {
                 self::server_error();
             }
 
@@ -225,7 +223,7 @@
         }
 
         // Authenticate - get Roundcube user ID
-        if (!($auth'abort' ?? false) && ($userid = $this->login($auth'user', $auth'pass', $auth'host', $err))) {
+        if (!$auth'abort' && ($userid = $this->login($auth'user', $auth'pass', $auth'host', $err))) {
             // set real username
             $this->username = $auth'user';
             return $userid;
@@ -300,7 +298,6 @@
 
         // parse $host
         $a_host = parse_url($host);
-        $port = null;
         if ($a_host'host') {
             $host = $a_host'host';
             $ssl = (isset($a_host'scheme') && in_array($a_host'scheme', array('ssl','imaps','tls'))) ? $a_host'scheme' : null;

kolab-syncroton-2.3.22.tar.gz/lib/kolab_sync_data_email.php Changed

@@ -633,7 +633,7 @@
     }
 
     /**
-     * Delete an email (or move to Trash)
+     * delete entry
      *
      * @param string                         $folderId
      * @param string                         $serverId
@@ -648,26 +648,17 @@
             throw new Syncroton_Exception_Status_Sync(Syncroton_Exception_Status_Sync::SYNC_SERVER_ERROR);
         }
 
-        // Note: If DeletesAsMoves is not specified in the request, its default is 1 (true).
-
         // move message to trash folder
-        if ((!isset($collection->deletesAsMoves) || !empty($collection->deletesAsMoves))
+        if ($collection->deletesAsMoves
             && strlen($trash)
             && $trash != $msg'foldername'
             && $this->storage->folder_exists($trash)
         ) {
             $this->storage->move_message($msg'uid', $trash, $msg'foldername');
         }
-        // delete the message
+        // set delete flag
         else {
-            // According to the ActiveSync spec. "If the DeletesAsMoves element is set to false,
-            // the deletion is PERMANENT.", therefore we delete the message, and not flag as deleted.
-            $this->storage->delete_message($msg'uid', $msg'foldername');
-
-            // FIXME: We could consider acting according to the 'flag_for_deletion' setting.
-            //        Don't forget about 'read_when_deleted' setting then.
-            // $this->storage->set_flag($msg'uid', 'DELETED', $msg'foldername');
-            // $this->storage->set_flag($msg'uid', 'SEEN', $msg'foldername');
+            $this->storage->set_flag($msg'uid', 'DELETED', $msg'foldername');
         }
     }

plesk.kolab_syncroton.inc.php Changed

Build Results
RPM Lint

Refresh

Comments for request 3114 0

Request History

mollekopf created request over 1 year ago

New release

mollekopf accepted request over 1 year ago