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

weard checks in flame nozzle #5593

Open
3 tasks done
cuberound opened this issue Jan 29, 2024 · 0 comments
Open
3 tasks done

weard checks in flame nozzle #5593

cuberound opened this issue Jan 29, 2024 · 0 comments
Labels
Bug Admin events aren't bugs

Comments

@cuberound
Copy link
Contributor

Testmerges

#5086 #5392 #5452 #5526 #5506 #5519 #5007

Round ID

No response

Description of the bug

if(gun.flags_gun_features & GUN_TRIGGER_SAFETY)
    to_chat(user, SPAN_WARNING("\The [gun] isn't lit!"))
    return

if(!istype(gun.current_mag, /obj/item/ammo_magazine/flamer_tank))
    to_chat(user, SPAN_WARNING("\The [gun] needs a flamer tank installed!"))
    return 

in /obj/item/attachable/attached_gun/flamer_nozzle/fire_attachment are never reached, the safty gives standard safty warning and the tank does not show any message at all

What's the difference with what should have happened?

recive warning about missing tank at least

How do we reproduce this bug?

1.take flamethrower
2.attach nozzle and select it
3.try firing without tank installed = no message
...

Issue Bingo

@cuberound cuberound added the Bug Admin events aren't bugs label Jan 29, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug Admin events aren't bugs
Projects
None yet
Development

No branches or pull requests

1 participant