Merge pull request #176 from gnat42/patch-7

split out the code that updates an entity from writeItem function

src/Ddeboer/DataImport/Writer/DoctrineWriter.php

@@ -192,7 +192,25 @@ public function writeItem(array $item)
         $entity = $this->findOrCreateItem($item);
 
         $this->loadAssociationObjectsToEntity($item, $entity);
+        $this->updateEntity($item, $entity);
 
+        $this->entityManager->persist($entity);
+
+        if (($this->counter % $this->batchSize) == 0) {
+            $this->entityManager->flush();
+            $this->entityManager->clear($this->entityName);
+        }
+
+        return $this;
+    }
+
+    /**
+     * 
+     * @param array $item
+     * @param object $entity
+     */
+    protected function updateEntity(array $item, $entity)
+    {
         $fieldNames = array_merge($this->entityMetadata->getFieldNames(), $this->entityMetadata->getAssociationNames());
         foreach ($fieldNames as $fieldName) {
 
@@ -213,16 +231,7 @@ public function writeItem(array $item)
                 $setter = 'set' . ucfirst($fieldName);
                 $this->setValue($entity, $value, $setter);
             }
-        }
-
-        $this->entityManager->persist($entity);
-
-        if (($this->counter % $this->batchSize) == 0) {
-            $this->entityManager->flush();
-            $this->entityManager->clear($this->entityName);
-        }
-
-        return $this;
+        }        
     }
 
     /**