Merge PR #23 from @stefanheimes

Fixing a bug that caused all data to be copied twice.

Author: stefanheimes

src/EventListener/BackendEventListener.php

@@ -26,8 +26,10 @@
 use Contao\System;
 use ContaoCommunityAlliance\DcGeneral\Event\PostDuplicateModelEvent;
 use ContaoCommunityAlliance\DcGeneral\Event\PostPasteModelEvent;
+use Doctrine\DBAL\ArrayParameterType;
 use Doctrine\DBAL\Connection;
 use MetaModels\DcGeneral\Data\Model;
+use MetaModels\IFactory;
 
 /**
  * Handles event operations on tl_metamodel_dcasetting
@@ -50,13 +52,20 @@ class BackendEventListener
      */
     private Connection $connection;
 
+    /**
+     * @var IFactory
+     */
+    private IFactory $factory;
+
     /**
      * The ArticleContent constructor.
      *
      * @param Connection $connection The database connection.
      */
-    public function __construct(Connection $connection = null)
-    {
+    public function __construct(
+        IFactory $factory,
+        Connection $connection = null,
+    ) {
         if (null === $connection) {
             // @codingStandardsIgnoreStart
             @trigger_error(
@@ -68,6 +77,7 @@ public function __construct(Connection $connection = null)
             assert($connection instanceof Connection);
         }
         $this->connection = $connection;
+        $this->factory = $factory;
     }
 
     /**
@@ -130,14 +140,33 @@ public function handlePostPasteModel(PostPasteModelEvent $event)
      */
     private function duplicateContentEntries(string $strTable, int $intSourceId, int $intDestinationId): void
     {
+        $metaModel = $this->factory->getMetaModel($strTable);
+        if ($metaModel === null) {
+            return;
+        }
+
+        $colNamesToCopy = [];
+        $attributes = $metaModel->getAttributes();
+        foreach ($attributes as $attribute) {
+            if ($attribute->get('type') === 'contentarticle') {
+                $colNamesToCopy[] = $attribute->getColName();
+            }
+        }
+
+        if (empty($colNamesToCopy)) {
+            return;
+        }
+
         $objContent = $this->connection
             ->createQueryBuilder()
             ->select('*')
             ->from('tl_content', 't')
             ->where('t.pid=:id')
             ->andWhere('t.ptable=:ptable')
+            ->andWhere('t.mm_slot IN (:slots)')
             ->setParameter('id', $intSourceId)
             ->setParameter('ptable', $strTable)
+            ->setParameter('slots', $colNamesToCopy, ArrayParameterType::STRING)
             ->executeQuery();
 
         while ($row = $objContent->fetchAssociative()) {

src/Resources/config/listeners.yml

@@ -2,6 +2,7 @@ services:
   MetaModels\AttributeContentArticleBundle\EventListener\BackendEventListener:
     public: false
     arguments:
+      - '@metamodels.factory'
       - '@database_connection'
     tags:
       - name: kernel.event_listener