Bump up version

SQL/sqlite.initial.sql

@@ -34,18 +34,6 @@ CREATE TABLE ident_switch
   smtp_port
     integer
     CHECK(smtp_port > 0 AND smtp_port <= 65535),
-	smtp_auth
-		smallint
-		NOT NULL
-		DEFAULT 1,
-	drafts_mbox
-		varchar(64),
-	sent_mbox
-		varchar(64),
-	junk_mbox
-		varchar(64),
-	trash_mbox
-		varchar(64),
 	UNIQUE (user_id, label)
 );
 CREATE INDEX IX_ident_switch_user_id ON ident_switch(user_id);

SQL/sqlite/2020121600.sql

@@ -1,92 +0,0 @@
-PRAGMA foreign_keys=off;
-BEGIN TRANSACTION;
-
-ALTER TABLE ident_switch RENAME TO ident_switch_old;
-
-CREATE TABLE ident_switch
-(
-	id
-		integer
-		PRIMARY KEY,
-	user_id
-		integer
-		NOT NULL
-		REFERENCES users(user_id) ON DELETE CASCADE ON UPDATE CASCADE,
-	iid
-		integer
-		NOT NULL
-		REFERENCES identities(identity_id) ON DELETE CASCADE ON UPDATE CASCADE
-		UNIQUE,
-	username
-		varchar(64),
-	password
-		varchar(64),
-	imap_host
-		varchar(64),
-	imap_port
-		integer
-		CHECK(imap_port > 0 AND imap_port <= 65535),
-	imap_delimiter
-		char(1),
-	label
-		varchar(32),
-	flags
-		integer
-		NOT NULL
-		DEFAULT(0),
-	smtp_host
-		varchar(64),
-	smtp_port
-		integer
-		CHECK(smtp_port > 0 AND smtp_port <= 65535),
-	smtp_auth
-		smallint
-		NOT NULL
-		DEFAULT 1,
-	drafts_mbox
-		varchar(64),
-	sent_mbox
-		varchar(64),
-	junk_mbox
-		varchar(64),
-	trash_mbox
-		varchar(64),
-	UNIQUE (user_id, label)
-);
-CREATE INDEX IX_ident_switch_user_id ON ident_switch(user_id);
-CREATE INDEX IX_ident_switch_iid on ident_switch(iid);
-
-INSERT OR ROLLBACK INTO
-  ident_switch (
-    id,
-    user_id,
-    iid,
-    username,
-    password,
-    imap_host,
-    imap_port,
-    label,
-    flags,
-    smtp_host,
-	smtp_port
-  )
-SELECT
-  id,
-  user_id,
-  iid,
-  username,
-  password,
-  imap_host,
-  imap_port,
-  label,
-  flags,
-  smtp_host,
-  smtp_port
-FROM
-  ident_switch_old;
-
-DROP TABLE
-  ident_switch_old;
-
-COMMIT;
-PRAGMA foreign_keys=on;

composer.json

@@ -5,10 +5,11 @@
 	"homepage": "https://bitbucket.org/BoresExpress/ident_switch",
 	"keywords": ["identity", "imap", "mail", "switch"],
     "license": "GPL-3.0+",
+    "version": "4.3.1",
     "authors": [
         {
             "name": "Boris Gulay",
-            "email": "boris@gulay.name",
+            "email": "boris@boressoft.ru",
             "role": "Developer"
         }
     ],

ident_switch.php

@@ -4,7 +4,7 @@
  *
  * This plugin allows fast switching between accounts.
  *
- * @version 4.4.2
+ * @version 1.0
  * @author Boris Gulay
  * @url 
  */
@@ -69,7 +69,7 @@ class ident_switch extends rcube_plugin
 		{
 			$defaultKey = $type . '_mbox_default' . self::MY_POSTFIX;
 			$otherKey = $type . '_mbox' . self::MY_POSTFIX;
-			$val = isset($_SESSION[$otherKey]) ? $_SESSION[$otherKey] : $_SESSION[$defaultKey];
+			$val = $_SESSION[$otherKey] ? $_SESSION[$otherKey] : $_SESSION[$defaultKey];
 			$rc->config->set($type . '_mbox', $val);
 		}
 
@@ -107,7 +107,7 @@ class ident_switch extends rcube_plugin
 		elseif (ctype_digit($iid_s))
 			$iid = intval($iid_s);
 		
-		$accNames = array(isset($_SESSION['global_alias']) ? $_SESSION['global_alias'] : $rc->user->data['username']);
+		$accNames = array($_SESSION['global_alias'] ? $_SESSION['global_alias'] : $rc->user->data['username']);
 		$accValues = array(-1);
 		$accSelected = -1;
 
@@ -159,20 +159,7 @@ class ident_switch extends rcube_plugin
 	{
 		$iid = $_SESSION['iid' . self::MY_POSTFIX];
 		if (!is_numeric($iid) || $iid == -1)
-		{
-			self::write_log('no identity switch is selected... trying to find related smtp server from the from header');
-			$requestFrom = rcube_utils::get_input_value('_from', rcube_utils::INPUT_POST);
-			if (empty($requestFrom)) {
-				self::write_log('no _from post parameter found... falling back to original default config');
 			return $args;
-			} else {
-				$iid = intval($requestFrom);
-				if ($iid == 0) {
-					self::write_log('falling back to original default config as _from post field is no integer: ' . $_POST['_from']); 
-					return $args;
-				}
-			}
-		}
 
 		$rc = rcmail::get_instance();
 
@@ -192,32 +179,16 @@ class ident_switch extends rcube_plugin
 
 			$args['smtp_user'] = $r['username'];
 			$args['smtp_pass'] = $r['smtp_auth'] == self::SMTP_AUTH_IMAP ? $rc->decrypt($r['password']) : '';
-
+			$args['smtp_server'] = $r['smtp_host'] ? $r['smtp_host'] : 'localhost'; // Default SMTP host here
-			# In 1.6 smtp_server was renamed to smtp_host and includes port. smtp_port was depricated.
+			$args['smtp_port'] = $r['smtp_port'] ? $r['smtp_port'] : 587; // Default SMTP port here
-			$verParts = explode('.', RCMAIL_VERSION, 3);
-			$paramSmtpHost = 'smtp_host';
-			$paramSmtpPort = null;
-			if (intval($verParts[0]) <= 1 && intval($verParts[1]) < 6)
-			{
-				$paramSmtpHost = 'smtp_server';
-				$paramSmtpPort = 'smtp_port';
-			}
-
-			$args[$paramSmtpHost] = $r['smtp_host'] ? $r['smtp_host'] : 'localhost'; // Default SMTP host here
 
 			if ($r['flags'] & self::DB_SECURE_IMAP_TLS)
 			{
-				if (strpos($args[$paramSmtpHost], ':') !== false)
+				if (strpos($args['smtp_server'], ':') !== false)
 					self::write_log('SMTP server already contains protocol, ignoring TLS flag.');
 				else
-					$args[$paramSmtpHost] = 'tls://' . $args[$paramSmtpHost];
+					$args['smtp_server'] = 'tls://' . $args['smtp_server'];
 			}
-
-			$smtpPort = $r['smtp_port'] ? $r['smtp_port'] : 587; // Default SMTP port here
-			if ($paramSmtpPort)
-				$args[$paramSmtpPort] = $smtpPort;
-			else
-				$args[$paramSmtpHost] .= ':' . $smtpPort;
 		}
 
 		return $args;
@@ -687,8 +658,6 @@ class ident_switch extends rcube_plugin
 					$rc->session->remove($k);
 				}
 			}
-			$delimiter = isset($config['imap_delimiter']) ? $config['imap_delimiter']:'.';
-            $_SESSION['imap_delimiter'] = $delimiter;
 			$_SESSION['username'] = $rc->user->data['username'];
 			$_SESSION['password'] = $_SESSION['password' . self::MY_POSTFIX];
 			$_SESSION['iid' . self::MY_POSTFIX] = -1;
@@ -696,7 +665,7 @@ class ident_switch extends rcube_plugin
 			foreach (rcube_storage::$folder_types as $type)
 			{
 				$otherKey = $type . '_mbox' . self::MY_POSTFIX;
-				if (isset($_SESSION[$otherKey]))
+				if ($_SESSION[$otherKey])
 					$rc->session->remove($otherKey);
 			}
 		}
@@ -724,7 +693,7 @@ class ident_switch extends rcube_plugin
 					{
 						if (strncasecmp($k, 'storage', 7) === 0 && substr_compare($k, self::MY_POSTFIX, -$my_postfix_len, $my_postfix_len) !== 0)
 						{
-							if (!isset($_SESSION[$k . self::MY_POSTFIX]))
+							if (!$_SESSION[$k . self::MY_POSTFIX])
 								$_SESSION[$k . self::MY_POSTFIX] = $_SESSION[$k];
 
 							$rc->session->remove($k);
@@ -734,7 +703,7 @@ class ident_switch extends rcube_plugin
 					$moreToSave = array('password', 'imap_delimiter');
 					foreach ($moreToSave as $k)
 					{
-						if (!isset($_SESSION[$k . self::MY_POSTFIX]))
+						if (!$_SESSION[$k . self::MY_POSTFIX])
 							$_SESSION[$k . self::MY_POSTFIX] = $_SESSION[$k];
 
 						$rc->session->remove($k);
@@ -755,7 +724,7 @@ class ident_switch extends rcube_plugin
 				if ($ssl && strncasecmp($host, $hostProtocol, strlen($hostProtocol)) !== 0)
 					$host = $hostProtocol . $host;
 
-				$delimiter = isset($r['imap_delimiter']) ? $r['imap_delimiter'] : '.'; // Default delimiter here
+				$delimiter = $r['imap_delimiter'] ? $r['imap_delimiter'] : '.'; // Default delimiter here
 
 				$_SESSION['storage_host'] = $host;
 				$_SESSION['storage_ssl'] = $ssl;
@@ -767,7 +736,7 @@ class ident_switch extends rcube_plugin
 
 				foreach (rcube_storage::$folder_types as $type)
 				{
-					if (isset($r[$type . '_mbox'])) {
+					if ($r[$type . '_mbox']) {
 						$otherKey = $type . '_mbox' . self::MY_POSTFIX;
 						$_SESSION[$otherKey] = $r[$type . '_mbox'];
 					}

localization/sl_SI.inc

@@ -1,76 +0,0 @@
-<?php
-/*
- * Localization file for ident_switch plugin
- */
-$labels = array();
-
-// Plugin ident_switch
-$labels['form.common.caption'] = 'Razširitev ident_switch';
-
-// Enabled
-$labels['form.common.enabled'] = 'Omogočeno';
-
-// Label
-$labels['form.common.label'] = 'Oznaka';
-
-// Value in \'Label\' field is too long (32 chars max).
-$labels['err.label.long'] = 'Število znakov v polju \'Oznaka\' presega maksimalno število znakov (max 32 znakov).';
-
-
-// IMAP
-$labels['form.imap.caption'] = 'IMAP';
-
-// Server host name
-$labels['form.imap.host'] = 'Ime oz. naslov IMAP strežnika';
-
-// Secure connection (TLS)
-$labels['form.imap.tls'] = 'Varna povezava (TLS)';
-
-// Port
-$labels['form.imap.port'] = 'Vrata';
-
-// Folder hierarchy delimiter
-$labels['form.imap.delimiter'] = 'Ločilo hierarhije map';
-
-// Username
-$labels['form.imap.username'] = 'Uporabniško ime';
-
-// Password
-$labels['form.imap.password'] = 'Geslo';
-
-// Value in \'Username\' field is too long (64 chars max).
-$labels['err.user.long'] = 'Število znakov v polju \'Uporabniško ime\' presega maksimalno število znakov (max 64 znakov).';
-
-
-// SMTP
-$labels['form.smtp.caption'] = 'SMTP';
-
-// Server host name
-$labels['form.smtp.host'] = 'Ime oz. naslov SMTP strežnika';
-
-// Secure connection (TLS)
-$labels['form.smtp.tls'] = 'Varna povezava (TLS)';
-
-// Port
-$labels['form.smtp.port'] = 'Vrata';
-
-// Authorization
-$labels['form.smtp.auth'] = 'Avtorizacija';
-
-// As IMAP
-$labels['form.smtp.auth.imap'] = 'Tako kot IMAP';
-
-// None
-$labels['form.smtp.auth.none'] = 'Brez';
-
-
-// Errors
-
-// Value in \'Server host name\' field is too long (64 chars max).
-$labels['err.host.long'] = 'Število znakov v polju \'Ime oz. naslov IMAP strežnika\' presega maksimalno število znakov (max 64 znakov).';
-
-// Value in \'Port\' field must be a number.
-$labels['err.port.num'] = 'Vrednost \'Vrata\' mora biti številka.';
-
-// Value in \'Port\' field must be between 1 and 65535.
-$labels['err.port.range'] = 'Vrednost v polju \'Vrata\' mora biti med 1 in 65535.';