* Only fetch editable nodes if edit tools are enabled.

* Always disable checkboxes in UI and grey out if not sensible in the current context.

Author: andys-sidefx

Plugins/HoudiniEngineUnity/Editor/UI/HEU_HoudiniAssetUI.cs

@@ -505,18 +505,14 @@ private void DrawAssetOptions(HEU_HoudiniAsset asset, SerializedObject assetObje
 
                     EditorGUILayout.EndHorizontal();
 
-                    if (asset.NumAttributeStores() > 0)
-                    {
-                        HEU_EditorUI.DrawPropertyField(assetObject, "_editableNodesToolsEnabled",
-                            "Enable Editable Node Tools",
-                            "Displays Editable Node Tools and generates the node's geometry, if asset has editable nodes.");
-                    }
+                    HEU_EditorUI.DrawPropertyField(assetObject, "_editableNodesToolsEnabled",
+                        "Enable Editable Node Tools",
+                        "Displays Editable Node Tools and generates the node's geometry, if asset has editable nodes.");
 
-                    if (asset.NumHandles() > 0)
-                    {
+                    GUI.enabled = (asset.NumHandles() > 0);
                         HEU_EditorUI.DrawPropertyField(assetObject, "_handlesEnabled", "Enable Handles",
                             "Creates Houdini Handles if asset has them.");
-                    }
+                    GUI.enabled = true;
 
                     EditorGUILayout.Space();
 

Plugins/HoudiniEngineUnity/Scripts/Asset/HEU_HoudiniAsset.cs

@@ -3876,7 +3876,7 @@ private void UpdateAllObjectNodes(HEU_SessionBase session)
         private HEU_ObjectNode CreateObjectNode(HEU_SessionBase session, ref HAPI_ObjectInfo objectInfo, ref HAPI_Transform objectTranform)
         {
             HEU_ObjectNode objectNode = ScriptableObject.CreateInstance<HEU_ObjectNode>();
-            objectNode.Initialize(session, objectInfo, objectTranform, this, _useOutputNodes);
+            objectNode.Initialize(session, objectInfo, objectTranform, this, _useOutputNodes, _editableNodesToolsEnabled);
             return objectNode;
         }
 

Plugins/HoudiniEngineUnity/Scripts/Asset/HEU_ObjectNode.cs

@@ -237,7 +237,7 @@ private void SyncWithObjectInfo(HEU_SessionBase session)
         }
 
         internal void Initialize(HEU_SessionBase session, HAPI_ObjectInfo objectInfo, HAPI_Transform objectTranform,
-            HEU_HoudiniAsset parentAsset, bool bUseOutputNodes)
+            HEU_HoudiniAsset parentAsset, bool bUseOutputNodes, bool bGetEditableNodes)
         {
             _objectInfo = objectInfo;
             _objectTransform = objectTranform;
@@ -248,7 +248,7 @@ internal void Initialize(HEU_SessionBase session, HAPI_ObjectInfo objectInfo, HA
             // Translate transform to Unity (TODO)
             List<HAPI_GeoInfo> geoInfos = new List<HAPI_GeoInfo>();
 
-            HEU_HAPIUtility.GatherAllAssetGeoInfos(session, parentAsset.AssetInfo, objectInfo, bUseOutputNodes,
+            HEU_HAPIUtility.GatherAllAssetGeoInfos(session, parentAsset.AssetInfo, objectInfo, bUseOutputNodes, bGetEditableNodes,
                 ref geoInfos);
             int numGeoInfos = geoInfos.Count;
             for (int i = 0; i < numGeoInfos; ++i)
@@ -418,9 +418,15 @@ internal void UpdateObject(HEU_SessionBase session, bool bForceUpdate)
 
 
                 bool useOutputNodes = true;
-                if (ParentAsset) useOutputNodes = ParentAsset.UseOutputNodes;
+                bool getEditableNodes = true;
 
-                HEU_HAPIUtility.GatherAllAssetGeoInfos(session, ParentAsset.AssetInfo, _objectInfo, useOutputNodes,
+                if (ParentAsset)
+                {
+                    useOutputNodes = ParentAsset.UseOutputNodes;
+                    getEditableNodes = ParentAsset.EditableNodesToolsEnabled;
+                }
+
+                HEU_HAPIUtility.GatherAllAssetGeoInfos(session, ParentAsset.AssetInfo, _objectInfo, useOutputNodes, getEditableNodes,
                     ref postCookGeoInfos);
 
                 // Now for each geo node that are present after cooking, we check if its

Plugins/HoudiniEngineUnity/Scripts/Core/HEU_HAPIUtility.cs

@@ -1535,8 +1535,12 @@ public static bool GetOutputIndex(HEU_SessionBase session, HAPI_NodeId nodeId, r
             return false;
         }
 
-        static internal void GatherAllAssetGeoInfos(HEU_SessionBase session, HAPI_AssetInfo assetInfo, HAPI_ObjectInfo objectInfo,
-            bool bUseOutputNodes, ref List<HAPI_GeoInfo> outGeoInfos)
+        static internal void GatherAllAssetGeoInfos(HEU_SessionBase session, 
+            HAPI_AssetInfo assetInfo, 
+            HAPI_ObjectInfo objectInfo,
+            bool bUseOutputNodes, 
+            bool bGetEditableNodes,
+            ref List<HAPI_GeoInfo> outGeoInfos)
         {
             if (outGeoInfos == null) outGeoInfos = new List<HAPI_GeoInfo>();
 
@@ -1564,8 +1568,17 @@ static internal void GatherAllAssetGeoInfos(HEU_SessionBase session, HAPI_AssetI
 
             // Get editable nodes, cook em, then create geo nodes for them
             HAPI_NodeId[] editableNodes = null;
-            HEU_SessionManager.GetComposedChildNodeList(session, assetInfo.nodeId, (int)HAPI_NodeType.HAPI_NODETYPE_SOP,
-                (int)HAPI_NodeFlags.HAPI_NODEFLAGS_EDITABLE, true, out editableNodes, false);
+            if (bGetEditableNodes)
+            {
+                HEU_SessionManager.GetComposedChildNodeList(session, assetInfo.nodeId, (int)HAPI_NodeType.HAPI_NODETYPE_SOP,
+                    (int)HAPI_NodeFlags.HAPI_NODEFLAGS_EDITABLE, true, out editableNodes, false);
+
+                if (editableNodes == null || editableNodes.Length == 0)
+                {
+                    HEU_Logger.LogWarning("Edit tools are enabled but no editable nodes were found in the HDA. Ensure the HDA is unlocked in Houdini and recook.");
+                }
+            }
+
             if (editableNodes != null)
             {
                 foreach (HAPI_NodeId editNodeID in editableNodes)