@base44/app-plugin-commerce 0.1.13 → 0.1.15

package/src/commerce/utils/variants.js

@@ -264,13 +264,31 @@ export function priceRange(product, variations, selection = {}) {
   return { min: Math.min(...prices), max: Math.max(...prices), on_sale: onSale, count: prices.length };
 }
 
+/**
+ * Normalize a stored image entry to the one shape the UI renders. Stored images
+ * are not uniform — product images carry `position`, variation images don't,
+ * and no sub-field is required by the schema — so this guarantees consumers
+ * `{src, name, alt}` with a non-empty `src`, or `null` when there is nothing
+ * renderable. Run gallery entries through it too: `product.images.map(normalizeImage)`.
+ *
+ * @param {object|null|undefined} img a stored image entry (`product.images[n]` or `variation.image`)
+ * @returns {{src: string, name: string, alt: string}|null}
+ */
+export function normalizeImage(img) {
+  const src = typeof img?.src === "string" ? img.src.trim() : "";
+  if (!src) return null;
+  const name = img.name ?? "";
+  return { src, name, alt: img.alt || name };
+}
+
 /**
  * The fields to display for the current state, resolved variation-first with a
  * fallback to the parent. `image` is the variation's own image when it has one
  * — highlight it in the gallery rather than replacing the gallery.
  *
- * `image` is `{src, name, alt}`, or `null` when the product has no images at all
- * — the one case needing a UI placeholder.
+ * `image` is **always** the normalized `{src, name, alt}` (never a raw stored
+ * entry — see `normalizeImage`), or `null` when neither the variation nor the
+ * product has a renderable image — the one case needing a UI placeholder.
  *
  * @returns {{price: number|null, regular_price: number|null, sale_price: number|null,
  *            on_sale: boolean, sku: string, stock_status: string,
@@ -296,7 +314,7 @@ export function displayFields(product, variation) {
       ? (variation.manage_stock === "yes" ? (variation.stock_quantity ?? null) : (product?.stock_quantity ?? null))
       : (product?.stock_quantity ?? null),
     backorders: pick("backorders", "no"),
-    image: variation?.image?.src ? variation.image : (product?.images?.[0] ?? null),
+    image: normalizeImage(variation?.image) ?? normalizeImage(product?.images?.[0]),
     weight: pick("weight"),
     dimensions: pick("dimensions"),
     description: variation?.description || product?.description || "",