Completed
Push — product/missing-methods ( 0cfbf7 )
by Kamil
25:41
created
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']->getId(), [], [], static::$authorizedHeaderWithAccept);
125
+        $this->client->request('GET', '/api/currencies/' . $currencies['currency_1']->getId(), [], [], 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']->getId(), [], [], static::$authorizedHeaderWithContentType);
154
+        $this->client->request('PUT', '/api/currencies/' . $currencies['currency_2']->getId(), [], [], 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']->getId(), [], [], static::$authorizedHeaderWithContentType, $data);
175
+        $this->client->request('PUT', '/api/currencies/' . $currencies['currency_2']->getId(), [], [], 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']->getId(), [], [], static::$authorizedHeaderWithAccept);
180
+        $this->client->request('GET', '/api/currencies/' . $currencies['currency_2']->getId(), [], [], 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']->getId(), [], [], static::$authorizedHeaderWithContentType, $data);
209
+        $this->client->request('PATCH', '/api/currencies/' . $currencies['currency_2']->getId(), [], [], 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']->getId(), [], [], static::$authorizedHeaderWithAccept);
214
+        $this->client->request('GET', '/api/currencies/' . $currencies['currency_2']->getId(), [], [], 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']->getId(), [], [], static::$authorizedHeaderWithContentType, []);
235
+        $this->client->request('DELETE', '/api/currencies/' . $currencies['currency_1']->getId(), [], [], 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']->getId(), [], [], static::$authorizedHeaderWithAccept);
240
+        $this->client->request('GET', '/api/currencies/' . $currencies['currency_1']->getId(), [], [], 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']->getId().'/provinces/'.$countryData['province_BE_limburg']->getId(), [], [], static::$authorizedHeaderWithContentType);
43
+        $this->client->request('GET', '/api/countries/' . $countryData['country_BE']->getId() . '/provinces/' . $countryData['province_BE_limburg']->getId(), [], [], 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']->getId().'/provinces/'.$countryData['province_BE_limburg']->getId(), [], [], static::$authorizedHeaderWithContentType);
54
+        $this->client->request('DELETE', '/api/countries/' . $countryData['country_BE']->getId() . '/provinces/' . $countryData['province_BE_limburg']->getId(), [], [], 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']->getId().'/provinces/'.$countryData['province_BE_limburg']->getId(), [], [], static::$authorizedHeaderWithContentType);
59
+        $this->client->request('GET', '/api/countries/' . $countryData['country_BE']->getId() . '/provinces/' . $countryData['province_BE_limburg']->getId(), [], [], 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.
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']->getId(), [], [], static::$authorizedHeaderWithAccept);
99
+        $this->client->request('GET', '/api/countries/' . $countries['country_NL']->getId(), [], [], 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.
src/Sylius/Behat/Context/Setup/ShippingContext.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -218,6 +218,6 @@
 block discarded – undo
218 218
      */
219 219
     private function generateCodeFromNameAndZone($shippingMethodName, $zoneCode = null)
220 220
     {
221
-        return str_replace([' ', '-'], '_', strtolower($shippingMethodName)).'_'.strtolower($zoneCode);
221
+        return str_replace([' ', '-'], '_', strtolower($shippingMethodName)) . '_' . strtolower($zoneCode);
222 222
     }
223 223
 }
Please login to merge, or discard this patch.
src/Sylius/Behat/Context/Setup/OrderContext.php 1 patch
Spacing   +3 added lines, -3 removed lines patch added patch discarded remove patch
@@ -310,7 +310,7 @@  discard block
 block discarded – undo
310 310
     {
311 311
         $customer = $user->getCustomer();
312 312
         for ($i = 0; $i < $numberOfOrders; $i++) {
313
-            $order = $this->createOrder($customer, '#00000'.$i);
313
+            $order = $this->createOrder($customer, '#00000' . $i);
314 314
             $order->setPaymentState(PaymentInterface::STATE_COMPLETED);
315 315
             $order->setCompletedAt(new \DateTime());
316 316
 
@@ -358,7 +358,7 @@  discard block
 block discarded – undo
358 358
         $total = $this->getPriceFromString($total);
359 359
 
360 360
         for ($i = 0; $i < $numberOfOrders; $i++) {
361
-            $order = $this->createOrder($customers[rand(0, $numberOfCustomers - 1)], '#'.uniqid());
361
+            $order = $this->createOrder($customers[rand(0, $numberOfCustomers - 1)], '#' . uniqid());
362 362
             $order->setPaymentState(PaymentInterface::STATE_COMPLETED);
363 363
             $order->setCompletedAt(new \DateTime());
364 364
 
@@ -502,7 +502,7 @@  discard block
 block discarded – undo
502 502
             $customer = $this->customerFactory->createNew();
503 503
             $customer->setEmail(sprintf('john%[email protected]', uniqid()));
504 504
             $customer->setFirstname('John');
505
-            $customer->setLastname('Doe'.$i);
505
+            $customer->setLastname('Doe' . $i);
506 506
 
507 507
             $customers[] = $customer;
508 508
 
Please login to merge, or discard this patch.
src/Sylius/Behat/Context/Setup/TaxationContext.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -212,6 +212,6 @@
 block discarded – undo
212 212
      */
213 213
     private function getCodeFromNameAndZoneCode($taxRateName, $zoneCode)
214 214
     {
215
-        return $this->getCodeFromName($taxRateName).'_'.strtolower($zoneCode);
215
+        return $this->getCodeFromName($taxRateName) . '_' . strtolower($zoneCode);
216 216
     }
217 217
 }
Please login to merge, or discard this patch.
src/Sylius/Behat/Context/Ui/CheckoutContext.php 1 patch
Spacing   +3 added lines, -3 removed lines patch added patch discarded remove patch
@@ -530,7 +530,7 @@  discard block
 block discarded – undo
530 530
      */
531 531
     public function iShouldSeeThisShippingAddressAsShippingAddress($fullName)
532 532
     {
533
-        $address = $this->sharedStorage->get('shipping_address_'.StringInflector::nameToLowercaseCode($fullName));
533
+        $address = $this->sharedStorage->get('shipping_address_' . StringInflector::nameToLowercaseCode($fullName));
534 534
         Assert::true(
535 535
             $this->summaryPage->hasShippingAddress($address),
536 536
             'Shipping address is improper.'
@@ -542,7 +542,7 @@  discard block
 block discarded – undo
542 542
      */
543 543
     public function iShouldSeeThisBillingAddressAsBillingAddress($fullName)
544 544
     {
545
-        $address = $this->sharedStorage->get('billing_address_'.StringInflector::nameToLowercaseCode($fullName));
545
+        $address = $this->sharedStorage->get('billing_address_' . StringInflector::nameToLowercaseCode($fullName));
546 546
         Assert::true(
547 547
             $this->summaryPage->hasBillingAddress($address),
548 548
             'Billing address is improper.'
@@ -622,7 +622,7 @@  discard block
 block discarded – undo
622 622
     {
623 623
         Assert::true(
624 624
             $this->summaryPage->hasShippingTotal($price),
625
-            sprintf('The shipping total should be %s, but it is not.',$price)
625
+            sprintf('The shipping total should be %s, but it is not.', $price)
626 626
         );
627 627
     }
628 628
 
Please login to merge, or discard this patch.
src/Sylius/Behat/Context/Cli/InstallerContext.php 1 patch
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -200,7 +200,7 @@  discard block
 block discarded – undo
200 200
     {
201 201
         $this->dialog = $this->command->getHelper('dialog');
202 202
         $inputString = join(PHP_EOL, $this->inputChoices);
203
-        $this->dialog->setInputStream($this->getInputStream($inputString.PHP_EOL));
203
+        $this->dialog->setInputStream($this->getInputStream($inputString . PHP_EOL));
204 204
 
205 205
         $this->tester->execute(['command' => $name]);
206 206
     }
@@ -211,7 +211,7 @@  discard block
 block discarded – undo
211 211
     private function iExecuteCommandAndConfirm($name)
212 212
     {
213 213
         $this->dialog = $this->command->getHelper('dialog');
214
-        $inputString = 'y'.PHP_EOL;
214
+        $inputString = 'y' . PHP_EOL;
215 215
         $this->dialog->setInputStream($this->getInputStream($inputString));
216 216
 
217 217
         $this->tester->execute(['command' => $name]);
Please login to merge, or discard this patch.
src/Sylius/Behat/Extension/MultiContainerExtension/Loader/XmlFileLoader.php 1 patch
Spacing   +7 added lines, -7 removed lines patch added patch discarded remove patch
@@ -177,7 +177,7 @@  discard block
 block discarded – undo
177 177
                 if (in_array($key, ['factory-class', 'factory-method', 'factory-service'])) {
178 178
                     @trigger_error(sprintf('The "%s" attribute of service "%s" in file "%s" is deprecated since version 2.6 and will be removed in 3.0. Use the "factory" element instead.', $key, (string) $service->getAttribute('id'), $file), E_USER_DEPRECATED);
179 179
                 }
180
-                $method = 'set'.str_replace('-', '', $key);
180
+                $method = 'set' . str_replace('-', '', $key);
181 181
                 $definition->$method(XmlUtils::phpize($value));
182 182
             }
183 183
         }
@@ -384,7 +384,7 @@  discard block
 block discarded – undo
384 384
             // this is used by DefinitionDecorator to overwrite a specific
385 385
             // argument of the parent definition
386 386
             if ($arg->hasAttribute('index')) {
387
-                $key = 'index_'.$arg->getAttribute('index');
387
+                $key = 'index_' . $arg->getAttribute('index');
388 388
             }
389 389
 
390 390
             switch ($arg->getAttribute('type')) {
@@ -442,7 +442,7 @@  discard block
 block discarded – undo
442 442
      */
443 443
     public function validateSchema(\DOMDocument $dom)
444 444
     {
445
-        $schemaLocations = ['http://symfony.com/schema/dic/services' => str_replace('\\', '/', __DIR__.'/schema/dic/services/services-1.0.xsd')];
445
+        $schemaLocations = ['http://symfony.com/schema/dic/services' => str_replace('\\', '/', __DIR__ . '/schema/dic/services/services-1.0.xsd')];
446 446
 
447 447
         if ($element = $dom->documentElement->getAttributeNS('http://www.w3.org/2001/XMLSchema-instance', 'schemaLocation')) {
448 448
             $items = preg_split('/\s+/', $element);
@@ -452,7 +452,7 @@  discard block
 block discarded – undo
452 452
                 }
453 453
 
454 454
                 if (($extension = $this->container->getExtension($items[$i])) && false !== $extension->getXsdValidationBasePath()) {
455
-                    $path = str_replace($extension->getNamespace(), str_replace('\\', '/', $extension->getXsdValidationBasePath()).'/', $items[$i + 1]);
455
+                    $path = str_replace($extension->getNamespace(), str_replace('\\', '/', $extension->getXsdValidationBasePath()) . '/', $items[$i + 1]);
456 456
 
457 457
                     if (!is_file($path)) {
458 458
                         throw new RuntimeException(sprintf('Extension "%s" references a non-existent XSD file "%s"', get_class($extension), $path));
@@ -475,10 +475,10 @@  discard block
 block discarded – undo
475 475
                     $parts = explode('/', str_replace('\\', '/', $tmpfile));
476 476
                 }
477 477
             }
478
-            $drive = '\\' === DIRECTORY_SEPARATOR ? array_shift($parts).'/' : '';
479
-            $location = 'file:///'.$drive.implode('/', array_map('rawurlencode', $parts));
478
+            $drive = '\\' === DIRECTORY_SEPARATOR ? array_shift($parts) . '/' : '';
479
+            $location = 'file:///' . $drive . implode('/', array_map('rawurlencode', $parts));
480 480
 
481
-            $imports .= sprintf('  <xsd:import namespace="%s" schemaLocation="%s" />'."\n", $namespace, $location);
481
+            $imports .= sprintf('  <xsd:import namespace="%s" schemaLocation="%s" />' . "\n", $namespace, $location);
482 482
         }
483 483
 
484 484
         $source = <<<EOF
Please login to merge, or discard this patch.