grommunio / grommunio-web

plugins/files/php/Files/Core/Util/class.stringutil.php

Indentation +68 added lines, -68 removed lines

@@ -11,77 +11,77 @@
 
 class StringUtil
 {
-	/**
-	 * Return size to human readable filesize
-	 *
-	 * @static
-	 * @param int $filesize in bytes
-	 * @param int $decimals digits
-	 *
-	 * @return string human readable filesize
-	 */
-	static function human_filesize($bytes, $decimals = 2)
-	{
-		$sz = ' KMGTP';
-		$factor = (int)floor((strlen($bytes) - 1) / 3);
-		return sprintf("%.{$decimals}f", $bytes / pow(1024, $factor)) . " " . @$sz[$factor] . "B";
-	}
+    /**
+     * Return size to human readable filesize
+     *
+     * @static
+     * @param int $filesize in bytes
+     * @param int $decimals digits
+     *
+     * @return string human readable filesize
+     */
+    static function human_filesize($bytes, $decimals = 2)
+    {
+        $sz = ' KMGTP';
+        $factor = (int)floor((strlen($bytes) - 1) / 3);
+        return sprintf("%.{$decimals}f", $bytes / pow(1024, $factor)) . " " . @$sz[$factor] . "B";
+    }
 
-	/**
-	 * check if a string starts with a specific char
-	 *
-	 * @static
-	 * @param string $haystack
-	 * @param string $needle
-	 *
-	 * @return boolean true if $needle is found in $haystack
-	 */
-	static function startsWith($haystack, $needle)
-	{
-		return !strncmp($haystack, $needle, strlen($needle));
-	}
+    /**
+     * check if a string starts with a specific char
+     *
+     * @static
+     * @param string $haystack
+     * @param string $needle
+     *
+     * @return boolean true if $needle is found in $haystack
+     */
+    static function startsWith($haystack, $needle)
+    {
+        return !strncmp($haystack, $needle, strlen($needle));
+    }
 
-	/**
-	 * Creates a random string
-	 *
-	 * @static
-	 * @param int length The length of the random string
-	 *
-	 * @return string a random string
-	 */
-	static function randomstring($length = 6)
-	{
-		// $chars - all allowed characters
-		$chars = "abcdefghijklmnopqrstuvwxyzABCDEFGHIJKLMNOPQRSTUVWXYZ1234567890";
+    /**
+     * Creates a random string
+     *
+     * @static
+     * @param int length The length of the random string
+     *
+     * @return string a random string
+     */
+    static function randomstring($length = 6)
+    {
+        // $chars - all allowed characters
+        $chars = "abcdefghijklmnopqrstuvwxyzABCDEFGHIJKLMNOPQRSTUVWXYZ1234567890";
 
-		srand((double)microtime() * 1000000);
-		$i = 0;
-		$pass = "";
-		while ($i < $length) {
-			$num = rand() % strlen($chars);
-			$tmp = substr($chars, $num, 1);
-			$pass = $pass . $tmp;
-			$i++;
-		}
-		return $pass;
-	}
+        srand((double)microtime() * 1000000);
+        $i = 0;
+        $pass = "";
+        while ($i < $length) {
+            $num = rand() % strlen($chars);
+            $tmp = substr($chars, $num, 1);
+            $pass = $pass . $tmp;
+            $i++;
+        }
+        return $pass;
+    }
 
-	/**
-	 * check if a string ends with a specific char
-	 *
-	 * @static
-	 * @param string $haystack
-	 * @param string $needle
-	 *
-	 * @return boolean true if $haystack ends with $needle
-	 */
-	static function endsWith($haystack, $needle)
-	{
-		$length = strlen($needle);
-		if ($length == 0) {
-			return true;
-		}
+    /**
+     * check if a string ends with a specific char
+     *
+     * @static
+     * @param string $haystack
+     * @param string $needle
+     *
+     * @return boolean true if $haystack ends with $needle
+     */
+    static function endsWith($haystack, $needle)
+    {
+        $length = strlen($needle);
+        if ($length == 0) {
+            return true;
+        }
 
-		return (substr($haystack, -$length) === $needle);
-	}
+        return (substr($haystack, -$length) === $needle);
+    }
 }

Spacing +2 added lines, -2 removed lines

@@ -23,7 +23,7 @@ 
 	static function human_filesize($bytes, $decimals = 2)
 	{
 		$sz = ' KMGTP';
-		$factor = (int)floor((strlen($bytes) - 1) / 3);
+		$factor = (int) floor((strlen($bytes) - 1) / 3);
 		return sprintf("%.{$decimals}f", $bytes / pow(1024, $factor)) . " " . @$sz[$factor] . "B";
 	}
 
@@ -54,7 +54,7 @@ 
 		// $chars - all allowed characters
 		$chars = "abcdefghijklmnopqrstuvwxyzABCDEFGHIJKLMNOPQRSTUVWXYZ1234567890";
 
-		srand((double)microtime() * 1000000);
+		srand((double) microtime() * 1000000);
 		$i = 0;
 		$pass = "";
 		while ($i < $length) {

plugins/files/php/Files/Core/Util/util.php

Indentation +1 added lines, -1 removed lines

@@ -11,7 +11,7 @@
 //	if (!mb_detect_encoding($str, "UTF-8", true)) {

 //		$str = utf8_encode($str);

 //	}

-	return $str;

+    return $str;

 }

 

 

plugins/files/php/Files/Core/Util/class.arrayutil.php

Indentation +153 added lines, -153 removed lines

@@ -4,157 +4,157 @@
 
 class ArrayUtil
 {
-	/**
-	 * Sort multidimensional array by any key
-	 *
-	 * @static
-	 * @param array $arr the array to sort
-	 * @param string $key the key to sort
-	 * @param string $dir ASC or DESC sort direction
-	 *
-	 * @return array the sorted array
-	 */
-	static function sort_by_key($arr, $key, $dir)
-	{
-		global $key2sort;
-		$key2sort = $key;
-
-		if ($dir == "DESC") {
-			usort($arr, array('self', 'invsort'));
-		} else {
-			usort($arr, array('self', 'sort'));
-		}
-		return ($arr);
-	}
-
-	/**
-	 * Sort multidimensional properties array by any key
-	 *
-	 * @static
-	 * @param array $arr the array to sort
-	 * @param string $key the key to sort
-	 * @param string $dir ASC or DESC sort direction
-	 *
-	 * @return array the sorted array
-	 */
-	static function sort_props_by_key($arr, $key, $dir)
-	{
-		global $key2sort;
-		$key2sort = $key;
-
-		if ($dir == "DESC") {
-			usort($arr, array('self', 'invpropsort'));
-		} else {
-			usort($arr, array('self', 'propsort'));
-		}
-		return ($arr);
-	}
-
-	/**
-	 * compare function for multidimensional array sorting
-	 *
-	 * @static
-	 * @param array $a this argument will be compared with argument $b
-	 * @param array $b this argument will be compared with argument $a
-	 *
-	 * @return int compare value. If $a < $b the return value will be -1.
-	 */
-	static function sort($a, $b)
-	{
-		global $key2sort;
-
-		if ($a['isFolder'] == $b['isFolder']) {
-			if (is_numeric($a[$key2sort]) && is_numeric($b[$key2sort])) {
-				if ($a[$key2sort] == $b[$key2sort]) {
-					return 0;
-				}
-				return ($a[$key2sort] < $b[$key2sort]) ? -1 : 1;
-			} else {
-				return (strcasecmp($a[$key2sort], $b[$key2sort]));
-			}
-		}
-
-		return ((int)$a['isFolder'] - (int)$b['isFolder']);
-	}
-
-	/**
-	 * inverse compare function for multidimensional array sorting
-	 *
-	 * @static
-	 * @param array $a this argument will be compared with argument $b
-	 * @param array $b this argument will be compared with argument $a
-	 *
-	 * @return int compare value. If $a < $b the return value will be 1.
-	 */
-	static function invsort($a, $b)
-	{
-		global $key2sort;
-
-		if ($a['isFolder'] == $b['isFolder']) {
-			if (is_numeric($a[$key2sort]) && is_numeric($b[$key2sort])) {
-				if ($a[$key2sort] == $b[$key2sort]) {
-					return 0;
-				}
-				return ($a[$key2sort] < $b[$key2sort]) ? 1 : -1;
-			} else {
-				return (-1 * strcasecmp($a[$key2sort], $b[$key2sort]));
-			}
-		}
-
-		return ((int)$b['isFolder'] - (int)$a['isFolder']);
-	}
-
-	/**
-	 * compare function for multidimensional array sorting
-	 *
-	 * @static
-	 * @param array $a this argument will be compared with argument $b
-	 * @param array $b this argument will be compared with argument $a
-	 *
-	 * @return int compare value. If $a < $b the return value will be -1.
-	 */
-	static function propsort($a, $b)
-	{
-		global $key2sort;
-
-		if ($a['props']['type'] == $b['props']['type']) {
-			if (is_numeric($a['props'][$key2sort]) && is_numeric($b['props'][$key2sort])) {
-				if ($a['props'][$key2sort] == $b['props'][$key2sort]) {
-					return 0;
-				}
-				return ($a['props'][$key2sort] < $b['props'][$key2sort]) ? -1 : 1;
-			} else {
-				return (strcasecmp($a['props'][$key2sort], $b['props'][$key2sort]));
-			}
-		}
-
-		return ($a['props']['type'] - $b['props']['type']);
-	}
-
-	/**
-	 * inverse compare function for multidimensional array sorting
-	 *
-	 * @static
-	 * @param array $a this argument will be compared with argument $b
-	 * @param array $b this argument will be compared with argument $a
-	 *
-	 * @return int compare value. If $a < $b the return value will be 1.
-	 */
-	static function invpropsort($a, $b)
-	{
-		global $key2sort;
-
-		if ($a['props']['type'] == $b['props']['type']) {
-			if (is_numeric($a['props'][$key2sort]) && is_numeric($b['props'][$key2sort])) {
-				if ($a['props'][$key2sort] == $b['props'][$key2sort]) {
-					return 0;
-				}
-				return ($a['props'][$key2sort] < $b['props'][$key2sort]) ? 1 : -1;
-			} else {
-				return (-1 * strcasecmp($a['props'][$key2sort], $b['props'][$key2sort]));
-			}
-		}
-
-		return ($b['props']['type'] - $a['props']['type']);
-	}
+    /**
+     * Sort multidimensional array by any key
+     *
+     * @static
+     * @param array $arr the array to sort
+     * @param string $key the key to sort
+     * @param string $dir ASC or DESC sort direction
+     *
+     * @return array the sorted array
+     */
+    static function sort_by_key($arr, $key, $dir)
+    {
+        global $key2sort;
+        $key2sort = $key;
+
+        if ($dir == "DESC") {
+            usort($arr, array('self', 'invsort'));
+        } else {
+            usort($arr, array('self', 'sort'));
+        }
+        return ($arr);
+    }
+
+    /**
+     * Sort multidimensional properties array by any key
+     *
+     * @static
+     * @param array $arr the array to sort
+     * @param string $key the key to sort
+     * @param string $dir ASC or DESC sort direction
+     *
+     * @return array the sorted array
+     */
+    static function sort_props_by_key($arr, $key, $dir)
+    {
+        global $key2sort;
+        $key2sort = $key;
+
+        if ($dir == "DESC") {
+            usort($arr, array('self', 'invpropsort'));
+        } else {
+            usort($arr, array('self', 'propsort'));
+        }
+        return ($arr);
+    }
+
+    /**
+     * compare function for multidimensional array sorting
+     *
+     * @static
+     * @param array $a this argument will be compared with argument $b
+     * @param array $b this argument will be compared with argument $a
+     *
+     * @return int compare value. If $a < $b the return value will be -1.
+     */
+    static function sort($a, $b)
+    {
+        global $key2sort;
+
+        if ($a['isFolder'] == $b['isFolder']) {
+            if (is_numeric($a[$key2sort]) && is_numeric($b[$key2sort])) {
+                if ($a[$key2sort] == $b[$key2sort]) {
+                    return 0;
+                }
+                return ($a[$key2sort] < $b[$key2sort]) ? -1 : 1;
+            } else {
+                return (strcasecmp($a[$key2sort], $b[$key2sort]));
+            }
+        }
+
+        return ((int)$a['isFolder'] - (int)$b['isFolder']);
+    }
+
+    /**
+     * inverse compare function for multidimensional array sorting
+     *
+     * @static
+     * @param array $a this argument will be compared with argument $b
+     * @param array $b this argument will be compared with argument $a
+     *
+     * @return int compare value. If $a < $b the return value will be 1.
+     */
+    static function invsort($a, $b)
+    {
+        global $key2sort;
+
+        if ($a['isFolder'] == $b['isFolder']) {
+            if (is_numeric($a[$key2sort]) && is_numeric($b[$key2sort])) {
+                if ($a[$key2sort] == $b[$key2sort]) {
+                    return 0;
+                }
+                return ($a[$key2sort] < $b[$key2sort]) ? 1 : -1;
+            } else {
+                return (-1 * strcasecmp($a[$key2sort], $b[$key2sort]));
+            }
+        }
+
+        return ((int)$b['isFolder'] - (int)$a['isFolder']);
+    }
+
+    /**
+     * compare function for multidimensional array sorting
+     *
+     * @static
+     * @param array $a this argument will be compared with argument $b
+     * @param array $b this argument will be compared with argument $a
+     *
+     * @return int compare value. If $a < $b the return value will be -1.
+     */
+    static function propsort($a, $b)
+    {
+        global $key2sort;
+
+        if ($a['props']['type'] == $b['props']['type']) {
+            if (is_numeric($a['props'][$key2sort]) && is_numeric($b['props'][$key2sort])) {
+                if ($a['props'][$key2sort] == $b['props'][$key2sort]) {
+                    return 0;
+                }
+                return ($a['props'][$key2sort] < $b['props'][$key2sort]) ? -1 : 1;
+            } else {
+                return (strcasecmp($a['props'][$key2sort], $b['props'][$key2sort]));
+            }
+        }
+
+        return ($a['props']['type'] - $b['props']['type']);
+    }
+
+    /**
+     * inverse compare function for multidimensional array sorting
+     *
+     * @static
+     * @param array $a this argument will be compared with argument $b
+     * @param array $b this argument will be compared with argument $a
+     *
+     * @return int compare value. If $a < $b the return value will be 1.
+     */
+    static function invpropsort($a, $b)
+    {
+        global $key2sort;
+
+        if ($a['props']['type'] == $b['props']['type']) {
+            if (is_numeric($a['props'][$key2sort]) && is_numeric($b['props'][$key2sort])) {
+                if ($a['props'][$key2sort] == $b['props'][$key2sort]) {
+                    return 0;
+                }
+                return ($a['props'][$key2sort] < $b['props'][$key2sort]) ? 1 : -1;
+            } else {
+                return (-1 * strcasecmp($a['props'][$key2sort], $b['props'][$key2sort]));
+            }
+        }
+
+        return ($b['props']['type'] - $a['props']['type']);
+    }
 }
\ No newline at end of file

Spacing +2 added lines, -2 removed lines

@@ -74,7 +74,7 @@ 
 			}
 		}
 
-		return ((int)$a['isFolder'] - (int)$b['isFolder']);
+		return ((int) $a['isFolder'] - (int) $b['isFolder']);
 	}
 
 	/**
@@ -101,7 +101,7 @@ 
 			}
 		}
 
-		return ((int)$b['isFolder'] - (int)$a['isFolder']);
+		return ((int) $b['isFolder'] - (int) $a['isFolder']);
 	}
 
 	/**

plugins/files/php/Files/Core/Util/class.logger.php

Indentation +18 added lines, -18 removed lines

@@ -4,24 +4,24 @@
 // TODO: implement logging levels
 class Logger
 {
-	public static function log($context, $msg)
-	{
-		if (PLUGIN_FILESBROWSER_LOGLEVEL === "DEBUG" || PLUGIN_FILESBROWSER_LOGLEVEL === "NORMAL") {
-			error_log("[INFO][$context] " . print_r($msg, true));
-		}
-	}
+    public static function log($context, $msg)
+    {
+        if (PLUGIN_FILESBROWSER_LOGLEVEL === "DEBUG" || PLUGIN_FILESBROWSER_LOGLEVEL === "NORMAL") {
+            error_log("[INFO][$context] " . print_r($msg, true));
+        }
+    }
 
-	public static function error($context, $msg)
-	{
-		if (PLUGIN_FILESBROWSER_LOGLEVEL === "ERROR" || PLUGIN_FILESBROWSER_LOGLEVEL === "DEBUG" || PLUGIN_FILESBROWSER_LOGLEVEL === "NORMAL") {
-			error_log("[ERROR][$context] " . print_r($msg, true));
-		}
-	}
+    public static function error($context, $msg)
+    {
+        if (PLUGIN_FILESBROWSER_LOGLEVEL === "ERROR" || PLUGIN_FILESBROWSER_LOGLEVEL === "DEBUG" || PLUGIN_FILESBROWSER_LOGLEVEL === "NORMAL") {
+            error_log("[ERROR][$context] " . print_r($msg, true));
+        }
+    }
 
-	public static function debug($context, $msg)
-	{
-		if (PLUGIN_FILESBROWSER_LOGLEVEL === "DEBUG") {
-			error_log("[DBG][$context] " . print_r($msg, true));
-		}
-	}
+    public static function debug($context, $msg)
+    {
+        if (PLUGIN_FILESBROWSER_LOGLEVEL === "DEBUG") {
+            error_log("[DBG][$context] " . print_r($msg, true));
+        }
+    }
 }
\ No newline at end of file

plugins/files/php/Files/Core/class.exception.php

Indentation +30 added lines, -30 removed lines

@@ -5,37 +5,37 @@
 
 class Exception extends \Exception
 {	
-	/**
-	 * The exception title to show as a message box title at client side.
-	 */
-	public $title = null;
+    /**
+     * The exception title to show as a message box title at client side.
+     */
+    public $title = null;
 
-	/**
-	 * @constructor
-	 * @param string $message The error message
-	 * @param int $code The error code
-	 */
-	public function __construct($message, $code = 0)
-	{
-		parent::__construct($message, $code);
-	}
+    /**
+     * @constructor
+     * @param string $message The error message
+     * @param int $code The error code
+     */
+    public function __construct($message, $code = 0)
+    {
+        parent::__construct($message, $code);
+    }
 
-	/**
-	 * Function sets title of an exception that will be sent to the client side
-	 * to show it to user.
-	 * @param string $title title of an exception.
-	 */
-	public function setTitle($title)
-	{
-		$this->title = $title;
-	}
+    /**
+     * Function sets title of an exception that will be sent to the client side
+     * to show it to user.
+     * @param string $title title of an exception.
+     */
+    public function setTitle($title)
+    {
+        $this->title = $title;
+    }
 	
-	/**
-	 * @return string returns title that should be sent to client to display as a message box
-	 * title.
-	 */
-	public function getTitle()
-	{
-		return $this->title;
-	}
+    /**
+     * @return string returns title that should be sent to client to display as a message box
+     * title.
+     */
+    public function getTitle()
+    {
+        return $this->title;
+    }
 }

plugins/files/php/Files/Core/class.recipienthandler.php

Indentation +34 added lines, -34 removed lines

@@ -13,40 +13,40 @@
 
 class RecipientHandler
 {
-	const LOG_CONTEXT = "RecipientHandler"; // Context for the Logger
-
-	public static function doGetRecipients()
-	{
-		// parse account id.
-		// wo only need to parse one string because it is
-		// only possible to download files from one backend at a time.
-		if (isset($_GET["ids"])) {
-			$tmpId = $_GET["ids"][0];
-		} else {
-			$tmpId = $_GET["id"];
-		}
-		$accountID = substr($tmpId, 3, (strpos($tmpId, '/') - 3));
-
-		// Initialize the account and backendstore
-		$accountStore = new \Files\Core\AccountStore();
-		$backendStore = \Files\Backend\BackendStore::getInstance();
-
-		$account = $accountStore->getAccount($accountID);
-
-		// initialize the backend
-		$initializedBackend = $backendStore->getInstanceOfBackend($account->getBackend());
-		$initializedBackend->init_backend($account->getBackendConfig());
-
-		try {
-			$initializedBackend->open();
-		} catch (\Files\Backend\Exception $e) {
-			Logger::error(self::LOG_CONTEXT, "Could not open the backend: " . $e->getMessage());
+    const LOG_CONTEXT = "RecipientHandler"; // Context for the Logger
+
+    public static function doGetRecipients()
+    {
+        // parse account id.
+        // wo only need to parse one string because it is
+        // only possible to download files from one backend at a time.
+        if (isset($_GET["ids"])) {
+            $tmpId = $_GET["ids"][0];
+        } else {
+            $tmpId = $_GET["id"];
+        }
+        $accountID = substr($tmpId, 3, (strpos($tmpId, '/') - 3));
+
+        // Initialize the account and backendstore
+        $accountStore = new \Files\Core\AccountStore();
+        $backendStore = \Files\Backend\BackendStore::getInstance();
+
+        $account = $accountStore->getAccount($accountID);
+
+        // initialize the backend
+        $initializedBackend = $backendStore->getInstanceOfBackend($account->getBackend());
+        $initializedBackend->init_backend($account->getBackendConfig());
+
+        try {
+            $initializedBackend->open();
+        } catch (\Files\Backend\Exception $e) {
+            Logger::error(self::LOG_CONTEXT, "Could not open the backend: " . $e->getMessage());
                         echo json_encode(array('success' => false, 'response' => $e->getCode(), 'message' => $e->getMessage()));
-			die();
-		}
-		$responsedata = $initializedBackend->getRecipients($_GET["query"]);
-		header('Content-Type: application/json');
-		echo json_encode($responsedata);
-	}
+            die();
+        }
+        $responsedata = $initializedBackend->getRecipients($_GET["query"]);
+        header('Content-Type: application/json');
+        echo json_encode($responsedata);
+    }
 
 }

plugins/files/php/Files/Core/class.accountstore.php

Indentation +365 added lines, -365 removed lines

@@ -18,369 +18,369 @@
 
 class AccountStore
 {
-	const LOG_CONTEXT = "AccountStore"; // Context for the Logger
-	const ACCOUNT_STORAGE_PATH = "zarafa/v1/plugins/files/accounts";
-	const ACCOUNT_VERSION = 1;
-
-	/**
-	 * @var Account[] Account array
-	 */
-	private $accounts = [];
-
-	/**
-	 *
-	 */
-	function __construct()
-	{
-		$this->initialiseAccounts();
-	}
-
-	/**
-	 * @param $name
-	 * @param $backend
-	 * @param Array $backendConfig Backend specific account settings
-	 *     like username, password, serveraddress, ...
-	 *
-	 * @return Account
-	 */
-	public function createAccount($name, $backend, $backendConfig)
-	{
-		$newID = $this->createNewId($backendConfig); // create id out of the configuration
-
-		// create instance of backend to get features
-		$backendStore = BackendStore::getInstance();
-		$backendInstance = $backendStore->getInstanceOfBackend($backend);
-		$features = $backendInstance->getAvailableFeatures();
-
-		// check backend_config for validity
-		$status = $this->checkBackendConfig($backendInstance, $backendConfig);
-
-		// get sequence number
-		$sequence = $this->getNewSequenceNumber();
-
-		$newAccount = new Account($newID, strip_tags($name), $status[0], $status[1], strip_tags($backend), $backendConfig, $features, $sequence, false);
-
-		// now store all the values to the user settings
-		$GLOBALS["settings"]->set(self::ACCOUNT_STORAGE_PATH . "/" . $newID . "/id", $newAccount->getId());
-		$GLOBALS["settings"]->set(self::ACCOUNT_STORAGE_PATH . "/" . $newID . "/name", $newAccount->getName());
-		$GLOBALS["settings"]->set(self::ACCOUNT_STORAGE_PATH . "/" . $newID . "/status", $newAccount->getStatus());
-		$GLOBALS["settings"]->set(self::ACCOUNT_STORAGE_PATH . "/" . $newID . "/status_description", $newAccount->getStatusDescription());
-		$GLOBALS["settings"]->set(self::ACCOUNT_STORAGE_PATH . "/" . $newID . "/backend", $newAccount->getBackend());
-		$GLOBALS["settings"]->set(self::ACCOUNT_STORAGE_PATH . "/" . $newID . "/account_sequence", $newAccount->getSequence());
-		// User defined accounts are never administrative. So set cannot_change to false.
-		$GLOBALS["settings"]->set(self::ACCOUNT_STORAGE_PATH . "/" . $newID . "/cannot_change", false);
-		$GLOBALS["settings"]->set(self::ACCOUNT_STORAGE_PATH . "/" . $newID . "/backend_config/version", self::ACCOUNT_VERSION);
-		// store all backend configurations
-		foreach ($newAccount->getBackendConfig() as $key => $value) {
-			if ($key !== "version") {
-				$GLOBALS["settings"]->set(self::ACCOUNT_STORAGE_PATH . "/" . $newID . "/backend_config/" . $key, $this->encryptBackendConfigProperty($value, self::ACCOUNT_VERSION));
-			}
-		}
-
-		// store all features
-		foreach ($newAccount->getFeatures() as $feature) {
-			$GLOBALS["settings"]->set(self::ACCOUNT_STORAGE_PATH . "/" . $newID . "/backend_features/" . $feature, true);
-		}
-
-		$GLOBALS["settings"]->saveSettings(); // save to MAPI storage
-
-		// add account to our local store after it was saved to the zarafa-settings
-		$this->accounts[$newID] = $newAccount;
-
-		return $newAccount;
-	}
-
-	/**
-	 * @param Account $account
-	 *
-	 * @return Account
-	 */
-	public function updateAccount($account)
-	{
-		$accId = $account->getId();
-		$isAdministrativeAccount = $account->getCannotChangeFlag();
-
-		// create instance of backend to get features
-		$backendStore = BackendStore::getInstance();
-		$backendInstance = $backendStore->getInstanceOfBackend($account->getBackend());
-		$features = $backendInstance->getAvailableFeatures();
-		$account->setFeatures($features);
-
-		// check backend_config for validity
-		$status = $this->checkBackendConfig($backendInstance, $account->getBackendConfig());
-		$account->setStatus($status[0]); // update status
-		$account->setStatusDescription($status[1]); // update status description
-
-		// add account to local store
-		$this->accounts[$accId] = $account;
-
-		// save values to MAPI settings
-		// now store all the values to the user settings
-		// but if we have an administrative account only save the account sequence
-		if (!$isAdministrativeAccount) {
-			$GLOBALS["settings"]->set(self::ACCOUNT_STORAGE_PATH . "/" . $accId . "/name", $account->getName());
-			$GLOBALS["settings"]->set(self::ACCOUNT_STORAGE_PATH . "/" . $accId . "/status", $account->getStatus());
-			$GLOBALS["settings"]->set(self::ACCOUNT_STORAGE_PATH . "/" . $accId . "/status_description", $account->getStatusDescription());
-			$GLOBALS["settings"]->set(self::ACCOUNT_STORAGE_PATH . "/" . $accId . "/backend", $account->getBackend());
-
-			$acc = $account->getBackendConfig();
-			$version = 0;
-			if (isset($acc["version"])) {
-				$version = $acc["version"];
-			}
-
-			// Unable to decrypt, don't update
-			if ($version == 0 && !defined('FILES_PASSWORD_IV') && !defined('FILES_PASSWORD_KEY'))  {
-				Logger::error(self::LOG_CONTEXT, "Unable to update the account to as FILES_PASSWORD_IV/FILES_PASSWORD_KEY is not set");
-			} else {
-				// store all backend configurations
-				foreach ($account->getBackendConfig() as $key => $value) {
-					if ($key !== "version") {
-						$GLOBALS["settings"]->set(self::ACCOUNT_STORAGE_PATH . "/" . $accId . "/backend_config/" . $key, $this->encryptBackendConfigProperty($value, self::ACCOUNT_VERSION));
-					}
-				}
-
-				$GLOBALS["settings"]->set(self::ACCOUNT_STORAGE_PATH . "/" . $accId . "/backend_config/version", self::ACCOUNT_VERSION);
-			}
-
-			// store all features
-			foreach ($account->getFeatures() as $feature) {
-				$GLOBALS["settings"]->set(self::ACCOUNT_STORAGE_PATH . "/" . $accId . "/backend_features/" . $feature, true);
-			}
-		}
-		// when getSequence returns 0, there is no account_sequence setting yet. So create one.
-		$account_sequence = ($account->getSequence() === 0 ? $this->getNewSequenceNumber() : $account->getSequence());
-		$GLOBALS["settings"]->set(self::ACCOUNT_STORAGE_PATH . "/" . $accId . "/account_sequence", $account_sequence);
-
-		$GLOBALS["settings"]->saveSettings(); // save to MAPI storage
-
-		return $account;
-	}
-
-	/**
-	 * Delete account from local store and from the MAPI settings
-	 *
-	 * @param $accountId
-	 *
-	 * @return bool
-	 */
-	public function deleteAccount($accountId)
-	{
-		$account = $this->getAccount($accountId);
-		// Do not allow deleting administrative accounts, but fail silently.
-		if (!$account->getCannotChangeFlag()) {
-			$GLOBALS["settings"]->delete(self::ACCOUNT_STORAGE_PATH . "/" . $accountId);
-			$GLOBALS["settings"]->saveSettings(); // save to MAPI storage
-		}
-
-		return true;
-	}
-
-	/**
-	 * Return the instance of the local account
-	 *
-	 * @param $accountId
-	 *
-	 * @return Account
-	 */
-	public function getAccount($accountId)
-	{
-		return $this->accounts[$accountId];
-	}
-
-	/**
-	 * @return Account[] all Accounts
-	 */
-	public function getAllAccounts()
-	{
-		return $this->accounts;
-	}
-
-	/**
-	 * Initialize the accountstore. Reads all accountinformation from the MAPI settings.
-	 */
-	private function initialiseAccounts()
-	{
-		// Parse accounts from the Settings
-		$tmpAccs = $GLOBALS["settings"]->get(self::ACCOUNT_STORAGE_PATH);
-
-		if (is_array($tmpAccs)) {
-			$this->accounts = array();
-
-			foreach ($tmpAccs as $acc) {
-				// set backend_features if it is not set to prevent warning
-				if (!isset($acc["backend_features"])) {
-					$acc["backend_features"] = array();
-				}
-				// account_sequence was introduced later. So set and save it if missing.
-				if (!isset($acc["account_sequence"])) {
-					$acc["account_sequence"] = $this->getNewSequenceNumber();
-					Logger::debug(self::LOG_CONTEXT, "Account sequence missing. New seq: " . $acc["account_sequence"]);
-					$GLOBALS["settings"]->set(self::ACCOUNT_STORAGE_PATH . "/" . $acc["id"] . "/account_sequence", $acc["account_sequence"]);
-					$GLOBALS["settings"]->saveSettings();
-				}
-				// cannot_change flag was introduced later. So set it to false and save it if missing.
-				if (!isset($acc["cannot_change"])) {
-					$acc["cannot_change"] = false;
-					Logger::debug(self::LOG_CONTEXT, "Cannot change flag missing. Setting to false.");
-					$GLOBALS["settings"]->set(self::ACCOUNT_STORAGE_PATH . "/" . $acc["id"] . "/cannot_change", false);
-					$GLOBALS["settings"]->saveSettings();
-				}
-
-				$backend_config = $acc["backend_config"];
-				$version = 0;
-
-				if (isset($acc["backend_config"]) && isset($acc["backend_config"]["version"])) {
-					$version = $acc["backend_config"]["version"];
-				}
-
-				if (($version === 0 && defined('FILES_PASSWORD_IV') && defined('FILES_PASSWORD_KEY')) || $version === self::ACCOUNT_VERSION) {
-					$backend_config = $this->decryptBackendConfig($acc["backend_config"], $version);
-					// version is lost after decryption, add it again
-					$backend_config["version"] = $version;
-				} else if ($version === 0) {
-					Logger::error(self::LOG_CONTEXT, "FILES_PASSWORD_IV or FILES_PASSWORD_KEY not set, unable to decrypt backend configuration");
-				} else {
-					Logger::error(self::LOG_CONTEXT, "Unsupported account version $version, unable to decrypt backend configuration");
-				}
-
-				$this->accounts[$acc["id"]] = new Account($acc["id"],
-					$acc["name"],
-					$acc["status"],
-					$acc["status_description"],
-					$acc["backend"],
-					$backend_config,
-					array_keys($acc["backend_features"]),
-					$acc["account_sequence"],
-					$acc["cannot_change"]
-				);
-			}
-		}
-		Logger::debug(self::LOG_CONTEXT, "Found " . count($this->accounts) . " accounts.");
-	}
-
-	/**
-	 * @param AbstractBackend $backendInstance
-	 * @param Array $backendConfig Backend specific account settings
-	 *     like username, password, serveraddress, ...
-	 *
-	 * @return Array
-	 */
-	private function checkBackendConfig($backendInstance, $backendConfig)
-	{
-		$status = Account::STATUS_NEW;
-		$description = _('Account is ready to use.');
-		try {
-			$backendInstance->init_backend($backendConfig);
-			$backendInstance->open();
-			$backendInstance->ls("/");
-			$status = Account::STATUS_OK;
-		} catch (BackendException $e) {
-			$status = Account::STATUS_ERROR;
-			$description = $e->getMessage();
-
-			Logger::error(self::LOG_CONTEXT, "Account check failed: " . $description);
-		}
-
-		return array($status, $description);
-	}
-
-	/**
-	 * @param Array $backendConfig Backend specific account settings
-	 *     like username, password, serveraddress, ...
-	 *
-	 * @return  an unique id
-	 */
-	private function createNewId($backendConfig)
-	{
-		// lets create a hash
-		return md5(json_encode($backendConfig) . time()); // json_encode is faster than serialize
-	}
-
-	/**
-	 * Generate a new sequence number. It will always be the highest used sequence number +1.
-	 *
-	 * @return int
-	 */
-	private function getNewSequenceNumber() {
-		$seq = 0;
-		foreach($this->accounts as $acc) {
-			if($acc->getSequence() > $seq) {
-				$seq = $acc->getSequence();
-			}
-		}
-
-		return $seq + 1;
-	}
-
-	/**
-	 * Decrypt the backend configuration using the standard grommunio Web key.
-	 *
-	 * @param Array $backendConfig Backend specific account settings
-	 *     like username, password, serveraddress, ...
-	 * @return array
-	 */
-	private function decryptBackendConfig($backendConfig, $version=0) {
-		$decBackendConfig = array();
-
-		foreach($backendConfig as $key => $value) {
-			if ($key !== "version") {
-				try {
-					$decBackendConfig[$key] = $this->decryptBackendConfigProperty($value, $version);
-				} catch (Exception $e) {
-					Logger::error(self::LOG_CONTEXT, sprintf("Unable to decrypt backend configuration: '%s'", $e->getMessage()));
-				}
-			}
-		}
-
-		return $decBackendConfig;
-	}
-
-	/**
-	 * Encrypt the given string.
-	 *
-	 * @param $value
-	 * @param $version the storage version used to identify what encryption to use
-	 * @return string
-	 */
-	private function encryptBackendConfigProperty($value, $version=0) {
-		if ($version == self::ACCOUNT_VERSION && !is_bool($value)) {
-			$nonce = random_bytes(SODIUM_CRYPTO_SECRETBOX_NONCEBYTES);
-			$key = $GLOBALS["operations"]->getFilesEncryptionKey();
-			$encrypted = sodium_crypto_secretbox($value, $nonce, $key);
-			$value = bin2hex($nonce) . bin2hex($encrypted);
-		} else if ($version !== self::ACCOUNT_VERSION) {
-			throw Exception("Unable to encrypt backend configuration unsupported version $version");
-		}
-
-		return $value;
-	}
-
-	/**
-	 * Decrypt the given string.
-	 *
-	 * @param $value
-	 * @param $version the storage version used to identify what encryption to use
-	 * @return string
-	 */
-	private function decryptBackendConfigProperty($value, $version=0) {
-		if (is_bool($value)) {
-			return $value;
-		}
-
-		if ($version == self::ACCOUNT_VERSION) {
-			$value = hex2bin($value);
-			$nonce = substr($value, 0, SODIUM_CRYPTO_SECRETBOX_NONCEBYTES);
-			$encrypted = substr($value, SODIUM_CRYPTO_SECRETBOX_NONCEBYTES, strlen($value));
-			$key = $GLOBALS["operations"]->getFilesEncryptionKey();
-			$value = sodium_crypto_secretbox_open($encrypted, $nonce, $key);
-
-			// Decryption failed, password might have changed
-			if ($value === false) {
-				throw new Exception("invalid password");
-			}
-		}
-
-		return $value;
-	}
+    const LOG_CONTEXT = "AccountStore"; // Context for the Logger
+    const ACCOUNT_STORAGE_PATH = "zarafa/v1/plugins/files/accounts";
+    const ACCOUNT_VERSION = 1;
+
+    /**
+     * @var Account[] Account array
+     */
+    private $accounts = [];
+
+    /**
+     *
+     */
+    function __construct()
+    {
+        $this->initialiseAccounts();
+    }
+
+    /**
+     * @param $name
+     * @param $backend
+     * @param Array $backendConfig Backend specific account settings
+     *     like username, password, serveraddress, ...
+     *
+     * @return Account
+     */
+    public function createAccount($name, $backend, $backendConfig)
+    {
+        $newID = $this->createNewId($backendConfig); // create id out of the configuration
+
+        // create instance of backend to get features
+        $backendStore = BackendStore::getInstance();
+        $backendInstance = $backendStore->getInstanceOfBackend($backend);
+        $features = $backendInstance->getAvailableFeatures();
+
+        // check backend_config for validity
+        $status = $this->checkBackendConfig($backendInstance, $backendConfig);
+
+        // get sequence number
+        $sequence = $this->getNewSequenceNumber();
+
+        $newAccount = new Account($newID, strip_tags($name), $status[0], $status[1], strip_tags($backend), $backendConfig, $features, $sequence, false);
+
+        // now store all the values to the user settings
+        $GLOBALS["settings"]->set(self::ACCOUNT_STORAGE_PATH . "/" . $newID . "/id", $newAccount->getId());
+        $GLOBALS["settings"]->set(self::ACCOUNT_STORAGE_PATH . "/" . $newID . "/name", $newAccount->getName());
+        $GLOBALS["settings"]->set(self::ACCOUNT_STORAGE_PATH . "/" . $newID . "/status", $newAccount->getStatus());
+        $GLOBALS["settings"]->set(self::ACCOUNT_STORAGE_PATH . "/" . $newID . "/status_description", $newAccount->getStatusDescription());
+        $GLOBALS["settings"]->set(self::ACCOUNT_STORAGE_PATH . "/" . $newID . "/backend", $newAccount->getBackend());
+        $GLOBALS["settings"]->set(self::ACCOUNT_STORAGE_PATH . "/" . $newID . "/account_sequence", $newAccount->getSequence());
+        // User defined accounts are never administrative. So set cannot_change to false.
+        $GLOBALS["settings"]->set(self::ACCOUNT_STORAGE_PATH . "/" . $newID . "/cannot_change", false);
+        $GLOBALS["settings"]->set(self::ACCOUNT_STORAGE_PATH . "/" . $newID . "/backend_config/version", self::ACCOUNT_VERSION);
+        // store all backend configurations
+        foreach ($newAccount->getBackendConfig() as $key => $value) {
+            if ($key !== "version") {
+                $GLOBALS["settings"]->set(self::ACCOUNT_STORAGE_PATH . "/" . $newID . "/backend_config/" . $key, $this->encryptBackendConfigProperty($value, self::ACCOUNT_VERSION));
+            }
+        }
+
+        // store all features
+        foreach ($newAccount->getFeatures() as $feature) {
+            $GLOBALS["settings"]->set(self::ACCOUNT_STORAGE_PATH . "/" . $newID . "/backend_features/" . $feature, true);
+        }
+
+        $GLOBALS["settings"]->saveSettings(); // save to MAPI storage
+
+        // add account to our local store after it was saved to the zarafa-settings
+        $this->accounts[$newID] = $newAccount;
+
+        return $newAccount;
+    }
+
+    /**
+     * @param Account $account
+     *
+     * @return Account
+     */
+    public function updateAccount($account)
+    {
+        $accId = $account->getId();
+        $isAdministrativeAccount = $account->getCannotChangeFlag();
+
+        // create instance of backend to get features
+        $backendStore = BackendStore::getInstance();
+        $backendInstance = $backendStore->getInstanceOfBackend($account->getBackend());
+        $features = $backendInstance->getAvailableFeatures();
+        $account->setFeatures($features);
+
+        // check backend_config for validity
+        $status = $this->checkBackendConfig($backendInstance, $account->getBackendConfig());
+        $account->setStatus($status[0]); // update status
+        $account->setStatusDescription($status[1]); // update status description
+
+        // add account to local store
+        $this->accounts[$accId] = $account;
+
+        // save values to MAPI settings
+        // now store all the values to the user settings
+        // but if we have an administrative account only save the account sequence
+        if (!$isAdministrativeAccount) {
+            $GLOBALS["settings"]->set(self::ACCOUNT_STORAGE_PATH . "/" . $accId . "/name", $account->getName());
+            $GLOBALS["settings"]->set(self::ACCOUNT_STORAGE_PATH . "/" . $accId . "/status", $account->getStatus());
+            $GLOBALS["settings"]->set(self::ACCOUNT_STORAGE_PATH . "/" . $accId . "/status_description", $account->getStatusDescription());
+            $GLOBALS["settings"]->set(self::ACCOUNT_STORAGE_PATH . "/" . $accId . "/backend", $account->getBackend());
+
+            $acc = $account->getBackendConfig();
+            $version = 0;
+            if (isset($acc["version"])) {
+                $version = $acc["version"];
+            }
+
+            // Unable to decrypt, don't update
+            if ($version == 0 && !defined('FILES_PASSWORD_IV') && !defined('FILES_PASSWORD_KEY'))  {
+                Logger::error(self::LOG_CONTEXT, "Unable to update the account to as FILES_PASSWORD_IV/FILES_PASSWORD_KEY is not set");
+            } else {
+                // store all backend configurations
+                foreach ($account->getBackendConfig() as $key => $value) {
+                    if ($key !== "version") {
+                        $GLOBALS["settings"]->set(self::ACCOUNT_STORAGE_PATH . "/" . $accId . "/backend_config/" . $key, $this->encryptBackendConfigProperty($value, self::ACCOUNT_VERSION));
+                    }
+                }
+
+                $GLOBALS["settings"]->set(self::ACCOUNT_STORAGE_PATH . "/" . $accId . "/backend_config/version", self::ACCOUNT_VERSION);
+            }
+
+            // store all features
+            foreach ($account->getFeatures() as $feature) {
+                $GLOBALS["settings"]->set(self::ACCOUNT_STORAGE_PATH . "/" . $accId . "/backend_features/" . $feature, true);
+            }
+        }
+        // when getSequence returns 0, there is no account_sequence setting yet. So create one.
+        $account_sequence = ($account->getSequence() === 0 ? $this->getNewSequenceNumber() : $account->getSequence());
+        $GLOBALS["settings"]->set(self::ACCOUNT_STORAGE_PATH . "/" . $accId . "/account_sequence", $account_sequence);
+
+        $GLOBALS["settings"]->saveSettings(); // save to MAPI storage
+
+        return $account;
+    }
+
+    /**
+     * Delete account from local store and from the MAPI settings
+     *
+     * @param $accountId
+     *
+     * @return bool
+     */
+    public function deleteAccount($accountId)
+    {
+        $account = $this->getAccount($accountId);
+        // Do not allow deleting administrative accounts, but fail silently.
+        if (!$account->getCannotChangeFlag()) {
+            $GLOBALS["settings"]->delete(self::ACCOUNT_STORAGE_PATH . "/" . $accountId);
+            $GLOBALS["settings"]->saveSettings(); // save to MAPI storage
+        }
+
+        return true;
+    }
+
+    /**
+     * Return the instance of the local account
+     *
+     * @param $accountId
+     *
+     * @return Account
+     */
+    public function getAccount($accountId)
+    {
+        return $this->accounts[$accountId];
+    }
+
+    /**
+     * @return Account[] all Accounts
+     */
+    public function getAllAccounts()
+    {
+        return $this->accounts;
+    }
+
+    /**
+     * Initialize the accountstore. Reads all accountinformation from the MAPI settings.
+     */
+    private function initialiseAccounts()
+    {
+        // Parse accounts from the Settings
+        $tmpAccs = $GLOBALS["settings"]->get(self::ACCOUNT_STORAGE_PATH);
+
+        if (is_array($tmpAccs)) {
+            $this->accounts = array();
+
+            foreach ($tmpAccs as $acc) {
+                // set backend_features if it is not set to prevent warning
+                if (!isset($acc["backend_features"])) {
+                    $acc["backend_features"] = array();
+                }
+                // account_sequence was introduced later. So set and save it if missing.
+                if (!isset($acc["account_sequence"])) {
+                    $acc["account_sequence"] = $this->getNewSequenceNumber();
+                    Logger::debug(self::LOG_CONTEXT, "Account sequence missing. New seq: " . $acc["account_sequence"]);
+                    $GLOBALS["settings"]->set(self::ACCOUNT_STORAGE_PATH . "/" . $acc["id"] . "/account_sequence", $acc["account_sequence"]);
+                    $GLOBALS["settings"]->saveSettings();
+                }
+                // cannot_change flag was introduced later. So set it to false and save it if missing.
+                if (!isset($acc["cannot_change"])) {
+                    $acc["cannot_change"] = false;
+                    Logger::debug(self::LOG_CONTEXT, "Cannot change flag missing. Setting to false.");
+                    $GLOBALS["settings"]->set(self::ACCOUNT_STORAGE_PATH . "/" . $acc["id"] . "/cannot_change", false);
+                    $GLOBALS["settings"]->saveSettings();
+                }
+
+                $backend_config = $acc["backend_config"];
+                $version = 0;
+
+                if (isset($acc["backend_config"]) && isset($acc["backend_config"]["version"])) {
+                    $version = $acc["backend_config"]["version"];
+                }
+
+                if (($version === 0 && defined('FILES_PASSWORD_IV') && defined('FILES_PASSWORD_KEY')) || $version === self::ACCOUNT_VERSION) {
+                    $backend_config = $this->decryptBackendConfig($acc["backend_config"], $version);
+                    // version is lost after decryption, add it again
+                    $backend_config["version"] = $version;
+                } else if ($version === 0) {
+                    Logger::error(self::LOG_CONTEXT, "FILES_PASSWORD_IV or FILES_PASSWORD_KEY not set, unable to decrypt backend configuration");
+                } else {
+                    Logger::error(self::LOG_CONTEXT, "Unsupported account version $version, unable to decrypt backend configuration");
+                }
+
+                $this->accounts[$acc["id"]] = new Account($acc["id"],
+                    $acc["name"],
+                    $acc["status"],
+                    $acc["status_description"],
+                    $acc["backend"],
+                    $backend_config,
+                    array_keys($acc["backend_features"]),
+                    $acc["account_sequence"],
+                    $acc["cannot_change"]
+                );
+            }
+        }
+        Logger::debug(self::LOG_CONTEXT, "Found " . count($this->accounts) . " accounts.");
+    }
+
+    /**
+     * @param AbstractBackend $backendInstance
+     * @param Array $backendConfig Backend specific account settings
+     *     like username, password, serveraddress, ...
+     *
+     * @return Array
+     */
+    private function checkBackendConfig($backendInstance, $backendConfig)
+    {
+        $status = Account::STATUS_NEW;
+        $description = _('Account is ready to use.');
+        try {
+            $backendInstance->init_backend($backendConfig);
+            $backendInstance->open();
+            $backendInstance->ls("/");
+            $status = Account::STATUS_OK;
+        } catch (BackendException $e) {
+            $status = Account::STATUS_ERROR;
+            $description = $e->getMessage();
+
+            Logger::error(self::LOG_CONTEXT, "Account check failed: " . $description);
+        }
+
+        return array($status, $description);
+    }
+
+    /**
+     * @param Array $backendConfig Backend specific account settings
+     *     like username, password, serveraddress, ...
+     *
+     * @return  an unique id
+     */
+    private function createNewId($backendConfig)
+    {
+        // lets create a hash
+        return md5(json_encode($backendConfig) . time()); // json_encode is faster than serialize
+    }
+
+    /**
+     * Generate a new sequence number. It will always be the highest used sequence number +1.
+     *
+     * @return int
+     */
+    private function getNewSequenceNumber() {
+        $seq = 0;
+        foreach($this->accounts as $acc) {
+            if($acc->getSequence() > $seq) {
+                $seq = $acc->getSequence();
+            }
+        }
+
+        return $seq + 1;
+    }
+
+    /**
+     * Decrypt the backend configuration using the standard grommunio Web key.
+     *
+     * @param Array $backendConfig Backend specific account settings
+     *     like username, password, serveraddress, ...
+     * @return array
+     */
+    private function decryptBackendConfig($backendConfig, $version=0) {
+        $decBackendConfig = array();
+
+        foreach($backendConfig as $key => $value) {
+            if ($key !== "version") {
+                try {
+                    $decBackendConfig[$key] = $this->decryptBackendConfigProperty($value, $version);
+                } catch (Exception $e) {
+                    Logger::error(self::LOG_CONTEXT, sprintf("Unable to decrypt backend configuration: '%s'", $e->getMessage()));
+                }
+            }
+        }
+
+        return $decBackendConfig;
+    }
+
+    /**
+     * Encrypt the given string.
+     *
+     * @param $value
+     * @param $version the storage version used to identify what encryption to use
+     * @return string
+     */
+    private function encryptBackendConfigProperty($value, $version=0) {
+        if ($version == self::ACCOUNT_VERSION && !is_bool($value)) {
+            $nonce = random_bytes(SODIUM_CRYPTO_SECRETBOX_NONCEBYTES);
+            $key = $GLOBALS["operations"]->getFilesEncryptionKey();
+            $encrypted = sodium_crypto_secretbox($value, $nonce, $key);
+            $value = bin2hex($nonce) . bin2hex($encrypted);
+        } else if ($version !== self::ACCOUNT_VERSION) {
+            throw Exception("Unable to encrypt backend configuration unsupported version $version");
+        }
+
+        return $value;
+    }
+
+    /**
+     * Decrypt the given string.
+     *
+     * @param $value
+     * @param $version the storage version used to identify what encryption to use
+     * @return string
+     */
+    private function decryptBackendConfigProperty($value, $version=0) {
+        if (is_bool($value)) {
+            return $value;
+        }
+
+        if ($version == self::ACCOUNT_VERSION) {
+            $value = hex2bin($value);
+            $nonce = substr($value, 0, SODIUM_CRYPTO_SECRETBOX_NONCEBYTES);
+            $encrypted = substr($value, SODIUM_CRYPTO_SECRETBOX_NONCEBYTES, strlen($value));
+            $key = $GLOBALS["operations"]->getFilesEncryptionKey();
+            $value = sodium_crypto_secretbox_open($encrypted, $nonce, $key);
+
+            // Decryption failed, password might have changed
+            if ($value === false) {
+                throw new Exception("invalid password");
+            }
+        }
+
+        return $value;
+    }
 }

Spacing +7 added lines, -7 removed lines

@@ -130,7 +130,7 @@ 
 			}
 
 			// Unable to decrypt, don't update
-			if ($version == 0 && !defined('FILES_PASSWORD_IV') && !defined('FILES_PASSWORD_KEY'))  {
+			if ($version == 0 && !defined('FILES_PASSWORD_IV') && !defined('FILES_PASSWORD_KEY')) {
 				Logger::error(self::LOG_CONTEXT, "Unable to update the account to as FILES_PASSWORD_IV/FILES_PASSWORD_KEY is not set");
 			} else {
 				// store all backend configurations
@@ -304,8 +304,8 @@ 
 	 */
 	private function getNewSequenceNumber() {
 		$seq = 0;
-		foreach($this->accounts as $acc) {
-			if($acc->getSequence() > $seq) {
+		foreach ($this->accounts as $acc) {
+			if ($acc->getSequence() > $seq) {
 				$seq = $acc->getSequence();
 			}
 		}
@@ -320,10 +320,10 @@ 
 	 *     like username, password, serveraddress, ...
 	 * @return array
 	 */
-	private function decryptBackendConfig($backendConfig, $version=0) {
+	private function decryptBackendConfig($backendConfig, $version = 0) {
 		$decBackendConfig = array();
 
-		foreach($backendConfig as $key => $value) {
+		foreach ($backendConfig as $key => $value) {
 			if ($key !== "version") {
 				try {
 					$decBackendConfig[$key] = $this->decryptBackendConfigProperty($value, $version);
@@ -343,7 +343,7 @@ 
 	 * @param $version the storage version used to identify what encryption to use
 	 * @return string
 	 */
-	private function encryptBackendConfigProperty($value, $version=0) {
+	private function encryptBackendConfigProperty($value, $version = 0) {
 		if ($version == self::ACCOUNT_VERSION && !is_bool($value)) {
 			$nonce = random_bytes(SODIUM_CRYPTO_SECRETBOX_NONCEBYTES);
 			$key = $GLOBALS["operations"]->getFilesEncryptionKey();
@@ -363,7 +363,7 @@ 
 	 * @param $version the storage version used to identify what encryption to use
 	 * @return string
 	 */
-	private function decryptBackendConfigProperty($value, $version=0) {
+	private function decryptBackendConfigProperty($value, $version = 0) {
 		if (is_bool($value)) {
 			return $value;
 		}

plugins/files/php/Files/Core/class.downloadhandler.php

Indentation +144 added lines, -144 removed lines

@@ -19,148 +19,148 @@
 
 class DownloadHandler
 {
-	const LOG_CONTEXT = "DownloadHandler"; // Context for the Logger
-
-	public static function doDownload()
-	{
-		// parse account id.
-		// wo only need to parse one string because it is
-		// only possible to download files from one backend at a time.
-		if (isset($_GET["ids"])) {
-			$tmpId = $_GET["ids"][0];
-		} else {
-			$tmpId = $_GET["id"];
-		}
-		$accountID = substr($tmpId, 3, (strpos($tmpId, '/') - 3));
-
-		// Initialize the account and backendstore
-		$accountStore = new \Files\Core\AccountStore();
-		$backendStore = \Files\Backend\BackendStore::getInstance();
-
-		$account = $accountStore->getAccount($accountID);
-
-		// initialize the backend
-		$initializedBackend = $backendStore->getInstanceOfBackend($account->getBackend());
-		$initializedBackend->init_backend($account->getBackendConfig());
-
-		try {
-			$initializedBackend->open();
-		} catch (\Files\Backend\Exception $e) {
-			Logger::error(self::LOG_CONTEXT, "Could not open the backend: " . $e->getMessage());
-
-			if ((isset($_GET["inline"]) && $_GET["inline"] == "false") || (isset($_GET["contentDispositionType"]) && $_GET["contentDispositionType"] == "attachment")) {
-				// Javascript error message
-				echo "<script>alert('" . _('File backend not responding. Please try again later.') . "');</script>";
-			} else {
-				// Text error message that is shown in the preview box
-				echo _('File backend not responding. Please try again later.');
-			}
-			die();
-		}
-
-		try {
-			if (isset($_GET["ids"])) {
-				$zip = new \ZipArchive;
-				$zipname = TMP_PATH . '/files_' . date("dmY_Hi") . '.zip';
-				Logger::debug(self::LOG_CONTEXT, "Download file tmp path: " . $zipname);
-				$res = $zip->open($zipname, \ZipArchive::CREATE);
-				if ($res !== true) {
-					Logger::error(self::LOG_CONTEXT, "Zip creation failed: " . $res);
-					echo "<script>alert('" . _('Zip file generation failed. Please inform the administrator.') . "');</script>";
-
-					die();
-				}
-				$i = 0;
-				$tmpfiles = array();
-				foreach ($_GET["ids"] as $id) {
-					// relative node ID. We need to trim off the #R# and account ID
-					$relNodeId = substr($id, strpos($id, '/'));
-
-					$tmpfiles[$i] = tempnam(TMP_PATH, stripslashes(base64_encode($relNodeId)));
-					$initializedBackend->get_file($relNodeId, $tmpfiles[$i]);
-
-					$res = $zip->addFile($tmpfiles[$i], PathUtil::getFilenameFromPath($relNodeId));
-					$i++;
-					if ($res !== true) {
-						Logger::error(self::LOG_CONTEXT, "Zip addFile failed: " . $res . " file: " . $tmpfiles[$i] . " id: " . $relNodeId);
-						echo "<script>alert('" . _('Zip file generation failed. Please inform the administrator.') . "');</script>";
-
-						die();
-					}
-				}
-				$zip->close();
-
-				// no caching
-				header('Content-Disposition: attachment; filename="' . basename($zipname) . '"');
-				header("Expires: 0"); // set expiration time
-				header("Cache-Control: must-revalidate, post-check=0, pre-check=0");
-				header('Content-Length: ' . filesize($zipname));
-				header('Content-Type: application/zip');
-				readfile($zipname);
-				unlink($zipname);
-				foreach ($tmpfiles as $tmpfile) {
-					unlink($tmpfile);
-				}
-				die();
-			} else {
-				// relative node ID. We need to trim off the #R# and account ID
-				$relNodeId = substr($_GET["id"], strpos($_GET["id"], '/'));
-				$stream = false;
-
-				$tmpfile;
-				if (!$initializedBackend->supports(\Files\Backend\BackendStore::FEATURE_STREAMING)) {
-					$tmpfile = tempnam(TMP_PATH, stripslashes(base64_encode($relNodeId)));
-					$initializedBackend->get_file($relNodeId, $tmpfile);
-					$filesize = filesize($tmpfile);
-				} else {
-					$gpi = $initializedBackend->gpi($relNodeId);
-					$stream = true;
-					$filesize = $gpi["getcontentlength"];
-				}
-
-				$mime = PathUtil::get_mime($relNodeId);
-
-				// set headers here
-				if ((isset($_GET["inline"]) && $_GET["inline"] == "false") || (isset($_GET["contentDispositionType"]) && $_GET["contentDispositionType"] == "attachment")) {
-					header('Content-Disposition: attachment; filename="' . PathUtil::getFilenameFromPath($relNodeId) . '"');
-				}
-
-				// no caching
-				header("Expires: 0"); // set expiration time
-				header("Cache-Control: must-revalidate, post-check=0, pre-check=0");
-				header('Content-Length: ' . $filesize);
-				header('Content-Type: ' . $mime);
-				flush();
-
-				if (!$stream) {
-					// print the downloaded file
-					readfile($tmpfile);
-					ignore_user_abort(true);
-					unlink($tmpfile);
-				} else {
-					// stream the file directly from the backend - much faster
-					$fh = $initializedBackend->getStreamReader($relNodeId);
-					while (!feof($fh)) {
-						set_time_limit(0);
-						print fread($fh, 4096);
-						ob_flush();
-						flush();
-					}
-					fclose($fh);
-				}
-				die();
-			}
-		} catch (\Files\Backend\Exception $e) {
-			Logger::error(self::LOG_CONTEXT, "Downloading failed: " . $e->getMessage());
-
-			if (isset($_GET["inline"]) && $_GET["inline"] == "false") {
-				// Javascript error message
-				echo "<script>alert('" . _('This file is no longer available. Please reload the folder.') . "');</script>";
-			} else {
-				// Text error message that is shown in the preview box
-				echo _('This file is no longer available. Please reload the folder.');
-			}
-			die();
-		}
-	}
+    const LOG_CONTEXT = "DownloadHandler"; // Context for the Logger
+
+    public static function doDownload()
+    {
+        // parse account id.
+        // wo only need to parse one string because it is
+        // only possible to download files from one backend at a time.
+        if (isset($_GET["ids"])) {
+            $tmpId = $_GET["ids"][0];
+        } else {
+            $tmpId = $_GET["id"];
+        }
+        $accountID = substr($tmpId, 3, (strpos($tmpId, '/') - 3));
+
+        // Initialize the account and backendstore
+        $accountStore = new \Files\Core\AccountStore();
+        $backendStore = \Files\Backend\BackendStore::getInstance();
+
+        $account = $accountStore->getAccount($accountID);
+
+        // initialize the backend
+        $initializedBackend = $backendStore->getInstanceOfBackend($account->getBackend());
+        $initializedBackend->init_backend($account->getBackendConfig());
+
+        try {
+            $initializedBackend->open();
+        } catch (\Files\Backend\Exception $e) {
+            Logger::error(self::LOG_CONTEXT, "Could not open the backend: " . $e->getMessage());
+
+            if ((isset($_GET["inline"]) && $_GET["inline"] == "false") || (isset($_GET["contentDispositionType"]) && $_GET["contentDispositionType"] == "attachment")) {
+                // Javascript error message
+                echo "<script>alert('" . _('File backend not responding. Please try again later.') . "');</script>";
+            } else {
+                // Text error message that is shown in the preview box
+                echo _('File backend not responding. Please try again later.');
+            }
+            die();
+        }
+
+        try {
+            if (isset($_GET["ids"])) {
+                $zip = new \ZipArchive;
+                $zipname = TMP_PATH . '/files_' . date("dmY_Hi") . '.zip';
+                Logger::debug(self::LOG_CONTEXT, "Download file tmp path: " . $zipname);
+                $res = $zip->open($zipname, \ZipArchive::CREATE);
+                if ($res !== true) {
+                    Logger::error(self::LOG_CONTEXT, "Zip creation failed: " . $res);
+                    echo "<script>alert('" . _('Zip file generation failed. Please inform the administrator.') . "');</script>";
+
+                    die();
+                }
+                $i = 0;
+                $tmpfiles = array();
+                foreach ($_GET["ids"] as $id) {
+                    // relative node ID. We need to trim off the #R# and account ID
+                    $relNodeId = substr($id, strpos($id, '/'));
+
+                    $tmpfiles[$i] = tempnam(TMP_PATH, stripslashes(base64_encode($relNodeId)));
+                    $initializedBackend->get_file($relNodeId, $tmpfiles[$i]);
+
+                    $res = $zip->addFile($tmpfiles[$i], PathUtil::getFilenameFromPath($relNodeId));
+                    $i++;
+                    if ($res !== true) {
+                        Logger::error(self::LOG_CONTEXT, "Zip addFile failed: " . $res . " file: " . $tmpfiles[$i] . " id: " . $relNodeId);
+                        echo "<script>alert('" . _('Zip file generation failed. Please inform the administrator.') . "');</script>";
+
+                        die();
+                    }
+                }
+                $zip->close();
+
+                // no caching
+                header('Content-Disposition: attachment; filename="' . basename($zipname) . '"');
+                header("Expires: 0"); // set expiration time
+                header("Cache-Control: must-revalidate, post-check=0, pre-check=0");
+                header('Content-Length: ' . filesize($zipname));
+                header('Content-Type: application/zip');
+                readfile($zipname);
+                unlink($zipname);
+                foreach ($tmpfiles as $tmpfile) {
+                    unlink($tmpfile);
+                }
+                die();
+            } else {
+                // relative node ID. We need to trim off the #R# and account ID
+                $relNodeId = substr($_GET["id"], strpos($_GET["id"], '/'));
+                $stream = false;
+
+                $tmpfile;
+                if (!$initializedBackend->supports(\Files\Backend\BackendStore::FEATURE_STREAMING)) {
+                    $tmpfile = tempnam(TMP_PATH, stripslashes(base64_encode($relNodeId)));
+                    $initializedBackend->get_file($relNodeId, $tmpfile);
+                    $filesize = filesize($tmpfile);
+                } else {
+                    $gpi = $initializedBackend->gpi($relNodeId);
+                    $stream = true;
+                    $filesize = $gpi["getcontentlength"];
+                }
+
+                $mime = PathUtil::get_mime($relNodeId);
+
+                // set headers here
+                if ((isset($_GET["inline"]) && $_GET["inline"] == "false") || (isset($_GET["contentDispositionType"]) && $_GET["contentDispositionType"] == "attachment")) {
+                    header('Content-Disposition: attachment; filename="' . PathUtil::getFilenameFromPath($relNodeId) . '"');
+                }
+
+                // no caching
+                header("Expires: 0"); // set expiration time
+                header("Cache-Control: must-revalidate, post-check=0, pre-check=0");
+                header('Content-Length: ' . $filesize);
+                header('Content-Type: ' . $mime);
+                flush();
+
+                if (!$stream) {
+                    // print the downloaded file
+                    readfile($tmpfile);
+                    ignore_user_abort(true);
+                    unlink($tmpfile);
+                } else {
+                    // stream the file directly from the backend - much faster
+                    $fh = $initializedBackend->getStreamReader($relNodeId);
+                    while (!feof($fh)) {
+                        set_time_limit(0);
+                        print fread($fh, 4096);
+                        ob_flush();
+                        flush();
+                    }
+                    fclose($fh);
+                }
+                die();
+            }
+        } catch (\Files\Backend\Exception $e) {
+            Logger::error(self::LOG_CONTEXT, "Downloading failed: " . $e->getMessage());
+
+            if (isset($_GET["inline"]) && $_GET["inline"] == "false") {
+                // Javascript error message
+                echo "<script>alert('" . _('This file is no longer available. Please reload the folder.') . "');</script>";
+            } else {
+                // Text error message that is shown in the preview box
+                echo _('This file is no longer available. Please reload the folder.');
+            }
+            die();
+        }
+    }
 }

plugins/files/php/Files/Core/class.account.php

Indentation +219 added lines, -219 removed lines

@@ -11,223 +11,223 @@
 
 class Account
 {
-	private $id;
-	private $name; // account name - for better usability
-	private $status;
-	private $statusDescription;
-	private $backend;
-	private $backendConfig; // This array will hold the backend configuration variables
-	private $features;
-	private $sequence; // for ordering
-	private $cannot_change; // for locking accounts
-
-	/**
-	 * Status variables
-	 */
-	const STATUS_NEW = "new";
-	const STATUS_OK = "ok";
-	const STATUS_ERROR = "err";
-	const STATUS_UNKNOWN = "unk";
-
-	/**
-	 * @param       $id
-	 * @param       $name
-	 * @param       $status
-	 * @param       $statusDescription
-	 * @param       $backend
-	 * @param       $backendConfig
-	 * @param array $features
-	 */
-	function __construct($id, $name, $status, $statusDescription, $backend, $backendConfig, $features = array(), $sequence, $cannot_change)
-	{
-		$this->id = $id;
-		$this->name = $name;
-		$this->status = $status;
-		$this->statusDescription = $statusDescription;
-		$this->backend = $backend;
-		$this->backendConfig = $backendConfig;
-		$this->features = $features;
-		$this->sequence = $sequence;
-		$this->cannot_change = $cannot_change;
-	}
-
-	/**
-	 * @return mixed
-	 */
-	public function getBackend()
-	{
-		return $this->backend;
-	}
-
-	/**
-	 * @param mixed $backend
-	 */
-	public function setBackend($backend)
-	{
-		$this->backend = $backend;
-	}
-
-	/**
-	 * @return mixed
-	 */
-	public function getId()
-	{
-		return $this->id;
-	}
-
-	/**
-	 * @param mixed $id
-	 */
-	public function setId($id)
-	{
-		$this->id = $id;
-	}
-
-	/**
-	 * @return mixed
-	 */
-	public function getName()
-	{
-		return $this->name;
-	}
-
-	/**
-	 * @param mixed $name
-	 */
-	public function setName($name)
-	{
-		$this->name = $name;
-	}
-
-	/**
-	 * @return mixed
-	 */
-	public function getStatus()
-	{
-		return $this->status;
-	}
-
-	/**
-	 * @param mixed $status
-	 */
-	public function setStatus($status)
-	{
-		$this->status = $status;
-	}
-
-	/**
-	 * @return mixed
-	 */
-	public function getBackendConfig()
-	{
-		// always add the accountID to the backendConfig
-		$this->backendConfig["current_account_id"] = $this->getId();
-
-		return $this->backendConfig;
-	}
-
-	/**
-	 * @param mixed $backendConfig
-	 */
-	public function setBackendConfig($backendConfig)
-	{
-		$this->backendConfig = $backendConfig;
-	}
-
-	/**
-	 * @return mixed
-	 */
-	public function getFeatures()
-	{
-		return $this->features;
-	}
-
-	/**
-	 * @param mixed $features
-	 */
-	public function setFeatures($features)
-	{
-		$this->features = $features;
-	}
-
-	/**
-	 * @return string
-	 */
-	public function getStatusDescription()
-	{
-		return $this->statusDescription;
-	}
-
-	/**
-	 * @param string $statusDescription
-	 */
-	public function setStatusDescription($statusDescription)
-	{
-		$this->statusDescription = $statusDescription;
-	}
-
-	/**
-	 * @param $property
-	 *
-	 * @return mixed
-	 */
-	public function getConfigValue($property)
-	{
-		if (is_array($this->backendConfig) && in_array($property, $this->backendConfig)) {
-			return $this->backendConfig[$property];
-		}
-
-		return false;
-	}
-
-	/**
-	 * @param $property
-	 * @param $value
-	 */
-	public function setConfigValue($property, $value)
-	{
-		if (!is_array($this->backendConfig)) {
-			$this->backendConfig = array();
-		}
-		$this->backendConfig[$property] = $value;
-	}
-
-	/**
-	 * This function is executed before the account gets deleted.
-	 */
-	public function beforeDelete() {
-		$backendstore = BackendStore::getInstance();
-		$backendinstance = $backendstore->getInstanceOfBackend($this->backend);
-		$backendinstance->init_backend($this->backendConfig);
-		$backendinstance->open();
-		$backendinstance->beforeDeleteAccount($this);
-	}
-
-	/**
-	 * @return int
-	 */
-	public function getSequence()
-	{
-		if(!$this->sequence) {
-			$this->sequence = 0;
-		}
-
-		return $this->sequence;
-	}
-
-	/**
-	 * @param int $sequence
-	 */
-	public function setSequence($sequence)
-	{
-		$this->sequence = $sequence;
-	}
-
-	/**
-	 * @return boolean
-	 */
-	public function getCannotChangeFlag()
-	{
-		return isset($this->cannot_change) ? $this->cannot_change : false;
-	}
+    private $id;
+    private $name; // account name - for better usability
+    private $status;
+    private $statusDescription;
+    private $backend;
+    private $backendConfig; // This array will hold the backend configuration variables
+    private $features;
+    private $sequence; // for ordering
+    private $cannot_change; // for locking accounts
+
+    /**
+     * Status variables
+     */
+    const STATUS_NEW = "new";
+    const STATUS_OK = "ok";
+    const STATUS_ERROR = "err";
+    const STATUS_UNKNOWN = "unk";
+
+    /**
+     * @param       $id
+     * @param       $name
+     * @param       $status
+     * @param       $statusDescription
+     * @param       $backend
+     * @param       $backendConfig
+     * @param array $features
+     */
+    function __construct($id, $name, $status, $statusDescription, $backend, $backendConfig, $features = array(), $sequence, $cannot_change)
+    {
+        $this->id = $id;
+        $this->name = $name;
+        $this->status = $status;
+        $this->statusDescription = $statusDescription;
+        $this->backend = $backend;
+        $this->backendConfig = $backendConfig;
+        $this->features = $features;
+        $this->sequence = $sequence;
+        $this->cannot_change = $cannot_change;
+    }
+
+    /**
+     * @return mixed
+     */
+    public function getBackend()
+    {
+        return $this->backend;
+    }
+
+    /**
+     * @param mixed $backend
+     */
+    public function setBackend($backend)
+    {
+        $this->backend = $backend;
+    }
+
+    /**
+     * @return mixed
+     */
+    public function getId()
+    {
+        return $this->id;
+    }
+
+    /**
+     * @param mixed $id
+     */
+    public function setId($id)
+    {
+        $this->id = $id;
+    }
+
+    /**
+     * @return mixed
+     */
+    public function getName()
+    {
+        return $this->name;
+    }
+
+    /**
+     * @param mixed $name
+     */
+    public function setName($name)
+    {
+        $this->name = $name;
+    }
+
+    /**
+     * @return mixed
+     */
+    public function getStatus()
+    {
+        return $this->status;
+    }
+
+    /**
+     * @param mixed $status
+     */
+    public function setStatus($status)
+    {
+        $this->status = $status;
+    }
+
+    /**
+     * @return mixed
+     */
+    public function getBackendConfig()
+    {
+        // always add the accountID to the backendConfig
+        $this->backendConfig["current_account_id"] = $this->getId();
+
+        return $this->backendConfig;
+    }
+
+    /**
+     * @param mixed $backendConfig
+     */
+    public function setBackendConfig($backendConfig)
+    {
+        $this->backendConfig = $backendConfig;
+    }
+
+    /**
+     * @return mixed
+     */
+    public function getFeatures()
+    {
+        return $this->features;
+    }
+
+    /**
+     * @param mixed $features
+     */
+    public function setFeatures($features)
+    {
+        $this->features = $features;
+    }
+
+    /**
+     * @return string
+     */
+    public function getStatusDescription()
+    {
+        return $this->statusDescription;
+    }
+
+    /**
+     * @param string $statusDescription
+     */
+    public function setStatusDescription($statusDescription)
+    {
+        $this->statusDescription = $statusDescription;
+    }
+
+    /**
+     * @param $property
+     *
+     * @return mixed
+     */
+    public function getConfigValue($property)
+    {
+        if (is_array($this->backendConfig) && in_array($property, $this->backendConfig)) {
+            return $this->backendConfig[$property];
+        }
+
+        return false;
+    }
+
+    /**
+     * @param $property
+     * @param $value
+     */
+    public function setConfigValue($property, $value)
+    {
+        if (!is_array($this->backendConfig)) {
+            $this->backendConfig = array();
+        }
+        $this->backendConfig[$property] = $value;
+    }
+
+    /**
+     * This function is executed before the account gets deleted.
+     */
+    public function beforeDelete() {
+        $backendstore = BackendStore::getInstance();
+        $backendinstance = $backendstore->getInstanceOfBackend($this->backend);
+        $backendinstance->init_backend($this->backendConfig);
+        $backendinstance->open();
+        $backendinstance->beforeDeleteAccount($this);
+    }
+
+    /**
+     * @return int
+     */
+    public function getSequence()
+    {
+        if(!$this->sequence) {
+            $this->sequence = 0;
+        }
+
+        return $this->sequence;
+    }
+
+    /**
+     * @param int $sequence
+     */
+    public function setSequence($sequence)
+    {
+        $this->sequence = $sequence;
+    }
+
+    /**
+     * @return boolean
+     */
+    public function getCannotChangeFlag()
+    {
+        return isset($this->cannot_change) ? $this->cannot_change : false;
+    }
 }

Spacing +1 added lines, -1 removed lines

@@ -208,7 +208,7 @@
 	 */
 	public function getSequence()
 	{
-		if(!$this->sequence) {
+		if (!$this->sequence) {
 			$this->sequence = 0;
 		}