[Bug]: Use parameters for joins in Customer Segment (#549) (#557)

* [Bug]: Use parameters for joins in Customer Segment (#549)

* Use parameters for joins

* Use array_keys

* Set param type

* Use non deprecated version

* Apply php-cs-fixer changes

* artifact bump

* fix stan

* fix

* Apply php-cs-fixer changes

* Update composer.json

* Update composer.json

* Update composer.json

* Try different parameter type

* Update CustomerSegment.php

* fix

* Apply php-cs-fixer changes

* fix

* fix

* Apply php-cs-fixer changes

---------

Co-authored-by: Matthias Schuhmayer <[email protected]>
Co-authored-by: kingjia90 <[email protected]>
Co-authored-by: mattamon <[email protected]>

Author: kingjia90

.github/workflows/static-analysis.yml

@@ -64,7 +64,7 @@ jobs:
 
             - name: "Upload baseline file"
               if: ${{ failure() }}
-              uses: actions/upload-artifact@v2
+              uses: actions/upload-artifact@v4
               with:
                   name: phpstan-baseline.neon
                   path: phpstan-baseline.neon

composer.json

@@ -14,7 +14,7 @@
   },
   "require": {
     "php": "~8.1.0 || ~8.2.0",
-    "doctrine/dbal": "^3.4.2",
+    "doctrine/dbal": "^3.6.0",
     "doctrine/migrations": "^3.0.2",
     "dragonmantank/cron-expression": "^3.0.2",
     "drewm/mailchimp-api": "^2.2",

src/CustomerList/Filter/CustomerSegment.php

@@ -19,6 +19,7 @@
 use CustomerManagementFrameworkBundle\Listing\Filter\OnCreateQueryFilterInterface;
 use CustomerManagementFrameworkBundle\Service\MariaDb;
 use Doctrine\DBAL\Query\QueryBuilder;
+use InvalidArgumentException;
 use Pimcore\Model\DataObject;
 use Pimcore\Model\DataObject\Listing as CoreListing;
 
@@ -69,7 +70,7 @@ public function __construct(array $segments, DataObject\CustomerSegmentGroup $se
     {
         $this->identifier = $this->buildIdentifier($segmentGroup);
         $this->segmentGroup = $segmentGroup;
-        $this->type = $type;
+        $this->type = $type === self::OPERATOR_AND ? self::OPERATOR_AND : self::OPERATOR_OR;
 
         foreach ($segments as $segment) {
             $this->addCustomerSegment($segment);
@@ -119,7 +120,7 @@ protected function addCustomerSegment(DataObject\CustomerSegment $segment)
     {
         if ($segment->getGroup() && null !== $this->segmentGroup) {
             if ($segment->getGroup()->getId() !== $this->segmentGroup->getId()) {
-                throw new \InvalidArgumentException('Segment does not belong to the defined segment group');
+                throw new InvalidArgumentException('Segment does not belong to the defined segment group');
             }
         }
 
@@ -187,6 +188,7 @@ protected function applyAndQuery(CoreListing\Concrete $listing, QueryBuilder $qu
      *
      * @param string $joinName
      * @param int|array $conditionValue
+     *
      */
     protected function addJoin(
         CoreListing\Concrete $listing,
@@ -212,20 +214,24 @@ protected function addJoin(
         );
 
         $condition = $baseCondition;
-
         if ($this->type === self::OPERATOR_OR) {
-            // must match any of the passed IDs
+            $valueKeys = array_keys($conditionValue);
+            $isNumericArray = $valueKeys == array_filter($valueKeys, 'is_numeric');
+            if (!$isNumericArray) {
+                $valueKeys = [0];
+            }
+
             $condition .= sprintf(
                 ' AND %1$s.dest_id IN (%2$s)',
                 $joinName,
-                implode(',', $conditionValue)
+                implode(',', $valueKeys)
             );
         } else {
             // runs an extra join for every ID - all joins must match
             $condition .= sprintf(
-                ' AND %1$s.dest_id = %2$s',
+                ' AND %1$s.dest_id = %2$d',
                 $joinName,
-                $conditionValue
+                is_numeric($conditionValue) ? $conditionValue : 0
             );
         }
 

src/CustomerProvider/ObjectNamingScheme/DefaultObjectNamingScheme.php

@@ -152,15 +152,13 @@ private function extractNamingScheme(CustomerInterface $customer, $namingScheme)
         foreach ($namingScheme as $i => $namingSchemeItem) {
             preg_match_all('/{([a-zA-Z0-9]*)}/', $namingSchemeItem, $matchedPlaceholder);
 
-            if (sizeof($matchedPlaceholder)) {
-                foreach ($matchedPlaceholder[0] as $j => $placeholder) {
-                    $field = $matchedPlaceholder[1][$j];
-
-                    $getter = 'get'.ucfirst($field);
-                    if (method_exists($customer, $getter)) {
-                        $value = (string)$customer->$getter();
-                        $namingScheme[$i] = str_replace($placeholder, $value, $namingScheme[$i]);
-                    }
+            foreach ($matchedPlaceholder[0] as $j => $placeholder) {
+                $field = $matchedPlaceholder[1][$j];
+
+                $getter = 'get'.ucfirst($field);
+                if (method_exists($customer, $getter)) {
+                    $value = (string)$customer->$getter();
+                    $namingScheme[$i] = str_replace($placeholder, $value, $namingScheme[$i]);
                 }
             }
             $namingScheme[$i] = trim($namingScheme[$i]) ?: '--';