Merge pull request #307 from lchrusciel/cleanup

Codebase cleanup
Sylius · Sep 3, 2018 · 782d2d1 · 782d2d1
2 parents becb1d8 + 92f5031
commit 782d2d1
Show file tree

Hide file tree

Showing 162 changed files with 495 additions and 2,056 deletions.
diff --git a/src/Checker/PromotionCouponEligibilityChecker.php b/src/Checker/PromotionCouponEligibilityChecker.php
@@ -14,20 +14,12 @@
 
 final class PromotionCouponEligibilityChecker implements PromotionCouponEligibilityCheckerInterface
 {
-    /**
-     * @var PromotionEligibilityCheckerInterface
-     */
+    /** @var PromotionEligibilityCheckerInterface */
     private $promotionEligibilityChecker;
 
-    /**
-     * @var PromotionCouponEligibilityCheckerInterface
-     */
+    /** @var PromotionCouponEligibilityCheckerInterface */
     private $couponEligibilityChecker;
 
-    /**
-     * @param PromotionEligibilityCheckerInterface $promotionEligibilityChecker
-     * @param PromotionCouponEligibilityCheckerInterface $couponEligibilityChecker
-     */
     public function __construct(
         PromotionEligibilityCheckerInterface $promotionEligibilityChecker,
         PromotionCouponEligibilityCheckerInterface $couponEligibilityChecker
@@ -36,9 +28,7 @@ public function __construct(
         $this->couponEligibilityChecker = $couponEligibilityChecker;
     }
 
-    /**
-     * {@inheritdoc}
-     */
+    /** {@inheritdoc} */
     public function isEligible(PromotionSubjectInterface $cart, PromotionCouponInterface $coupon): bool
     {
         /** @var OrderInterface $cart */

diff --git a/src/Command/AddCoupon.php b/src/Command/AddCoupon.php
@@ -6,37 +6,23 @@
 
 final class AddCoupon
 {
-    /**
-     * @var string
-     */
+    /** @var string */
     private $orderToken;
 
-    /**
-     * @var string
-     */
+    /** @var string */
     private $couponCode;
 
-    /**
-     * @param string $orderToken
-     * @param string $couponCode
-     */
     public function __construct(string $orderToken, string $couponCode)
     {
         $this->orderToken = $orderToken;
         $this->couponCode = $couponCode;
     }
 
-    /**
-     * @return string
-     */
     public function orderToken(): string
     {
         return $this->orderToken;
     }
 
-    /**
-     * @return string
-     */
     public function couponCode(): string
     {
         return $this->couponCode;

diff --git a/src/Command/AddProductReviewByCode.php b/src/Command/AddProductReviewByCode.php
@@ -6,34 +6,22 @@
 
 final class AddProductReviewByCode
 {
-    /**
-     * @var string
-     */
+    /** @var string */
     private $productCode;
 
-    /**
-     * @var string
-     */
+    /** @var string */
     private $channelCode;
 
-    /**
-     * @var string
-     */
+    /** @var string */
     private $title;
 
-    /**
-     * @var int
-     */
+    /** @var int */
     private $rating;
 
-    /**
-     * @var string
-     */
+    /** @var string */
     private $comment;
 
-    /**
-     * @var string
-     */
+    /** @var string */
     private $email;
 
     public function __construct(string $productCode, string $channelCode, string $title, int $rating, string $comment, string $email)

diff --git a/src/Command/AddProductReviewBySlug.php b/src/Command/AddProductReviewBySlug.php
@@ -6,34 +6,22 @@
 
 final class AddProductReviewBySlug
 {
-    /**
-     * @var string
-     */
+    /** @var string */
     private $productSlug;
 
-    /**
-     * @var string
-     */
+    /** @var string */
     private $channelCode;
 
-    /**
-     * @var string
-     */
+    /** @var string */
     private $title;
 
-    /**
-     * @var int
-     */
+    /** @var int */
     private $rating;
 
-    /**
-     * @var string
-     */
+    /** @var string */
     private $comment;
 
-    /**
-     * @var string
-     */
+    /** @var string */
     private $email;
 
     public function __construct(string $productSlug, string $channelCode, string $title, int $rating, string $comment, string $email)

diff --git a/src/Command/AddressOrder.php b/src/Command/AddressOrder.php
@@ -8,52 +8,32 @@
 
 final class AddressOrder
 {
-    /**
-     * @var string
-     */
+    /** @var string */
     private $orderToken;
 
-    /**
-     * @var Address
-     */
+    /** @var Address */
     private $address;
 
-    /**
-     * @var Address
-     */
+    /** @var Address */
     private $billingAddress;
 
-    /**
-     * @param string $orderToken
-     * @param Address $shippingAddress
-     * @param Address $billingAddress
-     */
     public function __construct(string $orderToken, Address $shippingAddress, Address $billingAddress)
     {
         $this->orderToken = $orderToken;
         $this->address = $shippingAddress;
         $this->billingAddress = $billingAddress;
     }
 
-    /**
-     * @return string
-     */
     public function orderToken(): string
     {
         return $this->orderToken;
     }
 
-    /**
-     * @return Address
-     */
     public function shippingAddress(): Address
     {
         return $this->address;
     }
 
-    /**
-     * @return Address
-     */
     public function billingAddress(): Address
     {
         return $this->billingAddress;

diff --git a/src/Command/ChangeItemQuantity.php b/src/Command/ChangeItemQuantity.php
@@ -8,26 +8,15 @@
 
 final class ChangeItemQuantity
 {
-    /**
-     * @var string
-     */
+    /** @var string */
     private $orderToken;
 
-    /**
-     * @var mixed
-     */
+    /** @var mixed */
     private $itemIdentifier;
 
-    /**
-     * @var int
-     */
+    /** @var int */
     private $quantity;
 
-    /**
-     * @param string $orderToken
-     * @param mixed $itemIdentifier
-     * @param int $quantity
-     */
     public function __construct(string $orderToken, $itemIdentifier, int $quantity)
     {
         Assert::greaterThan($quantity, 0, 'Quantity should be greater than 0');
@@ -37,25 +26,16 @@ public function __construct(string $orderToken, $itemIdentifier, int $quantity)
         $this->quantity = $quantity;
     }
 
-    /**
-     * @return string
-     */
     public function orderToken(): string
     {
         return $this->orderToken;
     }
 
-    /**
-     * @return mixed
-     */
     public function itemIdentifier()
     {
         return $this->itemIdentifier;
     }
 
-    /**
-     * @return int
-     */
     public function quantity(): int
     {
         return $this->quantity;

diff --git a/src/Command/ChoosePaymentMethod.php b/src/Command/ChoosePaymentMethod.php
@@ -6,52 +6,32 @@
 
 final class ChoosePaymentMethod
 {
-    /**
-     * @var mixed
-     */
+    /** @var mixed */
     private $paymentIdentifier;
 
-    /**
-     * @var string
-     */
+    /** @var string */
     private $paymentMethod;
 
-    /**
-     * @var string
-     */
+    /** @var string */
     private $orderToken;
 
-    /**
-     * @param string $orderToken
-     * @param mixed $paymentIdentifier
-     * @param string $paymentMethod
-     */
     public function __construct(string $orderToken, $paymentIdentifier, string $paymentMethod)
     {
         $this->orderToken = $orderToken;
         $this->paymentIdentifier = $paymentIdentifier;
         $this->paymentMethod = $paymentMethod;
     }
 
-    /**
-     * @return string
-     */
     public function orderToken(): string
     {
         return $this->orderToken;
     }
 
-    /**
-     * @return mixed
-     */
     public function paymentIdentifier()
     {
         return $this->paymentIdentifier;
     }
 
-    /**
-     * @return string
-     */
     public function paymentMethod(): string
     {
         return $this->paymentMethod;

diff --git a/src/Command/ChooseShippingMethod.php b/src/Command/ChooseShippingMethod.php
@@ -6,52 +6,32 @@
 
 final class ChooseShippingMethod
 {
-    /**
-     * @var mixed
-     */
+    /** @var mixed */
     private $shipmentIdentifier;
 
-    /**
-     * @var string
-     */
+    /** @var string */
     private $shippingMethod;
 
-    /**
-     * @var string
-     */
+    /** @var string */
     private $orderToken;
 
-    /**
-     * @param string $orderToken
-     * @param mixed $shipmentIdentifier
-     * @param string $shippingMethod
-     */
     public function __construct(string $orderToken, $shipmentIdentifier, string $shippingMethod)
     {
         $this->orderToken = $orderToken;
         $this->shipmentIdentifier = $shipmentIdentifier;
         $this->shippingMethod = $shippingMethod;
     }
 
-    /**
-     * @return string
-     */
     public function orderToken(): string
     {
         return $this->orderToken;
     }
 
-    /**
-     * @return mixed
-     */
     public function shipmentIdentifier()
     {
         return $this->shipmentIdentifier;
     }
 
-    /**
-     * @return string
-     */
     public function shippingMethod(): string
     {
         return $this->shippingMethod;