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/Object.php

Spacing +5 added lines, -5 removed lines

@@ -56,7 +56,7 @@ 
      */
     protected function getCache($key, $default)
     {
-        if (!isset($this->cache[$key])) {
+        if(!isset($this->cache[$key])) {
             $this->cache[$key] = is_callable($default) ? $default() : $default;
         }
         return $this->cache[$key];
@@ -73,10 +73,10 @@ 
      */
     protected function getCacheItem($key, $item, $defaultKey, $defaultItem = null)
     {
-        if (empty($cache = $this->getCache($key, $defaultKey))) {
+        if(empty($cache = $this->getCache($key, $defaultKey))) {
             return $defaultItem;
         }
-        if (!is_array($cache)) {
+        if(!is_array($cache)) {
             throw new DirectAdminException("Cache item $key is not an array");
         }
         return isset($cache[$item]) ? $cache[$item] : $defaultItem;
@@ -121,7 +121,7 @@ 
      */
     public static function toObjectArray(array $items, $class, UserContext $context)
     {
-        return array_combine($items, array_map(function ($item) use ($class, $context) {
+        return array_combine($items, array_map(function($item) use ($class, $context) {
             return new $class($item, $context);
         }, $items));
     }
@@ -136,7 +136,7 @@ 
      */
     public static function toRichObjectArray(array $items, $class, UserContext $context)
     {
-        array_walk($items, function (&$value, $name) use ($class, $context) {
+        array_walk($items, function(&$value, $name) use ($class, $context) {
             $value = new $class($name, $context, $value);
         });
         return $items;

Braces +6 added lines, -3 removed lines

@@ -56,7 +56,8 @@ 
      */
     protected function getCache($key, $default)
     {
-        if (!isset($this->cache[$key])) {
+        if (!isset($this->cache[$key]))
+        {
             $this->cache[$key] = is_callable($default) ? $default() : $default;
         }
         return $this->cache[$key];
@@ -73,10 +74,12 @@ 
      */
     protected function getCacheItem($key, $item, $defaultKey, $defaultItem = null)
     {
-        if (empty($cache = $this->getCache($key, $defaultKey))) {
+        if (empty($cache = $this->getCache($key, $defaultKey)))
+        {
             return $defaultItem;
         }
-        if (!is_array($cache)) {
+        if (!is_array($cache))
+        {
             throw new DirectAdminException("Cache item $key is not an array");
         }
         return isset($cache[$item]) ? $cache[$item] : $defaultItem;

src/DirectAdmin/Objects/Users/Reseller.php

Spacing +1 added lines, -1 removed lines

@@ -56,7 +56,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 a reseller');
         }
         return $context->impersonateReseller($this->getUsername());

Braces +2 added lines, -1 removed lines

@@ -56,7 +56,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 a reseller');
         }
         return $context->impersonateReseller($this->getUsername());

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/Domain.php

Spacing +6 added lines, -6 removed lines

@@ -69,7 +69,7 @@ 
         $this->domainName = $data['domain'];
 
         // Determine owner
-        if ($data['username'] === $context->getUsername()) {
+        if($data['username'] === $context->getUsername()) {
             $this->owner = $context->getContextUser();
         } else {
             throw new DirectAdminException('Could not determine relationship between context user and domain');
@@ -219,7 +219,7 @@ 
      */
     public function getDomainNames()
     {
-        return $this->getCache('domainNames', function () {
+        return $this->getCache('domainNames', function() {
             $list = array_merge($this->aliases, $this->pointers, [$this->getDomainName()]);
             sort($list);
             return $list;
@@ -231,7 +231,7 @@ 
      */
     public function getForwarders()
     {
-        return $this->getCache(self::CACHE_FORWARDERS, function () {
+        return $this->getCache(self::CACHE_FORWARDERS, function() {
             $forwarders = $this->getContext()->invokeGet('EMAIL_FORWARDERS', [
                 'domain' => $this->getDomainName(),
             ]);
@@ -244,7 +244,7 @@ 
      */
     public function getMailboxes()
     {
-        return $this->getCache(self::CACHE_MAILBOXES, function () {
+        return $this->getCache(self::CACHE_MAILBOXES, function() {
             $boxes = $this->getContext()->invokeGet('POP', [
                 'domain' => $this->getDomainName(),
                 'action' => 'full_list',
@@ -274,7 +274,7 @@ 
      */
     public function getSubdomains()
     {
-        return $this->getCache(self::CACHE_SUBDOMAINS, function () {
+        return $this->getCache(self::CACHE_SUBDOMAINS, function() {
             $subs = $this->getContext()->invokeGet('SUBDOMAINS', ['domain' => $this->getDomainName()]);
             $subs = array_combine($subs, $subs);
             return DomainObject::toDomainObjectArray($subs, Subdomain::class, $this);
@@ -296,7 +296,7 @@ 
             'action' => $action,
             'domain' => $this->domainName,
         ], $parameters));
-        if ($clearCache) {
+        if($clearCache) {
             $this->clearCache();
         }
         return $response;

Braces +7 added lines, -3 removed lines

@@ -69,9 +69,12 @@ 
         $this->domainName = $data['domain'];
 
         // Determine owner
-        if ($data['username'] === $context->getUsername()) {
+        if ($data['username'] === $context->getUsername())
+        {
             $this->owner = $context->getContextUser();
-        } else {
+        }
+        else
+        {
             throw new DirectAdminException('Could not determine relationship between context user and domain');
         }
 
@@ -296,7 +299,8 @@ 
             'action' => $action,
             'domain' => $this->domainName,
         ], $parameters));
-        if ($clearCache) {
+        if ($clearCache)
+        {
             $this->clearCache();
         }
         return $response;

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);
         }
     }