Merge pull request #3 from AymenMech/fix/multiple_flush_causing_latency

Fix - Multiple flush causing latency
hmonglee · Sep 9, 2021 · e446d62 · e446d62
2 parents 33bda7c + a571047
commit e446d62
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 2 deletions.
diff --git a/src/Doctrine.php b/src/Doctrine.php
@@ -94,6 +94,7 @@ public function import(MessageCatalogueInterface $catalogue, array $options = []
                 $message = new Message($key, $domain, $locale, $translation);
                 $this->create($message);
             }
+            $this->manager->flush();
         }
     }
 }
diff --git a/src/Service/TranslationManager.php b/src/Service/TranslationManager.php
@@ -83,7 +83,6 @@ public function createTranslation(MessageInterface $message)
             ->setStatus($message->getKey() === $message->getTranslation() ? Translation::STATUS_DRAFT : Translation::STATUS_PUBLISHED);
 
         $this->entityManager->persist($translation);
-        $this->entityManager->flush();
     }
 
     /**
@@ -100,7 +99,6 @@ public function updateTranslation(MessageInterface $message)
         }
 
         $translation->setTranslation($message->getTranslation());
-        $this->entityManager->flush();
     }
 
     /**
@@ -136,6 +134,11 @@ private function findOneBy($locale, $domain, $key)
         ]);
     }
 
+    public function flush(): void
+    {
+        $this->entityManager->flush();
+    }
+
     /**
      * @return ObjectRepository
      */