contentoh-components-library 21.6.32 → 21.6.34

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

@@ -181,7 +181,7 @@ const RetailerProductEditionView = ({
   const [statusArray, setStatusArray] = useState([]);
   const [rejectAll, setRejectAll] = useState(false);
 
-  // Variantes: mismo comportamiento que la ficha de la aplicación de clientes.
+  // Padre/hijo: mismo comportamiento que la ficha de la aplicación de clientes.
   const [showLinkVariants, setShowLinkVariants] = useState(false);
   const [family, setFamily] = useState(null);
   const [showFamily, setShowFamily] = useState(false);
@@ -782,7 +782,7 @@ const RetailerProductEditionView = ({
   };
 
   /*
-    Trae la familia: las variantes del producto, o sus hermanas si el abierto es
+    Trae la familia: los hijos del producto, o sus hermanos si el abierto es
     una de ellas. Se pide al abrir la sección y no al montar la pantalla: la
     mayoría de los productos no son familia.
   */
@@ -824,16 +824,16 @@ const RetailerProductEditionView = ({
   }, [state.product?.id_article]);
 
   // El conteo sale de la familia recién consultada, no del producto.
-  const variantesDelProducto = family?.error
+  const hijosDelProducto = family?.error
     ? 0
     : (family?.variants ?? []).filter((v) => !v.isCurrent).length;
 
-  const esVarianteDeOtro = family?.error
+  const esHijoDeOtro = family?.error
     ? !!state.product?.parentArticleId
     : !!family?.parent;
 
   /*
-    En Creators la lista de familia es INFORMATIVA: no se salta a otra variante.
+    En Creators la lista de familia es INFORMATIVA: no se salta a otro producto.
 
     Saltar exige precargar el producto destino, y el endpoint que hace eso en la
     aplicación de clientes —read-articles-v2— filtra por la compañía del usuario:
@@ -842,15 +842,16 @@ const RetailerProductEditionView = ({
 
     Navegar sin precargar sería peor que no ofrecerlo: esta pantalla lee el
     producto de sessionStorage, así que cambiaría la URL y seguiría mostrando el
-    producto anterior, dando a entender que se cambió de variante cuando no.
+    producto anterior, dando a entender que se cambió de producto cuando no.
 
     Para habilitarlo hace falta una forma de traer un producto por id sin filtro
     de compañía, respetando el alcance del perfil interno (§5.1 del contrato).
   */
 
   /*
-    Refresca tras ligar, sin recargar: el producto sale de sessionStorage, así que
-    un refresh devolvería el mismo objeto y el contador seguiría igual.
+    Refresca tras ligar, sin recargar la página: el producto sale de
+    sessionStorage, así que un F5 devolvería el mismo objeto y el contador
+    seguiría igual.
   */
   const handleOnLinkedVariants = (result) => {
     if (!result) return;
@@ -869,31 +870,77 @@ const RetailerProductEditionView = ({
           };
 
     dispatch({ type: "SET_PRODUCT", payload: updatedProduct });
-    setFamily(null);
 
     try {
       sessionStorage.setItem("productSelected", JSON.stringify(updatedProduct));
     } catch (error) {
       console.error("linkVariants/sessionStorage: ", error);
     }
+
+    /*
+      Volver a preguntar, no vaciar.
+
+      Antes esto hacía `setFamily(null)` para invalidar la caché, y en esta
+      pantalla eso tenía el efecto contrario al buscado: el bloque de parentesco
+      se decide con `esHijoDeOtro`, que sale de `family`, no del producto. Con
+      `family` en null el bloque desaparecía, así que después de ligar la
+      pantalla se quedaba SIN ninguna señal de que el producto ya tenía padre
+      —justo lo que el usuario acababa de hacer—.
+    */
+    loadFamily(true);
+
+    /*
+      Y se recargan los servicios, porque ligar cambia la herencia de cada campo
+      en el servidor: al planchar, lo que era propio pasa a heredado y los chips
+      tienen que decirlo. Sin esto la ficha sigue mostrando el estado anterior
+      hasta que alguien cierra y vuelve a abrir el producto.
+
+      Sin `enableLoading` para no parpadear el esqueleto encima de una pantalla
+      que ya está pintada.
+    */
+    loadData(false);
   };
 
   /*
     Acción "Volver a heredar" de los chips. Repinta en sitio con lo que devuelve
     el endpoint: recargar la ficha se llevaría lo que haya sin guardar.
   */
-  const handleOnRestoreInheritance = async (attributeId) => {
+  /*
+    `kind` distingue de qué pestaña salió el chip. Las descripciones viven en
+    `rel_article_description` y la ficha técnica en `article_data`: el endpoint
+    recibe una lista o la otra.
+  */
+  const handleOnRestoreInheritance = async (attributeId, kind = "attribute") => {
     if (!attributeId || !state.product?.id_article) return;
 
+    const esDescripcion = kind === "description";
+
     try {
       const res = await axios.post(
         `${process.env.REACT_APP_VARIANTS_ENDPOINT}/restore`,
-        { articleId: state.product.id_article, attributes: [Number(attributeId)] },
+        {
+          articleId: state.product.id_article,
+          ...(esDescripcion
+            ? { descriptions: [Number(attributeId)] }
+            : { attributes: [Number(attributeId)] }),
+        },
         { headers: { Authorization: token } }
       );
 
       const data = readPayload(res);
 
+      if (esDescripcion) {
+        dispatch({
+          type: "UPDATE_DESCRIPTIONS_INPUTS",
+          payload: (data.descriptionValues || []).map((item) => ({
+            attributeId: item.id_description,
+            value: item.value,
+            isOverridden: 0,
+          })),
+        });
+        return;
+      }
+
       dispatch({
         type: "UPDATE_DATASHEETS_INPUTS",
         payload: (data.values || []).map((item) => ({
@@ -933,10 +980,10 @@ const RetailerProductEditionView = ({
         await saveDescriptions(token);
         break;
       case "Ficha técnica": {
-        // Guardar un padre con variantes abre la decisión de propagar. Sin este
-        // paso el cambio se queda solo en el padre y las variantes se van
+        // 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
         // separando de él sin que nadie lo note.
-        const esPadre = variantesDelProducto > 0;
+        const esPadre = hijosDelProducto > 0;
         const cambios = cambiosDeFichaTecnica();
 
         if (esPadre && cambios.length > 0) {
@@ -1930,9 +1977,9 @@ const RetailerProductEditionView = ({
           </div>
 
           <div className="product-information">
-            {/* Solo si el producto NO es ya variante de alguien: ligarlo otra vez
+            {/* Solo si el producto NO es ya hijo de alguien: ligarlo otra vez
                 devolvería VARIANT_ALREADY_LINKED sin excepción. */}
-            {!esVarianteDeOtro && (
+            {!esHijoDeOtro && (
               <div className="variants-actions">
                 <button
                   type="button"
@@ -1941,7 +1988,7 @@ const RetailerProductEditionView = ({
                 >
                   Ligar productos
                 </button>
-                {variantesDelProducto > 0 && (
+                {hijosDelProducto > 0 && (
                   <button
                     type="button"
                     className="variants-count"
@@ -1951,15 +1998,15 @@ const RetailerProductEditionView = ({
                       loadFamily();
                     }}
                   >
-                    {variantesDelProducto} variante
-                    {variantesDelProducto === 1 ? "" : "s"}
+                    {hijosDelProducto} hijo
+                    {hijosDelProducto === 1 ? "" : "s"}
                     <span aria-hidden="true">{showFamily ? " ▴" : " ▾"}</span>
                   </button>
                 )}
               </div>
             )}
 
-            {esVarianteDeOtro && (
+            {esHijoDeOtro && (
               <div className="variants-actions">
                 <button
                   type="button"
@@ -1970,7 +2017,7 @@ const RetailerProductEditionView = ({
                     loadFamily();
                   }}
                 >
-                  Variante de otro producto
+                  Hijo de otro producto
                   <span aria-hidden="true">{showFamily ? " ▴" : " ▾"}</span>
                 </button>
               </div>
@@ -2000,7 +2047,7 @@ const RetailerProductEditionView = ({
                     className={`family-row${variant.isCurrent ? " current" : ""}`}
                   >
                     <span className="role">
-                      {variant.isCurrent ? "Estás aquí" : "Variante"}
+                      {variant.isCurrent ? "Estás aquí" : "Hijo"}
                     </span>
                     <span className="name">{variant.name}</span>
                     <span className="upc">{variant.upc}</span>
@@ -2008,7 +2055,7 @@ const RetailerProductEditionView = ({
                 ))}
 
                 {family && !family.error && !family.variants?.length && (
-                  <p className="loading">Este producto no tiene variantes.</p>
+                  <p className="loading">Este producto no tiene hijos.</p>
                 )}
               </div>
             )}
@@ -2095,6 +2142,10 @@ const RetailerProductEditionView = ({
                         activeSection={state.active_tab}
                         inputGroup={state.descriptions_inputs[0]}
                         updatedDescriptions={state.updated_descriptions_inputs}
+                        /* El parentesco es del producto, no del servicio: los
+                           dos endpoints lo devuelven igual. */
+                        variant={state.services?.datasheets?.variant}
+                        onRestoreInheritance={handleOnRestoreInheritance}
                         articleId={state.product?.id_article}
                         version={state.product.version}
                         auditInputGroup={auditDescriptions[0]}