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

Egg orphans and fragile egg fixes #7554

Merged
merged 7 commits into from
Nov 12, 2024
Merged

Conversation

Drulikar
Copy link
Contributor

@Drulikar Drulikar commented Nov 10, 2024

About the pull request

This PR fixes a couple bugs with xeno eggs and also adds a mechanic where eggs that have lost their weeds will become fragile. These eggs use the CARRIER_EGG_MAXIMUM_LIFE value (5 mins) but will convert back to normal eggs if their hive weeds are restored. If this doesn't discourage the weird behavior of placing and destroying pylons, we can just entirely remove the logic that restores the eggs to normal eggs.

For fixes, there was a check on egg/click that prevented egg interaction if you weren't close or are a ghost, but it needed to first call parent first for examine logic. Then there was an issue where the timer for egg carrier eggs would get deleted if called the Burst proc (but it failed to check if the burst was from just someone retrieving the hugger)

Forsaken variants of the new eggs will last indefinitely.

Explain why it's good for the game

Right now its possible to make eggs even on semi-weedable surfaces. This PR still allows this but they will only last 5 minutes if they don't have their weeds restored in time. In general I am okay with xenos creating traps for marines to get hugged, but doing so through weird mechanics like cycling a pylon construction needs to be discouraged.

Testing Photographs and Procedure

Screenshots & Videos

https://youtu.be/Hbyk6OikyjU

Changelog

🆑 Drathek
balance: Eggs that lose their hive weeds will now turn fragile (like egg carrier eggs) lasting 5 minutes.
fix: Fixed an exploit allowing egg carrier eggs to last indefinitely.
fix: Fixed the inability to examine planted eggs from further than one tile.
/:cl:

@Drulikar Drulikar added Testmerge Candidate we'll test this while you're asleep and the server has 10 players Balance Approved This PR has had its balance and gameplay-affecting aspects approved. Cry to the Head-maint about it. labels Nov 10, 2024
@cmss13-ci cmss13-ci bot added Balance You need to be a professional veteran game maintainer to comprehend what is being done here. Fix Fix one bug, make ten more labels Nov 10, 2024
@harryob harryob added this pull request to the merge queue Nov 12, 2024
Merged via the queue into cmss13-devs:master with commit 3b2a9c0 Nov 12, 2024
28 checks passed
cmss13-ci bot added a commit that referenced this pull request Nov 12, 2024
@Drulikar Drulikar deleted the Egg_Orphans branch November 12, 2024 08:55
@Doom-external
Copy link

This is only a semi fix for #7405, as they can still expand weeds on tiles they shouldn't normally be able to.

@Drulikar
Copy link
Contributor Author

This is only a semi fix for #7405, as they can still expand weeds on tiles they shouldn't normally be able to.

Right now its possible to make eggs even on semi-weedable surfaces. This PR still allows this but they will only last 5 minutes if they don't have their weeds restored in time. In general I am okay with xenos creating traps for marines to get hugged, but doing so through weird mechanics like cycling a pylon construction needs to be discouraged.

github-merge-queue bot pushed a commit that referenced this pull request Nov 21, 2024
# About the pull request

This PR is a follow up to #7554 bumping the timer from 6 mins (from 5
mins) since the hive core reconstruction time (`HIVECORE_COOLDOWN`) is 5
mins (to allow 1 minute window to restore a hive w/o all eggs popping).

# Explain why it's good for the game

Should xenos manage to hold their original hive location to rebuild a
hive core, this should make it possible to recover most eggs planted
there.

# Changelog
:cl: Drathek
balance: Increased orphan egg time from 5 mins to 6 mins.
/:cl:
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Balance Approved This PR has had its balance and gameplay-affecting aspects approved. Cry to the Head-maint about it. Balance You need to be a professional veteran game maintainer to comprehend what is being done here. Fix Fix one bug, make ten more Testmerge Candidate we'll test this while you're asleep and the server has 10 players
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Egg Planting Exploit substained egg limit bypass
3 participants