fix(product, dashboard): manual cleanup of uploaded images (#10254)

* manual cleanup

* add changeset

* remove thumbnail on deletion

.changeset/shy-gorillas-allow.md

@@ -0,0 +1,6 @@
+---
+"@medusajs/dashboard": patch
+"@medusajs/product": patch
+---
+
+fix(product,dashboard): Avoid duplicating images

packages/admin/dashboard/src/routes/products/product-media/components/edit-product-media-form/edit-product-media-form.tsx

@@ -132,8 +132,8 @@ export const EditProductMediaForm = ({ product }: ProductMediaViewProps) => {
 
     await mutateAsync(
       {
-        images: withUpdatedUrls.map((file) => ({ url: file.url })),
+        images: withUpdatedUrls.map((file) => ({ url: file.url, id: file.id })),
-        thumbnail: thumbnail,
+        thumbnail: thumbnail || null,
       },
       {
         onSuccess: () => {

packages/modules/product/src/migrations/Migration20241122120331.ts

@@ -19,6 +19,12 @@ export class Migration20241122120331 extends Migration {
       where pi.image_id = i.id;
     `);
 
+    // Delete orphaned images
+    this.addSql(`
+      delete from "image"
+      where product_id is null;
+    `);
+
     this.addSql('alter table if exists "image" alter column "product_id" set not null;');
     this.addSql('alter table if exists "image" add constraint "image_product_id_foreign" foreign key ("product_id") references "product" ("id") on update cascade on delete cascade;');
     this.addSql('drop table if exists "product_images" cascade;');

packages/modules/product/src/services/product-module-service.ts

@@ -1690,6 +1690,16 @@ export default class ProductModuleService
                 sharedContext
               )
             upsertedProduct.images = productImages
+
+            await this.productImageService_.delete(
+              {
+                product_id: upsertedProduct.id,
+                id: {
+                  $nin: productImages.map(({ id }) => id),
+                },
+              },
+              sharedContext
+            )
           } else {
             await this.productImageService_.delete(
               { product_id: upsertedProduct.id },