Fix paper not supported message

[Snyk] Upgrade org.projectlombok:lombok from 1.18.30 to 1.18.32

.github/workflows/maven.yml

@@ -10,7 +10,7 @@ jobs:
     runs-on: ubuntu-latest
     strategy:
       matrix:
-        java-version: [ 8, 11, 17 ]
+        java-version: [ 8, 11, 17, 21 ]
     steps:
       - name: Checkout
         uses: actions/checkout@v3

.github/workflows/release.yml

@@ -13,10 +13,10 @@ jobs:
 
     steps:
     - uses: actions/checkout@v3
-    - name: Set up JDK 17
+    - name: Set up JDK 21
       uses: actions/setup-java@v3
       with:
-        java-version: 17
+        java-version: 21
         distribution: temurin
         server-id: github
         settings-path: ${{ github.workspace }} # location for the settings.xml file

.github/workflows/sonarcloud.yml

@@ -14,10 +14,10 @@ jobs:
       - uses: actions/checkout@v3
         with:
           fetch-depth: 0  # Shallow clones should be disabled for a better relevancy of analysis
-      - name: Set up JDK 11
+      - name: Set up JDK 21
         uses: actions/setup-java@v3
         with:
-          java-version: 11
+          java-version: 21
           distribution: temurin
       - name: Cache SonarCloud packages
         uses: actions/cache@v3

Components/Minepacks-Bootstrap-Paper/resources/paper-plugin.yml

@@ -1,6 +1,6 @@
 name: "Minepacks"
-author: "GeorgH93"
-version: "${version}"
+author: "${author}"
+version: "${pluginVersion}"
 api-version: "1.19"
 
 main: "at.pcgamingfreaks.MinepacksStandalone.Bukkit.Minepacks"
@@ -10,7 +10,13 @@ dependencies:
   - name: PCGF_PluginLib
     required: false
     bootstrap: true
+  - name: MVdWPlaceholderAPI
+    required: false
+  - name: PlaceholderAPI
+    required: false
 
 load-after:
   - name: PCGF_PluginLib
-    bootstrap: true
+    bootstrap: true
+  - name: MVdWPlaceholderAPI
+  - name: PlaceholderAPI

Minepacks-API/pom.xml

@@ -18,7 +18,7 @@
 		<dependency>
 			<groupId>at.pcgamingfreaks</groupId>
 			<artifactId>PluginLib</artifactId>
-			<version>1.0.17-SNAPSHOT</version>
+			<version>${pcgfPluginLibVersion}</version>
 			<scope>provided</scope>
 		</dependency>
 	</dependencies>

Minepacks-API/src/at/pcgamingfreaks/Minepacks/Bukkit/API/Backpack.java

@@ -27,6 +27,7 @@ import org.jetbrains.annotations.NotNull;
 import org.jetbrains.annotations.Nullable;
 
 import java.util.Map;
+import java.util.UUID;
 
 @SuppressWarnings("unused")
 public interface Backpack extends InventoryHolder
@@ -35,9 +36,20 @@ public interface Backpack extends InventoryHolder
 	 * Gets the owner of the backpack.
 	 *
 	 * @return The owner of the backpack;
+	 * @deprecated Use the {@link Backpack#getOwnerId()} function instead.
 	 */
+	@Deprecated
 	@NotNull OfflinePlayer getOwner();
 
+	/**
+	 * Gets the id of the player owning the backpack.
+	 *
+	 * @return The uuid of the owning player.
+	 */
+	UUID getOwnerId();
+
+	@Nullable Player getOwnerPlayer();
+
 	/**
 	 * Let a given player open this backpack.
 	 *

Minepacks-API/src/at/pcgamingfreaks/Minepacks/Bukkit/API/Callback.java

@@ -21,5 +21,5 @@ public interface Callback<T>
 {
 	void onResult(T done);
 
-	void onFail();
+	default void onFail() { /* In most cases this can be ignored since it will not happen with most db queries used */ }
 }

Minepacks-API/src/at/pcgamingfreaks/Minepacks/Bukkit/API/Events/InventoryClearEvent.java

@@ -35,12 +35,12 @@ public class InventoryClearEvent extends Event implements Cancellable
 	/**
 	 * The player whose inventory should be cleared.
 	 */
-	@Getter final private Player player;
+	@Getter private final Player player;
 
 	/**
 	 * The command sender who has requested the inventory to be cleared.
 	 */
-	@Getter final private CommandSender sender;
+	@Getter private final CommandSender sender;
 
 	// Bukkit handler stuff
 	private static final HandlerList handlers = new HandlerList();

Minepacks-API/src/at/pcgamingfreaks/Minepacks/Bukkit/API/Events/InventoryClearedEvent.java

@@ -31,12 +31,12 @@ public class InventoryClearedEvent extends Event
 	/**
 	 * The player whose inventory has been cleared.
 	 */
-	@Getter final private Player player;
+	@Getter private final Player player;
 
 	/**
 	 * The command sender who has requested the inventory to be cleared.
 	 */
-	@Getter final private CommandSender sender;
+	@Getter private final CommandSender sender;
 
 	// Bukkit handler stuff
 	private static final HandlerList handlers = new HandlerList();

Minepacks-API/src/at/pcgamingfreaks/Minepacks/Bukkit/API/MinepacksCommand.java

@@ -30,15 +30,20 @@ import org.jetbrains.annotations.Nullable;
 import java.lang.reflect.Method;
 import java.util.ArrayList;
 import java.util.List;
+import java.util.logging.Level;
 
 /**
  * Only available if the plugin is not running in standalone mode!
  */
 public abstract class MinepacksCommand extends SubCommand
 {
-	private static MinepacksPlugin minepacksPlugin = null;
-	private static Method showHelp = null;
+	@SuppressWarnings("FieldMayBeFinal")
+	private static MinepacksPlugin minepacksPlugin = null; // Will be set by reflection
+	@SuppressWarnings("FieldMayBeFinal")
+	private static Method showHelp = null; // Will be set by reflection
+	@SuppressWarnings("FieldMayBeFinal") // Will be overwritten by reflection
 	private static Message messageNoPermission   = new Message(ChatColor.RED + "You don't have the permission to do that.");
+	@SuppressWarnings("FieldMayBeFinal") // Will be overwritten by reflection
 	private static Message messageNotFromConsole = new Message(ChatColor.RED + "This command can't be used from console!");
 	protected final JavaPlugin plugin;
 	private final boolean playerOnly;
@@ -52,7 +57,7 @@ public abstract class MinepacksCommand extends SubCommand
 	 * @param description The description of the command.
 	 * @param aliases     List of aliases for that command.
 	 */
-	public MinepacksCommand(@NotNull JavaPlugin plugin, @NotNull String name, @NotNull String description, @Nullable String... aliases)
+	protected MinepacksCommand(@NotNull JavaPlugin plugin, @NotNull String name, @NotNull String description, @Nullable String... aliases)
 	{
 		this(plugin, name, description, null, aliases);
 	}
@@ -66,7 +71,7 @@ public abstract class MinepacksCommand extends SubCommand
 	 * @param permission  The permission to be checked for this command. Players without the permission neither can use the command nor will they see it in help.
 	 * @param aliases     List of aliases for that command.
 	 */
-	public MinepacksCommand(@NotNull JavaPlugin plugin, @NotNull String name, @NotNull String description, @Nullable String permission, @Nullable String... aliases)
+	protected MinepacksCommand(@NotNull JavaPlugin plugin, @NotNull String name, @NotNull String description, @Nullable String permission, @Nullable String... aliases)
 	{
 		this(plugin, name, description, permission, false, aliases);
 	}
@@ -81,7 +86,7 @@ public abstract class MinepacksCommand extends SubCommand
 	 * @param playerOnly  Limits the command to players, console can't use and can't see the command.
 	 * @param aliases     List of aliases for that command.
 	 */
-	public MinepacksCommand(@NotNull JavaPlugin plugin, @NotNull String name, @NotNull String description, @Nullable String permission, boolean playerOnly, @Nullable String... aliases)
+	protected MinepacksCommand(@NotNull JavaPlugin plugin, @NotNull String name, @NotNull String description, @Nullable String permission, boolean playerOnly, @Nullable String... aliases)
 	{
 		super(name, description, permission, aliases);
 		this.plugin = plugin;
@@ -173,7 +178,7 @@ public abstract class MinepacksCommand extends SubCommand
 	 * @param usedMainCommandAlias The used backpack alias to replace the /backpack with the used alias.
 	 */
 	@Override
-	public void showHelp(@NotNull CommandSender sendTo, @NotNull String usedMainCommandAlias)
+	public void showHelp(final @NotNull CommandSender sendTo, final @NotNull String usedMainCommandAlias)
 	{
 		try
 		{
@@ -181,7 +186,7 @@ public abstract class MinepacksCommand extends SubCommand
 		}
 		catch(Exception e)
 		{
-			e.printStackTrace();
+			plugin.getLogger().log(Level.SEVERE, e, () -> "Failed to execute command " + usedMainCommandAlias);
 		}
 	}
 

Minepacks/pom.xml

@@ -16,10 +16,6 @@
 	<url>https://www.spigotmc.org/resources/19286/</url>
 
 	<properties>
-		<author>GeorgH93</author>
-		<maven.build.timestamp.format>yyyyMMddHHmmss</maven.build.timestamp.format>
-		<timestamp>${maven.build.timestamp}</timestamp>
-		<version>${project.version}-T${timestamp}</version>
 		<dependencies>PCGF_PluginLib</dependencies>
 		<soft-dependencies/>
 		<mainClass>${project.groupId}.${project.artifactId}.Bukkit.${project.artifactId}</mainClass>
@@ -220,7 +216,7 @@
 			<properties>
 				<version>${project.version}-Release</version>
 				<dependencies/>
-				<soft-dependencies>PCGF_PluginLib</soft-dependencies>
+				<soft-dependencies>, PCGF_PluginLib</soft-dependencies>
 				<mainClass>${project.groupId}.${project.artifactId}.Bukkit.${project.artifactId}BadRabbit</mainClass>
 				<releaseType>Release</releaseType>
 			</properties>

Minepacks/resources/lang/en.yml

@@ -16,6 +16,7 @@ Language:
     NotAllowedInBackpack_SendMethod: "action_bar"
     DontRemoveShortcut: "&cYou must not remove the backpack shortcut from your inventory!"
     DontRemoveShortcut_SendMethod: "action_bar"
+    BackpackShrunk: "&cYour backpack shrunk! Some items fell to the ground!"
     Open:
       #Parameter: {TimeLeft} time in seconds till the backpack can be reopened, {TimeSpanLeft} time formatted as string till the backpack can be reopened
       Cooldown: "[{\"text\":\"Please wait \",\"color\":\"dark_green\"},{\"text\":\"{TimeSpanLeft}\",\"hoverEvent\":{\"action\":\"show_text\",\"value\":\"{TimeLeft} seconds\"}},{\"text\":\" till you reopen your backpack.\"}]"

Minepacks/resources/lang/ru.yml

@@ -1,4 +1,4 @@
-# To simplify the customisation and the translation process please check out the editor: https://ptp.pcgamingfreaks.at
+# To simplify the customisation and translation process please check out the editor: https://ptp.pcgamingfreaks.at
 
 Language:
   NotFromConsole: "&cЭту команду нельзя использовать в консоли."
@@ -23,6 +23,10 @@ Language:
       WrongGameMode: "Вам не разрешено открывать рюкзак в текущем режиме игры."
     Clean:
       BackpackCleaned: "Рюкзак очищен."
+      BackpackCleanedBy: "Ваш рюкзак был очищен игроком {DisplayName}&r."
+      BackpackCleanedOther: "{DisplayName}'s&r рюкзак был очищен."
+    Sort:
+      Sorted: "Рюкзак отсортирован."
     Help:
       Header: "&6### Команды Minepacks ###"
       Footer: "&6#############################"
@@ -35,7 +39,7 @@ Language:
       NoUpdate: "[\"\",{\"text\":\"Нет доступных обновлений.\",\"color\":\"gold\"}]"
       UpdateFail: "[\"\",{\"text\":\"При проверке обновлений произошла ошибка! Пожалуйста проверьте консоль!\",\"color\":\"red\"}]"
       # You can change this message if you like to, but don't cry if the link isn't linking to the plugin anymore!
-      UpdateAvailable: "[{\"text\":\"Доступно обновление! Пожалуйста, посетите сайт \\\"\",\"color\":\"green\"},{\"text\":\"${project.url}\",\"color\":\"yellow\",\"underlined\":true,\"clickEvent\":{\"action\":\"open_url\",\"value\":\"${project.url}\"}},{\"text\":\"\\\" чтобы скачать его!\"}]"
+      UpdateAvailable: "[{\"text\":\"Доступно обновление! Пожалуйста, посетите сайт \\\"\",\"color\":\"green\"},{\"text\":\"https://www.spigotmc.org/resources/19286/\",\"color\":\"yellow\",\"underlined\":true,\"clickEvent\":{\"action\":\"open_url\",\"value\":\"https://www.spigotmc.org/resources/19286/\"}},{\"text\":\"\\\" чтобы скачать его!\"}]"
     Backup:
       Created: "Рюкзак был успешно скопирован."
       NoBackpack: "У игрока нет рюкзака, либо он пустой."
@@ -43,7 +47,7 @@ Language:
       BackupsPerPage: 10
       Headline: "[\"\",{\"text\":\"Резервные копии\",\"color\":\"yellow\"},{\"text\":\" - \",\"color\":\"white\"},{\"text\":\"страница {CurrentPage}/{MaxPage}\",\"color\":\"gold\"}]"
       Footer: "[{\"text\":\"<<< Предыдущая <<<\",\"color\":\"gray\",\"clickEvent\":{\"action\":\"run_command\",\"value\":\"/{MainCommand} {SubCommand} {CurrentPage}--\"},\"hoverEvent\":{\"action\":\"show_text\",\"value\":\"/{MainCommand} {SubCommand} {CurrentPage}--\"}},{\"text\":\" Страница {CurrentPage}/{MaxPage} \",\"color\":\"gold\"},{\"text\":\">>> Следующая >>>\",\"color\":\"gray\",\"clickEvent\":{\"action\":\"run_command\",\"value\":\"/{MainCommand} {SubCommand} {CurrentPage}++\"},\"hoverEvent\":{\"action\":\"show_text\",\"value\":\"/{MainCommand} {SubCommand} {CurrentPage}++\"}}]"
-      BackupEntry: "[\"\",{\"text\":\"{BackupIdentifier}\",\"clickEvent\":{\"action\":\"suggest_command\",\"value\":\"/{MainCommand} {SubCommand} {BackupIdentifier}\"},\"hoverEvent\":{\"action\":\"show_text\",\"value\":\"Пользователь: {BackupPlayerName} ({BackupPlayerUUID})\\nСоздана: {BackupDate}\"}}]"
+      BackupEntry: "[\"\",{\"text\":\"{BackupIdentifier}\",\"clickEvent\":{\"action\":\"suggest_command\",\"value\":\"/{MainCommand} {SubCommand} {BackupIdentifier}\"},\"hoverEvent\":{\"action\":\"show_text\",\"value\":\"Пользователь: {BackupPlayerName} ({BackupPlayerUUID})\\\nСоздана: {BackupDate}\"}}]"
       NoValidBackup: "Резервная копия {BackupIdentifier} не найдена"
       NoUserToRestoreToFound: "Не найдено действительного пользователя или копии для восстановления"
       # No Json!!!
@@ -51,11 +55,20 @@ Language:
       # No Json!!!
       DateFormat: "dd.MM.yy HH:mm:ss"
       Restored: "Резервная копия успешно восстановлена."
+    InventoryClear:
+      UnknownPlayer: "&cНе удалось найти игрока {Name}!"
+      Cleared: "Инвентарь очищен."
+      ClearedOther: "{DisplayName}'s&r инвентарь был очищен."
+      ClearedOtherTarget: "Ваш инвентарь был очищен игроком {DisplayName}&r."
+    Pickup:
+      ToggleOn: "&7Автоматический сбор предметов был переключен на &aВКЛ&7."
+      ToggleOff: "&7Автоматический сбор предметов был переключен на &cВЫКЛ&7."
   Commands:
     HelpFormat: "[\"\",{\"text\":\"/{MainCommand} {SubCommand} {Parameters}\",\"clickEvent\":{\"action\":\"suggest_command\",\"value\":\"/{MainCommand} {SubCommand}\"}},{\"text\":\" - \",\"color\":\"white\"},{\"text\":\"{Description}\",\"color\":\"aqua\"}]"
     PlayerNameVariable: "player_name"
     Description:
       Backpack: "Открыть ваш рюкзак."
+      Sort: "Отсортировать ваш рюкзак."
       Clean: "Очистить ваш рюкзак."
       CleanOthers: "Очистить рюкзак другого игрока."
       OpenOthers: "Открыть рюкзак другого игрока."
@@ -63,10 +76,12 @@ Language:
       Update: "Проверить плагин на наличие новых версий."
       Version: "Посмотреть информацию о плагине и о его зависимостях."
       Backup: "Создать резервную копию рюкзака игрока."
+      BackupEveryone: "Создает резервную копию всех, кто в данный момент находится в сети."
       Restore: "Восстановить резервную копию."
       RestoreList: "Посмотреть список всех доступных резервных копий."
       Help: "Посмотреть все команды и их описание."
       Migrate: "Изменить тип используемой базы данных."
+      Pickup: "Переключить состояние автоматического подбора, когда инвентарь заполнен."
 
 Command:
   Backpack:
@@ -74,6 +89,8 @@ Command:
     - bp
   Open:
     - open
+  Sort:
+    - sort
   Clean:
     - clean
     - clear
@@ -93,10 +110,16 @@ Command:
     - version
   Help:
     - help
+  InventoryClear:
+    - clear
+    - inventoryclear
+    - clean
+  Pickup:
+    - pickup
 
 # Will be shown in the console during startup
 LanguageName: "russian"
-Author: "MrZillaGold"
+Author: "maz1lovo"
 
 # Language file version. Don't touch it!
-Version: 17
+Version: 20

Minepacks/resources/plugin.yml

@@ -3,10 +3,10 @@ author: "${author}"
 website: "${project.url}"
 main: "${mainClass}"
 description: "${project.description}"
-version: "${version}"
+version: "${pluginVersion}"
 api-version: "1.13"
 depend: [${dependencies}]
-softdepend: [${soft-dependencies}]
+softdepend: [MVdWPlaceholderAPI, PlaceholderAPI ${soft-dependencies}]
 
 permissions:
   backpack.*:

Minepacks/src/at/pcgamingfreaks/Minepacks/Bukkit/Backpack.java

@@ -1,5 +1,5 @@
 /*
- *   Copyright (C) 2020 GeorgH93
+ *   Copyright (C) 2024 GeorgH93
  *
  *   This program is free software: you can redistribute it and/or modify
  *   it under the terms of the GNU General Public License as published by
@@ -12,17 +12,17 @@
  *   GNU General Public License for more details.
  *
  *   You should have received a copy of the GNU General Public License
- *   along with this program. If not, see <https://www.gnu.org/licenses/>.
+ *   along with this program. If not, see <http://www.gnu.org/licenses/>.
  */
 
 package at.pcgamingfreaks.Minepacks.Bukkit;
 
 import at.pcgamingfreaks.Bukkit.MCVersion;
+import at.pcgamingfreaks.Bukkit.Message.Message;
 import at.pcgamingfreaks.Bukkit.Util.InventoryUtils;
 import at.pcgamingfreaks.Minepacks.Bukkit.Database.Helper.InventoryCompressor;
 import at.pcgamingfreaks.Util.StringUtils;
-import lombok.AccessLevel;
-import lombok.Setter;
+
 import org.bukkit.Bukkit;
 import org.bukkit.Location;
 import org.bukkit.OfflinePlayer;
@@ -32,26 +32,35 @@ import org.bukkit.inventory.ItemStack;
 import org.jetbrains.annotations.NotNull;
 import org.jetbrains.annotations.Nullable;
 
+import lombok.AccessLevel;
+import lombok.Getter;
+import lombok.Setter;
+
 import java.util.ArrayList;
 import java.util.List;
 import java.util.Map;
+import java.util.UUID;
 import java.util.concurrent.ConcurrentHashMap;
 
 public class Backpack implements at.pcgamingfreaks.Minepacks.Bukkit.API.Backpack
 {
 	@Setter(AccessLevel.PACKAGE) private static ShrinkApproach shrinkApproach = ShrinkApproach.COMPRESS;
-	private static Object titleOwn;
-	private static String titleOtherFormat;
+	@Setter(AccessLevel.PACKAGE) private static Message messageBackpackShrunk = new Message("Backpack shrunk!");
+	private static Object titleOwnGlobal;
+	private static String titleFormat, titleOtherFormat;
+	private final Object titleOwn;
 	private final String titleOther;
-	private final OfflinePlayer owner;
+	@Getter private final UUID ownerId;
 	private final Map<Player, Boolean> opened = new ConcurrentHashMap<>(); //Thanks Minecraft 1.14
 	private Inventory bp;
-	private int size, ownerID;
+	@Getter private int size;
+	@Getter @Setter private int ownerDatabaseId;
 	private boolean hasChanged;
 
 	public static void setTitle(final @NotNull String title, final @NotNull String titleOther)
 	{
-		titleOwn = InventoryUtils.prepareTitleForOpenInventoryWithCustomTitle(title);
+		titleOwnGlobal = title.contains("%s") ? null : InventoryUtils.prepareTitleForOpenInventoryWithCustomTitle(title);
+		titleFormat = title;
 		titleOtherFormat = titleOther;
 	}
 
@@ -72,11 +81,14 @@ public class Backpack implements at.pcgamingfreaks.Minepacks.Bukkit.API.Backpack
 			size = 54;
 			Minepacks.getInstance().getLogger().warning("Backpacks with more than 6 rows are no longer supported on Minecraft 1.14 and up!");
 		}
-		this.owner = owner;
+		this.ownerId = owner.getUniqueId();
 		titleOther = StringUtils.limitLength(String.format(titleOtherFormat, owner.getName()), 32);
 		bp = Bukkit.createInventory(this, size, titleOther);
 		this.size = size;
-		ownerID = ID;
+		ownerDatabaseId = ID;
+
+		if (titleOwnGlobal != null) titleOwn = titleOwnGlobal;
+		else titleOwn = InventoryUtils.prepareTitleForOpenInventoryWithCustomTitle(String.format(titleFormat, owner.getName()));
 	}
 	
 	public Backpack(final OfflinePlayer owner, ItemStack[] backpack, final int ID)
@@ -108,41 +120,44 @@ public class Backpack implements at.pcgamingfreaks.Minepacks.Bukkit.API.Backpack
 		}
 		bp.setContents(backpack);
 	}
-	
-	public int getOwnerID()
+
+	@Override
+	@Deprecated
+	public @NotNull OfflinePlayer getOwner()
 	{
-		return ownerID;
-	}
-	
-	public void setOwnerID(int id)
-	{
-		ownerID = id;
+		return Bukkit.getServer().getOfflinePlayer(ownerId);
 	}
 
 	@Override
-	public @NotNull OfflinePlayer getOwner()
+	public @Nullable Player getOwnerPlayer()
 	{
-		return owner;
+		return Bukkit.getServer().getPlayer(ownerId);
 	}
 
-	private void checkResize()
+	public void checkResize()
 	{
-		if(owner.isOnline())
+		Player owner = Bukkit.getServer().getPlayer(this.ownerId);
+		if(owner != null)
 		{
-			Player owner = this.owner.getPlayer();
-			if(owner != null && owner.hasPermission(Permissions.USE))
+			if(owner.hasPermission(Permissions.USE))
 			{
 				int size = Minepacks.getInstance().getBackpackPermSize(owner);
 				if(size != bp.getSize())
 				{
+					boolean dropped = false;
 					List<ItemStack> items = setSize(size);
 					for(ItemStack i : items)
 					{
 						if(i != null)
 						{
 							owner.getWorld().dropItemNaturally(owner.getLocation(), i);
+							dropped = true;
 						}
 					}
+					if (dropped)
+					{
+						messageBackpackShrunk.send(owner);
+					}
 				}
 			}
 		}
@@ -153,7 +168,7 @@ public class Backpack implements at.pcgamingfreaks.Minepacks.Bukkit.API.Backpack
 	{
 		checkResize();
 		opened.put(player, editable);
-		if(owner.equals(player)) InventoryUtils.openInventoryWithCustomTitlePrepared(player, bp, titleOwn);
+		if(ownerId.equals(player.getUniqueId())) InventoryUtils.openInventoryWithCustomTitlePrepared(player, bp, titleOwn);
 		else player.openInventory(bp);
 	}
 
@@ -194,12 +209,6 @@ public class Backpack implements at.pcgamingfreaks.Minepacks.Bukkit.API.Backpack
 		return opened.containsKey(player) && opened.get(player);
 	}
 
-	@Override
-	public int getSize()
-	{
-		return size;
-	}
-	
 	public @NotNull List<ItemStack> setSize(int newSize)
 	{
 		opened.forEach((key, value) -> key.closeInventory()); // Close all open views of the inventory

Minepacks/src/at/pcgamingfreaks/Minepacks/Bukkit/Command/ClearCommand.java

@@ -69,15 +69,15 @@ public class ClearCommand extends MinepacksCommand
 					if(backpack != null)
 					{
 						backpack.clear();
-						if(commandSender.equals(backpack.getOwner()))
+						if(commandSender instanceof Player && ((Player) commandSender).getUniqueId().equals(backpack.getOwnerId()))
 						{
 							messageCleared.send(commandSender);
 						}
 						else
 						{
-							if(backpack.getOwner().isOnline())
+							Player owner = backpack.getOwnerPlayer();
+							if(owner != null)
 							{
-								Player owner = backpack.getOwner().getPlayer();
 								messageClearedOther.send(commandSender, owner);
 								messageClearedBy.send(owner, commandSender);
 							}

Minepacks/src/at/pcgamingfreaks/Minepacks/Bukkit/Command/CommandManager.java

@@ -1,5 +1,5 @@
 /*
- *   Copyright (C) 2023 GeorgH93
+ *   Copyright (C) 2024 GeorgH93
  *
  *   This program is free software: you can redistribute it and/or modify
  *   it under the terms of the GNU General Public License as published by
@@ -37,6 +37,7 @@ import org.jetbrains.annotations.NotNull;
 import java.util.Collection;
 import java.util.List;
 import java.util.Locale;
+import java.util.logging.Level;
 
 public class CommandManager extends CommandExecutorWithSubCommandsGeneric<MinepacksCommand> /*if_not[STANDALONE]*/ implements MinepacksCommandManager /*end[STANDALONE]*/
 {
@@ -67,8 +68,7 @@ public class CommandManager extends CommandExecutorWithSubCommandsGeneric<Minepa
 		}
 		catch(Exception e)
 		{
-			plugin.getLogger().warning(ConsoleColor.RED + "Unable to set the help format. Default format will be used.\nMore details:" + ConsoleColor.RESET);
-			e.printStackTrace();
+			plugin.getLogger().log(Level.SEVERE, e, () -> ConsoleColor.RED + "Unable to set the help format. Default format will be used.\nMore details:" + ConsoleColor.RESET);
 		}
 
 		// Init backpack commands
@@ -126,11 +126,8 @@ public class CommandManager extends CommandExecutorWithSubCommandsGeneric<Minepa
 			WorldBlacklistMode disabled = plugin.isDisabled((Player) sender);
 			if(disabled != WorldBlacklistMode.None)
 			{
-				switch(disabled)
-				{
-					case Message: plugin.messageWorldDisabled.send(sender); break;
-					case MissingPermission: plugin.messageNoPermission.send(sender); break;
-				}
+				if (disabled == WorldBlacklistMode.Message) plugin.messageWorldDisabled.send(sender);
+				else if (disabled == WorldBlacklistMode.MissingPermission) plugin.messageNoPermission.send(sender);
 				return null;
 			}
 		}

Minepacks/src/at/pcgamingfreaks/Minepacks/Bukkit/Command/DebugCommand.java

@@ -25,6 +25,7 @@ import at.pcgamingfreaks.Message.MessageClickEvent;
 import at.pcgamingfreaks.Message.MessageColor;
 import at.pcgamingfreaks.Message.MessageFormat;
 import at.pcgamingfreaks.Minepacks.Bukkit.API.MinepacksCommand;
+import at.pcgamingfreaks.Minepacks.Bukkit.API.Backpack;
 import at.pcgamingfreaks.Minepacks.Bukkit.Minepacks;
 import at.pcgamingfreaks.Minepacks.Bukkit.Permissions;
 
@@ -48,7 +49,9 @@ import lombok.SneakyThrows;
 import java.io.*;
 import java.nio.charset.StandardCharsets;
 import java.nio.file.Files;
+import java.util.ArrayList;
 import java.util.List;
+import java.util.logging.Level;
 
 public class DebugCommand extends MinepacksCommand
 {
@@ -67,18 +70,16 @@ public class DebugCommand extends MinepacksCommand
 		builder = new MessageBuilder("All data has been collected!", MessageColor.GREEN, MessageFormat.BOLD).appendNewLine();
 		builder.append("You can now interact with your game again.").appendNewLine();
 		builder.append("The collected data can be found in your plugins directory inside the 'debug.txt' file.").appendNewLine();
-		builder.append("Please upload this fiel to ");
+		builder.append("Please upload this file to ");
 		builder.append("https://pastebin.com/", MessageColor.YELLOW, MessageFormat.UNDERLINE).onClick(MessageClickEvent.ClickEventAction.OPEN_URL, "https://pastebin.com/");
 		builder.append(" and send the link to the developer.");
 		messageDone = builder.getMessage();
 	}
 
 	@SneakyThrows
-	@Override
-	public void execute(@NotNull CommandSender commandSender, @NotNull String mainCommandAlias, @NotNull String alias, @NotNull String[] args)
+	private void debugSystem(final @NotNull CommandSender commandSender)
 	{
-		if(writer != null) return;
-		Player sender = (Player) commandSender;
+		final Player sender = (Player) commandSender;
 		messageStart.send(sender);
 
 		File debugFile = new File(plugin.getDataFolder(), "debug.txt");
@@ -128,15 +129,100 @@ public class DebugCommand extends MinepacksCommand
 				writer.close();
 				writer = null;
 			}
-			catch(Exception e) {e.printStackTrace();}
+			catch(Exception e) { plugin.getLogger().log(Level.SEVERE, "Error while writing debug file.", e);}
 			sender.getInventory().setItem(0, slot);
 			messageDone.send(sender);
 		}, 30*20L);
 	}
 
+	@SneakyThrows
+	@Override
+	public void execute(@NotNull CommandSender commandSender, @NotNull String mainCommandAlias, @NotNull String alias, @NotNull String[] args)
+	{
+		if(writer != null) return;
+		if (args.length == 2 && args[0].equals("permissions"))
+		{
+			Player player = Bukkit.getServer().getPlayer(args[1]);
+			if (player == null)
+			{
+				commandSender.sendMessage("Player " + args[1] + " is offline.");
+				return;
+			}
+			commandSender.sendMessage("### Permissions for " + player.getName() + " ###");
+			for(String perm : Permissions.getPermissions())
+			{
+				commandSender.sendMessage(perm + ": " + player.hasPermission(perm));
+			}
+			commandSender.sendMessage("###############################");
+		}
+		else if (args.length == 2 && args[0].equals("size"))
+		{
+			executeSize(commandSender, args[1]);
+		}
+		else
+		{
+			debugSystem(commandSender);
+		}
+	}
+
+	void executeSize(final @NotNull CommandSender commandSender, final @NotNull String playerName)
+	{
+		Player player = Bukkit.getServer().getPlayer(playerName);
+		if (player == null)
+		{
+			commandSender.sendMessage("Player " + playerName + " is offline.");
+			return;
+		}
+		Backpack bp = Minepacks.getInstance().getBackpackCachedOnly(player);
+		int bpSize = -1, bpInvSize = -1, sizeShouldBe = Minepacks.getInstance().getBackpackPermSize(player);
+		String actualSize = "backpack not loaded", actualSizeInventory = "backpack not loaded";
+		if (bp != null)
+		{
+			bpSize = bp.getSize();
+			bpInvSize = bp.getInventory().getSize();
+			actualSize = String.valueOf(bpSize);
+			actualSizeInventory = String.valueOf(bpInvSize);
+		}
+		commandSender.sendMessage("### Backpack size for " + player.getName() + " ###");
+		commandSender.sendMessage("Size: " + actualSize);
+		commandSender.sendMessage("Inventory Size: " + actualSizeInventory);
+		commandSender.sendMessage("Should be: " + sizeShouldBe);
+		if (bpSize != sizeShouldBe && bp != null)
+		{
+			commandSender.sendMessage("Size mismatch detected, attempt resize ...");
+			((at.pcgamingfreaks.Minepacks.Bukkit.Backpack) bp).checkResize();
+			if (bp.getSize() != sizeShouldBe)
+			{
+				commandSender.sendMessage("Failed to resize backpack.");
+			}
+			else
+			{
+				commandSender.sendMessage("Resized backpack successfully.");
+			}
+		}
+		if (bp != null && bp.getSize() != bp.getInventory().getSize())
+		{
+			commandSender.sendMessage("Inventory size does not match backpack size!");
+		}
+		commandSender.sendMessage("Player class: " + player.getClass().getName());
+		commandSender.sendMessage("###############################");
+	}
+
 	@Override
 	public List<String> tabComplete(@NotNull CommandSender commandSender, @NotNull String mainCommandAlias, @NotNull String alias, @NotNull String[] args)
 	{
+		if (args.length == 1)
+		{
+			List<String> completeList = new ArrayList<>(3);
+			if ("size".startsWith(args[0])) { completeList.add("size"); }
+			if ("system".startsWith(args[0])) { completeList.add("system"); }
+			if ("permissions".startsWith(args[0])) { completeList.add("permissions"); }
+			return completeList;
+		}
+		else if (args.length == 2)
+		{
+			return Utils.getPlayerNamesStartingWith(args[args.length - 1], null);
+		}
 		return null;
 	}
 

Minepacks/src/at/pcgamingfreaks/Minepacks/Bukkit/Command/OpenCommand.java

@@ -64,42 +64,52 @@ public class OpenCommand extends MinepacksCommand
 		Player player = (Player) sender;
 		if(args.length == 0)
 		{
-			if(getMinepacksPlugin().isPlayerGameModeAllowed(player))
-			{
-				if(plugin.getCooldownManager() != null && !player.hasPermission(Permissions.NO_COOLDOWN))
-				{
-					long cd = plugin.getCooldownManager().getRemainingCooldown(player);
-					if(cd > 0)
-					{
-						TimeSpan ts = TimeSpan.fromMilliseconds(cd);
-						messageCooldown.send(sender, cd / 1000f, ts.toString());
-						return;
-					}
-					plugin.getCooldownManager().setCooldown(player);
-				}
-				plugin.openBackpack(player, player, true);
-			}
-			else
-			{
-				messageWrongGameMode.send(player, player.getGameMode());
-			}
+			executeSelf(player);
 		}
 		else
 		{
-			if(player.hasPermission(Permissions.OTHERS))
+			executeOther(player, args[0]);
+		}
+	}
+
+	void executeSelf(Player player)
+	{
+		if(getMinepacksPlugin().isPlayerGameModeAllowed(player))
+		{
+			if(plugin.getCooldownManager() != null && !player.hasPermission(Permissions.NO_COOLDOWN))
 			{
-				OfflinePlayer target = Bukkit.getPlayer(args[0]);
-				if(target == null)
+				long cd = plugin.getCooldownManager().getRemainingCooldown(player);
+				if(cd > 0)
 				{
-					//noinspection deprecation
-					target = Bukkit.getOfflinePlayer(args[0]);
+					TimeSpan ts = TimeSpan.fromMilliseconds(cd);
+					messageCooldown.send(player, cd / 1000f, ts.toString());
+					return;
 				}
-				plugin.openBackpack(player, target, player.hasPermission(Permissions.OTHERS_EDIT));
+				plugin.getCooldownManager().setCooldown(player);
 			}
-			else
+			plugin.openBackpack(player, player, true);
+		}
+		else
+		{
+			messageWrongGameMode.send(player, player.getGameMode());
+		}
+	}
+
+	void executeOther(Player player, String name)
+	{
+		if(player.hasPermission(Permissions.OTHERS))
+		{
+			OfflinePlayer target = Bukkit.getPlayer(name);
+			if(target == null)
 			{
-				plugin.messageNoPermission.send(player);
+				//noinspection deprecation
+				target = Bukkit.getOfflinePlayer(name);
 			}
+			plugin.openBackpack(player, target, player.hasPermission(Permissions.OTHERS_EDIT));
+		}
+		else
+		{
+			plugin.messageNoPermission.send(player);
 		}
 	}
 

Minepacks/src/at/pcgamingfreaks/Minepacks/Bukkit/Command/RestoreCommand.java

@@ -1,5 +1,5 @@
 /*
- *   Copyright (C) 2020 GeorgH93
+ *   Copyright (C) 2023 GeorgH93
  *
  *   This program is free software: you can redistribute it and/or modify
  *   it under the terms of the GNU General Public License as published by
@@ -20,8 +20,6 @@ package at.pcgamingfreaks.Minepacks.Bukkit.Command;
 import at.pcgamingfreaks.Bukkit.Message.Message;
 import at.pcgamingfreaks.Command.HelpData;
 import at.pcgamingfreaks.Message.MessageClickEvent;
-import at.pcgamingfreaks.Minepacks.Bukkit.API.Backpack;
-import at.pcgamingfreaks.Minepacks.Bukkit.API.Callback;
 import at.pcgamingfreaks.Minepacks.Bukkit.API.MinepacksCommand;
 import at.pcgamingfreaks.Minepacks.Bukkit.Minepacks;
 import at.pcgamingfreaks.Minepacks.Bukkit.Permissions;
@@ -114,17 +112,10 @@ public class RestoreCommand extends MinepacksCommand
 				messageNoUserFound.send(sender);
 				return;
 			}
-			getMinepacksPlugin().getBackpack(target, new Callback<Backpack>() {
-				@Override
-				public void onResult(Backpack backpack)
-				{
-					backpack.getInventory().setContents(items);
-					backpack.setChanged();
-					messageRestored.send(sender);
-				}
-
-				@Override
-				public void onFail() {}
+			getMinepacksPlugin().getBackpack(target, backpack -> {
+				backpack.getInventory().setContents(items);
+				backpack.setChanged();
+				messageRestored.send(sender);
 			});
 		}
 		else
@@ -133,21 +124,32 @@ public class RestoreCommand extends MinepacksCommand
 		}
 	}
 
-	private void listBackups(final @NotNull CommandSender sender, final @NotNull String mainCommandAlias, final @NotNull String alias, final @NotNull String[] args)
+	private int parsePageNr(final @NotNull CommandSender sender, final @NotNull String[] args)
 	{
-		int page = 0;
 		if(args.length == 2)
 		{
 			try
 			{
-				page = StringUtils.parsePageNumber(args[1]);
+				return StringUtils.parsePageNumber(args[1]);
 			}
 			catch(NumberFormatException ignored)
 			{
 				((Minepacks) getMinepacksPlugin()).messageNotANumber.send(sender);
-				return;
 			}
 		}
+		return 0;
+	}
+
+	private String formatUUID(final @NotNull String uuidString)
+	{
+		if (uuidString.contains("-")) return uuidString;
+		return uuidString.replaceAll("(\\w{8})(\\w{4})(\\w{4})(\\w{4})(\\w{12})", "$1-$2-$3-$4-$5");
+	}
+
+	private void listBackups(final @NotNull CommandSender sender, final @NotNull String mainCommandAlias, final @NotNull String alias, final @NotNull String[] args)
+	{
+		int page = parsePageNr(sender, args);
+
 		ArrayList<String> backups = ((Minepacks) getMinepacksPlugin()).getDatabase().getBackups();
 		int pages = backups.size() / elementsPerPage + 1;
 		page = Math.min(page, pages - 1);
@@ -158,7 +160,7 @@ public class RestoreCommand extends MinepacksCommand
 		{
 			String backup = backups.get(offset++), uuid = "No UUID", date = "Unknown";
 			String[] components = backup.split("_");
-			if(components.length == 3) uuid = (components[1].contains("-")) ? components[1] : components[1].replaceAll("(\\w{8})(\\w{4})(\\w{4})(\\w{4})(\\w{12})", "$1-$2-$3-$4-$5");
+			if(components.length == 3) uuid = formatUUID(components[1]);
 			if(dateFormat != null && (components.length == 2 || components.length == 3))
 			{
 				try

Minepacks/src/at/pcgamingfreaks/Minepacks/Bukkit/Command/ShortcutCommand.java

@@ -0,0 +1,68 @@
+/*
+ *   Copyright (C) 2023 GeorgH93
+ *
+ *   This program is free software: you can redistribute it and/or modify
+ *   it under the terms of the GNU General Public License as published by
+ *   the Free Software Foundation, either version 3 of the License, or
+ *   (at your option) any later version.
+ *
+ *   This program is distributed in the hope that it will be useful,
+ *   but WITHOUT ANY WARRANTY; without even the implied warranty of
+ *   MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
+ *   GNU General Public License for more details.
+ *
+ *   You should have received a copy of the GNU General Public License
+ *   along with this program. If not, see <http://www.gnu.org/licenses/>.
+ */
+
+package at.pcgamingfreaks.Minepacks.Bukkit.Command;
+
+import at.pcgamingfreaks.Command.HelpData;
+import at.pcgamingfreaks.Minepacks.Bukkit.API.MinepacksCommand;
+import at.pcgamingfreaks.Minepacks.Bukkit.Listener.ItemShortcut;
+import at.pcgamingfreaks.Minepacks.Bukkit.Minepacks;
+import at.pcgamingfreaks.Minepacks.Bukkit.Permissions;
+
+import org.bukkit.Bukkit;
+import org.bukkit.command.CommandSender;
+import org.bukkit.entity.Player;
+import org.jetbrains.annotations.NotNull;
+import org.jetbrains.annotations.Nullable;
+
+import java.util.List;
+
+public class ShortcutCommand extends MinepacksCommand
+{
+	private final ItemShortcut itemShortcut;
+
+	public ShortcutCommand(Minepacks plugin, final @NotNull ItemShortcut itemShortcut)
+	{
+		super(plugin, "shortcut", ""/*plugin.getLanguage().getTranslated("Commands.Description.Shortcut")*/, Permissions.USE, true, plugin.getLanguage().getCommandAliases("Shortcut"));
+		this.itemShortcut = itemShortcut;
+	}
+
+	@Override
+	public void execute(@NotNull CommandSender sender, @NotNull String mainCommandAlias, @NotNull String alias, @NotNull String[] args)
+	{
+		if (args.length == 1 && sender.hasPermission(Permissions.OTHERS))
+		{
+			Player p = Bukkit.getPlayer(args[0]);
+			if (p != null && p.hasPermission(Permissions.USE)) itemShortcut.addItem(p);
+		}
+		else
+		{
+			itemShortcut.addItem((Player) sender);
+		}
+	}
+
+	@Override
+	public List<String> tabComplete(@NotNull CommandSender sender, @NotNull String mainCommandAlias, @NotNull String alias, @NotNull String[] args)
+	{
+		return null;
+	}
+	@Override
+	public @Nullable List<HelpData> getHelp(@NotNull CommandSender requester)
+	{
+		return null;
+	}
+}

Minepacks/src/at/pcgamingfreaks/Minepacks/Bukkit/Command/SortCommand.java

@@ -1,5 +1,5 @@
 /*
- *   Copyright (C) 2020 GeorgH93
+ *   Copyright (C) 2023 GeorgH93
  *
  *   This program is free software: you can redistribute it and/or modify
  *   it under the terms of the GNU General Public License as published by
@@ -18,8 +18,6 @@
 package at.pcgamingfreaks.Minepacks.Bukkit.Command;
 
 import at.pcgamingfreaks.Bukkit.Message.Message;
-import at.pcgamingfreaks.Minepacks.Bukkit.API.Backpack;
-import at.pcgamingfreaks.Minepacks.Bukkit.API.Callback;
 import at.pcgamingfreaks.Minepacks.Bukkit.API.MinepacksCommand;
 import at.pcgamingfreaks.Minepacks.Bukkit.Database.Helper.InventoryCompressor;
 import at.pcgamingfreaks.Minepacks.Bukkit.Minepacks;
@@ -44,24 +42,16 @@ public class SortCommand extends MinepacksCommand
 	public void execute(final @NotNull CommandSender commandSender, @NotNull String mainCommandAlias, @NotNull String alias, @NotNull String[] args)
 	{
 		final Player player = (Player) commandSender;
-		getMinepacksPlugin().getBackpack(player, new Callback<Backpack>()
-		{
-			@Override
-			public void onResult(Backpack backpack)
+		getMinepacksPlugin().getBackpack(player, backpack -> {
+			InventoryCompressor compressor = new InventoryCompressor(backpack.getInventory().getContents());
+			if(!compressor.sort().isEmpty())
 			{
-				InventoryCompressor compressor = new InventoryCompressor(backpack.getInventory().getContents());
-				if(!compressor.sort().isEmpty())
-				{
-					plugin.getLogger().warning("Failed to sort backpack!"); //this should not happen
-					return;
-				}
-				backpack.getInventory().setContents(compressor.getTargetStacks());
-				backpack.setChanged();
-				messageSorted.send(player);
+				plugin.getLogger().warning("Failed to sort backpack!"); //this should not happen
+				return;
 			}
-
-			@Override
-			public void onFail() {} // This should not happen
+			backpack.getInventory().setContents(compressor.getTargetStacks());
+			backpack.setChanged();
+			messageSorted.send(player);
 		});
 	}
 

Minepacks/src/at/pcgamingfreaks/Minepacks/Bukkit/CooldownManager.java

@@ -1,5 +1,5 @@
 /*
- *   Copyright (C) 2019 GeorgH93
+ *   Copyright (C) 2023 GeorgH93
  *
  *   This program is free software: you can redistribute it and/or modify
  *   it under the terms of the GNU General Public License as published by
@@ -17,7 +17,6 @@
 
 package at.pcgamingfreaks.Minepacks.Bukkit;
 
-import at.pcgamingfreaks.Minepacks.Bukkit.API.Callback;
 import org.bukkit.entity.Player;
 import org.bukkit.event.EventHandler;
 import org.bukkit.event.EventPriority;
@@ -91,16 +90,7 @@ public class CooldownManager extends BukkitRunnable implements Listener
 		{
 			final UUID uuid = event.getPlayer().getUniqueId();
 			cooldowns.put(uuid, System.currentTimeMillis() + cooldown); // Temporary cooldown till the data is loaded from the database
-			plugin.getDatabase().getCooldown(event.getPlayer(), new Callback<Long>() {
-				@Override
-				public void onResult(Long dbCooldownTime)
-				{
-					cooldowns.put(uuid, dbCooldownTime);
-				}
-
-				@Override
-				public void onFail() {}
-			});
+			plugin.getDatabase().getCooldown(event.getPlayer(), dbCooldownTime -> cooldowns.put(uuid, dbCooldownTime));
 		}
 		else if(addOnJoin)
 		{

Minepacks/src/at/pcgamingfreaks/Minepacks/Bukkit/Database/Config.java

@@ -1,5 +1,5 @@
 /*
- *   Copyright (C) 2023 GeorgH93
+ *   Copyright (C) 2024 GeorgH93
  *
  *   This program is free software: you can redistribute it and/or modify
  *   it under the terms of the GNU General Public License as published by
@@ -12,7 +12,7 @@
  *   GNU General Public License for more details.
  *
  *   You should have received a copy of the GNU General Public License
- *   along with this program. If not, see <https://www.gnu.org/licenses/>.
+ *   along with this program. If not, see <http://www.gnu.org/licenses/>.
  */
 
 package at.pcgamingfreaks.Minepacks.Bukkit.Database;
@@ -31,11 +31,13 @@ import at.pcgamingfreaks.Minepacks.Bukkit.Minepacks;
 import at.pcgamingfreaks.Minepacks.Bukkit.ShrinkApproach;
 import at.pcgamingfreaks.Minepacks.MagicValues;
 import at.pcgamingfreaks.Version;
+
 import org.bukkit.*;
 import org.jetbrains.annotations.NotNull;
 
 import java.io.FileNotFoundException;
 import java.util.*;
+import java.util.logging.Level;
 
 public class Config extends Configuration implements DatabaseConnectionConfiguration, ILanguageConfiguration
 {
@@ -97,7 +99,7 @@ public class Config extends Configuration implements DatabaseConnectionConfigura
 		}
 		catch(FileNotFoundException e)
 		{
-			e.printStackTrace();
+			logger.log(Level.SEVERE, "Failed to set database type", e);
 		}
 	}
 
@@ -131,7 +133,7 @@ public class Config extends Configuration implements DatabaseConnectionConfigura
 				Boolean detectedOnlineMode = Utils.getBungeeOrVelocityOnlineMode();
 				if (detectedOnlineMode != null)
 				{
-					logger.info("Detected online mode in paper config: " + detectedOnlineMode);
+					logger.log(Level.INFO, "Detected online mode in paper config: {0}", detectedOnlineMode);
 					return detectedOnlineMode;
 				}
 				logger.warning("When using BungeeCord please make sure to set the UUID_Type config option explicitly!");
@@ -169,12 +171,12 @@ public class Config extends Configuration implements DatabaseConnectionConfigura
 
 	public String getBPTitleOther()
 	{
-		return ChatColor.translateAlternateColorCodes('&', getConfigE().getString("BackpackTitleOther", "{OwnerName} Backpack").replaceAll("%", "%%").replaceAll("\\{OwnerName}", "%s"));
+		return ChatColor.translateAlternateColorCodes('&', getConfigE().getString("BackpackTitleOther", "{OwnerName} Backpack").replace("%", "%%").replace("{OwnerName}", "%s"));
 	}
 
 	public String getBPTitle()
 	{
-		return ChatColor.translateAlternateColorCodes('&', getConfigE().getString("BackpackTitle", "Backpack"));
+		return ChatColor.translateAlternateColorCodes('&', getConfigE().getString("BackpackTitle", "Backpack").replace("%", "%%").replace("{OwnerName}", "%s"));
 	}
 
 	public boolean getDropOnDeath()
@@ -208,7 +210,7 @@ public class Config extends Configuration implements DatabaseConnectionConfigura
 		}
 		catch(IllegalArgumentException ignored)
 		{
-			logger.warning("Unknown ShrinkApproach \"" + approach + "\"!");
+			logger.log(Level.WARNING, "Unknown ShrinkApproach \"{0}\"!", approach);
 			return ShrinkApproach.SORT;
 		}
 	}
@@ -226,7 +228,7 @@ public class Config extends Configuration implements DatabaseConnectionConfigura
 		{
 			return channel;
 		}
-		else logger.info("Unknown update Channel: " + channel);
+		else logger.log(Level.INFO, "Unknown update Channel: {0}", channel);
 		return null;
 	}
 
@@ -305,7 +307,7 @@ public class Config extends Configuration implements DatabaseConnectionConfigura
 				gameModes.add(gm);
 			}
 		}
-		if(gameModes.size() < 1)
+		if(gameModes.isEmpty())
 		{
 			logger.info("No game-mode's allowed, allowing: " + GameMode.SURVIVAL.name());
 			gameModes.add(GameMode.SURVIVAL);
@@ -329,11 +331,13 @@ public class Config extends Configuration implements DatabaseConnectionConfigura
 		return getConfigE().getDouble("FullInventory.CollectRadius", 1.5); // in blocks
 	}
 
-	public boolean isFullInvToggleAllowed() {
+	public boolean isFullInvToggleAllowed()
+	{
 		return getConfigE().getBoolean("FullInventory.IsToggleAllowed", false);
 	}
 
-	public boolean isFullInvEnabledOnJoin() {
+	public boolean isFullInvEnabledOnJoin()
+	{
 		return getFullInvCollect();
 	}
 	//endregion
@@ -514,14 +518,17 @@ public class Config extends Configuration implements DatabaseConnectionConfigura
 		return null;
 	}
 
+	private static final @NotNull String DEFAULT_SOUND_OPEN = MCVersion.isNewerOrEqualThan(MCVersion.MC_1_11) ? "BLOCK_SHULKER_BOX_OPEN" : (MCVersion.isNewerOrEqualThan(MCVersion.MC_1_9_2) ? "BLOCK_CHEST_OPEN" : "CHEST_OPEN");
+	private static final @NotNull String DEFAULT_SOUND_CLOSE = MCVersion.isNewerOrEqualThan(MCVersion.MC_1_11) ? "BLOCK_SHULKER_BOX_CLOSE" : (MCVersion.isNewerOrEqualThan(MCVersion.MC_1_9_2) ? "BLOCK_CHEST_CLOSE" : "CHEST_CLOSE");
+
 	public Sound getOpenSound()
 	{
-		return getSound("OpenSound", MCVersion.isNewerOrEqualThan(MCVersion.MC_1_11) ? "BLOCK_SHULKER_BOX_OPEN" : (MCVersion.isNewerOrEqualThan(MCVersion.MC_1_9_2) ? "BLOCK_CHEST_OPEN" : "CHEST_OPEN"));
+		return getSound("OpenSound", DEFAULT_SOUND_OPEN);
 	}
 
 	public Sound getCloseSound()
 	{
-		return getSound("CloseSound", MCVersion.isNewerOrEqualThan(MCVersion.MC_1_11) ? "BLOCK_SHULKER_BOX_CLOSE" : (MCVersion.isNewerOrEqualThan(MCVersion.MC_1_9_2) ? "BLOCK_CHEST_CLOSE" : "CHEST_CLOSE"));
+		return getSound("CloseSound", DEFAULT_SOUND_CLOSE);
 	}
 	//endregion
 

Minepacks/src/at/pcgamingfreaks/Minepacks/Bukkit/Database/Database.java

@@ -121,14 +121,14 @@ public abstract class Database implements Listener
 		catch(IllegalStateException ignored) {}
 		catch(Exception e)
 		{
-			e.printStackTrace();
+			plugin.getLogger().log(Level.SEVERE, "Failed to initialize database.", e);
 		}
 		return null;
 	}
 
 	public void backup(@NotNull Backpack backpack)
 	{
-		writeBackup(backpack.getOwner().getName(), getPlayerFormattedUUID(backpack.getOwner()), itsSerializer.getUsedSerializer(), itsSerializer.serialize(backpack.getInventory()));
+		writeBackup(backpack.getOwner().getName(), getPlayerFormattedUUID(backpack.getOwnerId()), itsSerializer.getUsedSerializer(), itsSerializer.serialize(backpack.getInventory()));
 	}
 
 	protected void writeBackup(@Nullable String userName, @NotNull String userIdentifier, final int usedSerializer, final byte[] data)
@@ -172,7 +172,12 @@ public abstract class Database implements Listener
 
 	protected String getPlayerFormattedUUID(OfflinePlayer player)
 	{
-		return (useUUIDSeparators) ? player.getUniqueId().toString() : player.getUniqueId().toString().replace("-", "");
+		return getPlayerFormattedUUID(player.getUniqueId());
+	}
+
+	protected String getPlayerFormattedUUID(UUID uuid)
+	{
+		return (useUUIDSeparators) ? uuid.toString() : uuid.toString().replace("-", "");
 	}
 
 	public @NotNull Collection<Backpack> getLoadedBackpacks()
@@ -238,7 +243,6 @@ public abstract class Database implements Listener
 
 	public void unloadBackpack(Backpack backpack)
 	{
-		plugin.getLogger().log(Level.INFO, "Unloading backpack of {0} ({1})", new Object[] {backpack.getOwner().getName(), backpack.getOwner().getUniqueId()});
 		if (forceSaveOnUnload)
 		{
 			backpack.forceSave();
@@ -247,7 +251,7 @@ public abstract class Database implements Listener
 		{
 			backpack.save();
 		}
-		backpacks.remove(backpack.getOwner().getUniqueId());
+		backpacks.remove(backpack.getOwnerId());
 	}
 
 	public void asyncLoadBackpack(final OfflinePlayer player)

Minepacks/src/at/pcgamingfreaks/Minepacks/Bukkit/Database/Files.java

@@ -1,5 +1,5 @@
 /*
- *   Copyright (C) 2020 GeorgH93
+ *   Copyright (C) 2024 GeorgH93
  *
  *   This program is free software: you can redistribute it and/or modify
  *   it under the terms of the GNU General Public License as published by
@@ -12,7 +12,7 @@
  *   GNU General Public License for more details.
  *
  *   You should have received a copy of the GNU General Public License
- *   along with this program. If not, see <https://www.gnu.org/licenses/>.
+ *   along with this program. If not, see <http://www.gnu.org/licenses/>.
  */
 
 package at.pcgamingfreaks.Minepacks.Bukkit.Database;
@@ -20,7 +20,8 @@ package at.pcgamingfreaks.Minepacks.Bukkit.Database;
 import at.pcgamingfreaks.Minepacks.Bukkit.API.Callback;
 import at.pcgamingfreaks.Minepacks.Bukkit.Backpack;
 import at.pcgamingfreaks.Minepacks.Bukkit.Minepacks;
-import at.pcgamingfreaks.UUIDConverter;
+import at.pcgamingfreaks.UUID.UuidConverter;
+
 import org.bukkit.OfflinePlayer;
 import org.bukkit.entity.Player;
 import org.bukkit.inventory.ItemStack;
@@ -30,6 +31,8 @@ import org.jetbrains.annotations.Nullable;
 import java.io.File;
 import java.io.FileInputStream;
 import java.io.FileOutputStream;
+import java.util.UUID;
+import java.util.logging.Level;
 import java.util.logging.Logger;
 
 public class Files extends Database
@@ -37,10 +40,12 @@ public class Files extends Database
 	public static final String EXT = ".backpack", EXT_REGEX = "\\.backpack", FOLDER_NAME = "backpacks";
 
 	private final File saveFolder;
+	private final UuidConverter converter;
 	
 	public Files(Minepacks plugin)
 	{
 		super(plugin);
+		converter = new UuidConverter(plugin.getLogger());
 		maxAge *= 24 * 3600000L;
 		saveFolder = new File(this.plugin.getDataFolder(), FOLDER_NAME);
 		if(!saveFolder.exists())
@@ -62,64 +67,67 @@ public class Files extends Database
 		// Files are stored with the users name or the uuid, there is no reason to update anything
 	}
 
+	private String getUuidFromFileName(String fileName)
+	{
+		String name = fileName.substring(0, fileName.length() - EXT.length());
+		UUID uuid = (onlineUUIDs) ? converter.getUUID(name, true) : UuidConverter.getOfflineModeUUID(name);
+		return getPlayerFormattedUUID(uuid);
+	}
+
+	private void tryRename(File file, File newFileName)
+	{
+		if (!file.renameTo(newFileName))
+		{
+			plugin.getLogger().log(Level.WARNING, () -> "Failed to rename file (" + file.getAbsolutePath() + " to " + newFileName.getAbsolutePath() + ").");
+		}
+	}
+
 	private void checkFiles()
 	{
 		File[] allFiles = saveFolder.listFiles((dir, name) -> name.endsWith(EXT));
 		if(allFiles == null) return;
 		for (File file : allFiles)
 		{
-			if(maxAge > 0 && System.currentTimeMillis() - file.lastModified() > maxAge) // Check if the file is older then x days
+			if(maxAge > 0 && System.currentTimeMillis() - file.lastModified() > maxAge) // Check if the file is older than x days
 			{
 				if(!file.delete())
 				{
 					plugin.getLogger().warning("Failed to delete file (" + file.getAbsolutePath() + ").");
 				}
-				continue; // We don't have to check if the file name is correct cause we have the deleted the file
+				continue; // We don't have to check if the file name is correct because we have the deleted the file
 			}
 			int len = file.getName().length() - EXT.length();
 			if(len <= 16) // It's a player name
 			{
-				if(!file.renameTo(new File(saveFolder, UUIDConverter.getUUIDFromName(file.getName().substring(0, len), onlineUUIDs, useUUIDSeparators) + EXT)))
-				{
-					plugin.getLogger().warning("Failed to rename file (" + file.getAbsolutePath() + ").");
-				}
+				tryRename(file, new File(saveFolder, getUuidFromFileName(file.getName()) + EXT));
 			}
-			else // It's an UUID
+			else // It's a UUID
 			{
 				if(file.getName().contains("-"))
 				{
 					if(!useUUIDSeparators)
 					{
-						if(!file.renameTo(new File(saveFolder, file.getName().replaceAll("-", ""))))
-						{
-							plugin.getLogger().warning("Failed to rename file (" + file.getAbsolutePath() + ").");
-						}
+						tryRename(file, new File(saveFolder, file.getName().replaceAll("-", "")));
 					}
 				}
-				else
+				else if(useUUIDSeparators)
 				{
-					if(useUUIDSeparators)
-					{
-						if(!file.renameTo(new File(saveFolder, file.getName().replaceAll("(\\w{8})(\\w{4})(\\w{4})(\\w{4})(\\w{12})" + EXT_REGEX, "$1-$2-$3-$4-$5" + EXT))))
-						{
-							plugin.getLogger().warning("Failed to rename file (" + file.getAbsolutePath() + ").");
-						}
-					}
+					tryRename(file, new File(saveFolder, file.getName().replaceAll("(\\w{8})(\\w{4})(\\w{4})(\\w{4})(\\w{12})" + EXT_REGEX, "$1-$2-$3-$4-$5" + EXT)));
 				}
 			}
 		}
 	}
 	
-	private String getFileName(OfflinePlayer player)
+	private String getFileName(UUID uuid)
 	{
-		return getPlayerFormattedUUID(player) + EXT;
+		return getPlayerFormattedUUID(uuid) + EXT;
 	}
 	
 	// DB Functions
 	@Override
 	public void saveBackpack(Backpack backpack)
 	{
-		File save = new File(saveFolder, getFileName(backpack.getOwner()));
+		File save = new File(saveFolder, getFileName(backpack.getOwnerId()));
 		try(FileOutputStream fos = new FileOutputStream(save))
 		{
 			fos.write(itsSerializer.getUsedSerializer());
@@ -128,14 +136,14 @@ public class Files extends Database
 		}
 		catch(Exception e)
 		{
-			e.printStackTrace();
+			plugin.getLogger().log(Level.SEVERE, "Failed to save backpack.", e);
 		}
 	}
 
 	@Override
 	protected void loadBackpack(final OfflinePlayer player, final Callback<Backpack> callback)
 	{ //TODO this needs to be done async!
-		File save = new File(saveFolder, getFileName(player));
+		File save = new File(saveFolder, getFileName(player.getUniqueId()));
 		ItemStack[] itemStacks = readFile(itsSerializer, save, plugin.getLogger());
 		if(itemStacks != null)
 		{
@@ -161,7 +169,7 @@ public class Files extends Database
 			}
 			catch(Exception e)
 			{
-				e.printStackTrace();
+				logger.log(Level.WARNING, "Failed to read backpack.", e);
 			}
 		}
 		return null;

Minepacks/src/at/pcgamingfreaks/Minepacks/Bukkit/Database/InventorySerializer.java

@@ -20,10 +20,12 @@ package at.pcgamingfreaks.Minepacks.Bukkit.Database;
 import at.pcgamingfreaks.Bukkit.ItemStackSerializer.ItemStackSerializer;
 import at.pcgamingfreaks.Bukkit.MCVersion;
 import at.pcgamingfreaks.ConsoleColor;
+
 import lombok.Getter;
 import org.bukkit.inventory.Inventory;
 import org.bukkit.inventory.ItemStack;
 
+import java.util.logging.Level;
 import java.util.logging.Logger;
 
 public class InventorySerializer
@@ -49,13 +51,16 @@ public class InventorySerializer
 		}
 		catch(Exception e)
 		{
-			e.printStackTrace();
+			logger.log(Level.SEVERE, "Failed to produce serializer!", e);
 		}
 		if(serializer == null)
 		{
 			logger.severe("NBTItemStackSerializer does not support your Minecraft version!\nFalling back to BukkitItemStackSerializer! This most likely is wrong!");
-			usedSerializer = 0;
-			serializer = BUKKIT_ITEM_STACK_SERIALIZER;
+			if (MCVersion.isOlderThan(MCVersion.MC_NMS_1_8_R1))
+			{
+				usedSerializer = 0;
+				serializer = BUKKIT_ITEM_STACK_SERIALIZER;
+			}
 		}
 		this.serializer = serializer;
 		this.usedSerializer = usedSerializer;

Minepacks/src/at/pcgamingfreaks/Minepacks/Bukkit/Database/MySQL.java

@@ -38,7 +38,7 @@ public class MySQL extends SQL
 	protected void updateQueriesForDialect()
 	{
 		queryDeleteOldBackpacks = "DELETE FROM {TableBackpacks} WHERE {FieldBPLastUpdate} + INTERVAL {VarMaxAge} day < NOW()";
-		queryUpdateBp = queryUpdateBp.replaceAll("\\{NOW}", "NOW()");
+		queryUpdateBp = queryUpdateBp.replace("{NOW}", "NOW()");
 	}
 
 	@Override

Minepacks/src/at/pcgamingfreaks/Minepacks/Bukkit/Database/SQL.java

@@ -1,5 +1,5 @@
 /*
- *   Copyright (C) 2023 GeorgH93
+ *   Copyright (C) 2024 GeorgH93
  *
  *   This program is free software: you can redistribute it and/or modify
  *   it under the terms of the GNU General Public License as published by
@@ -32,6 +32,7 @@ import org.intellij.lang.annotations.Language;
 import org.jetbrains.annotations.NotNull;
 
 import java.sql.*;
+import java.util.logging.Level;
 
 public abstract class SQL extends Database
 {
@@ -64,7 +65,7 @@ public abstract class SQL extends Database
 			}
 			catch(SQLException e)
 			{
-				e.printStackTrace();
+				plugin.getLogger().log(Level.SEVERE, "Failed to delete old backpacks! Error: {0}", e.getMessage());
 			}
 		}
 		// Delete old cooldowns
@@ -72,11 +73,11 @@ public abstract class SQL extends Database
 		{
 			try(Connection connection = getConnection())
 			{
-				DBTools.runStatement(connection, queryDeleteOldCooldowns, System.currentTimeMillis());
+				DBTools.runStatement(connection, queryDeleteOldCooldowns, new Timestamp(System.currentTimeMillis()));
 			}
 			catch(SQLException e)
 			{
-				e.printStackTrace();
+				plugin.getLogger().log(Level.SEVERE, "Failed to delete cooldowns! Error: {0}", e.getMessage());
 			}
 		}
 	}
@@ -115,7 +116,7 @@ public abstract class SQL extends Database
 		}
 		catch(SQLException e)
 		{
-			e.printStackTrace();
+			plugin.getLogger().log(Level.SEVERE, "Failed to check UUIDs! Error: {0}", e.getMessage());
 		}
 	}
 
@@ -163,10 +164,10 @@ public abstract class SQL extends Database
 	protected String replacePlaceholders(@Language("SQL") String query)
 	{
 		return query.replaceAll("(\\{\\w+})", "`$1`").replaceAll("`(\\{\\w+})`_(\\w+)", "`$1_$2`").replaceAll("fk_`(\\{\\w+})`_`(\\{\\w+})`_`(\\{\\w+})`", "`fk_$1_$2_$3`") // Fix name formatting
-				.replaceAll("\\{TablePlayers}", tablePlayers).replaceAll("\\{FieldName}", fieldPlayerName).replaceAll("\\{FieldUUID}", fieldPlayerUUID).replaceAll("\\{FieldPlayerID}", fieldPlayerID) // Players
-				.replaceAll("\\{TableBackpacks}", tableBackpacks).replaceAll("\\{FieldBPOwner}", fieldBpOwner).replaceAll("\\{FieldBPITS}", fieldBpIts) // Backpacks
-				.replaceAll("\\{FieldBPVersion}", fieldBpVersion).replaceAll("\\{FieldBPLastUpdate}", fieldBpLastUpdate) // Backpacks
-				.replaceAll("\\{TableCooldowns}", tableCooldowns).replaceAll("\\{FieldCDPlayer}", fieldCdPlayer).replaceAll("\\{FieldCDTime}", fieldCdTime); // Cooldowns
+				.replace("{TablePlayers}", tablePlayers).replace("{FieldName}", fieldPlayerName).replace("{FieldUUID}", fieldPlayerUUID).replace("{FieldPlayerID}", fieldPlayerID) // Players
+				.replace("{TableBackpacks}", tableBackpacks).replace("{FieldBPOwner}", fieldBpOwner).replace("{FieldBPITS}", fieldBpIts) // Backpacks
+				.replace("{FieldBPVersion}", fieldBpVersion).replace("{FieldBPLastUpdate}", fieldBpLastUpdate) // Backpacks
+				.replace("{TableCooldowns}", tableCooldowns).replace("{FieldCDPlayer}", fieldCdPlayer).replace("{FieldCDTime}", fieldCdTime); // Cooldowns
 	}
 
 	protected void runStatementAsync(final String query, final Object... args)
@@ -186,8 +187,7 @@ public abstract class SQL extends Database
 		}
 		catch(SQLException e)
 		{
-			plugin.getLogger().severe("Query: " + query);
-			e.printStackTrace();
+			plugin.getLogger().log(Level.SEVERE, e, () -> "Query: " + query);
 		}
 	}
 
@@ -202,8 +202,8 @@ public abstract class SQL extends Database
 	public void saveBackpack(final Backpack backpack)
 	{
 		final byte[] data = itsSerializer.serialize(backpack.getInventory());
-		final int id = backpack.getOwnerID(), usedSerializer = itsSerializer.getUsedSerializer();
-		final String nameOrUUID = getPlayerFormattedUUID(backpack.getOwner()), name = backpack.getOwner().getName();
+		final int id = backpack.getOwnerDatabaseId(), usedSerializer = itsSerializer.getUsedSerializer();
+		final String nameOrUUID = getPlayerFormattedUUID(backpack.getOwnerId()), name = backpack.getOwner().getName();
 
 		Runnable runnable = () -> {
 			try(Connection connection = getConnection())
@@ -219,7 +219,7 @@ public abstract class SQL extends Database
 							{
 								final int newID = rs.getInt(fieldPlayerID);
 								DBTools.runStatement(connection, queryInsertBp, newID, data, usedSerializer);
-								plugin.getServer().getScheduler().runTask(plugin, () -> backpack.setOwnerID(newID));
+								plugin.getServer().getScheduler().runTask(plugin, () -> backpack.setOwnerDatabaseId(newID));
 							}
 							else
 							{
@@ -236,8 +236,7 @@ public abstract class SQL extends Database
 			}
 			catch(SQLException e)
 			{
-				plugin.getLogger().warning("Failed to save backpack in database! Error: " + e.getMessage());
-				e.printStackTrace();
+				plugin.getLogger().log(Level.SEVERE, "Failed to save backpack in database! Error: {0}", e.getMessage());
 				writeBackup(name, nameOrUUID, usedSerializer, data);
 			}
 		};
@@ -289,7 +288,7 @@ public abstract class SQL extends Database
 			}
 			catch(SQLException e)
 			{
-				e.printStackTrace();
+				plugin.getLogger().log(Level.SEVERE, "Failed to load backpack! Error: {0}", e.getMessage());
 				plugin.getServer().getScheduler().runTask(plugin, callback::onFail);
 			}
 		});
@@ -317,7 +316,7 @@ public abstract class SQL extends Database
 			}
 			catch(SQLException e)
 			{
-				e.printStackTrace();
+				plugin.getLogger().log(Level.SEVERE, "Failed to load cooldown! Error: {0}", e.getMessage());
 				plugin.getServer().getScheduler().runTask(plugin, () -> callback.onResult(0L));
 			}
 		});

Minepacks/src/at/pcgamingfreaks/Minepacks/Bukkit/Database/UnCacheStrategies/Interval.java

@@ -37,7 +37,7 @@ public class Interval extends UnCacheStrategy implements Runnable
 	{
 		for(Backpack backpack : cache.getLoadedBackpacks())
 		{
-			if(!backpack.getOwner().isOnline() && !backpack.isOpen())
+			if(backpack.getOwnerPlayer() == null && !backpack.isOpen())
 			{
 				this.cache.unloadBackpack(backpack);
 			}

Minepacks/src/at/pcgamingfreaks/Minepacks/Bukkit/Database/UnCacheStrategies/IntervalChecked.java

@@ -22,6 +22,7 @@ import at.pcgamingfreaks.Minepacks.Bukkit.Database.Database;
 import at.pcgamingfreaks.Minepacks.Bukkit.Minepacks;
 
 import org.bukkit.Bukkit;
+import org.bukkit.OfflinePlayer;
 
 public class IntervalChecked extends UnCacheStrategy implements Runnable
 {
@@ -42,7 +43,8 @@ public class IntervalChecked extends UnCacheStrategy implements Runnable
 		long currentTime = System.currentTimeMillis() - delay;
 		for(Backpack backpack : cache.getLoadedBackpacks())
 		{
-			if(!backpack.getOwner().isOnline() && backpack.getOwner().getLastPlayed() < currentTime && !backpack.isOpen())
+			OfflinePlayer owner = backpack.getOwner();
+			if(!owner.isOnline() && owner.getLastPlayed() < currentTime && !backpack.isOpen())
 			{
 				this.cache.unloadBackpack(backpack);
 			}

Minepacks/src/at/pcgamingfreaks/Minepacks/Bukkit/Database/UnCacheStrategies/UnCacheStrategy.java

@@ -24,7 +24,7 @@ public abstract class UnCacheStrategy
 {
 	protected Database cache;
 
-	public UnCacheStrategy(Database cache)
+	protected UnCacheStrategy(Database cache)
 	{
 		this.cache = cache;
 	}

Minepacks/src/at/pcgamingfreaks/Minepacks/Bukkit/ItemsCollector.java

@@ -64,7 +64,7 @@ public class ItemsCollector extends BukkitRunnable
 		itemFilter = plugin.getItemFilter();
 	}
 
-	private boolean canUseAutoPickup(Player player)
+	public boolean canUseAutoPickup(Player player)
 	{
 		// Check if player can use in world
 		if(plugin.isDisabled(player) != WorldBlacklistMode.None) return false;

Minepacks/src/at/pcgamingfreaks/Minepacks/Bukkit/Listener/BackpackEventListener.java

@@ -55,14 +55,13 @@ public class BackpackEventListener extends MinepacksListener
 				backpack.save();
 			}
 			backpack.close(closer);
-			if(event.getPlayer().getUniqueId().equals(backpack.getOwner().getUniqueId()))
+			if(event.getPlayer().getUniqueId().equals(backpack.getOwnerId()))
 			{
 				messageOwnBackpackClose.send(closer);
 			}
 			else
 			{
-				OfflinePlayer owner = backpack.getOwner();
-				messageOtherBackpackClose.send(closer, owner);
+				messageOtherBackpackClose.send(closer, backpack.getOwner());
 			}
 			if(closeSound != null)
 			{

Minepacks/src/at/pcgamingfreaks/Minepacks/Bukkit/Listener/DropOnDeath.java

@@ -1,5 +1,5 @@
 /*
- *   Copyright (C) 2022 GeorgH93
+ *   Copyright (C) 2023 GeorgH93
  *
  *   This program is free software: you can redistribute it and/or modify
  *   it under the terms of the GNU General Public License as published by
@@ -17,8 +17,6 @@
 
 package at.pcgamingfreaks.Minepacks.Bukkit.Listener;
 
-import at.pcgamingfreaks.Minepacks.Bukkit.API.Backpack;
-import at.pcgamingfreaks.Minepacks.Bukkit.API.Callback;
 import at.pcgamingfreaks.Minepacks.Bukkit.API.Events.BackpackDropOnDeathEvent;
 import at.pcgamingfreaks.Minepacks.Bukkit.API.WorldBlacklistMode;
 import at.pcgamingfreaks.Minepacks.Bukkit.Minepacks;
@@ -48,21 +46,13 @@ public class DropOnDeath extends MinepacksListener
 		if (!player.hasPermission(Permissions.KEEP_ON_DEATH))
 		{
 			final Location location = player.getLocation();
-			plugin.getBackpack(player, new Callback<Backpack>()
-			{
-				@Override
-				public void onResult(Backpack backpack)
+			plugin.getBackpack(player, backpack -> {
+				BackpackDropOnDeathEvent event1 = new BackpackDropOnDeathEvent(player, backpack);
+				plugin.getServer().getPluginManager().callEvent(event1);
+				if(!event1.isCancelled())
 				{
-					BackpackDropOnDeathEvent event = new BackpackDropOnDeathEvent(player, backpack);
-					plugin.getServer().getPluginManager().callEvent(event);
-					if(!event.isCancelled())
-					{
-						backpack.drop(location);
-					}
+					backpack.drop(location);
 				}
-
-				@Override
-				public void onFail() {}
 			});
 		}
 	}

Minepacks/src/at/pcgamingfreaks/Minepacks/Bukkit/Listener/ItemShortcut.java

@@ -91,7 +91,7 @@ public class ItemShortcut extends MinepacksListener
 		return itemDisplayName != null && itemNameNoReset.equals(itemDisplayName.replace(ChatColor.RESET.toString(), ""));
 	}
 
-	private void addItem(Player player)
+	public void addItem(Player player)
 	{
 		if(player.hasPermission(Permissions.USE))
 		{
@@ -120,6 +120,17 @@ public class ItemShortcut extends MinepacksListener
 				player.getInventory().addItem(HeadUtils.fromBase64(value, itemName, MINEPACKS_UUID));
 			}
 		}
+		else
+		{
+			for(ItemStack itemStack : player.getInventory())
+			{
+				if(isItemShortcut(itemStack))
+				{
+					player.getInventory().remove(itemStack);
+					return;
+				}
+			}
+		}
 	}
 
 	private void removeItem(Player player)

Minepacks/src/at/pcgamingfreaks/Minepacks/Bukkit/Minepacks.java

@@ -1,5 +1,5 @@
 /*
- *   Copyright (C) 2023 GeorgH93
+ *   Copyright (C) 2024 GeorgH93
  *
  *   This program is free software: you can redistribute it and/or modify
  *   it under the terms of the GNU General Public License as published by
@@ -27,19 +27,21 @@ import at.pcgamingfreaks.Minepacks.Bukkit.API.Backpack;
 import at.pcgamingfreaks.Minepacks.Bukkit.API.*;
 import at.pcgamingfreaks.Minepacks.Bukkit.Command.CommandManager;
 import at.pcgamingfreaks.Minepacks.Bukkit.Command.InventoryClearCommand;
+import at.pcgamingfreaks.Minepacks.Bukkit.Command.ShortcutCommand;
 import at.pcgamingfreaks.Minepacks.Bukkit.Database.Config;
 import at.pcgamingfreaks.Minepacks.Bukkit.Database.Database;
 import at.pcgamingfreaks.Minepacks.Bukkit.Database.Language;
 import at.pcgamingfreaks.Minepacks.Bukkit.Listener.ItemFilter;
 import at.pcgamingfreaks.Minepacks.Bukkit.Listener.*;
+import at.pcgamingfreaks.Minepacks.Bukkit.Placeholder.PlaceholderManager;
 import at.pcgamingfreaks.Minepacks.Bukkit.SpecialInfoWorker.NoDatabaseWorker;
 import at.pcgamingfreaks.Minepacks.MagicValues;
 import at.pcgamingfreaks.Plugin.IPlugin;
 import at.pcgamingfreaks.ServerType;
-import at.pcgamingfreaks.Util.StringUtils;
 import at.pcgamingfreaks.Updater.UpdateResponseCallback;
+import at.pcgamingfreaks.Util.StringUtils;
 import at.pcgamingfreaks.Version;
-import lombok.Getter;
+
 import org.bukkit.Bukkit;
 import org.bukkit.GameMode;
 import org.bukkit.OfflinePlayer;
@@ -52,6 +54,8 @@ import org.bukkit.plugin.java.JavaPlugin;
 import org.jetbrains.annotations.NotNull;
 import org.jetbrains.annotations.Nullable;
 
+import lombok.Getter;
+
 import java.io.File;
 import java.util.Collection;
 import java.util.Locale;
@@ -59,12 +63,12 @@ import java.util.Set;
 
 public class Minepacks extends JavaPlugin implements MinepacksPlugin, IPlugin
 {
-	private static Minepacks instance = null;
+	@Getter private static Minepacks instance = null;
 
 	private ManagedUpdater updater = null;
 	private Config config;
 	private Language lang;
-	private Database database;
+	@Getter private Database database;
 
 	public Message messageNoPermission, messageInvalidBackpack, messageWorldDisabled, messageNotFromConsole, messageNotANumber;
 
@@ -79,11 +83,7 @@ public class Minepacks extends JavaPlugin implements MinepacksPlugin, IPlugin
 	private ItemFilter itemFilter = null;
 	private Sound openSound = null;
 	private ItemShortcut shortcut = null;
-
-	public static Minepacks getInstance()
-	{
-		return instance;
-	}
+	@Getter private PlaceholderManager placeholderManager = null;
 
 	@Override
 	public boolean isRunningInStandaloneMode()
@@ -123,7 +123,14 @@ public class Minepacks extends JavaPlugin implements MinepacksPlugin, IPlugin
 
 	private boolean checkMcVersion()
 	{
-		if(MCVersion.is(MCVersion.UNKNOWN) || !MCVersion.isUUIDsSupportAvailable() || MCVersion.isNewerThan(MCVersion.MC_NMS_1_20_R1))
+		if ((MCVersion.isNewerThan(MCVersion.MC_NMS_1_20_R3) || MCVersion.is(MCVersion.UNKNOWN)) && ServerType.isPaperCompatible())
+		{
+			getLogger().warning("Minepacks is not yet compatible with Paper 1.20.5 and up!");
+			getLogger().warning("To use this plugin please switch to spigot or wait for an update.");
+			this.setEnabled(false);
+			return false;
+		}
+		if (MCVersion.is(MCVersion.UNKNOWN) || !MCVersion.isUUIDsSupportAvailable() || MCVersion.isNewerThan(MCVersion.MC_NMS_1_20_R4))
 		{
 			this.warnOnVersionIncompatibility();
 			this.setEnabled(false);
@@ -195,6 +202,7 @@ public class Minepacks extends JavaPlugin implements MinepacksPlugin, IPlugin
 		maxSize = config.getBackpackMaxSize();
 		at.pcgamingfreaks.Minepacks.Bukkit.Backpack.setShrinkApproach(config.getShrinkApproach());
 		at.pcgamingfreaks.Minepacks.Bukkit.Backpack.setTitle(config.getBPTitle(), config.getBPTitleOther());
+		at.pcgamingfreaks.Minepacks.Bukkit.Backpack.setMessageBackpackShrunk(lang.getMessage("Ingame.BackpackShrunk"));
 		messageNotFromConsole  = lang.getMessage("NotFromConsole");
 		messageNoPermission    = lang.getMessage("Ingame.NoPermission");
 		messageInvalidBackpack = lang.getMessage("Ingame.InvalidBackpack");
@@ -218,18 +226,21 @@ public class Minepacks extends JavaPlugin implements MinepacksPlugin, IPlugin
 		{
 			shortcut = new ItemShortcut(this);
 			pluginManager.registerEvents(shortcut, this);
+			commandManager.registerSubCommand(new ShortcutCommand(this, shortcut));
 		}
 		else shortcut = null;
 		if(config.isWorldWhitelistMode()) pluginManager.registerEvents(new WorldBlacklistUpdater(this), this);
 		//endregion
 		if(config.getFullInvCollect() || config.isFullInvToggleAllowed()) collector = new ItemsCollector(this);
 		worldBlacklist = config.getWorldBlacklist();
-		worldBlacklistMode = (worldBlacklist.size() == 0) ? WorldBlacklistMode.None : config.getWorldBlockMode();
+		worldBlacklistMode = (worldBlacklist.isEmpty()) ? WorldBlacklistMode.None : config.getWorldBlockMode();
 
 		gameModes = config.getAllowedGameModes();
 		if(config.getCommandCooldown() > 0) cooldownManager = new CooldownManager(this);
 
 		openSound = config.getOpenSound();
+
+		placeholderManager = new PlaceholderManager(this);
 	}
 
 	private void unload()
@@ -240,6 +251,7 @@ public class Minepacks extends JavaPlugin implements MinepacksPlugin, IPlugin
 			inventoryClearCommand.close();
 			inventoryClearCommand = null;
 		}
+		if (placeholderManager != null) { placeholderManager.close(); placeholderManager = null; }
 		if(collector != null) collector.close();
 		if(commandManager != null) commandManager.close();
 		if(collector != null) collector.cancel();
@@ -260,11 +272,9 @@ public class Minepacks extends JavaPlugin implements MinepacksPlugin, IPlugin
 
 	public void warnOnVersionIncompatibility()
 	{
-		String name = Bukkit.getServer().getClass().getPackage().getName();
-		String[] version = name.substring(name.lastIndexOf('.') + 2).split("_");
 		getLogger().warning(ConsoleColor.RED + "################################" + ConsoleColor.RESET);
 		getLogger().warning(ConsoleColor.RED + String.format("Your minecraft version (MC %1$s) is currently not compatible with this plugins version (%2$s). " +
-				                                                     "Please check for updates!", version[0] + "." + version[1], getDescription().getVersion()) + ConsoleColor.RESET);
+				                                                     "Please check for updates!", Bukkit.getServer().getVersion(), getDescription().getVersion()) + ConsoleColor.RESET);
 		getLogger().warning(ConsoleColor.RED + "################################" + ConsoleColor.RESET);
 		Utils.blockThread(5);
 	}
@@ -279,11 +289,6 @@ public class Minepacks extends JavaPlugin implements MinepacksPlugin, IPlugin
 		return lang;
 	}
 
-	public Database getDatabase()
-	{
-		return database;
-	}
-
 	@Override
 	public void openBackpack(@NotNull final Player opener, @NotNull final OfflinePlayer owner, final boolean editable)
 	{
@@ -299,17 +304,7 @@ public class Minepacks extends JavaPlugin implements MinepacksPlugin, IPlugin
 	@Override
 	public void openBackpack(@NotNull Player opener, @NotNull OfflinePlayer owner, boolean editable, @Nullable String title)
 	{
-		database.getBackpack(owner, new Callback<Backpack>()
-		{
-			@Override
-			public void onResult(Backpack backpack)
-			{
-				openBackpack(opener, backpack, editable, title);
-			}
-
-			@Override
-			public void onFail() {}
-		});
+		database.getBackpack(owner, backpack -> openBackpack(opener, backpack, editable, title));
 	}
 
 	@Override
@@ -318,11 +313,8 @@ public class Minepacks extends JavaPlugin implements MinepacksPlugin, IPlugin
 		WorldBlacklistMode disabled = isDisabled(opener);
 		if(disabled != WorldBlacklistMode.None)
 		{
-			switch(disabled)
-			{
-				case Message: messageWorldDisabled.send(opener); break;
-				case MissingPermission: messageNoPermission.send(opener); break;
-			}
+			if (disabled == WorldBlacklistMode.Message) messageWorldDisabled.send(opener);
+			else if (disabled == WorldBlacklistMode.MissingPermission) messageNoPermission.send(opener);
 			return;
 		}
 		if(backpack == null)
@@ -377,7 +369,7 @@ public class Minepacks extends JavaPlugin implements MinepacksPlugin, IPlugin
 	}
 
 	@Override
-	public WorldBlacklistMode isDisabled(Player player)
+	public @NotNull WorldBlacklistMode isDisabled(final @NotNull Player player)
 	{
 		if(worldBlacklistMode == WorldBlacklistMode.None || (worldBlacklistMode != WorldBlacklistMode.NoPlugin && player.hasPermission(Permissions.IGNORE_WORLD_BLACKLIST))) return WorldBlacklistMode.None;
 		if(worldBlacklist.contains(player.getWorld().getName().toLowerCase(Locale.ROOT))) return worldBlacklistMode;
@@ -408,7 +400,8 @@ public class Minepacks extends JavaPlugin implements MinepacksPlugin, IPlugin
 		return shortcut.isItemShortcut(itemStack);
 	}
 
-	public ItemsCollector getItemsCollector() {
+	public ItemsCollector getItemsCollector()
+	{
 		return collector;
 	}
 

Minepacks/src/at/pcgamingfreaks/Minepacks/Bukkit/Permissions.java

@@ -17,6 +17,13 @@
 
 package at.pcgamingfreaks.Minepacks.Bukkit;
 
+import lombok.SneakyThrows;
+
+import java.lang.reflect.Field;
+import java.lang.reflect.Modifier;
+import java.util.ArrayList;
+import java.util.List;
+
 public class Permissions
 {
 	public static final String BASE = "backpack.";
@@ -41,4 +48,29 @@ public class Permissions
 
 	public static final String INVENTORY_CLEAR = "clearInventory";
 	public static final String INVENTORY_CLEAR_OTHER = "clearInventory.other";
+
+	@SneakyThrows
+	public static List<String> getPermissions()
+	{
+		Field[] fields = Permissions.class.getDeclaredFields();
+		List<String> permissions = new ArrayList<>(fields.length);
+		for(Field field : fields)
+		{
+			if (field.getType().equals(String.class) && Modifier.isStatic(field.getModifiers()))
+			{
+				String val = ((String) field.get(null));
+				if (!val.endsWith("."))
+				{
+					permissions.add(val);
+				}
+			}
+		}
+		for (int i = 1; i < 10; i++)
+		{
+			permissions.add("backpack.size." + i);
+		}
+		return permissions;
+	}
+
+	private Permissions() {}
 }

Minepacks/src/at/pcgamingfreaks/Minepacks/Bukkit/Placeholder/PlaceholderManager.java

@@ -0,0 +1,36 @@
+/*
+ *   Copyright (C) 2023 GeorgH93
+ *
+ *   This program is free software: you can redistribute it and/or modify
+ *   it under the terms of the GNU General Public License as published by
+ *   the Free Software Foundation, either version 3 of the License, or
+ *   (at your option) any later version.
+ *
+ *   This program is distributed in the hope that it will be useful,
+ *   but WITHOUT ANY WARRANTY; without even the implied warranty of
+ *   MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
+ *   GNU General Public License for more details.
+ *
+ *   You should have received a copy of the GNU General Public License
+ *   along with this program. If not, see <http://www.gnu.org/licenses/>.
+ */
+
+package at.pcgamingfreaks.Minepacks.Bukkit.Placeholder;
+
+import at.pcgamingfreaks.Minepacks.Bukkit.Minepacks;
+import at.pcgamingfreaks.Minepacks.Bukkit.Placeholder.Replacer.AutoPickupEnabled;
+
+public final class PlaceholderManager extends at.pcgamingfreaks.Bukkit.Placeholder.PlaceholderManager
+{
+	public PlaceholderManager(Minepacks plugin)
+	{
+		super(plugin);
+	}
+
+	@Override
+	protected void generatePlaceholdersMap()
+	{
+		Minepacks plugin = (Minepacks) getPlugin();
+		registerPlaceholder(new AutoPickupEnabled(plugin));
+	}
+}

Minepacks/src/at/pcgamingfreaks/Minepacks/Bukkit/Placeholder/Replacer/AutoPickupEnabled.java

@@ -0,0 +1,53 @@
+/*
+ *   Copyright (C) 2023 GeorgH93
+ *
+ *   This program is free software: you can redistribute it and/or modify
+ *   it under the terms of the GNU General Public License as published by
+ *   the Free Software Foundation, either version 3 of the License, or
+ *   (at your option) any later version.
+ *
+ *   This program is distributed in the hope that it will be useful,
+ *   but WITHOUT ANY WARRANTY; without even the implied warranty of
+ *   MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
+ *   GNU General Public License for more details.
+ *
+ *   You should have received a copy of the GNU General Public License
+ *   along with this program. If not, see <http://www.gnu.org/licenses/>.
+ */
+
+package at.pcgamingfreaks.Minepacks.Bukkit.Placeholder.Replacer;
+
+import at.pcgamingfreaks.Bukkit.Placeholder.PlaceholderName;
+import at.pcgamingfreaks.Bukkit.Placeholder.PlaceholderReplacerBase;
+import at.pcgamingfreaks.Minepacks.Bukkit.ItemsCollector;
+import at.pcgamingfreaks.Minepacks.Bukkit.Minepacks;
+
+import org.bukkit.OfflinePlayer;
+import org.bukkit.entity.Player;
+import org.jetbrains.annotations.Nullable;
+
+@PlaceholderName(aliases = "IsAutoPickupEnabled")
+public class AutoPickupEnabled extends PlaceholderReplacerBase
+{
+	private final Minepacks plugin;
+
+	public AutoPickupEnabled(Minepacks mp)
+	{
+		this.plugin = mp;
+	}
+
+	@Override
+	public @Nullable String replace(OfflinePlayer player)
+	{
+		ItemsCollector collector = plugin.getItemsCollector();
+		if (collector != null)
+		{
+			if (player instanceof Player)
+			{
+				return collector.canUseAutoPickup((Player) player) ? "true" : "false";
+			}
+			return "offline";
+		}
+		return "disabled";
+	}
+}

pom.xml

@@ -7,16 +7,21 @@
 	<packaging>pom</packaging>
 
 	<properties>
-		<revision>2.4.21</revision>
+		<revision>2.4.29</revision>
 		<project.build.sourceEncoding>UTF-8</project.build.sourceEncoding>
 		<project.reporting.outputEncoding>UTF-8</project.reporting.outputEncoding>
 
 		<configFileVersion>34</configFileVersion>
-		<languageFileVersion>20</languageFileVersion>
-		<pcgfPluginLibVersion>1.0.39-SNAPSHOT</pcgfPluginLibVersion>
+		<languageFileVersion>21</languageFileVersion>
+		<pcgfPluginLibVersion>1.0.39.4-SNAPSHOT</pcgfPluginLibVersion>
 
 		<bukkitVersion>1.15.2-R0.1-SNAPSHOT</bukkitVersion>
-		<mavenShade.version>3.4.1</mavenShade.version>
+		<mavenShade.version>3.5.3</mavenShade.version>
+
+		<author>GeorgH93</author>
+		<maven.build.timestamp.format>yyyyMMddHHmmss</maven.build.timestamp.format>
+		<timestamp>${maven.build.timestamp}</timestamp>
+		<pluginVersion>${project.version}-T${timestamp}</pluginVersion>
 
 		<sonar.organization>georgh93</sonar.organization>
 		<sonar.host.url>https://sonarcloud.io</sonar.host.url>
@@ -37,7 +42,7 @@
 	</ciManagement>
 
 	<name>Minepacks-Parent</name>
-	<description>Minepacks is a backpack plugin with different backpack sizes, multi language support and SQLite and MySQL storage support.</description>
+	<description>Minepacks is a backpack plugin with different backpack sizes, multi-language support and SQLite and MySQL storage support.</description>
 	<url>https://www.spigotmc.org/resources/19286/</url>
 	<inceptionYear>2014</inceptionYear>
 	<licenses>
@@ -76,7 +81,7 @@
 		<dependency>
 			<groupId>org.projectlombok</groupId>
 			<artifactId>lombok</artifactId>
-			<version>1.18.26</version>
+			<version>1.18.32</version>
 			<scope>provided</scope>
 		</dependency>
 	</dependencies>