npm - contentoh-components-library - Versions diffs - 21.6.49 → 21.6.51 - Mend

contentoh-components-library 21.6.49 → 21.6.51

This diff represents the content of publicly available package versions that have been released to one of the supported registries. The information contained in this diff is provided for informational purposes only and reflects changes between package versions as they appear in their respective public registries.

Files changed (9) hide show

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

@@ -17,6 +17,8 @@ var _react = require("react");
 var _styles = require("./styles");
+var _nonPropagable = require("./non-propagable");
 var _jsxRuntime = require("react/jsx-runtime");
 /*
@@ -46,7 +48,7 @@ var UMBRAL_SEGUNDO_PLANO = 20;
 var PropagateChangesModal = function PropagateChangesModal(_ref) {
   var _ref$changedAttribute = _ref.changedAttributes,
-      changedAttributes = _ref$changedAttribute === void 0 ? [] : _ref$changedAttribute,
+      cambiosRecibidos = _ref$changedAttribute === void 0 ? [] : _ref$changedAttribute,
       _ref$variants = _ref.variants,
       variants = _ref$variants === void 0 ? [] : _ref$variants,
       _ref$groupMembers = _ref.groupMembers,
@@ -58,6 +60,15 @@ var PropagateChangesModal = function PropagateChangesModal(_ref) {
       _ref$saving = _ref.saving,
       saving = _ref$saving === void 0 ? false : _ref$saving;
+  /*
+    Segundo filtro, a propósito redundante: quien llama ya debería quitarlos —de
+    eso depende que el diálogo no se abra por un cambio que no propaga nada—,
+    pero ofrecer el UPC y descartarlo en el servidor es exactamente el camino
+    por el que un usuario concluye que la propagación no sirve. Vale más
+    repetirlo aquí que confiar en que ningún llamador futuro lo olvide.
+  */
+  var changedAttributes = (0, _nonPropagable.soloPropagables)(cambiosRecibidos);
   var _useState = (0, _react.useState)([]),
       _useState2 = (0, _slicedToArray2.default)(_useState, 2),
       chosenAttributes = _useState2[0],

package/dist/components/organisms/PropagateChangesModal/non-propagable.js ADDED Viewed

@@ -0,0 +1,45 @@
+"use strict";
+Object.defineProperty(exports, "__esModule", {
+  value: true
+});
+exports.soloPropagables = exports.UPC_ATTRIBUTE_ID = exports.SKU_ATTRIBUTE_ID = exports.NON_PROPAGABLE_ATTRIBUTES = void 0;
+/*
+  Atributos que la propagación nunca copia.
+  UPC y SKU son la IDENTIDAD del producto, y una variante existe precisamente
+  porque la suya es distinta: el acuerdo lo dice literal —"cambian UPC/color/
+  talla"—. Bajarlos del padre dejaría dos productos con el mismo código de
+  barras, que es lo contrario de una variante.
+  El servidor ya los descarta y esa es la defensa de verdad: la exclusión vive
+  allá, no aquí, y aunque esta pantalla los mandara no se copiarían. Lo de este
+  archivo es otra cosa —que el diálogo no los OFREZCA—, porque ofrecer una
+  opción que se descarta en silencio es peor que no ofrecerla: el usuario elige
+  el UPC, guarda, no ve error, y concluye que la propagación está rota.
+  Los ids son los mismos que `variants/utils/constants.js` en el backend. Están
+  duplicados porque no hay un contrato compartido entre repos; si allá cambian,
+  aquí hay que cambiarlos.
+*/
+var UPC_ATTRIBUTE_ID = 683;
+exports.UPC_ATTRIBUTE_ID = UPC_ATTRIBUTE_ID;
+var SKU_ATTRIBUTE_ID = 40001;
+exports.SKU_ATTRIBUTE_ID = SKU_ATTRIBUTE_ID;
+var NON_PROPAGABLE_ATTRIBUTES = [UPC_ATTRIBUTE_ID, SKU_ATTRIBUTE_ID];
+/**
+ * Quita de una lista de cambios los que no se pueden propagar.
+ * @param {Array<{id: number}>} cambios
+ */
+exports.NON_PROPAGABLE_ATTRIBUTES = NON_PROPAGABLE_ATTRIBUTES;
+var soloPropagables = function soloPropagables() {
+  var cambios = arguments.length > 0 && arguments[0] !== undefined ? arguments[0] : [];
+  return cambios.filter(function (c) {
+    return !NON_PROPAGABLE_ATTRIBUTES.includes(Number(c === null || c === void 0 ? void 0 : c.id));
+  });
+};
+exports.soloPropagables = soloPropagables;

package/dist/components/pages/ProviderProductEdition/index.js CHANGED Viewed

@@ -73,6 +73,8 @@ var _VariantGroupModal = require("../../organisms/VariantGroupModal");
 var _PropagateChangesModal = require("../../organisms/PropagateChangesModal");
+var _nonPropagable = require("../../organisms/PropagateChangesModal/non-propagable");
 var _api = require("../../organisms/LinkVariantsModal/api");
 var _ProductImageModal = require("../../organisms/ProductImageModal");
@@ -94,7 +96,7 @@ var _AiProductEdition = require("../../../contexts/AiProductEdition");
 var _jsxRuntime = require("react/jsx-runtime");
 var ProviderProductEditionView = function ProviderProductEditionView(_ref) {
-  var _location$state4, _state$product2, _state$product3, _state$product4, _state$product5, _state$product6, _state$product7, _state$product15, _family$variants2, _variantGroup$members2, _variantGroup$group, _state$product20, _state$product21, _state$product22, _state$active_retaile4, _state$product23, _state$product24, _state$product25, _variantGroup$group2, _location$state5, _location$state6, _state$active_retaile5, _state$active_retaile6, _state$images_values$2, _state$images_values10, _state$images_values11, _state$active_retaile7, _state$product26, _state$product27, _state$product28, _state$product29, _family$variants3, _family$variants4, _family$variants5, _variantGroup$members3, _variantGroup$members4, _state$active_percent, _state$active_percent2, _state$active_retaile8, _state$active_retaile9, _state$active_retaile10, _state$product35, _state$descriptions_i, _state$services2, _state$services2$data, _state$product36, _state$product37, _state$datasheets_inp, _state$datasheets_inp2, _state$datasheets_inp3, _state$product38, _state$images_values12;
+  var _state$product, _location$state4, _state$product3, _state$product4, _state$product5, _state$product6, _state$product7, _state$product8, _state$product17, _state$product22, _family$variants2, _variantGroup$members2, _variantGroup$group, _state$product23, _state$product24, _state$product25, _state$active_retaile4, _state$product26, _state$product27, _state$product28, _variantGroup$group2, _location$state5, _location$state6, _state$active_retaile5, _state$active_retaile6, _state$images_values$2, _state$images_values10, _state$images_values11, _state$active_retaile7, _state$product29, _state$product30, _state$product31, _state$product32, _family$variants3, _family$variants4, _family$variants5, _variantGroup$members3, _variantGroup$members4, _state$active_percent, _state$active_percent2, _state$active_retaile8, _state$active_retaile9, _state$active_retaile10, _state$product38, _state$descriptions_i, _state$services2, _state$services2$data, _state$product39, _state$product40, _state$datasheets_inp, _state$datasheets_inp2, _state$datasheets_inp3, _state$product41, _state$images_values12;
   var tabsSections = _ref.tabsSections,
       _ref$productSelected = _ref.productSelected,
@@ -174,6 +176,17 @@ var ProviderProductEditionView = function ProviderProductEditionView(_ref) {
       _useState18 = (0, _slicedToArray2.default)(_useState17, 2),
       showGroupModal = _useState18[0],
       setShowGroupModal = _useState18[1];
+  /*
+    Un hijo no se agrupa como variante (decisión de producto, 2026-09-25).
+     Agrupar declara «estos son el mismo producto en otra presentación», y un
+    hijo ya declaró de dónde deriva; tener las dos cosas lo dejaba con dos
+    respuestas distintas a la misma pregunta. El servidor lo rechaza con
+    VARIANT_GROUP_NOT_FOR_CHILD, así que ofrecer el botón aquí sería ofrecer
+    algo que siempre falla.
+  */
+  var puedeAgruparse = !((_state$product = state.product) !== null && _state$product !== void 0 && _state$product.parentArticleId);
   var _useState19 = (0, _react.useState)(null),
       _useState20 = (0, _slicedToArray2.default)(_useState19, 2),
@@ -358,10 +371,10 @@ var ProviderProductEditionView = function ProviderProductEditionView(_ref) {
   (0, _react.useEffect)(function () {
     if (!state.product || !state.active_retailer) return;
     console.log("Fetch services and percentages");
-    var _state$product = state.product,
-        id_article = _state$product.id_article,
-        version = _state$product.version,
-        id_order = _state$product.id_order;
+    var _state$product2 = state.product,
+        id_article = _state$product2.id_article,
+        version = _state$product2.version,
+        id_order = _state$product2.id_order;
     var _state$active_retaile = state.active_retailer,
         category = _state$active_retaile.category,
         id_category = _state$active_retaile.id_category,
@@ -519,7 +532,7 @@ var ProviderProductEditionView = function ProviderProductEditionView(_ref) {
     }();
     fetchData();
-  }, [(_state$product2 = state.product) === null || _state$product2 === void 0 ? void 0 : _state$product2.datasheet_status, (_state$product3 = state.product) === null || _state$product3 === void 0 ? void 0 : _state$product3.description_status, (_state$product4 = state.product) === null || _state$product4 === void 0 ? void 0 : _state$product4.images_status, (_state$product5 = state.product) === null || _state$product5 === void 0 ? void 0 : _state$product5.status, (_state$product6 = state.product) === null || _state$product6 === void 0 ? void 0 : _state$product6.percentages, (_state$product7 = state.product) === null || _state$product7 === void 0 ? void 0 : _state$product7.version, state.active_retailer, token, linkNonce]); // Actualizador de campos del ui para que coincidan con el retailer activo
+  }, [(_state$product3 = state.product) === null || _state$product3 === void 0 ? void 0 : _state$product3.datasheet_status, (_state$product4 = state.product) === null || _state$product4 === void 0 ? void 0 : _state$product4.description_status, (_state$product5 = state.product) === null || _state$product5 === void 0 ? void 0 : _state$product5.images_status, (_state$product6 = state.product) === null || _state$product6 === void 0 ? void 0 : _state$product6.status, (_state$product7 = state.product) === null || _state$product7 === void 0 ? void 0 : _state$product7.percentages, (_state$product8 = state.product) === null || _state$product8 === void 0 ? void 0 : _state$product8.version, state.active_retailer, token, linkNonce]); // Actualizador de campos del ui para que coincidan con el retailer activo
   (0, _react.useEffect)(function () {
     if (!state.active_retailer) return;
@@ -578,9 +591,9 @@ var ProviderProductEditionView = function ProviderProductEditionView(_ref) {
   }, [state.services]); // Helper functions
   var canShowValidationButtons = function canShowValidationButtons() {
-    var _state$product8;
+    var _state$product9;
-    if (!((_state$product8 = state.product) !== null && _state$product8 !== void 0 && _state$product8.statusByRetailer)) return false;
+    if (!((_state$product9 = state.product) !== null && _state$product9 !== void 0 && _state$product9.statusByRetailer)) return false;
     try {
       var currentService = (0, _utils.getConceptByTab)(state.active_tab); // const currentStatus = state.product[`${currentService}_status`];
@@ -701,9 +714,9 @@ var ProviderProductEditionView = function ProviderProductEditionView(_ref) {
   };
   var handleOnClickSaveImages = function handleOnClickSaveImages() {
-    var _state$product9, _state$product9$servi;
+    var _state$product10, _state$product10$serv;
-    if (((_state$product9 = state.product) === null || _state$product9 === void 0 ? void 0 : (_state$product9$servi = _state$product9.services) === null || _state$product9$servi === void 0 ? void 0 : _state$product9$servi.images) === 1) {
+    if (((_state$product10 = state.product) === null || _state$product10 === void 0 ? void 0 : (_state$product10$serv = _state$product10.services) === null || _state$product10$serv === void 0 ? void 0 : _state$product10$serv.images) === 1) {
       updateImages();
     }
   };
@@ -858,7 +871,7 @@ var ProviderProductEditionView = function ProviderProductEditionView(_ref) {
     var retailerId = (_state$active_retaile2 = state.active_retailer) === null || _state$active_retaile2 === void 0 ? void 0 : _state$active_retaile2.id_retailer;
     var inputs = (_state$services$datas = (_state$services = state.services) === null || _state$services === void 0 ? void 0 : (_state$services$datas2 = _state$services.datasheets) === null || _state$services$datas2 === void 0 ? void 0 : _state$services$datas2.inputs) !== null && _state$services$datas !== void 0 ? _state$services$datas : {};
-    return ((_state$updated_datash = state.updated_datasheets_inputs) !== null && _state$updated_datash !== void 0 ? _state$updated_datash : []).map(function (item) {
+    var cambios = ((_state$updated_datash = state.updated_datasheets_inputs) !== null && _state$updated_datash !== void 0 ? _state$updated_datash : []).map(function (item) {
       var _input$name;
       var input = inputs["".concat(item.attributeId, "_").concat(retailerId)];
@@ -868,6 +881,14 @@ var ProviderProductEditionView = function ProviderProductEditionView(_ref) {
         value: item.value
       };
     });
+    /*
+      UPC y SKU se quitan aquí y no dentro del diálogo porque esta lista también
+      decide SI el diálogo se abre. Editar solo el UPC no tiene nada que
+      propagar, así que preguntarlo sería pedirle al usuario una decisión sobre
+      un conjunto vacío.
+    */
+    return (0, _nonPropagable.soloPropagables)(cambios);
   };
   var handleOnClickSave = function handleOnClickSave() {
@@ -882,7 +903,7 @@ var ProviderProductEditionView = function ProviderProductEditionView(_ref) {
       case "Ficha técnica":
         {
-          var _variantGroup$members, _state$product10;
+          var _variantGroup$members, _state$product11, _state$product12;
           // 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
@@ -896,7 +917,21 @@ var ProviderProductEditionView = function ProviderProductEditionView(_ref) {
           var companeras = ((_variantGroup$members = variantGroup === null || variantGroup === void 0 ? void 0 : variantGroup.members) !== null && _variantGroup$members !== void 0 ? _variantGroup$members : []).filter(function (m) {
             return !m.isCurrent;
           });
-          var hayDestinos = ((_state$product10 = state.product) === null || _state$product10 === void 0 ? void 0 : _state$product10.variantsCount) > 0 || companeras.length > 0;
+          /*
+            Un HIJO no propaga a nadie, y hay que decirlo explícitamente.
+             Sin este corte el diálogo se abría desde la ficha de un hijo y le
+            ofrecía a sus HERMANOS como si fueran sus hijos: `family.variants`
+            mira la familia desde el padre —eso es lo correcto para navegar entre
+            hermanos— pero aquí se estaba leyendo como «los destinos de este
+            producto». El servidor resolvía los destinos por su cuenta y no
+            escribía nada, así que el usuario elegía hermanos, guardaba, y no
+            pasaba nada: un no-operativo silencioso.
+             Un hijo tampoco puede estar agrupado desde 2026-09-25, así que para
+            él no queda ningún destino posible.
+          */
+          var esHijo = !!((_state$product11 = state.product) !== null && _state$product11 !== void 0 && _state$product11.parentArticleId);
+          var hayDestinos = !esHijo && (((_state$product12 = state.product) === null || _state$product12 === void 0 ? void 0 : _state$product12.variantsCount) > 0 || companeras.length > 0);
           var cambios = cambiosDeFichaTecnica();
           if (hayDestinos && cambios.length > 0) {
@@ -963,7 +998,7 @@ var ProviderProductEditionView = function ProviderProductEditionView(_ref) {
   var loadFamily = /*#__PURE__*/function () {
     var _ref7 = (0, _asyncToGenerator2.default)( /*#__PURE__*/_regenerator.default.mark(function _callee3() {
       var forzar,
-          _state$product11,
+          _state$product13,
           res,
           _args3 = arguments;
@@ -984,7 +1019,7 @@ var ProviderProductEditionView = function ProviderProductEditionView(_ref) {
               _context3.prev = 3;
               _context3.next = 6;
               return _axios.default.post("".concat(process.env.REACT_APP_VARIANTS_ENDPOINT, "/family"), {
-                articleId: (_state$product11 = state.product) === null || _state$product11 === void 0 ? void 0 : _state$product11.id_article
+                articleId: (_state$product13 = state.product) === null || _state$product13 === void 0 ? void 0 : _state$product13.id_article
               }, {
                 headers: {
                   Authorization: token
@@ -1163,14 +1198,14 @@ var ProviderProductEditionView = function ProviderProductEditionView(_ref) {
   var handleOnUnlinked = function handleOnUnlinked(result) {
-    var _state$product$varian, _state$product12, _updatedProduct$varia;
+    var _state$product$varian, _state$product14, _updatedProduct$varia;
     if (!result) return;
     var updatedProduct = result.mode === "self" ? (0, _objectSpread4.default)((0, _objectSpread4.default)({}, state.product), {}, {
       parentArticleId: null,
       variantsCount: 0
     }) : (0, _objectSpread4.default)((0, _objectSpread4.default)({}, state.product), {}, {
-      variantsCount: Math.max(((_state$product$varian = (_state$product12 = state.product) === null || _state$product12 === void 0 ? void 0 : _state$product12.variantsCount) !== null && _state$product$varian !== void 0 ? _state$product$varian : 1) - 1, 0)
+      variantsCount: Math.max(((_state$product$varian = (_state$product14 = state.product) === null || _state$product14 === void 0 ? void 0 : _state$product14.variantsCount) !== null && _state$product$varian !== void 0 ? _state$product$varian : 1) - 1, 0)
     });
     dispatch({
       type: "SET_PRODUCT",
@@ -1220,7 +1255,7 @@ var ProviderProductEditionView = function ProviderProductEditionView(_ref) {
   var loadVariantGroup = /*#__PURE__*/function () {
     var _ref9 = (0, _asyncToGenerator2.default)( /*#__PURE__*/_regenerator.default.mark(function _callee5() {
       var forzar,
-          _state$product13,
+          _state$product15,
           res,
           data,
           _args5 = arguments;
@@ -1242,7 +1277,7 @@ var ProviderProductEditionView = function ProviderProductEditionView(_ref) {
               _context5.prev = 3;
               _context5.next = 6;
               return _axios.default.post("".concat(process.env.REACT_APP_VARIANTS_ENDPOINT, "/group"), {
-                articleId: (_state$product13 = state.product) === null || _state$product13 === void 0 ? void 0 : _state$product13.id_article
+                articleId: (_state$product15 = state.product) === null || _state$product15 === void 0 ? void 0 : _state$product15.id_article
               }, {
                 headers: {
                   Authorization: token
@@ -1280,16 +1315,16 @@ var ProviderProductEditionView = function ProviderProductEditionView(_ref) {
   }();
   (0, _react.useEffect)(function () {
-    var _state$product14;
+    var _state$product16;
-    if (!((_state$product14 = state.product) !== null && _state$product14 !== void 0 && _state$product14.id_article)) return;
+    if (!((_state$product16 = state.product) !== null && _state$product16 !== void 0 && _state$product16.id_article)) return;
     loadVariantGroup(true); // eslint-disable-next-line react-hooks/exhaustive-deps
-  }, [(_state$product15 = state.product) === null || _state$product15 === void 0 ? void 0 : _state$product15.id_article]); // Saca el producto del grupo. Los demás miembros siguen agrupados entre ellos;
+  }, [(_state$product17 = state.product) === null || _state$product17 === void 0 ? void 0 : _state$product17.id_article]); // Saca el producto del grupo. Los demás miembros siguen agrupados entre ellos;
   // si era el último, el grupo deja de existir, porque no vive en ninguna tabla.
   var handleOnLeaveGroup = /*#__PURE__*/function () {
     var _ref10 = (0, _asyncToGenerator2.default)( /*#__PURE__*/_regenerator.default.mark(function _callee6() {
-      var _state$product16, res;
+      var _state$product18, res;
       return _regenerator.default.wrap(function _callee6$(_context6) {
         while (1) {
@@ -1298,7 +1333,7 @@ var ProviderProductEditionView = function ProviderProductEditionView(_ref) {
               _context6.prev = 0;
               _context6.next = 3;
               return _axios.default.post("".concat(process.env.REACT_APP_VARIANTS_ENDPOINT, "/leave"), {
-                articleId: (_state$product16 = state.product) === null || _state$product16 === void 0 ? void 0 : _state$product16.id_article
+                articleId: (_state$product18 = state.product) === null || _state$product18 === void 0 ? void 0 : _state$product18.id_article
               }, {
                 headers: {
                   Authorization: token
@@ -1436,11 +1471,11 @@ var ProviderProductEditionView = function ProviderProductEditionView(_ref) {
   var handleOnLinkedVariants = function handleOnLinkedVariants(result) {
-    var _state$product$varian2, _state$product17, _result$linked, _result$parentArticle;
+    var _state$product$varian2, _state$product19, _result$linked, _result$parentArticle;
     if (!result) return;
     var updatedProduct = result.mode === "children" ? (0, _objectSpread4.default)((0, _objectSpread4.default)({}, state.product), {}, {
-      variantsCount: ((_state$product$varian2 = (_state$product17 = state.product) === null || _state$product17 === void 0 ? void 0 : _state$product17.variantsCount) !== null && _state$product$varian2 !== void 0 ? _state$product$varian2 : 0) + ((_result$linked = result.linked) !== null && _result$linked !== void 0 ? _result$linked : []).length
+      variantsCount: ((_state$product$varian2 = (_state$product19 = state.product) === null || _state$product19 === void 0 ? void 0 : _state$product19.variantsCount) !== null && _state$product$varian2 !== void 0 ? _state$product$varian2 : 0) + ((_result$linked = result.linked) !== null && _result$linked !== void 0 ? _result$linked : []).length
     }) : (0, _objectSpread4.default)((0, _objectSpread4.default)({}, state.product), {}, {
       parentArticleId: (_result$parentArticle = result.parentArticleId) !== null && _result$parentArticle !== void 0 ? _result$parentArticle : null,
       variantsCount: 0
@@ -1499,7 +1534,7 @@ var ProviderProductEditionView = function ProviderProductEditionView(_ref) {
   var handleOnRestoreInheritance = /*#__PURE__*/function () {
     var _ref12 = (0, _asyncToGenerator2.default)( /*#__PURE__*/_regenerator.default.mark(function _callee8(attributeId) {
-      var _state$product18;
+      var _state$product20;
       var kind,
           esDescripcion,
@@ -1512,7 +1547,7 @@ var ProviderProductEditionView = function ProviderProductEditionView(_ref) {
             case 0:
               kind = _args8.length > 1 && _args8[1] !== undefined ? _args8[1] : "attribute";
-              if (!(!attributeId || !((_state$product18 = state.product) !== null && _state$product18 !== void 0 && _state$product18.id_article))) {
+              if (!(!attributeId || !((_state$product20 = state.product) !== null && _state$product20 !== void 0 && _state$product20.id_article))) {
                 _context8.next = 3;
                 break;
               }
@@ -1622,7 +1657,7 @@ var ProviderProductEditionView = function ProviderProductEditionView(_ref) {
   var sendEvaluation = /*#__PURE__*/function () {
     var _ref13 = (0, _asyncToGenerator2.default)( /*#__PURE__*/_regenerator.default.mark(function _callee9(result) {
-      var _state$product$id_ord, _state$active_retaile3, _state$product$status, _state$product$status2, _state$product19;
+      var _state$product$id_ord, _state$active_retaile3, _state$product$status, _state$product$status2, _state$product21;
       var concept, articleId, orderId, retailerId, evalStatus, data, _newStatuses$newStatu, _state$services_data, res, newStatuses, serviceStatus, updatedStatusByRetailer, updatedProduct, updatedServicesData;
@@ -1638,7 +1673,7 @@ var ProviderProductEditionView = function ProviderProductEditionView(_ref) {
               articleId = state.product.id_article;
               orderId = (_state$product$id_ord = state.product.id_order) !== null && _state$product$id_ord !== void 0 ? _state$product$id_ord : state.product.orderId;
               retailerId = (_state$active_retaile3 = state.active_retailer) === null || _state$active_retaile3 === void 0 ? void 0 : _state$active_retaile3.id_retailer;
-              evalStatus = ((_state$product$status = state.product.statusByRetailer) === null || _state$product$status === void 0 ? void 0 : (_state$product$status2 = _state$product$status[retailerId]) === null || _state$product$status2 === void 0 ? void 0 : _state$product$status2[concept]) || state.product["".concat(concept, "_status")] || ((_state$product19 = state.product) === null || _state$product19 === void 0 ? void 0 : _state$product19.version_status);
+              evalStatus = ((_state$product$status = state.product.statusByRetailer) === null || _state$product$status === void 0 ? void 0 : (_state$product$status2 = _state$product$status[retailerId]) === null || _state$product$status2 === void 0 ? void 0 : _state$product$status2[concept]) || state.product["".concat(concept, "_status")] || ((_state$product21 = state.product) === null || _state$product21 === void 0 ? void 0 : _state$product21.version_status);
               data = {
                 articleId: articleId,
                 orderId: orderId,
@@ -2122,7 +2157,7 @@ var ProviderProductEditionView = function ProviderProductEditionView(_ref) {
         jwt: token
       }), pendingPropagation && /*#__PURE__*/(0, _jsxRuntime.jsx)(_PropagateChangesModal.PropagateChangesModal, {
         changedAttributes: pendingPropagation.cambios,
-        variants: ((_family$variants2 = family === null || family === void 0 ? void 0 : family.variants) !== null && _family$variants2 !== void 0 ? _family$variants2 : []).filter(function (v) {
+        variants: (_state$product22 = state.product) !== null && _state$product22 !== void 0 && _state$product22.parentArticleId ? [] : ((_family$variants2 = family === null || family === void 0 ? void 0 : family.variants) !== null && _family$variants2 !== void 0 ? _family$variants2 : []).filter(function (v) {
           return !v.isCurrent;
         }),
         groupMembers: ((_variantGroup$members2 = variantGroup === null || variantGroup === void 0 ? void 0 : variantGroup.members) !== null && _variantGroup$members2 !== void 0 ? _variantGroup$members2 : []).filter(function (m) {
@@ -2138,9 +2173,9 @@ var ProviderProductEditionView = function ProviderProductEditionView(_ref) {
           saveDatasheets(token, propagation !== null && propagation !== void 0 ? propagation : undefined);
         }
       }), showLinkVariants && /*#__PURE__*/(0, _jsxRuntime.jsx)(_LinkVariantsModal.LinkVariantsModal, {
-        articleId: (_state$product20 = state.product) === null || _state$product20 === void 0 ? void 0 : _state$product20.id_article,
-        articleName: (_state$product21 = state.product) === null || _state$product21 === void 0 ? void 0 : _state$product21.name,
-        articleUpc: (_state$product22 = state.product) === null || _state$product22 === void 0 ? void 0 : _state$product22.upc,
+        articleId: (_state$product23 = state.product) === null || _state$product23 === void 0 ? void 0 : _state$product23.id_article,
+        articleName: (_state$product24 = state.product) === null || _state$product24 === void 0 ? void 0 : _state$product24.name,
+        articleUpc: (_state$product25 = state.product) === null || _state$product25 === void 0 ? void 0 : _state$product25.upc,
         activeRetailerId: (_state$active_retaile4 = state.active_retailer) === null || _state$active_retaile4 === void 0 ? void 0 : _state$active_retaile4.id_retailer,
         token: token,
         onClose: function onClose() {
@@ -2148,9 +2183,9 @@ var ProviderProductEditionView = function ProviderProductEditionView(_ref) {
         },
         onLinked: handleOnLinkedVariants
       }), showGroupModal && /*#__PURE__*/(0, _jsxRuntime.jsx)(_VariantGroupModal.VariantGroupModal, {
-        articleId: (_state$product23 = state.product) === null || _state$product23 === void 0 ? void 0 : _state$product23.id_article,
-        articleName: (_state$product24 = state.product) === null || _state$product24 === void 0 ? void 0 : _state$product24.name,
-        articleUpc: (_state$product25 = state.product) === null || _state$product25 === void 0 ? void 0 : _state$product25.upc,
+        articleId: (_state$product26 = state.product) === null || _state$product26 === void 0 ? void 0 : _state$product26.id_article,
+        articleName: (_state$product27 = state.product) === null || _state$product27 === void 0 ? void 0 : _state$product27.name,
+        articleUpc: (_state$product28 = state.product) === null || _state$product28 === void 0 ? void 0 : _state$product28.upc,
         currentGroup: (_variantGroup$group2 = variantGroup === null || variantGroup === void 0 ? void 0 : variantGroup.group) !== null && _variantGroup$group2 !== void 0 ? _variantGroup$group2 : null,
         token: token,
         onClose: function onClose() {
@@ -2309,7 +2344,7 @@ var ProviderProductEditionView = function ProviderProductEditionView(_ref) {
             activeImage: (_state$images_values10 = state.images_values) !== null && _state$images_values10 !== void 0 && _state$images_values10.values ? (_state$images_values11 = state.images_values) === null || _state$images_values11 === void 0 ? void 0 : _state$images_values11.values[state.current_image] : {},
             assignationsImages: assig,
             retailerSelected: (_state$active_retaile7 = state.active_retailer) === null || _state$active_retaile7 === void 0 ? void 0 : _state$active_retaile7.id_retailer,
-            imagesStatus: ((_state$product26 = state.product) === null || _state$product26 === void 0 ? void 0 : _state$product26.images_status) || "-",
+            imagesStatus: ((_state$product29 = state.product) === null || _state$product29 === void 0 ? void 0 : _state$product29.images_status) || "-",
             isRetailer: isRetailer,
             setShowVersionSelector: setShowVersionSelector,
             version: state.product.version,
@@ -2322,7 +2357,7 @@ var ProviderProductEditionView = function ProviderProductEditionView(_ref) {
           className: "product-information",
           children: /*#__PURE__*/(0, _jsxRuntime.jsxs)("div", {
             className: "product-information",
-            children: [!((_state$product27 = state.product) !== null && _state$product27 !== void 0 && _state$product27.parentArticleId) && /*#__PURE__*/(0, _jsxRuntime.jsxs)("div", {
+            children: [!((_state$product30 = state.product) !== null && _state$product30 !== void 0 && _state$product30.parentArticleId) && /*#__PURE__*/(0, _jsxRuntime.jsxs)("div", {
               className: "variants-actions",
               children: [/*#__PURE__*/(0, _jsxRuntime.jsx)("button", {
                 type: "button",
@@ -2331,7 +2366,7 @@ var ProviderProductEditionView = function ProviderProductEditionView(_ref) {
                   return setShowLinkVariants(true);
                 },
                 children: "Ligar productos"
-              }), ((_state$product28 = state.product) === null || _state$product28 === void 0 ? void 0 : _state$product28.variantsCount) > 0 && /*#__PURE__*/(0, _jsxRuntime.jsxs)("button", {
+              }), ((_state$product31 = state.product) === null || _state$product31 === void 0 ? void 0 : _state$product31.variantsCount) > 0 && /*#__PURE__*/(0, _jsxRuntime.jsxs)("button", {
                 type: "button",
                 className: "variants-count",
                 "aria-expanded": showFamily,
@@ -2346,7 +2381,7 @@ var ProviderProductEditionView = function ProviderProductEditionView(_ref) {
                   children: showFamily ? " ▴" : " ▾"
                 })]
               })]
-            }), ((_state$product29 = state.product) === null || _state$product29 === void 0 ? void 0 : _state$product29.parentArticleId) && /*#__PURE__*/(0, _jsxRuntime.jsx)("div", {
+            }), ((_state$product32 = state.product) === null || _state$product32 === void 0 ? void 0 : _state$product32.parentArticleId) && /*#__PURE__*/(0, _jsxRuntime.jsx)("div", {
               className: "variants-actions",
               children: /*#__PURE__*/(0, _jsxRuntime.jsxs)("button", {
                 type: "button",
@@ -2395,13 +2430,13 @@ var ProviderProductEditionView = function ProviderProductEditionView(_ref) {
                   className: "desvincular",
                   title: "Dejar de ser hijo de este producto",
                   onClick: function onClick() {
-                    var _state$product30, _state$product31, _state$product32;
+                    var _state$product33, _state$product34, _state$product35;
                     return setPorDesvincular({
                       mode: "self",
-                      childArticleId: (_state$product30 = state.product) === null || _state$product30 === void 0 ? void 0 : _state$product30.id_article,
-                      childName: (_state$product31 = state.product) === null || _state$product31 === void 0 ? void 0 : _state$product31.name,
-                      childUpc: (_state$product32 = state.product) === null || _state$product32 === void 0 ? void 0 : _state$product32.upc,
+                      childArticleId: (_state$product33 = state.product) === null || _state$product33 === void 0 ? void 0 : _state$product33.id_article,
+                      childName: (_state$product34 = state.product) === null || _state$product34 === void 0 ? void 0 : _state$product34.name,
+                      childUpc: (_state$product35 = state.product) === null || _state$product35 === void 0 ? void 0 : _state$product35.upc,
                       parentName: family.parent.name,
                       parentUpc: family.parent.upc
                     });
@@ -2433,15 +2468,15 @@ var ProviderProductEditionView = function ProviderProductEditionView(_ref) {
                     className: "desvincular",
                     title: "Desvincular este hijo",
                     onClick: function onClick() {
-                      var _state$product33, _state$product34;
+                      var _state$product36, _state$product37;
                       return setPorDesvincular({
                         mode: "child",
                         childArticleId: variant.articleId,
                         childName: variant.name,
                         childUpc: variant.upc,
-                        parentName: (_state$product33 = state.product) === null || _state$product33 === void 0 ? void 0 : _state$product33.name,
-                        parentUpc: (_state$product34 = state.product) === null || _state$product34 === void 0 ? void 0 : _state$product34.upc
+                        parentName: (_state$product36 = state.product) === null || _state$product36 === void 0 ? void 0 : _state$product36.name,
+                        parentUpc: (_state$product37 = state.product) === null || _state$product37 === void 0 ? void 0 : _state$product37.upc
                       });
                     },
                     children: "Desvincular"
@@ -2457,7 +2492,7 @@ var ProviderProductEditionView = function ProviderProductEditionView(_ref) {
                 onClick: openMultipleEdition,
                 children: preparandoEdicionMultiple ? "Abriendo…" : "Editar los ".concat(family.variants.length, " hijos juntos")
               })]
-            }), /*#__PURE__*/(0, _jsxRuntime.jsxs)("div", {
+            }), puedeAgruparse && /*#__PURE__*/(0, _jsxRuntime.jsxs)("div", {
               className: "variants-actions",
               children: [/*#__PURE__*/(0, _jsxRuntime.jsx)("button", {
                 type: "button",
@@ -2480,7 +2515,7 @@ var ProviderProductEditionView = function ProviderProductEditionView(_ref) {
                   children: showGroup ? " ▴" : " ▾"
                 })]
               })]
-            }), showGroup && (variantGroup === null || variantGroup === void 0 ? void 0 : variantGroup.group) && /*#__PURE__*/(0, _jsxRuntime.jsxs)("div", {
+            }), puedeAgruparse && showGroup && (variantGroup === null || variantGroup === void 0 ? void 0 : variantGroup.group) && /*#__PURE__*/(0, _jsxRuntime.jsxs)("div", {
               className: "variants-family",
               children: [((_variantGroup$members3 = variantGroup.members) !== null && _variantGroup$members3 !== void 0 ? _variantGroup$members3 : []).filter(function (miembro) {
                 return !miembro.isCurrent;
@@ -2568,7 +2603,7 @@ var ProviderProductEditionView = function ProviderProductEditionView(_ref) {
             }), /*#__PURE__*/(0, _jsxRuntime.jsx)("div", {
               className: "services-information-container " + (imageLayout && state.active_tab === "Imágenes" ? "image-services" : ""),
               children: state.saving ? /*#__PURE__*/(0, _jsxRuntime.jsx)(_Loading.Loading, {}) : /*#__PURE__*/(0, _jsxRuntime.jsxs)(_jsxRuntime.Fragment, {
-                children: [state.active_tab === "Descripción" && (((_state$product35 = state.product) === null || _state$product35 === void 0 ? void 0 : _state$product35.description_status) !== "NS" ? (_state$descriptions_i = state.descriptions_inputs) !== null && _state$descriptions_i !== void 0 && _state$descriptions_i[0] ? /*#__PURE__*/(0, _jsxRuntime.jsx)(_InputGroup.InputGroup, {
+                children: [state.active_tab === "Descripción" && (((_state$product38 = state.product) === null || _state$product38 === void 0 ? void 0 : _state$product38.description_status) !== "NS" ? (_state$descriptions_i = state.descriptions_inputs) !== null && _state$descriptions_i !== void 0 && _state$descriptions_i[0] ? /*#__PURE__*/(0, _jsxRuntime.jsx)(_InputGroup.InputGroup, {
                   activeSection: state.active_tab,
                   inputGroup: state.descriptions_inputs[0],
                   updatedDescriptions: state.updated_descriptions_inputs
@@ -2579,7 +2614,7 @@ var ProviderProductEditionView = function ProviderProductEditionView(_ref) {
                   ,
                   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$product36 = state.product) === null || _state$product36 === void 0 ? void 0 : _state$product36.id_article,
+                  articleId: (_state$product39 = state.product) === null || _state$product39 === void 0 ? void 0 : _state$product39.id_article,
                   version: state.product.version,
                   isShowbox: false,
                   dinamicHeight: true,
@@ -2591,7 +2626,7 @@ var ProviderProductEditionView = function ProviderProductEditionView(_ref) {
                 }) : /*#__PURE__*/(0, _jsxRuntime.jsx)(_ScreenHeader.ScreenHeader, {
                   text: "No cuentas con este servicio",
                   headerType: "input-name-header"
-                })), state.active_tab === "Ficha técnica" && (((_state$product37 = state.product) === null || _state$product37 === void 0 ? void 0 : _state$product37.datasheet_status) !== "NS" ? /*#__PURE__*/(0, _jsxRuntime.jsx)(_jsxRuntime.Fragment, {
+                })), state.active_tab === "Ficha técnica" && (((_state$product40 = state.product) === null || _state$product40 === void 0 ? void 0 : _state$product40.datasheet_status) !== "NS" ? /*#__PURE__*/(0, _jsxRuntime.jsx)(_jsxRuntime.Fragment, {
                   children: ((_state$datasheets_inp = state.datasheets_inputs) === null || _state$datasheets_inp === void 0 ? void 0 : (_state$datasheets_inp2 = _state$datasheets_inp[0]) === null || _state$datasheets_inp2 === void 0 ? void 0 : (_state$datasheets_inp3 = _state$datasheets_inp2.data) === null || _state$datasheets_inp3 === void 0 ? void 0 : _state$datasheets_inp3.length) > 0 ? state.datasheets_inputs[0].data.map(function (dataGroup, index) {
                     var _state$datasheets_inp4, _state$services3, _state$services3$data, _state$services$datas3;
@@ -2621,7 +2656,7 @@ var ProviderProductEditionView = function ProviderProductEditionView(_ref) {
                 }) : /*#__PURE__*/(0, _jsxRuntime.jsx)(_ScreenHeader.ScreenHeader, {
                   text: "No cuentas con este servicio",
                   headerType: "input-name-header"
-                })), state.active_tab === "Imágenes" && (((_state$product38 = state.product) === null || _state$product38 === void 0 ? void 0 : _state$product38.images_status) !== "NS" ? /*#__PURE__*/(0, _jsxRuntime.jsxs)(_jsxRuntime.Fragment, {
+                })), state.active_tab === "Imágenes" && (((_state$product41 = state.product) === null || _state$product41 === void 0 ? void 0 : _state$product41.images_status) !== "NS" ? /*#__PURE__*/(0, _jsxRuntime.jsxs)(_jsxRuntime.Fragment, {
                   children: [!imageLayout && /*#__PURE__*/(0, _jsxRuntime.jsx)(_GalleryHeader.GalleryHeader, {
                     checkAll: state.all_image_checked // Determina si el checkbox "Seleccionar todo" está marcado
                     ,

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

@@ -81,6 +81,8 @@ var _VariantGroupModal = require("../../organisms/VariantGroupModal");
 var _PropagateChangesModal = require("../../organisms/PropagateChangesModal");
+var _nonPropagable = require("../../organisms/PropagateChangesModal/non-propagable");
 var _api = require("../../organisms/LinkVariantsModal/api");
 var _genericModalCheck = _interopRequireDefault(require("../../../assets/images/genericModal/genericModalCheck.svg"));
@@ -1314,6 +1316,18 @@ var RetailerProductEditionView = function RetailerProductEditionView(_ref) {
     return !v.isCurrent;
   }).length;
   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);
+  /*
+    Un hijo no se agrupa como variante (decisión de producto, 2026-09-25).
+     Agrupar declara «estos son el mismo producto en otra presentación», y un
+    hijo ya declaró de dónde deriva. El servidor lo rechaza con
+    VARIANT_GROUP_NOT_FOR_CHILD, así que ofrecer el botón sería ofrecer algo
+    que siempre falla.
+     Se apoya en `esHijoDeOtro` y no en `state.product` porque en esta pantalla
+    el producto sale de sessionStorage: la señal buena es la familia recién
+    consultada, con el producto solo como respaldo si esa consulta falló.
+  */
+  var puedeAgruparse = !esHijoDeOtro;
   /*
     Abre otro producto de la familia.
      Cambiar la URL no basta: esta pantalla no lee el producto de la ruta sino de
@@ -1797,7 +1811,7 @@ var RetailerProductEditionView = function RetailerProductEditionView(_ref) {
     var retailerId = (_state$active_retaile4 = state.active_retailer) === null || _state$active_retaile4 === void 0 ? void 0 : _state$active_retaile4.id_retailer;
     var inputs = (_state$services$datas = (_state$services = state.services) === null || _state$services === void 0 ? void 0 : (_state$services$datas2 = _state$services.datasheets) === null || _state$services$datas2 === void 0 ? void 0 : _state$services$datas2.inputs) !== null && _state$services$datas !== void 0 ? _state$services$datas : {};
-    return ((_state$updated_datash = state.updated_datasheets_inputs) !== null && _state$updated_datash !== void 0 ? _state$updated_datash : []).map(function (item) {
+    var cambios = ((_state$updated_datash = state.updated_datasheets_inputs) !== null && _state$updated_datash !== void 0 ? _state$updated_datash : []).map(function (item) {
       var _input$name;
       var input = inputs["".concat(item.attributeId, "_").concat(retailerId)];
@@ -1807,11 +1821,19 @@ var RetailerProductEditionView = function RetailerProductEditionView(_ref) {
         value: item.value
       };
     });
+    /*
+      UPC y SKU se quitan aquí y no dentro del diálogo porque esta lista también
+      decide SI el diálogo se abre. Editar solo el UPC no tiene nada que
+      propagar, así que preguntarlo sería pedirle al usuario una decisión sobre
+      un conjunto vacío.
+    */
+    return (0, _nonPropagable.soloPropagables)(cambios);
   };
   var handleOnClickSave = /*#__PURE__*/function () {
     var _ref16 = (0, _asyncToGenerator2.default)( /*#__PURE__*/_regenerator.default.mark(function _callee11() {
-      var _familia$variants, _variantGroup$members, cambios, familia, hijos, companeras;
+      var _familia, _familia$variants, _variantGroup$members, cambios, familia, esHijo, hijos, companeras;
       return _regenerator.default.wrap(function _callee11$(_context11) {
         while (1) {
@@ -1826,7 +1848,7 @@ var RetailerProductEditionView = function RetailerProductEditionView(_ref) {
             case 2:
               _context11.t0 = state.active_tab;
-              _context11.next = _context11.t0 === "Descripción" ? 5 : _context11.t0 === "Ficha técnica" ? 8 : _context11.t0 === "Imágenes" ? 33 : 36;
+              _context11.next = _context11.t0 === "Descripción" ? 5 : _context11.t0 === "Ficha técnica" ? 8 : _context11.t0 === "Imágenes" ? 34 : 37;
               break;
             case 5:
@@ -1834,7 +1856,7 @@ var RetailerProductEditionView = function RetailerProductEditionView(_ref) {
               return saveDescriptions(token);
             case 7:
-              return _context11.abrupt("break", 37);
+              return _context11.abrupt("break", 38);
             case 8:
               cambios = cambiosDeFichaTecnica();
@@ -1848,7 +1870,7 @@ var RetailerProductEditionView = function RetailerProductEditionView(_ref) {
               return saveDatasheets(token);
             case 12:
-              return _context11.abrupt("break", 37);
+              return _context11.abrupt("break", 38);
             case 13:
               /*
@@ -1921,7 +1943,17 @@ var RetailerProductEditionView = function RetailerProductEditionView(_ref) {
               return _context11.abrupt("return");
             case 25:
-              hijos = ((_familia$variants = familia.variants) !== null && _familia$variants !== void 0 ? _familia$variants : []).filter(function (v) {
+              /*
+                Un HIJO no propaga a nadie, y hay que decirlo explícitamente.
+                 `familia.variants` mira la familia desde el PADRE —eso es lo correcto
+                para navegar entre hermanos— así que desde la ficha de un hijo esa
+                lista son sus HERMANOS. Leerla como «los destinos de este producto»
+                hacía que el diálogo le ofreciera heredar a sus hermanos. El servidor
+                resolvía los destinos por su cuenta y no escribía nada, de modo que el
+                usuario elegía, guardaba, y no pasaba nada: un no-operativo silencioso.
+              */
+              esHijo = !!((_familia = familia) !== null && _familia !== void 0 && _familia.parent);
+              hijos = esHijo ? [] : ((_familia$variants = familia.variants) !== null && _familia$variants !== void 0 ? _familia$variants : []).filter(function (v) {
                 return !v.isCurrent;
               });
               /*
@@ -1932,10 +1964,11 @@ var RetailerProductEditionView = function RetailerProductEditionView(_ref) {
               companeras = ((_variantGroup$members = variantGroup === null || variantGroup === void 0 ? void 0 : variantGroup.members) !== null && _variantGroup$members !== void 0 ? _variantGroup$members : []).filter(function (m) {
                 return !m.isCurrent;
-              });
+              }); // Un hijo tampoco puede estar agrupado desde 2026-09-25, así que para
+              // él no queda ningún destino posible.
-              if (!(hijos.length > 0 || companeras.length > 0)) {
-                _context11.next = 30;
+              if (!(!esHijo && (hijos.length > 0 || companeras.length > 0))) {
+                _context11.next = 31;
                 break;
               }
@@ -1944,28 +1977,28 @@ var RetailerProductEditionView = function RetailerProductEditionView(_ref) {
               });
               return _context11.abrupt("return");
-            case 30:
-              _context11.next = 32;
+            case 31:
+              _context11.next = 33;
               return saveDatasheets(token);
-            case 32:
-              return _context11.abrupt("break", 37);
             case 33:
-              _context11.next = 35;
-              return updateImages(token);
+              return _context11.abrupt("break", 38);
-            case 35:
-              return _context11.abrupt("break", 37);
+            case 34:
+              _context11.next = 36;
+              return updateImages(token);
             case 36:
-              return _context11.abrupt("break", 37);
+              return _context11.abrupt("break", 38);
             case 37:
-              _context11.next = 39;
+              return _context11.abrupt("break", 38);
+            case 38:
+              _context11.next = 40;
               return loadData(false);
-            case 39:
+            case 40:
             case "end":
               return _context11.stop();
           }
@@ -2894,7 +2927,7 @@ var RetailerProductEditionView = function RetailerProductEditionView(_ref) {
       headerTop: headerTop,
       children: [pendingPropagation && /*#__PURE__*/(0, _jsxRuntime.jsx)(_PropagateChangesModal.PropagateChangesModal, {
         changedAttributes: pendingPropagation.cambios,
-        variants: ((_family$variants2 = family === null || family === void 0 ? void 0 : family.variants) !== null && _family$variants2 !== void 0 ? _family$variants2 : []).filter(function (v) {
+        variants: esHijoDeOtro ? [] : ((_family$variants2 = family === null || family === void 0 ? void 0 : family.variants) !== null && _family$variants2 !== void 0 ? _family$variants2 : []).filter(function (v) {
           return !v.isCurrent;
         }),
         groupMembers: ((_variantGroup$members2 = variantGroup === null || variantGroup === void 0 ? void 0 : variantGroup.members) !== null && _variantGroup$members2 !== void 0 ? _variantGroup$members2 : []).filter(function (m) {
@@ -3239,7 +3272,7 @@ var RetailerProductEditionView = function RetailerProductEditionView(_ref) {
               className: "loading",
               children: avisoFamilia
             })]
-          }), /*#__PURE__*/(0, _jsxRuntime.jsxs)("div", {
+          }), puedeAgruparse && /*#__PURE__*/(0, _jsxRuntime.jsxs)("div", {
             className: "variants-actions",
             children: [/*#__PURE__*/(0, _jsxRuntime.jsx)("button", {
               type: "button",
@@ -3262,7 +3295,7 @@ var RetailerProductEditionView = function RetailerProductEditionView(_ref) {
                 children: showGroup ? " ▴" : " ▾"
               })]
             })]
-          }), showGroup && (variantGroup === null || variantGroup === void 0 ? void 0 : variantGroup.group) && /*#__PURE__*/(0, _jsxRuntime.jsxs)("div", {
+          }), puedeAgruparse && showGroup && (variantGroup === null || variantGroup === void 0 ? void 0 : variantGroup.group) && /*#__PURE__*/(0, _jsxRuntime.jsxs)("div", {
             className: "variants-family",
             children: [((_variantGroup$members3 = variantGroup.members) !== null && _variantGroup$members3 !== void 0 ? _variantGroup$members3 : []).filter(function (miembro) {
               return !miembro.isCurrent;

package/package.json CHANGED Viewed

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

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

@@ -10,6 +10,7 @@ import {
   Note,
   Footer,
 } from "./styles";
+import { soloPropagables } from "./non-propagable";
 /*
   Diálogo de propagación al guardar un producto padre.
@@ -38,7 +39,7 @@ import {
 const UMBRAL_SEGUNDO_PLANO = 20;
 export const PropagateChangesModal = ({
-  changedAttributes = [],
+  changedAttributes: cambiosRecibidos = [],
   variants = [],
   groupMembers = [],
   groupCode = null,
@@ -46,6 +47,15 @@ export const PropagateChangesModal = ({
   onConfirm,
   saving = false,
 }) => {
+  /*
+    Segundo filtro, a propósito redundante: quien llama ya debería quitarlos —de
+    eso depende que el diálogo no se abra por un cambio que no propaga nada—,
+    pero ofrecer el UPC y descartarlo en el servidor es exactamente el camino
+    por el que un usuario concluye que la propagación no sirve. Vale más
+    repetirlo aquí que confiar en que ningún llamador futuro lo olvide.
+  */
+  const changedAttributes = soloPropagables(cambiosRecibidos);
   const [chosenAttributes, setChosenAttributes] = useState([]);
   /*

package/src/components/organisms/PropagateChangesModal/non-propagable.js ADDED Viewed

@@ -0,0 +1,29 @@
+/*
+  Atributos que la propagación nunca copia.
+  UPC y SKU son la IDENTIDAD del producto, y una variante existe precisamente
+  porque la suya es distinta: el acuerdo lo dice literal —"cambian UPC/color/
+  talla"—. Bajarlos del padre dejaría dos productos con el mismo código de
+  barras, que es lo contrario de una variante.
+  El servidor ya los descarta y esa es la defensa de verdad: la exclusión vive
+  allá, no aquí, y aunque esta pantalla los mandara no se copiarían. Lo de este
+  archivo es otra cosa —que el diálogo no los OFREZCA—, porque ofrecer una
+  opción que se descarta en silencio es peor que no ofrecerla: el usuario elige
+  el UPC, guarda, no ve error, y concluye que la propagación está rota.
+  Los ids son los mismos que `variants/utils/constants.js` en el backend. Están
+  duplicados porque no hay un contrato compartido entre repos; si allá cambian,
+  aquí hay que cambiarlos.
+*/
+export const UPC_ATTRIBUTE_ID = 683;
+export const SKU_ATTRIBUTE_ID = 40001;
+export const NON_PROPAGABLE_ATTRIBUTES = [UPC_ATTRIBUTE_ID, SKU_ATTRIBUTE_ID];
+/**
+ * Quita de una lista de cambios los que no se pueden propagar.
+ * @param {Array<{id: number}>} cambios
+ */
+export const soloPropagables = (cambios = []) =>
+  cambios.filter((c) => !NON_PROPAGABLE_ATTRIBUTES.includes(Number(c?.id)));

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

@@ -42,6 +42,7 @@ import { LinkVariantsModal } from "../../organisms/LinkVariantsModal";
 import { UnlinkVariantModal } from "../../organisms/UnlinkVariantModal";
 import { VariantGroupModal } from "../../organisms/VariantGroupModal";
 import { PropagateChangesModal } from "../../organisms/PropagateChangesModal";
+import { soloPropagables } from "../../organisms/PropagateChangesModal/non-propagable";
 import { readPayload } from "../../organisms/LinkVariantsModal/api";
 import { ProductImageModal } from "../../organisms/ProductImageModal";
 import { VersionSelector } from "../../organisms/VersionSelector";
@@ -95,6 +96,17 @@ const ProviderProductEditionView = ({
   const [variantGroup, setVariantGroup] = useState(null);
   const [showGroup, setShowGroup] = useState(false);
   const [showGroupModal, setShowGroupModal] = useState(false);
+  /*
+    Un hijo no se agrupa como variante (decisión de producto, 2026-09-25).
+    Agrupar declara «estos son el mismo producto en otra presentación», y un
+    hijo ya declaró de dónde deriva; tener las dos cosas lo dejaba con dos
+    respuestas distintas a la misma pregunta. El servidor lo rechaza con
+    VARIANT_GROUP_NOT_FOR_CHILD, así que ofrecer el botón aquí sería ofrecer
+    algo que siempre falla.
+  */
+  const puedeAgruparse = !state.product?.parentArticleId;
   const [pendingPropagation, setPendingPropagation] = useState(null);
   const [preparandoPropagacion, setPreparandoPropagacion] = useState(false);
   const [preparandoEdicionMultiple, setPreparandoEdicionMultiple] =
@@ -712,7 +724,7 @@ const ProviderProductEditionView = ({
     const retailerId = state.active_retailer?.id_retailer;
     const inputs = state.services?.datasheets?.inputs ?? {};
-    return (state.updated_datasheets_inputs ?? []).map((item) => {
+    const cambios = (state.updated_datasheets_inputs ?? []).map((item) => {
       const input = inputs[`${item.attributeId}_${retailerId}`];
       return {
         id: Number(item.attributeId),
@@ -720,6 +732,14 @@ const ProviderProductEditionView = ({
         value: item.value,
       };
     });
+    /*
+      UPC y SKU se quitan aquí y no dentro del diálogo porque esta lista también
+      decide SI el diálogo se abre. Editar solo el UPC no tiene nada que
+      propagar, así que preguntarlo sería pedirle al usuario una decisión sobre
+      un conjunto vacío.
+    */
+    return soloPropagables(cambios);
   };
   const handleOnClickSave = () => {
@@ -742,8 +762,24 @@ const ProviderProductEditionView = ({
         const companeras = (variantGroup?.members ?? []).filter(
           (m) => !m.isCurrent
         );
+        /*
+          Un HIJO no propaga a nadie, y hay que decirlo explícitamente.
+          Sin este corte el diálogo se abría desde la ficha de un hijo y le
+          ofrecía a sus HERMANOS como si fueran sus hijos: `family.variants`
+          mira la familia desde el padre —eso es lo correcto para navegar entre
+          hermanos— pero aquí se estaba leyendo como «los destinos de este
+          producto». El servidor resolvía los destinos por su cuenta y no
+          escribía nada, así que el usuario elegía hermanos, guardaba, y no
+          pasaba nada: un no-operativo silencioso.
+          Un hijo tampoco puede estar agrupado desde 2026-09-25, así que para
+          él no queda ningún destino posible.
+        */
+        const esHijo = !!state.product?.parentArticleId;
         const hayDestinos =
-          state.product?.variantsCount > 0 || companeras.length > 0;
+          !esHijo &&
+          (state.product?.variantsCount > 0 || companeras.length > 0);
         const cambios = cambiosDeFichaTecnica();
         if (hayDestinos && cambios.length > 0) {
@@ -1650,7 +1686,11 @@ const ProviderProductEditionView = ({
         {pendingPropagation && (
           <PropagateChangesModal
             changedAttributes={pendingPropagation.cambios}
-            variants={(family?.variants ?? []).filter((v) => !v.isCurrent)}
+            variants={
+              state.product?.parentArticleId
+                ? []
+                : (family?.variants ?? []).filter((v) => !v.isCurrent)
+            }
             groupMembers={(variantGroup?.members ?? []).filter(
               (m) => !m.isCurrent
             )}
@@ -2058,6 +2098,7 @@ const ProviderProductEditionView = ({
                   código compartido que dice "estos son el mismo producto en
                   otra presentación". Mezclarlos en una sola caja haría creer
                   que agrupar hereda algo, que es justo lo que no pasa. */}
+              {puedeAgruparse && (
               <div className="variants-actions">
                 <button
                   type="button"
@@ -2079,8 +2120,9 @@ const ProviderProductEditionView = ({
                   </button>
                 )}
               </div>
+              )}
-              {showGroup && variantGroup?.group && (
+              {puedeAgruparse && showGroup && variantGroup?.group && (
                 <div className="variants-family">
                   {/* Los hermanos de grupo NO se listan como padre/hijo: aquí
                       nadie hereda de nadie, así que no hay rol que mostrar,

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

@@ -54,6 +54,7 @@ import { LinkVariantsModal } from "../../organisms/LinkVariantsModal";
 import { UnlinkVariantModal } from "../../organisms/UnlinkVariantModal";
 import { VariantGroupModal } from "../../organisms/VariantGroupModal";
 import { PropagateChangesModal } from "../../organisms/PropagateChangesModal";
+import { soloPropagables } from "../../organisms/PropagateChangesModal/non-propagable";
 import { readPayload } from "../../organisms/LinkVariantsModal/api";
 import successIcon from "../../../assets/images/genericModal/genericModalCheck.svg";
@@ -197,6 +198,7 @@ const RetailerProductEditionView = ({
   const [variantGroup, setVariantGroup] = useState(null);
   const [showGroup, setShowGroup] = useState(false);
   const [showGroupModal, setShowGroupModal] = useState(false);
   // Qué vínculo se está por romper, o null si no hay diálogo abierto. Guarda al
   // HIJO, que es quien lleva la relación y lo que pide la acción `unlink`.
   const [porDesvincular, setPorDesvincular] = useState(null);
@@ -901,6 +903,20 @@ const RetailerProductEditionView = ({
     ? !!state.product?.parentArticleId
     : !!family?.parent;
+  /*
+    Un hijo no se agrupa como variante (decisión de producto, 2026-09-25).
+    Agrupar declara «estos son el mismo producto en otra presentación», y un
+    hijo ya declaró de dónde deriva. El servidor lo rechaza con
+    VARIANT_GROUP_NOT_FOR_CHILD, así que ofrecer el botón sería ofrecer algo
+    que siempre falla.
+    Se apoya en `esHijoDeOtro` y no en `state.product` porque en esta pantalla
+    el producto sale de sessionStorage: la señal buena es la familia recién
+    consultada, con el producto solo como respaldo si esa consulta falló.
+  */
+  const puedeAgruparse = !esHijoDeOtro;
   /*
     Abre otro producto de la familia.
@@ -1236,7 +1252,7 @@ const RetailerProductEditionView = ({
     const retailerId = state.active_retailer?.id_retailer;
     const inputs = state.services?.datasheets?.inputs ?? {};
-    return (state.updated_datasheets_inputs ?? []).map((item) => {
+    const cambios = (state.updated_datasheets_inputs ?? []).map((item) => {
       const input = inputs[`${item.attributeId}_${retailerId}`];
       return {
         id: Number(item.attributeId),
@@ -1244,6 +1260,14 @@ const RetailerProductEditionView = ({
         value: item.value,
       };
     });
+    /*
+      UPC y SKU se quitan aquí y no dentro del diálogo porque esta lista también
+      decide SI el diálogo se abre. Editar solo el UPC no tiene nada que
+      propagar, así que preguntarlo sería pedirle al usuario una decisión sobre
+      un conjunto vacío.
+    */
+    return soloPropagables(cambios);
   };
   const handleOnClickSave = async () => {
@@ -1317,7 +1341,20 @@ const RetailerProductEditionView = ({
           return;
         }
-        const hijos = (familia.variants ?? []).filter((v) => !v.isCurrent);
+        /*
+          Un HIJO no propaga a nadie, y hay que decirlo explícitamente.
+          `familia.variants` mira la familia desde el PADRE —eso es lo correcto
+          para navegar entre hermanos— así que desde la ficha de un hijo esa
+          lista son sus HERMANOS. Leerla como «los destinos de este producto»
+          hacía que el diálogo le ofreciera heredar a sus hermanos. El servidor
+          resolvía los destinos por su cuenta y no escribía nada, de modo que el
+          usuario elegía, guardaba, y no pasaba nada: un no-operativo silencioso.
+        */
+        const esHijo = !!familia?.parent;
+        const hijos = esHijo
+          ? []
+          : (familia.variants ?? []).filter((v) => !v.isCurrent);
         /*
           También cuentan las compañeras de grupo: un producto puede no tener
@@ -1328,7 +1365,9 @@ const RetailerProductEditionView = ({
           (m) => !m.isCurrent
         );
-        if (hijos.length > 0 || companeras.length > 0) {
+        // 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 });
           return;
         }
@@ -2092,7 +2131,11 @@ const RetailerProductEditionView = ({
         {pendingPropagation && (
           <PropagateChangesModal
             changedAttributes={pendingPropagation.cambios}
-            variants={(family?.variants ?? []).filter((v) => !v.isCurrent)}
+            variants={
+              esHijoDeOtro
+                ? []
+                : (family?.variants ?? []).filter((v) => !v.isCurrent)
+            }
             groupMembers={(variantGroup?.members ?? []).filter(
               (m) => !m.isCurrent
             )}
@@ -2499,6 +2542,7 @@ const RetailerProductEditionView = ({
                 herencia y tiene jerarquía; esto es solo un código compartido
                 que dice "estos son el mismo producto en otra presentación".
                 Juntarlos haría creer que agrupar hereda algo, que no pasa. */}
+            {puedeAgruparse && (
             <div className="variants-actions">
               <button
                 type="button"
@@ -2520,8 +2564,9 @@ const RetailerProductEditionView = ({
                 </button>
               )}
             </div>
+            )}
-            {showGroup && variantGroup?.group && (
+            {puedeAgruparse && showGroup && variantGroup?.group && (
               <div className="variants-family">
                 {(variantGroup.members ?? [])
                   .filter((miembro) => !miembro.isCurrent)