CleverRaven/Cataclysm-DDA

Remove FIRE_CONTAINER from oven furniture definition

Open

#56.343 aperta il 25 mar 2022

Vedi su GitHub
 (2 commenti) (0 reazioni) (0 assegnatari)C++ (3991 fork)batch import
Fields / Furniture / Terrain / TrapsGood First Issue

Metriche repository

Star
 (9264 star)
Metriche merge PR
 (Merge medio 6g 5h) (292 PR mergiate in 30 g)

Descrizione

Is your feature request related to a problem? Please describe.

Electric ovens IRL aren’t all that great at containing an actual fire, as they’re heat insulated with flame retardant insulation, but they’re not fireproof. Setting logs on fire in an oven would not only damage the electric heating elements and render it unusable, but it would also risk burning through the thin sheet metal bottom of the oven.

Solution you would like.

Removal of the FIRE_CONTAINER flag from f_oven. It only appears that the mapgen placed ovens are this way - the user placed oven appliance is not a fire container.

Describe alternatives you have considered.

No response

Additional context

This is an easy first issue. I can throw up a PR this weekend if nobody gets to it.

Guida contributor

Direzione di ricerca
Trova il file di definizione dell'arredamento del forno (probabilmente `f oven` in un file JSON sotto `data/`) e rimuovi il flag `FIRE CONTAINER` da esso. Assicurati che solo i forni generati dalla mappa siano influenzati, se esiste una definizione separata per i forni posizionati dal giocatore.
Tech stack
cpp
Dominio
game dev
Tipo issue
Funzionalità
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
1
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
Meno di 1 ora
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Attiva
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
Git
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
85