odiseoteam / SyliusReportPlugin

spec/Controller/ReportControllerSpec.php

Spacing +2 added lines, -2 removed lines

@@ -117,7 +117,7 @@ 
         $singleResourceProvider->get($configuration, $repository)->willReturn($resource);
 
         $configuration->isHtmlRequest()->willReturn(true);
-        $configuration->getTemplate(ResourceActions::SHOW . '.html')->willReturn('SyliusShopBundle:Product:show.html.twig');
+        $configuration->getTemplate(ResourceActions::SHOW.'.html')->willReturn('SyliusShopBundle:Product:show.html.twig');
 
         $eventDispatcher->dispatch(ResourceActions::SHOW, $configuration, $resource)->shouldBeCalled();
 
@@ -173,7 +173,7 @@ 
 
     private function getViewComparingCallback(View $expectedView)
     {
-        return function ($value) use ($expectedView) {
+        return function($value) use ($expectedView) {
             if (!$value instanceof View) {
                 return false;
             }

src/Repository/OrderRepository.php

Spacing +1 added lines, -1 removed lines

@@ -16,7 +16,7 @@
         return $this->createQueryBuilder('o')
             ->select('o.id, o.number')
             ->andWhere('o.number LIKE :number')
-            ->setParameter('number', '%' . $number . '%')
+            ->setParameter('number', '%'.$number.'%')
             ->getQuery()
             ->getArrayResult()
         ;

src/DataFetcher/TimePeriodDataFetcher.php

Spacing +8 added lines, -8 removed lines

@@ -63,7 +63,7 @@ 
         $labels = [];
         foreach ($labelsAux as $label)
         {
-            if(!in_array($label, ['MonthDate', 'YearDate', 'DateDate']))
+            if (!in_array($label, ['MonthDate', 'YearDate', 'DateDate']))
             {
                 $labels[] = $label;
             }
@@ -85,7 +85,7 @@ 
         $labels = [];
         foreach ($labelsAux as $label)
         {
-            if(!in_array($label, ['MonthDate', 'YearDate', 'DateDate']))
+            if (!in_array($label, ['MonthDate', 'YearDate', 'DateDate']))
             {
                 $labels[] = preg_replace('/(?!^)[A-Z]{2,}(?=[A-Z][a-z])|[A-Z][a-z]/', ' $0', $label);
             }
@@ -158,21 +158,21 @@ 
         }
         $labels = array_keys($datas[0]);
         $datesMedia = [];
-        foreach($datas as $data)
+        foreach ($datas as $data)
         {
             $date = new \DateTime($data[$labels[0]]);
             $dateFormated = $date->format($configuration['timePeriod']['presentationFormat']);
-            $currentDateMedia = isset($datesMedia[$dateFormated])?$datesMedia[$dateFormated]:array('quantity' => 0, 'media' => 0);
-            $currentDateMedia['quantity'] = $currentDateMedia['quantity']+1;
-            $currentDateMedia['media'] = $currentDateMedia['media']+$data[$labels[1]];
+            $currentDateMedia = isset($datesMedia[$dateFormated]) ? $datesMedia[$dateFormated] : array('quantity' => 0, 'media' => 0);
+            $currentDateMedia['quantity'] = $currentDateMedia['quantity'] + 1;
+            $currentDateMedia['media'] = $currentDateMedia['media'] + $data[$labels[1]];
             $datesMedia[$dateFormated] = $currentDateMedia;
         }
         $fetched = [];
-        foreach($datesMedia as $date => $dateMedia)
+        foreach ($datesMedia as $date => $dateMedia)
         {
             $fetched[] = [
                 $labels[0] => $date,
-                $labels[1] => round($dateMedia['media']/$dateMedia['quantity'], 1)
+                $labels[1] => round($dateMedia['media'] / $dateMedia['quantity'], 1)
             ];
         }
         return $fetched;

src/Controller/ReportController.php

Spacing +4 added lines, -4 removed lines

@@ -58,7 +58,7 @@ 
 
         if ($configuration->isHtmlRequest()) {
             $view
-                ->setTemplate($configuration->getTemplate(ResourceActions::SHOW . '.html'))
+                ->setTemplate($configuration->getTemplate(ResourceActions::SHOW.'.html'))
                 ->setTemplateVar($this->metadata->getName())
                 ->setData([
                     'configuration' => $configuration,
@@ -92,7 +92,7 @@ 
         $configuration = $request->get('configuration');
         $currencyProvider = $this->get('sylius.currency_provider');
 
-        if(isset($configuration['timePeriod']))
+        if (isset($configuration['timePeriod']))
         {
             $configuration['timePeriod']['start'] = new \DateTime($configuration['timePeriod']['start']['date'], new \DateTimeZone($configuration['timePeriod']['start']['timezone']));
             $configuration['timePeriod']['end'] = new \DateTime($configuration['timePeriod']['end']['date'], new \DateTimeZone($configuration['timePeriod']['end']['timezone']));
@@ -191,13 +191,13 @@ 
         $responseData = [];
         foreach ($data->getData() as $key => $value)
         {
-            if(is_array($value))
+            if (is_array($value))
             {
                 foreach ($value as $k => $v)
                 {
                     $responseData[] = [$k, $v];
                 }
-            }else {
+            } else {
                 $responseData[] = [$key, $value];
             }
         }

Braces +1 added lines, -1 removed lines

@@ -197,7 +197,7 @@
                 {
                     $responseData[] = [$k, $v];
                 }
-            }else {
+            } else {
                 $responseData[] = [$key, $value];
             }
         }

src/Filter/QueryFilter.php

Spacing +25 added lines, -25 removed lines

@@ -69,7 +69,7 @@ 
         $selectPeriod = '';
         $selectGroupBy = '';
         foreach ($configuration['groupBy'] as $groupByElement) {
-            if(strlen($selectPeriod) > 0) {
+            if (strlen($selectPeriod) > 0) {
                 $selectPeriod .= ', ';
                 $selectGroupBy .= ',';
             }
@@ -90,7 +90,7 @@ 
      */
     public function addLeftJoin($join, $alias): string
     {
-        if(!isset($this->joins[$join])) {
+        if (!isset($this->joins[$join])) {
             $this->joins[$join] = $alias;
 
             $this->qb->leftJoin($join, $alias);
@@ -136,19 +136,19 @@ 
      */
     public function addChannel(array $configuration = [], $field = null): void
     {
-        if(isset($configuration['channel']) && $configuration['channel'] != null) {
+        if (isset($configuration['channel']) && $configuration['channel'] != null) {
             $storeIds = [];
 
-            if($configuration['channel'] instanceof ChannelInterface) {
+            if ($configuration['channel'] instanceof ChannelInterface) {
                 $storeIds[] = $configuration['channel']->getId();
-            }else if(is_array($configuration['channel']) && !in_array(0, $configuration['channel'])) {
+            } else if (is_array($configuration['channel']) && !in_array(0, $configuration['channel'])) {
                 $storeIds = $configuration['channel'];
             }
 
-            if(!(count($storeIds) > 0))
+            if (!(count($storeIds) > 0))
                 return;
 
-            if(!$field) {
+            if (!$field) {
                 $alias = $this->qb->getRootAliases()[0];
                 $field = $alias.'.channel';
             }
@@ -164,10 +164,10 @@ 
      */
     public function addUserGender(array $configuration = []): void
     {
-        if(isset($configuration['userGender']) && $configuration['userGender'] != null) {
+        if (isset($configuration['userGender']) && $configuration['userGender'] != null) {
             $rootAlias = $cAlias = $this->qb->getRootAliases()[0];
 
-            if(!$this->hasRootEntity(Customer::class))
+            if (!$this->hasRootEntity(Customer::class))
                 $cAlias = $this->addLeftJoin($rootAlias.'.customer', 'c');
 
             $this->qb
@@ -181,20 +181,20 @@ 
      */
     public function addUserBuyer(array $configuration = []): void
     {
-        if(isset($configuration['userBuyer']) && $configuration['userBuyer'] != null && $configuration['userBuyer'] != 'both') {
+        if (isset($configuration['userBuyer']) && $configuration['userBuyer'] != null && $configuration['userBuyer'] != 'both') {
             $rootAlias = $this->qb->getRootAliases()[0];
 
-            if(!$this->hasRootEntity(Customer::class))
+            if (!$this->hasRootEntity(Customer::class))
                 $this->addLeftJoin($rootAlias.'.customer', 'c');
 
             $oAlias = $this->addLeftJoin('c.orders', 'o');
 
-            if($configuration['userBuyer'] == 'yes') {
+            if ($configuration['userBuyer'] == 'yes') {
                 $this->qb
                     ->andWhere($this->qb->expr()->isNotNull($oAlias.'.checkoutCompletedAt'))
                     ->andWhere($oAlias.'.itemsTotal > 0')
                 ;
-            }else {
+            } else {
                 $this->qb
                     ->andWhere($this->qb->expr()->isNull($oAlias.'.checkoutCompletedAt'))
                 ;
@@ -209,10 +209,10 @@ 
     public function addUserCountry(array $configuration = [], string $addressType = 'shipping'): void
     {
         $type = 'user'.ucfirst($addressType).'Country';
-        if(isset($configuration[$type]) && $configuration[$type] != null) {
+        if (isset($configuration[$type]) && $configuration[$type] != null) {
             $rootAlias = $cAlias = $this->qb->getRootAliases()[0];
 
-            if(!$this->hasRootEntity(Customer::class))
+            if (!$this->hasRootEntity(Customer::class))
                 $cAlias = $this->addLeftJoin($rootAlias.'.customer', 'c');
 
             $caAlias = $this->addLeftJoin($cAlias.'.addresses', 'c'.substr($addressType, 0, 1).'a');
@@ -230,10 +230,10 @@ 
     public function addUserProvince(array $configuration = [], string $addressType = 'shipping'): void
     {
         $type = 'user'.ucfirst($addressType).'Province';
-        if(isset($configuration[$type]) && $configuration[$type] != null) {
+        if (isset($configuration[$type]) && $configuration[$type] != null) {
             $rootAlias = $cAlias = $this->qb->getRootAliases()[0];
 
-            if(!$this->hasRootEntity(Customer::class))
+            if (!$this->hasRootEntity(Customer::class))
                 $cAlias = $this->addLeftJoin($rootAlias.'.customer', 'c');
 
             $caAlias = $this->addLeftJoin($cAlias.'.addresses', 'c'.substr($addressType, 0, 1).'a');
@@ -251,10 +251,10 @@ 
     public function addUserCity(array $configuration = [], string $addressType = 'shipping'): void
     {
         $type = 'user'.ucfirst($addressType).'City';
-        if(isset($configuration[$type]) && $configuration[$type] != null) {
+        if (isset($configuration[$type]) && $configuration[$type] != null) {
             $rootAlias = $cAlias = $this->qb->getRootAliases()[0];
 
-            if(!$this->hasRootEntity(Customer::class))
+            if (!$this->hasRootEntity(Customer::class))
                 $cAlias = $this->addLeftJoin($rootAlias.'.customer', 'c');
 
             $caAlias = $this->addLeftJoin($cAlias.'.addresses', 'c'.substr($addressType, 0, 1).'a');
@@ -272,10 +272,10 @@ 
     public function addUserPostcode(array $configuration = [], string $addressType = 'shipping'): void
     {
         $type = 'user'.ucfirst($addressType).'Postcode';
-        if(isset($configuration[$type]) && $configuration[$type] != null) {
+        if (isset($configuration[$type]) && $configuration[$type] != null) {
             $rootAlias = $cAlias = $this->qb->getRootAliases()[0];
 
-            if(!$this->hasRootEntity(Customer::class))
+            if (!$this->hasRootEntity(Customer::class))
                 $cAlias = $this->addLeftJoin($rootAlias.'.customer', 'c');
 
             $caAlias = $this->addLeftJoin($cAlias.'.addresses', 'c'.substr($addressType, 0, 1).'2a');
@@ -292,7 +292,7 @@ 
      */
     public function addProduct(array $configuration = [], string $field = 'p.id'): void
     {
-        if(isset($configuration['product']) && $configuration['product'] != null) {
+        if (isset($configuration['product']) && $configuration['product'] != null) {
             $this->qb
                 ->andWhere($this->qb->expr()->in($field, $configuration['product']))
             ;
@@ -305,7 +305,7 @@ 
      */
     public function addProductBrand(array $configuration = [], string $field = 'p.vendor'): void
     {
-        if(isset($configuration['productBrand']) && $configuration['productBrand'] != null) {
+        if (isset($configuration['productBrand']) && $configuration['productBrand'] != null) {
             $this->qb
                 ->andWhere($this->qb->expr()->in($field, $configuration['productBrand']))
             ;
@@ -318,7 +318,7 @@ 
      */
     public function addProductCategory(array $configuration = [], string $field = 'pt.id'): void
     {
-        if(isset($configuration['productCategory']) && $configuration['productCategory'] != null) {
+        if (isset($configuration['productCategory']) && $configuration['productCategory'] != null) {
             $this->qb
                 ->andWhere($this->qb->expr()->in($field, $configuration['productCategory']))
             ;
@@ -331,7 +331,7 @@ 
      */
     public function addOrderNumbers(array $configuration = [], string $field = 'o.id'): void
     {
-        if(isset($configuration['orderNumbers']) && $configuration['orderNumbers'] != null) {
+        if (isset($configuration['orderNumbers']) && $configuration['orderNumbers'] != null) {
             $this->qb
                 ->andWhere($this->qb->expr()->in($field, $configuration['orderNumbers']))
             ;

Braces +23 added lines, -16 removed lines

@@ -141,12 +141,13 @@ 
 
             if($configuration['channel'] instanceof ChannelInterface) {
                 $storeIds[] = $configuration['channel']->getId();
-            }else if(is_array($configuration['channel']) && !in_array(0, $configuration['channel'])) {
+            } else if(is_array($configuration['channel']) && !in_array(0, $configuration['channel'])) {
                 $storeIds = $configuration['channel'];
             }
 
-            if(!(count($storeIds) > 0))
-                return;
+            if(!(count($storeIds) > 0)) {
+                            return;
+            }
 
             if(!$field) {
                 $alias = $this->qb->getRootAliases()[0];
@@ -167,8 +168,9 @@ 
         if(isset($configuration['userGender']) && $configuration['userGender'] != null) {
             $rootAlias = $cAlias = $this->qb->getRootAliases()[0];
 
-            if(!$this->hasRootEntity(Customer::class))
-                $cAlias = $this->addLeftJoin($rootAlias.'.customer', 'c');
+            if(!$this->hasRootEntity(Customer::class)) {
+                            $cAlias = $this->addLeftJoin($rootAlias.'.customer', 'c');
+            }
 
             $this->qb
                 ->andWhere($this->qb->expr()->in($cAlias.'.gender', $configuration['userGender']))
@@ -184,8 +186,9 @@ 
         if(isset($configuration['userBuyer']) && $configuration['userBuyer'] != null && $configuration['userBuyer'] != 'both') {
             $rootAlias = $this->qb->getRootAliases()[0];
 
-            if(!$this->hasRootEntity(Customer::class))
-                $this->addLeftJoin($rootAlias.'.customer', 'c');
+            if(!$this->hasRootEntity(Customer::class)) {
+                            $this->addLeftJoin($rootAlias.'.customer', 'c');
+            }
 
             $oAlias = $this->addLeftJoin('c.orders', 'o');
 
@@ -194,7 +197,7 @@ 
                     ->andWhere($this->qb->expr()->isNotNull($oAlias.'.checkoutCompletedAt'))
                     ->andWhere($oAlias.'.itemsTotal > 0')
                 ;
-            }else {
+            } else {
                 $this->qb
                     ->andWhere($this->qb->expr()->isNull($oAlias.'.checkoutCompletedAt'))
                 ;
@@ -212,8 +215,9 @@ 
         if(isset($configuration[$type]) && $configuration[$type] != null) {
             $rootAlias = $cAlias = $this->qb->getRootAliases()[0];
 
-            if(!$this->hasRootEntity(Customer::class))
-                $cAlias = $this->addLeftJoin($rootAlias.'.customer', 'c');
+            if(!$this->hasRootEntity(Customer::class)) {
+                            $cAlias = $this->addLeftJoin($rootAlias.'.customer', 'c');
+            }
 
             $caAlias = $this->addLeftJoin($cAlias.'.addresses', 'c'.substr($addressType, 0, 1).'a');
 
@@ -233,8 +237,9 @@ 
         if(isset($configuration[$type]) && $configuration[$type] != null) {
             $rootAlias = $cAlias = $this->qb->getRootAliases()[0];
 
-            if(!$this->hasRootEntity(Customer::class))
-                $cAlias = $this->addLeftJoin($rootAlias.'.customer', 'c');
+            if(!$this->hasRootEntity(Customer::class)) {
+                            $cAlias = $this->addLeftJoin($rootAlias.'.customer', 'c');
+            }
 
             $caAlias = $this->addLeftJoin($cAlias.'.addresses', 'c'.substr($addressType, 0, 1).'a');
 
@@ -254,8 +259,9 @@ 
         if(isset($configuration[$type]) && $configuration[$type] != null) {
             $rootAlias = $cAlias = $this->qb->getRootAliases()[0];
 
-            if(!$this->hasRootEntity(Customer::class))
-                $cAlias = $this->addLeftJoin($rootAlias.'.customer', 'c');
+            if(!$this->hasRootEntity(Customer::class)) {
+                            $cAlias = $this->addLeftJoin($rootAlias.'.customer', 'c');
+            }
 
             $caAlias = $this->addLeftJoin($cAlias.'.addresses', 'c'.substr($addressType, 0, 1).'a');
 
@@ -275,8 +281,9 @@ 
         if(isset($configuration[$type]) && $configuration[$type] != null) {
             $rootAlias = $cAlias = $this->qb->getRootAliases()[0];
 
-            if(!$this->hasRootEntity(Customer::class))
-                $cAlias = $this->addLeftJoin($rootAlias.'.customer', 'c');
+            if(!$this->hasRootEntity(Customer::class)) {
+                            $cAlias = $this->addLeftJoin($rootAlias.'.customer', 'c');
+            }
 
             $caAlias = $this->addLeftJoin($cAlias.'.addresses', 'c'.substr($addressType, 0, 1).'2a');
 

src/Form/Builder/QueryFilterFormBuilder.php

Spacing +5 added lines, -5 removed lines

@@ -306,7 +306,7 @@ 
         {
             $cityLabel = ucfirst(strtolower($address->getCity())).', '.$address->getCountryCode();
 
-            if(!in_array($cityLabel, $choices))
+            if (!in_array($cityLabel, $choices))
             {
                 $choices[$cityLabel] = $address->getCity();
             }
@@ -324,7 +324,7 @@ 
 
         /** @var ProvinceInterface $province */
         foreach ($provinces as $province) {
-            if(!isset($provincesLabel[$province->getCode()])) {
+            if (!isset($provincesLabel[$province->getCode()])) {
                 $provincesLabel[$province->getCode()] = $province->getName();
             }
         }
@@ -332,9 +332,9 @@ 
         /** @var AddressInterface $address */
         foreach ($addresses as $address)
         {
-            $provinceLabel = isset($provincesLabel[$address->getProvinceCode()])?ucfirst(strtolower($provincesLabel[$address->getProvinceCode()])).', '.$address->getCountryCode():null;
+            $provinceLabel = isset($provincesLabel[$address->getProvinceCode()]) ?ucfirst(strtolower($provincesLabel[$address->getProvinceCode()])).', '.$address->getCountryCode() : null;
 
-            if($provinceLabel && !in_array($provinceLabel, $choices))
+            if ($provinceLabel && !in_array($provinceLabel, $choices))
             {
                 $choices[$provinceLabel] = $address->getProvinceCode();
             }
@@ -353,7 +353,7 @@ 
         {
             $postcode = $address->getPostcode();
 
-            if(!in_array($postcode, $choices))
+            if (!in_array($postcode, $choices))
             {
                 $choices[$postcode] = $postcode;
             }