pamil / Sylius

src/Sylius/Bundle/CustomerBundle/DependencyInjection/SyliusCustomerExtension.php

Spacing +1 added lines, -1 removed lines

@@ -28,7 +28,7 @@
      */
     public function load(array $config, ContainerBuilder $container)
     {
-        $loader = new XmlFileLoader($container, new FileLocator(__DIR__.'/../Resources/config'));
+        $loader = new XmlFileLoader($container, new FileLocator(__DIR__ . '/../Resources/config'));
         $loader->load('services.xml');
         $loader->load('twig.xml');
     }

tests/Controller/OauthTokenApiTest.php

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

@@ -35,7 +35,7 @@ 
             "username": "api@sylius.com",
             "password": "sylius"
         }
-EOT;
+eot;
 
         $this->client->request('POST', '/api/oauth/v2/token', [], [], ['CONTENT_TYPE' => 'application/json'], $data);
 
@@ -58,7 +58,7 @@ 
             "grant_type": "refresh_token",
             "refresh_token": "SampleRefreshTokenODllODY4ZTQyOThlNWIyMjA1ZDhmZjE1ZDYyMGMwOTUxOWM2NGFmNGRjNjQ2NDBhMDVlNGZjMmQ0YzgyNDM2Ng"
         }
-EOT;
+eot;
 
         $this->client->request('POST', '/api/oauth/v2/token', [], [], ['CONTENT_TYPE' => 'application/json'], $data);
 

tests/Controller/CheckoutAddressingApiTest.php

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

@@ -73,7 +73,7 @@ 
                 "email": "john@doe.com"
             }
         }
-EOT;
+eot;
 
         $url = sprintf('/api/checkouts/addressing/%d', $checkoutData['order1']->getId());
         $this->client->request('PUT', $url, [], [], static::$authorizedHeaderWithContentType, $data);
@@ -107,7 +107,7 @@ 
                 "email": "john@doe.com"
             }
         }
-EOT;
+eot;
 
         $url = sprintf('/api/checkouts/addressing/%d', $checkoutData['order1']->getId());
         $this->client->request('PUT', $url, [], [], static::$authorizedHeaderWithContentType, $data);
@@ -142,7 +142,7 @@ 
                 "email": "john@doe.com"
             }
         }
-EOT;
+eot;
 
         $url = sprintf('/api/checkouts/addressing/%d', $checkoutData['order1']->getId());
         $this->client->request('PUT', $url, [], [], static::$authorizedHeaderWithContentType, $data);
@@ -184,7 +184,7 @@ 
                 "email": "john@doe.com"
             }
         }
-EOT;
+eot;
 
         $url = sprintf('/api/checkouts/addressing/%d', $checkoutData['order1']->getId());
         $this->client->request('PUT', $url, [], [], static::$authorizedHeaderWithContentType, $data);
@@ -223,7 +223,7 @@ 
                 "email": "john@doe.com"
             }
         }
-EOT;
+eot;
 
         $orderId = $checkoutData['order1']->getId();
 
@@ -243,7 +243,7 @@ 
             },
             "differentBillingAddress": false
         }
-EOT;
+eot;
 
         $url = sprintf('/api/checkouts/addressing/%d', $checkoutData['order1']->getId());
         $this->client->request('PUT', $url, [], [], static::$authorizedHeaderWithContentType, $newData);
@@ -277,7 +277,7 @@ 
                 "email": "john@doe.com"
             }
         }
-EOT;
+eot;
 
         $orderId = $checkoutData['order1']->getId();
 
@@ -299,7 +299,7 @@ 
             },
             "differentBillingAddress": false
         }
-EOT;
+eot;
 
         $url = sprintf('/api/checkouts/addressing/%d', $checkoutData['order1']->getId());
         $this->client->request('PUT', $url, [], [], static::$authorizedHeaderWithContentType, $newAddressData);
@@ -333,7 +333,7 @@ 
                 "email": "john@doe.com"
             }
         }
-EOT;
+eot;
 
         $orderId = $checkoutData['order1']->getId();
 
@@ -356,7 +356,7 @@ 
             },
             "differentBillingAddress": false
         }
-EOT;
+eot;
 
         $url = sprintf('/api/checkouts/addressing/%d', $checkoutData['order1']->getId());
         $this->client->request('PUT', $url, [], [], static::$authorizedHeaderWithContentType, $newAddressData);

tests/Controller/CheckoutPaymentApiTest.php

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/CheckoutShippingApiTest.php

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

@@ -86,7 +86,7 @@ 
                 }
             ]
         }
-EOT;
+eot;
 
         $url = sprintf('/api/checkouts/select-shipping/%d', $orderId);
         $this->client->request('PUT', $url, [], [], static::$authorizedHeaderWithContentType, $data);
@@ -116,7 +116,7 @@ 
                 }
             ]
         }
-EOT;
+eot;
 
         $url = sprintf('/api/checkouts/select-shipping/%d', $orderId);
         $this->client->request('PUT', $url, [], [], static::$authorizedHeaderWithContentType, $data);
@@ -151,7 +151,7 @@ 
                 }
             ]
         }
-EOT;
+eot;
 
         $url = sprintf('/api/checkouts/select-shipping/%d', $orderId);
         $this->client->request('PUT', $url, [], [], static::$authorizedHeaderWithContentType, $data);
@@ -182,7 +182,7 @@ 
                 }
             ]
         }
-EOT;
+eot;
 
         $url = sprintf('/api/checkouts/select-shipping/%d', $orderId);
         $this->client->request('PUT', $url, [], [], static::$authorizedHeaderWithContentType, $data);

tests/Controller/ZoneApiTest.php

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();

Spacing +8 added lines, -8 removed lines

@@ -147,7 +147,7 @@ 
         $this->loadFixturesFromFile('authentication/api_administrator.yml');
         $zones = $this->loadFixturesFromFile('resources/zones.yml');
 
-        $this->client->request('GET', '/api/v1/zones/'.$zones['zone_eu']->getCode(), [], [], static::$authorizedHeaderWithAccept);
+        $this->client->request('GET', '/api/v1/zones/' . $zones['zone_eu']->getCode(), [], [], static::$authorizedHeaderWithAccept);
 
         $response = $this->client->getResponse();
         $this->assertResponse($response, 'zone/show_response', Response::HTTP_OK);
@@ -186,7 +186,7 @@ 
         $this->loadFixturesFromFile('resources/countries.yml');
         $zones = $this->loadFixturesFromFile('resources/zones.yml');
 
-        $this->client->request('PUT', '/api/v1/zones/'.$zones['zone_eu']->getCode(), [], [], static::$authorizedHeaderWithContentType);
+        $this->client->request('PUT', '/api/v1/zones/' . $zones['zone_eu']->getCode(), [], [], static::$authorizedHeaderWithContentType);
 
         $response = $this->client->getResponse();
         $this->assertResponse($response, 'zone/update_validation_fail_response', Response::HTTP_BAD_REQUEST);
@@ -214,12 +214,12 @@ 
         }
 EOT;
 
-        $this->client->request('PUT', '/api/v1/zones/'.$zones['zone_eu']->getCode(), [], [], static::$authorizedHeaderWithContentType, $data);
+        $this->client->request('PUT', '/api/v1/zones/' . $zones['zone_eu']->getCode(), [], [], static::$authorizedHeaderWithContentType, $data);
 
         $response = $this->client->getResponse();
         $this->assertResponseCode($response, Response::HTTP_NO_CONTENT);
 
-        $this->client->request('GET', '/api/v1/zones/'.$zones['zone_eu']->getCode(), [], [], static::$authorizedHeaderWithAccept);
+        $this->client->request('GET', '/api/v1/zones/' . $zones['zone_eu']->getCode(), [], [], static::$authorizedHeaderWithAccept);
 
         $response = $this->client->getResponse();
         $this->assertResponse($response, 'zone/update_response', Response::HTTP_OK);
@@ -265,12 +265,12 @@ 
         }
 EOT;
 
-        $this->client->request('PATCH', '/api/v1/zones/'.$zones['zone_eu']->getCode(), [], [], static::$authorizedHeaderWithContentType, $data);
+        $this->client->request('PATCH', '/api/v1/zones/' . $zones['zone_eu']->getCode(), [], [], static::$authorizedHeaderWithContentType, $data);
 
         $response = $this->client->getResponse();
         $this->assertResponseCode($response, Response::HTTP_NO_CONTENT);
 
-        $this->client->request('GET', '/api/v1/zones/'.$zones['zone_eu']->getCode(), [], [], static::$authorizedHeaderWithAccept);
+        $this->client->request('GET', '/api/v1/zones/' . $zones['zone_eu']->getCode(), [], [], static::$authorizedHeaderWithAccept);
 
         $response = $this->client->getResponse();
         $this->assertResponse($response, 'zone/update_response', Response::HTTP_OK);
@@ -308,12 +308,12 @@ 
         $this->loadFixturesFromFile('authentication/api_administrator.yml');
         $zones = $this->loadFixturesFromFile('resources/zones.yml');
 
-        $this->client->request('DELETE', '/api/v1/zones/'.$zones['zone_eu']->getCode(), [], [], static::$authorizedHeaderWithContentType);
+        $this->client->request('DELETE', '/api/v1/zones/' . $zones['zone_eu']->getCode(), [], [], static::$authorizedHeaderWithContentType);
 
         $response = $this->client->getResponse();
         $this->assertResponseCode($response, Response::HTTP_NO_CONTENT);
 
-        $this->client->request('GET', '/api/v1/zones/'.$zones['zone_eu']->getCode(), [], [], static::$authorizedHeaderWithAccept);
+        $this->client->request('GET', '/api/v1/zones/' . $zones['zone_eu']->getCode(), [], [], static::$authorizedHeaderWithAccept);
 
         $response = $this->client->getResponse();
         $this->assertResponseCode($response, Response::HTTP_NOT_FOUND);

tests/Controller/CheckoutApiTestCase.php

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();

src/Sylius/Behat/Page/Shop/Checkout/CompletePageInterface.php

Doc Comments +13 added lines

@@ -20,6 +20,7 @@ 
 {
     /**
      * @param string $paymentMethod
+     * @return void
      */
     public function selectPaymentMethod($paymentMethod);
     
@@ -37,12 +38,24 @@ 
      */
     public function getItemSubtotal($itemName);
 
+    /**
+     * @return void
+     */
     public function nextStep();
 
+    /**
+     * @return void
+     */
     public function changeShippingMethod();
 
+    /**
+     * @return void
+     */
     public function changeShippingMethodByStepLabel();
 
+    /**
+     * @return void
+     */
     public function changeAddressByStepLabel();
 
     /**

Unused Use Statements -1 removed lines

@@ -15,7 +15,6 @@
 use Sylius\Component\Core\Model\AddressInterface;
 use Sylius\Component\Core\Model\ProductInterface;
 use Sylius\Component\Core\Model\ShippingMethodInterface;
-use Sylius\Component\Payment\Model\PaymentMethodInterface;
 
 /**
  * @author Mateusz Zalewski <mateusz.zalewski@lakion.com>

src/Sylius/Component/Core/StateResolver/OrderPaymentStateResolver.php

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

@@ -13,11 +13,11 @@
 
 use SM\Factory\FactoryInterface;
 use SM\StateMachine\StateMachineInterface;
-use Sylius\Component\Order\Model\OrderInterface;
-use Sylius\Component\Order\StateResolver\StateResolverInterface;
 use Sylius\Component\Core\Model\ShipmentInterface;
 use Sylius\Component\Core\OrderShippingStates;
 use Sylius\Component\Core\OrderShippingTransitions;
+use Sylius\Component\Order\Model\OrderInterface;
+use Sylius\Component\Order\StateResolver\StateResolverInterface;
 
 /**
  * @author Paweł Jędrzejewski <pawel@sylius.org>