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

Fix Weed Nodes in Walls #3765

Merged
merged 5 commits into from
Jul 5, 2023
Merged

Conversation

Drulikar
Copy link
Contributor

@Drulikar Drulikar commented Jun 29, 2023

About the pull request

This PR makes it so weed nodes cannot be placed in a turf with density set true (such as when you build a resin wall on top of yourself) or a dense structure (such as a window frame). Nodes still can be placed under doors while they are open. Marked it as balance but feel free to change it to fix if you think it isn't balance.

If theres a situation where a weed node should be able to be placed in a turf with density let me know and I can make the check exclusively for resin walls. I can't think of any though.

Explain why it's good for the game

Fixes #3745
Fixes #3770

Testing Photographs and Procedure

Screenshots & Videos

image

Changelog

🆑 Drathek
balance: Weed nodes can no longer be placed in walls or window frames (or any turf or structure with density)
refactor: Refactored the plant weeds ability code
/:cl:

# Conflicts:
#	code/modules/mob/living/carbon/xenomorph/abilities/general_powers.dm
@github-actions github-actions bot added Balance You need to be a professional veteran game maintainer to comprehend what is being done here. Refactor Make the code harder to read labels Jun 29, 2023
@morrowwolf morrowwolf added this pull request to the merge queue Jul 5, 2023
Merged via the queue into cmss13-devs:master with commit d4bec12 Jul 5, 2023
@Drulikar Drulikar deleted the Fix_Wall_Weed_Nodes branch July 5, 2023 06:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Balance You need to be a professional veteran game maintainer to comprehend what is being done here. Refactor Make the code harder to read
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Weed nodes can be placed under window frames Weed nodes can be placed under walls
3 participants