Skip to content
This repository was archived by the owner on Oct 12, 2025. It is now read-only.

Conversation

@Nocolm
Copy link

@Nocolm Nocolm commented Oct 29, 2024

Avez vous lu le Code de Conduite?: oui

Votre code se compile t-il en local ?: oui

Avez-vous supprimez les imports inutilisés ?: oui ( normalement )

Decrivez vos changements

Clairement et avec des screenshots si nécessaires
J'ai coder un évènement pour halloween rajoutant : une dimension temporaire ou il y a un manoir / une lootbox pour halloween / des "hats" ( chapeaux ) / des bonbons / un shop hebdomadaire et un shop de bonbon accessible par une commande / ( pour les modos une commande pour désactiver l'événement au cas où) / des drops customs pour les mobs dans le manoir / une nouvelle quête hebdomadaire .

Copy link

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

⚠️ Première pull request de cette personne

@MathiasDPX
Copy link
Contributor

MathiasDPX commented Oct 29, 2024

Tu as laissé des debugs
Il y as besoin d'un pack ia ?
Pourquoi je suis collaborateur dans la cmd évents ?

Copy link
Member

@iambibi iambibi left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Attention ma review est plutot vide, je conseille qu'un autre developpeur dise ce qu'il en pense


} else {

player.sendMessage(ChatColor.RED + "Aucun évenement en cours revenez plus tard");
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Mets comme ça tes messages,

MessageManager.sendMessageType(getOwner(), "Message", Prefix.CONTEST, MessageType.SUCCESS, true);
Ton prefix est def dans fr.communaywen.core.utils.constant.Prefix

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ok je vais voir ça

private final int enchant_level;


WeeklyItemType(String itemType, String itemName, String displayName, String material, int data, int cost, String money, String moneyName,String lore, int enchant_level){
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Essaie d'avoir un ItemStack + cost + money + moneyName + namespace item adder

Copy link
Author

@Nocolm Nocolm Oct 29, 2024

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

heu pour les livres enchantés pas sûr que ça va marcher


}

private void GiveItems(Player player, ItemStack item){
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

GiveItems => giveItems

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ok.

@iambibi
Copy link
Member

iambibi commented Oct 29, 2024

Tu as laissé des debugs Il y as besoin d'un pack ia ? Pourquoi je suis collaborateur dans la cmd évents ?

  • Il y a un menu custom
  • Il a repris le systeme de la dimension des reves


String savedWeeklyItem = getConfig().getString("weekly-item", WeeklyItemType.LUCKY_BLOCK.name());
WeeklyItemType weeklyItemType = WeeklyItemType.valueOf(savedWeeklyItem);
Bukkit.getConsoleSender().sendMessage("Le WeeklyItemType a été charger : " + getConfig().getString("weekly-item"));
Copy link
Member

@iambibi iambibi Oct 29, 2024

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Euh tu as rien ajouter dans le config.yml?

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Si ....

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Screenshot_20241029_121932_Chrome

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Non.

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

mais normalement si ?
image

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Montre moi que j'ai tord.

Copy link
Author

@Nocolm Nocolm Oct 29, 2024

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

mais je viens de te montrer que c'est sauvegarder sur le config.yml du plugin je peux rien faire d'autre pour te le montrer.

edit : la sauvegarde s'ajoute dans le config.yml au premier lancement de cleui_ci

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Si tu le dis on verra sur le serveur ptdr

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

image
tu vois ça devrais etre marqué dans les fichiers modifié

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ba je sais pas parce que sur mon ide sa apparait pas mais quand j'ouvre le config.yml seul il est présent

@Nocolm
Copy link
Author

Nocolm commented Oct 29, 2024

Tu as laissé des debugs Il y as besoin d'un pack ia ? Pourquoi je suis collaborateur dans la cmd évents ?

Mince je pensais les avoir tous retirer

@Nocolm
Copy link
Author

Nocolm commented Oct 30, 2024

j'allais oublier pour la dim faut remplacer le fichier créer par celui-ci sinon le manoir ba il existe pas
Mansion.zip

@Nocolm
Copy link
Author

Nocolm commented Nov 2, 2024

ducoup est-ce que c'est un code propre pour être ajouter au serveur ou pas ?


this.RewardType = 1;
Items = Material.DIAMOND;
Items = Material.EMERALD;
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

On ne met pas de majuscule au début des noms de variables.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Items => items

@EventHandler
public void onEntityKill(EntityDeathEvent event) {

EnumSet<EntityType> MansionEntities = EnumSet.of(EntityType.ZOMBIE_VILLAGER, EntityType.SKELETON, EntityType.SPIDER, EntityType.CAVE_SPIDER, EntityType.WITHER_SKELETON, EntityType.VINDICATOR, EntityType.WITCH);
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

MansionEntities => mansionEntities

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

normalement c'est corrigé

@sonarqubecloud
Copy link

Quality Gate Failed Quality Gate failed

Failed conditions
7 Security Hotspots
22.4% Duplication on New Code (required ≤ 3%)
D Reliability Rating on New Code (required ≥ A)

See analysis details on SonarQube Cloud

Catch issues before they fail your Quality Gate with our IDE extension SonarQube for IDE

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants