Completed
Push — admin-api-bundle ( f175a5...04cc3d )
by Kamil
19:55
created
src/Sylius/Behat/Context/Ui/Shop/CheckoutContext.php 1 patch
Doc Comments   +2 added lines patch added patch discarded remove patch
@@ -130,6 +130,8 @@
 block discarded – undo
130 130
     /**
131 131
      * @Given I have proceeded order with :shippingMethodName shipping method and :paymentMethodName payment
132 132
      * @When I proceed with :shippingMethodName shipping method and :paymentMethodName payment
133
+     * @param string $shippingMethodName
134
+     * @param string $paymentMethodName
133 135
      */
134 136
     public function iProceedOrderWithShippingMethodAndPayment($shippingMethodName, $paymentMethodName)
135 137
     {
Please login to merge, or discard this patch.
src/Sylius/Behat/Context/Ui/Shop/Checkout/CheckoutCompleteContext.php 1 patch
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -100,7 +100,7 @@  discard block
 block discarded – undo
100 100
      */
101 101
     public function iShouldSeeThisShippingAddressAsShippingAddress($fullName)
102 102
     {
103
-        $address = $this->sharedStorage->get('shipping_address_'.StringInflector::nameToLowercaseCode($fullName));
103
+        $address = $this->sharedStorage->get('shipping_address_' . StringInflector::nameToLowercaseCode($fullName));
104 104
 
105 105
         Assert::true($this->completePage->hasShippingAddress($address));
106 106
     }
@@ -110,7 +110,7 @@  discard block
 block discarded – undo
110 110
      */
111 111
     public function iShouldSeeThisBillingAddressAsBillingAddress($fullName)
112 112
     {
113
-        $address = $this->sharedStorage->get('billing_address_'.StringInflector::nameToLowercaseCode($fullName));
113
+        $address = $this->sharedStorage->get('billing_address_' . StringInflector::nameToLowercaseCode($fullName));
114 114
 
115 115
         Assert::true($this->completePage->hasBillingAddress($address));
116 116
     }
Please login to merge, or discard this patch.
tests/Controller/LocaleApiTest.php 2 patches
Spacing   +3 added lines, -3 removed lines patch added patch discarded remove patch
@@ -71,7 +71,7 @@  discard block
 block discarded – undo
71 71
         $this->loadFixturesFromFile('authentication/api_administrator.yml');
72 72
         $locales = $this->loadFixturesFromFile('resources/locales.yml');
73 73
 
74
-        $this->client->request('GET', '/api/v1/locales/'.$locales['locale_en_US']->getCode(), [], [], static::$authorizedHeaderWithAccept);
74
+        $this->client->request('GET', '/api/v1/locales/' . $locales['locale_en_US']->getCode(), [], [], static::$authorizedHeaderWithAccept);
75 75
 
76 76
         $response = $this->client->getResponse();
77 77
         $this->assertResponse($response, 'locale/show_response', Response::HTTP_OK);
@@ -156,12 +156,12 @@  discard block
 block discarded – undo
156 156
         $this->loadFixturesFromFile('authentication/api_administrator.yml');
157 157
         $locales = $this->loadFixturesFromFile('resources/locales.yml');
158 158
 
159
-        $this->client->request('DELETE', '/api/v1/locales/'.$locales['locale_en_US']->getCode(), [], [], static::$authorizedHeaderWithContentType, []);
159
+        $this->client->request('DELETE', '/api/v1/locales/' . $locales['locale_en_US']->getCode(), [], [], static::$authorizedHeaderWithContentType, []);
160 160
 
161 161
         $response = $this->client->getResponse();
162 162
         $this->assertResponseCode($response, Response::HTTP_NO_CONTENT);
163 163
 
164
-        $this->client->request('GET', '/api/v1/locales/'.$locales['locale_en_US']->getId(), [], [], static::$authorizedHeaderWithAccept);
164
+        $this->client->request('GET', '/api/v1/locales/' . $locales['locale_en_US']->getId(), [], [], static::$authorizedHeaderWithAccept);
165 165
 
166 166
         $response = $this->client->getResponse();
167 167
         $this->assertResponse($response, 'error/not_found_response', Response::HTTP_NOT_FOUND);
Please login to merge, or discard this patch.
Upper-Lower-Casing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -102,7 +102,7 @@
 block discarded – undo
102 102
         {
103 103
             "code": "es_ES"
104 104
         }
105
-EOT;
105
+eot;
106 106
 
107 107
         $this->client->request('POST', '/api/v1/locales/', [], [], static::$authorizedHeaderWithContentType, $data);
108 108
 
Please login to merge, or discard this patch.
src/Sylius/Behat/Context/Ui/Shop/Checkout/CheckoutShippingContext.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -196,6 +196,6 @@
 block discarded – undo
196 196
      */
197 197
     public function iShouldBeCheckingOutAs($email)
198 198
     {
199
-        Assert::same($this->selectShippingPage->getPurchaserEmail(), 'Checking out as '.$email.'.');
199
+        Assert::same($this->selectShippingPage->getPurchaserEmail(), 'Checking out as ' . $email . '.');
200 200
     }
201 201
 }
Please login to merge, or discard this patch.
src/Sylius/Behat/Context/Setup/ProductContext.php 2 patches
Unused Use Statements   -2 removed lines patch added patch discarded remove patch
@@ -16,7 +16,6 @@  discard block
 block discarded – undo
16 16
 use Behat\Mink\Element\NodeElement;
17 17
 use Doctrine\Common\Persistence\ObjectManager;
18 18
 use Sylius\Behat\Service\SharedStorageInterface;
19
-use Sylius\Component\Attribute\Factory\AttributeFactoryInterface;
20 19
 use Sylius\Component\Core\Formatter\StringInflector;
21 20
 use Sylius\Component\Core\Model\ChannelInterface;
22 21
 use Sylius\Component\Core\Model\ChannelPricingInterface;
@@ -34,7 +33,6 @@  discard block
 block discarded – undo
34 33
 use Sylius\Component\Product\Resolver\ProductVariantResolverInterface;
35 34
 use Sylius\Component\Resource\Factory\FactoryInterface;
36 35
 use Sylius\Component\Resource\Model\TranslationInterface;
37
-use Sylius\Component\Resource\Repository\RepositoryInterface;
38 36
 use Sylius\Component\Shipping\Model\ShippingCategoryInterface;
39 37
 use Sylius\Component\Taxation\Model\TaxCategoryInterface;
40 38
 use Symfony\Component\HttpFoundation\File\UploadedFile;
Please login to merge, or discard this patch.
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -651,7 +651,7 @@
 block discarded – undo
651 651
 
652 652
         /** @var ImageInterface $productImage */
653 653
         $productImage = $this->productImageFactory->createNew();
654
-        $productImage->setFile(new UploadedFile($filesPath.$imagePath, basename($imagePath)));
654
+        $productImage->setFile(new UploadedFile($filesPath . $imagePath, basename($imagePath)));
655 655
         $productImage->setType($imageType);
656 656
         $this->imageUploader->upload($productImage);
657 657
 
Please login to merge, or discard this patch.
src/Sylius/Behat/Context/Ui/Shop/ProductContext.php 1 patch
Unused Use Statements   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -13,9 +13,9 @@
 block discarded – undo
13 13
 
14 14
 use Behat\Behat\Context\Context;
15 15
 use Behat\Mink\Element\NodeElement;
16
+use Sylius\Behat\Page\Shop\ProductReview\IndexPageInterface as ProductReviewIndexPageInterface;
16 17
 use Sylius\Behat\Page\Shop\Product\IndexPageInterface;
17 18
 use Sylius\Behat\Page\Shop\Product\ShowPageInterface;
18
-use Sylius\Behat\Page\Shop\ProductReview\IndexPageInterface as ProductReviewIndexPageInterface;
19 19
 use Sylius\Component\Core\Model\ProductInterface;
20 20
 use Sylius\Component\Core\Model\TaxonInterface;
21 21
 use Webmozart\Assert\Assert;
Please login to merge, or discard this patch.
src/Sylius/Bundle/CurrencyBundle/DependencyInjection/Configuration.php 1 patch
Unused Use Statements   -1 removed lines patch added patch discarded remove patch
@@ -11,7 +11,6 @@
 block discarded – undo
11 11
 
12 12
 namespace Sylius\Bundle\CurrencyBundle\DependencyInjection;
13 13
 
14
-use Sylius\Bundle\CurrencyBundle\Doctrine\ORM\ExchangeRateRepository;
15 14
 use Sylius\Bundle\CurrencyBundle\Form\Type\CurrencyType;
16 15
 use Sylius\Bundle\CurrencyBundle\Form\Type\ExchangeRateType;
17 16
 use Sylius\Bundle\ResourceBundle\Controller\ResourceController;
Please login to merge, or discard this patch.
src/Sylius/Bundle/ResourceBundle/Controller/RequestConfiguration.php 2 patches
Doc Comments   +4 added lines, -4 removed lines patch added patch discarded remove patch
@@ -171,7 +171,7 @@  discard block
 block discarded – undo
171 171
     /**
172 172
      * @param $name
173 173
      *
174
-     * @return mixed|null|string
174
+     * @return string
175 175
      */
176 176
     public function getRedirectRoute($name)
177 177
     {
@@ -231,7 +231,7 @@  discard block
 block discarded – undo
231 231
     }
232 232
 
233 233
     /**
234
-     * @param object|null $resource
234
+     * @param \Sylius\Component\Resource\Model\ResourceInterface|null $resource
235 235
      *
236 236
      * @return array
237 237
      */
@@ -350,7 +350,7 @@  discard block
 block discarded – undo
350 350
     }
351 351
 
352 352
     /**
353
-     * @param $parameter
353
+     * @param string $parameter
354 354
      * @param array $defaults
355 355
      *
356 356
      * @return array
@@ -454,7 +454,7 @@  discard block
 block discarded – undo
454 454
     }
455 455
 
456 456
     /**
457
-     * @return mixed|null
457
+     * @return string
458 458
      */
459 459
     public function getSerializationVersion()
460 460
     {
Please login to merge, or discard this patch.
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -163,7 +163,7 @@  discard block
 block discarded – undo
163 163
      */
164 164
     public function getRouteName($name)
165 165
     {
166
-        $sectionPrefix = $this->getSection() ? $this->getSection().'_' : '';
166
+        $sectionPrefix = $this->getSection() ? $this->getSection() . '_' : '';
167 167
 
168 168
         return sprintf('%s_%s%s_%s', $this->metadata->getApplicationName(), $sectionPrefix, $this->metadata->getName(), $name);
169 169
     }
@@ -205,7 +205,7 @@  discard block
 block discarded – undo
205 205
             return '';
206 206
         }
207 207
 
208
-        return '#'.$redirect['hash'];
208
+        return '#' . $redirect['hash'];
209 209
     }
210 210
 
211 211
     /**
Please login to merge, or discard this patch.
src/Sylius/Bundle/ReviewBundle/DependencyInjection/Configuration.php 1 patch
Unused Use Statements   +1 added lines, -2 removed lines patch added patch discarded remove patch
@@ -12,12 +12,11 @@
 block discarded – undo
12 12
 namespace Sylius\Bundle\ReviewBundle\DependencyInjection;
13 13
 
14 14
 use Sylius\Bundle\ResourceBundle\Controller\ResourceController;
15
-use Sylius\Bundle\ResourceBundle\Doctrine\ORM\EntityRepository;
16 15
 use Sylius\Bundle\ResourceBundle\SyliusResourceBundle;
17 16
 use Sylius\Bundle\ReviewBundle\Form\Type\ReviewType;
18 17
 use Sylius\Component\Resource\Factory\Factory;
19
-use Sylius\Component\Review\Model\ReviewerInterface;
20 18
 use Sylius\Component\Review\Model\ReviewInterface;
19
+use Sylius\Component\Review\Model\ReviewerInterface;
21 20
 use Symfony\Component\Config\Definition\Builder\ArrayNodeDefinition;
22 21
 use Symfony\Component\Config\Definition\Builder\TreeBuilder;
23 22
 use Symfony\Component\Config\Definition\ConfigurationInterface;
Please login to merge, or discard this patch.