unify SMAPI versions for SDV 1.11 and 1.2 using compile switches where needed (#264)

1 files changed, 90 insertions, 73 deletions

diff --git a/src/StardewModdingAPI/StardewModdingAPI.config.json b/src/StardewModdingAPI/StardewModdingAPI.config.json
index 9438c621..ba698d35 100644
--- a/src/StardewModdingAPI/StardewModdingAPI.config.json
+++ b/src/StardewModdingAPI/StardewModdingAPI.config.json
@@ -27,6 +27,10 @@ This file contains advanced configuration for SMAPI. You generally shouldn't cha
    * Changing this field is not recommended and may destabilise your game.
    */
   "ModCompatibility": [
+
+    /****
+    ** Stardew Valley 1.2+ only
+    ****/
     {
       "Name": "AccessChestAnywhere",
       "ID": "AccessChestAnywhere",
@@ -34,9 +38,94 @@ This file contains advanced configuration for SMAPI. You generally shouldn't cha
       "Compatibility": "AssumeBroken",
       "UpdateUrl": "http://www.nexusmods.com/stardewvalley/mods/257",
       "UnofficialUpdateUrl": "http://www.nexusmods.com/stardewvalley/mods/518",
-      "Notes": "Crashes with 'Method not found: Void StardewValley.Item.set_Name(System.String)'."
+      "Notes": "Crashes with 'Method not found: Void StardewValley.Item.set_Name(System.String)'.",
+      "OnlyStardewValleyBeta": true
+    },
+    {
+      "Name": "Chests Anywhere",
+      "ID": "ChestsAnywhere",
+      "UpperVersion": "1.8.2",
+      "Compatibility": "AssumeBroken",
+      "UpdateUrl": "http://www.nexusmods.com/stardewvalley/mods/518",
+      "Notes": "Crashes with 'Method not found: Void StardewValley.Menus.TextBox.set_Highlighted(Boolean)'.",
+      "OnlyStardewValleyBeta": true
+    },
+    {
+      "Name": "Chests Anywhere",
+      "ID": "Pathoschild.ChestsAnywhere",
+      "UpperVersion": "1.9-beta",
+      "Compatibility": "AssumeBroken",
+      "UpdateUrl": "http://www.nexusmods.com/stardewvalley/mods/518",
+      "Notes": "ID changed in 1.9. Crashes with InvalidOperationException: 'The menu doesn't seem to have a player inventory'.",
+      "OnlyStardewValleyBeta": true
+    },
+    {
+      "Name": "CJB Automation",
+      "ID": "CJBAutomation",
+      "UpperVersion": "1.4",
+      "Compatibility": "AssumeBroken",
+      "UpdateUrl": "http://www.nexusmods.com/stardewvalley/mods/211",
+      "Notes": "Crashes with 'Method not found: Void StardewValley.Item.set_Name(System.String)'.",
+      "OnlyStardewValleyBeta": true
+    },
+    {
+      "Name": "Cooking Skill",
+      "ID": "CookingSkill",
+      "UpperVersion": "1.0.3",
+      "Compatibility": "AssumeBroken",
+      "UpdateUrl": "http://www.nexusmods.com/stardewvalley/mods/522",
+      "Notes": "Crashes with 'Method not found: Void StardewValley.Buff..ctor(Int32, Int32, Int32, Int32, Int32, Int32, Int32, Int32, Int32, Int32, Int32, Int32, Int32, System.String)'.",
+      "OnlyStardewValleyBeta": true
+    },
+    {
+      "Name": "Extended Fridge",
+      "ID": "Mystra007ExtendedFridge",
+      "UpperVersion": "1.0",
+      "Compatibility": "AssumeBroken",
+      "UpdateUrl": "http://www.nexusmods.com/stardewvalley/mods/485",
+      "Notes": "Actual upper version is 0.94, but mod incorrectly sets it to 1.0 in the manifest. Crashes with 'Field not found: StardewValley.Game1.mouseCursorTransparency'.",
+      "OnlyStardewValleyBeta": true
+    },
+    {
+      "Name": "Get Dressed",
+      "ID": "GetDressed.dll",
+      "UpperVersion": "3.2",
+      "Compatibility": "AssumeBroken",
+      "UpdateUrl": "http://www.nexusmods.com/stardewvalley/mods/331",
+      "Notes": "Crashes with NullReferenceException in GameEvents.UpdateTick.",
+      "OnlyStardewValleyBeta": true
+    },
+    {
+      "Name": "Lookup Anything",
+      "ID": "LookupAnything",
+      "UpperVersion": "1.10",
+      "Compatibility": "AssumeBroken",
+      "UpdateUrl": "http://www.nexusmods.com/stardewvalley/mods/541",
+      "Notes": "Crashes with FormatException when looking up NPCs.",
+      "OnlyStardewValleyBeta": true
+    },
+    {
+      "Name": "Lookup Anything",
+      "ID": "Pathoschild.LookupAnything",
+      "UpperVersion": "1.10.1",
+      "Compatibility": "AssumeBroken",
+      "UpdateUrl": "http://www.nexusmods.com/stardewvalley/mods/541",
+      "Notes": "ID changed in 1.10.1. Crashes with FormatException when looking up NPCs.",
+      "OnlyStardewValleyBeta": true
     },
     {
+      "Name": "Teleporter",
+      "ID": "Teleporter",
+      "UpperVersion": "1.0.2",
+      "Compatibility": "AssumeBroken",
+      "UpdateUrl": "http://community.playstarbound.com/resources/4374",
+      "Notes": "Crashes with 'InvalidOperationException: The StardewValley.Menus.MapPage object doesn't have a private 'points' instance field'.",
+      "OnlyStardewValleyBeta": true
+    },
+
+    /****
+    ** Any version of Stardew Valley
+    ****/    {
       "Name": "Almighty Tool",
       "ID": "AlmightyTool.dll",
       "UpperVersion": "1.1.1",
@@ -69,30 +158,6 @@ This file contains advanced configuration for SMAPI. You generally shouldn't cha
       "Notes": "Not compatible with Stardew Valley 1.1+"
     },
     {
-      "Name": "Chests Anywhere",
-      "ID": "ChestsAnywhere",
-      "UpperVersion": "1.8.2",
-      "Compatibility": "AssumeBroken",
-      "UpdateUrl": "http://www.nexusmods.com/stardewvalley/mods/518",
-      "Notes": "Crashes with 'Method not found: Void StardewValley.Menus.TextBox.set_Highlighted(Boolean)'."
-    },
-    {
-      "Name": "Chests Anywhere",
-      "ID": "Pathoschild.ChestsAnywhere",
-      "UpperVersion": "1.9-beta",
-      "Compatibility": "AssumeBroken",
-      "UpdateUrl": "http://www.nexusmods.com/stardewvalley/mods/518",
-      "Notes": "ID changed in 1.9. Crashes with InvalidOperationException: 'The menu doesn't seem to have a player inventory'."
-    },
-    {
-      "Name": "CJB Automation",
-      "ID": "CJBAutomation",
-      "UpperVersion": "1.4",
-      "Compatibility": "AssumeBroken",
-      "UpdateUrl": "http://www.nexusmods.com/stardewvalley/mods/211",
-      "Notes": "Crashes with 'Method not found: Void StardewValley.Item.set_Name(System.String)'."
-    },
-    {
       "Name": "CJB Cheats Menu",
       "ID": "CJBCheatsMenu",
       "UpperVersion": "1.12",
@@ -115,14 +180,6 @@ This file contains advanced configuration for SMAPI. You generally shouldn't cha
       "Notes": "Uses SMAPI's internal SGame class."
     },
     {
-      "Name": "Cooking Skill",
-      "ID": "CookingSkill",
-      "UpperVersion": "1.0.3",
-      "Compatibility": "AssumeBroken",
-      "UpdateUrl": "http://www.nexusmods.com/stardewvalley/mods/522",
-      "Notes": "Crashes with 'Method not found: Void StardewValley.Buff..ctor(Int32, Int32, Int32, Int32, Int32, Int32, Int32, Int32, Int32, Int32, Int32, Int32, Int32, System.String)'."
-    },
-    {
       "Name": "Enemy Health Bars",
       "ID": "SPDHealthBar",
       "UpperVersion": "1.7",
@@ -139,38 +196,6 @@ This file contains advanced configuration for SMAPI. You generally shouldn't cha
       "Notes": "Uses obsolete StardewModdingAPI.Inheritance.SObject until 1.6.1; then crashes until 1.6.4 ('Entoarox Framework requested an immediate game shutdown: Fatal error attempting to update player tick properties System.NullReferenceException: Object reference not set to an instance of an object. at Entoarox.Framework.PlayerHelper.Update(Object s, EventArgs e)')."
     },
     {
-      "Name": "Extended Fridge",
-      "ID": "Mystra007ExtendedFridge",
-      "UpperVersion": "1.0",
-      "Compatibility": "AssumeBroken",
-      "UpdateUrl": "http://www.nexusmods.com/stardewvalley/mods/485",
-      "Notes": "Actual upper version is 0.94, but mod incorrectly sets it to 1.0 in the manifest. Crashes with 'Field not found: StardewValley.Game1.mouseCursorTransparency'."
-    },
-    {
-      "Name": "Get Dressed",
-      "ID": "GetDressed.dll",
-      "UpperVersion": "3.2",
-      "Compatibility": "AssumeBroken",
-      "UpdateUrl": "http://www.nexusmods.com/stardewvalley/mods/331",
-      "Notes": "Crashes with NullReferenceException in GameEvents.UpdateTick."
-    },
-    {
-      "Name": "Lookup Anything",
-      "ID": "LookupAnything",
-      "UpperVersion": "1.10",
-      "Compatibility": "AssumeBroken",
-      "UpdateUrl": "http://www.nexusmods.com/stardewvalley/mods/541",
-      "Notes": "Crashes with FormatException when looking up NPCs."
-    },
-    {
-      "Name": "Lookup Anything",
-      "ID": "Pathoschild.LookupAnything",
-      "UpperVersion": "1.10.1",
-      "Compatibility": "AssumeBroken",
-      "UpdateUrl": "http://www.nexusmods.com/stardewvalley/mods/541",
-      "Notes": "ID changed in 1.10.1. Crashes with FormatException when looking up NPCs."
-    },
-    {
       "Name": "Makeshift Multiplayer",
       "ID": "StardewValleyMP",
       "Compatibility": "AssumeBroken",
@@ -228,14 +253,6 @@ This file contains advanced configuration for SMAPI. You generally shouldn't cha
       "Notes": "Crashes with 'Method not found: Void StardewModdingAPI.Command.CallCommand(System.String)'."
     },
     {
-      "Name": "Teleporter",
-      "ID": "Teleporter",
-      "UpperVersion": "1.0.2",
-      "Compatibility": "AssumeBroken",
-      "UpdateUrl": "http://community.playstarbound.com/resources/4374",
-      "Notes": "Crashes with 'InvalidOperationException: The StardewValley.Menus.MapPage object doesn't have a private 'points' instance field'."
-    },
-    {
       "Name": "Zoryn's Better RNG",
       "ID": "76b6d1e1-f7ba-4d72-8c32-5a1e6d2716f6",
       "UpperVersion": "1.5",