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

Pacification Prevents Throwing Weapons #1433

Merged

Conversation

angelofallars
Copy link
Contributor

@angelofallars angelofallars commented Jan 5, 2025

Description

Prevents entities who are Pacified (through the Pacifist trait or being a thief) from throwing items that deal damage on hit or embed.

Technical Details

Two components will prevent throwing if they deal any damage: DamageOtherOnHitComponent and EmbedPassiveComponent.

The pacifist check on EmbeddableProjectileComponent has been removed, because just because an item is embeddable does not mean they deal damage.

Media

Throw attempt with the Pacifist trait

image

Changelog

🆑 Skubman

  • fix: Pacified characters (Pacifist trait and thieves) can no longer throw items that deal throwing damage.

@github-actions github-actions bot added Changes: C# Changes any cs files Changes: YML Changes any yml files labels Jan 5, 2025
@github-actions github-actions bot added the Status: Needs Review Someone please review this label Jan 5, 2025
@github-actions github-actions bot added the Status: Merge Conflict FIX YOUR PR AAAGH label Jan 5, 2025
Copy link
Contributor

github-actions bot commented Jan 5, 2025

This pull request has conflicts, please resolve those before we can evaluate the pull request.

@github-actions github-actions bot removed the Status: Merge Conflict FIX YOUR PR AAAGH label Jan 5, 2025
@VMSolidus VMSolidus merged commit 71e3c03 into Simple-Station:master Jan 5, 2025
9 of 10 checks passed
SimpleStation14 added a commit that referenced this pull request Jan 5, 2025
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: YML Changes any yml files Status: Needs Review Someone please review this
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants