GlowstoneMC/Glowstone

Named sound id are wrong in some cases

Open

#519 ouverte le 5 juil. 2017

Voir sur GitHub
 (2 commentaires) (0 réactions) (0 assignés)Java (278 forks)batch import
Category: server featurePriority: minorType: inconsistencyhelp wanted

Métriques du dépôt

Stars
 (1 825 stars)
Métriques de merge PR
 (Aucune PR mergée en 30 j)

Description

Hello,

I wanted to implement equipment sounds for armor stand (see #471). "Sound.ITEM_ARMOR_EQUIP_LEATHER" maps to id "minecraft:item.armor.equip.leather". According to http://pokechu22.github.io/Burger/1.11.html#sounds however, it should be "minecraft:item.armor.equip_leather".

The same is the case for multiple, if not all, of the armor_equip sounds.

regards Postremus

Want to back this issue? Post a bounty on it! We accept bounties via Bountysource.

Guide contributeur

Direction de recherche
Étudiez le mappage des ID de sons dans le code source de Glowstone, probablement dans une classe comme Sound.java. Comparez le mappage actuel avec les ID corrects de la ressource Burger (http://pokechu22.github.io/Burger/1.11.html#sounds). Mettez à jour les chaînes de mappage pour utiliser des tirets bas au lieu de points là où ils sont incorrects, par exemple, changez 'item.armor.equip.leather' en 'item.armor.equip leather'. Vérifiez par rapport à tous les sons d'équipement d'armure listés dans l'issue et assurez la cohérence avec les identifiants de sons réels du jeu.
Stack technique
java
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
Java
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
75