Chunk Loading problems

So, I'm messing around trying to create a plugin which teleports an entity from one location to another when right clicked. It works completely fine at short distances, but when I try to teleport an entity from an unloaded Chunk, nothing seems to happen. The teleport() method simply doesn't do anything, and manually removing and respawning the entity does work, but it doesn't actually remove the original entity. So I draw the logical conclusion that the unloaded Chunks is the problem. Furthermore, if the entity is in the spawn chunks when teleported, everything works fine, confirming my hypothesis.

I then tried to load the chunk where the entity was when teleporting, but to no success, I also tried loading 5x5 chunks around it to make it an entity-processing chunk (I think that's how it works..), but again, to no success. So my question is, how do I manually load a Chunk so the entity can be teleported away?

 

Have you tried loading the chunk and teleporting the entity a few ticks later?

 

@FisheyLP
Yep, didn't work either.. Here's my code btw:

Code:java

package com.bringholm.featherteleport;
 
import org.apache.commons.lang.WordUtils;
import org.bukkit.Chunk;
import org.bukkit.Location;
import org.bukkit.Material;
import org.bukkit.World;
import org.bukkit.entity.Animals;
import org.bukkit.entity.Entity;
import org.bukkit.event.EventHandler;
import org.bukkit.event.Listener;
import org.bukkit.event.block.Action;
import org.bukkit.event.player.PlayerInteractAtEntityEvent;
import org.bukkit.event.player.PlayerInteractEvent;
import org.bukkit.inventory.EquipmentSlot;
import org.bukkit.plugin.java.JavaPlugin;
 
import java.util.HashMap;
import java.util.UUID;
 
import static org.bukkit.ChatColor.GOLD;
 
public class FeatherTeleport extends JavaPlugin implements Listener {
    private HashMap<UUID, Entity> selectedMobs = new HashMap<>();
    private static FeatherTeleport main;
 
    public FeatherTeleport() {
        main = this;
    }
 
    public static FeatherTeleport getInstance() {
        return main;
    }
 
    @Override
    public void onEnable() {
        this.getServer().getPluginManager().registerEvents(this, this);
    }
 
    @EventHandler
    public void onEntityRightClick(PlayerInteractAtEntityEvent e) {
        if (e.getHand() == EquipmentSlot.HAND && e.getPlayer().getInventory().getItemInMainHand().getType() == Material.FEATHER) {
            if (e.getRightClicked() instanceof Animals) {
                selectedMobs.put(e.getPlayer().getUniqueId(), e.getRightClicked());
                e.getPlayer().sendMessage(GOLD + "Selected " + WordUtils.capitalizeFully(e.getRightClicked().getType().toString()) + " to teleport!");
            }
        }
    }
 
    @EventHandler
    public void onRightClick(PlayerInteractEvent e) {
        if (e.getItem() == null) {
            return;
        }
        if (e.getAction() == Action.RIGHT_CLICK_BLOCK && e.getHand() == EquipmentSlot.HAND && e.getItem().getType() == Material.FEATHER) {
            if (selectedMobs.containsKey(e.getPlayer().getUniqueId())) {
                Location location = e.getClickedBlock().getLocation().add(0.5, 1, 0.5);
                Entity entity = selectedMobs.get(e.getPlayer().getUniqueId());
                if (!entity.getLocation().getWorld().equals(e.getPlayer().getWorld())) {
                    e.getPlayer().sendMessage(GOLD + "Cannot teleport entities between worlds!");
                    return;
                }
                if (!location.getBlock().getType().isSolid()) {
                    loadChunks(entity.getLocation().getChunk());
                    entity.teleport(location);
                    e.getPlayer().sendMessage(GOLD + "Teleported " + WordUtils.capitalizeFully(entity.getType().toString()) + "!");
                    selectedMobs.remove(e.getPlayer().getUniqueId());
                } else {
                    e.getPlayer().sendMessage(GOLD + "Unsafe location to teleport mob to!");
                }
            }
        }
    }
 
    private void loadChunks(Chunk chunk) {
        World world = chunk.getWorld();
        for (int x = chunk.getX() - 6; x <= chunk.getX() + 6; x++) {
            for (int z = chunk.getZ() - 6; z <= chunk.getZ() + 6; z++) {
                world.loadChunk(x, z);
            }
        }
    }
}

 

There is no reason to have this. If you have other classes, pass the main instance of this class through the other class's constructor.

Remember, the item in the player's hand can be null.

Your main problem:
What may be happening is that the chunks are unloading right after you load them. Try listening to ChunkUnloadEvent to make sure those chunks are not unloading. Also, try checking if the chunk is actually loaded using (Chunk#isLoaded()).

 

@Zombie_Striker

1. That's just part of my part of my plugin template I forgot to delete
2. I do a nullcheck right before that?
3. I can't believe I didn't think of that.. That was the problem, thank you very much!

 

@Zombie_Striker
Actually, it isn't fixed, although I have made progress. Using the ChunkUnloadEvent I can prevent the chunks from unloading, provided that they have already been loaded by the player, but I can't seem to artificially load them. Chunk#load() doesn't work, and loading a 21x21 grid of chunks around the chunks doesn't seem to load it either..