omines / directadmin

src/DirectAdmin/DirectAdmin.php

Spacing +3 added lines, -3 removed lines

@@ -127,7 +127,7 @@ 
     public function invoke($method, $command, $options = [])
     {
         $result = $this->rawRequest($method, '/CMD_API_' . $command, $options);
-        if (!empty($result['error'])) {
+        if(!empty($result['error'])) {
             throw new DirectAdminException("$method to $command failed: $result[details] ($result[text])");
         }
         return Conversion::sanitizeArray($result);
@@ -157,12 +157,12 @@ 
     {
         try {
             $response = $this->connection->request($method, $uri, $options);
-            if ($response->getHeader('Content-Type')[0] == 'text/html') {
+            if($response->getHeader('Content-Type')[0] == 'text/html') {
                 throw new DirectAdminException("DirectAdmin API returned a text/html body. Requested {$uri} via {$method}. Responded: " . strip_tags($response->getBody()->getContents()));
             }
             $body = $response->getBody()->getContents();
             return Conversion::responseToArray($body);
-        } catch (TransferException $exception) {
+        } catch(TransferException $exception) {
             // Rethrow anything that causes a network issue
             throw new DirectAdminException("Request to {$uri} using {$method} failed", 0, $exception);
         }

Braces +9 added lines, -4 removed lines

@@ -127,7 +127,8 @@ 
     public function invoke($method, $command, $options = [])
     {
         $result = $this->rawRequest($method, '/CMD_API_' . $command, $options);
-        if (!empty($result['error'])) {
+        if (!empty($result['error']))
+        {
             throw new DirectAdminException("$method to $command failed: $result[details] ($result[text])");
         }
         return Conversion::sanitizeArray($result);
@@ -155,14 +156,18 @@ 
      */
     private function rawRequest($method, $uri, $options)
     {
-        try {
+        try
+        {
             $response = $this->connection->request($method, $uri, $options);
-            if ($response->getHeader('Content-Type')[0] == 'text/html') {
+            if ($response->getHeader('Content-Type')[0] == 'text/html')
+            {
                 throw new DirectAdminException("DirectAdmin API returned a text/html body. Requested {$uri} via {$method}. Responded: " . strip_tags($response->getBody()->getContents()));
             }
             $body = $response->getBody()->getContents();
             return Conversion::responseToArray($body);
-        } catch (TransferException $exception) {
+        }
+        catch (TransferException $exception)
+        {
             // Rethrow anything that causes a network issue
             throw new DirectAdminException("Request to {$uri} using {$method} failed", 0, $exception);
         }

src/DirectAdmin/Objects/BaseObject.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/Domain.php

Spacing +7 added lines, -7 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');
@@ -152,7 +152,7 @@ 
             'from' => $domain,
             'action' => 'add',
         ];
-        if ($alias) {
+        if($alias) {
             $parameters['alias'] = 'yes';
             $list = &$this->aliases;
         } else {
@@ -243,7 +243,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;
@@ -255,7 +255,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(),
             ]);
@@ -268,7 +268,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',
@@ -298,7 +298,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);
@@ -320,7 +320,7 @@ 
             'action' => $action,
             'domain' => $this->domainName,
         ], $parameters));
-        if ($clearCache) {
+        if($clearCache) {
             $this->clearCache();
         }
         return $response;

Braces +12 added lines, -5 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');
         }
 
@@ -152,10 +155,13 @@ 
             'from' => $domain,
             'action' => 'add',
         ];
-        if ($alias) {
+        if ($alias)
+        {
             $parameters['alias'] = 'yes';
             $list = &$this->aliases;
-        } else {
+        }
+        else
+        {
             $list = &$this->pointers;
         }
         $this->getContext()->invokePost('DOMAIN_POINTER', $parameters);
@@ -320,7 +326,8 @@ 
             'action' => $action,
             'domain' => $this->domainName,
         ], $parameters));
-        if ($clearCache) {
+        if ($clearCache)
+        {
             $this->clearCache();
         }
         return $response;