Skip to content

Commit

Permalink
Merge branch '1.19.4_prep'
Browse files Browse the repository at this point in the history
# Conflicts:
#	src/main/java/adhdmc/villagerinfo/Config/Defaults.java
#	src/main/java/adhdmc/villagerinfo/Config/Message.java
  • Loading branch information
RhythmicSys committed Mar 23, 2023
2 parents 0c408d7 + d84089a commit 80945a9
Show file tree
Hide file tree
Showing 16 changed files with 211 additions and 334 deletions.
2 changes: 1 addition & 1 deletion pom.xml
Original file line number Diff line number Diff line change
Expand Up @@ -6,7 +6,7 @@

<groupId>ADHDMC</groupId>
<artifactId>VillagerInfo</artifactId>
<version>2.5</version>
<version>2.6</version>
<packaging>jar</packaging>

<name>VillagerInfo</name>
Expand Down
19 changes: 12 additions & 7 deletions src/main/java/adhdmc/villagerinfo/Commands/CommandHandler.java
Original file line number Diff line number Diff line change
@@ -1,14 +1,14 @@
package adhdmc.villagerinfo.Commands;

import adhdmc.villagerinfo.Config.ConfigValidator;
import adhdmc.villagerinfo.Config.Message;
import adhdmc.villagerinfo.Config.VIMessage;
import adhdmc.villagerinfo.VillagerInfo;
import net.kyori.adventure.text.minimessage.MiniMessage;
import net.kyori.adventure.text.minimessage.tag.resolver.Placeholder;
import org.bukkit.command.Command;
import org.bukkit.command.CommandExecutor;
import org.bukkit.command.CommandSender;
import org.bukkit.command.TabExecutor;
import org.jetbrains.annotations.NotNull;

import java.util.*;

Expand All @@ -19,7 +19,7 @@ public class CommandHandler implements CommandExecutor, TabExecutor {

//TY Peashooter101
@Override
public List<String> onTabComplete(CommandSender sender, Command command, String label, String[] args) {
public List<String> onTabComplete(@NotNull CommandSender sender, @NotNull Command command, @NotNull String label, String[] args) {
ArrayList<String> subCommands1 = new ArrayList<>(Arrays.asList("help", "toggle", "reload"));
if (args.length == 1) {
return subCommands1;
Expand All @@ -29,10 +29,15 @@ public List<String> onTabComplete(CommandSender sender, Command command, String


@Override
public boolean onCommand(CommandSender sender, Command cmd, String label, String[] args) {
public boolean onCommand(@NotNull CommandSender sender, @NotNull Command cmd, @NotNull String label, String[] args) {
//Checking for arguments
if (args.length == 0) {
String url = VillagerInfo.getInstance().getDescription().getWebsite();
String url;
if (VillagerInfo.getInstance().getDescription().getWebsite() != null) {
url = VillagerInfo.getInstance().getDescription().getWebsite();
} else {
url = "https://github.com/ADHDMC/VillagerInfo";
}
String version = VillagerInfo.getInstance().getDescription().getVersion();
List<String> authors = new ArrayList<>();
for (String authorName : VillagerInfo.getInstance().getDescription().getAuthors()) {
Expand All @@ -48,12 +53,12 @@ public boolean onCommand(CommandSender sender, Command cmd, String label, String
);
return true;
}
//if has an argument, check to see if it's contained in the list of arguments
//if there is an argument, check to see if it's contained in the list of arguments
String command = args[0].toLowerCase();
if (subcommandList.containsKey(command)) {
subcommandList.get(command).execute(sender, Arrays.copyOfRange(args, 1, args.length));
} else {
sender.sendMessage(miniMessage.deserialize(Message.NO_COMMAND.getMessage()));
sender.sendMessage(miniMessage.deserialize(VIMessage.NO_COMMAND.getMessage()));
}
return true;
}
Expand Down
Original file line number Diff line number Diff line change
@@ -1,7 +1,7 @@
package adhdmc.villagerinfo.Commands.SubCommands;

import adhdmc.villagerinfo.Commands.SubCommand;
import adhdmc.villagerinfo.Config.Message;
import adhdmc.villagerinfo.Config.VIMessage;
import adhdmc.villagerinfo.Config.Perms;
import adhdmc.villagerinfo.VillagerInfo;
import net.kyori.adventure.text.minimessage.MiniMessage;
Expand All @@ -19,14 +19,14 @@ public HelpCommand() {
public void execute(CommandSender sender, String[] args) {
MiniMessage miniMessage = VillagerInfo.getMiniMessage();

if (sender.hasPermission(Perms.USE.getPerm())) {
sender.sendMessage(miniMessage.deserialize(Message.PREFIX.getMessage()));
sender.sendMessage(miniMessage.deserialize(Message.HELP_MAIN.getMessage()));
sender.sendMessage(miniMessage.deserialize(Message.HELP_TOGGLE.getMessage()));
sender.sendMessage(miniMessage.deserialize(Message.HELP_RELOAD.getMessage()));
if (sender.hasPermission(Perms.USE.getVIPerm())) {
sender.sendMessage(miniMessage.deserialize(VIMessage.PLUGIN_PREFIX.getMessage()));
sender.sendMessage(miniMessage.deserialize(VIMessage.HELP_MAIN.getMessage()));
sender.sendMessage(miniMessage.deserialize(VIMessage.HELP_TOGGLE.getMessage()));
sender.sendMessage(miniMessage.deserialize(VIMessage.HELP_RELOAD.getMessage()));
return;
}
sender.sendMessage(miniMessage.deserialize(Message.NO_PERMISSION.getMessage()));
sender.sendMessage(miniMessage.deserialize(VIMessage.NO_PERMISSION.getMessage()));
}

@Override
Expand Down

This file was deleted.

Original file line number Diff line number Diff line change
Expand Up @@ -2,7 +2,8 @@

import adhdmc.villagerinfo.Commands.SubCommand;
import adhdmc.villagerinfo.Config.ConfigValidator;
import adhdmc.villagerinfo.Config.Message;
import adhdmc.villagerinfo.Config.LocaleConfig;
import adhdmc.villagerinfo.Config.VIMessage;
import adhdmc.villagerinfo.Config.Perms;
import adhdmc.villagerinfo.VillagerInfo;
import net.kyori.adventure.text.minimessage.MiniMessage;
Expand All @@ -21,13 +22,13 @@ public ReloadCommand() {
@Override
public void execute(CommandSender sender, String[] args) {
MiniMessage miniMessage = VillagerInfo.getMiniMessage();
if (!(sender instanceof Player) || sender.hasPermission(Perms.RELOAD.getPerm())) {
if (!(sender instanceof Player) || sender.hasPermission(Perms.RELOAD.getVIPerm())) {
VillagerInfo.getInstance().reloadConfig();
VillagerInfo.getLocaleConfig().reloadConfig();
LocaleConfig.getInstance().reloadLocale();
ConfigValidator.configValidator();
sender.sendMessage(miniMessage.deserialize(Message.CONFIG_RELOADED.getMessage()));
sender.sendMessage(miniMessage.deserialize(VIMessage.CONFIG_RELOADED.getMessage()));
} else {
sender.sendMessage(miniMessage.deserialize((Message.NO_PERMISSION.getMessage())));
sender.sendMessage(miniMessage.deserialize((VIMessage.NO_PERMISSION.getMessage())));
}
}

Expand Down
Original file line number Diff line number Diff line change
@@ -1,7 +1,7 @@
package adhdmc.villagerinfo.Commands.SubCommands;

import adhdmc.villagerinfo.Commands.SubCommand;
import adhdmc.villagerinfo.Config.Message;
import adhdmc.villagerinfo.Config.VIMessage;
import adhdmc.villagerinfo.Config.Perms;
import adhdmc.villagerinfo.VillagerInfo;
import net.kyori.adventure.text.minimessage.MiniMessage;
Expand All @@ -22,20 +22,20 @@ public ToggleCommand() {
@Override
public void execute(CommandSender sender, String[] args) {
if (!(sender instanceof Player)) {
sender.sendMessage(miniMessage.deserialize(Message.NOT_A_PLAYER.getMessage()));
sender.sendMessage(miniMessage.deserialize(VIMessage.NOT_A_PLAYER.getMessage()));
return;
}
if (!(sender.hasPermission(Perms.TOGGLE.getPerm()))) {
sender.sendMessage(miniMessage.deserialize(Message.NO_PERMISSION.getMessage()));
if (!(sender.hasPermission(Perms.TOGGLE.getVIPerm()))) {
sender.sendMessage(miniMessage.deserialize(VIMessage.NO_PERMISSION.getMessage()));
return;
}
if (toggleSetting((Player) sender)) {
sender.sendMessage(miniMessage.deserialize(Message.PREFIX.getMessage())
.append(miniMessage.deserialize(Message.TOGGLE_ON.getMessage())));
sender.sendMessage(miniMessage.deserialize(VIMessage.PLUGIN_PREFIX.getMessage())
.append(miniMessage.deserialize(VIMessage.TOGGLE_ON.getMessage())));
return;
}
sender.sendMessage(miniMessage.deserialize(Message.PREFIX.getMessage())
.append(miniMessage.deserialize(Message.TOGGLE_OFF.getMessage())));
sender.sendMessage(miniMessage.deserialize(VIMessage.PLUGIN_PREFIX.getMessage())
.append(miniMessage.deserialize(VIMessage.TOGGLE_OFF.getMessage())));
}

private boolean toggleSetting(Player player) {
Expand Down
4 changes: 0 additions & 4 deletions src/main/java/adhdmc/villagerinfo/Config/ConfigValidator.java
Original file line number Diff line number Diff line change
Expand Up @@ -4,10 +4,7 @@
import org.bukkit.Sound;
import org.bukkit.configuration.file.FileConfiguration;

import java.util.Collections;
import java.util.HashMap;
import java.util.Locale;
import java.util.Map;

public class ConfigValidator {

Expand Down Expand Up @@ -50,7 +47,6 @@ public static void configValidator() {
soundPitch = 1.5F;
}
ToggleSetting.reloadToggles();
Message.reloadLocale();
}

public static float getSoundPitch() {
Expand Down
70 changes: 0 additions & 70 deletions src/main/java/adhdmc/villagerinfo/Config/Defaults.java

This file was deleted.

66 changes: 27 additions & 39 deletions src/main/java/adhdmc/villagerinfo/Config/LocaleConfig.java
Original file line number Diff line number Diff line change
@@ -1,57 +1,45 @@
package adhdmc.villagerinfo.Config;

import adhdmc.villagerinfo.VillagerInfo;
import org.bukkit.configuration.InvalidConfigurationException;
import org.bukkit.configuration.file.FileConfiguration;
import org.bukkit.configuration.file.YamlConfiguration;

import java.io.File;
import java.io.IOException;
import java.io.InputStream;
import java.io.InputStreamReader;
import java.util.Set;

public class LocaleConfig {
private final VillagerInfo plugin;
private static final String localeName = "locale.yml";
private YamlConfiguration localeConfig = null;
private File localeFile = null;

public LocaleConfig(VillagerInfo plugin) {
this.plugin = plugin;
}
private static LocaleConfig instance;
private final String fileName = "locale.yml";
private final File localeFile = new File(VillagerInfo.getInstance().getDataFolder(), fileName);
private final FileConfiguration localeConfig = new YamlConfiguration();

public void reloadConfig() {
if (this.localeFile == null) {
this.localeFile = new File(this.plugin.getDataFolder(), localeName);
}
this.localeConfig = YamlConfiguration.loadConfiguration(this.localeFile);
this.localeConfig.options().copyDefaults(true);
InputStream defaultStream = this.plugin.getResource(localeName);
if (defaultStream != null) {
YamlConfiguration defaultConfig = YamlConfiguration.loadConfiguration(new InputStreamReader(defaultStream));
this.localeConfig.setDefaults(defaultConfig);
}

private LocaleConfig() {
if (!localeFile.exists()) VillagerInfo.getInstance().saveResource(fileName, false);
reloadLocale();
}

public YamlConfiguration getlocaleConfig() {
if (this.localeConfig == null) {
reloadConfig();
}
return this.localeConfig;
public static LocaleConfig getInstance() {
if (instance == null) instance = new LocaleConfig();
return instance;
}

public void saveConfig() {
getlocaleConfig();
if (this.localeConfig == null || this.localeFile == null) {
return;
}
try {
this.getlocaleConfig().save(this.localeFile);
} catch (IOException e) {
plugin.getLogger().severe("[saveConfig()] Could not save config to " + this.localeFile);
e.printStackTrace();
}
if (!this.localeFile.exists()) {
this.plugin.saveResource(localeName, false);
public FileConfiguration getLocale() { return localeConfig; }

public void reloadLocale() {
try { localeConfig.load(localeFile); }
catch (IOException | InvalidConfigurationException e) { e.printStackTrace(); }
Set<String> keys = localeConfig.getKeys(false);
for (String key : keys) {
try {
VIMessage message = VIMessage.valueOf(key);
message.setMessage(localeConfig.getString(key, message.getMessage()));
} catch (IllegalArgumentException e) {
VillagerInfo.getVillagerInfoLogger().warning(VIMessage.LOGGER_INVALID_LOCALE_KEY.getMessage() + key);
}
}
}
}

Loading

0 comments on commit 80945a9

Please sign in to comment.