Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Monkeys and Kobolds are too clumsy to grind #33819

Open
RadioAndrea opened this issue Dec 11, 2024 · 3 comments · May be fixed by #33844
Open

Monkeys and Kobolds are too clumsy to grind #33819

RadioAndrea opened this issue Dec 11, 2024 · 3 comments · May be fixed by #33844
Labels
A: General Interactions Area: General in-game interactions that don't relate to another area. D3: Low Difficulty: Some codebase knowledge required. Issue: Replicated This issue has been replicated by another user. P2: Raised Priority: Item has a raised priority, indicating it might get increased maintainer attention. S: Needs Content PR Merged Status: Requires an existing SS14 PR to be merged first. T: Bugfix Type: Bugs and/or bugfixes

Comments

@RadioAndrea
Copy link

Description

When attempting to grind a kobold or monkey by clicking and dragging them into the biomass reclaimer, they instead bonk their heads and cannot be grinded.

Reproduction

  1. Take a dead zombified monkey/kobold
  2. clickdrag them into the biomass reclaimer
  3. Watch as you smash their head into it
  4. Monkey/kobold unground

Screenshots
image

Additional context
Not sure if this is only zombified monkeys and kobolds or also normal dead ones too.

@github-actions github-actions bot added the S: Untriaged Status: Indicates an item has not been triaged and doesn't have appropriate labels. label Dec 11, 2024
@MilenVolf
Copy link
Contributor

Any mob with ClumsyComponent will fail to grind in biomass reclaimer. That's because of the changes with events for climbing in Clumsy system refactor made in #31147. Clumsy system uses SelfBeforeClimbEvent and after that cancels it, this leads to return in ClimbSystem in Climb method, so other events are just not being risen.
That also causes inability for a clumsy mob to be climbed on something by somebody (I also found out that in that way you can bonk a clumsy person to death 💀)

@beck-thompson
Copy link
Contributor

I knew I'd miss something with that... The old way it was doing was very suspect but I assumed they were probably doing it for a reason 😭

@beck-thompson beck-thompson added T: Bugfix Type: Bugs and/or bugfixes P2: Raised Priority: Item has a raised priority, indicating it might get increased maintainer attention. D3: Low Difficulty: Some codebase knowledge required. Issue: Replicated This issue has been replicated by another user. A: General Interactions Area: General in-game interactions that don't relate to another area. and removed S: Untriaged Status: Indicates an item has not been triaged and doesn't have appropriate labels. labels Dec 11, 2024
@RadioAndrea
Copy link
Author

Transitive property of clumsy

@ScarKy0 ScarKy0 added the S: Needs Content PR Merged Status: Requires an existing SS14 PR to be merged first. label Dec 13, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A: General Interactions Area: General in-game interactions that don't relate to another area. D3: Low Difficulty: Some codebase knowledge required. Issue: Replicated This issue has been replicated by another user. P2: Raised Priority: Item has a raised priority, indicating it might get increased maintainer attention. S: Needs Content PR Merged Status: Requires an existing SS14 PR to be merged first. T: Bugfix Type: Bugs and/or bugfixes
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants