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

package/es/LiteKitchenConfigurator.js

@@ -8,7 +8,7 @@ import _possibleConstructorReturn from "@babel/runtime/helpers/esm/possibleConst
 import _getPrototypeOf from "@babel/runtime/helpers/esm/getPrototypeOf";
 import _inherits from "@babel/runtime/helpers/esm/inherits";
 import _taggedTemplateLiteral from "@babel/runtime/helpers/esm/taggedTemplateLiteral";
-var _excluded = ["width", "height", "extractedState", "measurementUnit"];
+var _excluded = ["width", "height", "extractedState"];
 var _templateObject, _templateObject2;
 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; }
 function _objectSpread(e) { for (var r = 1; r < arguments.length; r++) { var t = null != arguments[r] ? arguments[r] : {}; r % 2 ? ownKeys(Object(t), !0).forEach(function (r) { _defineProperty(e, r, t[r]); }) : Object.getOwnPropertyDescriptors ? Object.defineProperties(e, Object.getOwnPropertyDescriptors(t)) : ownKeys(Object(t)).forEach(function (r) { Object.defineProperty(e, r, Object.getOwnPropertyDescriptor(t, r)); }); } return e; }
@@ -121,7 +121,7 @@ var LiteKitchenConfigurator = /*#__PURE__*/function (_Component) {
     _this.neverShowInput = /*#__PURE__*/React.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;
@@ -214,7 +214,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",
@@ -224,7 +224,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",
@@ -234,7 +234,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",
@@ -251,7 +251,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",
@@ -261,7 +261,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",
@@ -271,7 +271,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",
@@ -281,7 +281,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",
@@ -291,7 +291,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",
@@ -301,7 +301,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",
@@ -311,7 +311,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",
@@ -332,6 +332,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 = _toConsumableArray(plugins);
       newplugins.forEach(function (newplugin) {
         return newplugin(store, stateExtractor);
@@ -343,21 +345,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) {
         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",
@@ -367,9 +366,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) {
@@ -378,15 +374,11 @@ var LiteKitchenConfigurator = /*#__PURE__*/function (_Component) {
       var len = 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,
@@ -401,8 +393,6 @@ var LiteKitchenConfigurator = /*#__PURE__*/function (_Component) {
         this._cachedExtractedState = merged;
         return merged;
       }
-
-      // no merge needed
       this._cachedViewer2DKey = cacheKey;
       this._cachedExtractedState = extractedState;
       return extractedState;
@@ -414,7 +404,6 @@ var LiteKitchenConfigurator = /*#__PURE__*/function (_Component) {
         width = _this$props3.width,
         height = _this$props3.height,
         extractedState = _this$props3.extractedState,
-        measurementUnit = _this$props3.measurementUnit,
         props = _objectWithoutProperties(_this$props3, _excluded);
       var _this$state = this.state,
         savePopupVisible = _this$state.savePopupVisible,
@@ -424,8 +413,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;
@@ -477,13 +464,11 @@ LiteKitchenConfigurator.propTypes = {
   width: PropTypes.number.isRequired,
   height: PropTypes.number.isRequired,
   stateExtractor: PropTypes.func.isRequired,
-  sidebarComponents: PropTypes.array,
-  footerbarComponents: PropTypes.array,
-  customContents: PropTypes.object,
-  softwareSignature: PropTypes.string,
   configData: PropTypes.object,
   onInternalEvent: PropTypes.func,
-  extractedState: PropTypes.object // <-- new injected prop
+  extractedState: PropTypes.object,
+  // ✅ injected by connect
+  externalEvent: PropTypes.object
 };
 LiteKitchenConfigurator.contextTypes = {
   store: PropTypes.object.isRequired
@@ -506,7 +491,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/es/LiteRenderer.js

@@ -35,11 +35,11 @@ import LiteKitchenConfigurator from "./LiteKitchenConfigurator";
 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;
@@ -125,7 +125,6 @@ function initCatalogOnce(catalog) {
 
 // ----------------- store factory -----------------
 function createInstanceStore() {
-  // keep your initial state semantics identical
   var AppState = Map({
     KitchenConfigurator: new State()
   });
@@ -135,8 +134,6 @@ function createInstanceStore() {
       return PlannerReducer(plannerState, action);
     });
   };
-
-  // keep devtools behavior identical; guard against missing window
   var enhancer = !isProduction && isBrowser && window.devToolsExtension ? window.devToolsExtension({
     features: {
       pause: true,
@@ -156,8 +153,6 @@ function createInstanceStore() {
   };
   return createStore(reducer, null, enhancer);
 }
-
-// ----------------- plugins -----------------
 function createPlugins() {
   return [Keyboard(), ConsoleDebugger()];
 }
@@ -171,28 +166,29 @@ export default function LiteRenderer(props) {
     onError = props.onError,
     passThrough = _objectWithoutProperties(props, _excluded);
 
-  // ---- instance-scoped store + catalog (no module-level globals) ----
+  // ✅ instance-scoped store (no module singleton)
   var storeRef = useRef(null);
   if (!storeRef.current) storeRef.current = createInstanceStore();
+
+  // ✅ instance-scoped catalog (no module singleton)
   var catalogRef = useRef(null);
   if (!catalogRef.current) {
     catalogRef.current = new Catalog();
     initCatalogOnce(catalogRef.current);
   }
 
-  // ---- plugins init once per instance + cleanup support ----
-  var pluginCleanupsRef = useRef([]);
+  // ✅ stable plugins array
   var plugins = useMemo(function () {
     return createPlugins();
-  }, []); // stable
+  }, []);
 
+  // ✅ plugin lifecycle: supports cleanup if plugin returns function
+  var pluginCleanupsRef = useRef([]);
   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;
@@ -202,9 +198,8 @@ export default 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) {
@@ -252,8 +247,7 @@ export default 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;