contentoh-components-library 21.6.36 → 21.6.37

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

@@ -16,6 +16,10 @@ var _jsxRuntime = require("react/jsx-runtime");
  *
  * 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.
+ *
+ * OJO: "propio" ya NO significa "a salvo de la propagación". Desde el cambio del
+ * 2026-09-01 la propagación sobrescribe también estos valores; el aviso de que va
+ * a pasar vive en el diálogo de propagación, no aquí. El tooltip lo dice así.
  * "Heredado" es el caso normal y se mantiene discreto: si gritara en cada campo,
  * la ficha entera se volvería ruido.
  *
@@ -51,9 +55,9 @@ var InheritanceTag = function InheritanceTag(_ref) {
     className: "inheritance-tag ".concat(state, " ").concat(className),
     children: [/*#__PURE__*/(0, _jsxRuntime.jsx)("span", {
       className: "tag-label",
-      title: isOverridden ? "Valor propio de este producto hijo. La propagación del padre no lo modifica." : "Valor heredado".concat(parentUpc ? " del producto ".concat(parentUpc) : " del producto padre", "."),
+      title: isOverridden ? "Valor capturado en este producto hijo, distinto al del padre. Si el padre propaga este atributo, se reemplaza." : "Valor heredado".concat(parentUpc ? " del producto ".concat(parentUpc) : " del producto padre", "."),
       children: isOverridden ? "Propio" : "Heredado"
-    }), isOverridden && onRestore && /*#__PURE__*/(0, _jsxRuntime.jsx)("button", {
+    }), Boolean(isOverridden) && onRestore && /*#__PURE__*/(0, _jsxRuntime.jsx)("button", {
       type: "button",
       className: "tag-action",
       onClick: onRestore,

package/dist/components/molecules/TagAndInput/index.js

@@ -420,7 +420,7 @@ var TagAndInput = function TagAndInput(_ref) {
   return /*#__PURE__*/(0, _jsxRuntime.jsxs)(_styles.Container, {
     inputType: inputType,
     className: "input-container",
-    children: [(label === null || label === void 0 ? void 0 : label.length) && /*#__PURE__*/(0, _jsxRuntime.jsxs)("div", {
+    children: [Boolean(label === null || label === void 0 ? void 0 : label.length) && /*#__PURE__*/(0, _jsxRuntime.jsxs)("div", {
       className: "title-container",
       children: [/*#__PURE__*/(0, _jsxRuntime.jsxs)("div", {
         className: "titles",

package/dist/components/molecules/TagAndInputv2/index.js

@@ -54,7 +54,7 @@ var TagAndInputv2 = function TagAndInputv2(_ref) {
   return /*#__PURE__*/(0, _jsxRuntime.jsxs)(_styles.Container, {
     inputType: inputType,
     className: "input-container",
-    children: [(label === null || label === void 0 ? void 0 : label.length) && /*#__PURE__*/(0, _jsxRuntime.jsxs)("div", {
+    children: [Boolean(label === null || label === void 0 ? void 0 : label.length) && /*#__PURE__*/(0, _jsxRuntime.jsxs)("div", {
       className: "title-container",
       children: [/*#__PURE__*/(0, _jsxRuntime.jsx)(_ScreenHeaderv.ScreenHeaderv2, {
         text: label,

package/dist/components/organisms/PropagateChangesModal/index.js

@@ -9,6 +9,8 @@ exports.PropagateChangesModal = void 0;
 
 var _toConsumableArray2 = _interopRequireDefault(require("@babel/runtime/helpers/esm/toConsumableArray"));
 
+var _objectSpread2 = _interopRequireDefault(require("@babel/runtime/helpers/esm/objectSpread2"));
+
 var _slicedToArray2 = _interopRequireDefault(require("@babel/runtime/helpers/esm/slicedToArray"));
 
 var _react = require("react");
@@ -27,7 +29,10 @@ var _jsxRuntime = require("react/jsx-runtime");
 
   props (* requerido):
     - *changedAttributes: [{ id, name, value }] lo que se acaba de editar
-    - *variants: [{ articleId, name, upc }] los hijos activos del padre
+    - *variants: [{ articleId, name, upc, overriddenAttributes }] los hijos
+        activos del padre. `overriddenAttributes` son los atributos que ese hijo
+        capturó a mano; la propagación los sobrescribe, y este diálogo es el
+        único lugar donde el usuario puede enterarse antes de que pase.
     - *onCancel: cerrar sin guardar nada
     - *onConfirm: (propagation | null) => void
         null            → guardar solo el padre
@@ -67,6 +72,31 @@ var PropagateChangesModal = function PropagateChangesModal(_ref) {
   var destinatarios = scope === "all" ? variants.length : chosenChildren.length;
   var puedePropagar = chosenAttributes.length > 0 && destinatarios > 0;
   var enSegundoPlano = destinatarios > UMBRAL_SEGUNDO_PLANO;
+  /*
+    A quién se le va a pisar un valor capturado a mano.
+     La propagación sobrescribe: eso se decidió a propósito, porque el usuario ya
+    eligió atributo por atributo y hijo por hijo. Pero sobrescribir en silencio
+    es otra cosa, y desde que se quitaron las etiquetas de «Heredado» y «Propio»
+    este diálogo es el único momento en que alguien puede saberlo.
+     Se recalcula con lo que está marcado ahora, no al abrir: si el usuario
+    desmarca el atributo conflictivo, el aviso desaparece — que es justo la
+    salida que el aviso debería estar ofreciendo.
+  */
+
+  var destinos = scope === "all" ? variants : variants.filter(function (item) {
+    return chosenChildren.includes(item.articleId);
+  });
+  var conValorPropio = destinos.map(function (hijo) {
+    var _hijo$overriddenAttri;
+
+    return (0, _objectSpread2.default)((0, _objectSpread2.default)({}, hijo), {}, {
+      pisados: ((_hijo$overriddenAttri = hijo.overriddenAttributes) !== null && _hijo$overriddenAttri !== void 0 ? _hijo$overriddenAttri : []).filter(function (id) {
+        return chosenAttributes.includes(Number(id));
+      })
+    });
+  }).filter(function (hijo) {
+    return hijo.pisados.length > 0;
+  });
 
   var toggleAttribute = function toggleAttribute(id) {
     setChosenAttributes(function (current) {
@@ -173,8 +203,11 @@ var PropagateChangesModal = function PropagateChangesModal(_ref) {
               })
             }, variant.articleId);
           })
-        }), /*#__PURE__*/(0, _jsxRuntime.jsx)(_styles.Note, {
-          children: "Un hijo que haya editado alguno de estos atributos por su cuenta conserva su valor. Para devolverlo al del padre est\xE1 la acci\xF3n \xABVolver a heredar\xBB en la ficha de ese hijo."
+        }), conValorPropio.length > 0 && /*#__PURE__*/(0, _jsxRuntime.jsxs)(_styles.Note, {
+          className: "accent",
+          children: [conValorPropio.length === 1 ? "Un hijo tiene un valor propio en lo que elegiste y se va a reemplazar: " : "".concat(conValorPropio.length, " hijos tienen valores propios en lo que elegiste y se van a reemplazar: "), conValorPropio.map(function (hijo) {
+            return "".concat(hijo.name || "Producto ".concat(hijo.articleId), " (").concat(hijo.pisados.length, ")");
+          }).join(", "), ". Si no quieres pisarlos, desmarca ese atributo o ese hijo."]
         }), enSegundoPlano && /*#__PURE__*/(0, _jsxRuntime.jsxs)(_styles.Note, {
           className: "accent",
           children: ["Son m\xE1s de ", UMBRAL_SEGUNDO_PLANO, " hijos, as\xED que la propagaci\xF3n corre en segundo plano: el guardado termina antes de que todas est\xE9n actualizadas."]

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

@@ -1597,7 +1597,8 @@ var RetailerProductEditionView = function RetailerProductEditionView(_ref) {
 
   var handleOnClickSave = /*#__PURE__*/function () {
     var _ref12 = (0, _asyncToGenerator2.default)( /*#__PURE__*/_regenerator.default.mark(function _callee8() {
-      var esPadre, cambios;
+      var _familia$variants, cambios, familia, hijos;
+
       return _regenerator.default.wrap(function _callee8$(_context8) {
         while (1) {
           switch (_context8.prev = _context8.next) {
@@ -1611,7 +1612,7 @@ var RetailerProductEditionView = function RetailerProductEditionView(_ref) {
 
             case 2:
               _context8.t0 = state.active_tab;
-              _context8.next = _context8.t0 === "Descripción" ? 5 : _context8.t0 === "Ficha técnica" ? 8 : _context8.t0 === "Imágenes" ? 17 : 20;
+              _context8.next = _context8.t0 === "Descripción" ? 5 : _context8.t0 === "Ficha técnica" ? 8 : _context8.t0 === "Imágenes" ? 32 : 35;
               break;
 
             case 5:
@@ -1619,57 +1620,134 @@ var RetailerProductEditionView = function RetailerProductEditionView(_ref) {
               return saveDescriptions(token);
 
             case 7:
-              return _context8.abrupt("break", 21);
+              return _context8.abrupt("break", 36);
 
             case 8:
-              // 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 = hijosDelProducto > 0;
               cambios = cambiosDeFichaTecnica();
 
-              if (!(esPadre && cambios.length > 0)) {
-                _context8.next = 14;
+              if (!(cambios.length === 0)) {
+                _context8.next = 13;
                 break;
               }
 
+              _context8.next = 12;
+              return saveDatasheets(token);
+
+            case 12:
+              return _context8.abrupt("break", 36);
+
+            case 13:
+              /*
+                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.
+                 Si tiene hijos se resuelve AQUÍ, esperando la respuesta, en vez de
+                mirar la copia en memoria. `family` es una consulta asíncrona: si
+                todavía no llegó, o si falló, `hijosDelProducto` vale 0 — o sea que
+                «todavía no sé» se leía igual que «no tiene hijos», y el padre se
+                guardaba sin ofrecer la propagación. En silencio, que es lo peor: el
+                usuario creía haber bajado el cambio a los hijos.
+                 La ficha de la aplicación de clientes no tiene el problema porque ahí
+                `state.product.variantsCount` viene del listado; aquí el producto sale
+                del endpoint de reportes, que no trae el parentesco.
+              */
               setPreparandoPropagacion(true);
-              loadFamily().then(function () {
-                return setPendingPropagation({
-                  cambios: cambios
-                });
-              }).finally(function () {
-                return setPreparandoPropagacion(false);
+              familia = null;
+              _context8.prev = 15;
+              _context8.next = 18;
+              return loadFamily(true);
+
+            case 18:
+              familia = _context8.sent;
+
+            case 19:
+              _context8.prev = 19;
+              setPreparandoPropagacion(false);
+              return _context8.finish(19);
+
+            case 22:
+              if (!(!familia || familia.error)) {
+                _context8.next = 25;
+                break;
+              }
+
+              dispatch({
+                type: "SET_MODAL",
+                payload: {
+                  show: true,
+                  title: "No se pudo consultar la familia",
+                  message: "No pudimos saber si este producto tiene hijos, así que no sabemos si hay que bajarles el cambio. Vuelve a intentar el guardado; si sigue fallando, guarda y revisa los hijos a mano.",
+                  image: _genericModalWarning.default,
+                  buttons: [{
+                    text: "Entendido",
+                    buttonType: "general-white-button",
+                    action: function action() {
+                      return dispatch({
+                        type: "SET_MODAL",
+                        payload: {
+                          show: false
+                        }
+                      });
+                    }
+                  }, {
+                    text: "Guardar solo este producto",
+                    buttonType: "general-button-default",
+                    action: function action() {
+                      dispatch({
+                        type: "SET_MODAL",
+                        payload: {
+                          show: false
+                        }
+                      });
+                      saveDatasheets(token);
+                    }
+                  }]
+                }
               });
               return _context8.abrupt("return");
 
-            case 14:
-              _context8.next = 16;
+            case 25:
+              hijos = ((_familia$variants = familia.variants) !== null && _familia$variants !== void 0 ? _familia$variants : []).filter(function (v) {
+                return !v.isCurrent;
+              });
+
+              if (!(hijos.length > 0)) {
+                _context8.next = 29;
+                break;
+              }
+
+              setPendingPropagation({
+                cambios: cambios
+              });
+              return _context8.abrupt("return");
+
+            case 29:
+              _context8.next = 31;
               return saveDatasheets(token);
 
-            case 16:
-              return _context8.abrupt("break", 21);
+            case 31:
+              return _context8.abrupt("break", 36);
 
-            case 17:
-              _context8.next = 19;
+            case 32:
+              _context8.next = 34;
               return updateImages(token);
 
-            case 19:
-              return _context8.abrupt("break", 21);
+            case 34:
+              return _context8.abrupt("break", 36);
 
-            case 20:
-              return _context8.abrupt("break", 21);
+            case 35:
+              return _context8.abrupt("break", 36);
 
-            case 21:
-              _context8.next = 23;
+            case 36:
+              _context8.next = 38;
               return loadData(false);
 
-            case 23:
+            case 38:
             case "end":
               return _context8.stop();
           }
         }
-      }, _callee8);
+      }, _callee8, null, [[15,, 19, 22]]);
     }));
 
     return function handleOnClickSave() {

package/package.json

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

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

@@ -7,6 +7,10 @@ import { Container } from "./styles";
  *
  * 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.
+ *
+ * OJO: "propio" ya NO significa "a salvo de la propagación". Desde el cambio del
+ * 2026-09-01 la propagación sobrescribe también estos valores; el aviso de que va
+ * a pasar vive en el diálogo de propagación, no aquí. El tooltip lo dice así.
  * "Heredado" es el caso normal y se mantiene discreto: si gritara en cada campo,
  * la ficha entera se volvería ruido.
  *
@@ -41,14 +45,20 @@ export const InheritanceTag = ({
         className="tag-label"
         title={
           isOverridden
-            ? "Valor propio de este producto hijo. La propagación del padre no lo modifica."
+            ? "Valor capturado en este producto hijo, distinto al del padre. Si el padre propaga este atributo, se reemplaza."
             : `Valor heredado${parentUpc ? ` del producto ${parentUpc}` : " del producto padre"}.`
         }
       >
         {isOverridden ? "Propio" : "Heredado"}
       </span>
 
-      {isOverridden && onRestore && (
+      {/* `Boolean`, no `isOverridden` a secas.
+
+          `is_overridden` llega como NÚMERO (0/1) desde la API y el reducer. Con
+          `{isOverridden && ...}`, un 0 se evalúa a 0 y React IMPRIME el texto
+          «0» — ese era el cero que aparecía en el hueco de la acción justo
+          después de pulsar «Volver a heredar», que es cuando el valor pasa a 0. */}
+      {Boolean(isOverridden) && onRestore && (
         <button type="button" className="tag-action" onClick={onRestore}>
           {restoreLabel}
         </button>

package/src/components/molecules/TagAndInput/index.js

@@ -321,7 +321,9 @@ export const TagAndInput = ({
       className={"input-container"}
       key={`generalTagInput-${inputType}`}
     >
-      {label?.length && (
+      {/* `Boolean`, no `label?.length`: con la etiqueta en cadena vacía la
+          expresión valía 0 y React imprimía un «0» en la ficha. */}
+      {Boolean(label?.length) && (
         <div className="title-container">
           <div className="titles">
             <ScreenHeader

package/src/components/molecules/TagAndInputv2/index.js

@@ -46,7 +46,9 @@ export const TagAndInputv2 = ({
       className={"input-container"}
       key={`generalTagInput-${inputType}`}
     >
-      {label?.length && (
+      {/* `Boolean`, no `label?.length`: con la etiqueta en cadena vacía la
+          expresión valía 0 y React imprimía un «0» en la ficha. */}
+      {Boolean(label?.length) && (
         <div className="title-container">
           <ScreenHeaderv2
             text={label}

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

@@ -21,7 +21,10 @@ import {
 
   props (* requerido):
     - *changedAttributes: [{ id, name, value }] lo que se acaba de editar
-    - *variants: [{ articleId, name, upc }] los hijos activos del padre
+    - *variants: [{ articleId, name, upc, overriddenAttributes }] los hijos
+        activos del padre. `overriddenAttributes` son los atributos que ese hijo
+        capturó a mano; la propagación los sobrescribe, y este diálogo es el
+        único lugar donde el usuario puede enterarse antes de que pase.
     - *onCancel: cerrar sin guardar nada
     - *onConfirm: (propagation | null) => void
         null            → guardar solo el padre
@@ -49,6 +52,31 @@ export const PropagateChangesModal = ({
   const puedePropagar = chosenAttributes.length > 0 && destinatarios > 0;
   const enSegundoPlano = destinatarios > UMBRAL_SEGUNDO_PLANO;
 
+  /*
+    A quién se le va a pisar un valor capturado a mano.
+
+    La propagación sobrescribe: eso se decidió a propósito, porque el usuario ya
+    eligió atributo por atributo y hijo por hijo. Pero sobrescribir en silencio
+    es otra cosa, y desde que se quitaron las etiquetas de «Heredado» y «Propio»
+    este diálogo es el único momento en que alguien puede saberlo.
+
+    Se recalcula con lo que está marcado ahora, no al abrir: si el usuario
+    desmarca el atributo conflictivo, el aviso desaparece — que es justo la
+    salida que el aviso debería estar ofreciendo.
+  */
+  const destinos = scope === "all"
+    ? variants
+    : variants.filter((item) => chosenChildren.includes(item.articleId));
+
+  const conValorPropio = destinos
+    .map((hijo) => ({
+      ...hijo,
+      pisados: (hijo.overriddenAttributes ?? []).filter((id) =>
+        chosenAttributes.includes(Number(id))
+      ),
+    }))
+    .filter((hijo) => hijo.pisados.length > 0);
+
   const toggleAttribute = (id) => {
     setChosenAttributes((current) =>
       current.includes(id)
@@ -155,13 +183,20 @@ export const PropagateChangesModal = ({
             </List>
           )}
 
-          {/* La diferencia con "ligar", y la que sorprende si no se dice: aquí
-              lo que el hijo editó a mano NO se pisa. */}
-          <Note>
-            Un hijo que haya editado alguno de estos atributos por su cuenta
-            conserva su valor. Para devolverlo al del padre está la acción
-            «Volver a heredar» en la ficha de ese hijo.
-          </Note>
+          {/* Lo que sorprende si no se dice: aquí se pisa lo que el hijo
+              hubiera capturado a mano. Se nombra a quién, no solo cuántos: con
+              «2 hijos» nadie sabe si le importa, con los nombres sí. */}
+          {conValorPropio.length > 0 && (
+            <Note className="accent">
+              {conValorPropio.length === 1
+                ? "Un hijo tiene un valor propio en lo que elegiste y se va a reemplazar: "
+                : `${conValorPropio.length} hijos tienen valores propios en lo que elegiste y se van a reemplazar: `}
+              {conValorPropio
+                .map((hijo) => `${hijo.name || `Producto ${hijo.articleId}`} (${hijo.pisados.length})`)
+                .join(", ")}
+              . Si no quieres pisarlos, desmarca ese atributo o ese hijo.
+            </Note>
+          )}
 
           {enSegundoPlano && (
             <Note className="accent">

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

@@ -1131,17 +1131,73 @@ const RetailerProductEditionView = ({
         await 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
-        // separando de él sin que nadie lo note.
-        const esPadre = hijosDelProducto > 0;
         const cambios = cambiosDeFichaTecnica();
 
-        if (esPadre && cambios.length > 0) {
-          setPreparandoPropagacion(true);
-          loadFamily()
-            .then(() => setPendingPropagation({ cambios }))
-            .finally(() => setPreparandoPropagacion(false));
+        if (cambios.length === 0) {
+          await saveDatasheets(token);
+          break;
+        }
+
+        /*
+          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.
+
+          Si tiene hijos se resuelve AQUÍ, esperando la respuesta, en vez de
+          mirar la copia en memoria. `family` es una consulta asíncrona: si
+          todavía no llegó, o si falló, `hijosDelProducto` vale 0 — o sea que
+          «todavía no sé» se leía igual que «no tiene hijos», y el padre se
+          guardaba sin ofrecer la propagación. En silencio, que es lo peor: el
+          usuario creía haber bajado el cambio a los hijos.
+
+          La ficha de la aplicación de clientes no tiene el problema porque ahí
+          `state.product.variantsCount` viene del listado; aquí el producto sale
+          del endpoint de reportes, que no trae el parentesco.
+        */
+        setPreparandoPropagacion(true);
+        let familia = null;
+        try {
+          familia = await loadFamily(true);
+        } finally {
+          setPreparandoPropagacion(false);
+        }
+
+        // No se pudo saber si tiene hijos. Guardar de todas formas repetiría el
+        // defecto que se está corrigiendo, así que se dice y se deja la decisión.
+        if (!familia || familia.error) {
+          dispatch({
+            type: "SET_MODAL",
+            payload: {
+              show: true,
+              title: "No se pudo consultar la familia",
+              message:
+                "No pudimos saber si este producto tiene hijos, así que no sabemos si hay que bajarles el cambio. Vuelve a intentar el guardado; si sigue fallando, guarda y revisa los hijos a mano.",
+              image: warningIcon,
+              buttons: [
+                {
+                  text: "Entendido",
+                  buttonType: "general-white-button",
+                  action: () =>
+                    dispatch({ type: "SET_MODAL", payload: { show: false } }),
+                },
+                {
+                  text: "Guardar solo este producto",
+                  buttonType: "general-button-default",
+                  action: () => {
+                    dispatch({ type: "SET_MODAL", payload: { show: false } });
+                    saveDatasheets(token);
+                  },
+                },
+              ],
+            },
+          });
+          return;
+        }
+
+        const hijos = (familia.variants ?? []).filter((v) => !v.isCurrent);
+
+        if (hijos.length > 0) {
+          setPendingPropagation({ cambios });
           return;
         }