Merge pull request #2680 from coreshop/coding-standard/refactor-4.0

[CS] Refactor
coreshop · Aug 12, 2024 · f26277d · f26277d
2 parents f4766cb + c43e3ae
commit f26277d
Show file tree

Hide file tree

Showing 4 changed files with 11 additions and 13 deletions.
diff --git a/src/CoreShop/Bundle/MoneyBundle/EventListener/IndexActionSettingsSubscriber.php b/src/CoreShop/Bundle/MoneyBundle/EventListener/IndexActionSettingsSubscriber.php
@@ -26,9 +26,8 @@ class IndexActionSettingsSubscriber implements EventSubscriberInterface
 {
     public function __construct(
         private int $decimalPrecision,
-        private int $decimalFactor
-    )
-    {
+        private int $decimalFactor,
+    ) {
     }
 
     public static function getSubscribedEvents()

diff --git a/src/CoreShop/Component/Product/Rule/Action/PriceActionProcessor.php b/src/CoreShop/Component/Product/Rule/Action/PriceActionProcessor.php
@@ -38,7 +38,7 @@ public function getPrice($subject, array $context, array $configuration): int
         if (isset($context['unitDefinition']) && $context['unitDefinition'] instanceof ProductUnitDefinitionInterface) {
             throw new NoRetailPriceFoundException(__CLASS__);
         }
-        
+
         Assert::keyExists($context, 'base_currency');
         Assert::isInstanceOf($context['base_currency'], CurrencyInterface::class);
 

diff --git a/src/CoreShop/Component/Variant/AttributeCollector.php b/src/CoreShop/Component/Variant/AttributeCollector.php
@@ -31,9 +31,9 @@
 
 class AttributeCollector implements AttributeCollectorInterface
 {
-
-    public function __construct(private EventDispatcherInterface $eventDispatcher)
-    {
+    public function __construct(
+        private EventDispatcherInterface $eventDispatcher,
+    ) {
     }
 
     /**
@@ -57,10 +57,9 @@ public function getAttributes(array $products, bool $showInList = false): array
     {
         $resolvedGroups = [];
         foreach ($products as $product) {
-
             $event = $this->eventDispatcher->dispatch(new VariantAvailabilityEvent($product), 'coreshop.attribute.collector.preCondition');
 
-            if (!$event->isConditionMet()){
+            if (!$event->isConditionMet()) {
                 continue;
             }
 
@@ -86,7 +85,6 @@ public function getAttributes(array $products, bool $showInList = false): array
                     $group->setGroup($attributeGroup);
                     $group->setType(get_class($attribute));
 
-
                     $resolvedGroups[$attributeGroup->getId()] = $group;
                 } else {
                     $group = $resolvedGroups[$attributeGroup->getId()];

diff --git a/src/CoreShop/Component/Variant/Event/VariantAvailabilityEvent.php b/src/CoreShop/Component/Variant/Event/VariantAvailabilityEvent.php
@@ -25,8 +25,9 @@ class VariantAvailabilityEvent extends Event
 {
     private bool $conditionMet = true;
 
-    public function __construct(private ProductVariantAwareInterface $product)
-    {
+    public function __construct(
+        private ProductVariantAwareInterface $product,
+    ) {
     }
 
     public function getProduct(): ProductVariantAwareInterface
@@ -43,4 +44,4 @@ public function isConditionMet(): bool
     {
         return $this->conditionMet;
     }
-}
+}