kitchen-simulator 3.16.0-test-renderer-fix → 3.16.1-test-renderer-fix

package/lib/LiteKitchenConfigurator.js

@@ -31,7 +31,7 @@ var _translator = _interopRequireDefault(require("./translator/translator"));
 var _objectsUtils = require("./utils/objects-utils");
 var _version = require("./version");
 var _isolateEventHandler = require("./utils/isolate-event-handler");
-var _excluded = ["width", "height", "extractedState", "measurementUnit"];
+var _excluded = ["width", "height", "extractedState"];
 var _templateObject, _templateObject2; // LiteKitchenConfigurator.jsx
 function _interopRequireWildcard(e, t) { if ("function" == typeof WeakMap) var r = new WeakMap(), n = new WeakMap(); return (_interopRequireWildcard = function _interopRequireWildcard(e, t) { if (!t && e && e.__esModule) return e; var o, i, f = { __proto__: null, "default": e }; if (null === e || "object" != _typeof(e) && "function" != typeof e) return f; if (o = t ? n : r) { if (o.has(e)) return o.get(e); o.set(e, f); } for (var _t in e) "default" !== _t && {}.hasOwnProperty.call(e, _t) && ((i = (o = Object.defineProperty) && Object.getOwnPropertyDescriptor(e, _t)) && (i.get || i.set) ? o(f, _t, i) : f[_t] = e[_t]); return f; })(e, t); }
 function ownKeys(e, r) { var t = Object.keys(e); if (Object.getOwnPropertySymbols) { var o = Object.getOwnPropertySymbols(e); r && (o = o.filter(function (r) { return Object.getOwnPropertyDescriptor(e, r).enumerable; })), t.push.apply(t, o); } return t; }
@@ -129,7 +129,7 @@ var LiteKitchenConfigurator = /*#__PURE__*/function (_Component) {
     _this.neverShowInput = /*#__PURE__*/_react["default"].createRef(null);
     _this.setShowProperty = _this.setShowProperty.bind(_this);
 
-    // cache to avoid immutable merge allocations every render
+    // cache for viewer2D init so we don't allocate merges every render
     _this._cachedViewer2DKey = null;
     _this._cachedExtractedState = null;
     return _this;
@@ -222,7 +222,7 @@ var LiteKitchenConfigurator = /*#__PURE__*/function (_Component) {
         floorOpened: true
       });
       var el = document.getElementById('make_floorplan_inactive');
-      el && ((_el$parentElement = el.parentElement) === null || _el$parentElement === void 0 ? void 0 : _el$parentElement.parentElement) && (el.parentElement.parentElement.style.zIndex = 999);
+      if (el !== null && el !== void 0 && (_el$parentElement = el.parentElement) !== null && _el$parentElement !== void 0 && _el$parentElement.parentElement) el.parentElement.parentElement.style.zIndex = 999;
     }
   }, {
     key: "openCabinet",
@@ -232,7 +232,7 @@ var LiteKitchenConfigurator = /*#__PURE__*/function (_Component) {
         cabinetOpened: true
       });
       var el = document.getElementById('add_cabinets_inactive');
-      el && ((_el$parentElement2 = el.parentElement) === null || _el$parentElement2 === void 0 ? void 0 : _el$parentElement2.parentElement) && (el.parentElement.parentElement.style.zIndex = 999);
+      if (el !== null && el !== void 0 && (_el$parentElement2 = el.parentElement) !== null && _el$parentElement2 !== void 0 && _el$parentElement2.parentElement) el.parentElement.parentElement.style.zIndex = 999;
     }
   }, {
     key: "toggleDoorStyle",
@@ -242,7 +242,7 @@ var LiteKitchenConfigurator = /*#__PURE__*/function (_Component) {
         doorStyleOpen: visible
       });
       var el = document.getElementById('select_doorstyle_inactive');
-      el && ((_el$parentElement3 = el.parentElement) === null || _el$parentElement3 === void 0 ? void 0 : _el$parentElement3.parentElement) && (el.parentElement.parentElement.style.zIndex = visible ? 999 : 6);
+      if (el !== null && el !== void 0 && (_el$parentElement3 = el.parentElement) !== null && _el$parentElement3 !== void 0 && _el$parentElement3.parentElement) el.parentElement.parentElement.style.zIndex = visible ? 999 : 6;
     }
   }, {
     key: "replaceCabinet",
@@ -259,7 +259,7 @@ var LiteKitchenConfigurator = /*#__PURE__*/function (_Component) {
         applianceOpened: true
       });
       var el = document.getElementById('add_appliances_inactive');
-      el && ((_el$parentElement4 = el.parentElement) === null || _el$parentElement4 === void 0 ? void 0 : _el$parentElement4.parentElement) && (el.parentElement.parentElement.style.zIndex = 999);
+      if (el !== null && el !== void 0 && (_el$parentElement4 = el.parentElement) !== null && _el$parentElement4 !== void 0 && _el$parentElement4.parentElement) el.parentElement.parentElement.style.zIndex = 999;
     }
   }, {
     key: "openFinishing",
@@ -269,7 +269,7 @@ var LiteKitchenConfigurator = /*#__PURE__*/function (_Component) {
         finishingOpened: true
       });
       var el = document.getElementById('finishing_touches_inactive');
-      el && ((_el$parentElement5 = el.parentElement) === null || _el$parentElement5 === void 0 ? void 0 : _el$parentElement5.parentElement) && (el.parentElement.parentElement.style.zIndex = 999);
+      if (el !== null && el !== void 0 && (_el$parentElement5 = el.parentElement) !== null && _el$parentElement5 !== void 0 && _el$parentElement5.parentElement) el.parentElement.parentElement.style.zIndex = 999;
     }
   }, {
     key: "onReviewQuoteClicked",
@@ -279,7 +279,7 @@ var LiteKitchenConfigurator = /*#__PURE__*/function (_Component) {
         reviewQuotePopupOpened: visible
       });
       var el = document.getElementById('review_quote_inactive');
-      el && ((_el$parentElement6 = el.parentElement) === null || _el$parentElement6 === void 0 ? void 0 : _el$parentElement6.parentElement) && (el.parentElement.parentElement.style.zIndex = visible ? 999 : 6);
+      if (el !== null && el !== void 0 && (_el$parentElement6 = el.parentElement) !== null && _el$parentElement6 !== void 0 && _el$parentElement6.parentElement) el.parentElement.parentElement.style.zIndex = visible ? 999 : 6;
     }
   }, {
     key: "closeFloorTB",
@@ -289,7 +289,7 @@ var LiteKitchenConfigurator = /*#__PURE__*/function (_Component) {
         floorOpened: false
       });
       var el = document.getElementById('make_floorplan_inactive');
-      el && ((_el$parentElement7 = el.parentElement) === null || _el$parentElement7 === void 0 ? void 0 : _el$parentElement7.parentElement) && (el.parentElement.parentElement.style.zIndex = 6);
+      if (el !== null && el !== void 0 && (_el$parentElement7 = el.parentElement) !== null && _el$parentElement7 !== void 0 && _el$parentElement7.parentElement) el.parentElement.parentElement.style.zIndex = 6;
     }
   }, {
     key: "closeCabinetTB",
@@ -299,7 +299,7 @@ var LiteKitchenConfigurator = /*#__PURE__*/function (_Component) {
         cabinetOpened: false
       });
       var el = document.getElementById('add_cabinets_inactive');
-      el && ((_el$parentElement8 = el.parentElement) === null || _el$parentElement8 === void 0 ? void 0 : _el$parentElement8.parentElement) && (el.parentElement.parentElement.style.zIndex = 6);
+      if (el !== null && el !== void 0 && (_el$parentElement8 = el.parentElement) !== null && _el$parentElement8 !== void 0 && _el$parentElement8.parentElement) el.parentElement.parentElement.style.zIndex = 6;
     }
   }, {
     key: "closeFinishingTB",
@@ -309,7 +309,7 @@ var LiteKitchenConfigurator = /*#__PURE__*/function (_Component) {
         finishingOpened: false
       });
       var el = document.getElementById('finishing_touches_inactive');
-      el && ((_el$parentElement9 = el.parentElement) === null || _el$parentElement9 === void 0 ? void 0 : _el$parentElement9.parentElement) && (el.parentElement.parentElement.style.zIndex = 6);
+      if (el !== null && el !== void 0 && (_el$parentElement9 = el.parentElement) !== null && _el$parentElement9 !== void 0 && _el$parentElement9.parentElement) el.parentElement.parentElement.style.zIndex = 6;
     }
   }, {
     key: "closeApplianceTB",
@@ -319,7 +319,7 @@ var LiteKitchenConfigurator = /*#__PURE__*/function (_Component) {
         applianceOpened: false
       });
       var el = document.getElementById('add_appliances_inactive');
-      el && ((_el$parentElement0 = el.parentElement) === null || _el$parentElement0 === void 0 ? void 0 : _el$parentElement0.parentElement) && (el.parentElement.parentElement.style.zIndex = 6);
+      if (el !== null && el !== void 0 && (_el$parentElement0 = el.parentElement) !== null && _el$parentElement0 !== void 0 && _el$parentElement0.parentElement) el.parentElement.parentElement.style.zIndex = 6;
     }
   }, {
     key: "getChildContext",
@@ -340,6 +340,8 @@ var LiteKitchenConfigurator = /*#__PURE__*/function (_Component) {
       var _this$props = this.props,
         stateExtractor = _this$props.stateExtractor,
         plugins = _this$props.plugins;
+
+      // keep old behavior: run plugins once on mount
       var newplugins = (0, _toConsumableArray2["default"])(plugins);
       newplugins.forEach(function (newplugin) {
         return newplugin(store, stateExtractor);
@@ -351,21 +353,18 @@ var LiteKitchenConfigurator = /*#__PURE__*/function (_Component) {
       var _extractedState$getIn;
       var _this$props2 = this.props,
         externalEvent = _this$props2.externalEvent,
-        stateExtractor = _this$props2.stateExtractor,
         extractedState = _this$props2.extractedState,
         projectActions = _this$props2.projectActions,
         catalog = _this$props2.catalog;
 
-      // handle external events (same behavior as before)
+      // same behavior: handle external event when it changes
       if (prevProps.externalEvent !== externalEvent) {
         (0, _isolateEventHandler.handleExternalEvent)(this.props);
       }
 
-      // init catalog once ready flag says not ready (same behavior)
+      // same behavior: init catalog until ready
       var catalogReady = extractedState === null || extractedState === void 0 || (_extractedState$getIn = extractedState.getIn) === null || _extractedState$getIn === void 0 ? void 0 : _extractedState$getIn.call(extractedState, ['catalog', 'ready']);
-      if (!catalogReady) {
-        projectActions.initCatalog(catalog);
-      }
+      if (!catalogReady) projectActions.initCatalog(catalog);
     }
   }, {
     key: "isProjectEmpty",
@@ -375,9 +374,6 @@ var LiteKitchenConfigurator = /*#__PURE__*/function (_Component) {
       var layer = layers.get(selectedLayer);
       return layer.areas.size + layer.lines.size + layer.holes.size + layer.items.size === 0;
     }
-
-    // Compute viewer2D transform only when needed, and cache it.
-    // This preserves the exact visual result but avoids merge allocations every render.
   }, {
     key: "getExtractedStateWithViewer2DInit",
     value: function getExtractedStateWithViewer2DInit(extractedState, width, height) {
@@ -386,15 +382,11 @@ var LiteKitchenConfigurator = /*#__PURE__*/function (_Component) {
       var len = (0, _convertUnitsLite.convert)(_scene.width).from(_scene.unit).to('cm');
       var _viewer2D = extractedState.getIn(['viewer2D']);
       if (!_viewer2D || _viewer2D.size <= 0) return extractedState;
-
-      // build a stable cache key based on inputs that affect the result
       var v = _viewer2D.toJS();
-      var cacheKey = "".concat(width, ":").concat(height, ":").concat(len, ":").concat(v.e, ":").concat(v.f, ":").concat(v.viewerWidth, ":").concat(v.viewerHeight, ":").concat(v.SVGWidth, ":").concat(v.SVGHeight);
+      var cacheKey = "".concat(width, ":").concat(height, ":").concat(len, ":").concat(v.e, ":").concat(v.f, ":").concat(v.viewerWidth, ":").concat(v.viewerHeight, ":").concat(v.SVGWidth, ":").concat(v.SVGHeight, ":").concat(v.a, ":").concat(v.d);
       if (this._cachedViewer2DKey === cacheKey && this._cachedExtractedState) {
         return this._cachedExtractedState;
       }
-
-      // your original logic
       if (v.e === 0 && v.f === 0) {
         _viewer2D = _viewer2D.merge({
           e: v.viewerWidth / 2 - v.SVGWidth / 2,
@@ -409,8 +401,6 @@ var LiteKitchenConfigurator = /*#__PURE__*/function (_Component) {
         this._cachedExtractedState = merged;
         return merged;
       }
-
-      // no merge needed
       this._cachedViewer2DKey = cacheKey;
       this._cachedExtractedState = extractedState;
       return extractedState;
@@ -422,7 +412,6 @@ var LiteKitchenConfigurator = /*#__PURE__*/function (_Component) {
         width = _this$props3.width,
         height = _this$props3.height,
         extractedState = _this$props3.extractedState,
-        measurementUnit = _this$props3.measurementUnit,
         props = (0, _objectWithoutProperties2["default"])(_this$props3, _excluded);
       var _this$state = this.state,
         savePopupVisible = _this$state.savePopupVisible,
@@ -432,8 +421,6 @@ var LiteKitchenConfigurator = /*#__PURE__*/function (_Component) {
         myProjectsOpen = _this$state.myProjectsOpen;
       var contentW = width - toolbarW;
       var contentH = height;
-      var headerW = width;
-      var headerH = 60;
       var extracted = this.getExtractedStateWithViewer2DInit(extractedState, width, height);
       var firstVisit = this.state.wizardStepOpend && this.isProjectEmpty(extracted.scene);
       var allVisible = firstVisit || signOpen || myProjectsOpen;
@@ -485,13 +472,11 @@ LiteKitchenConfigurator.propTypes = {
   width: _propTypes["default"].number.isRequired,
   height: _propTypes["default"].number.isRequired,
   stateExtractor: _propTypes["default"].func.isRequired,
-  sidebarComponents: _propTypes["default"].array,
-  footerbarComponents: _propTypes["default"].array,
-  customContents: _propTypes["default"].object,
-  softwareSignature: _propTypes["default"].string,
   configData: _propTypes["default"].object,
   onInternalEvent: _propTypes["default"].func,
-  extractedState: _propTypes["default"].object // <-- new injected prop
+  extractedState: _propTypes["default"].object,
+  // ✅ injected by connect
+  externalEvent: _propTypes["default"].object
 };
 LiteKitchenConfigurator.contextTypes = {
   store: _propTypes["default"].object.isRequired
@@ -514,7 +499,8 @@ LiteKitchenConfigurator.defaultProps = {
   configData: {}
 };
 
-// IMPORTANT: select only the extracted slice so top component doesn't rerender on every dispatch
+// ✅ Only select the slice you actually use.
+// This preserves behavior but massively reduces rerenders.
 function mapStateToProps(reduxState, ownProps) {
   var stateExtractor = ownProps.stateExtractor || function (s) {
     return s;

package/lib/LiteRenderer.js

@@ -43,11 +43,11 @@ function _isNativeReflectConstruct() { try { var t = !Boolean.prototype.valueOf.
 var isBrowser = typeof window !== 'undefined';
 if (isBrowser) window.THREE = THREE;
 
-// ---- Sentry init: keep behavior but avoid re-init on multiple mounts ----
+// ---- Keep prior Sentry behavior but ensure it only initializes once ----
 var __sentryInited = false;
 function ensureSentryInit() {
-  // assumes isProduction is defined in your build env like before
   if (!isBrowser) return;
+  // assumes isProduction exists in your build just like today
   if (!isProduction) return;
   if (__sentryInited) return;
   __sentryInited = true;
@@ -133,7 +133,6 @@ function initCatalogOnce(catalog) {
 
 // ----------------- store factory -----------------
 function createInstanceStore() {
-  // keep your initial state semantics identical
   var AppState = (0, _immutable.Map)({
     KitchenConfigurator: new _models.State()
   });
@@ -143,8 +142,6 @@ function createInstanceStore() {
       return (0, _reducer["default"])(plannerState, action);
     });
   };
-
-  // keep devtools behavior identical; guard against missing window
   var enhancer = !isProduction && isBrowser && window.devToolsExtension ? window.devToolsExtension({
     features: {
       pause: true,
@@ -164,8 +161,6 @@ function createInstanceStore() {
   };
   return (0, _redux.createStore)(reducer, null, enhancer);
 }
-
-// ----------------- plugins -----------------
 function createPlugins() {
   return [(0, _export2.Keyboard)(), (0, _export2.ConsoleDebugger)()];
 }
@@ -179,28 +174,29 @@ function LiteRenderer(props) {
     onError = props.onError,
     passThrough = (0, _objectWithoutProperties2["default"])(props, _excluded);
 
-  // ---- instance-scoped store + catalog (no module-level globals) ----
+  // ✅ instance-scoped store (no module singleton)
   var storeRef = (0, _react.useRef)(null);
   if (!storeRef.current) storeRef.current = createInstanceStore();
+
+  // ✅ instance-scoped catalog (no module singleton)
   var catalogRef = (0, _react.useRef)(null);
   if (!catalogRef.current) {
     catalogRef.current = new _catalog["default"]();
     initCatalogOnce(catalogRef.current);
   }
 
-  // ---- plugins init once per instance + cleanup support ----
-  var pluginCleanupsRef = (0, _react.useRef)([]);
+  // ✅ stable plugins array
   var plugins = (0, _react.useMemo)(function () {
     return createPlugins();
-  }, []); // stable
+  }, []);
 
+  // ✅ plugin lifecycle: supports cleanup if plugin returns function
+  var pluginCleanupsRef = (0, _react.useRef)([]);
   (0, _react.useEffect)(function () {
     var store = storeRef.current;
     var stateExtractor = function stateExtractor(state) {
       return state.get('KitchenConfigurator');
     };
-
-    // plugins may or may not return cleanup; support both
     var cleanups = [];
     var _iterator = _createForOfIteratorHelper(plugins),
       _step;
@@ -210,9 +206,8 @@ function LiteRenderer(props) {
         try {
           var maybeCleanup = p === null || p === void 0 ? void 0 : p(store, stateExtractor);
           if (typeof maybeCleanup === 'function') cleanups.push(maybeCleanup);
-        } catch (e) {
-          // keep behavior: don't crash; send to onError buffer
-          // (actual emitting is handled below)
+        } catch (_unused4) {
+          // keep prior behavior: do not crash
         }
       }
     } catch (err) {
@@ -260,8 +255,7 @@ function LiteRenderer(props) {
     };
     try {
       onError === null || onError === void 0 || onError(payload);
-    } catch (_unused4) {}
-    // keep your debug
+    } catch (_unused5) {}
     // eslint-disable-next-line no-console
     console.debug('[LiteRenderer:onError]', payload);
     lastEmittedRef.current = payload;