wrap product copy around a DB::transaction()

bagisto · Aug 5, 2020 · 73a210f · 73a210f
1 parent ebb6fa0
commit 73a210f
Showing 1 changed file with 15 additions and 3 deletions.
diff --git a/packages/Webkul/Product/src/Repositories/ProductRepository.php b/packages/Webkul/Product/src/Repositories/ProductRepository.php
@@ -484,11 +484,23 @@ public function copy(Product $originalProduct): Product
             throw new Exception(trans('admin::app.response.booking-can-not-be-copied'));
         }
 
-        $copiedProduct = $this->persistCopiedProduct($originalProduct);
+        DB::beginTransaction();
 
-        $this->persistAttributeValues($originalProduct, $copiedProduct);
+        try {
+            $copiedProduct = $this->persistCopiedProduct($originalProduct);
 
-        $this->persistRelations($originalProduct, $copiedProduct);
+            $this->persistAttributeValues($originalProduct, $copiedProduct);
+
+            $this->persistRelations($originalProduct, $copiedProduct);
+        } catch (Exception $e) {
+            DB::rollBack();
+
+            report($e);
+
+            throw $e;
+        }
+
+        DB::commit();
 
         return $copiedProduct;
     }