CleverRaven/Cataclysm-DDA

Remove FIRE_CONTAINER from oven furniture definition

Open

#56 343 ouverte le 25 mars 2022

Voir sur GitHub
 (2 commentaires) (0 réactions) (0 assignés)C++ (3 991 forks)batch import
Fields / Furniture / Terrain / TrapsGood First Issue

Métriques du dépôt

Stars
 (9 264 stars)
Métriques de merge PR
 (Merge moyen 6j 5h) (292 PRs mergées en 30 j)

Description

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.

Guide contributeur

Direction de recherche
Localisez le fichier de définition du meuble four (probablement `f oven` dans un fichier JSON sous `data/`) et supprimez le drapeau `FIRE CONTAINER` de celui ci. Assurez vous que seuls les fours générés par la carte sont affectés, s'il existe une définition distincte pour les fours placés par le joueur.
Stack technique
cpp
Domaine
game dev
Type d'issue
Fonctionnalité
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
1
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
Moins d'une heure
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Active
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Claire
Prérequis
Git
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
85