BB-19484: Email campaign sending cusumer error (#28154)

- fix Account contactInformation virtual field configuration
- fix query converter for case when two more joins, one of which related to another one added in the configuration of the single virtual field
- add Account Contact information as an "email" contact information to allow to use it in the marketing campaigns
- add functional test
- add behat tests
- fix unit tests
- fix data fixture
- fix bc break in query converter

Author: Dmitriev Alexandr

src/Oro/Bundle/ActionBundle/Provider/OriginalUrlProvider.php

@@ -47,11 +47,15 @@ public function __construct(
      *
      * @param ButtonSearchContext|null $buttonSearchContext
      *
-     * @return string
+     * @return string|null
      */
-    public function getOriginalUrl(ButtonSearchContext $buttonSearchContext = null)
+    public function getOriginalUrl(ButtonSearchContext $buttonSearchContext = null): ?string
     {
         $originalUrl = $this->getMasterRequestUri();
+        if (null === $originalUrl) {
+            return null;
+        }
+
         if (null === $buttonSearchContext) {
             return $originalUrl;
         }
@@ -70,10 +74,23 @@ public function getOriginalUrl(ButtonSearchContext $buttonSearchContext = null)
     }
 
     /**
-     * @return string
+     * @return string|null
      */
-    private function getMasterRequestUri()
+    private function getMasterRequestUri(): ?string
     {
-        return $this->requestStack->getMasterRequest()->getRequestUri();
+        $request = $this->requestStack->getMasterRequest();
+
+        /**
+         * Based on the "requestStack" contract "MasterRequest" could be "null". it could happens in case
+         * if this service is called during the console command work.
+         *
+         * Example of such case is the building of the MarketingList datasource, where datagrid used as a source of
+         * the data.
+         */
+        if (!$request) {
+            return null;
+        }
+
+        return $request->getRequestUri();
     }
 }

src/Oro/Bundle/ActionBundle/Tests/Unit/Provider/OriginalUrlProviderTest.php

@@ -45,6 +45,11 @@ public function testGetOriginalUrl()
         $this->assertEquals('example.com', $this->urlProvider->getOriginalUrl());
     }
 
+    public function testGetOriginalUrlReturnNullIfRequestIsNotDefined()
+    {
+        $this->assertNull($this->urlProvider->getOriginalUrl());
+    }
+
     public function testGetOriginalUrlWhenDatagridIsSet()
     {
         $datagridName = 'quotes-grid';

src/Oro/Bundle/QueryDesignerBundle/QueryDesigner/AbstractQueryConverter.php

@@ -1333,7 +1333,11 @@ protected function registerVirtualColumnQueryAliases($query, $mainEntityJoinId)
                     }
                     $joinType     = $mapItem['type'];
                     $join         = $query['join'][$joinType][$mapItem['key']];
-                    $parentJoinId = $this->getParentJoinIdForVirtualColumnJoin($join['join'], $mainEntityJoinId);
+                    $parentJoinId = $this->getParentJoinIdForVirtualColumn(
+                        $joinMap,
+                        $join['join'],
+                        $mainEntityJoinId
+                    );
                     if (null !== $parentJoinId) {
                         $alias    = $join['alias'];
                         $joinId   = $this->buildJoinIdentifier($join, $parentJoinId, $joinType);
@@ -1348,7 +1352,8 @@ protected function registerVirtualColumnQueryAliases($query, $mainEntityJoinId)
                         if (!isset($mapItem['processed'])) {
                             $joinType     = $mapItem['type'];
                             $join         = $query['join'][$joinType][$mapItem['key']];
-                            $parentJoinId = $this->getParentJoinIdForVirtualColumnJoin(
+                            $parentJoinId = $this->getParentJoinIdForVirtualColumn(
+                                $joinMap,
                                 $join['join'],
                                 $mainEntityJoinId
                             );
@@ -1374,6 +1379,32 @@ protected function registerVirtualColumnQueryAliases($query, $mainEntityJoinId)
                         ['processed' => true, 'joinId' => $joinId]
                     );
                     unset($joinMap[$alias]);
+
+                    /**
+                     * It is required to update parentAliases in the map with the new alias,
+                     * because there could be a case when one of the joins use another one and in this case
+                     * second join alias will start to be invalid after replacement.
+                     *
+                     * Example of such virtual field con figuration:
+                     * select:
+                     *     expr:         defaultContactEmails.email
+                     *     return_type:  string
+                     * join:
+                     *     left:
+                     *         - { join: entity.defaultContact, alias: defaultContact }
+                     *         - {
+                     *             join: defaultContact.emails,
+                     *             alias: defaultContactEmails,
+                     *             conditionType: 'WITH',
+                     *             condition: 'defaultContactEmails.primary = true'
+                     *           }
+                     * Example of $joinMap value for this configuration of the virtual field above:
+                     *      [
+                     *          '__tmp1__' => ['['type'=> left, 'key' => 0, 'parentAlias' => t1],
+                     *          '__tmp2__' => ['['type'=> left, 'key' => 1, 'parentAlias' => __tmp1__],
+                     *      ]
+                     */
+                    $joinMap = $this->replaceParentAliasInJoinMapItems($joinMap, $alias, $newAlias);
                 }
             }
         }
@@ -1402,6 +1433,36 @@ protected function findExistingTableAlias($virtualColumnJoinId, $virtualColumnJo
         return $foundAlias;
     }
 
+    /**
+     * Method added to avoid problem with backward compatibility and will be removed in the next release.
+     * Instead signature of the method:
+     * @see \Oro\Bundle\QueryDesignerBundle\QueryDesigner\AbstractQueryConverter::getParentJoinIdForVirtualColumnJoin
+     * will be changed to fix a bug: $joinMap parameter was not passed to the function but it is used in it to get
+     * correct parent join id from the already processed joins
+     *
+     * @param array  $joinMap
+     * @param string $joinExpr
+     * @param string $mainEntityJoinId
+     *
+     * @return string|null
+     */
+    private function getParentJoinIdForVirtualColumn($joinMap, $joinExpr, $mainEntityJoinId)
+    {
+        $parentJoinId = $this->getParentJoinIdForVirtualColumnJoin($joinExpr, $mainEntityJoinId);
+        if (!$parentJoinId) {
+            $parts = explode('.', $joinExpr, 2);
+
+            $parentAlias = count($parts) === 2
+                ? $parts[0]
+                : null;
+            if (isset($joinMap[$parentAlias]['processed'])) {
+                $parentJoinId = $joinMap[$parentAlias]['joinId'];
+            }
+        }
+
+        return $parentJoinId;
+    }
+
     /**
      * @param string $joinExpr
      * @param string $mainEntityJoinId
@@ -1494,6 +1555,24 @@ protected function replaceTableAliasesInVirtualColumnQuery(&$query, $aliases)
         $query['select']['expr'] = QueryUtils::replaceTableAliasesInSelectExpr($query['select']['expr'], $aliases);
     }
 
+    /**
+     * Actualize parent alias
+     *
+     * @param array  $joinMap
+     * @param string $from
+     * @param string $to
+     */
+    private function replaceParentAliasInJoinMapItems(array $joinMap, string $from, string $to): array
+    {
+        foreach ($joinMap as &$mapItem) {
+            if (isset($mapItem['parentAlias']) && $mapItem['parentAlias'] === $from) {
+                $mapItem['parentAlias'] = $to;
+            }
+        }
+
+        return $joinMap;
+    }
+
     /**
      * Replaces all table aliases declared in the virtual column query with unique aliases for built query
      *