pamil / Sylius

tests/Controller/ShippingCategoryApiTest.php

Spacing +8 added lines, -8 removed lines

@@ -109,7 +109,7 @@ 
         $this->loadFixturesFromFile('authentication/api_administrator.yml');
         $shippingCategories = $this->loadFixturesFromFile('resources/shipping_categories.yml');
 
-        $this->client->request('GET', '/api/shipping-categories/'.$shippingCategories['shipping_category_1']->getId(), [], [], [
+        $this->client->request('GET', '/api/shipping-categories/' . $shippingCategories['shipping_category_1']->getId(), [], [], [
             'HTTP_Authorization' => 'Bearer SampleTokenNjZkNjY2MDEwMTAzMDkxMGE0OTlhYzU3NzYyMTE0ZGQ3ODcyMDAwM2EwMDZjNDI5NDlhMDdlMQ',
             'ACCEPT' => 'application/json',
         ]);
@@ -144,7 +144,7 @@ 
         $this->loadFixturesFromFile('authentication/api_administrator.yml');
         $shippingCategories = $this->loadFixturesFromFile('resources/shipping_categories.yml');
 
-        $this->client->request('PUT', '/api/shipping-categories/'.$shippingCategories['shipping_category_1']->getId(), [], [], [
+        $this->client->request('PUT', '/api/shipping-categories/' . $shippingCategories['shipping_category_1']->getId(), [], [], [
             'HTTP_Authorization' => 'Bearer SampleTokenNjZkNjY2MDEwMTAzMDkxMGE0OTlhYzU3NzYyMTE0ZGQ3ODcyMDAwM2EwMDZjNDI5NDlhMDdlMQ',
             'CONTENT_TYPE' => 'application/json',
         ]);
@@ -166,7 +166,7 @@ 
         }
 EOT;
 
-        $this->client->request('PUT', '/api/shipping-categories/'.$shippingCategories['shipping_category_1']->getId(), [], [], [
+        $this->client->request('PUT', '/api/shipping-categories/' . $shippingCategories['shipping_category_1']->getId(), [], [], [
             'HTTP_Authorization' => 'Bearer SampleTokenNjZkNjY2MDEwMTAzMDkxMGE0OTlhYzU3NzYyMTE0ZGQ3ODcyMDAwM2EwMDZjNDI5NDlhMDdlMQ',
             'CONTENT_TYPE' => 'application/json',
         ], $data);
@@ -174,7 +174,7 @@ 
         $response = $this->client->getResponse();
         $this->assertResponseCode($response, Response::HTTP_NO_CONTENT);
 
-        $this->client->request('GET', '/api/shipping-categories/'.$shippingCategories['shipping_category_1']->getId(), [], [], [
+        $this->client->request('GET', '/api/shipping-categories/' . $shippingCategories['shipping_category_1']->getId(), [], [], [
             'HTTP_Authorization' => 'Bearer SampleTokenNjZkNjY2MDEwMTAzMDkxMGE0OTlhYzU3NzYyMTE0ZGQ3ODcyMDAwM2EwMDZjNDI5NDlhMDdlMQ',
             'ACCEPT' => 'application/json',
         ]);
@@ -209,7 +209,7 @@ 
         }
 EOT;
 
-        $this->client->request('PATCH', '/api/shipping-categories/'.$shippingCategories['shipping_category_1']->getId(), [], [], [
+        $this->client->request('PATCH', '/api/shipping-categories/' . $shippingCategories['shipping_category_1']->getId(), [], [], [
             'HTTP_Authorization' => 'Bearer SampleTokenNjZkNjY2MDEwMTAzMDkxMGE0OTlhYzU3NzYyMTE0ZGQ3ODcyMDAwM2EwMDZjNDI5NDlhMDdlMQ',
             'CONTENT_TYPE' => 'application/json',
         ], $data);
@@ -217,7 +217,7 @@ 
         $response = $this->client->getResponse();
         $this->assertResponseCode($response, Response::HTTP_NO_CONTENT);
 
-        $this->client->request('GET', '/api/shipping-categories/'.$shippingCategories['shipping_category_1']->getId(), [], [], [
+        $this->client->request('GET', '/api/shipping-categories/' . $shippingCategories['shipping_category_1']->getId(), [], [], [
             'HTTP_Authorization' => 'Bearer SampleTokenNjZkNjY2MDEwMTAzMDkxMGE0OTlhYzU3NzYyMTE0ZGQ3ODcyMDAwM2EwMDZjNDI5NDlhMDdlMQ',
             'ACCEPT' => 'application/json',
         ]);
@@ -244,7 +244,7 @@ 
         $this->loadFixturesFromFile('authentication/api_administrator.yml');
         $shippingCategories = $this->loadFixturesFromFile('resources/shipping_categories.yml');
 
-        $this->client->request('DELETE', '/api/shipping-categories/'.$shippingCategories['shipping_category_1']->getId(), [], [], [
+        $this->client->request('DELETE', '/api/shipping-categories/' . $shippingCategories['shipping_category_1']->getId(), [], [], [
             'HTTP_Authorization' => 'Bearer SampleTokenNjZkNjY2MDEwMTAzMDkxMGE0OTlhYzU3NzYyMTE0ZGQ3ODcyMDAwM2EwMDZjNDI5NDlhMDdlMQ',
             'CONTENT_TYPE' => 'application/json',
         ], []);
@@ -252,7 +252,7 @@ 
         $response = $this->client->getResponse();
         $this->assertResponseCode($response, Response::HTTP_NO_CONTENT);
 
-        $this->client->request('GET', '/api/shipping-categories/'.$shippingCategories['shipping_category_1']->getId(), [], [], [
+        $this->client->request('GET', '/api/shipping-categories/' . $shippingCategories['shipping_category_1']->getId(), [], [], [
             'HTTP_Authorization' => 'Bearer SampleTokenNjZkNjY2MDEwMTAzMDkxMGE0OTlhYzU3NzYyMTE0ZGQ3ODcyMDAwM2EwMDZjNDI5NDlhMDdlMQ',
             'ACCEPT' => 'application/json',
         ]);

Upper-Lower-Casing +3 added lines, -3 removed lines

@@ -34,7 +34,7 @@ 
             },
             "author": "Christie Golden"
         }
-EOT;
+eot;
 
         $this->client->request('POST', '/books/', [], [], ['CONTENT_TYPE' => 'application/json'], $data);
         $response = $this->client->getResponse();
@@ -61,7 +61,7 @@ 
             },
             "author": "Christie Golden"
         }
-EOT;
+eot;
 
         $this->client->request('PUT', '/books/'. $objects["book1"]->getId(), [], [], ['CONTENT_TYPE' => 'application/json'], $data);
         $response = $this->client->getResponse();
@@ -80,7 +80,7 @@ 
         {
             "author": "Christie Golden"
         }
-EOT;
+eot;
 
         $this->client->request('PATCH', '/books/'. $objects["book1"]->getId(), [], [], ['CONTENT_TYPE' => 'application/json'], $data);
         $response = $this->client->getResponse();

tests/Controller/CustomerApiTest.php

Spacing +9 added lines, -9 removed lines

@@ -187,7 +187,7 @@ 
         $this->loadFixturesFromFile('authentication/api_administrator.yml');
         $customers = $this->loadFixturesFromFile('resources/customers.yml');
 
-        $this->client->request('GET', '/api/customers/'.$customers['customer_Barry']->getId(), [], [], static::$authorizedHeaderWithAccept);
+        $this->client->request('GET', '/api/customers/' . $customers['customer_Barry']->getId(), [], [], static::$authorizedHeaderWithAccept);
 
         $response = $this->client->getResponse();
         $this->assertResponse($response, 'customer/show_response', Response::HTTP_OK);
@@ -201,7 +201,7 @@ 
         $this->loadFixturesFromFile('authentication/api_administrator.yml');
         $customers = $this->loadFixturesFromFile('resources/customers.yml');
 
-        $this->client->request('GET', '/api/customers/'.$customers['customer_Roy']->getId(), [], [], static::$authorizedHeaderWithAccept);
+        $this->client->request('GET', '/api/customers/' . $customers['customer_Roy']->getId(), [], [], static::$authorizedHeaderWithAccept);
 
         $response = $this->client->getResponse();
         $this->assertResponse($response, 'customer/show_with_user_response', Response::HTTP_OK);
@@ -239,7 +239,7 @@ 
         $this->loadFixturesFromFile('authentication/api_administrator.yml');
         $customers = $this->loadFixturesFromFile('resources/customers.yml');
 
-        $this->client->request('PUT', '/api/customers/'.$customers['customer_Oliver']->getId(), [], [], static::$authorizedHeaderWithContentType);
+        $this->client->request('PUT', '/api/customers/' . $customers['customer_Oliver']->getId(), [], [], static::$authorizedHeaderWithContentType);
 
         $response = $this->client->getResponse();
         $this->assertResponse($response, 'customer/update_validation_fail_response', Response::HTTP_BAD_REQUEST);
@@ -263,12 +263,12 @@ 
         }
 EOT;
 
-        $this->client->request('PUT', '/api/customers/'.$customers['customer_Oliver']->getId(), [], [], static::$authorizedHeaderWithContentType, $data);
+        $this->client->request('PUT', '/api/customers/' . $customers['customer_Oliver']->getId(), [], [], static::$authorizedHeaderWithContentType, $data);
 
         $response = $this->client->getResponse();
         $this->assertResponseCode($response, Response::HTTP_NO_CONTENT);
 
-        $this->client->request('GET', '/api/customers/'.$customers['customer_Oliver']->getId(), [], [], static::$authorizedHeaderWithAccept);
+        $this->client->request('GET', '/api/customers/' . $customers['customer_Oliver']->getId(), [], [], static::$authorizedHeaderWithAccept);
 
         $response = $this->client->getResponse();
         $this->assertResponse($response, 'customer/update_response', Response::HTTP_OK);
@@ -303,12 +303,12 @@ 
         }
 EOT;
 
-        $this->client->request('PATCH', '/api/customers/'.$customers['customer_Oliver']->getId(), [], [], static::$authorizedHeaderWithContentType, $data);
+        $this->client->request('PATCH', '/api/customers/' . $customers['customer_Oliver']->getId(), [], [], static::$authorizedHeaderWithContentType, $data);
 
         $response = $this->client->getResponse();
         $this->assertResponseCode($response, Response::HTTP_NO_CONTENT);
 
-        $this->client->request('GET', '/api/customers/'.$customers['customer_Oliver']->getId(), [], [], static::$authorizedHeaderWithAccept);
+        $this->client->request('GET', '/api/customers/' . $customers['customer_Oliver']->getId(), [], [], static::$authorizedHeaderWithAccept);
 
         $response = $this->client->getResponse();
         $this->assertResponse($response, 'customer/partial_update_response', Response::HTTP_OK);
@@ -346,12 +346,12 @@ 
         $this->loadFixturesFromFile('authentication/api_administrator.yml');
         $customers = $this->loadFixturesFromFile('resources/customers.yml');
 
-        $this->client->request('DELETE', '/api/customers/'.$customers['customer_Oliver']->getId(), [], [], static::$authorizedHeaderWithContentType);
+        $this->client->request('DELETE', '/api/customers/' . $customers['customer_Oliver']->getId(), [], [], static::$authorizedHeaderWithContentType);
 
         $response = $this->client->getResponse();
         $this->assertResponseCode($response, Response::HTTP_NO_CONTENT);
 
-        $this->client->request('GET', '/api/customers/'.$customers['customer_Oliver']->getId(), [], [], static::$authorizedHeaderWithAccept);
+        $this->client->request('GET', '/api/customers/' . $customers['customer_Oliver']->getId(), [], [], static::$authorizedHeaderWithAccept);
 
         $response = $this->client->getResponse();
         $this->assertResponse($response, 'error/not_found_response', Response::HTTP_NOT_FOUND);

Upper-Lower-Casing +5 added lines, -5 removed lines

@@ -73,7 +73,7 @@ 
                 "enabled": "true"
             }
         }
-EOT;
+eot;
 
         $this->client->request('POST', '/api/customers/', [], [], static::$authorizedHeaderWithContentType, $data);
 
@@ -96,7 +96,7 @@ 
             "email": "john.diggle@yahoo.com",
             "gender": "m"
         }
-EOT;
+eot;
 
         $this->client->request('POST', '/api/customers/', [], [], static::$authorizedHeaderWithContentType, $data);
 
@@ -122,7 +122,7 @@ 
                 "plainPassword" : "testPassword"
             }
         }
-EOT;
+eot;
 
         $this->client->request('POST', '/api/customers/', [], [], static::$authorizedHeaderWithContentType, $data);
 
@@ -261,7 +261,7 @@ 
             "email": "john.diggle@example.com",
             "gender": "m"
         }
-EOT;
+eot;
 
         $this->client->request('PUT', '/api/customers/'.$customers['customer_Oliver']->getId(), [], [], static::$authorizedHeaderWithContentType, $data);
 
@@ -301,7 +301,7 @@ 
             "firstName": "John",
             "lastName": "Doe"
         }
-EOT;
+eot;
 
         $this->client->request('PATCH', '/api/customers/'.$customers['customer_Oliver']->getId(), [], [], static::$authorizedHeaderWithContentType, $data);
 

src/Sylius/Behat/Context/Setup/TaxationContext.php

Spacing +1 added lines, -1 removed lines

@@ -212,6 +212,6 @@
      */
     private function getCodeFromNameAndZoneCode($taxRateName, $zoneCode)
     {
-        return $this->getCodeFromName($taxRateName).'_'.strtolower($zoneCode);
+        return $this->getCodeFromName($taxRateName) . '_' . strtolower($zoneCode);
     }
 }

Doc Comments +1 added lines, -1 removed lines

@@ -67,7 +67,7 @@
      *
      * @throws \RuntimeException
      *
-     * @return string
+     * @return integer
      */
     private function tryToMapPaymentMethodName($paymentMethodName)
     {

Unused Use Statements +1 added lines, -1 removed lines

@@ -15,9 +15,9 @@
 use Sylius\Behat\Page\Admin\Channel\IndexPageInterface;
 use Sylius\Behat\Page\Admin\Channel\UpdatePageInterface;
 use Sylius\Behat\Page\Shop\HomePageInterface;
+use Sylius\Behat\Service\SharedStorageInterface;
 use Sylius\Bundle\ThemeBundle\Model\ThemeInterface;
 use Sylius\Component\Core\Model\ChannelInterface;
-use Sylius\Behat\Service\SharedStorageInterface;
 use Webmozart\Assert\Assert;
 
 /**

src/Sylius/Behat/Context/Ui/CheckoutContext.php

Spacing +3 added lines, -3 removed lines

@@ -530,7 +530,7 @@ 
      */
     public function iShouldSeeThisShippingAddressAsShippingAddress($fullName)
     {
-        $address = $this->sharedStorage->get('shipping_address_'.StringInflector::nameToLowercaseCode($fullName));
+        $address = $this->sharedStorage->get('shipping_address_' . StringInflector::nameToLowercaseCode($fullName));
         Assert::true(
             $this->summaryPage->hasShippingAddress($address),
             'Shipping address is improper.'
@@ -542,7 +542,7 @@ 
      */
     public function iShouldSeeThisBillingAddressAsBillingAddress($fullName)
     {
-        $address = $this->sharedStorage->get('billing_address_'.StringInflector::nameToLowercaseCode($fullName));
+        $address = $this->sharedStorage->get('billing_address_' . StringInflector::nameToLowercaseCode($fullName));
         Assert::true(
             $this->summaryPage->hasBillingAddress($address),
             'Billing address is improper.'
@@ -622,7 +622,7 @@ 
     {
         Assert::true(
             $this->summaryPage->hasShippingTotal($price),
-            sprintf('The shipping total should be %s, but it is not.',$price)
+            sprintf('The shipping total should be %s, but it is not.', $price)
         );
     }
 

Indentation +1 added lines, -1 removed lines

@@ -1261,7 +1261,7 @@
     {
         Assert::true(
             $this->selectPaymentPage->isNextStepButtonUnavailable(),
-           'The "next step" button should be disabled.'
+            'The "next step" button should be disabled.'
         );
     }
 

src/Sylius/Behat/Context/Cli/InstallerContext.php

Spacing +2 added lines, -2 removed lines

@@ -200,7 +200,7 @@ 
     {
         $this->dialog = $this->command->getHelper('dialog');
         $inputString = join(PHP_EOL, $this->inputChoices);
-        $this->dialog->setInputStream($this->getInputStream($inputString.PHP_EOL));
+        $this->dialog->setInputStream($this->getInputStream($inputString . PHP_EOL));
 
         $this->tester->execute(['command' => $name]);
     }
@@ -211,7 +211,7 @@ 
     private function iExecuteCommandAndConfirm($name)
     {
         $this->dialog = $this->command->getHelper('dialog');
-        $inputString = 'y'.PHP_EOL;
+        $inputString = 'y' . PHP_EOL;
         $this->dialog->setInputStream($this->getInputStream($inputString));
 
         $this->tester->execute(['command' => $name]);

src/Sylius/Behat/Page/Shop/Account/Order/ShowPage.php

Spacing +1 added lines, -1 removed lines

@@ -159,7 +159,7 @@
             (stripos($elementText, $customerName) !== false) &&
             (stripos($elementText, $street) !== false) &&
             (stripos($elementText, $city) !== false) &&
-            (stripos($elementText, $countryName.' '.$postcode) !== false)
+            (stripos($elementText, $countryName . ' ' . $postcode) !== false)
         ;
     }
 }

src/Sylius/Behat/Page/Admin/Country/UpdatePage.php

Spacing +3 added lines, -3 removed lines

@@ -40,7 +40,7 @@ 
     {
         $provinces = $this->getElement('provinces');
 
-        return $provinces->has('css', '[value = "'.$provinceName.'"]');
+        return $provinces->has('css', '[value = "' . $provinceName . '"]');
     }
 
     /**
@@ -50,7 +50,7 @@ 
     {
         $provinces = $this->getElement('provinces');
 
-        return $provinces->has('css', '[value = "'.$provinceCode.'"]');
+        return $provinces->has('css', '[value = "' . $provinceCode . '"]');
     }
 
     /**
@@ -137,7 +137,7 @@ 
         if ($this->isThereProvince($provinceName)) {
             $provinces = $this->getElement('provinces');
 
-            $item = $provinces->find('css', 'div[data-form-collection="item"] input[value="'.$provinceName.'"]')->getParent();
+            $item = $provinces->find('css', 'div[data-form-collection="item"] input[value="' . $provinceName . '"]')->getParent();
             $item->fillField('Name', '');
         }
     }

src/Sylius/Behat/Page/Admin/Order/ShowPage.php

Spacing +2 added lines, -2 removed lines

@@ -385,7 +385,7 @@ 
             (stripos($elementText, $customerName) !== false) &&
             (stripos($elementText, $street) !== false) &&
             (stripos($elementText, $city) !== false) &&
-            (stripos($elementText, $countryName.' '.$postcode) !== false)
+            (stripos($elementText, $countryName . ' ' . $postcode) !== false)
         ;
     }
 
@@ -402,7 +402,7 @@ 
             ['item' => $itemName]
         );
 
-        return $rows[0]->find('css', '.'.$property)->getText();
+        return $rows[0]->find('css', '.' . $property)->getText();
     }
 
     /**

src/Sylius/Bundle/UiBundle/Twig/PercentageExtension.php

Spacing +1 added lines, -1 removed lines

@@ -35,7 +35,7 @@
     {
         $percentage = $number * 100;
 
-        return $percentage.' %';
+        return $percentage . ' %';
     }
 
     /**