Fishing nets onTick

[duke605]

Hello,

I'm making a fishing net mod and have no idea what method to use for the block to deposit stuff in a nearby chest. I was using randomDisplayTick() but that was only called server side and I need it to be called both server and client side. I was looking at onUpdateTick() but that has to be set before hand and it's only called once.

 

any ideas?

[Yagoki]

create a tile entity and use the updateEntity() method

[duke605]

updateEntity() seems to only be giving me the remote world as well

[Yagoki]

that's odd. Can I see your block and the tile entity?

[duke605]

BlockFishingNet

 

 

package duke605.mini.block;

 

 

import java.util.Random;

 

 

import cpw.mods.fml.client.FMLClientHandler;

import cpw.mods.fml.relauncher.Side;

import cpw.mods.fml.relauncher.SideOnly;

 

 

import duke605.mini.Mini;

import duke605.mini.tileentity.TileEntityFishingNet;

import net.minecraft.block.Block;

import net.minecraft.block.ITileEntityProvider;

import net.minecraft.block.material.Material;

import net.minecraft.client.Minecraft;

import net.minecraft.client.gui.GuiScreen;

import net.minecraft.creativetab.CreativeTabs;

import net.minecraft.entity.item.EntityItem;

import net.minecraft.entity.player.EntityPlayer;

import net.minecraft.item.Item;

import net.minecraft.item.ItemStack;

import net.minecraft.tileentity.TileEntity;

import net.minecraft.tileentity.TileEntityChest;

import net.minecraft.world.World;

 

 

public class BlockFishingNet extends Block implements ITileEntityProvider {

 

 

public BlockFishingNet(int par1) {

super(par1, Material.cloth);

this.setCreativeTab(Mini.customTab);

this.setLightOpacity(0);

}

 

public int getRenderBlockPass() {

        return 1;

    }

 

public boolean isOpaqueCube() {

        return false;

    }

 

public boolean renderAsNormalBlock() {

        return false;

    }

 

public boolean onBlockActivated(World world, int x, int y, int z, EntityPlayer player, int par6, float par7, float par8, float par9) {

int[] chestCoords;

if (!GuiScreen.isShiftKeyDown() && this.blockID == Mini.fishingNetClogged.blockID){

world.setBlock(x, y, z, Mini.fishingNet.blockID);

return true;

}

return false;

    }

 

 

@Override

public TileEntity createNewTileEntity(World world) {

return new TileEntityFishingNet();

}

}

 

 

 

TileEntityFishingNet

 

 

package duke605.mini.tileentity;

 

import net.minecraft.block.Block;

import net.minecraft.item.Item;

import net.minecraft.item.ItemStack;

import net.minecraft.tileentity.TileEntity;

import net.minecraft.tileentity.TileEntityChest;

import net.minecraft.world.World;

 

public class TileEntityFishingNet extends TileEntity{

public void updateEntity() {

World world = worldObj;

int x = this.xCoord;

int y = this.yCoord;

int z = this.zCoord;

 

 

int[] chestCoords;

if (isInWater(world, x, y ,z) && (chestCoords = findChest(world, x, y, z)) != null) {

((TileEntityChest)world.getBlockTileEntity(chestCoords[0], chestCoords[1], chestCoords[2])).setInventorySlotContents(1, new ItemStack(Item.appleGold, 1));

}

}

 

private boolean isInWater(World world, int x, int y, int z) {

if (isWater(world.getBlockId(x+1, y, z+1)) || isWater(world.getBlockId(x, y, z+1)) || isWater(world.getBlockId(x+1, y, z)) ||

isWater(world.getBlockId(x-1, y, z-1)) || isWater(world.getBlockId(x, y, z+1)) || isWater(world.getBlockId(x-1, y, z)))

return true;

 

return false;

}

 

private int[] findChest(World world, int xPos, int yPos, int zPos) {

for (int x = xPos-16;x < xPos+16;x++) {

for (int y = yPos-16;y < yPos+16;y++) {

for (int z = zPos-16;z < zPos+16;z++) {

if (world.getBlockId(x, y, z) == Block.chest.blockID)

return new int[] {x, y, z};

}

}

}

return null;

}

 

private boolean isWater(int id) {

if (id == Block.waterMoving.blockID || id == Block.waterStill.blockID)

return true;

 

return false;

}

}

 

 

[Yagoki]

This works:

public class TestBlock extends Block
{
public TestBlock(int par1, Material par2Material)
{
	super(par1, par2Material);
}

@Override
public boolean hasTileEntity(int metadata)
{
	return true;
}

@Override
public TileEntity createTileEntity(World world, int metadata)
{
	return new TestTileEntity();
}
}

 

the tile entity (nothing special here):

public class TestTileEntity extends TileEntity
{

@Override
public void updateEntity()
{
	System.out.println("Hello World : " + this.worldObj.isRemote);
}

}

 

this was called on both sides, not sure what was wrong with yours, it could be something with ITileProvider, but I couldn't see anything obvious as to what it may be.

 

Your other option is to extend ContainerBlock instead of using ITileProvideras this also works

[duke605]

thanks, i got it