Ajout du /profil#1341
Conversation
|
Yeeeee enfin |
|
Par contre rename ta PR correctement stp |
ar1hurgit
changed the title
iambibi
changed the title
| private static final int IDENTITY_SLOT = 13; | ||
| private static final int CITY_SLOT = 15; | ||
| private static final int PLAYTIME_SLOT = 17; | ||
| private static final int CLOSE_SLOT = 35; |
There was a problem hiding this comment.
Pas obligé de mettre les slots comme ça en static final, tu peux directement les mettre dans la map
There was a problem hiding this comment.
j'aime bien ça fait plus clair
| : TranslationManager.translation("feature.profile.status.offline") | ||
| .color(NamedTextColor.GRAY); | ||
|
|
||
| content.put(IDENTITY_SLOT, new ItemMenuBuilder(this, getHead(target), meta -> { |
There was a problem hiding this comment.
Ptet vaut mieux utiliser la méthode ItemUtils.getPlayerHead
| meta.lore(TranslationManager.translationLore( | ||
| "feature.profile.item.identity.lore", | ||
| status | ||
| ).stream().map(line -> line.color(NamedTextColor.GRAY)).toList()); |
There was a problem hiding this comment.
change directement la couleur dans les translations, et fais ça pour tout
| content.put(IDENTITY_SLOT, createIdentityItem()); | ||
| content.put(CITY_SLOT, createCityItem()); | ||
| content.put(PLAYTIME_SLOT, createPlaytimeItem()); | ||
| content.put(CLOSE_SLOT, createCloseItem()); |
There was a problem hiding this comment.
btw tu peux simplement mettre le contenu des méthodes dans la map
There was a problem hiding this comment.
en fonction de la taille des methodes ça peut être un peu plus clean comme ça
|
Ah oui et tu peux aussi respecter le template qu'on a fait pour les PRs |
| private String getTargetName() { | ||
| String name = target.getName(); | ||
| if (name != null && !name.isBlank()) { | ||
| return name; | ||
| } | ||
| return TranslationManager.translationString("feature.profile.player.unknown"); | ||
| } |
There was a problem hiding this comment.
je vois pas tellement l'utilité on a un PlayerNameCache qui fait le meme travail
| import java.util.Set; | ||
|
|
||
| @Credit(developers = {"ar1hurgit"}) | ||
| public class ProfileManager extends Feature implements HasCommands, HasListeners { |
There was a problem hiding this comment.
enleve ItemsAdder de ton serveur est dit moi si le serveur charge bien?
Co-authored-by: iambibi_ <89582596+iambibi@users.noreply.github.com>
|
je re review cette PR |
iambibi
left a comment
iambibi
left a comment
There was a problem hiding this comment.
ça me parait ok, juste enleve items adder comme plugin et envoie moi le log de ton serveur pls (via mclogs)
| import java.util.Set; | ||
|
|
||
| @Credit(developers = {"ar1hurgit"}) | ||
| public class ProfileManager extends Feature implements HasCommands, HasListeners { |
5 participants
close #303
ajout
/profile