Skip to content

Commit f867a6e

Browse files
committed
fix(barrel): Build issue
1 parent 89fd0a9 commit f867a6e

8 files changed

Lines changed: 13 additions & 12 deletions

File tree

src/main/java/com/dre/brewery/Barrel.java

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -164,6 +164,11 @@ public static void onUpdate() {
164164
}
165165
}
166166

167+
public static @NotNull List<Barrel> getBarrels(UUID worldUuid) {
168+
List<Barrel> worldBarrels = barrels.get(worldUuid);
169+
return worldBarrels == null ? List.of() : worldBarrels;
170+
}
171+
167172
public boolean hasPermsOpen(Player player, PlayerInteractEvent event) {
168173
if (isLarge()) {
169174
if (!player.hasPermission("brewery.openbarrel.big")) {

src/main/java/com/dre/brewery/commands/subcommands/ShowStatsCommand.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -40,7 +40,7 @@ public void execute(BreweryPlugin breweryPlugin, Lang lang, CommandSender sender
4040

4141
Logging.msg(sender, "Drunk Players: " + BPlayer.numDrunkPlayers());
4242
Logging.msg(sender, "Brews created: " + BreweryPlugin.getInstance().getBreweryStats().brewsCreated);
43-
Logging.msg(sender, "Barrels built: " + Barrel.barrels.size());
43+
Logging.msg(sender, "Barrels built: " + Barrel.getAllBarrels().size());
4444
Logging.msg(sender, "Cauldrons boiling: " + BCauldron.bcauldrons.size());
4545
Logging.msg(sender, "Number of Recipes: " + BRecipe.getAllRecipes().size());
4646
Logging.msg(sender, "Wakeups: " + Wakeup.wakeups.size());

src/main/java/com/dre/brewery/integration/bstats/BreweryStats.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -79,7 +79,7 @@ public void setupBStats() {
7979
Metrics metrics = new Metrics(BreweryPlugin.getInstance(), BSTATS_ID);
8080
metrics.addCustomChart(new SingleLineChart("drunk_players", BPlayer::numDrunkPlayers));
8181
metrics.addCustomChart(new SingleLineChart("brews_in_existence", () -> brewsCreated));
82-
metrics.addCustomChart(new SingleLineChart("barrels_built", Barrel.barrels::size));
82+
metrics.addCustomChart(new SingleLineChart("barrels_built", Barrel.getAllBarrels()::size));
8383
metrics.addCustomChart(new SingleLineChart("cauldrons_boiling", BCauldron.bcauldrons::size));
8484
metrics.addCustomChart(new AdvancedPie("brew_quality", () -> {
8585
Map<String, Integer> map = new HashMap<>(8);

src/main/java/com/dre/brewery/integration/bstats/BreweryXStats.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -108,7 +108,7 @@ public void setupBStats() {
108108
metrics.addCustomChart(new SimplePie("branch", this::getBranch));
109109

110110
metrics.addCustomChart(new SingleLineChart("drunk_players", BPlayer::numDrunkPlayers));
111-
metrics.addCustomChart(new SingleLineChart("barrels_built", Barrel.barrels::size));
111+
metrics.addCustomChart(new SingleLineChart("barrels_built", Barrel.getAllBarrels()::size));
112112
metrics.addCustomChart(new SingleLineChart("cauldrons_boiling", BCauldron.bcauldrons::size));
113113

114114
} catch (Exception | LinkageError e) {

src/main/java/com/dre/brewery/integration/listeners/movecraft/MovecraftUtil.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -39,7 +39,7 @@ public class MovecraftUtil {
3939
public static Set<Barrel> barrelsOnCraft(HitBox hitBox, World craftWorld) {
4040
Set<Barrel> output = new HashSet<>();
4141

42-
for (Barrel barrel : Barrel.barrels) {
42+
for (Barrel barrel : Barrel.getBarrels(craftWorld.getUID())) {
4343
Location location = barrel.getSpigot().getLocation();
4444

4545
if (!Objects.equals(location.getWorld(), craftWorld)) {

src/main/java/com/dre/brewery/integration/listeners/movecraft/SinkListener.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -35,7 +35,7 @@ public class SinkListener implements Listener {
3535
@EventHandler
3636
public void onSink(CraftSinkEvent event) {
3737
HitBox hitBox = event.getCraft().getHitBox();
38-
ArrayList<Barrel> list = new ArrayList<>(Barrel.barrels);
38+
ArrayList<Barrel> list = new ArrayList<>(Barrel.getBarrels(event.getCraft().getWorld().getUID()));
3939

4040
for (Barrel barrel : list) {
4141
Location location = barrel.getSpigot().getLocation().clone();

src/main/java/com/dre/brewery/listeners/WorldListener.java

Lines changed: 2 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -35,16 +35,12 @@ public void onWorldLoad(WorldLoadEvent event) {
3535
dataManager.getAllBarrels()
3636
.thenAcceptAsync(barrels -> barrels.stream()
3737
.filter(barrel -> barrel.getSpigot().getWorld().equals(event.getWorld()))
38-
.forEach(Barrel.getBarrels()::add)
38+
.forEach(Barrel::registerBarrel)
3939
);
4040
}
4141

4242
@EventHandler(priority = EventPriority.MONITOR)
4343
public void onWorldLoad(WorldUnloadEvent event) {
44-
dataManager.getAllBarrels()
45-
.thenAcceptAsync(barrels -> barrels.stream()
46-
.filter(barrel -> barrel.getSpigot().getWorld().equals(event.getWorld()))
47-
.forEach(Barrel.getBarrels()::remove)
48-
);
44+
Barrel.onUnload(event.getWorld());
4945
}
5046
}

src/main/java/com/dre/brewery/storage/BData.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -477,7 +477,7 @@ public static void loadWorldData(String uuid, World world) {
477477
}
478478
if (!initBarrelFutures.isEmpty()) {
479479
FutureUtil.mergeFutures(initBarrelFutures)
480-
.thenAcceptAsync(Barrel.barrels::addAll);
480+
.thenAcceptAsync(barrels -> barrels.forEach(Barrel::registerBarrel));
481481
}
482482
if (!initWakeups.isEmpty()) {
483483
Wakeup.wakeups.addAll(initWakeups);

0 commit comments

Comments
 (0)