contentoh-components-library 21.6.51 → 21.6.53

package/src/components/pages/ProviderProductEdition/index.js

@@ -742,14 +742,64 @@ const ProviderProductEditionView = ({
     return soloPropagables(cambios);
   };
 
+  /*
+    Lo que se acaba de editar en la pestaña de descripciones.
+
+    Va aparte de `cambiosDeFichaTecnica` porque son tablas distintas y el
+    endpoint recibe una lista o la otra. Aquí NO se filtran UPC ni SKU: esos son
+    atributos de ficha técnica y no aparecen en esta pestaña.
+  */
+  const cambiosDeDescripciones = () => {
+    const retailerId = state.active_retailer?.id_retailer;
+    const inputs = state.services?.descriptions?.inputs ?? {};
+
+    return (state.updated_descriptions_inputs ?? []).map((item) => {
+      const input = inputs[`${item.attributeId}_${retailerId}`];
+      return {
+        id: Number(item.attributeId),
+        name: input?.name ?? `Descripción ${item.attributeId}`,
+        value: item.value,
+      };
+    });
+  };
+
+  /*
+    Destinos posibles para lo que se guarde en este producto: sus hijos y sus
+    compañeras de grupo. Un hijo no propaga a nadie.
+  */
+  const destinosPosibles = () => {
+    if (state.product?.parentArticleId) return 0;
+    const companeras = (variantGroup?.members ?? []).filter((m) => !m.isCurrent);
+    return (state.product?.variantsCount ?? 0) + companeras.length;
+  };
+
   const handleOnClickSave = () => {
     // `preparandoPropagacion` cubre la ventana entre el clic y la apertura del
     // diálogo: sin esto, dos clics seguidos lanzan dos consultas de familia.
     if (state.saving || preparandoPropagacion) return;
     switch (state.active_tab) {
-      case "Descripción":
+      case "Descripción": {
+        /*
+          Guardar una descripción en un padre también abre la decisión de
+          propagar. Sin esto, el hijo heredaba la descripción al ligarse y se
+          quedaba con la versión vieja en cuanto el padre la corregía, sin que
+          nadie lo notara.
+        */
+        const cambiosDesc = cambiosDeDescripciones();
+
+        if (destinosPosibles() > 0 && cambiosDesc.length > 0) {
+          setPreparandoPropagacion(true);
+          loadFamily()
+            .then(() =>
+              setPendingPropagation({ cambios: cambiosDesc, kind: "description" })
+            )
+            .finally(() => setPreparandoPropagacion(false));
+          return;
+        }
+
         saveDescriptions(token);
         break;
+      }
       case "Ficha técnica": {
         // Guardar un padre con hijos abre la decisión de propagar. Sin este
         // paso el cambio se queda solo en el padre y los hijos se van
@@ -788,7 +838,7 @@ const ProviderProductEditionView = ({
           // justo lo contrario de lo que va a preguntar.
           setPreparandoPropagacion(true);
           loadFamily()
-            .then(() => setPendingPropagation({ cambios }))
+            .then(() => setPendingPropagation({ cambios, kind: "datasheet" }))
             .finally(() => setPreparandoPropagacion(false));
           return;
         }
@@ -1686,6 +1736,7 @@ const ProviderProductEditionView = ({
         {pendingPropagation && (
           <PropagateChangesModal
             changedAttributes={pendingPropagation.cambios}
+            kind={pendingPropagation.kind ?? "datasheet"}
             variants={
               state.product?.parentArticleId
                 ? []
@@ -1698,8 +1749,29 @@ const ProviderProductEditionView = ({
             saving={state.saving}
             onCancel={() => setPendingPropagation(null)}
             onConfirm={(propagation) => {
+              const esDescripcion = pendingPropagation.kind === "description";
               setPendingPropagation(null);
-              saveDatasheets(token, propagation ?? undefined);
+
+              if (!propagation) {
+                // Guardar sin propagar: el producto se guarda solo.
+                if (esDescripcion) saveDescriptions(token);
+                else saveDatasheets(token);
+                return;
+              }
+
+              /*
+                El diálogo habla de «atributos» porque es el mismo componente
+                para las dos pestañas. Al guardar hay que ponerlo en la lista
+                que corresponde: el servidor recibe `attributes` o
+                `descriptions`, nunca una por la otra.
+              */
+              const { attributes, ...destinos } = propagation;
+
+              if (esDescripcion) {
+                saveDescriptions(token, { ...destinos, descriptions: attributes });
+              } else {
+                saveDatasheets(token, propagation);
+              }
             }}
           />
         )}

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

@@ -34,7 +34,7 @@ const ProductEditionContext = createContext();
 export const ProviderProductEditionProvider = ({ children }) => {
   const [state, dispatch] = useReducer(productEditionReducer, initialState);
 
-  const saveDescriptions = async (token) => {
+  const saveDescriptions = async (token, propagation) => {
     if (state.updated_descriptions_inputs.length === 0) return;
 
     dispatch({ type: "SET_SAVING", payload: true });
@@ -42,6 +42,7 @@ export const ProviderProductEditionProvider = ({ children }) => {
     const dataObject = {
       articleId: state.product?.id_article,
       articleData: state.updated_descriptions_inputs,
+      ...(propagation ? { propagation } : {}),
     };
 
     // console.log({dataObject});

package/src/components/pages/RetailerProductEdition/index.js

@@ -1270,13 +1270,68 @@ const RetailerProductEditionView = ({
     return soloPropagables(cambios);
   };
 
+  /*
+    Lo que se acaba de editar en la pestaña de descripciones.
+
+    Va aparte de `cambiosDeFichaTecnica` porque son tablas distintas y el
+    endpoint recibe una lista o la otra. Aquí NO se filtran UPC ni SKU: esos son
+    atributos de ficha técnica y no aparecen en esta pestaña.
+  */
+  const cambiosDeDescripciones = () => {
+    const retailerId = state.active_retailer?.id_retailer;
+    const inputs = state.services?.descriptions?.inputs ?? {};
+
+    return (state.updated_descriptions_inputs ?? []).map((item) => {
+      const input = inputs[`${item.attributeId}_${retailerId}`];
+      return {
+        id: Number(item.attributeId),
+        name: input?.name ?? `Descripción ${item.attributeId}`,
+        value: item.value,
+      };
+    });
+  };
+
   const handleOnClickSave = async () => {
     if (state.saving || preparandoPropagacion) return;
 
     switch (state.active_tab) {
-      case "Descripción":
+      case "Descripción": {
+        /*
+          Guardar una descripción en un padre también abre la decisión de
+          propagar. Sin esto, el hijo heredaba la descripción al ligarse y se
+          quedaba con la versión vieja en cuanto el padre la corregía, sin que
+          nadie lo notara.
+        */
+        const cambiosDesc = cambiosDeDescripciones();
+        const companerasDesc = (variantGroup?.members ?? []).filter(
+          (m) => !m.isCurrent
+        );
+
+        if (esHijoDeOtro || cambiosDesc.length === 0) {
+          await saveDescriptions(token);
+          break;
+        }
+
+        setPreparandoPropagacion(true);
+        let familiaDesc = null;
+        try {
+          familiaDesc = await loadFamily(true);
+        } finally {
+          setPreparandoPropagacion(false);
+        }
+
+        const hijosDesc = (familiaDesc?.variants ?? []).filter(
+          (v) => !v.isCurrent
+        );
+
+        if (hijosDesc.length > 0 || companerasDesc.length > 0) {
+          setPendingPropagation({ cambios: cambiosDesc, kind: "description" });
+          return;
+        }
+
         await saveDescriptions(token);
         break;
+      }
       case "Ficha técnica": {
         const cambios = cambiosDeFichaTecnica();
 
@@ -1368,7 +1423,7 @@ const RetailerProductEditionView = ({
         // Un hijo tampoco puede estar agrupado desde 2026-09-25, así que para
         // él no queda ningún destino posible.
         if (!esHijo && (hijos.length > 0 || companeras.length > 0)) {
-          setPendingPropagation({ cambios });
+          setPendingPropagation({ cambios, kind: "datasheet" });
           return;
         }
 
@@ -2131,6 +2186,7 @@ const RetailerProductEditionView = ({
         {pendingPropagation && (
           <PropagateChangesModal
             changedAttributes={pendingPropagation.cambios}
+            kind={pendingPropagation.kind ?? "datasheet"}
             variants={
               esHijoDeOtro
                 ? []
@@ -2143,8 +2199,28 @@ const RetailerProductEditionView = ({
             saving={state.saving}
             onCancel={() => setPendingPropagation(null)}
             onConfirm={(propagation) => {
+              const esDescripcion = pendingPropagation.kind === "description";
               setPendingPropagation(null);
-              saveDatasheets(token, propagation ?? undefined);
+
+              if (!propagation) {
+                if (esDescripcion) saveDescriptions(token);
+                else saveDatasheets(token);
+                return;
+              }
+
+              /*
+                El diálogo habla de «atributos» porque es el mismo componente
+                para las dos pestañas. Al guardar hay que ponerlo en la lista
+                que corresponde: el servidor recibe `attributes` o
+                `descriptions`, nunca una por la otra.
+              */
+              const { attributes, ...destinos } = propagation;
+
+              if (esDescripcion) {
+                saveDescriptions(token, { ...destinos, descriptions: attributes });
+              } else {
+                saveDatasheets(token, propagation);
+              }
             }}
           />
         )}