omines / directadmin

src/DirectAdmin/Context/ResellerContext.php

Spacing +1 added lines, -1 removed lines

@@ -83,7 +83,7 @@
     public function deleteAccounts(array $usernames)
     {
         $options = ['confirmed' => 'Confirm', 'delete' => 'yes'];
-        foreach (array_values($usernames) as $idx => $username) {
+        foreach(array_values($usernames) as $idx => $username) {
             $options["select{$idx}"] = $username;
         }
         $this->invokePost('SELECT_USERS', $options);

Braces +2 added lines, -1 removed lines

@@ -83,7 +83,8 @@
     public function deleteAccounts(array $usernames)
     {
         $options = ['confirmed' => 'Confirm', 'delete' => 'yes'];
-        foreach (array_values($usernames) as $idx => $username) {
+        foreach (array_values($usernames) as $idx => $username)
+        {
             $options["select{$idx}"] = $username;
         }
         $this->invokePost('SELECT_USERS', $options);

src/DirectAdmin/Context/UserContext.php

Spacing +3 added lines, -3 removed lines

@@ -34,13 +34,13 @@ 
     public function __construct(DirectAdmin $connection, $validate = false)
     {
         parent::__construct($connection);
-        if ($validate) {
+        if($validate) {
             $classMap = [
                 DirectAdmin::ACCOUNT_TYPE_ADMIN => AdminContext::class,
                 DirectAdmin::ACCOUNT_TYPE_RESELLER => ResellerContext::class,
                 DirectAdmin::ACCOUNT_TYPE_USER => self::class,
             ];
-            if ($classMap[$this->getType()] != get_class($this)) {
+            if($classMap[$this->getType()] != get_class($this)) {
                 throw new DirectAdminException('Validation mismatch on context construction');
             }
         }
@@ -63,7 +63,7 @@ 
      */
     public function getContextUser()
     {
-        if (!isset($this->user)) {
+        if(!isset($this->user)) {
             $this->user = User::fromConfig($this->invokeGet('SHOW_USER_CONFIG'), $this);
         }
         return $this->user;

Braces +6 added lines, -3 removed lines

@@ -34,13 +34,15 @@ 
     public function __construct(DirectAdmin $connection, $validate = false)
     {
         parent::__construct($connection);
-        if ($validate) {
+        if ($validate)
+        {
             $classMap = [
                 DirectAdmin::ACCOUNT_TYPE_ADMIN => AdminContext::class,
                 DirectAdmin::ACCOUNT_TYPE_RESELLER => ResellerContext::class,
                 DirectAdmin::ACCOUNT_TYPE_USER => self::class,
             ];
-            if ($classMap[$this->getType()] != get_class($this)) {
+            if ($classMap[$this->getType()] != get_class($this))
+            {
                 throw new DirectAdminException('Validation mismatch on context construction');
             }
         }
@@ -63,7 +65,8 @@ 
      */
     public function getContextUser()
     {
-        if (!isset($this->user)) {
+        if (!isset($this->user))
+        {
             $this->user = User::fromConfig($this->invokeGet('SHOW_USER_CONFIG'), $this);
         }
         return $this->user;

src/DirectAdmin/Objects/DomainObject.php

Spacing +1 added lines, -1 removed lines

@@ -70,7 +70,7 @@
      */
     public static function toDomainObjectArray(array $items, $class, Domain $domain)
     {
-        array_walk($items, function (&$value, $name) use ($class, $domain) {
+        array_walk($items, function(&$value, $name) use ($class, $domain) {
             $value = new $class($name, $domain, $value);
         });
         return $items;

src/DirectAdmin/Objects/Users/User.php

Spacing +12 added lines, -12 removed lines

@@ -43,7 +43,7 @@ 
     public function __construct($name, UserContext $context, $config = null)
     {
         parent::__construct($name, $context);
-        if (isset($config)) {
+        if(isset($config)) {
             $this->setCache(self::CACHE_CONFIG, $config);
         }
     }
@@ -163,7 +163,7 @@ 
      */
     public function getDefaultDomain()
     {
-        if (empty($name = $this->getConfig('domain'))) {
+        if(empty($name = $this->getConfig('domain'))) {
             return null;
         }
         return $this->getDomain($name);
@@ -204,11 +204,11 @@ 
      */
     public function getDatabases()
     {
-        return $this->getCache(self::CACHE_DATABASES, function () {
+        return $this->getCache(self::CACHE_DATABASES, function() {
             $databases = [];
-            foreach ($this->getSelfManagedContext()->invokeGet('DATABASES') as $fullName) {
+            foreach($this->getSelfManagedContext()->invokeGet('DATABASES') as $fullName) {
                 list($user, $db) = explode('_', $fullName, 2);
-                if ($this->getUsername() != $user) {
+                if($this->getUsername() != $user) {
                     throw new DirectAdminException('Username incorrect on database ' . $fullName);
                 }
                 $databases[$db] = new Database($db, $this, $this->getSelfManagedContext());
@@ -223,7 +223,7 @@ 
      */
     public function getDomain($domainName)
     {
-        if (!isset($this->domains)) {
+        if(!isset($this->domains)) {
             $this->getDomains();
         }
         return isset($this->domains[$domainName]) ? $this->domains[$domainName] : null;
@@ -234,8 +234,8 @@ 
      */
     public function getDomains()
     {
-        if (!isset($this->domains)) {
-            if (!$this->isSelfManaged()) {
+        if(!isset($this->domains)) {
+            if(!$this->isSelfManaged()) {
                 $this->domains = $this->impersonate()->getDomains();
             } else {
                 $this->domains = Object::toRichObjectArray($this->getContext()->invokeGet('ADDITIONAL_DOMAINS'), Domain::class, $this->getContext());
@@ -282,7 +282,7 @@ 
     public function impersonate()
     {
         /** @var ResellerContext $context */
-        if (!($context = $this->getContext()) instanceof ResellerContext) {
+        if(!($context = $this->getContext()) instanceof ResellerContext) {
             throw new DirectAdminException('You need to be at least a reseller to impersonate');
         }
         return $context->impersonateUser($this->getUsername());
@@ -348,7 +348,7 @@ 
     public static function fromConfig($config, UserContext $context)
     {
         $name = $config['username'];
-        switch ($config['usertype']) {
+        switch($config['usertype']) {
             case DirectAdmin::ACCOUNT_TYPE_USER:
                 return new self($name, $context, $config);
             case DirectAdmin::ACCOUNT_TYPE_RESELLER:
@@ -368,7 +368,7 @@ 
      */
     private function getConfig($item)
     {
-        return $this->getCacheItem(self::CACHE_CONFIG, $item, function () {
+        return $this->getCacheItem(self::CACHE_CONFIG, $item, function() {
             return $this->loadConfig();
         });
     }
@@ -381,7 +381,7 @@ 
      */
     private function getUsage($item)
     {
-        return $this->getCacheItem(self::CACHE_USAGE, $item, function () {
+        return $this->getCacheItem(self::CACHE_USAGE, $item, function() {
             return $this->getContext()->invokeGet('SHOW_USER_USAGE', ['user' => $this->getUsername()]);
         });
     }

Braces +21 added lines, -10 removed lines

@@ -43,7 +43,8 @@ 
     public function __construct($name, UserContext $context, $config = null)
     {
         parent::__construct($name, $context);
-        if (isset($config)) {
+        if (isset($config))
+        {
             $this->setCache(self::CACHE_CONFIG, $config);
         }
     }
@@ -163,7 +164,8 @@ 
      */
     public function getDefaultDomain()
     {
-        if (empty($name = $this->getConfig('domain'))) {
+        if (empty($name = $this->getConfig('domain')))
+        {
             return null;
         }
         return $this->getDomain($name);
@@ -206,9 +208,11 @@ 
     {
         return $this->getCache(self::CACHE_DATABASES, function () {
             $databases = [];
-            foreach ($this->getSelfManagedContext()->invokeGet('DATABASES') as $fullName) {
+            foreach ($this->getSelfManagedContext()->invokeGet('DATABASES') as $fullName)
+            {
                 list($user, $db) = explode('_', $fullName, 2);
-                if ($this->getUsername() != $user) {
+                if ($this->getUsername() != $user)
+                {
                     throw new DirectAdminException('Username incorrect on database ' . $fullName);
                 }
                 $databases[$db] = new Database($db, $this, $this->getSelfManagedContext());
@@ -223,7 +227,8 @@ 
      */
     public function getDomain($domainName)
     {
-        if (!isset($this->domains)) {
+        if (!isset($this->domains))
+        {
             $this->getDomains();
         }
         return isset($this->domains[$domainName]) ? $this->domains[$domainName] : null;
@@ -234,10 +239,14 @@ 
      */
     public function getDomains()
     {
-        if (!isset($this->domains)) {
-            if (!$this->isSelfManaged()) {
+        if (!isset($this->domains))
+        {
+            if (!$this->isSelfManaged())
+            {
                 $this->domains = $this->impersonate()->getDomains();
-            } else {
+            }
+            else
+            {
                 $this->domains = Object::toRichObjectArray($this->getContext()->invokeGet('ADDITIONAL_DOMAINS'), Domain::class, $this->getContext());
             }
         }
@@ -282,7 +291,8 @@ 
     public function impersonate()
     {
         /** @var ResellerContext $context */
-        if (!($context = $this->getContext()) instanceof ResellerContext) {
+        if (!($context = $this->getContext()) instanceof ResellerContext)
+        {
             throw new DirectAdminException('You need to be at least a reseller to impersonate');
         }
         return $context->impersonateUser($this->getUsername());
@@ -348,7 +358,8 @@ 
     public static function fromConfig($config, UserContext $context)
     {
         $name = $config['username'];
-        switch ($config['usertype']) {
+        switch ($config['usertype'])
+        {
             case DirectAdmin::ACCOUNT_TYPE_USER:
                 return new self($name, $context, $config);
             case DirectAdmin::ACCOUNT_TYPE_RESELLER:

src/DirectAdmin/Objects/Users/Admin.php

Spacing +1 added lines, -1 removed lines

@@ -35,7 +35,7 @@
     public function impersonate()
     {
         /** @var AdminContext $context */
-        if (!($context = $this->getContext()) instanceof AdminContext) {
+        if(!($context = $this->getContext()) instanceof AdminContext) {
             throw new DirectAdminException('You need to be an admin to impersonate another admin');
         }
         return $context->impersonateAdmin($this->getUsername());

Braces +2 added lines, -1 removed lines

@@ -35,7 +35,8 @@
     public function impersonate()
     {
         /** @var AdminContext $context */
-        if (!($context = $this->getContext()) instanceof AdminContext) {
+        if (!($context = $this->getContext()) instanceof AdminContext)
+        {
             throw new DirectAdminException('You need to be an admin to impersonate another admin');
         }
         return $context->impersonateAdmin($this->getUsername());

src/DirectAdmin/Objects/Email/Mailbox.php

Spacing +2 added lines, -2 removed lines

@@ -31,7 +31,7 @@ 
     public function __construct($prefix, Domain $domain, $config = null)
     {
         parent::__construct($prefix, $domain);
-        if (isset($config)) {
+        if(isset($config)) {
             $this->setCache(self::CACHE_DATA, is_string($config) ? \GuzzleHttp\Psr7\parse_query($config) : $config);
         }
     }
@@ -118,7 +118,7 @@ 
      */
     protected function getData($key)
     {
-        return $this->getCacheItem(self::CACHE_DATA, $key, function () {
+        return $this->getCacheItem(self::CACHE_DATA, $key, function() {
             $result = $this->getContext()->invokeGet('POP', [
                 'domain' => $this->getDomainName(),
                 'action' => 'full_list',

Braces +2 added lines, -1 removed lines

@@ -31,7 +31,8 @@
     public function __construct($prefix, Domain $domain, $config = null)
     {
         parent::__construct($prefix, $domain);
-        if (isset($config)) {
+        if (isset($config))
+        {
             $this->setCache(self::CACHE_DATA, is_string($config) ? \GuzzleHttp\Psr7\parse_query($config) : $config);
         }
     }

src/DirectAdmin/Objects/Email/Forwarder.php

Spacing +1 added lines, -1 removed lines

@@ -77,7 +77,7 @@
      */
     public function getAliases()
     {
-        return array_map(function ($domain) {
+        return array_map(function($domain) {
             return $this->getPrefix() . '@' . $domain;
         }, $this->getDomain()->getDomainNames());
     }

src/DirectAdmin/Objects/Database.php

Spacing +3 added lines, -3 removed lines

@@ -57,7 +57,7 @@ 
             'action' => 'create',
             'name' => $name,
         ];
-        if (!empty($password)) {
+        if(!empty($password)) {
             $options += ['user' => $username, 'passwd' => $password, 'passwd2' => $password];
         } else {
             $options += ['userlist' => $username];
@@ -83,13 +83,13 @@ 
      */
     public function getAccessHosts()
     {
-        return $this->getCache(self::CACHE_ACCESS_HOSTS, function () {
+        return $this->getCache(self::CACHE_ACCESS_HOSTS, function() {
             $accessHosts = $this->getContext()->invokeGet('DATABASES', [
                 'action' => 'accesshosts',
                 'db' => $this->getDatabaseName(),
             ]);
 
-            return array_map(function ($name) {
+            return array_map(function($name) {
                 return new Database\AccessHost($name, $this);
             }, $accessHosts);
         });

Braces +5 added lines, -2 removed lines

@@ -57,9 +57,12 @@
             'action' => 'create',
             'name' => $name,
         ];
-        if (!empty($password)) {
+        if (!empty($password))
+        {
             $options += ['user' => $username, 'passwd' => $password, 'passwd2' => $password];
-        } else {
+        }
+        else
+        {
             $options += ['userlist' => $username];
         }
         $user->getContext()->invokePost('DATABASES', $options);

src/DirectAdmin/Utility/Conversion.php

Spacing +4 added lines, -4 removed lines

@@ -39,7 +39,7 @@ 
     {
         $ukey = "u{$key}";
         unset($options[$ukey]);
-        if (array_key_exists($key, $options) && ($options[$key] === 'unlimited' || !isset($options[$key]))) {
+        if(array_key_exists($key, $options) && ($options[$key] === 'unlimited' || !isset($options[$key]))) {
             $options[$ukey] = 'ON';
         }
     }
@@ -52,7 +52,7 @@ 
      */
     public static function processUnlimitedOptions(array $options)
     {
-        foreach (['bandwidth', 'domainptr', 'ftp', 'mysql', 'nemailf', 'nemailml', 'nemailr', 'nemails',
+        foreach(['bandwidth', 'domainptr', 'ftp', 'mysql', 'nemailf', 'nemailml', 'nemailr', 'nemails',
                     'nsubdomains', 'quota', 'vdomains', ] as $key) {
             self::processUnlimitedOption($options, $key);
         }
@@ -67,7 +67,7 @@ 
      */
     public static function responseToArray($data)
     {
-        $unescaped = preg_replace_callback('/&#([0-9]{2})/', function ($val) {
+        $unescaped = preg_replace_callback('/&#([0-9]{2})/', function($val) {
             return chr($val[1]);
         }, $data);
         return \GuzzleHttp\Psr7\parse_query($unescaped);
@@ -81,7 +81,7 @@ 
      */
     public static function sanitizeArray($result)
     {
-        if (count($result) == 1 && isset($result['list[]'])) {
+        if(count($result) == 1 && isset($result['list[]'])) {
             $result = $result['list[]'];
         }
         return is_array($result) ? $result : [$result];

Braces +6 added lines, -3 removed lines

@@ -39,7 +39,8 @@ 
     {
         $ukey = "u{$key}";
         unset($options[$ukey]);
-        if (array_key_exists($key, $options) && ($options[$key] === 'unlimited' || !isset($options[$key]))) {
+        if (array_key_exists($key, $options) && ($options[$key] === 'unlimited' || !isset($options[$key])))
+        {
             $options[$ukey] = 'ON';
         }
     }
@@ -53,7 +54,8 @@ 
     public static function processUnlimitedOptions(array $options)
     {
         foreach (['bandwidth', 'domainptr', 'ftp', 'mysql', 'nemailf', 'nemailml', 'nemailr', 'nemails',
-                    'nsubdomains', 'quota', 'vdomains', ] as $key) {
+                    'nsubdomains', 'quota', 'vdomains', ] as $key)
+        {
             self::processUnlimitedOption($options, $key);
         }
         return $options;
@@ -81,7 +83,8 @@ 
      */
     public static function sanitizeArray($result)
     {
-        if (count($result) == 1 && isset($result['list[]'])) {
+        if (count($result) == 1 && isset($result['list[]']))
+        {
             $result = $result['list[]'];
         }
         return is_array($result) ? $result : [$result];