fix: disable bundle decorator for joja membership (#8)

Author: gottyduke

Informant/Implementation/Decorator/BundleDecorator.cs

@@ -28,11 +28,15 @@ public BundleDecorator(IModHelper modHelper)
 
     public bool HasDecoration(Item input)
     {
+        if (Game1.MasterPlayer.mailReceived.Contains("JojaMember")) {
+            return false;
+        }
+
         if (_bundles.Any() && input is SObject obj && !obj.bigCraftable.Value) {
 
             int[]? allowedAreas;
 
-            if (!Game1.player.mailReceived.Contains("canReadJunimoText")) {
+            if (!Game1.MasterPlayer.mailReceived.Contains("canReadJunimoText")) {
                 // if player can't read Junimo text, they can't have bundles yet
                 allowedAreas = null;
             } else {

Informant/Implementation/Decorator/ShippingBinDecorator.cs

@@ -31,7 +31,7 @@ public bool HasDecoration(Item input)
                 return false;
             }
             var parentSheetIndex = input.ParentSheetIndex.ToString();
-            var alreadyShipped = Game1.player.basicShipped.ContainsKey(parentSheetIndex) ? Game1.player.basicShipped[parentSheetIndex] : 0;
+            var alreadyShipped = Game1.MasterPlayer.basicShipped.ContainsKey(parentSheetIndex) ? Game1.MasterPlayer.basicShipped[parentSheetIndex] : 0;
 
             if (!Ship15Items.Contains(input.ParentSheetIndex)) {
                 // we only need to ship this item once
@@ -58,7 +58,7 @@ public Decoration Decorate(Item input)
             return null;
         }
         var parentSheetIndex = input.ParentSheetIndex.ToString();
-        var alreadyShipped = Game1.player.basicShipped.ContainsKey(parentSheetIndex) ? Game1.player.basicShipped[parentSheetIndex] : 0;
+        var alreadyShipped = Game1.MasterPlayer.basicShipped.ContainsKey(parentSheetIndex) ? Game1.MasterPlayer.basicShipped[parentSheetIndex] : 0;
         const int needToBeShipped = 15;
         if (alreadyShipped >= needToBeShipped) {
             // we don't need to ship anything? why even show the decorator?

Informant/Implementation/NewRecipeDisplayable.cs

@@ -75,8 +75,8 @@ private static void DrawOverlayIfNecessary(ClickableTextureComponent __instance,
 
         // recipe.timesCrafted is not updated it seems
         var timesCrafted = recipe.isCookingRecipe
-                ? (Game1.player.recipesCooked.ContainsKey(recipe.getIndexOfMenuView()) ? Game1.player.recipesCooked[recipe.getIndexOfMenuView()] : 0)
-                : (Game1.player.craftingRecipes.ContainsKey(recipe.name) ? Game1.player.craftingRecipes[recipe.name] : 0);
+                ? (Game1.MasterPlayer.recipesCooked.ContainsKey(recipe.getIndexOfMenuView()) ? Game1.MasterPlayer.recipesCooked[recipe.getIndexOfMenuView()] : 0)
+                : (Game1.MasterPlayer.craftingRecipes.ContainsKey(recipe.name) ? Game1.MasterPlayer.craftingRecipes[recipe.name] : 0);
         if (timesCrafted > 0) {
             // we are on the recipe page, have a recipe which was already craftet? nice, it's not new
             return;