Merge branch 'MAUT-11602' into development

acquia · Jun 25, 2024 · 8a1fae0 · 8a1fae0
2 parents 736d890 + 850a962
commit 8a1fae0
Show file tree

Hide file tree

Showing 4 changed files with 51 additions and 22 deletions.
diff --git a/CustomFieldType/DateOperatorTrait.php b/CustomFieldType/DateOperatorTrait.php
@@ -0,0 +1,33 @@
+<?php
+
+declare(strict_types=1);
+
+namespace MauticPlugin\CustomObjectsBundle\CustomFieldType;
+
+trait DateOperatorTrait
+{
+    /**
+     * Remove operators that are supported only by segment filters.
+     *
+     * @return string[]
+     */
+    public function getOperatorOptions(): array
+    {
+        $options = parent::getOperatorOptions();
+
+        unset($options['between'], $options['!between'], $options['inLast'], $options['inNext']);
+
+        return $options;
+    }
+
+    /**
+     * @return mixed[]
+     */
+    public function getOperators(): array
+    {
+        $allOperators     = parent::getOperators();
+        $allowedOperators = array_flip(['=', '!=', 'gt', 'gte', 'lt', 'lte', 'empty', '!empty', 'between', '!between', 'inLast', 'inNext']);
+
+        return array_intersect_key($allOperators, $allowedOperators);
+    }
+}
diff --git a/CustomFieldType/DateTimeType.php b/CustomFieldType/DateTimeType.php
@@ -17,6 +17,8 @@
 
 class DateTimeType extends AbstractCustomFieldType
 {
+    use DateOperatorTrait;
+
     /**
      * @var string
      */
@@ -70,17 +72,6 @@ public function getEntityClass(): string
         return CustomFieldValueDateTime::class;
     }
 
-    /**
-     * @return mixed[]
-     */
-    public function getOperators(): array
-    {
-        $allOperators     = parent::getOperators();
-        $allowedOperators = array_flip(['=', '!=', 'gt', 'gte', 'lt', 'lte', 'empty', '!empty', 'between', '!between', 'inLast', 'inNext']);
-
-        return array_intersect_key($allOperators, $allowedOperators);
-    }
-
     /**
      * {@inheritdoc}
      */

diff --git a/CustomFieldType/DateType.php b/CustomFieldType/DateType.php
@@ -16,6 +16,8 @@
 
 class DateType extends AbstractCustomFieldType
 {
+    use DateOperatorTrait;
+
     /**
      * @var string
      */
@@ -70,17 +72,6 @@ public function getEntityClass(): string
         return CustomFieldValueDate::class;
     }
 
-    /**
-     * @return mixed[]
-     */
-    public function getOperators(): array
-    {
-        $allOperators     = parent::getOperators();
-        $allowedOperators = array_flip(['=', '!=', 'gt', 'gte', 'lt', 'lte', 'empty', '!empty', 'between', '!between', 'inLast', 'inNext']);
-
-        return array_intersect_key($allOperators, $allowedOperators);
-    }
-
     /**
      * {@inheritdoc}
      */

diff --git a/CustomFieldType/IntType.php b/CustomFieldType/IntType.php
@@ -41,6 +41,20 @@ public function createValueEntity(CustomField $customField, CustomItem $customIt
         return new CustomFieldValueInt($customField, $customItem, (int) $value);
     }
 
+    /**
+     * Remove operators that are supported only by segment filters.
+     *
+     * @return string[]
+     */
+    public function getOperatorOptions(): array
+    {
+        $options = parent::getOperatorOptions();
+
+        unset($options['between'], $options['!between']);
+
+        return $options;
+    }
+
     /**
      * @return mixed[]
      */