Fix code style

sequra · Jun 12, 2024 · 62908c7 · 62908c7
1 parent 172f5de
commit 62908c7
Show file tree

Hide file tree

Showing 180 changed files with 634 additions and 520 deletions.
diff --git a/src/BusinessLogic/AdminAPI/Connection/ConnectionController.php b/src/BusinessLogic/AdminAPI/Connection/ConnectionController.php
@@ -112,7 +112,7 @@ public function validateConnectionData(ConnectionRequest $connectionRequest): Re
     {
         try {
             $this->connectionService->isConnectionDataValid($connectionRequest->transformToDomainModel());
-        } catch (BadMerchantIdException|InvalidEnvironmentException|WrongCredentialsException|HttpRequestException $e) {
+        } catch (BadMerchantIdException | InvalidEnvironmentException | WrongCredentialsException | HttpRequestException $e) {
             return new ErrorResponse($e);
         }
 

diff --git a/src/BusinessLogic/AdminAPI/Connection/Requests/OnboardingRequest.php b/src/BusinessLogic/AdminAPI/Connection/Requests/OnboardingRequest.php
@@ -48,8 +48,7 @@ public function __construct(
         string $password,
         bool $sendStatisticalData,
         ?string $merchantId = null
-    )
-    {
+    ) {
         $this->environment = $environment;
         $this->merchantId = $merchantId;
         $this->username = $username;

diff --git a/src/BusinessLogic/AdminAPI/CountryConfiguration/CountryConfigurationController.php b/src/BusinessLogic/AdminAPI/CountryConfiguration/CountryConfigurationController.php
@@ -36,8 +36,7 @@ class CountryConfigurationController
     public function __construct(
         CountryConfigurationService $countryConfigurationService,
         SellingCountriesService $sellingCountriesService
-    )
-    {
+    ) {
         $this->countryConfigurationService = $countryConfigurationService;
         $this->sellingCountriesService = $sellingCountriesService;
     }

diff --git a/src/BusinessLogic/AdminAPI/GeneralSettings/GeneralSettingsController.php b/src/BusinessLogic/AdminAPI/GeneralSettings/GeneralSettingsController.php
@@ -32,10 +32,9 @@ class GeneralSettingsController
      * @param CategoryService $categoryService
      */
     public function __construct(
-        GeneralSettingsService   $generalSettingsService,
-        CategoryService          $categoryService
-    )
-    {
+        GeneralSettingsService $generalSettingsService,
+        CategoryService $categoryService
+    ) {
         $this->generalSettingsService = $generalSettingsService;
         $this->categoryService = $categoryService;
     }

diff --git a/src/BusinessLogic/AdminAPI/GeneralSettings/Requests/GeneralSettingsRequest.php b/src/BusinessLogic/AdminAPI/GeneralSettings/Requests/GeneralSettingsRequest.php
@@ -50,8 +50,7 @@ public function __construct(
         ?array $allowedIPAddresses,
         ?array $excludedProducts,
         ?array $excludedCategories
-    )
-    {
+    ) {
         $this->sendOrderReportsPeriodicallyToSeQura = $sendOrderReportsPeriodicallyToSeQura;
         $this->showSeQuraCheckoutAsHostedPage = $showSeQuraCheckoutAsHostedPage;
         $this->allowedIPAddresses = $allowedIPAddresses;

diff --git a/src/BusinessLogic/AdminAPI/Integration/IntegrationController.php b/src/BusinessLogic/AdminAPI/Integration/IntegrationController.php
@@ -42,8 +42,7 @@ public function __construct(
         VersionService $versionService,
         Configuration $configurationService,
         UIStateService $stateService
-    )
-    {
+    ) {
         $this->versionService = $versionService;
         $this->configurationService = $configurationService;
         $this->stateService = $stateService;

diff --git a/src/BusinessLogic/AdminAPI/OrderStatusSettings/OrderStatusSettingsController.php b/src/BusinessLogic/AdminAPI/OrderStatusSettings/OrderStatusSettingsController.php
@@ -35,9 +35,8 @@ class OrderStatusSettingsController
      */
     public function __construct(
         OrderStatusSettingsService $orderStatusSettingsService,
-        ShopOrderStatusesService   $shopOrderStatusesService
-    )
-    {
+        ShopOrderStatusesService $shopOrderStatusesService
+    ) {
         $this->orderStatusSettingsService = $orderStatusSettingsService;
         $this->shopOrderStatusesService = $shopOrderStatusesService;
     }

diff --git a/src/BusinessLogic/AdminAPI/PaymentMethods/Responses/PaymentMethodsResponse.php b/src/BusinessLogic/AdminAPI/PaymentMethods/Responses/PaymentMethodsResponse.php
@@ -28,7 +28,7 @@ public function __construct(array $paymentMethods)
     /**
      * @inheritDoc
      */
-    public function toArray():array
+    public function toArray(): array
     {
         $methods = [];
         foreach ($this->paymentMethods as $paymentMethod) {

diff --git a/src/BusinessLogic/AdminAPI/PaymentMethods/Responses/ProductsResponse.php b/src/BusinessLogic/AdminAPI/PaymentMethods/Responses/ProductsResponse.php
@@ -31,4 +31,4 @@ public function toArray(): array
     {
         return $this->products;
     }
-}
+}
diff --git a/src/BusinessLogic/AdminAPI/PromotionalWidgets/PromotionalWidgetsController.php b/src/BusinessLogic/AdminAPI/PromotionalWidgets/PromotionalWidgetsController.php
@@ -26,8 +26,7 @@ class PromotionalWidgetsController
      */
     public function __construct(
         WidgetSettingsService $widgetSettingsService
-    )
-    {
+    ) {
         $this->widgetSettingsService = $widgetSettingsService;
     }
 

diff --git a/src/BusinessLogic/AdminAPI/PromotionalWidgets/Requests/WidgetSettingsRequest.php b/src/BusinessLogic/AdminAPI/PromotionalWidgets/Requests/WidgetSettingsRequest.php
@@ -63,17 +63,16 @@ class WidgetSettingsRequest extends Request
      * @param array $messagesBelowLimit
      */
     public function __construct(
-        bool    $enabled,
+        bool $enabled,
         ?string $assetsKey,
-        bool    $displayOnProductPage,
-        bool    $showInstallmentsInProductListing,
-        bool    $showInstallmentsInCartPage,
-        string  $miniWidgetSelector,
-        string   $widgetConfiguration,
-        array   $messages = [],
-        array   $messagesBelowLimit = []
-    )
-    {
+        bool $displayOnProductPage,
+        bool $showInstallmentsInProductListing,
+        bool $showInstallmentsInCartPage,
+        string $miniWidgetSelector,
+        string $widgetConfiguration,
+        array $messages = [],
+        array $messagesBelowLimit = []
+    ) {
         $this->enabled = $enabled;
         $this->assetsKey = $assetsKey;
         $this->displayOnProductPage = $displayOnProductPage;

diff --git a/src/BusinessLogic/AdminAPI/Response/ErrorResponse.php b/src/BusinessLogic/AdminAPI/Response/ErrorResponse.php
@@ -33,11 +33,12 @@ public function __construct(Throwable $error)
 
     /**
      * Implementation is swallowing all undefined calls to avoid undefined method call exceptions when
+     *
      * @see ErrorHandlingAspect already hanled the API call exception but because of chaining calle will trigger
      * API controller messages on instance of the @see self.
      *
-     * @param $methodName
-     * @param $arguments
+     * @param  $methodName
+     * @param  $arguments
      * @return self Already handled error response
      */
     public function __call($methodName, $arguments)

diff --git a/src/BusinessLogic/Bootstrap/Aspect/Aspect.php b/src/BusinessLogic/Bootstrap/Aspect/Aspect.php
@@ -11,4 +11,3 @@ interface Aspect
 {
     public function applyOn(callable $callee, array $params = []);
 }
-
diff --git a/src/BusinessLogic/Bootstrap/Aspect/Aspects.php b/src/BusinessLogic/Bootstrap/Aspect/Aspects.php
@@ -76,7 +76,7 @@ public function __call($methodName, $arguments)
             return $this->aspect->applyOn([$this->subject, $methodName], $arguments);
         }
 
-        return $this->aspect->applyOn(function() use ($methodName, $arguments) {
+        return $this->aspect->applyOn(function () use ($methodName, $arguments) {
             $subject = ServiceRegister::getService($this->subjectClassName);
 
             return call_user_func_array([$subject, $methodName], $arguments);

diff --git a/src/BusinessLogic/BootstrapComponent.php b/src/BusinessLogic/BootstrapComponent.php
@@ -613,7 +613,9 @@ protected static function initEvents(): void
 
         EventBus::getInstance()->when(TickEvent::class, TickEventListener::class . '::handle');
 
-        /** @var QueueItemStateTransitionEventBus $queueBus */
+        /**
+ * @var QueueItemStateTransitionEventBus $queueBus
+*/
         $queueBus = ServiceRegister::getService(QueueItemStateTransitionEventBus::CLASS_NAME);
 
         $queueBus->when(

diff --git a/src/BusinessLogic/CheckoutAPI/CheckoutAPI.php b/src/BusinessLogic/CheckoutAPI/CheckoutAPI.php
@@ -29,7 +29,7 @@ public static function get(): object
     }
 
     /**
-     * @param string $storeId
+     * @param  string $storeId
      * @return SolicitationController
      */
     public function solicitation(string $storeId): object

diff --git a/src/BusinessLogic/CheckoutAPI/Solicitation/Response/SolicitationResponse.php b/src/BusinessLogic/CheckoutAPI/Solicitation/Response/SolicitationResponse.php
@@ -24,6 +24,7 @@ class SolicitationResponse extends Response
 
     /**
      * SolicitationResponse constructor.
+     *
      * @param SeQuraOrder $order
      * @param SeQuraPaymentMethod[] $availablePaymentMethods
      */
@@ -53,7 +54,7 @@ public function toArray(): array
     {
         return [
             'order' => $this->order->toArray(),
-            'availablePaymentMethods' => array_map(static function(SeQuraPaymentMethod $paymentMethod) {
+            'availablePaymentMethods' => array_map(static function (SeQuraPaymentMethod $paymentMethod) {
                 return [
                     'product' => $paymentMethod->getProduct(),
                     'title' => $paymentMethod->getTitle(),

diff --git a/src/BusinessLogic/DataAccess/ConnectionData/Repositories/ConnectionDataRepository.php b/src/BusinessLogic/DataAccess/ConnectionData/Repositories/ConnectionDataRepository.php
@@ -78,7 +78,9 @@ public function setConnectionData(ConnectionData $connectionData): void
      */
     public function getOldestConnectionSettingsStoreId(): ?string
     {
-        /** @var ConnectionDataEntity $connectionData */
+        /**
+ * @var ConnectionDataEntity $connectionData
+*/
         $connectionData = $this->repository->selectOne(new QueryFilter());
 
         return $connectionData ? $connectionData->getStoreId() : null;
@@ -89,7 +91,9 @@ public function getOldestConnectionSettingsStoreId(): ?string
      */
     public function getAllConnectionSettingsStores(): array
     {
-        /** @var ConnectionDataEntity[] $entities */
+        /**
+ * @var ConnectionDataEntity[] $entities
+*/
         $entities = $this->repository->select();
 
         return $entities ? array_map(function ($entity) {
@@ -110,7 +114,7 @@ protected function getConnectionDataEntity(): ?ConnectionDataEntity
         $queryFilter->where('storeId', Operators::EQUALS, $this->storeContext->getStoreId());
 
         /**
-         * @var  ConnectionDataEntity $connectionData
+         * @var ConnectionDataEntity $connectionData
          */
         $connectionData = $this->repository->selectOne($queryFilter);
 

diff --git a/...nessLogic/DataAccess/CountryConfiguration/Repositories/CountryConfigurationRepository.php b/...nessLogic/DataAccess/CountryConfiguration/Repositories/CountryConfigurationRepository.php
@@ -84,7 +84,9 @@ protected function getCountryConfigurationEntity(): ?CountryConfigurationEntity
         $queryFilter = new QueryFilter();
         $queryFilter->where('storeId', Operators::EQUALS, $this->storeContext->getStoreId());
 
-        /** @var  CountryConfigurationEntity $countryConfiguration */
+        /**
+ * @var CountryConfigurationEntity $countryConfiguration
+*/
         $countryConfiguration = $this->repository->selectOne($queryFilter);
 
         return $countryConfiguration;

diff --git a/src/BusinessLogic/DataAccess/GeneralSettings/Repositories/GeneralSettingsRepository.php b/src/BusinessLogic/DataAccess/GeneralSettings/Repositories/GeneralSettingsRepository.php
@@ -85,7 +85,9 @@ protected function getGeneralSettingsEntity(): ?GeneralSettingsEntity
         $queryFilter = new QueryFilter();
         $queryFilter->where('storeId', Operators::EQUALS, $this->storeContext->getStoreId());
 
-        /** @var  GeneralSettingsEntity $generalSettings */
+        /**
+ * @var GeneralSettingsEntity $generalSettings
+*/
         $generalSettings = $this->repository->selectOne($queryFilter);
 
         return $generalSettings;

diff --git a/src/BusinessLogic/DataAccess/Order/Repositories/SeQuraOrderRepository.php b/src/BusinessLogic/DataAccess/Order/Repositories/SeQuraOrderRepository.php
@@ -33,7 +33,9 @@ public function getByShopReference(string $shopOrderReference): ?SeQuraOrder
         $filter = new QueryFilter();
         $filter->where('orderRef1', Operators::EQUALS, $shopOrderReference);
 
-        /** @var SeQuraOrder|null $result */
+        /**
+ * @var SeQuraOrder|null $result
+*/
         $result =  $this->repository->selectOne($filter);
 
         return $result;
@@ -44,7 +46,9 @@ public function getOrderBatchByShopReferences(array $shopOrderReferences): array
         $filter = new QueryFilter();
         $filter->where('orderRef1', Operators::IN, $shopOrderReferences);
 
-        /** @var SeQuraOrder[] $result */
+        /**
+ * @var SeQuraOrder[] $result
+*/
         $result =  $this->repository->select($filter);
 
         return $result;
@@ -55,7 +59,9 @@ public function getByCartId(string $cartId): ?SeQuraOrder
         $filter = new QueryFilter();
         $filter->where('cartId', Operators::EQUALS, $cartId);
 
-        /** @var SeQuraOrder|null $result */
+        /**
+ * @var SeQuraOrder|null $result
+*/
         $result =  $this->repository->selectOne($filter);
 
         return $result;
@@ -66,7 +72,9 @@ public function getByOrderReference(string $sequraOrderReference): ?SeQuraOrder
         $filter = new QueryFilter();
         $filter->where('reference', Operators::EQUALS, $sequraOrderReference);
 
-        /** @var SeQuraOrder|null $result */
+        /**
+ * @var SeQuraOrder|null $result
+*/
         $result =  $this->repository->selectOne($filter);
 
         return $result;

diff --git a/src/BusinessLogic/DataAccess/OrderSettings/Repositories/OrderStatusMappingRepository.php b/src/BusinessLogic/DataAccess/OrderSettings/Repositories/OrderStatusMappingRepository.php
@@ -84,7 +84,9 @@ protected function getOrderStatusMappingsEntity(): ?OrderStatusSettings
         $queryFilter = new QueryFilter();
         $queryFilter->where('storeId', Operators::EQUALS, $this->storeContext->getStoreId());
 
-        /** @noinspection PhpIncompatibleReturnTypeInspection */
+        /**
+ * @noinspection PhpIncompatibleReturnTypeInspection
+*/
         return $this->repository->selectOne($queryFilter);
     }
 }
diff --git a/src/BusinessLogic/DataAccess/PromotionalWidgets/Repositories/WidgetSettingsRepository.php b/src/BusinessLogic/DataAccess/PromotionalWidgets/Repositories/WidgetSettingsRepository.php
@@ -81,9 +81,11 @@ protected function getWidgetSettingsEntity(): ?WidgetSettingsEntity
         $queryFilter = new QueryFilter();
         $queryFilter->where('storeId', Operators::EQUALS, $this->storeContext->getStoreId());
 
-        /** @var  WidgetSettingsEntity $widgetSettings */
+        /**
+ * @var WidgetSettingsEntity $widgetSettings
+*/
         $widgetSettings = $this->repository->selectOne($queryFilter);
 
         return $widgetSettings;
     }
-}
+}
diff --git a/src/BusinessLogic/DataAccess/SendReport/Repositories/SendReportRepository.php b/src/BusinessLogic/DataAccess/SendReport/Repositories/SendReportRepository.php
@@ -106,7 +106,9 @@ public function getReportSendingContexts(): array
         $queryFilter = new QueryFilter();
         $queryFilter->where('sendReportTime', Operators::LESS_OR_EQUAL_THAN, $now);
 
-        /** @var SendReportEntity[] $result */
+        /**
+ * @var SendReportEntity[] $result
+*/
         $result = $this->repository->select($queryFilter);
 
         return $result ? array_map(function ($entity) {
@@ -128,7 +130,9 @@ protected function getSendReportEntity(?string $context = null): ?SendReportEnti
         $queryFilter = new QueryFilter();
         $queryFilter->where('context', Operators::EQUALS, $context ?? $this->storeContext->getStoreId());
 
-        /** @var SendReportEntity $statisticalData */
+        /**
+ * @var SendReportEntity $statisticalData
+*/
         $statisticalData = $this->repository->selectOne($queryFilter);
 
         return $statisticalData;

diff --git a/src/BusinessLogic/DataAccess/StatisticalData/Repositories/StatisticalDataRepository.php b/src/BusinessLogic/DataAccess/StatisticalData/Repositories/StatisticalDataRepository.php
@@ -85,7 +85,9 @@ protected function getStatisticalDataEntity(): ?StatisticalDataEntity
         $queryFilter = new QueryFilter();
         $queryFilter->where('storeId', Operators::EQUALS, $this->storeContext->getStoreId());
 
-        /** @var  StatisticalDataEntity $statisticalData */
+        /**
+ * @var StatisticalDataEntity $statisticalData
+*/
         $statisticalData = $this->repository->selectOne($queryFilter);
 
         return $statisticalData;

diff --git a/src/BusinessLogic/DataAccess/TransactionLog/Repositories/TransactionLogRepository.php b/src/BusinessLogic/DataAccess/TransactionLog/Repositories/TransactionLogRepository.php
@@ -108,14 +108,16 @@ public function getItemByExecutionId(int $executionId): ?TransactionLog
             ->where('executionId', Operators::EQUALS, $executionId)
             ->orderBy('id', QueryFilter::ORDER_DESC);
 
-        /** @var TransactionLog|null $transactionLog */
+        /**
+ * @var TransactionLog|null $transactionLog
+*/
         $transactionLog = $this->repository->selectOne($queryFilter);
 
         return $transactionLog;
     }
 
     /**
-     * @param DateTime|null $disconnectTime
+     * @param  DateTime|null $disconnectTime
      * @return int
      *
      * @throws QueryFilterInvalidParamException
@@ -142,7 +144,9 @@ public function findByMerchantReference(string $merchantReference): ?Transaction
         $queryFilter = new QueryFilter();
         $queryFilter->where('merchantReference', Operators::EQUALS, $merchantReference);
 
-        /** @var TransactionLog|null $transactionLog */
+        /**
+ * @var TransactionLog|null $transactionLog
+*/
         $transactionLog = $this->repository->selectOne($queryFilter);
 
         return $transactionLog;
@@ -204,7 +208,9 @@ protected function getTransactionLogEntity(string $merchantReference): ?Transact
         $queryFilter->where('storeId', Operators::EQUALS, $this->storeContext->getStoreId())
             ->where('merchantReference', Operators::EQUALS, $merchantReference);
 
-        /** @var TransactionLog|null $transactionLog */
+        /**
+ * @var TransactionLog|null $transactionLog
+*/
         $transactionLog = $this->repository->selectOne($queryFilter);
 
         return $transactionLog;
-Original file line number
+Diff line change
@@ Expand Up / @@ -31,4 +31,4 @@ public function toArray(): array @@
         {
             return $this->products;
         }
-    }
+    }
Original file line number	Diff line number	Diff line change
Expand Up		@@ -11,4 +11,3 @@ interface Aspect
		{
		public function applyOn(callable $callee, array $params = []);
		}