Area.rearrange() buggy?

Miwarre · Dec 30th 2016

I am receiving wrong results by Area.rearrange() if (some of?) the start point coordinates are greater than the ending point's. In practice, both points become equal to the original ending point.

Looking into the matter, I noticed it uses code like this (and similarly for other conditions or other coordinates):

Java

if (startChunk.x == endChunk.x)
{
    if (startBlock.x > endBlock.x)
    {
        int cx = endChunk.x;
        int bx = endBlock.x;
        startChunk.x = endChunk.x;
        startBlock.x = endBlock.x;
        endChunk.x = cx;
        endBlock.x = bx;
        arrange = true;
    }
}

Display More

which in fact makes both end and start chunk/block equal to the original end chunk/block. I suppose it should be instead:

Java

if (startChunk.x == endChunk.x)
{
    if (startBlock.x > endBlock.x)
    {
        int cx = endChunk.x;
        int bx = endBlock.x;
        endChunk.x = startChunk.x;
        endBlock.x = endBlock.x;
        startChunk.x = cx;
        startBlock.x = bx;
        arrange = true;
    }
}

Display More

Am I wrong?

Miwarre · Feb 15th 2017

This issue seems to be still there...

red51 · Feb 15th 2017

That's true This will be fixed with the next hotfix (since all changes to the API require a server update) or the next update

Participate now!