Merge pull request #718 from qsb-dev/dev

1.5.0
- Fix some quantum objects in NH mods. They should move around now, at least.
- Fix most items in NH mods. Custom item logic will not work, but you can do normal item things (pick up, drop, insert into slot).
- Fix chat/HUD working in custom systems. It still does not work in the eye yet.

Author: JohnCorby

.editorconfig

@@ -0,0 +1,2 @@
+[*.cs]
+indent_style = tab

QSB-NH/QSBNH.cs

@@ -6,6 +6,7 @@
 using OWML.Common;
 using QSB;
 using QSB.HUD;
+using QSB.HUD.Messages;
 using QSB.Messaging;
 using QSB.Utility;
 using UnityEngine;
@@ -33,6 +34,8 @@ private void OnWakeUp()
 			// Allow time for MultiplayerHUDManager.OnWakeUp to run
 			Delay.RunNextFrame(() =>
 			{
+				var triggers = new List<PlanetTrigger>();
+
 				var currentPlanets = NewHorizons.Main.BodyDict[NewHorizons.Main.Instance.CurrentStarSystem];
 				foreach (var planet in currentPlanets)
 				{
@@ -62,8 +65,20 @@ private void OnWakeUp()
 
 					if (!nhAstro.isVanilla)
 					{
-						MultiplayerHUDManager.CreateTrigger(nhAstro.GetRootSector().gameObject, astroObjName);
+						triggers.Add(MultiplayerHUDManager.CreateTrigger(nhAstro.GetRootSector().gameObject, astroObjName));
+					}
+				}
+
+				foreach (var trigger in triggers)
+				{
+					if (!trigger._sector.ContainsOccupant(DynamicOccupant.Player))
+					{
+						continue;
 					}
+
+					MultiplayerHUDManager.HUDIconStack.Push(trigger.PlanetID);
+					new PlanetMessage(trigger.PlanetID).Send();
+					break;
 				}
 			});
 		}

QSB/HUD/MultiplayerHUDManager.cs

@@ -375,24 +375,46 @@ Sprite GetSprite(AstroObject.Name name)
 
 		CreatePlanetToSprite();
 
-		CreateTrigger("TowerTwin_Body/Sector_TowerTwin", AstroObject.Name.TowerTwin);
-		CreateTrigger("CaveTwin_Body/Sector_CaveTwin", AstroObject.Name.CaveTwin);
-		CreateTrigger("TimberHearth_Body/Sector_TH", AstroObject.Name.TimberHearth);
-		CreateTrigger("Moon_Body/Sector_THM", AstroObject.Name.TimberMoon);
-		CreateTrigger("BrittleHollow_Body/Sector_BH", AstroObject.Name.BrittleHollow);
-		CreateTrigger("VolcanicMoon_Body/Sector_VM", AstroObject.Name.VolcanicMoon);
-		CreateTrigger("GiantsDeep_Body/Sector_GD", AstroObject.Name.GiantsDeep);
-		CreateTrigger("DarkBramble_Body/Sector_DB", AstroObject.Name.DarkBramble);
-		CreateTrigger("Comet_Body/Sector_CO", AstroObject.Name.Comet);
-		CreateTrigger("WhiteHole_Body/Sector_WhiteHole", AstroObject.Name.WhiteHole);
-		CreateTrigger("RingWorld_Body/Sector_RingWorld", AstroObject.Name.RingWorld); // TODO : this doesnt work????
-		CreateTrigger("QuantumMoon_Body/Sector_QuantumMoon", AstroObject.Name.QuantumMoon);
+		var planetTriggers = new List<PlanetTrigger>
+		{
+			CreateTrigger("TowerTwin_Body/Sector_TowerTwin", AstroObject.Name.TowerTwin),
+			CreateTrigger("CaveTwin_Body/Sector_CaveTwin", AstroObject.Name.CaveTwin),
+			CreateTrigger("TimberHearth_Body/Sector_TH", AstroObject.Name.TimberHearth),
+			CreateTrigger("Moon_Body/Sector_THM", AstroObject.Name.TimberMoon),
+			CreateTrigger("BrittleHollow_Body/Sector_BH", AstroObject.Name.BrittleHollow),
+			CreateTrigger("VolcanicMoon_Body/Sector_VM", AstroObject.Name.VolcanicMoon),
+			CreateTrigger("GiantsDeep_Body/Sector_GD", AstroObject.Name.GiantsDeep),
+			CreateTrigger("DarkBramble_Body/Sector_DB", AstroObject.Name.DarkBramble),
+			CreateTrigger("Comet_Body/Sector_CO", AstroObject.Name.Comet),
+			CreateTrigger("WhiteHole_Body/Sector_WhiteHole", AstroObject.Name.WhiteHole),
+			CreateTrigger("RingWorld_Body/Sector_RingWorld", AstroObject.Name.RingWorld),
+			CreateTrigger("QuantumMoon_Body/Sector_QuantumMoon", AstroObject.Name.QuantumMoon)
+		};
 
 		HUDIconStack.Clear();
 		HUDIconStack.Push("__SPACE__");
 
-		HUDIconStack.Push("TimberHearth");
-		new PlanetMessage("TimberHearth").Send();
+		foreach (var trigger in planetTriggers)
+		{
+			if (trigger == null)
+			{
+				continue;
+			}
+
+			if (trigger._sector == null)
+			{
+				continue;
+			}
+
+			if (!trigger._sector.ContainsOccupant(DynamicOccupant.Player))
+			{
+				continue;
+			}
+
+			HUDIconStack.Push(trigger.PlanetID);
+			new PlanetMessage(trigger.PlanetID).Send();
+			break;
+		}
 
 		_textChat = multiplayerGroup.transform.Find("TextChat");
 		var inputFieldGO = _textChat.Find("InputField");
@@ -561,6 +583,7 @@ public static PlanetTrigger CreateTrigger(GameObject parent, string name)
 			triggerGO.transform.SetParent(parent.transform, false);
 			triggerGO.SetActive(false);
 			var trigger = triggerGO.AddComponent<PlanetTrigger>();
+			trigger.Reset(); // Force _sector to be found immediately
 			trigger.PlanetID = name;
 			triggerGO.SetActive(true);
 			return trigger;

QSB/ItemSync/ItemManager.cs

@@ -1,11 +1,14 @@
 using Cysharp.Threading.Tasks;
+using HarmonyLib;
 using OWML.Common;
 using QSB.ItemSync.WorldObjects;
 using QSB.ItemSync.WorldObjects.Items;
 using QSB.ItemSync.WorldObjects.Sockets;
 using QSB.Utility;
 using QSB.Utility.Deterministic;
 using QSB.WorldSync;
+using System;
+using System.Collections.Generic;
 using System.Linq;
 using System.Threading;
 using UnityEngine;
@@ -21,21 +24,66 @@ public override async UniTask BuildWorldObjects(OWScene scene, CancellationToken
 		DebugLog.DebugWrite("Building OWItems...", MessageType.Info);
 
 		// Items
-		QSBWorldSync.Init<QSBNomaiConversationStone, NomaiConversationStone>();
-		QSBWorldSync.Init<QSBScrollItem, ScrollItem>();
-		QSBWorldSync.Init<QSBSharedStone, SharedStone>();
-		QSBWorldSync.Init<QSBSimpleLanternItem, SimpleLanternItem>();
 		QSBWorldSync.Init<QSBSlideReelItem, SlideReelItem>();
 		QSBWorldSync.Init<QSBWarpCoreItem, WarpCoreItem>();
 		// dream lantern and vision torch are set up in their own managers
+		// the rest can just use the generic thing below because they dont have special things using them
+
+		// Use the basic QSBItem class for any items that do not require custom code through a derived class (mod compatibility)
+		// QSB addons can still define their own QSBItem derived classes and they will just get skipped here
+		var handledItemTypes = new HashSet<Type>(GetHandledItemTypes()); // set cuz we do Contains below
+		DebugLog.DebugWrite($"Handled OWItem types (the rest will get generic QSBItem support) are: {handledItemTypes.Join()}");
+		var otherItemlistToInitFrom = QSBWorldSync.GetUnityObjects<OWItem>()
+			.Where(x => !handledItemTypes.Contains(x.GetType()))
+			.SortDeterministic();
+		// could make a subclass for this but i dont care, and would have to filter out from reflection thing below
+		QSBWorldSync.Init<QSBItem<OWItem>, OWItem>(otherItemlistToInitFrom);
 
 		// Sockets
 		QSBWorldSync.Init<QSBItemSocket, OWItemSocket>();
 
 		// other drop targets that don't already have world objects
+		// breaks if mod adds custom subclass of IItemDropTarget and then makes WorldObject of that but can just fix that when that happens
 		var listToInitFrom = QSBWorldSync.GetUnityObjects<MonoBehaviour>()
 			.Where(x => x is IItemDropTarget and not (RaftDock or RaftController or PrisonCellElevator))
 			.SortDeterministic();
 		QSBWorldSync.Init<QSBOtherDropTarget, MonoBehaviour>(listToInitFrom);
 	}
+
+	/// <summary>
+	/// Gets all types that extend QSBItem and returns the list of OWItem types that are already handled by dedicated classes
+	/// </summary>
+	private static IEnumerable<Type> GetHandledItemTypes()
+	{
+		var assemblies = QSBCore.Addons.Values
+			.Select(x => x.GetType().Assembly)
+			.Append(typeof(QSBCore).Assembly);
+
+		if (QSBCore.QSBNHAssembly != null)
+		{
+			assemblies = assemblies.Append(QSBCore.QSBNHAssembly);
+		}
+
+		// If the class inherits from QSBItem<T>, this will return what T is else null
+		static Type GetTypeFromQSBItem(Type type)
+		{
+			if (type.IsInterface || type.IsAbstract || type.BaseType == null)
+			{
+				return null;
+			}
+			if (type.BaseType.IsGenericType && type.BaseType.GetGenericTypeDefinition() == typeof(QSBItem<>))
+			{
+				return type.BaseType.GetGenericArguments()[0];
+			}
+			else
+			{
+				return GetTypeFromQSBItem(type.BaseType);
+			}
+		}
+
+		return assemblies.SelectMany(x => x.GetTypes())
+			.Select(GetTypeFromQSBItem)
+			.Where(x => x != null)
+			.OrderBy(x => x.FullName);
+	}
 }

QSB/ItemSync/WorldObjects/Items/QSBItem.cs

@@ -12,6 +12,8 @@
 
 namespace QSB.ItemSync.WorldObjects.Items;
 
+// not abstract so modded items can use this QSBItem<OWItem>
+// could use QSBOtherItem subclass but i dont feel like it
 public class QSBItem<T> : WorldObject<T>, IQSBItem
 	where T : OWItem
 {

QSB/ItemSync/WorldObjects/Items/QSBNomaiConversationStone.cs

@@ -3,4 +3,4 @@
 public class QSBWarpCoreItem : QSBItem<WarpCoreItem>
 {
 	public bool IsVesselCoreType() => AttachedObject.IsVesselCoreType();
-}
+}