Test Failed
Push — main ( 6fe7e7...953723 )
by Dylan
03:16 queued 01:13
created
src/CurlResponse.php 1 patch
Braces   +3 added lines, -1 removed lines patch added patch discarded remove patch
@@ -40,7 +40,9 @@
 block discarded – undo
40 40
     public function getJSON(): ?array
41 41
     {
42 42
         $data = json_decode($this->result, true);
43
-        if (!is_array($data)) return null;
43
+        if (!is_array($data)) {
44
+            return null;
45
+        }
44 46
         
45 47
         return $data;
46 48
     }
Please login to merge, or discard this patch.
src/App.php 1 patch
Braces   +3 added lines, -1 removed lines patch added patch discarded remove patch
@@ -44,7 +44,9 @@
 block discarded – undo
44 44
      */
45 45
     public function getAPIChallenge(): string
46 46
     {
47
-        if (!$this->_api_challenge) $this->_api_challenge = Utils::create_random_string(128);
47
+        if (!$this->_api_challenge) {
48
+            $this->_api_challenge = Utils::create_random_string(128);
49
+        }
48 50
         return $this->_api_challenge;
49 51
     }
50 52
 
Please login to merge, or discard this patch.
src/utils/Curl.php 1 patch
Braces   +27 added lines, -9 removed lines patch added patch discarded remove patch
@@ -39,9 +39,15 @@  discard block
 block discarded – undo
39 39
      */
40 40
     public function __construct(string $url = null, array $data = [], array $headers = [])
41 41
     {
42
-        if (!is_null($url)) $this->setURL($url);
43
-        foreach ($data as $name => $value)      $this->addDataParam($name, $value);
44
-        foreach ($headers as $name => $value)   $this->addHeader($name, $value);
42
+        if (!is_null($url)) {
43
+            $this->setURL($url);
44
+        }
45
+        foreach ($data as $name => $value) {
46
+            $this->addDataParam($name, $value);
47
+        }
48
+        foreach ($headers as $name => $value) {
49
+            $this->addHeader($name, $value);
50
+        }
45 51
     }
46 52
 
47 53
     /**
@@ -98,7 +104,9 @@  discard block
 block discarded – undo
98 104
 
99 105
     public function removeDataParam(string $name): Curl
100 106
     {
101
-        if (array_key_exists($name, $this->_data)) unset($this->_data[$name]);
107
+        if (array_key_exists($name, $this->_data)) {
108
+            unset($this->_data[$name]);
109
+        }
102 110
         return $this;
103 111
     }
104 112
 
@@ -127,7 +135,9 @@  discard block
 block discarded – undo
127 135
      */
128 136
     public function removeHeader(string $name): Curl
129 137
     {
130
-        if (array_key_exists($name, $this->_headers)) unset($this->_headers[$name]);
138
+        if (array_key_exists($name, $this->_headers)) {
139
+            unset($this->_headers[$name]);
140
+        }
131 141
         return $this;
132 142
     }
133 143
 
@@ -171,13 +181,17 @@  discard block
 block discarded – undo
171 181
         $request_uri    = $this->getURL();
172 182
 
173 183
         //  Headers
174
-        foreach ($this->getHeaders() as $k => $v) $send_headers[] = "{$k}: {$v}";
184
+        foreach ($this->getHeaders() as $k => $v) {
185
+            $send_headers[] = "{$k}: {$v}";
186
+        }
175 187
 
176 188
         // Request Data
177 189
         switch ($this->getMethod()) {
178 190
             case 'GET':
179 191
             case 'DELETE':
180
-                foreach ($this->getDataParams() as $name => $value) $request_uri = URL::setGetVar($name, $value, $request_uri);
192
+                foreach ($this->getDataParams() as $name => $value) {
193
+                    $request_uri = URL::setGetVar($name, $value, $request_uri);
194
+                }
181 195
                 break;
182 196
 
183 197
             case 'POST':
@@ -208,7 +222,9 @@  discard block
 block discarded – undo
208 222
         curl_setopt($ch, CURLINFO_HEADER_OUT, true);
209 223
         curl_setopt($ch, CURLOPT_AUTOREFERER, true);
210 224
 
211
-        if (!empty($send_headers))  curl_setopt($ch, CURLOPT_HTTPHEADER, $send_headers);
225
+        if (!empty($send_headers)) {
226
+            curl_setopt($ch, CURLOPT_HTTPHEADER, $send_headers);
227
+        }
212 228
         if (!is_null($post_data))   {
213 229
             curl_setopt($ch, CURLOPT_POST, 1);
214 230
             curl_setopt($ch, CURLOPT_POSTFIELDS, $post_data);
@@ -218,7 +234,9 @@  discard block
 block discarded – undo
218 234
         $http_code  = curl_getinfo($ch, CURLINFO_HTTP_CODE);
219 235
 
220 236
         $response = new CurlResponse($http_code, $result);
221
-        if ($this->_enable_cache && isset($cache_key)) self::$_cache[$cache_key] = $response;
237
+        if ($this->_enable_cache && isset($cache_key)) {
238
+            self::$_cache[$cache_key] = $response;
239
+        }
222 240
 
223 241
         return $response;
224 242
     }
Please login to merge, or discard this patch.
src/resource/ListResource.php 1 patch
Braces   +15 added lines, -5 removed lines patch added patch discarded remove patch
@@ -107,11 +107,15 @@  discard block
 block discarded – undo
107 107
 
108 108
             $this->_max_items = (int) $data['available_items'];
109 109
 
110
-            if (empty($data['items'])) return $this->_items[$page] = [];
110
+            if (empty($data['items'])) {
111
+                return $this->_items[$page] = [];
112
+            }
111 113
 
112 114
             foreach ($data['items'] as $item) {
113 115
                 $obj = ObjectFactory::make($this->getClient(), $item);
114
-                if (!$obj) continue;
116
+                if (!$obj) {
117
+                    continue;
118
+                }
115 119
 
116 120
                 $this->_items[$page][] = $obj;
117 121
             }
@@ -126,7 +130,9 @@  discard block
 block discarded – undo
126 130
      */
127 131
     public function offsetGet($offset): ?ObjectResource
128 132
     {
129
-        foreach ($this as $i => $obj) if ($i === $offset) return $obj;
133
+        foreach ($this as $i => $obj) {
134
+            if ($i === $offset) return $obj;
135
+        }
130 136
         return null;
131 137
     }
132 138
 
@@ -157,7 +163,9 @@  discard block
 block discarded – undo
157 163
      */
158 164
     public function offsetExists($offset): bool
159 165
     {
160
-        foreach ($this as $i => $obj) if ($offset === $i) return true;
166
+        foreach ($this as $i => $obj) {
167
+            if ($offset === $i) return true;
168
+        }
161 169
         return false;
162 170
     }
163 171
 
@@ -213,6 +221,8 @@  discard block
 block discarded – undo
213 221
      */
214 222
     public function first(): ?Model
215 223
     {
216
-        foreach ($this as $obj) return $obj;
224
+        foreach ($this as $obj) {
225
+            return $obj;
226
+        }
217 227
     }
218 228
 }
Please login to merge, or discard this patch.
src/Connector.php 1 patch
Braces   +3 added lines, -1 removed lines patch added patch discarded remove patch
@@ -45,7 +45,9 @@
 block discarded – undo
45 45
     public function __get(string $service): ?ApiService
46 46
     {
47 47
         $obj = ServiceFactory::inst($this, $service);
48
-        if (!$obj) throw new BadMethodException("Service for `{$service}` does not exist");
48
+        if (!$obj) {
49
+            throw new BadMethodException("Service for `{$service}` does not exist");
50
+        }
49 51
 
50 52
         return $obj;
51 53
     }
Please login to merge, or discard this patch.
src/factory/ServiceFactory.php 1 patch
Braces   +3 added lines, -1 removed lines patch added patch discarded remove patch
@@ -19,7 +19,9 @@
 block discarded – undo
19 19
     public static function inst(Connector $connector, string $service): ?ApiService
20 20
     {
21 21
         $service = strtolower($service);
22
-        if (!array_key_exists($service, ClassMap::SERVICES)) return null;
22
+        if (!array_key_exists($service, ClassMap::SERVICES)) {
23
+            return null;
24
+        }
23 25
 
24 26
         $cls = ClassMap::SERVICES[$service];
25 27
         return new $cls($connector);
Please login to merge, or discard this patch.
src/services/ApiService.php 1 patch
Braces   +3 added lines, -1 removed lines patch added patch discarded remove patch
@@ -98,7 +98,9 @@
 block discarded – undo
98 98
     {
99 99
         $curl = $this->getClient()->curl_api($url, 'DELETE', $params);
100 100
 
101
-        if ($curl->isValid()) return true;
101
+        if ($curl->isValid()) {
102
+            return true;
103
+        }
102 104
 
103 105
         throw new ApiException($curl->getError());
104 106
     }
Please login to merge, or discard this patch.
src/models/Address.php 1 patch
Braces   +3 added lines, -1 removed lines patch added patch discarded remove patch
@@ -59,7 +59,9 @@
 block discarded – undo
59 59
      */
60 60
     public function Customer(): ?Customer
61 61
     {
62
-        if (!$this->CustomerID) return null;
62
+        if (!$this->CustomerID) {
63
+            return null;
64
+        }
63 65
 
64 66
         /** @var Customer|null $customer */
65 67
         $customer = $this->getClient()->customers->fetch($this->CustomerID);
Please login to merge, or discard this patch.
src/resource/SimpleList.php 1 patch
Braces   +6 added lines, -2 removed lines patch added patch discarded remove patch
@@ -42,7 +42,9 @@  discard block
 block discarded – undo
42 42
 
43 43
             foreach ($list as $item) {
44 44
                 $obj = ObjectFactory::make($this->getClient(), $item);
45
-                if (!$obj) continue;
45
+                if (!$obj) {
46
+                    continue;
47
+                }
46 48
 
47 49
                 $this->_items[] = $obj;
48 50
             }
@@ -76,6 +78,8 @@  discard block
 block discarded – undo
76 78
      */
77 79
     public function first(): ?Model
78 80
     {
79
-        foreach ($this as $obj) return $obj;
81
+        foreach ($this as $obj) {
82
+            return $obj;
83
+        }
80 84
     }
81 85
 }
Please login to merge, or discard this patch.