make passenger list not null because its more clean to use the api that way

This commit is contained in:
Pyrbu 2024-12-15 10:40:28 +01:00
parent ca89bfe32d
commit 9477d4b500
2 changed files with 9 additions and 14 deletions

View file

@ -13,11 +13,7 @@ import lol.pyr.znpcsplus.util.Viewable;
import org.bukkit.entity.Player;
import org.bukkit.inventory.ItemStack;
import java.util.ArrayList;
import java.util.List;
import java.util.Set;
import java.util.UUID;
import java.util.stream.IntStream;
import java.util.*;
public class PacketEntity implements PropertyHolder {
private final PacketFactory packetFactory;
@ -75,7 +71,7 @@ public class PacketEntity implements PropertyHolder {
if (vehicleId != null) {
packetFactory.setPassengers(player, vehicleId, this.getEntityId());
}
if(passengers != null) {
if (passengers != null) {
packetFactory.setPassengers(player, this.getEntityId(), passengers.stream().mapToInt(Integer::intValue).toArray());
}
}
@ -141,7 +137,7 @@ public class PacketEntity implements PropertyHolder {
}
public List<Integer> getPassengers() {
return passengers;
return passengers == null ? Collections.emptyList() : passengers;
}
public void addPassenger(int entityId) {

View file

@ -20,6 +20,7 @@ import org.bukkit.Location;
import org.bukkit.World;
import org.bukkit.entity.Player;
import org.bukkit.inventory.ItemStack;
import org.jetbrains.annotations.NotNull;
import org.jetbrains.annotations.Nullable;
import java.util.*;
@ -248,7 +249,7 @@ public class NpcImpl extends Viewable implements Npc {
}
@Override
public @Nullable List<Integer> getPassengers() {
public @NotNull List<Integer> getPassengers() {
return entity.getPassengers();
}
@ -271,6 +272,4 @@ public class NpcImpl extends Viewable implements Npc {
public void setVehicleId(Integer vehicleId) {
entity.setVehicleId(vehicleId);
}
}