cmss13-devs / cmss13-devs/cmss13

weard checks in flame nozzle

Open
#5,593 0 comments 0 reactions 0 assignees View on GitHub
Bug
Dominant language
DM
Stars
170
Forks
940
Avg merge
5d 16h
Merged PRs (30d)
61

Description

### 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

- [X] Issue could be reproduced at least once
- [X] Issue happened in a recent (less than 7 days ago) round
- [X] Couldn't find an existing issue about this (https://github.com/cmss13-devs/cmss13/issues)

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.