Refactor SetLight node. Only takes in one light for now.

Author: freezy

Editor/Descriptors/SetLightUnitDescriptor.cs

@@ -0,0 +1,58 @@
+// Visual Pinball Engine
+// Copyright (C) 2021 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 Unity.VisualScripting;
+using VisualPinball.Unity.Editor;
+using IconSize = VisualPinball.Unity.Editor.IconSize;
+
+namespace VisualPinball.Unity.VisualScripting.Editor
+{
+	[Descriptor(typeof(SetLightUnit))]
+	public class SetLightUnitDescriptor : UnitDescriptor<SetLightUnit>
+	{
+		public SetLightUnitDescriptor(SetLightUnit target) : base(target)
+		{
+		}
+
+		protected override string DefinedSummary()
+		{
+			return "This node assigns a given value to a light or light group in the scene. \n\nNote that this doesn't pass through the gamelogic engine, thus no event will be triggered. However, it allows you to drive non-mapped lights as well.";
+		}
+
+		protected override EditorTexture DefinedIcon() => EditorTexture.Single(Unity.Editor.Icons.Light(IconSize.Large, IconColor.Orange));
+
+		protected override void DefinedPort(IUnitPort port, UnitPortDescription desc)
+		{
+			base.DefinedPort(port, desc);
+
+			switch (port.key) {
+				case nameof(SetLightUnit.LampComponent):
+					desc.summary = "The light component whose value you want to change. Assigning a light group will change all lights in the group.";
+					break;
+
+				case nameof(SetLightUnit.Value):
+					desc.summary = "The intensity to apply (0-1).";
+					break;
+
+				case nameof(SetLightUnit.ColorChannel):
+					desc.summary = "Which color channel to use. For non-RGB lights, use alpha.";
+					break;
+			}
+		}
+	}
+}

Editor/Descriptors/SetLightUnitDescriptor.cs.meta

@@ -0,0 +1,46 @@
+// Visual Pinball Engine
+// Copyright (C) 2021 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 UnityEditor;
+using UnityEngine;
+using VisualPinball.Unity.Editor;
+
+namespace VisualPinball.Unity.VisualScripting.Editor
+{
+	public class ObjectPickerInspector<T> : Inspector where T: class, IIdentifiableItemComponent
+	{
+		private readonly ObjectReferencePicker<T> _devicePicker;
+
+		public ObjectPickerInspector(Metadata metadata, string title, TableComponent tableComponent) : base(metadata)
+		{
+			_devicePicker = new ObjectReferencePicker<T>(title, tableComponent, true);
+		}
+
+		protected override float GetHeight(float width, GUIContent label) => EditorGUIUtility.singleLineHeight;
+
+		protected override void OnGUI(Rect position, GUIContent label)
+		{
+			position = BeginLabeledBlock(metadata, position, label);
+			_devicePicker.Render(position, (T)metadata.value, component => {
+				if (EndBlock(metadata)) {
+					metadata.RecordUndo();
+					metadata.value = component;
+				}
+			});
+		}
+	}
+}

Editor/Inspectors.meta

@@ -0,0 +1,46 @@
+// Visual Pinball Engine
+// Copyright (C) 2021 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 Unity.VisualScripting;
+
+namespace VisualPinball.Unity.VisualScripting.Editor
+{
+	[Widget(typeof(SetLightUnit))]
+	public sealed class SetLightUnitWidget : UnitWidget<SetLightUnit>
+	{
+		private ObjectPickerInspector<ILampDeviceComponent> _objectInspector;
+		private readonly Func<Metadata, ObjectPickerInspector<ILampDeviceComponent>> _setObjectInspectorConstructor;
+
+		public SetLightUnitWidget(FlowCanvas canvas, SetLightUnit unit) : base(canvas, unit)
+		{
+			var tc = reference.gameObject.GetComponentInParent<TableComponent>();
+			_setObjectInspectorConstructor = meta => new ObjectPickerInspector<ILampDeviceComponent>(meta, "Lamps", tc);
+		}
+
+		public override Inspector GetPortInspector(IUnitPort port, Metadata meta)
+		{
+			if (port == unit.LampComponent) {
+				InspectorProvider.instance.Renew(ref _objectInspector, meta, _setObjectInspectorConstructor);
+				return _objectInspector;
+			}
+			return base.GetPortInspector(port, meta);
+		}
+
+	}
+}

Editor/Inspectors/ObjectPickerInspector.cs

@@ -20,6 +20,7 @@
 namespace VisualPinball.Unity.VisualScripting
 {
 	[UnitTitle("Set Coil")]
+	[UnitSurtitle("Gamelogic Engine")]
 	[UnitCategory("Visual Pinball")]
 	public class SetCoilUnit : GleUnit
 	{

Editor/Inspectors/ObjectPickerInspector.cs.meta

@@ -33,13 +33,25 @@ public abstract class GleUnit : Unit, IGleUnit
 		[DoNotSerialize]
 		protected IGamelogicEngine Gle;
 
+		[DoNotSerialize]
+		protected Player Player;
+
 		protected bool AssertGle(Flow flow)
 		{
 			if (Gle != null) {
 				return true;
 			}
 			Gle = flow.stack.gameObject.GetComponentInParent<IGamelogicEngine>();
-			return Gle == null;
+			return Gle != null;
+		}
+
+		protected bool AssertPlayer(Flow flow)
+		{
+			if (Player != null) {
+				return true;
+			}
+			Player = flow.stack.gameObject.GetComponentInParent<Player>();
+			return Player != null;
 		}
 	}
 

Editor/Widgets/SetLightUnitWidget.cs

@@ -20,6 +20,7 @@
 namespace VisualPinball.Unity.VisualScripting
 {
 	[UnitTitle("Get Lamp Value")]
+	[UnitSurtitle("Gamelogic Engine")]
 	[UnitCategory("Visual Pinball")]
 	public class GetLampUnit : GleUnit
 	{