contentoh-components-library 21.6.50 → 21.6.51

package/src/components/pages/RetailerProductEdition/context/provider-product-edition.context.jsx

@@ -13,13 +13,6 @@ import successIcon from "../../../../assets/images/genericModal/genericModalChec
 import errorIcon from "../../../../assets/images/genericModal/errorModal.svg";
 import warningIcon from "../../../../assets/images/genericModal/genericModalWarning.svg";
 import { getConceptByTab } from "../utils";
-import {
-  assetKey,
-  buildSavePayload,
-  isPersisted,
-  toPayload,
-} from "../../../../global-files/digital-assets/assets.adapter";
-import { selectAssetsToSave } from "../../../../global-files/digital-assets/assets.reducer";
 
 const S3_BUCKET = process.env.REACT_APP_IMAGES_BUCKET;
 const REGION = "us-east-1";
@@ -561,198 +554,6 @@ export const ProviderProductEditionProvider = ({ children }) => {
     dispatch({ type: "SET_SAVING", payload: false });
   };
 
-  /*
-    Registra en la base los activos que ACABAN de subirse, sin esperar a que el
-    usuario guarde.
-
-    Es una corrección de expectativa, no una comodidad: al ver la barra de
-    progreso llegar al final, cualquiera da por hecho que el activo ya quedó
-    ligado al producto. Si entonces cierra la ficha, el archivo se queda en S3
-    sin fila en la base —invisible y sin forma de recuperarlo desde la
-    pantalla—.
-
-    Manda SOLO las altas recién subidas, con `updateAssets` y `deleteAssets`
-    vacíos. Un reetiquetado o un borrado que el usuario todavía no confirmó no
-    se aplica por haber soltado otro archivo: el botón de guardar sigue siendo
-    quien decide sobre esos.
-  */
-  const registerUploadedAssets = async (token, uploaded = []) => {
-    const listos = uploaded.filter((asset) => asset.src && !isPersisted(asset));
-    if (listos.length === 0) return;
-
-    const data = {
-      articleId: state.product?.id_article,
-      newAssets: listos.map(toPayload),
-      updateAssets: [],
-      deleteAssets: [],
-    };
-
-    if (state.product?.orderId || state.product?.id_order) {
-      data.orderId = state.product?.orderId || state.product?.id_order;
-    }
-
-    try {
-      const res = await axios.put(
-        process.env.REACT_APP_ARTICLE_DATA_ENDPOINT,
-        data,
-        { params: { asset: true }, headers: { Authorization: token } },
-      );
-
-      if (res.data.statusCode !== 200) throw new Error("Respuesta inesperada");
-
-      const body = JSON.parse(res.data.body);
-      const guardados = body.assets ?? [];
-
-      /*
-        Se le pone a cada activo el id que le tocó, emparejando por la clave de
-        S3, que es única.
-
-        A propósito NO se reemplaza la lista entera con lo que devolvió el
-        backend: eso borraría el rastro de lo que el usuario cambió o borró y
-        todavía no ha guardado. Aquí solo se completa lo que faltaba.
-      */
-      listos.forEach((asset) => {
-        const fila = guardados.find((row) => row.src === asset.src);
-        if (!fila) return;
-        dispatch({
-          type: "PATCH_ASSET",
-          payload: { key: assetKey(asset), changes: { id: fila.id, url: fila.url } },
-        });
-      });
-    } catch (error) {
-      console.log("Error registering assets:", error);
-      /*
-        El archivo SÍ está en S3; lo que falló es la fila. Se marca en el activo
-        y no en un modal porque el usuario puede estar soltando más archivos:
-        interrumpirlo con un diálogo por cada fallo sería peor que dejarlo
-        señalado en su renglón, donde además se ve cuál falló.
-      */
-      listos.forEach((asset) => dispatch({
-        type: "PATCH_ASSET",
-        payload: {
-          key: assetKey(asset),
-          changes: {
-            status: "error",
-            error: "Se subió el archivo pero no se pudo guardar. Usa el botón de guardar.",
-          },
-        },
-      }));
-    }
-  };
-
-  /*
-    Guarda los activos digitales: registra los nuevos, manda los cambios y pide
-    las bajas.
-
-    Va aparte de `updateImages` y no dentro, aunque las dos bandejas vivan en la
-    misma pestaña. Un activo no pasa por ninguna de las reglas de una imagen —ni
-    tipo de toma, ni obligatoriedad, ni cupo por cadena— y el backend los escribe
-    en otra tabla. Mezclarlos obligaría a partir en dos cada validación de
-    `updateImages`, que ya es la función más delicada de este archivo.
-
-    Igual que con las imágenes, aquí NO viaja ningún binario: el archivo ya está
-    en S3 y esto solo registra metadatos. Mandar el archivo en el cuerpo devuelve
-    413.
-  */
-  const saveAssets = async (token) => {
-    const payload = buildSavePayload(selectAssetsToSave(state));
-    const { newAssets, updateAssets, deleteAssets } = payload;
-
-    if (!newAssets.length && !updateAssets.length && !deleteAssets.length) {
-      return false;
-    }
-
-    /*
-      Un activo nuevo sin `key` es un archivo que todavía no llegó a S3.
-      Registrarlo crearía una fila apuntando a un objeto que no existe: la
-      bandeja lo mostraría y al descargarlo daría 404, sin forma de saber por qué.
-      Mejor no guardar nada y decirlo.
-    */
-    const sinSubir = newAssets.filter((asset) => !asset.key);
-    if (sinSubir.length > 0) {
-      dispatch({
-        type: "SET_MODAL",
-        payload: {
-          show: true,
-          title: "Faltan archivos por subir",
-          message: `${sinSubir.length} activo(s) todavía no se han subido. Espera a que termine la carga e intenta de nuevo.`,
-          image: warningIcon,
-        },
-      });
-      return false;
-    }
-
-    dispatch({ type: "SET_SAVING", payload: true });
-
-    // Sin `version`: un activo general es el mismo para todas las versiones del
-    // producto, así que ni la fila ni la clave de S3 la llevan.
-    const data = {
-      articleId: state.product?.id_article,
-      newAssets,
-      updateAssets,
-      deleteAssets,
-    };
-
-    if (state.product?.orderId || state.product?.id_order) {
-      data.orderId = state.product?.orderId || state.product?.id_order;
-    }
-
-    try {
-      const res = await axios.put(
-        process.env.REACT_APP_ARTICLE_DATA_ENDPOINT,
-        data,
-        {
-          params: { asset: true },
-          headers: { Authorization: token },
-        },
-      );
-
-      if (res.data.statusCode === 200) {
-        // Se olvida qué cambió y qué se borró: lo que hay en pantalla ya es lo
-        // que hay en la base. Sin esto, el siguiente guardado reenviaría todo.
-        dispatch({ type: "CLEAR_ASSETS_TRACKING" });
-        dispatch({
-          type: "SET_MODAL",
-          payload: {
-            show: true,
-            title: "",
-            message: "Activos digitales guardados con éxito",
-            image: successIcon,
-          },
-        });
-        /*
-          Devuelve `true` para que quien llama recargue.
-
-          Es imprescindible, no cortesía: los activos recién creados siguen en
-          memoria con su id temporal y sin el id de la base. Sin recargar, el
-          siguiente guardado los volvería a mandar como nuevos y se duplicarían.
-          Es exactamente el problema que ya tuvieron las imágenes.
-        */
-        return true;
-      }
-
-      return false;
-    } catch (error) {
-      console.log("Error saving assets:", error);
-      dispatch({
-        type: "SET_ERRORS",
-        payload: [...state.errors, error],
-      });
-      dispatch({
-        type: "SET_MODAL",
-        payload: {
-          show: true,
-          title: "Error",
-          message: "Hubo un error al guardar los activos digitales",
-          image: errorIcon,
-        },
-      });
-      return false;
-    } finally {
-      dispatch({ type: "SET_SAVING", payload: false });
-    }
-  };
-
   const createComment = async (body, token) => {
     const concept = getConceptByTab(state.active_tab);
     const data = {
@@ -781,8 +582,6 @@ export const ProviderProductEditionProvider = ({ children }) => {
     updateImages,
     saveImageAttrs,
     deleteImages,
-    saveAssets,
-    registerUploadedAssets,
     handleOnDownloadImages,
     createComment,
   };

package/src/components/pages/RetailerProductEdition/context/provider-product-edition.reducer.js

@@ -18,11 +18,6 @@ import {
   activeStateInitialState,
   activeStateReducer,
 } from "./reducers/active-state";
-import { ASSET_ACTIONS } from "../../../../global-files/digital-assets/assets.actions";
-import {
-  assetsInitialState,
-  assetsReducer,
-} from "../../../../global-files/digital-assets/assets.reducer";
 
 // Initial state
 export const initialState = {
@@ -30,7 +25,6 @@ export const initialState = {
   ...productInitialState,
   ...inputsInitialState,
   ...activeStateInitialState,
-  ...assetsInitialState,
 };
 
 // Action types
@@ -39,7 +33,6 @@ export const ACTIONS = {
   ...PRODUCT_ACTIONS,
   ...INPUTS_ACTIONS,
   ...ACTIVE_STATE_ACTIONS,
-  ...ASSET_ACTIONS,
 };
 
 // Reducer function
@@ -65,10 +58,5 @@ export const productEditionReducer = (state, action) => {
     return activeStateResult;
   }
 
-  const assetsResult = assetsReducer(state, action);
-  if (assetsResult !== null) {
-    return assetsResult;
-  }
-
   return state;
 };