-Updated version

-Removed an unused reference
-Did some general code cleanup

Author: solidDoWant

BaseModuleType.cs

@@ -29,13 +29,13 @@ public override GameObject loadPrefab(int sizeIndex)
             //Add collider to object for raycasting
             foreach (Transform transform in ModuleObjects[adjustedSizeIndex].transform)
             {
-                //Make the collider for the gameobject the same size/shape as the mesh
+                //Make the collider for the GameObject the same size/shape as the mesh
                 if (transform.gameObject.GetComponent<MeshFilter>() != null)
                 {
                     transform.gameObject.AddComponent<MeshCollider>().sharedMesh = transform.gameObject.GetComponent<MeshFilter>().sharedMesh;
                 }
 
-                //This is copied directly from PB code. Not really sure that it's needed, but I'll leave it in place incase sombody else is comparing this to
+                //This is copied directly from PB code. Not really sure that it's needed, but I'll leave it in place in case somebody else is comparing this to
                 //PB code
                 if (transform.gameObject.name.IsValidTag())
                 {

FrameworkMod.cs

@@ -1,13 +1,12 @@
 using System;
-using System.Reflection;
 using Planetbase;
-using UnityEngine;
 
 namespace PlanetbaseFramework
 {
     public class FrameworkMod : ModBase
     {
-        public new static Version ModVersion => Assembly.GetExecutingAssembly().GetName().Version;
+        public const string AssemblyVersion = "2.2.0.0";
+        public new static readonly Version ModVersion = new Version(AssemblyVersion);
 
         public FrameworkMod()
         {
@@ -23,11 +22,6 @@ public override void Init()
             InjectPatches();
         }
 
-        public static void Callback(string condition, string stackTrace, LogType type)
-        {
-
-        }
-
         private class ModTitleButton : TitleButton
         {
             public ModTitleButton() : base("mod_titlemenu") { }

ModBase.cs

@@ -188,7 +188,7 @@ private string[] GetAssetsMatchingFileType(string fileType, string subfolder = n
             return Directory.Exists(searchPath) ? Directory.GetFiles(searchPath, "*." + fileType) : new string[0];
         }
 
-        public HarmonyInstance GetHarmonyInstance() => Harmony ?? HarmonyInstance.Create(ModName);
+        public HarmonyInstance GetHarmonyInstance() => Harmony ?? (Harmony = HarmonyInstance.Create(ModName));
 
         public void InjectPatches()
         {
@@ -202,6 +202,7 @@ public void RegisterTitleButton(TitleButton button)
 
         /// <summary>
         /// Do any pre-load actions before embedded resources are loaded, such as removing existing folders/items.
+        /// Warning: this call will be made in the constructor before child initialization.
         /// </summary>
         /// <param name="resourceNames">The name of all the resources to be loaded</param>
         protected virtual void PreProcessEmbeddedResources(string[] resourceNames)
@@ -211,6 +212,7 @@ protected virtual void PreProcessEmbeddedResources(string[] resourceNames)
 
         /// <summary>
         /// Do any pre-load actions before an embedded resource is loaded, such as removing existing folders/items.
+        /// Warning: this call will be made in the constructor before child initialization.
         /// </summary>
         /// <param name="resourceName">The name of the resources being loaded</param>
         /// <returns>True if the resource should be loaded, false otherwise</returns>

ModListGameState.cs

@@ -6,7 +6,8 @@ namespace PlanetbaseFramework
     //This class was thrown together fairly quickly, but should be a decent example on making new gamestates.
     public class ModListGameState : GameState
     {
-        public GuiRenderer mGuiRenderer = new GuiRenderer();
+        private GuiRenderer Renderer { get; } = new GuiRenderer();
+
         public override bool isTitleState()
         {
             return true;
@@ -20,20 +21,26 @@ public override void onGui()
             }
 
             PrintLine("Loaded Mods:", 0);
-            for(int i = 0; i < ModLoader.ModList.Count; i++)
+            for(var i = 0; i < ModLoader.ModList.Count; i++)
             {
                 PrintLine(ModLoader.ModList[i].ModName, i + 1);
             }
 
-            if(this.mGuiRenderer.renderBackButton(new Vector2(Screen.width - GuiRenderer.getMenuButtonSize(FontSize.Huge).x, Screen.height - GuiRenderer.getMenuButtonSize(FontSize.Huge).y))){
+            if (Renderer.renderBackButton(
+                new Vector2(
+                    Screen.width - GuiRenderer.getMenuButtonSize(FontSize.Huge).x,
+                    Screen.height - GuiRenderer.getMenuButtonSize(FontSize.Huge).y
+                )
+            ))
+            {
                 GameManager.getInstance().setGameStateTitle();
             }
         }
 
         private void PrintLine(string text, int lineNumber)
         {
             Vector2 textLocation = new Vector2(50, 80);
-            GUIStyle labelStyle = mGuiRenderer.getLabelStyle(FontSize.Huge, FontStyle.Bold, TextAnchor.LowerLeft, FontType.Title);
+            GUIStyle labelStyle = Renderer.getLabelStyle(FontSize.Huge, FontStyle.Bold, TextAnchor.LowerLeft, FontType.Title);
             labelStyle.normal.textColor = Color.blue;
 
             GUI.Label(new Rect(textLocation.x, textLocation.y + (GuiRenderer.getMenuButtonSize(FontSize.Huge).y )* lineNumber, Screen.width, GuiRenderer.getMenuButtonSize(FontSize.Huge).y - 30), text, labelStyle);

PlanetbaseFramework.csproj

@@ -62,7 +62,6 @@
     </Reference>
     <Reference Include="System" />
     <Reference Include="System.Core" />
-    <Reference Include="System.Drawing" />
     <Reference Include="System.Xml" />
     <Reference Include="UnityEngine">
       <HintPath>..\..\..\..\..\Program Files (x86)\Steam\steamapps\common\Planetbase\Planetbase_Data\Managed\UnityEngine.dll</HintPath>

Properties/AssemblyInfo.cs

@@ -1,5 +1,6 @@
 using System.Reflection;
 using System.Runtime.InteropServices;
+using PlanetbaseFramework;
 
 // General Information about an assembly is controlled through the following 
 // set of attributes. Change these attribute values to modify the information
@@ -9,7 +10,7 @@
 [assembly: AssemblyConfiguration("")]
 [assembly: AssemblyCompany("solidDoWant")]
 [assembly: AssemblyProduct("PlanetbaseFramework")]
-[assembly: AssemblyCopyright("Copyright ©  2018")]
+[assembly: AssemblyCopyright("Copyright ©  2019")]
 [assembly: AssemblyTrademark("")]
 [assembly: AssemblyCulture("")]
 
@@ -31,5 +32,5 @@
 // You can specify all the values or you can default the Build and Revision Numbers 
 // by using the '*' as shown below:
 // [assembly: AssemblyVersion("1.0.*")]
-[assembly: AssemblyVersion("2.1.2.0")]
-[assembly: AssemblyFileVersion("1.0.0.0")]
+[assembly: AssemblyVersion(FrameworkMod.AssemblyVersion)]
+[assembly: AssemblyFileVersion(FrameworkMod.AssemblyVersion)]

Utils.cs

@@ -182,10 +182,7 @@ public static bool IsValidTag(this string toCheck)
             }
         }
 
-        public static bool Compare (this Type t1, Type t2)
-        {
-            return t1.FullName.Equals(t2.FullName);
-        }
+        public static bool Compare(this Type t1, Type t2) => t1.FullName != null && t1.FullName.Equals(t2.FullName);
 
         public static string[] ListEmbeddedFiles()
         {