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

Fixed inactive incendiary grenades not exploding if thrown in fire #34769

Merged
merged 2 commits into from
Oct 15, 2019
Merged

Fixed inactive incendiary grenades not exploding if thrown in fire #34769

merged 2 commits into from
Oct 15, 2019

Conversation

Night-Pryanik
Copy link
Contributor

Summary

SUMMARY: Bugfixes "Fixed inactive incendiary grenades not exploding if thrown in fire."

Purpose of change

Closes #32276 - Incendiary grenade activates in fire but doesn't explode.

Describe the solution

When thrown in fire, transformable items with ACT_IN_FIRE and BOMB flags will transform to their transform-targets and then become active.

Describe alternatives you've considered

Remove ACT_IN_FIRE flag from both types of incendiary grenades and add a explode_in_fire: true field to them. But that didn't worked out.

Testing

Spawned a fire. Spawned an inactive incendiary grenade. Threw grenade into a fire. Boom!

Additional context

None.

@Night-Pryanik Night-Pryanik added Items / Item Actions / Item Qualities Items and how they work and interact <Bugfix> This is a fix for a bug (or closes open issue) Fields / Furniture / Terrain / Traps Objects that are part of the map or its features. labels Oct 15, 2019
@ZhilkinSerg ZhilkinSerg merged commit 1dac8a6 into CleverRaven:master Oct 15, 2019
@Night-Pryanik Night-Pryanik deleted the inactive-incendiary-grenades-will-explode-when-thrown-in-fire branch October 15, 2019 09:50
Ramza13 pushed a commit to Ramza13/Cataclysm-DDA that referenced this pull request Oct 22, 2019
…leverRaven#34769)

* Transformable items with ACT_IN_FIRE and BOMB flags will transform into their "_act" variants and then become active if thrown in fire

* Get rid of hardcoded name and use target item id
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
<Bugfix> This is a fix for a bug (or closes open issue) Fields / Furniture / Terrain / Traps Objects that are part of the map or its features. Items / Item Actions / Item Qualities Items and how they work and interact
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Incendiary grenade activates in fire but doesn't explode
2 participants