@iobroker/gui-components 10.3.5 → 10.3.6

package/README.md

@@ -36,7 +36,7 @@ npm create vite@latest src -- --template react-ts
     "dependencies": {
         "@emotion/react": "^11.14.0",
         "@emotion/styled": "^11.14.1",
-        "@iobroker/gui-components": "^10.3.5",
+        "@iobroker/gui-components": "^10.3.6",
         "@mui/icons-material": "^9.0.1",
         "@mui/material": "^9.0.1",
         "react": "^19.2.5",
@@ -877,6 +877,12 @@ You can find the migration instructions:
 -->
 
 ## Changelog
+### 10.3.6 (2026-09-26)
+
+- (@GermanBluefox) Fixed: `Router.doNavigate` wrote the ID into the URL hash as it was, while `Router.getLocation` reads the hash by splitting it at "/" and decoding every part. An ID that contains a "/" - e.g. `ocpp.0./TACW1142021G1543.1.meterValues.Power_Active_Import` - was therefore cut off at its first slash on the way back, and its remainder landed in `arg`. The ID segment is URL-encoded now, so it survives the round trip, including the one a dialog makes when it only changes its tab (ioBroker.admin#3634). **Callers must pass the raw ID**: one that is already encoded is now encoded twice
+- (@GermanBluefox) Changed: without the expert mode, the object browser no longer offers to delete an object of an adapter - neither the button in the row nor the entry in the context menu nor the `Delete` key. Deleting such an object can stop the adapter from working, and it is created again on the next start anyway. What a user creates themselves (`0_userdata.*`, `alias.*`) can still be deleted without the expert mode, and with the expert mode everything can be deleted as before (ioBroker.admin#3639)
+- (@GermanBluefox) Fixed: the object browser threw `can't access property "<id>", this.objects is undefined` when a navigation target reached it before the object tree was read - which is what a route in the URL does. Such a target now waits for the tree instead of crashing the browser (ioBroker.admin#3634)
+
 ### 10.3.5 (2026-09-25)
 
 - (@GermanBluefox) Added function `findObjectIcon`

package/build/Components/ObjectBrowser/ObjectBrowserClass.d.ts

@@ -218,6 +218,16 @@ export declare class ObjectBrowserClass extends Component<ObjectBrowserProps, Ob
      * Called when component is unmounted.
      */
     componentWillUnmount(): void;
+    /**
+     * May this object be deleted without the expert mode?
+     *
+     * Deleting an object of an adapter can stop that adapter from working, and the adapter creates it
+     * again on the next start anyway. So outside the expert mode only the objects a user makes
+     * themselves can be deleted - those in `0_userdata.*` and `alias.*` (ioBroker.admin#3639).
+     *
+     * @param id ID of the object
+     */
+    isDeleteAllowed(id: string): boolean;
     /**
      * Show the deletion dialog for a given object
      */

package/build/Components/ObjectBrowser/ObjectBrowserClass.js

@@ -27,6 +27,11 @@ import * as contextMenu from './contextMenu';
 import { DEFAULT_DATE_FORMAT, DEFAULT_FILTER, ITEM_LEVEL, MIN_COLUMN_WIDTHS, SCREEN_WIDTHS, } from './constants';
 export { getSelectIdIconFromObjects };
 let objectsAlreadyLoaded = false;
+/**
+ * The namespaces whose objects a user creates themselves. Only these may be deleted without the
+ * expert mode; see {@link ObjectBrowserClass.isDeleteAllowed}.
+ */
+const USER_OWNED_NAMESPACES = ['0_userdata.', 'alias.'];
 export class ObjectBrowserClass extends Component {
     // do not define the type as null to save the performance, so we must check it every time
     info = {
@@ -787,11 +792,26 @@ export class ObjectBrowserClass extends Component {
         this.subscribes.clear();
         this.objects = {};
     }
+    /**
+     * May this object be deleted without the expert mode?
+     *
+     * Deleting an object of an adapter can stop that adapter from working, and the adapter creates it
+     * again on the next start anyway. So outside the expert mode only the objects a user makes
+     * themselves can be deleted - those in `0_userdata.*` and `alias.*` (ioBroker.admin#3639).
+     *
+     * @param id ID of the object
+     */
+    isDeleteAllowed(id) {
+        return this.state.filter.expertMode || USER_OWNED_NAMESPACES.some(prefix => id.startsWith(prefix));
+    }
     /**
      * Show the deletion dialog for a given object
      */
     showDeleteDialog(options) {
         const { id, obj, item } = options;
+        if (!this.isDeleteAllowed(id)) {
+            return;
+        }
         // calculate the number of children
         const keys = Object.keys(this.objects);
         keys.sort();
@@ -864,9 +884,12 @@ export class ObjectBrowserClass extends Component {
      */
     onSelect(toggleItem, isDouble, cb) {
         this.localStorage.setItem(`${this.props.dialogName || 'App'}.focused`, toggleItem);
+        // `objects` only exists once the tree has been read. A selection can arrive before that -
+        // from the URL, through `navigateTo` - and must not throw; such an ID is treated like one
+        // that is not in the tree, and `applyInitialNavigateTo` selects it properly after the load.
+        const obj = this.objects?.[toggleItem];
         if (!this.props.multiSelect) {
-            if (this.objects[toggleItem] &&
-                (!this.props.types || this.props.types.includes(this.objects[toggleItem].type))) {
+            if (obj && (!this.props.types || this.props.types.includes(obj.type))) {
                 this.localStorage.removeItem(`${this.props.dialogName || 'App'}.selectedNonObject`);
                 if (this.state.selected[0] !== toggleItem) {
                     this.setState({ selected: [toggleItem], selectedNonObject: '', focused: toggleItem }, () => {
@@ -890,8 +913,7 @@ export class ObjectBrowserClass extends Component {
                 });
             }
         }
-        else if (this.objects[toggleItem] &&
-            (!this.props.types || this.props.types.includes(this.objects[toggleItem].type))) {
+        else if (obj && (!this.props.types || this.props.types.includes(obj.type))) {
             this.localStorage.removeItem(`${this.props.dialogName || 'App'}.selectedNonObject`);
             const selected = [...this.state.selected];
             const pos = selected.indexOf(toggleItem);
@@ -2493,6 +2515,11 @@ export class ObjectBrowserClass extends Component {
     }
     /** Apply a navigation target coming from the parent (`navigateTo`): select + open the dialog. */
     applyNavigateTo(nav) {
+        if (!this.objects) {
+            // The tree is still being read. `applyInitialNavigateTo` runs once it is there and takes
+            // the target from the props again, so nothing is lost by waiting here.
+            return;
+        }
         this.applyingNav = true;
         const done = () => {
             this.applyingNav = false;