CleverRaven/Cataclysm-DDA

Allow modifying/repairing guns on the ground and around you.

Open

#42 443 ouverte le 25 juil. 2020

Voir sur GitHub
 (5 commentaires) (5 réactions) (0 assignés)C++ (3 991 forks)batch import
(P3 - Medium)<Suggestion / Discussion>Help WantedItems: Gunmod / ToolmodQuality of Life

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

Game version:

  • Game Version: 83ce24e [64-bit] Operating system:
  • OS: Linux
    • OS Version: LSB Version: n/a; Distributor ID: ManjaroLinux; Description: Manjaro Linux; Release: 20.0.3; Codename: Lysia; Tiles or curses:
  • Graphics Version: Tiles Mods active:
  • Mods loaded: [ Dark Days Ahead [dda], No Fungal Monsters [No_Fungi], Fuji's Military Profession Pack [fuji_mpp], SpeedyDex [speedydex], Stats Through Kills [stats_through_kills], Stats Through Skills [StatsThroughSkills] ]

Expected behavior

Attach weapon mods or repair guns located on the tile you are standing on and 1+ tile away from you, just like sewing/~tailor's kit, soldering iron, and welder for their respective items.

Actual behavior

To modify/repair guns they must be either in your inventory or wielded. To modify (but not repair) clothing with a tailor's kit they must be either in your inventory or wielded.

Steps to reproduce the behavior

Activate firearm/gunsmith repair kit with batteries with sufficient charge and anything on the ground you are standing on and on the ground adjacent to you is not accessible for repair.

Guide contributeur

Direction de recherche
Cette demande propose d'étendre la portée d'interaction pour la modification/réparation d'armes afin d'inclure les objets au sol jusqu'à 1 case de distance. Pour implémenter cela, vous devriez examiner le code existant pour des interactions similaires (par exemple, les kits de couture) dans des fichiers comme `src/iuse.cpp` et `src/activity handlers.cpp`. Recherchez des fonctions comme `iuse::sew` ou `iuse::repair item` pour comprendre le modèle. Envisagez de modifier l'activité concernée pour vérifier les objets au sol au lieu de seulement l'inventaire. Aucune PR liée ou conseil de mainteneur n'est présent, vous devrez donc peut être proposer une approche dans l'issue d'abord.
Stack technique
Aucun
Domaine
backend
Type d'issue
Bug
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
2
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
1-3 heures
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.
65