View GitHub Repository Sylius

Schedule Inspection
Subscribe
Completed
Push — symfony3-fqcn-sylius ( 26083b...02605f )
by Kamil
20:07
created
Status
Category
Toggle Patches
tests/Controller/ZoneApiTest.php 1 patch
Spacing   +8 added lines, -8 removed lines patch added patch discarded remove patch 
@@ -146,7 +146,7 @@  discard block
 
 block discarded – undo
146 146 
         $this->loadFixturesFromFile('authentication/api_administrator.yml');
147 147 
         $zones = $this->loadFixturesFromFile('resources/zones.yml');
148 148
149 
-        $this->client->request('GET', '/api/zones/'.$zones['zone_eu']->getCode(), [], [], static::$authorizedHeaderWithAccept);
149 
+        $this->client->request('GET', '/api/zones/' . $zones['zone_eu']->getCode(), [], [], static::$authorizedHeaderWithAccept);
150 150
151 151 
         $response = $this->client->getResponse();
152 152 
         $this->assertResponse($response, 'zone/show_response', Response::HTTP_OK);
 
@@ -184,7 +184,7 @@  discard block
 
 block discarded – undo
184 184 
         $this->loadFixturesFromFile('authentication/api_administrator.yml');
185 185 
         $zones = $this->loadFixturesFromFile('resources/zones.yml');
186 186
187 
-        $this->client->request('PUT', '/api/zones/'.$zones['zone_eu']->getCode(), [], [], static::$authorizedHeaderWithContentType);
187 
+        $this->client->request('PUT', '/api/zones/' . $zones['zone_eu']->getCode(), [], [], static::$authorizedHeaderWithContentType);
188 188
189 189 
         $response = $this->client->getResponse();
190 190 
         $this->assertResponse($response, 'zone/update_validation_fail_response', Response::HTTP_BAD_REQUEST);
 
@@ -211,12 +211,12 @@  discard block
 
 block discarded – undo
211 211 
         }
212 212 
 EOT;
213 213
214 
-        $this->client->request('PUT', '/api/zones/'.$zones['zone_eu']->getCode(), [], [], static::$authorizedHeaderWithContentType, $data);
214 
+        $this->client->request('PUT', '/api/zones/' . $zones['zone_eu']->getCode(), [], [], static::$authorizedHeaderWithContentType, $data);
215 215
216 216 
         $response = $this->client->getResponse();
217 217 
         $this->assertResponseCode($response, Response::HTTP_NO_CONTENT);
218 218
219 
-        $this->client->request('GET', '/api/zones/'.$zones['zone_eu']->getCode(), [], [], static::$authorizedHeaderWithAccept);
219 
+        $this->client->request('GET', '/api/zones/' . $zones['zone_eu']->getCode(), [], [], static::$authorizedHeaderWithAccept);
220 220
221 221 
         $response = $this->client->getResponse();
222 222 
         $this->assertResponse($response, 'zone/update_response', Response::HTTP_OK);
 
@@ -262,12 +262,12 @@  discard block
 
 block discarded – undo
262 262 
         }
263 263 
 EOT;
264 264
265 
-        $this->client->request('PATCH', '/api/zones/'.$zones['zone_eu']->getCode(), [], [], static::$authorizedHeaderWithContentType, $data);
265 
+        $this->client->request('PATCH', '/api/zones/' . $zones['zone_eu']->getCode(), [], [], static::$authorizedHeaderWithContentType, $data);
266 266
267 267 
         $response = $this->client->getResponse();
268 268 
         $this->assertResponseCode($response, Response::HTTP_NO_CONTENT);
269 269
270 
-        $this->client->request('GET', '/api/zones/'.$zones['zone_eu']->getCode(), [], [], static::$authorizedHeaderWithAccept);
270 
+        $this->client->request('GET', '/api/zones/' . $zones['zone_eu']->getCode(), [], [], static::$authorizedHeaderWithAccept);
271 271
272 272 
         $response = $this->client->getResponse();
273 273 
         $this->assertResponse($response, 'zone/update_response', Response::HTTP_OK);
 
@@ -305,12 +305,12 @@  discard block
 
 block discarded – undo
305 305 
         $this->loadFixturesFromFile('authentication/api_administrator.yml');
306 306 
         $zones = $this->loadFixturesFromFile('resources/zones.yml');
307 307
308 
-        $this->client->request('DELETE', '/api/zones/'.$zones['zone_eu']->getCode(), [], [], static::$authorizedHeaderWithContentType);
308 
+        $this->client->request('DELETE', '/api/zones/' . $zones['zone_eu']->getCode(), [], [], static::$authorizedHeaderWithContentType);
309 309
310 310 
         $response = $this->client->getResponse();
311 311 
         $this->assertResponseCode($response, Response::HTTP_NO_CONTENT);
312 312
313 
-        $this->client->request('GET', '/api/zones/'.$zones['zone_eu']->getCode(), [], [], static::$authorizedHeaderWithAccept);
313 
+        $this->client->request('GET', '/api/zones/' . $zones['zone_eu']->getCode(), [], [], static::$authorizedHeaderWithAccept);
314 314
315 315 
         $response = $this->client->getResponse();
316 316 
         $this->assertResponseCode($response, Response::HTTP_NOT_FOUND);
Please login to merge, or discard this patch.
tests/Controller/LocaleApiTest.php 1 patch
Spacing   +3 added lines, -3 removed lines patch added patch discarded remove patch 
@@ -62,7 +62,7 @@  discard block
 
 block discarded – undo
62 62 
         $this->loadFixturesFromFile('authentication/api_administrator.yml');
63 63 
         $locales = $this->loadFixturesFromFile('resources/locales.yml');
64 64
65 
-        $this->client->request('GET', '/api/locales/'.$locales['locale_en']->getCode(), [], [], static::$authorizedHeaderWithAccept);
65 
+        $this->client->request('GET', '/api/locales/' . $locales['locale_en']->getCode(), [], [], static::$authorizedHeaderWithAccept);
66 66
67 67 
         $response = $this->client->getResponse();
68 68 
         $this->assertResponse($response, 'locale/show_response', Response::HTTP_OK);
 
@@ -130,12 +130,12 @@  discard block
 
 block discarded – undo
130 130 
         $this->loadFixturesFromFile('authentication/api_administrator.yml');
131 131 
         $locales = $this->loadFixturesFromFile('resources/locales.yml');
132 132
133 
-        $this->client->request('DELETE', '/api/locales/'.$locales['locale_en']->getCode(), [], [], static::$authorizedHeaderWithContentType, []);
133 
+        $this->client->request('DELETE', '/api/locales/' . $locales['locale_en']->getCode(), [], [], static::$authorizedHeaderWithContentType, []);
134 134
135 135 
         $response = $this->client->getResponse();
136 136 
         $this->assertResponseCode($response, Response::HTTP_NO_CONTENT);
137 137
138 
-        $this->client->request('GET', '/api/locales/'.$locales['locale_en']->getId(), [], [], static::$authorizedHeaderWithAccept);
138 
+        $this->client->request('GET', '/api/locales/' . $locales['locale_en']->getId(), [], [], static::$authorizedHeaderWithAccept);
139 139
140 140 
         $response = $this->client->getResponse();
141 141 
         $this->assertResponse($response, 'error/not_found_response', Response::HTTP_NOT_FOUND);
Please login to merge, or discard this patch.
tests/Controller/CountryApiTest.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch 
@@ -96,7 +96,7 @@
 
 block discarded – undo
96 96 
         $this->loadFixturesFromFile('authentication/api_administrator.yml');
97 97 
         $countries = $this->loadFixturesFromFile('resources/countries.yml');
98 98
99 
-        $this->client->request('GET', '/api/countries/'.$countries['country_NL']->getCode(), [], [], static::$authorizedHeaderWithAccept);
99 
+        $this->client->request('GET', '/api/countries/' . $countries['country_NL']->getCode(), [], [], static::$authorizedHeaderWithAccept);
100 100
101 101 
         $response = $this->client->getResponse();
102 102 
         $this->assertResponse($response, 'country/show_response', Response::HTTP_OK);
Please login to merge, or discard this patch.
tests/Controller/CurrencyApiTest.php 1 patch
Spacing   +8 added lines, -8 removed lines patch added patch discarded remove patch 
@@ -122,7 +122,7 @@  discard block
 
 block discarded – undo
122 122 
         $this->loadFixturesFromFile('authentication/api_administrator.yml');
123 123 
         $currencies = $this->loadFixturesFromFile('resources/currencies.yml');
124 124
125 
-        $this->client->request('GET', '/api/currencies/'.$currencies['currency_1']->getCode(), [], [], static::$authorizedHeaderWithAccept);
125 
+        $this->client->request('GET', '/api/currencies/' . $currencies['currency_1']->getCode(), [], [], static::$authorizedHeaderWithAccept);
126 126
127 127 
         $response = $this->client->getResponse();
128 128 
         $this->assertResponse($response, 'currency/show_response', Response::HTTP_OK);
 
@@ -151,7 +151,7 @@  discard block
 
 block discarded – undo
151 151 
         $this->loadFixturesFromFile('authentication/api_administrator.yml');
152 152 
         $currencies = $this->loadFixturesFromFile('resources/currencies.yml');
153 153
154 
-        $this->client->request('PUT', '/api/currencies/'.$currencies['currency_2']->getCode(), [], [], static::$authorizedHeaderWithContentType);
154 
+        $this->client->request('PUT', '/api/currencies/' . $currencies['currency_2']->getCode(), [], [], static::$authorizedHeaderWithContentType);
155 155
156 156 
         $response = $this->client->getResponse();
157 157 
 
@@ -172,12 +172,12 @@  discard block
 
 block discarded – undo
172 172 
         }
173 173 
 EOT;
174 174
175 
-        $this->client->request('PUT', '/api/currencies/'.$currencies['currency_2']->getCode(), [], [], static::$authorizedHeaderWithContentType, $data);
175 
+        $this->client->request('PUT', '/api/currencies/' . $currencies['currency_2']->getCode(), [], [], static::$authorizedHeaderWithContentType, $data);
176 176
177 177 
         $response = $this->client->getResponse();
178 178 
         $this->assertResponseCode($response, Response::HTTP_NO_CONTENT);
179 179
180 
-        $this->client->request('GET', '/api/currencies/'.$currencies['currency_2']->getCode(), [], [], static::$authorizedHeaderWithAccept);
180 
+        $this->client->request('GET', '/api/currencies/' . $currencies['currency_2']->getCode(), [], [], static::$authorizedHeaderWithAccept);
181 181
182 182 
         $response = $this->client->getResponse();
183 183 
         $this->assertResponse($response, 'currency/update_response', Response::HTTP_OK);
 
@@ -206,12 +206,12 @@  discard block
 
 block discarded – undo
206 206 
         }
207 207 
 EOT;
208 208
209 
-        $this->client->request('PATCH', '/api/currencies/'.$currencies['currency_2']->getCode(), [], [], static::$authorizedHeaderWithContentType, $data);
209 
+        $this->client->request('PATCH', '/api/currencies/' . $currencies['currency_2']->getCode(), [], [], static::$authorizedHeaderWithContentType, $data);
210 210
211 211 
         $response = $this->client->getResponse();
212 212 
         $this->assertResponseCode($response, Response::HTTP_NO_CONTENT);
213 213
214 
-        $this->client->request('GET', '/api/currencies/'.$currencies['currency_2']->getCode(), [], [], static::$authorizedHeaderWithAccept);
214 
+        $this->client->request('GET', '/api/currencies/' . $currencies['currency_2']->getCode(), [], [], static::$authorizedHeaderWithAccept);
215 215
216 216 
         $response = $this->client->getResponse();
217 217 
         $this->assertResponse($response, 'currency/update_response', Response::HTTP_OK);
 
@@ -232,12 +232,12 @@  discard block
 
 block discarded – undo
232 232 
         $this->loadFixturesFromFile('authentication/api_administrator.yml');
233 233 
         $currencies = $this->loadFixturesFromFile('resources/currencies.yml');
234 234
235 
-        $this->client->request('DELETE', '/api/currencies/'.$currencies['currency_1']->getCode(), [], [], static::$authorizedHeaderWithContentType, []);
235 
+        $this->client->request('DELETE', '/api/currencies/' . $currencies['currency_1']->getCode(), [], [], static::$authorizedHeaderWithContentType, []);
236 236
237 237 
         $response = $this->client->getResponse();
238 238 
         $this->assertResponseCode($response, Response::HTTP_NO_CONTENT);
239 239
240 
-        $this->client->request('GET', '/api/currencies/'.$currencies['currency_1']->getCode(), [], [], static::$authorizedHeaderWithAccept);
240 
+        $this->client->request('GET', '/api/currencies/' . $currencies['currency_1']->getCode(), [], [], static::$authorizedHeaderWithAccept);
241 241
242 242 
         $response = $this->client->getResponse();
243 243 
         $this->assertResponse($response, 'error/not_found_response', Response::HTTP_NOT_FOUND);
Please login to merge, or discard this patch.
tests/Controller/ProvinceApiTest.php 1 patch
Spacing   +3 added lines, -3 removed lines patch added patch discarded remove patch 
@@ -40,7 +40,7 @@  discard block
 
 block discarded – undo
40 40 
         $this->loadFixturesFromFile('authentication/api_administrator.yml');
41 41 
         $countryData = $this->loadFixturesFromFile('resources/countries.yml');
42 42
43 
-        $this->client->request('GET', '/api/countries/'.$countryData['country_BE']->getCode().'/provinces/'.$countryData['province_BE_limburg']->getCode(), [], [], static::$authorizedHeaderWithContentType);
43 
+        $this->client->request('GET', '/api/countries/' . $countryData['country_BE']->getCode() . '/provinces/' . $countryData['province_BE_limburg']->getCode(), [], [], static::$authorizedHeaderWithContentType);
44 44
45 45 
         $response = $this->client->getResponse();
46 46 
         $this->assertResponse($response, 'province/show_response', Response::HTTP_OK);
 
@@ -51,12 +51,12 @@  discard block
 
 block discarded – undo
51 51 
         $this->loadFixturesFromFile('authentication/api_administrator.yml');
52 52 
         $countryData = $this->loadFixturesFromFile('resources/countries.yml');
53 53
54 
-        $this->client->request('DELETE', '/api/countries/'.$countryData['country_BE']->getCode().'/provinces/'.$countryData['province_BE_limburg']->getCode(), [], [], static::$authorizedHeaderWithContentType);
54 
+        $this->client->request('DELETE', '/api/countries/' . $countryData['country_BE']->getCode() . '/provinces/' . $countryData['province_BE_limburg']->getCode(), [], [], static::$authorizedHeaderWithContentType);
55 55
56 56 
         $response = $this->client->getResponse();
57 57 
         $this->assertResponseCode($response, Response::HTTP_NO_CONTENT);
58 58
59 
-        $this->client->request('GET', '/api/countries/'.$countryData['country_BE']->getCode().'/provinces/'.$countryData['province_BE_limburg']->getCode(), [], [], static::$authorizedHeaderWithContentType);
59 
+        $this->client->request('GET', '/api/countries/' . $countryData['country_BE']->getCode() . '/provinces/' . $countryData['province_BE_limburg']->getCode(), [], [], static::$authorizedHeaderWithContentType);
60 60
61 61 
         $response = $this->client->getResponse();
62 62 
         $this->assertResponse($response, 'error/not_found_response', Response::HTTP_NOT_FOUND);
Please login to merge, or discard this patch.
src/Sylius/Behat/Context/Setup/ShippingCategoryContext.php 1 patch
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch 
@@ -69,7 +69,7 @@  discard block
 
 block discarded – undo
69 69 
     public function theStoreHasAndShippingCategory($firstShippingCategoryName, $secondShippingCategoryName = null)
70 70 
     {
71 71 
         $this->createShippingCategory($firstShippingCategoryName);
72 
-        (null === $secondShippingCategoryName)? : $this->createShippingCategory($secondShippingCategoryName);
72 
+        (null === $secondShippingCategoryName) ?: $this->createShippingCategory($secondShippingCategoryName);
73 73 
     }
74 74
75 75 
     /**
 
@@ -78,7 +78,7 @@  discard block
 
 block discarded – undo
78 78 
     private function createShippingCategory($shippingCategoryName)
79 79 
     {
80 80 
         /** @var ShippingCategoryInterface $shippingCategory */
81 
-        $shippingCategory =  $this->shippingCategoryFactory->createNew();
81 
+        $shippingCategory = $this->shippingCategoryFactory->createNew();
82 82 
         $shippingCategory->setName($shippingCategoryName);
83 83 
         $shippingCategory->setCode(StringInflector::nameToCode($shippingCategoryName));
84 84
Please login to merge, or discard this patch.