From 65120d64efa09c4b8a4881d8c9a0175b4ce30812 Mon Sep 17 00:00:00 2001
From: Jesse Plamondon-Willard <Pathoschild@users.noreply.github.com>
Date: Fri, 16 Jun 2023 22:38:08 -0400
Subject: fix Context.HasRemotePlayers being true when there's no farmhands

---
 src/SMAPI/Context.cs | 5 ++---
 1 file changed, 2 insertions(+), 3 deletions(-)

(limited to 'src/SMAPI/Context.cs')

diff --git a/src/SMAPI/Context.cs b/src/SMAPI/Context.cs
index 978459e8..3e3ee784 100644
--- a/src/SMAPI/Context.cs
+++ b/src/SMAPI/Context.cs
@@ -88,7 +88,7 @@ namespace StardewModdingAPI
         public static bool IsSplitScreen => LocalMultiplayer.IsLocalMultiplayer();
 
         /// <summary>Whether there are players connected over the network.</summary>
-        public static bool HasRemotePlayers => Context.IsMultiplayer && !Game1.hasLocalClientsOnly;
+        public static bool HasRemotePlayers => Context.IsMultiplayer && !Game1.hasLocalClientsOnly && Game1.getOnlineFarmers().Count > 1;
 
         /// <summary>Whether the current player is the main player. This is always true in single-player, and true when hosting in multiplayer.</summary>
         public static bool IsMainPlayer => Game1.IsMasterGame && Context.ScreenId == 0 && TitleMenu.subMenu is not FarmhandMenu;
@@ -97,8 +97,7 @@ namespace StardewModdingAPI
         /*********
         ** Public methods
         *********/
-        /// <summary>Get whether a screen ID is still active.</summary>
-        /// <param name="id">The screen ID.</param>
+        /// <summary>Get whether a screen ID is still active.</summary> <param name="id">The screen ID.</param>
         public static bool HasScreenId(int id)
         {
             return Context.ActiveScreenIds.Contains(id);
-- 
cgit