darkorsa / shop

src/CartPresenter.php

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

@@ -11,15 +11,15 @@
 
 namespace Plane\Shop;
 
+use Money\Currencies\ISOCurrencies;
+use Money\Formatter\DecimalMoneyFormatter;
 use Money\Money;
 use Money\MoneyFormatter;
+use Plane\Shop\CartDiscountInterface;
 use Plane\Shop\CartInterface;
 use Plane\Shop\CartItemInterface;
-use Plane\Shop\Traits\CartCommon;
-use Money\Currencies\ISOCurrencies;
-use Plane\Shop\CartDiscountInterface;
 use Plane\Shop\CartPresenterInterface;
-use Money\Formatter\DecimalMoneyFormatter;
+use Plane\Shop\Traits\CartCommon;
 
 class CartPresenter implements CartPresenterInterface
 {

Spacing +3 added lines, -3 removed lines

@@ -68,10 +68,10 @@ 
     public function toArray(): array
     {
         $array = [];
-        $array['items'] = array_map(function (CartItemInterface $item) {
+        $array['items'] = array_map(function(CartItemInterface $item) {
             $itemArray = $item->toArray($this->getCurrency());
 
-            array_walk_recursive($itemArray, function (&$value, $key) {
+            array_walk_recursive($itemArray, function(&$value, $key) {
                 if ($value instanceof Money) {
                     $value = $this->priceFormatter->format($value);
                 }
@@ -96,7 +96,7 @@ 
             $array['payment']['fee'] = $this->paymentFee();
         }
 
-        $array['discounts'] = array_map(function (CartDiscountInterface $discount) {
+        $array['discounts'] = array_map(function(CartDiscountInterface $discount) {
             return [
                 'text'  => $discount->getDescription(),
                 'price' => $this->priceFormatter->format($discount->getPriceAfterDiscount())

src/Traits/CartCommon.php

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

@@ -11,11 +11,11 @@
 
 namespace Plane\Shop\Traits;
 
-use Plane\Shop\PaymentInterface;
+use Plane\Shop\CartDiscountInterface;
+use Plane\Shop\CartItemCollection;
 use Plane\Shop\CartItemInterface;
+use Plane\Shop\PaymentInterface;
 use Plane\Shop\ShippingInterface;
-use Plane\Shop\CartItemCollection;
-use Plane\Shop\CartDiscountInterface;
 
 trait CartCommon
 {

tests/CartTrait.php

Doc Comments +6 added lines

@@ -44,6 +44,9 @@ 
        $this->secondCartItem = new CartItem($product, 2);
     }
 
+    /**
+     * @return \Plane\Shop\PaymentInterface
+     */
     protected function getPaymentMock()
     {
         $payment = $this->createMock(Payment::class);
@@ -55,6 +58,9 @@ 
         return $payment;
     }
 
+    /**
+     * @return \Plane\Shop\ShippingInterface
+     */
     protected function getShippingMock()
     {
         $shipping  = $this->createMock(Shipping::class);

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

@@ -3,9 +3,9 @@
 namespace Plane\Shop\Tests;
 
 use Plane\Shop\Cart;
+use Plane\Shop\CartItem;
 use Plane\Shop\Payment;
 use Plane\Shop\Product;
-use Plane\Shop\CartItem;
 use Plane\Shop\Shipping;
 
 trait CartTrait

Spacing +1 added lines, -1 removed lines

@@ -57,7 +57,7 @@
 
     protected function getShippingMock()
     {
-        $shipping  = $this->createMock(Shipping::class);
+        $shipping = $this->createMock(Shipping::class);
         $shipping->method('getId')->willReturn(1);
         $shipping->method('getName')->willReturn('Test shipping');
         $shipping->method('getDescription')->willReturn('Shipping description');

tests/PaymentTest.php

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

@@ -2,9 +2,9 @@
 
 namespace Plane\Shop\Tests;
 
+use InvalidArgumentException;
 use Money\Money;
 use Plane\Shop\Payment;
-use InvalidArgumentException;
 
 /**
  * Payment test suite

tests/ProductTest.php

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

@@ -2,8 +2,8 @@
 
 namespace Plane\Shop\Tests;
 
-use Plane\Shop\Product;
 use InvalidArgumentException;
+use Plane\Shop\Product;
 
 /**
  * Product test suite

Spacing +8 added lines, -8 removed lines

@@ -72,15 +72,15 @@
 
         $array = $product->toArray(self::CURRENCY);
         
-        $this->assertSame(self::PRODUCT_OUTPUT['id'],           $array['id']);
-        $this->assertSame(self::PRODUCT_OUTPUT['name'],         $array['name']);
-        $this->assertSame(self::PRODUCT_OUTPUT['price'],        $this->getAmount($array['price']));
-        $this->assertSame(self::PRODUCT_OUTPUT['stock'],        $array['stock']);
-        $this->assertSame(self::PRODUCT_OUTPUT['taxRate'],      $array['taxRate']);
-        $this->assertSame(self::PRODUCT_OUTPUT['tax'],          $this->getAmount($array['tax']));
+        $this->assertSame(self::PRODUCT_OUTPUT['id'], $array['id']);
+        $this->assertSame(self::PRODUCT_OUTPUT['name'], $array['name']);
+        $this->assertSame(self::PRODUCT_OUTPUT['price'], $this->getAmount($array['price']));
+        $this->assertSame(self::PRODUCT_OUTPUT['stock'], $array['stock']);
+        $this->assertSame(self::PRODUCT_OUTPUT['taxRate'], $array['taxRate']);
+        $this->assertSame(self::PRODUCT_OUTPUT['tax'], $this->getAmount($array['tax']));
         $this->assertSame(self::PRODUCT_OUTPUT['priceWithTax'], $this->getAmount($array['priceWithTax']));
-        $this->assertSame(self::PRODUCT_OUTPUT['weight'],       $array['weight']);
-        $this->assertSame(self::PRODUCT_OUTPUT['imagePath'],    $array['imagePath']);
+        $this->assertSame(self::PRODUCT_OUTPUT['weight'], $array['weight']);
+        $this->assertSame(self::PRODUCT_OUTPUT['imagePath'], $array['imagePath']);
         
     }
 }

tests/ShippingTest.php

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

@@ -2,9 +2,9 @@
 
 namespace Plane\Shop\Tests;
 
+use InvalidArgumentException;
 use Money\Money;
 use Plane\Shop\Shipping;
-use InvalidArgumentException;
 
 /**
  * Shipping test suite

Spacing +1 added lines, -1 removed lines

@@ -39,7 +39,7 @@
 
     public function testCreateIncompleteObject()
     {
-        $input =self::SHIPPING_INPUT;
+        $input = self::SHIPPING_INPUT;
         unset($input['cost']);
 
         $this->expectException(InvalidArgumentException::class);

src/Cart.php

Spacing +6 added lines, -6 removed lines

@@ -74,7 +74,7 @@ 
     
     public function fill(CartItemCollection $collection): void
     {
-        array_map(function ($item) {
+        array_map(function($item) {
             $this->addItem($item);
         }, $collection->getItems());
     }
@@ -131,7 +131,7 @@ 
     public function itemsQuantity(): int
     {
         return (int) array_sum(
-            array_map(function (CartItemInterface $item) {
+            array_map(function(CartItemInterface $item) {
                 return $item->getQuantity();
             }, $this->items)
         );
@@ -140,7 +140,7 @@ 
     public function weight(): string
     {
         return (string) array_sum(
-            array_map(function (CartItemInterface $item) {
+            array_map(function(CartItemInterface $item) {
                 return $item->getWeightTotal();
             }, $this->items)
         );
@@ -148,7 +148,7 @@ 
 
     public function totalNet(): Money
     {
-        $total = array_map(function (CartItemInterface $item) {
+        $total = array_map(function(CartItemInterface $item) {
             return $item->getPriceTotal($this->currency);
         }, $this->items);
         
@@ -157,7 +157,7 @@ 
 
     public function totalGross(): Money
     {
-        $total = array_map(function (CartItemInterface $item) {
+        $total = array_map(function(CartItemInterface $item) {
             return $item->getPriceTotalWithTax($this->currency);
         }, $this->items);
         
@@ -166,7 +166,7 @@ 
     
     public function tax(): Money
     {
-        $total = array_map(function (CartItemInterface $item) {
+        $total = array_map(function(CartItemInterface $item) {
             return $item->getTaxTotal($this->currency);
         }, $this->items);
 

src/Product.php

Spacing +1 added lines, -1 removed lines

@@ -48,7 +48,7 @@
     {
         if (count(array_intersect_key(array_flip($this->requiredFields), $data)) !== count($this->requiredFields)) {
             throw new InvalidArgumentException(
-                'Cannot create object, required array keys: '. implode(', ', $this->requiredFields)
+                'Cannot create object, required array keys: ' . implode(', ', $this->requiredFields)
             );
         }
 

src/Shipping.php

Spacing +1 added lines, -1 removed lines

@@ -48,7 +48,7 @@
     {
         if (count(array_intersect_key(array_flip($this->requiredFields), $data)) !== count($this->requiredFields)) {
             throw new InvalidArgumentException(
-                'Cannot create object, required array keys: '. implode(', ', $this->requiredFields)
+                'Cannot create object, required array keys: ' . implode(', ', $this->requiredFields)
             );
         }