diff --git a/Content.Server/Nutrition/EntitySystems/DrinkSystem.cs b/Content.Server/Nutrition/EntitySystems/DrinkSystem.cs
index b8e334fb0dafc5..d9122ff278c387 100644
--- a/Content.Server/Nutrition/EntitySystems/DrinkSystem.cs
+++ b/Content.Server/Nutrition/EntitySystems/DrinkSystem.cs
@@ -17,6 +17,7 @@
using Content.Shared.DoAfter;
using Content.Shared.EntityEffects;
using Content.Shared.FixedPoint;
+using Content.Shared.Hands.EntitySystems;
using Content.Shared.IdentityManagement;
using Content.Shared.Interaction;
using Content.Shared.Interaction.Events;
@@ -48,6 +49,7 @@ public sealed class DrinkSystem : SharedDrinkSystem
[Dependency] private readonly SharedAppearanceSystem _appearance = default!;
[Dependency] private readonly SharedAudioSystem _audio = default!;
[Dependency] private readonly SharedDoAfterSystem _doAfter = default!;
+ [Dependency] private readonly SharedHandsSystem _hands = default!;
[Dependency] private readonly SharedInteractionSystem _interaction = default!;
[Dependency] private readonly SolutionContainerSystem _solutionContainer = default!;
[Dependency] private readonly StomachSystem _stomach = default!;
@@ -156,6 +158,9 @@ public void UpdateAppearance(EntityUid uid, DrinkComponent component)
_appearance.SetData(uid, FoodVisuals.Visual, drainAvailable.Float(), appearance);
}
+ ///
+ /// Tries to feed the drink item to the target entity
+ ///
private bool TryDrink(EntityUid user, EntityUid target, DrinkComponent drink, EntityUid item)
{
if (!HasComp(target))
@@ -210,9 +215,9 @@ private bool TryDrink(EntityUid user, EntityUid target, DrinkComponent drink, En
BreakOnDamage = true,
MovementThreshold = 0.01f,
DistanceThreshold = 1.0f,
- // Mice and the like can eat without hands.
- // TODO maybe set this based on some CanEatWithoutHands event or component?
- NeedHand = forceDrink,
+ // do-after will stop if item is dropped when trying to feed someone else
+ // or if the item started out in the user's own hands
+ NeedHand = forceDrink || _hands.IsHolding(user, item),
};
_doAfter.TryStartDoAfter(doAfterEventArgs);
diff --git a/Content.Server/Nutrition/EntitySystems/FoodSystem.cs b/Content.Server/Nutrition/EntitySystems/FoodSystem.cs
index 1862b4e19f1469..fc9d228b056a69 100644
--- a/Content.Server/Nutrition/EntitySystems/FoodSystem.cs
+++ b/Content.Server/Nutrition/EntitySystems/FoodSystem.cs
@@ -99,6 +99,9 @@ private void OnFeedFood(Entity entity, ref AfterInteractEvent arg
args.Handled = result.Handled;
}
+ ///
+ /// Tries to feed the food item to the target entity
+ ///
public (bool Success, bool Handled) TryFeed(EntityUid user, EntityUid target, EntityUid food, FoodComponent foodComp)
{
//Suppresses eating yourself and alive mobs
@@ -189,9 +192,9 @@ private void OnFeedFood(Entity entity, ref AfterInteractEvent arg
BreakOnDamage = true,
MovementThreshold = 0.01f,
DistanceThreshold = MaxFeedDistance,
- // Mice and the like can eat without hands.
- // TODO maybe set this based on some CanEatWithoutHands event or component?
- NeedHand = forceFeed,
+ // do-after will stop if item is dropped when trying to feed someone else
+ // or if the item started out in the user's own hands
+ NeedHand = forceFeed || _hands.IsHolding(user, food),
};
_doAfter.TryStartDoAfter(doAfterArgs);