Add set player state unit that allows selecting the variable.

Author: freezy

Editor/Inspectors/GleInspector.cs

@@ -0,0 +1,48 @@
+// Visual Pinball Engine
+// Copyright (C) 2022 freezy and VPE Team
+//
+// This program is free software: you can redistribute it and/or modify
+// it under the terms of the GNU General Public License as published by
+// the Free Software Foundation, either version 3 of the License, or
+// (at your option) any later version.
+//
+// This program is distributed in the hope that it will be useful,
+// but WITHOUT ANY WARRANTY; without even the implied warranty of
+// MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
+// GNU General Public License for more details.
+//
+// You should have received a copy of the GNU General Public License
+// along with this program. If not, see <https://www.gnu.org/licenses/>.
+
+using Unity.VisualScripting;
+using VisualPinball.Unity;
+using VisualPinball.Unity.VisualScripting;
+
+public abstract class GleInspector : Inspector
+{
+	protected GleInspector(Metadata metadata) : base(metadata) { }
+
+	private const string NoGleError = "Unable to find Gamelogic Engine in scene.";
+
+	protected VisualScriptingGamelogicEngine Gle {
+		get {
+			if (_gle != null) {
+				return _gle;
+			}
+			var tableComponent = TableSelector.Instance.SelectedOrFirstTable;
+			if (tableComponent != null) {
+				_gle = tableComponent.GetComponentInChildren<VisualScriptingGamelogicEngine>();
+			}
+			if (_gle == null && ErrorMessage == null) {
+				ErrorMessage = NoGleError;
+
+			} else if (_gle != null && ErrorMessage == NoGleError) {
+				ErrorMessage = null;
+			}
+			return _gle;
+		}
+	}
+	protected string ErrorMessage;
+
+	private VisualScriptingGamelogicEngine _gle;
+}

Editor/Inspectors/GleInspector.cs.meta

@@ -0,0 +1,72 @@
+// Visual Pinball Engine
+// Copyright (C) 2022 freezy and VPE Team
+//
+// This program is free software: you can redistribute it and/or modify
+// it under the terms of the GNU General Public License as published by
+// the Free Software Foundation, either version 3 of the License, or
+// (at your option) any later version.
+//
+// This program is distributed in the hope that it will be useful,
+// but WITHOUT ANY WARRANTY; without even the implied warranty of
+// MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
+// GNU General Public License for more details.
+//
+// You should have received a copy of the GNU General Public License
+// along with this program. If not, see <https://www.gnu.org/licenses/>.
+
+using System.Linq;
+using Unity.VisualScripting;
+using UnityEditor;
+using UnityEngine;
+using VisualPinball.Unity.VisualScripting;
+
+[Inspector(typeof(VisualScriptingPlayerStatePropertyDefinition))]
+public class VisualScriptingPlayerStatePropertyDefinitionInspector : GleInspector
+{
+	public VisualScriptingPlayerStatePropertyDefinitionInspector(Metadata metadata) : base(metadata) { }
+
+	protected override void OnGUI(Rect position, GUIContent label)
+	{
+		// can't get this from the flow
+		var gle = Gle;
+		if (gle != null) {
+			var propDefinitions = gle.PlayerStateDefinition;
+
+			if (propDefinitions.Count == 0) {
+				ErrorMessage = "No properties defined.";
+
+			} else {
+				var propNames = propDefinitions.Select(d => d.Name).ToArray();
+				var currentPropDef = metadata.value as VisualScriptingPlayerStatePropertyDefinition;
+				var playerPropDef = propDefinitions.FirstOrDefault(p => p.Id == currentPropDef!.Id);
+				var currentIndex = playerPropDef != null ? propDefinitions.IndexOf(playerPropDef) : 0;
+
+				var newIndex = EditorGUI.Popup(position, currentIndex, propNames);
+				if (EndBlock(metadata))
+				{
+					metadata.RecordUndo();
+					metadata.value = propDefinitions[newIndex];
+				}
+			}
+		}
+
+
+		if (ErrorMessage != null) {
+			position.height -= EditorGUIUtility.standardVerticalSpacing;
+			EditorGUI.HelpBox(position, ErrorMessage, MessageType.Error);
+		}
+	}
+
+	public override float GetAdaptiveWidth() => LudiqGUIUtility.currentInspectorWidth;
+
+	protected override float GetHeight(float width, GUIContent label)
+	{
+		if (ErrorMessage != null) {
+			var height = LudiqGUIUtility.GetHelpBoxHeight(ErrorMessage, MessageType.Error, width);
+			height += EditorGUIUtility.standardVerticalSpacing;
+			return height;
+		}
+
+		return EditorGUIUtility.singleLineHeight;
+	}
+}

Editor/Inspectors/VisualScriptingPlayerStatePropertyDefinitionInspector.cs

@@ -0,0 +1,26 @@
+// Visual Pinball Engine
+// Copyright (C) 2022 freezy and VPE Team
+//
+// This program is free software: you can redistribute it and/or modify
+// it under the terms of the GNU General Public License as published by
+// the Free Software Foundation, either version 3 of the License, or
+// (at your option) any later version.
+//
+// This program is distributed in the hope that it will be useful,
+// but WITHOUT ANY WARRANTY; without even the implied warranty of
+// MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
+// GNU General Public License for more details.
+//
+// You should have received a copy of the GNU General Public License
+// along with this program. If not, see <https://www.gnu.org/licenses/>.
+
+// using UnityEditor;
+//
+// namespace VisualPinball.Unity.VisualScripting.Editor
+// {
+// 	[CustomPropertyDrawer(typeof(VisualScriptingCoil))]
+// 	public class VisualScriptingPlayerStateDefinitionPropertyDrawer : PropertyDrawer
+// 	{
+// 	}
+// }
+

Editor/Inspectors/VisualScriptingPlayerStatePropertyDefinitionInspector.cs.meta

@@ -23,7 +23,9 @@ public abstract class GleUnitWidget<TUnit> : UnitWidget<TUnit> where TUnit : Uni
 		protected override NodeColorMix baseColor => GleAvailable ? NodeColorMix.TealReadable : new NodeColorMix { red = 1f, green = 0f, blue = 0f };
 		protected bool GameObjectAvailable => reference != null && reference.gameObject != null;
 		protected IGamelogicEngine Gle => reference.gameObject.GetComponentInParent<IGamelogicEngine>();
+		protected VisualScriptingGamelogicEngine VsGle => reference.gameObject.GetComponentInParent<VisualScriptingGamelogicEngine>();
 		private bool GleAvailable => GameObjectAvailable && Gle != null;
+		private bool VsGleAvailable => GameObjectAvailable && VsGle != null;
 
 		protected GleUnitWidget(FlowCanvas canvas, TUnit unit) : base(canvas, unit)
 		{

Editor/PropertyDrawers/VisualScriptingPlayerStateDefinitionPropertyDrawer.cs

@@ -0,0 +1,59 @@
+// Visual Pinball Engine
+// Copyright (C) 2022 freezy and VPE Team
+//
+// This program is free software: you can redistribute it and/or modify
+// it under the terms of the GNU General Public License as published by
+// the Free Software Foundation, either version 3 of the License, or
+// (at your option) any later version.
+//
+// This program is distributed in the hope that it will be useful,
+// but WITHOUT ANY WARRANTY; without even the implied warranty of
+// MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
+// GNU General Public License for more details.
+//
+// You should have received a copy of the GNU General Public License
+// along with this program. If not, see <https://www.gnu.org/licenses/>.
+
+// ReSharper disable UnusedType.Global
+
+using System;
+using System.Collections.Generic;
+using System.Linq;
+using Unity.VisualScripting;
+
+namespace VisualPinball.Unity.VisualScripting.Editor
+{
+	[Widget(typeof(PlayerStateSetUnit))]
+	public sealed class PlayerStateSetUnitWidget : GleUnitWidget<PlayerStateSetUnit>
+	{
+		public PlayerStateSetUnitWidget(FlowCanvas canvas, PlayerStateSetUnit unit) : base(canvas, unit)
+		{
+			_propertyNameInspectorConstructor = meta => new VariableNameInspector(meta, GetNameSuggestions);
+		}
+
+		protected override NodeColorMix baseColor => NodeColorMix.TealReadable;
+
+		private VariableNameInspector _propertyNameInspector;
+		private readonly Func<Metadata, VariableNameInspector> _propertyNameInspectorConstructor;
+
+		public override Inspector GetPortInspector(IUnitPort port, Metadata meta)
+		{
+			// if (port == unit.PropertyName) {
+			// 	InspectorProvider.instance.Renew(ref _propertyNameInspector, meta, _propertyNameInspectorConstructor);
+			//
+			// 	return _propertyNameInspector;
+			// }
+
+			return base.GetPortInspector(port, meta);
+		}
+
+		private IEnumerable<string> GetNameSuggestions()
+		{
+			if (!GameObjectAvailable) {
+				return new List<string>();
+			}
+			var gle = VsGle;
+			return gle == null ? new List<string>() : gle.PlayerStateDefinition.Select(prop => prop.Name).ToList();
+		}
+	}
+}

Editor/PropertyDrawers/VisualScriptingPlayerStateDefinitionPropertyDrawer.cs.meta

@@ -27,7 +27,7 @@ namespace VisualPinball.Unity.VisualScripting
 {
 	[DisallowMultipleComponent]
 	[AddComponentMenu("Visual Pinball/Gamelogic Engine/Visual Scripting Game Logic")]
-	public class VisualScriptingGamelogicEngine : MonoBehaviour, IGamelogicEngine
+	public class VisualScriptingGamelogicEngine : MonoBehaviour, IGamelogicEngine, ISerializationCallbackReceiver
 	{
 		public string Name => "Visual Scripting Gamelogic Engine";
 
@@ -151,5 +151,23 @@ public bool GetCoil(string id)
 		{
 			return _player.CoilStatuses.ContainsKey(id) && _player.CoilStatuses[id];
 		}
+
+
+		public void OnBeforeSerialize()
+		{
+			#if UNITY_EDITOR
+
+			var ids = new HashSet<string>();
+			foreach (var def in PlayerStateDefinition) {
+				if (!def.HasId || ids.Contains(def.Id)) {
+					def.GenerateId();
+				}
+				ids.Add(def.Id);
+			}
+			#endif
+		}
+		public void OnAfterDeserialize()
+		{
+		}
 	}
 }