Merge branch 'MAUT-11616' into development

Author: rahuld-dev

Form/Type/CampaignConditionFieldValueType.php

@@ -52,6 +52,20 @@ public function buildForm(FormBuilderInterface $builder, array $options): void
             $choices[$field->getLabel()] = $field->getId();
         }
 
+        $optionAttr = array_combine(
+            array_map(fn ($field) => $field->getLabel(), $fields),
+            array_map(
+                function ($field) {
+                    return [
+                        'data-operators'  => json_encode($field->getTypeObject()->getOperatorOptions()),
+                        'data-options'    => json_encode($field->getChoices()),
+                        'data-field-type' => $field->getType(),
+                    ];
+                },
+                $fields
+            )
+        );
+
         $builder->add(
             'field',
             ChoiceType::class,
@@ -62,16 +76,7 @@ public function buildForm(FormBuilderInterface $builder, array $options): void
                 'attr'     => [
                     'class' => 'form-control',
                 ],
-                'choice_attr' => array_map(
-                    function ($field) {
-                        return [
-                        'data-operators'  => json_encode($field->getTypeObject()->getOperatorOptions()),
-                        'data-options'    => json_encode($field->getChoices()),
-                        'data-field-type' => $field->getType(),
-                    ];
-                    },
-                    $fields
-                ),
+                'choice_attr' => $optionAttr,
             ]
         );
 

Tests/Functional/EventListener/CampaignConditionTest.php

@@ -67,4 +67,35 @@ public function testConditionForm(): void
         Assert::assertSame('=', $body['event']['properties']['properties']['operator']);
         Assert::assertSame('unicorn', $body['event']['properties']['properties']['value']);
     }
+
+    public function testVerifyDataOperatorAttrIsAvailableForFields(): void
+    {
+        $customObject = $this->createCustomObjectWithAllFields(self::$container, 'Campaign test object');
+        $crawler      = $this->client->request(
+            Request::METHOD_GET,
+            's/campaigns/events/new',
+            [
+                'campaignId'      => 'mautic_041b2a401f680fb0b644654af5ba0892f31f0697',
+                'type'            => "custom_item.{$customObject->getId()}.fieldvalue",
+                'eventType'       => 'condition',
+                'anchor'          => 'leadsource',
+                'anchorEventType' => 'source',
+            ],
+            [],
+            $this->createAjaxHeaders()
+        );
+
+        Assert::assertTrue($this->client->getResponse()->isOk(), $this->client->getResponse()->getContent());
+
+        $html = json_decode($this->client->getResponse()->getContent(), true)['newContent'];
+
+        $crawler->addHtmlContent($html);
+
+        $options = $crawler->filter('#campaignevent_properties_field')->filter('option'); // ->attr('data-operators');
+
+        /** @var \DOMElement $option */
+        foreach ($options as $option) {
+            Assert::assertNotEmpty($option->getAttribute('data-operators'));
+        }
+    }
 }

Tests/Unit/Form/Type/CampaignConditionFieldValueTypeTest.php

@@ -102,7 +102,7 @@ public function testBuildForm(): void
                             'class' => 'form-control',
                         ],
                         'choice_attr' => [
-                            42 => [
+                            'Cheese' => [
                                 'data-operators'  => '{"=":"a","!=":"b"}',
                                 'data-options'    => '[]',
                                 'data-field-type' => 'int',