Skip to content

Normal Drinker Vulp Trait and Speedup Drinking as MessyDrinker - #6431

Open
Coryler wants to merge 1 commit into
DeltaV-Station:masterfrom
Coryler:vulp-drink-traits-ops
Open

Normal Drinker Vulp Trait and Speedup Drinking as MessyDrinker#6431
Coryler wants to merge 1 commit into
DeltaV-Station:masterfrom
Coryler:vulp-drink-traits-ops

Conversation

@Coryler

@Coryler Coryler commented Aug 20, 2026

Copy link
Copy Markdown
Contributor

About the PR

Made Messy Drinker faster drinker and added a Trait which makes Vulps normal drinker again

Why / Balance

Makes Messy Drinker a bit more senseful as why you're messy. Also allows people who don't want to be that... to be don't.

Technical details

  • C# changes, addition by adding references to Messydrinker
  • ftl changes for traits name desc
  • yml for trait

Media

Screencast_20260820_102808.webm

Requirements

  • I have tested all added content and changes.
  • I have added media to this PR or it does not require an in-game showcase.

Licensing

Changelog

🆑

  • add: Normal Drinker Traits for Vulps
  • tweak: Messy Drinker speed for drinking drinks

@Coryler
Coryler requested a review from a team as a code owner August 20, 2026 08:33
@github-actions github-actions Bot added size/S 16-63 lines S: Needs Review Awaiting review from a Maintainer Changes: YML Changes any yml files Changes: C# Changes any cs files Changes: Localization Changes any ftl files labels Aug 20, 2026
@delta-v-bot delta-v-bot Bot added the Under Direction Review This pull request is under consideration from the Direction team. label Aug 20, 2026
@delta-v-bot

delta-v-bot Bot commented Aug 20, 2026

Copy link
Copy Markdown

Triaged by siphynith:
This PR requires a content review discussion, which will be held in public.

microbalancing pr world championship

You can view the discussion here and write comments starting with !discord to send messages into the thread.
The review duration has been set to 1 days.

@delta-v-bot delta-v-bot Bot added S: Content Approved This PR has been approved by a member of Content Review. and removed Under Direction Review This pull request is under consideration from the Direction team. labels Aug 21, 2026
@delta-v-bot

delta-v-bot Bot commented Aug 21, 2026

Copy link
Copy Markdown

CR consensus: Approved

it’s not that deep

Review closed by siphynith.

@SirWarock

Copy link
Copy Markdown
Contributor

As detailed in the Content-Review channel, the trait "Normal drinker" is subject to removal once the vulpkanine jump ability has been added.

This comment serves as a warning, and will be used in case complaints such as "But I've grown accustomed to the trait!" or "the trait is an inherent part of my character!" are made.

/// Drinking Speed Delay Reduction, which speeds up the drinking speed of Vulpkanin
/// </summary>
[DataField, AutoNetworkedField]
public FixedPoint2 DrinkDelayReduction = 0.5;

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Reduction implies that you're subtracting from the time, not using as a multiplier. Probably can be a float too.

Suggested change
public FixedPoint2 DrinkDelayReduction = 0.5;
public float DrinkSpeedMultiplier = 0.5f;

trait-hemophilia-desc = Your body fails to make blood clots.

trait-normal-drinker-name = Normal Drinker
trait-normal-drinker-desc = You're not like your kin. You drink sophisticated.

@ShepardToTheStars ShepardToTheStars Aug 23, 2026

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
trait-normal-drinker-desc = You're not like your kin. You drink sophisticated.
trait-normal-drinker-desc = You're not like your kin. You drink more sophisticated, but you do not drink nearly as fast as other vulpkanins.

Comment on lines +94 to +99
// Delta V - Begin Messy Drinker Speed
if (TryComp<MessyDrinkerComponent>(args.User, out var comp) && entity.Comp.Solution.ToLower() == MessyDrinkerSolution.ToLower())
{
args.Time *= comp.DrinkDelayReduction.Double();
}
// Delta V - End

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

  • I'm not a huge fan of putting this in IngestionSystem.Blockers.cs.
  • This doesn't check if the person drinking is the person who has MessyDrinker (aka it doesn't check are you drinking yourself or is someone feeding you).
  • Not a fan of hard-coding MessyDrinkerSolution

It may be better to put this in IngestionSystem.cs, right before the DoAfter and right after CanConsume, like so:

        // Check if despite being able to digest the item something is blocking us from eating.
        if (!CanConsume(args.User, entity, args.Ingested, out var solution, out var time))
            return;

        // BEGIN DeltaV - Messy Drinkers drink faster
        if (args.User == entity.Owner && TryComp<MessyDrinkerComponent>(entity, out var messyDrinker))
        {
            var proto = GetEdibleType(food);
            if (proto is { } edibleType && messyDrinker.SpillableTypes.Contains(edibleType))
                time *= messyDrinker.DrinkSpeedMultiplier;
        }
        // END DeltaV

        if (!_doAfter.TryStartDoAfter(GetEdibleDoAfterArgs(args.User, entity, food, time ?? TimeSpan.Zero)))
            return;

@ShepardToTheStars ShepardToTheStars added the S: Awaiting Changes Do not merge due to requested changes label Aug 23, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Changes: C# Changes any cs files Changes: Localization Changes any ftl files Changes: YML Changes any yml files S: Awaiting Changes Do not merge due to requested changes S: Content Approved This PR has been approved by a member of Content Review. S: Needs Review Awaiting review from a Maintainer size/S 16-63 lines

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants