Sign in to follow this  
Sklo:D

Creature Cages can't be unloaded in areas higher than 128

Recommended Posts

If you are in an area which has a surface height greater than 128 there is an overflow in a variable causing 129 to become -127. A very embarrassing bug, I am a bit disappointed.

How the hell can you think that byte would be a good idea here???

 

                final int tile = Server.surfaceMesh.getTile(performer.getTileX(), performer.getTileY());
                final byte height = (byte)Tiles.decodeHeight(tile);

                if (height < -17) {
                    performer.getCommunicator().sendNormalServerMessage("The water is too deep the creature would drown in the cage.");
                    return true;
                }

 

 

    byte height;
    int realHeight = 166;
    height = (byte) realHeight;
    
    System.out.print(realHeight); //166
    System.out.print(height); //-90 => leads to The water is too deep the creature would drown in the cage.... 166!!! levels over water!!

 

 

@Keenan

@Budda

Edited by Sklo:D
  • Like 2

Share this post


Link to post
Share on other sites

Sounds like the issue addressed here and fixed, but maybe the fix is not in WU build yet? It's very recent.

 

  • Like 1

Share this post


Link to post
Share on other sites

It's already fixed and ready for the next patch, if you still have issues with it after the next server update let me know.

  • Like 1

Share this post


Link to post
Share on other sites
1 hour ago, Darklords said:

It's already fixed and ready for the next patch, if you still have issues with it after the next server update let me know.

 

When is that likely to be? Roughly? September? 

 

 

Share this post


Link to post
Share on other sites

It was already fixed in WO, but appears the WU version didnt get it, we'll do a small update to correct that. 

 

Not september, no 

Share this post


Link to post
Share on other sites
On 7/4/2018 at 6:59 AM, Retrograde said:

It was already fixed in WO, but appears the WU version didnt get it, we'll do a small update to correct that. 

 

Not september, no 

 

Preparing for early december then.

Share this post


Link to post
Share on other sites

Create an account or sign in to comment

You need to be a member in order to leave a comment

Create an account

Sign up for a new account in our community. It's easy!

Register a new account

Sign in

Already have an account? Sign in here.

Sign In Now
Sign in to follow this