Registry-related get methods
Personne n'a encore pris cette issue.
Évaluation
- Difficulté
- 4/5
- Temps estimé
- 3-5 jours
- Accessibilité débutants
- 38/100
Piste de recherche
Commencez par localiser les méthodes de Registry, HolderGetter, HolderLookup, RegistryAccess et BootstapContext listées dans l’issue, puis comparez l’approche utilisée pour #10. Suivez les sites d’appel où les arguments de type ResourceKey ou TagKey proviennent de champs statiques et identifiez les variables locales concernées. Le travail est terminé lorsque les variables concernées utilisent systématiquement des noms dérivés de ces champs dans les méthodes listées.
Rédigé par le modèle d'indexation à partir du texte de l'issue.
Description
A lot of the methods on Registry, HolderGetter, HolderLookup, RegistryAccess take ResourceKey as the sole parameter returning something. A lot of the time, those ResourceKey arguments are obtained from static final fields somewhere which makes choosing the name for the local var very easy.
Something like
Holder<Structure> structure1 = registry.getHolder(BuiltinStructures.NETHER_FOSSIL);
can be changed to
Holder<Structure> netherForsilStructure = registry.getHolder(BuiltinStructures.NETHER_FOSSIL);
This is similar to #10 and how the various is and has methods are handled, by looking at the field used in the parameter.
A non-exhaustive list of methods that this could apply to are:
TagKey
TagKey#create(ResourceKey, ResourceLocation)
ResourceKey
ResourceKey#create(ResourceKey, ResourceLocation)
Registry
Registry#get(ResourceKey)Registry#getHolder(ResourceKey)Registry#getOptional(ResourceKey)Registry#getOrThrow(ResourceKey)Registry#containsKey(ResourceKey)Registry#getHolder(ResourceKey)Registry#getTag(TagKey)Registry#getTagOrEmpty(TagKey)Registry#getOrCreateTag(TagKey)
RegistryAccess, RegistryAccess$Frozen, HolderLookup$Provider
RegistryAccess#lookup(ResourceKey)RegistryAccess#lookupOrThrow(ResourceKey)RegistryAccess#registry(ResourceKey)RegistryAccess#registryOrThrow(ResourceKey)
HolderGetter, HolderLookup
HolderGetter#get(ResourceKey)HolderGetter#getOrThrow(ResourceKey)HolderGetter#get(TagKey)HolderGetter#getOrThrow(TagKey)
BootstapContext
(yes there is a typo in Bootstap in the vanilla source)
Bootstap#register(ResourceKey, Object, Lifecycle)Bootstap#register(ResourceKey, Object)Bootstap#lookup(ResourceKey)
- Langage dominant
- Java
- Étoiles
- 24
- Forks
- 1
- Merge moyen
- 2 min
- PR mergées (30 j)
- 1
Guide de contribution
Aucun guide de contribution indexé pour ce dépôt
Par où commencer
- Lisez l'issue en entier, puis le guide de contribution du projet.
- Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
- Forkez le dépôt et travaillez sur une branche.
- Ouvrez une pull request qui référence le numéro de l'issue.
Autres issues de PaperMC/codebook
-
Dependency Dashboard Ouverte
Difficulté 4/5 3-5 jours Accessibilité débutants 20/100
-
Difficulté 4/5 3-5 jours Accessibilité débutants 35/100
-
lvt names for catch clauses Ouverte
Difficulté 3/5 1-2 jours Accessibilité débutants 25/100
-
enhancement
Difficulté 5/5 Plus d'une semaine Accessibilité débutants 30/100
-
Output various reports Ouverteenhancement
Difficulté 4/5 3-5 jours Accessibilité débutants 35/100
Toutes les issues de PaperMC/codebook
Issues similaires
-
executions.Query — startDate and timeRange filters are sent with inverted comparison operators Ouvertearea/plugin
Difficulté 2/5 1-3 heures Accessibilité débutants 75/100
kestra-io/plugin-kestra#190 ·
-
litertlm-android AAR ships no consumer ProGuard rules → "mid == null" SIGABRT in minified apps Ouverte
Difficulté 2/5 1-3 heures Accessibilité débutants 70/100
google-ai-edge/LiteRT-LM#3739 ·
-
bug
Difficulté 2/5 1-3 heures Accessibilité débutants 75/100
-
Add canonical URLs and a sitemap Ouverte
Difficulté 2/5 1-3 heures Accessibilité débutants 75/100
integra-team-red/meet-map#249 ·
-
[Studio][Bug] Cancelled create-user dialog keeps the password and admin switch for the next attempt Ouverte
Difficulté 2/5 1-3 heures Accessibilité débutants 75/100
apache/rocketmq-dashboard#5064 ·