[1.6.4]Custom Portal Woes

[SilasOtoko]

I'm having several issues here, and I can't seem to pinpoint the problem. When I use a custom portal to get to my custom dimension, I run into one of two problems. 1) The player spawns underground or somewhere else nearby. 2) The custom portal destroys itself or does not spawn a proper working portal in the next dimension after going through it. Third issue I am trying to fix is the portal that fills in the space between the frame is still breakable as individual blocks. Any help appreciated.

 

Portal Block

 

public class blockTeleporter extends BlockPortal{

public blockTeleporter(int par1){

super(par1);

 

}

 

public void onEntityCollidedWithBlock(World world, int x, int y, int z, Entity entity){

if(entity.ridingEntity == null && entity.riddenByEntity == null && entity instanceof EntityPlayerMP){

EntityPlayerMP player = (EntityPlayerMP) entity;

MinecraftServer server = MinecraftServer.getServer();

 

if(player.timeUntilPortal > 0){

player.timeUntilPortal = 10;

}else if(player.dimension != Main.dimensionIDRune2){

player.timeUntilPortal = 10;

player.mcServer.getConfigurationManager().transferPlayerToDimension(player, Main.dimensionIDRune2, new TeleporterRuneDimension(server.worldServerForDimension(Main.dimensionIDRune2)));

}else{

player.timeUntilPortal = 10;

player.mcServer.getConfigurationManager().transferPlayerToDimension(player, 0, new TeleporterRuneDimension(server.worldServerForDimension(Main.dimensionIDRune2)));

 

}

}

}

 

public boolean tryToCreatePortal(World world, int x, int y, int z){

 

byte b = 0;

byte b1 = 0;

 

if(world.getBlockId(x-1, y, z) == Main.blockDimensionStone.blockID || world.getBlockId(x+1, y, z) == Main.blockDimensionStone.blockID){

b = 1;

 

}

 

if(world.getBlockId(x, y, z-1) == Main.blockDimensionStone.blockID || world.getBlockId(x, y, z+1) == Main.blockDimensionStone.blockID){

b1 = 1;

}

 

if(b == b1){

return false;

}else{

if(world.isAirBlock(x-b, y, z-b1)){

x-=b;

z-=b1;

}

 

for(int i = -1; i <= 2; i++){

for(int j = -1; j <= 3; j++){

boolean flag = (i == -1 || i == 2 || j == -1 || j == 3);

 

if(i != -1 && i != 2 || j != -1 && j != 3){

int k = world.getBlockId(x + (b*i), y + j, z+(b1*i));

boolean isAirBlock = world.isAirBlock(x + b*i, y+j, z + b1*i);

 

if(flag){

if(k != Main.blockDimensionStone.blockID){

return false;

}

}else if(!isAirBlock && k != Main.RuneFire.blockID){

return false;

}

}

}

}

 

for(int l = 0; l < 2; l++){

for(int l2 = 0; l2 < 3; l2++){

world.setBlock(x + b*l, y + l2, z + b1*l, Main.blockTeleporter.blockID, 0, 2);

}

}

return false;

}

 

 

}

 

//@Override

public void onNeighborBlockChange(World par1World, int par2, int par3, int par4, int par5)

    {

        byte b0 = 0;

        byte b1 = 1;

 

        if (par1World.getBlockId(par2 - 1, par3, par4) == this.blockID || par1World.getBlockId(par2 + 1, par3, par4) == this.blockID)

        {

            b0 = 1;

            b1 = 0;

        }

 

        int i1;

 

        for (i1 = par3; par1World.getBlockId(par2, i1 - 1, par4) == this.blockID; --i1)

        {

            ;

        }

 

        if (par1World.getBlockId(par2, i1 - 1, par4) != Main.blockDimensionStone.blockID)

        {

            par1World.setBlockToAir(par2, par3, par4);

        }

        else

        {

            int j1;

 

            for (j1 = 1; j1 < 4 && par1World.getBlockId(par2, i1 + j1, par4) == this.blockID; ++j1)

            {

                ;

            }

 

            if (j1 == 3 && par1World.getBlockId(par2, i1 + j1, par4) == Main.blockDimensionStone.blockID)

            {

                boolean flag = par1World.getBlockId(par2 - 1, par3, par4) == this.blockID || par1World.getBlockId(par2 + 1, par3, par4) == this.blockID;

                boolean flag1 = par1World.getBlockId(par2, par3, par4 - 1) == this.blockID || par1World.getBlockId(par2, par3, par4 + 1) == this.blockID;

 

                if (flag && flag1)

                {

                    par1World.setBlockToAir(par2, par3, par4);

                }

                else

                {

                    if ((par1World.getBlockId(par2 + b0, par3, par4 + b1) != Main.blockDimensionStone.blockID || par1World.getBlockId(par2 - b0, par3, par4 - b1) != this.blockID) && (par1World.getBlockId(par2 - b0, par3, par4 - b1) != Main.blockDimensionStone.blockID || par1World.getBlockId(par2 + b0, par3, par4 + b1) != this.blockID))

                    {

                        par1World.setBlockToAir(par2, par3, par4);

                    }

                }

            }

            else

            {

                par1World.setBlockToAir(par2, par3, par4);

            }

        }

    }

 

 

}

 

 

Teleporter

 

public class TeleporterRuneDimension extends Teleporter {

 

private final WorldServer worldServerInstance;

private final Random rand;

 

private final LongHashMap destinationCoordinateCache = new LongHashMap();

private final List destinationCoordinateKeys = new ArrayList();

 

public TeleporterRuneDimension(WorldServer par1WorldServer){

super(par1WorldServer);

 

this.worldServerInstance = par1WorldServer;

this.rand = new Random(par1WorldServer.getSeed());

}

 

public void placeInPortal(Entity entity, double x, double y, double z, float f){

if(this.worldServerInstance.provider.dimensionId != 1){

if(!this.placeInExistingPortal(entity, x, y, z, f)){

this.makePortal(entity);

this.placeInExistingPortal(entity, x, y, z, f);

 

}else{

int entity_x = MathHelper.floor_double(entity.posX);

int entity_y = MathHelper.floor_double(entity.posY)-1;

int entity_z = MathHelper.floor_double(entity.posZ);

byte b0 = 1;

byte b1 = 0;

 

for(int i= -2; i <= 2; i++){

for(int j = -2; j <= 2; j++){

for(int k = -1; k < 3; k++){

int x2 = entity_x + j*b0 + i*b1;

int y2 = entity_y + k;

int z2 = entity_z + j*b1 - 1*b0;

boolean flag = k < 0;

this.worldServerInstance.setBlock(x2, y2, z2, flag ? Main.blockDimensionStone.blockID : 0);

}

}

}

 

entity.setLocationAndAngles(entity_x, entity_y, entity_z, entity.rotationYaw, 0F);

entity.motionX = entity.motionY = entity.motionZ = 0D;

}

}

}

 

@Override

public boolean placeInExistingPortal(Entity par1Entity, double par2, double par4, double par6, float par8)

    {

        short short1 = 128;

        double d3 = -1.0D;

        int i = 0;

        int j = 0;

        int k = 0;

        int l = MathHelper.floor_double(par1Entity.posX);

        int i1 = MathHelper.floor_double(par1Entity.posZ);

        long j1 = ChunkCoordIntPair.chunkXZ2Int(l, i1);

        boolean flag = true;

        double d4;

        int k1;

 

        if (this.destinationCoordinateCache.containsItem(j1))

        {

            PortalPosition portalposition = (PortalPosition)this.destinationCoordinateCache.getValueByKey(j1);

            d3 = 0.0D;

            i = portalposition.posX;

            j = portalposition.posY;

            k = portalposition.posZ;

            portalposition.lastUpdateTime = this.worldServerInstance.getTotalWorldTime();

            flag = false;

        }

        else

        {

            for (k1 = l - short1; k1 <= l + short1; ++k1)

            {

                double d5 = (double)k1 + 0.5D - par1Entity.posX;

 

                for (int l1 = i1 - short1; l1 <= i1 + short1; ++l1)

                {

                    double d6 = (double)l1 + 0.5D - par1Entity.posZ;

 

                    for (int i2 = this.worldServerInstance.getActualHeight() - 1; i2 >= 0; --i2)

                    {

                        if (this.worldServerInstance.getBlockId(k1, i2, l1) == Main.blockTeleporter.blockID)

                        {

                            while (this.worldServerInstance.getBlockId(k1, i2 - 1, l1) == Main.blockTeleporter.blockID)

                            {

                                --i2;

                            }

 

                            d4 = (double)i2 + 0.5D - par1Entity.posY;

                            double d7 = d5 * d5 + d4 * d4 + d6 * d6;

 

                            if (d3 < 0.0D || d7 < d3)

                            {

                                d3 = d7;

                                i = k1;

                                j = i2;

                                k = l1;

                            }

                        }

                    }

                }

            }

        }

 

        if (d3 >= 0.0D)

        {

            if (flag)

            {

                this.destinationCoordinateCache.add(j1, new PortalPosition(this, i, j, k, this.worldServerInstance.getTotalWorldTime()));

                this.destinationCoordinateKeys.add(Long.valueOf(j1));

            }

 

            double d8 = (double)i + 0.5D;

            double d9 = (double)j + 0.5D;

            d4 = (double)k + 0.5D;

            int j2 = -1;

 

            if (this.worldServerInstance.getBlockId(i - 1, j, k) == Main.blockTeleporter.blockID)

            {

                j2 = 2;

            }

 

            if (this.worldServerInstance.getBlockId(i + 1, j, k) == Main.blockTeleporter.blockID)

            {

                j2 = 0;

            }

 

            if (this.worldServerInstance.getBlockId(i, j, k - 1) == Main.blockTeleporter.blockID)

            {

                j2 = 3;

            }

 

            if (this.worldServerInstance.getBlockId(i, j, k + 1) == Main.blockTeleporter.blockID)

            {

                j2 = 1;

            }

 

            int k2 = par1Entity.getTeleportDirection();

 

            if (j2 > -1)

            {

                int l2 = Direction.rotateLeft[j2];

                int i3 = Direction.offsetX[j2];

                int j3 = Direction.offsetZ[j2];

                int k3 = Direction.offsetX[l2];

                int l3 = Direction.offsetZ[l2];

                boolean flag1 = !this.worldServerInstance.isAirBlock(i + i3 + k3, j, k + j3 + l3) || !this.worldServerInstance.isAirBlock(i + i3 + k3, j + 1, k + j3 + l3);

                boolean flag2 = !this.worldServerInstance.isAirBlock(i + i3, j, k + j3) || !this.worldServerInstance.isAirBlock(i + i3, j + 1, k + j3);

 

                if (flag1 && flag2)

                {

                    j2 = Direction.rotateOpposite[j2];

                    l2 = Direction.rotateOpposite[l2];

                    i3 = Direction.offsetX[j2];

                    j3 = Direction.offsetZ[j2];

                    k3 = Direction.offsetX[l2];

                    l3 = Direction.offsetZ[l2];

                    k1 = i - k3;

                    d8 -= (double)k3;

                    int i4 = k - l3;

                    d4 -= (double)l3;

                    flag1 = !this.worldServerInstance.isAirBlock(k1 + i3 + k3, j, i4 + j3 + l3) || !this.worldServerInstance.isAirBlock(k1 + i3 + k3, j + 1, i4 + j3 + l3);

                    flag2 = !this.worldServerInstance.isAirBlock(k1 + i3, j, i4 + j3) || !this.worldServerInstance.isAirBlock(k1 + i3, j + 1, i4 + j3);

                }

 

                float f1 = 0.5F;

                float f2 = 0.5F;

 

                if (!flag1 && flag2)

                {

                    f1 = 1.0F;

                }

                else if (flag1 && !flag2)

                {

                    f1 = 0.0F;

                }

                else if (flag1 && flag2)

                {

                    f2 = 0.0F;

                }

 

                d8 += (double)((float)k3 * f1 + f2 * (float)i3);

                d4 += (double)((float)l3 * f1 + f2 * (float)j3);

                float f3 = 0.0F;

                float f4 = 0.0F;

                float f5 = 0.0F;

                float f6 = 0.0F;

 

                if (j2 == k2)

                {

                    f3 = 1.0F;

                    f4 = 1.0F;

                }

                else if (j2 == Direction.rotateOpposite[k2])

                {

                    f3 = -1.0F;

                    f4 = -1.0F;

                }

                else if (j2 == Direction.rotateRight[k2])

                {

                    f5 = 1.0F;

                    f6 = -1.0F;

                }

                else

                {

                    f5 = -1.0F;

                    f6 = 1.0F;

                }

 

                double d10 = par1Entity.motionX;

                double d11 = par1Entity.motionZ;

                par1Entity.motionX = d10 * (double)f3 + d11 * (double)f6;

                par1Entity.motionZ = d10 * (double)f5 + d11 * (double)f4;

                par1Entity.rotationYaw = par8 - (float)(k2 * 90) + (float)(j2 * 90);

            }

            else

            {

                par1Entity.motionX = par1Entity.motionY = par1Entity.motionZ = 0.0D;

            }

 

            par1Entity.setLocationAndAngles(d8, d9, d4, par1Entity.rotationYaw, par1Entity.rotationPitch);

            return true;

        }

        else

        {

            return false;

        }

    }

 

public boolean makePortal(Entity par1Entity)

    {

        byte b0 = 16;

        double d0 = -1.0D;

        int i = MathHelper.floor_double(par1Entity.posX);

        int j = MathHelper.floor_double(par1Entity.posY);

        int k = MathHelper.floor_double(par1Entity.posZ);

        int l = i;

        int i1 = j;

        int j1 = k;

        int k1 = 0;

        int l1 = this.rand.nextInt(4);

        int i2;

        double d1;

        double d2;

        int j2;

        int k2;

        int l2;

        int i3;

        int j3;

        int k3;

        int l3;

        int i4;

        int j4;

        int k4;

        double d3;

        double d4;

 

        for (i2 = i - b0; i2 <= i + b0; ++i2)

        {

            d1 = (double)i2 + 0.5D - par1Entity.posX;

 

            for (j2 = k - b0; j2 <= k + b0; ++j2)

            {

                d2 = (double)j2 + 0.5D - par1Entity.posZ;

                label274:

 

                for (k2 = this.worldServerInstance.getActualHeight() - 1; k2 >= 0; --k2)

                {

                    if (this.worldServerInstance.isAirBlock(i2, k2, j2))

                    {

                        while (k2 > 0 && this.worldServerInstance.isAirBlock(i2, k2 - 1, j2))

                        {

                            --k2;

                        }

 

                        for (i3 = l1; i3 < l1 + 4; ++i3)

                        {

                            l2 = i3 % 2;

                            k3 = 1 - l2;

 

                            if (i3 % 4 >= 2)

                            {

                                l2 = -l2;

                                k3 = -k3;

                            }

 

                            for (j3 = 0; j3 < 3; ++j3)

                            {

                                for (i4 = 0; i4 < 4; ++i4)

                                {

                                    for (l3 = -1; l3 < 4; ++l3)

                                    {

                                        k4 = i2 + (i4 - 1) * l2 + j3 * k3;

                                        j4 = k2 + l3;

                                        int l4 = j2 + (i4 - 1) * k3 - j3 * l2;

 

                                        if (l3 < 0 && !this.worldServerInstance.getBlockMaterial(k4, j4, l4).isSolid() || l3 >= 0 && !this.worldServerInstance.isAirBlock(k4, j4, l4))

                                        {

                                            continue label274;

                                        }

                                    }

                                }

                            }

 

                            d4 = (double)k2 + 0.5D - par1Entity.posY;

                            d3 = d1 * d1 + d4 * d4 + d2 * d2;

 

                            if (d0 < 0.0D || d3 < d0)

                            {

                                d0 = d3;

                                l = i2;

                                i1 = k2;

                                j1 = j2;

                                k1 = i3 % 4;

                            }

                        }

                    }

                }

            }

        }

 

        if (d0 < 0.0D)

        {

            for (i2 = i - b0; i2 <= i + b0; ++i2)

            {

                d1 = (double)i2 + 0.5D - par1Entity.posX;

 

                for (j2 = k - b0; j2 <= k + b0; ++j2)

                {

                    d2 = (double)j2 + 0.5D - par1Entity.posZ;

                    label222:

 

                    for (k2 = this.worldServerInstance.getActualHeight() - 1; k2 >= 0; --k2)

                    {

                        if (this.worldServerInstance.isAirBlock(i2, k2, j2))

                        {

                            while (k2 > 0 && this.worldServerInstance.isAirBlock(i2, k2 - 1, j2))

                            {

                                --k2;

                            }

 

                            for (i3 = l1; i3 < l1 + 2; ++i3)

                            {

                                l2 = i3 % 2;

                                k3 = 1 - l2;

 

                                for (j3 = 0; j3 < 4; ++j3)

                                {

                                    for (i4 = -1; i4 < 4; ++i4)

                                    {

                                        l3 = i2 + (j3 - 1) * l2;

                                        k4 = k2 + i4;

                                        j4 = j2 + (j3 - 1) * k3;

 

                                        if (i4 < 0 && !this.worldServerInstance.getBlockMaterial(l3, k4, j4).isSolid() || i4 >= 0 && !this.worldServerInstance.isAirBlock(l3, k4, j4))

                                        {

                                            continue label222;

                                        }

                                    }

                                }

 

                                d4 = (double)k2 + 0.5D - par1Entity.posY;

                                d3 = d1 * d1 + d4 * d4 + d2 * d2;

 

                                if (d0 < 0.0D || d3 < d0)

                                {

                                    d0 = d3;

                                    l = i2;

                                    i1 = k2;

                                    j1 = j2;

                                    k1 = i3 % 2;

                                }

                            }

                        }

                    }

                }

            }

        }

 

        int i5 = l;

        int j5 = i1;

        j2 = j1;

        int k5 = k1 % 2;

        int l5 = 1 - k5;

 

        if (k1 % 4 >= 2)

        {

            k5 = -k5;

            l5 = -l5;

        }

 

        boolean flag;

 

        if (d0 < 0.0D)

        {

            if (i1 < 70)

            {

                i1 = 70;

            }

 

            if (i1 > this.worldServerInstance.getActualHeight() - 10)

            {

                i1 = this.worldServerInstance.getActualHeight() - 10;

            }

 

            j5 = i1;

 

            for (k2 = -1; k2 <= 1; ++k2)

            {

                for (i3 = 1; i3 < 3; ++i3)

                {

                    for (l2 = -1; l2 < 3; ++l2)

                    {

                        k3 = i5 + (i3 - 1) * k5 + k2 * l5;

                        j3 = j5 + l2;

                        i4 = j2 + (i3 - 1) * l5 - k2 * k5;

                        flag = l2 < 0;

                        this.worldServerInstance.setBlock(k3, j3, i4, flag ? Main.blockDimensionStone.blockID : 0);

                    }

                }

            }

        }

 

        for (k2 = 0; k2 < 4; ++k2)

        {

            for (i3 = 0; i3 < 4; ++i3)

            {

                for (l2 = -1; l2 < 4; ++l2)

                {

                    k3 = i5 + (i3 - 1) * k5;

                    j3 = j5 + l2;

                    i4 = j2 + (i3 - 1) * l5;

                    flag = i3 == 0 || i3 == 3 || l2 == -1 || l2 == 3;

                    this.worldServerInstance.setBlock(k3, j3, i4, flag ? Main.blockDimensionStone.blockID : Main.blockTeleporter.blockID, 0, 2);

                }

            }

 

            for (i3 = 0; i3 < 4; ++i3)

            {

                for (l2 = -1; l2 < 4; ++l2)

                {

                    k3 = i5 + (i3 - 1) * k5;

                    j3 = j5 + l2;

                    i4 = j2 + (i3 - 1) * l5;

                    this.worldServerInstance.notifyBlocksOfNeighborChange(k3, j3, i4, this.worldServerInstance.getBlockId(k3, j3, i4));

                }

            }

        }

 

        return true;

    }

 

public void removeStalePortalLocations(long par1)

    {

        if (par1 % 100L == 0L)

        {

            Iterator iterator = this.destinationCoordinateKeys.iterator();

            long j = par1 - 600L;

 

            while (iterator.hasNext())

            {

                Long olong = (Long)iterator.next();

                PortalPosition portalposition = (PortalPosition)this.destinationCoordinateCache.getValueByKey(olong.longValue());

 

                if (portalposition == null || portalposition.lastUpdateTime < j)

                {

                    iterator.remove();

                    this.destinationCoordinateCache.remove(olong.longValue());

                }

            }

        }

    }

 

 

}

 

 

[target.san]

Sorry, but I doubt anyone would help you with such decription.

Some advices:

1. Rewrite your code so it would consist of autonomous parts, easy to read and debug. Now your code is a large blob of text no one would read.

2. Debug your code and identify part you're having trouble with

3. Formulate the problem in short and concise words and post it there if you haven't already solved it

[SilasOtoko]

The reason I gave all the code is because, like I said in the post, I do not have an idea which method is causing an issue. I'm not getting any sort of error in Eclipse from it. If I were able to pinpoint it down to exactly where I was having the issue and was able to more concisely identify the problem then I'd probably be able to figure it out on my own. I only asked if anyone was able to give any insight into it, maybe from personal experience or guesswork.

  I wish I could be more specific, but I can't. All I know is that my portal is not functioning like a portal should after the player passes through it into another dimension. The player does not spawn inside another portal like when using the vanilla portal.

[target.san]

IMO the reason you're having such trouble is because you've copypasted vanilla code almost 1:1. And we must confess to ourselves that cross-dimensional teleportation code in Minecraft is pretty bad.

I cannot help you identify your problems exactly though here are several advices:

1. Separate portal construction. I mean, fill all necessary data for portal and construct its exitpoint when you create one, not when you travel through it. This will allow you to pinpoint any portal construction issues.

2. I would recommend add tile entity for your portal block and store exitpoint there

3. teleportPlayerToDimension, along with your teleporter code, should only perform teleportation and nothing more.

4. As of block's breakability, I'd propose override onBlockPreDestroy and do portal death there

5. Write your code in a readable manner. Remember that MC sources are in no way a reference to some 'good' style since they're product of deobfuscation. The first step is to give sensible names to function arguments and local variables.