MM-29: Add functionality to map relation between website and sales_channel on the integration form (#54)

Author: alexandr-parkhomenko

src/Marello/Bundle/SalesBundle/Autocomplete/GroupSalesChannelHandler.php

@@ -22,45 +22,26 @@ protected function checkAllDependenciesInjected()
 
     /**
      * @param string $searchTerm
-     * @param int    $firstResult
-     * @param int    $maxResults
+     * @param int $firstResult
+     * @param int $maxResults
      * @return QueryBuilder
      */
     protected function prepareQueryBuilder($searchTerm, $firstResult, $maxResults)
     {
-        $queryBuilder = $this->entityRepository->createQueryBuilder('scg');
+        $queryBuilder = $this->entityRepository->getNonSystemSalesChannelBySearchTermQB($searchTerm);
         $queryBuilder
-            ->where($queryBuilder->expr()->like('LOWER(scg.name)', ':searchTerm'))
-            ->andWhere(($queryBuilder->expr()->eq('scg.system', $queryBuilder->expr()->literal(false))))
-            ->setParameter('searchTerm', '%' . mb_strtolower($searchTerm) . '%')
-            ->orderBy('scg.name', 'ASC')
             ->setFirstResult($firstResult)
             ->setMaxResults($maxResults);
 
         return $queryBuilder;
     }
 
     /**
-     * {@inheritdoc}
+     * {@inheritDoc}
      */
     protected function searchEntities($search, $firstResult, $maxResults)
     {
         $queryBuilder = $this->prepareQueryBuilder($search, $firstResult, $maxResults);
-        $query = $this->aclHelper->apply($queryBuilder, 'VIEW');
-
-        return $query->getResult();
-    }
-
-    /**
-     * {@inheritdoc}
-     */
-    protected function findById($query)
-    {
-        $parts = explode(';', $query);
-        $id = $parts[0];
-
-        $criteria = [$this->idFieldName => $id];
-
-        return [$this->entityRepository->findOneBy($criteria, null)];
+        return $this->aclHelper->apply($queryBuilder->getQuery())->getResult();
     }
 }

src/Marello/Bundle/SalesBundle/Entity/Repository/SalesChannelGroupRepository.php

@@ -3,6 +3,7 @@
 namespace Marello\Bundle\SalesBundle\Entity\Repository;
 
 use Doctrine\ORM\EntityRepository;
+use Doctrine\ORM\QueryBuilder;
 use Marello\Bundle\SalesBundle\Entity\SalesChannelGroup;
 use Oro\Bundle\IntegrationBundle\Entity\Channel as IntegrationChannel;
 use Oro\Bundle\SecurityBundle\ORM\Walker\AclHelper;
@@ -60,4 +61,20 @@ public function hasAttachedIntegration(int $salesChannelGroupId): bool
 
         return (bool) $qb->getQuery()->getResult();
     }
+
+    /**
+     * @param string $searchTerm
+     * @return QueryBuilder
+     */
+    public function getNonSystemSalesChannelBySearchTermQB(string $searchTerm): QueryBuilder
+    {
+        $qb = $this->createQueryBuilder('scg');
+        $qb
+            ->where($qb->expr()->like('LOWER(scg.name)', ':searchTerm'))
+            ->andWhere(($qb->expr()->eq('scg.system', $qb->expr()->literal(false))))
+            ->setParameter('searchTerm', '%' . mb_strtolower($searchTerm) . '%')
+            ->orderBy('scg.name', 'ASC');
+
+        return $qb;
+    }
 }

src/Marello/Bundle/SalesBundle/Entity/Repository/SalesChannelRepository.php

@@ -23,6 +23,34 @@ public function setAclHelper(AclHelper $aclHelper)
         $this->aclHelper = $aclHelper;
     }
 
+    /**
+     * @param string $searchTerm
+     * @param int $groupId
+     * @param array $skippedSalesChannelIds
+     * @return QueryBuilder
+     */
+    public function getActiveSalesChannelBySearchTermLimitedWithGroupIdQB(
+        string $searchTerm,
+        int $groupId,
+        array $skippedSalesChannelIds = []
+    ): QueryBuilder {
+        $qb = $this->getActiveChannelsQuery();
+        $qb
+            ->andWhere($qb->expr()->like('LOWER(sc.name)', ':searchTerm'))
+            ->andWhere($qb->expr()->eq('sc.group', ':salesChannelGroupId'))
+            ->setParameter('searchTerm', '%' . mb_strtolower($searchTerm) . '%')
+            ->setParameter('salesChannelGroupId', $groupId)
+            ->orderBy('sc.name', 'ASC');
+
+        if (!empty($skippedSalesChannelIds)) {
+            $qb
+                ->andWhere($qb->expr()->notIn('sc.id', ':skippedSalesChannelIds'))
+                ->setParameter('skippedSalesChannelIds', $skippedSalesChannelIds);
+        }
+
+        return $qb;
+    }
+
     /**
      * Return product prices for specified channel and productId
      *

src/Marello/Bundle/SalesBundle/Form/Type/SalesChannelType.php

@@ -5,7 +5,6 @@
 use Marello\Bundle\PricingBundle\Form\EventListener\CurrencySubscriber;
 use Marello\Bundle\SalesBundle\Entity\SalesChannel;
 use Oro\Bundle\CurrencyBundle\Form\Type\CurrencyType;
-use Oro\Bundle\FormBundle\Utils\FormUtils;
 use Oro\Bundle\LocaleBundle\Form\Type\LocalizationSelectType;
 use Symfony\Component\Form\AbstractType;
 use Symfony\Component\Form\Extension\Core\Type\CheckboxType;
@@ -70,10 +69,6 @@ public function preSetDataListener(FormEvent $event)
                 'label' => 'marello.sales.saleschannel.create_own_group.label'
             ]);
         }
-
-        if ($channel->getId()) {
-            FormUtils::replaceField($form, 'code', ['disabled' => true]);
-        }
     }
 
     /**

src/Marello/Bundle/SalesBundle/Resources/config/form.yml

@@ -11,7 +11,7 @@ services:
         class: 'Marello\Bundle\SalesBundle\Form\Type\SalesChannelGroupType'
         tags:
             - { name: form.type }
-             
+
     marello_sales_form_type.saleschannel_multi_select:
         class: 'Marello\Bundle\SalesBundle\Form\Type\SalesChannelMultiSelectType'
         arguments:

src/Marello/Bundle/SalesBundle/Resources/config/services.yml

@@ -47,7 +47,7 @@ services:
             - 'Marello\Bundle\SalesBundle\Entity\SalesChannelGroup'
             - ['name']
         tags:
-            - { name: oro_form.autocomplete.search_handler, alias: group_saleschannels }
+            - { name: oro_form.autocomplete.search_handler, alias: group_saleschannels, acl_resource: marello_product_view }
 
     marello_sales.active_saleschannels.form.autocomplete.search_handler:
         class: 'Marello\Bundle\SalesBundle\Autocomplete\ActiveSalesChannelHandler'