-
Notifications
You must be signed in to change notification settings - Fork 4.8k
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鈥檒l occasionally send you account related emails.
Already on GitHub? Sign in to your account
moon smile effect durations fix #81794
Conversation
Not that this needs this but Yeah this was the intent of the original pr |
@@ -28,7 +28,8 @@ | |||
/datum/action/cooldown/spell/pointed/moon_smile/cast(mob/living/carbon/human/cast_on) | |||
. = ..() | |||
/// The duration of these effects are based on sanity, mainly for flavor but also to make it a weaker alpha strike | |||
var/moon_smile_duration = (150 - cast_on.mob_mood.sanity) / 10 | |||
var/maximum_duration = 15 SECONDS | |||
var/moon_smile_duration = ((SANITY_MAXIMUM - cast_on.mob_mood.sanity) / (SANITY_MAXIMUM - SANITY_INSANE)) * maximum_duration |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
var/moon_smile_duration = ((SANITY_MAXIMUM - cast_on.mob_mood.sanity) / (SANITY_MAXIMUM - SANITY_INSANE)) * maximum_duration | |
var/moon_smile_duration = ((SANITY_MAXIMUM - cast_on.mob_mood.sanity) / SANITY_MAXIMUM) * maximum_duration |
the SANITY_INSANE value is 0 anyway.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I know, but just in case it ever changes, I use the define so that the scaling stays accurate.
About The Pull Request
Moon smile's durations had some syntax issues, causing their durations to be much lower than intended, and sanity having very low impact on it. Also adds a use of SANITY_MAXIMUM define, so that the calculation will still work if this value is ever changed.
Why It's Good For The Game
The moon did not smile. It was frowning. Frowning because it was bugged :( But with this fix it can smile again :)
Changelog
馃啈
fix: fixed some issues when calculating the duration of moon smile's effects
/:cl: