contentoh-components-library 21.6.32 → 21.6.34

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

@@ -234,7 +234,7 @@ function _regenerateVideoThumbnails() {
 }
 
 var RetailerProductEditionView = function RetailerProductEditionView(_ref) {
-  var _state$product10, _family$variants, _state$product11, _state$errors, _family$variants2, _state$product14, _state$product15, _state$product16, _state$active_retaile5, _location$state, _location$state2, _state$active_retaile6, _state$active_retaile7, _state$images_values$2, _state$images_values10, _state$images_values11, _state$product17, _state$active_retaile8, _family$variants3, _family$variants4, _state$active_percent, _state$active_percent2, _state$active_percent3, _state$active_retaile9, _state$active_retaile10, _state$active_retaile11, _state$product18, _state$product19, _state$product20, _state$datasheets_inp, _state$datasheets_inp2, _state$product21, _state$images_values12, _state$comment;
+  var _state$product10, _family$variants, _state$product11, _state$errors, _family$variants2, _state$product14, _state$product15, _state$product16, _state$active_retaile5, _location$state, _location$state2, _state$active_retaile6, _state$active_retaile7, _state$images_values$2, _state$images_values10, _state$images_values11, _state$product17, _state$active_retaile8, _family$variants3, _family$variants4, _state$active_percent, _state$active_percent2, _state$active_percent3, _state$active_retaile9, _state$active_retaile10, _state$active_retaile11, _state$product18, _state$services2, _state$services2$data, _state$product19, _state$product20, _state$datasheets_inp, _state$datasheets_inp2, _state$product21, _state$images_values12, _state$comment;
 
   var tabsSections = _ref.tabsSections,
       _ref$productSelected = _ref.productSelected,
@@ -310,7 +310,7 @@ var RetailerProductEditionView = function RetailerProductEditionView(_ref) {
   var _useState19 = (0, _react.useState)(false),
       _useState20 = (0, _slicedToArray2.default)(_useState19, 2),
       rejectAll = _useState20[0],
-      setRejectAll = _useState20[1]; // Variantes: mismo comportamiento que la ficha de la aplicación de clientes.
+      setRejectAll = _useState20[1]; // Padre/hijo: mismo comportamiento que la ficha de la aplicación de clientes.
 
 
   var _useState21 = (0, _react.useState)(false),
@@ -1128,7 +1128,7 @@ var RetailerProductEditionView = function RetailerProductEditionView(_ref) {
     }
   };
   /*
-    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.
   */
@@ -1215,26 +1215,27 @@ var RetailerProductEditionView = function RetailerProductEditionView(_ref) {
     loadFamily(true); // eslint-disable-next-line react-hooks/exhaustive-deps
   }, [(_state$product10 = state.product) === null || _state$product10 === void 0 ? void 0 : _state$product10.id_article]); // El conteo sale de la familia recién consultada, no del producto.
 
-  var variantesDelProducto = family !== null && family !== void 0 && family.error ? 0 : ((_family$variants = family === null || family === void 0 ? void 0 : family.variants) !== null && _family$variants !== void 0 ? _family$variants : []).filter(function (v) {
+  var hijosDelProducto = family !== null && family !== void 0 && family.error ? 0 : ((_family$variants = family === null || family === void 0 ? void 0 : family.variants) !== null && _family$variants !== void 0 ? _family$variants : []).filter(function (v) {
     return !v.isCurrent;
   }).length;
-  var esVarianteDeOtro = family !== null && family !== void 0 && family.error ? !!((_state$product11 = state.product) !== null && _state$product11 !== void 0 && _state$product11.parentArticleId) : !!(family !== null && family !== void 0 && family.parent);
+  var esHijoDeOtro = family !== null && family !== void 0 && family.error ? !!((_state$product11 = state.product) !== null && _state$product11 !== void 0 && _state$product11.parentArticleId) : !!(family !== null && family !== void 0 && 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:
     `AND a.company_id = <la del usuario>`. El equipo interno pertenece a ContentOh
     y trabaja productos de sus clientes, así que ahí no devuelve nada.
      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.
   */
 
   var handleOnLinkedVariants = function handleOnLinkedVariants(result) {
@@ -1251,51 +1252,107 @@ var RetailerProductEditionView = function RetailerProductEditionView(_ref) {
       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.
   */
 
+  /*
+    `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.
+  */
+
 
   var handleOnRestoreInheritance = /*#__PURE__*/function () {
     var _ref9 = (0, _asyncToGenerator2.default)( /*#__PURE__*/_regenerator.default.mark(function _callee6(attributeId) {
       var _state$product13;
 
-      var res, data;
+      var kind,
+          esDescripcion,
+          res,
+          data,
+          _args6 = arguments;
       return _regenerator.default.wrap(function _callee6$(_context6) {
         while (1) {
           switch (_context6.prev = _context6.next) {
             case 0:
+              kind = _args6.length > 1 && _args6[1] !== undefined ? _args6[1] : "attribute";
+
               if (!(!attributeId || !((_state$product13 = state.product) !== null && _state$product13 !== void 0 && _state$product13.id_article))) {
-                _context6.next = 2;
+                _context6.next = 3;
                 break;
               }
 
               return _context6.abrupt("return");
 
-            case 2:
-              _context6.prev = 2;
-              _context6.next = 5;
-              return _axios.default.post("".concat(process.env.REACT_APP_VARIANTS_ENDPOINT, "/restore"), {
-                articleId: state.product.id_article,
+            case 3:
+              esDescripcion = kind === "description";
+              _context6.prev = 4;
+              _context6.next = 7;
+              return _axios.default.post("".concat(process.env.REACT_APP_VARIANTS_ENDPOINT, "/restore"), (0, _objectSpread5.default)({
+                articleId: state.product.id_article
+              }, esDescripcion ? {
+                descriptions: [Number(attributeId)]
+              } : {
                 attributes: [Number(attributeId)]
-              }, {
+              }), {
                 headers: {
                   Authorization: token
                 }
               });
 
-            case 5:
+            case 7:
               res = _context6.sent;
               data = (0, _api.readPayload)(res);
+
+              if (!esDescripcion) {
+                _context6.next = 12;
+                break;
+              }
+
+              dispatch({
+                type: "UPDATE_DESCRIPTIONS_INPUTS",
+                payload: (data.descriptionValues || []).map(function (item) {
+                  return {
+                    attributeId: item.id_description,
+                    value: item.value,
+                    isOverridden: 0
+                  };
+                })
+              });
+              return _context6.abrupt("return");
+
+            case 12:
               dispatch({
                 type: "UPDATE_DATASHEETS_INPUTS",
                 payload: (data.values || []).map(function (item) {
@@ -1306,20 +1363,20 @@ var RetailerProductEditionView = function RetailerProductEditionView(_ref) {
                   };
                 })
               });
-              _context6.next = 13;
+              _context6.next = 18;
               break;
 
-            case 10:
-              _context6.prev = 10;
-              _context6.t0 = _context6["catch"](2);
+            case 15:
+              _context6.prev = 15;
+              _context6.t0 = _context6["catch"](4);
               console.error("restoreInheritance: ", _context6.t0);
 
-            case 13:
+            case 18:
             case "end":
               return _context6.stop();
           }
         }
-      }, _callee6, null, [[2, 10]]);
+      }, _callee6, null, [[4, 15]]);
     }));
 
     return function handleOnRestoreInheritance(_x3) {
@@ -1376,10 +1433,10 @@ var RetailerProductEditionView = function RetailerProductEditionView(_ref) {
               return _context7.abrupt("break", 21);
 
             case 8:
-              // 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.
-              esPadre = variantesDelProducto > 0;
+              esPadre = hijosDelProducto > 0;
               cambios = cambiosDeFichaTecnica();
 
               if (!(esPadre && cambios.length > 0)) {
@@ -2537,7 +2594,7 @@ var RetailerProductEditionView = function RetailerProductEditionView(_ref) {
           })]
         }), /*#__PURE__*/(0, _jsxRuntime.jsxs)("div", {
           className: "product-information",
-          children: [!esVarianteDeOtro && /*#__PURE__*/(0, _jsxRuntime.jsxs)("div", {
+          children: [!esHijoDeOtro && /*#__PURE__*/(0, _jsxRuntime.jsxs)("div", {
             className: "variants-actions",
             children: [/*#__PURE__*/(0, _jsxRuntime.jsx)("button", {
               type: "button",
@@ -2546,7 +2603,7 @@ var RetailerProductEditionView = function RetailerProductEditionView(_ref) {
                 return setShowLinkVariants(true);
               },
               children: "Ligar productos"
-            }), variantesDelProducto > 0 && /*#__PURE__*/(0, _jsxRuntime.jsxs)("button", {
+            }), hijosDelProducto > 0 && /*#__PURE__*/(0, _jsxRuntime.jsxs)("button", {
               type: "button",
               className: "variants-count",
               "aria-expanded": showFamily,
@@ -2556,12 +2613,12 @@ var RetailerProductEditionView = function RetailerProductEditionView(_ref) {
                 });
                 loadFamily();
               },
-              children: [variantesDelProducto, " variante", variantesDelProducto === 1 ? "" : "s", /*#__PURE__*/(0, _jsxRuntime.jsx)("span", {
+              children: [hijosDelProducto, " hijo", hijosDelProducto === 1 ? "" : "s", /*#__PURE__*/(0, _jsxRuntime.jsx)("span", {
                 "aria-hidden": "true",
                 children: showFamily ? " ▴" : " ▾"
               })]
             })]
-          }), esVarianteDeOtro && /*#__PURE__*/(0, _jsxRuntime.jsx)("div", {
+          }), esHijoDeOtro && /*#__PURE__*/(0, _jsxRuntime.jsx)("div", {
             className: "variants-actions",
             children: /*#__PURE__*/(0, _jsxRuntime.jsxs)("button", {
               type: "button",
@@ -2573,7 +2630,7 @@ var RetailerProductEditionView = function RetailerProductEditionView(_ref) {
                 });
                 loadFamily();
               },
-              children: ["Variante de otro producto", /*#__PURE__*/(0, _jsxRuntime.jsx)("span", {
+              children: ["Hijo de otro producto", /*#__PURE__*/(0, _jsxRuntime.jsx)("span", {
                 "aria-hidden": "true",
                 children: showFamily ? " ▴" : " ▾"
               })]
@@ -2603,7 +2660,7 @@ var RetailerProductEditionView = function RetailerProductEditionView(_ref) {
                 className: "family-row".concat(variant.isCurrent ? " current" : ""),
                 children: [/*#__PURE__*/(0, _jsxRuntime.jsx)("span", {
                   className: "role",
-                  children: variant.isCurrent ? "Estás aquí" : "Variante"
+                  children: variant.isCurrent ? "Estás aquí" : "Hijo"
                 }), /*#__PURE__*/(0, _jsxRuntime.jsx)("span", {
                   className: "name",
                   children: variant.name
@@ -2614,7 +2671,7 @@ var RetailerProductEditionView = function RetailerProductEditionView(_ref) {
               }, variant.articleId);
             }), family && !family.error && !((_family$variants4 = family.variants) !== null && _family$variants4 !== void 0 && _family$variants4.length) && /*#__PURE__*/(0, _jsxRuntime.jsx)("p", {
               className: "loading",
-              children: "Este producto no tiene variantes."
+              children: "Este producto no tiene hijos."
             })]
           }), /*#__PURE__*/(0, _jsxRuntime.jsx)(_FullProductNameHeader.FullProductNameHeader // states
           , {
@@ -2677,7 +2734,12 @@ var RetailerProductEditionView = function RetailerProductEditionView(_ref) {
               children: [state.active_tab === "Descripción" && (((_state$product18 = state.product) === null || _state$product18 === void 0 ? void 0 : _state$product18.description_status) !== "NS" ? /*#__PURE__*/(0, _jsxRuntime.jsx)(_InputGroup.InputGroup, {
                 activeSection: state.active_tab,
                 inputGroup: state.descriptions_inputs[0],
-                updatedDescriptions: state.updated_descriptions_inputs,
+                updatedDescriptions: state.updated_descriptions_inputs
+                /* El parentesco es del producto, no del servicio: los
+                   dos endpoints lo devuelven igual. */
+                ,
+                variant: (_state$services2 = state.services) === null || _state$services2 === void 0 ? void 0 : (_state$services2$data = _state$services2.datasheets) === null || _state$services2$data === void 0 ? void 0 : _state$services2$data.variant,
+                onRestoreInheritance: handleOnRestoreInheritance,
                 articleId: (_state$product19 = state.product) === null || _state$product19 === void 0 ? void 0 : _state$product19.id_article,
                 version: state.product.version,
                 auditInputGroup: auditDescriptions[0],
@@ -2689,7 +2751,7 @@ var RetailerProductEditionView = function RetailerProductEditionView(_ref) {
                 headerType: "input-name-header"
               })), state.active_tab === "Ficha técnica" && (((_state$product20 = state.product) === null || _state$product20 === void 0 ? void 0 : _state$product20.datasheet_status) !== "NS" ? /*#__PURE__*/(0, _jsxRuntime.jsx)(_jsxRuntime.Fragment, {
                 children: (_state$datasheets_inp = state.datasheets_inputs[0]) === null || _state$datasheets_inp === void 0 ? void 0 : (_state$datasheets_inp2 = _state$datasheets_inp.data) === null || _state$datasheets_inp2 === void 0 ? void 0 : _state$datasheets_inp2.map(function (dataGroup, index) {
-                  var _state$services2, _state$services2$data;
+                  var _state$services3, _state$services3$data;
 
                   return /*#__PURE__*/(0, _jsxRuntime.jsx)(_InputGroup.InputGroup, {
                     articleId: state.product.id_article,
@@ -2697,7 +2759,7 @@ var RetailerProductEditionView = function RetailerProductEditionView(_ref) {
                     activeSection: state.active_tab,
                     inputGroup: dataGroup,
                     dataInputs: state.datasheets_inputs[1],
-                    variant: (_state$services2 = state.services) === null || _state$services2 === void 0 ? void 0 : (_state$services2$data = _state$services2.datasheets) === null || _state$services2$data === void 0 ? void 0 : _state$services2$data.variant,
+                    variant: (_state$services3 = state.services) === null || _state$services3 === void 0 ? void 0 : (_state$services3$data = _state$services3.datasheets) === null || _state$services3$data === void 0 ? void 0 : _state$services3$data.variant,
                     onRestoreInheritance: handleOnRestoreInheritance,
                     auditInputs: auditDatasheets[1],
                     updatedDatasheets: state.updated_datasheets_inputs,

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "contentoh-components-library",
-  "version": "21.6.32",
+  "version": "21.6.34",
   "dependencies": {
     "@aws-amplify/auth": "^4.5.3",
     "@aws-amplify/datastore": "^3.11.0",

package/src/components/atoms/InheritanceTag/InheritanceTag.stories.js

@@ -31,9 +31,9 @@ PropioSinAccion.args = {
   isOverridden: true,
 };
 
-// En un producto que no es variante no se pinta nada.
-export const NoEsVariante = Template.bind({});
-NoEsVariante.args = {
+// En un producto que no es hijo de nadie no se pinta nada.
+export const NoEsHijo = Template.bind({});
+NoEsHijo.args = {
   isVariant: false,
   isOverridden: true,
 };

package/src/components/atoms/InheritanceTag/index.js

@@ -1,9 +1,9 @@
 import { Container } from "./styles";
 
 /**
- * Marca si el valor de un atributo viene del producto padre o es propio de la
- * variante. Solo tiene sentido dentro de una variante: en un producto sin padre
- * no se pinta nada, porque no hay de quién heredar.
+ * Marca si el valor de un atributo viene del producto padre o es propio del
+ * hijo. Solo tiene sentido dentro de un producto hijo: en uno sin padre no se
+ * pinta nada, porque no hay de quién heredar.
  *
  * El estado "propio" es el que carga información —ese campo se separó del padre—
  * así que es el que lleva acento y el que ofrece la acción de volver a heredar.
@@ -11,8 +11,8 @@ import { Container } from "./styles";
  * la ficha entera se volvería ruido.
  *
  * Un campo VACÍO no lleva marca: no heredó nada y tampoco tiene valor propio, así
- * que llamarlo "Heredado" es falso además de ruidoso. Sin esto la ficha de una
- * variante sale con el chip repetido en cada campo, incluidos los obligatorios en
+ * que llamarlo "Heredado" es falso además de ruidoso. Sin esto la ficha de un
+ * hijo sale con el chip repetido en cada campo, incluidos los obligatorios en
  * rojo que están sin capturar, y la marca deja de distinguir nada.
  *
  * Tampoco la lleva un atributo NO heredable. El UPC y el SKU son la identidad del
@@ -41,7 +41,7 @@ export const InheritanceTag = ({
         className="tag-label"
         title={
           isOverridden
-            ? "Valor propio de esta variante. La propagación del producto padre no lo modifica."
+            ? "Valor propio de este producto hijo. La propagación del padre no lo modifica."
             : `Valor heredado${parentUpc ? ` del producto ${parentUpc}` : " del producto padre"}.`
         }
       >

package/src/components/organisms/InputGroup/index.js

@@ -296,6 +296,18 @@ export const InputGroup = ({
                     dinamicHeight={dinamicHeight}
                     description={input?.description}
                     showTooltip={true}
+                    isVariant={variant?.isVariant}
+                    isOverridden={input?.isOverridden}
+                    inheritable={input?.inheritable}
+                    parentUpc={variant?.parent?.upc}
+                    /* Las descripciones viven en otra tabla que la ficha
+                       técnica, así que restore necesita saber cuál de las dos
+                       listas está recibiendo. */
+                    onRestoreInheritance={
+                      onRestoreInheritance
+                        ? (id) => onRestoreInheritance(id, "description")
+                        : undefined
+                    }
                     auditClass={
                       compare &&
                       !isEquals(