Skip to content

Commit 959ccfb

Browse files
authored
Updated Argonauts API integration to fix crashing (#1120)
1 parent 1642b7e commit 959ccfb

File tree

4 files changed

+13
-14
lines changed

4 files changed

+13
-14
lines changed

gradle.properties

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -33,8 +33,8 @@ guideme_version_range=[21.1.5, 21.2)
3333
# Optional dependency versions
3434
ae2_version=19.0.12-alpha
3535
ae2_version_range=[19.0.12-alpha, 20)
36-
argonauts_minecraft_version=1.20.4
37-
argonauts_version=1.2.0
36+
argonauts_minecraft_version=1.21
37+
argonauts_version=2.0.0-alpha.4
3838
athena_minecraft_version=1.21
3939
athena_version=4.0.0
4040
emi_minecraft_version=1.21

src/main/java/aztech/modern_industrialization/compat/argonauts/ArgonautsFacade.java

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -26,7 +26,7 @@
2626
import java.util.Collection;
2727
import java.util.List;
2828
import java.util.UUID;
29-
import net.minecraft.server.MinecraftServer;
29+
import net.minecraft.world.level.Level;
3030
import net.neoforged.fml.ModList;
3131

3232
public interface ArgonautsFacade {
@@ -45,5 +45,5 @@ private static ArgonautsFacade getInstance() {
4545
return (ms, uuid) -> List.of();
4646
}
4747

48-
Collection<UUID> getOtherPlayersInGuild(MinecraftServer server, UUID playerUuid);
48+
Collection<UUID> getOtherPlayersInGuild(Level level, UUID playerUuid);
4949
}

src/main/java/aztech/modern_industrialization/compat/argonauts/ArgonautsFacadeImpl.java

Lines changed: 8 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -23,24 +23,24 @@
2323
*/
2424
package aztech.modern_industrialization.compat.argonauts;
2525

26-
import earth.terrarium.argonauts.api.guild.GuildApi;
26+
import earth.terrarium.argonauts.api.teams.guild.GuildApi;
2727
import java.util.ArrayList;
2828
import java.util.Collection;
2929
import java.util.List;
3030
import java.util.UUID;
31-
import net.minecraft.server.MinecraftServer;
31+
import net.minecraft.world.level.Level;
3232

3333
public class ArgonautsFacadeImpl implements ArgonautsFacade {
3434
@Override
35-
public Collection<UUID> getOtherPlayersInGuild(MinecraftServer server, UUID playerUuid) {
36-
var guild = GuildApi.API.getPlayerGuild(server, playerUuid);
37-
if (guild == null) {
35+
public Collection<UUID> getOtherPlayersInGuild(Level level, UUID playerUuid) {
36+
var guild = GuildApi.API.getPlayerGuild(level, playerUuid);
37+
if (guild.isEmpty()) {
3838
return List.of();
3939
}
4040
List<UUID> out = new ArrayList<>();
41-
for (var member : guild.members()) {
42-
if (!member.profile().getId().equals(playerUuid)) {
43-
out.add(member.profile().getId());
41+
for (var memberUuid : guild.get().members().keySet()) {
42+
if (!memberUuid.equals(playerUuid)) {
43+
out.add(memberUuid);
4444
}
4545
}
4646
return out;

src/main/java/aztech/modern_industrialization/stats/PlayerStatistics.java

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -92,7 +92,6 @@ public void addUsedItems(ItemLike what, long amount) {
9292
}
9393

9494
public void addProducedItems(Level level, ItemLike what, long amount) {
95-
var server = Objects.requireNonNull(level.getServer());
9695
var item = what.asItem();
9796
producedItems.computeIfAbsent(item, i -> new StatisticValue()).add(amount);
9897

@@ -104,7 +103,7 @@ public void addProducedItems(Level level, ItemLike what, long amount) {
104103
// Make sure we only award the stats to other players once even if they have both FTB Teams and Argonauts.
105104
uuidCache.clear();
106105
uuidCache.addAll(FTBTeamsFacade.INSTANCE.getOtherPlayersInTeam(uuid));
107-
uuidCache.addAll(ArgonautsFacade.INSTANCE.getOtherPlayersInGuild(server, uuid));
106+
uuidCache.addAll(ArgonautsFacade.INSTANCE.getOtherPlayersInGuild(level, uuid));
108107

109108
for (var uuid : uuidCache) {
110109
data.get(uuid).awardStat(level, what, amount);

0 commit comments

Comments
 (0)