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

Pug Thralls cannot emote #16646

Closed
goonstation-issuebot opened this issue Oct 26, 2023 · 1 comment · Fixed by #16809
Closed

Pug Thralls cannot emote #16646

goonstation-issuebot opened this issue Oct 26, 2023 · 1 comment · Fixed by #16809
Assignees
Labels
C-Bug A bug that impacts usage of a feature P-Minor A bug that does not impact usage of a feature. These are often visual issues

Comments

@goonstation-issuebot
Copy link

Labels

[BUG][MINOR]

Description

Thralled pugs cannot emote through keybinds or typing it manually

Steps to reproduce

  1. Be pug
  2. Get thralled
  3. Attempt emoting

Expected Behavior

not this

Additional Information & Screenshots

Reported by: Bomb_Sirens
On server: Goonstation 1 Classic: Heisenbee
Active test merges: #16477 and #16449
Round log date: 2023-10-26-10-49
Reported on: 2023-10-26 12:08:19
Map: COGMAP

@keywordlabeler keywordlabeler bot added C-Bug A bug that impacts usage of a feature P-Minor A bug that does not impact usage of a feature. These are often visual issues labels Oct 26, 2023
@gear-space gear-space self-assigned this Nov 6, 2023
@gear-space
Copy link
Contributor

Related to #16561
Same root cause - pugs have a special emote trigger on oxy damage that is triggering when mob is dead

if (ispug(src))
var/mob/living/carbon/human/H = src
amount *= 2
H.emote(pick("wheeze", "cough", "sputter"))
src.oxyloss = max(0,src.oxyloss + amount)
return

Guessing that emote_allowed gets set to false/0 as part of doing the oxy damage emote, never gets set back to true/1 since the source is now a new entity after being thralled?

if(voluntary && !src.emote_allowed)
return

Also explains why they do a little gasp when revived as a thrall.

###Callstack when thrall comes back
/mob/living/carbon/take_oxygen_deprivation (goonstation\code\mob\living\carbon.dm:185)
/mob/living/carbon/full_heal (goonstation\code\mob\living\carbon.dm:131)
/mob/living/carbon/human/full_heal (goonstation\code\mob\living\carbon\human.dm:2349)
/datum/abilityHolder/vampire/make_thrall (goonstation\code\modules\antagonists\vampire\abilities\_vampire_ability_holder.dm:307)
/datum/action/bar/private/icon/vampire_enthrall_thrall/onEnd (goonstation\code\modules\antagonists\vampire\abilities\enthrall.dm:128)
/datum/action_controller/process (goonstation\code\datums\controllers\action_controls.dm:87)
/datum/controller/process/actions/doWork (goonstation\code\datums\controllers\process\actions.dm:17)
/datum/controller/process/process (goonstation\code\datums\controllers\process\_process.dm:165)
/datum/controller/processScheduler/runProcess (goonstation\code\datums\controllers\process_scheduler.dm:215)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
C-Bug A bug that impacts usage of a feature P-Minor A bug that does not impact usage of a feature. These are often visual issues
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants