@@ -56,7 +56,7 @@ discard block |
||
56 | 56 | */ |
57 | 57 | protected function getCache($key, $default) |
58 | 58 | { |
59 | - if (!isset($this->cache[$key])) { |
|
59 | + if(!isset($this->cache[$key])) { |
|
60 | 60 | $this->cache[$key] = is_callable($default) ? $default() : $default; |
61 | 61 | } |
62 | 62 | return $this->cache[$key]; |
@@ -73,10 +73,10 @@ discard block |
||
73 | 73 | */ |
74 | 74 | protected function getCacheItem($key, $item, $defaultKey, $defaultItem = null) |
75 | 75 | { |
76 | - if (empty($cache = $this->getCache($key, $defaultKey))) { |
|
76 | + if(empty($cache = $this->getCache($key, $defaultKey))) { |
|
77 | 77 | return $defaultItem; |
78 | 78 | } |
79 | - if (!is_array($cache)) { |
|
79 | + if(!is_array($cache)) { |
|
80 | 80 | throw new DirectAdminException("Cache item $key is not an array"); |
81 | 81 | } |
82 | 82 | return isset($cache[$item]) ? $cache[$item] : $defaultItem; |
@@ -121,7 +121,7 @@ discard block |
||
121 | 121 | */ |
122 | 122 | public static function toObjectArray(array $items, $class, UserContext $context) |
123 | 123 | { |
124 | - return array_combine($items, array_map(function ($item) use ($class, $context) { |
|
124 | + return array_combine($items, array_map(function($item) use ($class, $context) { |
|
125 | 125 | return new $class($item, $context); |
126 | 126 | }, $items)); |
127 | 127 | } |
@@ -136,7 +136,7 @@ discard block |
||
136 | 136 | */ |
137 | 137 | public static function toRichObjectArray(array $items, $class, UserContext $context) |
138 | 138 | { |
139 | - array_walk($items, function (&$value, $name) use ($class, $context) { |
|
139 | + array_walk($items, function(&$value, $name) use ($class, $context) { |
|
140 | 140 | $value = new $class($name, $context, $value); |
141 | 141 | }); |
142 | 142 | return $items; |
@@ -56,7 +56,8 @@ discard block |
||
56 | 56 | */ |
57 | 57 | protected function getCache($key, $default) |
58 | 58 | { |
59 | - if (!isset($this->cache[$key])) { |
|
59 | + if (!isset($this->cache[$key])) |
|
60 | + { |
|
60 | 61 | $this->cache[$key] = is_callable($default) ? $default() : $default; |
61 | 62 | } |
62 | 63 | return $this->cache[$key]; |
@@ -73,10 +74,12 @@ discard block |
||
73 | 74 | */ |
74 | 75 | protected function getCacheItem($key, $item, $defaultKey, $defaultItem = null) |
75 | 76 | { |
76 | - if (empty($cache = $this->getCache($key, $defaultKey))) { |
|
77 | + if (empty($cache = $this->getCache($key, $defaultKey))) |
|
78 | + { |
|
77 | 79 | return $defaultItem; |
78 | 80 | } |
79 | - if (!is_array($cache)) { |
|
81 | + if (!is_array($cache)) |
|
82 | + { |
|
80 | 83 | throw new DirectAdminException("Cache item $key is not an array"); |
81 | 84 | } |
82 | 85 | return isset($cache[$item]) ? $cache[$item] : $defaultItem; |
@@ -69,7 +69,7 @@ discard block |
||
69 | 69 | $this->domainName = $data['domain']; |
70 | 70 | |
71 | 71 | // Determine owner |
72 | - if ($data['username'] === $context->getUsername()) { |
|
72 | + if($data['username'] === $context->getUsername()) { |
|
73 | 73 | $this->owner = $context->getContextUser(); |
74 | 74 | } else { |
75 | 75 | throw new DirectAdminException('Could not determine relationship between context user and domain'); |
@@ -152,7 +152,7 @@ discard block |
||
152 | 152 | 'from' => $domain, |
153 | 153 | 'action' => 'add', |
154 | 154 | ]; |
155 | - if ($alias) { |
|
155 | + if($alias) { |
|
156 | 156 | $parameters['alias'] = 'yes'; |
157 | 157 | $list = &$this->aliases; |
158 | 158 | } else { |
@@ -243,7 +243,7 @@ discard block |
||
243 | 243 | */ |
244 | 244 | public function getDomainNames() |
245 | 245 | { |
246 | - return $this->getCache('domainNames', function () { |
|
246 | + return $this->getCache('domainNames', function() { |
|
247 | 247 | $list = array_merge($this->aliases, $this->pointers, [$this->getDomainName()]); |
248 | 248 | sort($list); |
249 | 249 | return $list; |
@@ -255,7 +255,7 @@ discard block |
||
255 | 255 | */ |
256 | 256 | public function getForwarders() |
257 | 257 | { |
258 | - return $this->getCache(self::CACHE_FORWARDERS, function () { |
|
258 | + return $this->getCache(self::CACHE_FORWARDERS, function() { |
|
259 | 259 | $forwarders = $this->getContext()->invokeGet('EMAIL_FORWARDERS', [ |
260 | 260 | 'domain' => $this->getDomainName(), |
261 | 261 | ]); |
@@ -268,7 +268,7 @@ discard block |
||
268 | 268 | */ |
269 | 269 | public function getMailboxes() |
270 | 270 | { |
271 | - return $this->getCache(self::CACHE_MAILBOXES, function () { |
|
271 | + return $this->getCache(self::CACHE_MAILBOXES, function() { |
|
272 | 272 | $boxes = $this->getContext()->invokeGet('POP', [ |
273 | 273 | 'domain' => $this->getDomainName(), |
274 | 274 | 'action' => 'full_list', |
@@ -298,7 +298,7 @@ discard block |
||
298 | 298 | */ |
299 | 299 | public function getSubdomains() |
300 | 300 | { |
301 | - return $this->getCache(self::CACHE_SUBDOMAINS, function () { |
|
301 | + return $this->getCache(self::CACHE_SUBDOMAINS, function() { |
|
302 | 302 | $subs = $this->getContext()->invokeGet('SUBDOMAINS', ['domain' => $this->getDomainName()]); |
303 | 303 | $subs = array_combine($subs, $subs); |
304 | 304 | return DomainObject::toDomainObjectArray($subs, Subdomain::class, $this); |
@@ -320,7 +320,7 @@ discard block |
||
320 | 320 | 'action' => $action, |
321 | 321 | 'domain' => $this->domainName, |
322 | 322 | ], $parameters)); |
323 | - if ($clearCache) { |
|
323 | + if($clearCache) { |
|
324 | 324 | $this->clearCache(); |
325 | 325 | } |
326 | 326 | return $response; |
@@ -69,9 +69,12 @@ discard block |
||
69 | 69 | $this->domainName = $data['domain']; |
70 | 70 | |
71 | 71 | // Determine owner |
72 | - if ($data['username'] === $context->getUsername()) { |
|
72 | + if ($data['username'] === $context->getUsername()) |
|
73 | + { |
|
73 | 74 | $this->owner = $context->getContextUser(); |
74 | - } else { |
|
75 | + } |
|
76 | + else |
|
77 | + { |
|
75 | 78 | throw new DirectAdminException('Could not determine relationship between context user and domain'); |
76 | 79 | } |
77 | 80 | |
@@ -152,10 +155,13 @@ discard block |
||
152 | 155 | 'from' => $domain, |
153 | 156 | 'action' => 'add', |
154 | 157 | ]; |
155 | - if ($alias) { |
|
158 | + if ($alias) |
|
159 | + { |
|
156 | 160 | $parameters['alias'] = 'yes'; |
157 | 161 | $list = &$this->aliases; |
158 | - } else { |
|
162 | + } |
|
163 | + else |
|
164 | + { |
|
159 | 165 | $list = &$this->pointers; |
160 | 166 | } |
161 | 167 | $this->getContext()->invokePost('DOMAIN_POINTER', $parameters); |
@@ -320,7 +326,8 @@ discard block |
||
320 | 326 | 'action' => $action, |
321 | 327 | 'domain' => $this->domainName, |
322 | 328 | ], $parameters)); |
323 | - if ($clearCache) { |
|
329 | + if ($clearCache) |
|
330 | + { |
|
324 | 331 | $this->clearCache(); |
325 | 332 | } |
326 | 333 | return $response; |
@@ -83,7 +83,7 @@ discard block |
||
83 | 83 | public function deleteAccounts(array $usernames) |
84 | 84 | { |
85 | 85 | $options = ['confirmed' => 'Confirm', 'delete' => 'yes']; |
86 | - foreach (array_values($usernames) as $idx => $username) { |
|
86 | + foreach(array_values($usernames) as $idx => $username) { |
|
87 | 87 | $options["select{$idx}"] = $username; |
88 | 88 | } |
89 | 89 | $this->invokePost('SELECT_USERS', $options); |
@@ -117,7 +117,7 @@ discard block |
||
117 | 117 | public function suspendAccounts(array $usernames, $suspend = true) |
118 | 118 | { |
119 | 119 | $options = ['suspend' => $suspend ? 'Suspend' : 'Unsuspend']; |
120 | - foreach (array_values($usernames) as $idx => $username) { |
|
120 | + foreach(array_values($usernames) as $idx => $username) { |
|
121 | 121 | $options["select{$idx}"] = $username; |
122 | 122 | } |
123 | 123 | $this->invokePost('SELECT_USERS', $options); |
@@ -83,7 +83,8 @@ discard block |
||
83 | 83 | public function deleteAccounts(array $usernames) |
84 | 84 | { |
85 | 85 | $options = ['confirmed' => 'Confirm', 'delete' => 'yes']; |
86 | - foreach (array_values($usernames) as $idx => $username) { |
|
86 | + foreach (array_values($usernames) as $idx => $username) |
|
87 | + { |
|
87 | 88 | $options["select{$idx}"] = $username; |
88 | 89 | } |
89 | 90 | $this->invokePost('SELECT_USERS', $options); |
@@ -117,7 +118,8 @@ discard block |
||
117 | 118 | public function suspendAccounts(array $usernames, $suspend = true) |
118 | 119 | { |
119 | 120 | $options = ['suspend' => $suspend ? 'Suspend' : 'Unsuspend']; |
120 | - foreach (array_values($usernames) as $idx => $username) { |
|
121 | + foreach (array_values($usernames) as $idx => $username) |
|
122 | + { |
|
121 | 123 | $options["select{$idx}"] = $username; |
122 | 124 | } |
123 | 125 | $this->invokePost('SELECT_USERS', $options); |