[FIXED] MCForge level loading not functioning correctly

[00pol]

MCSharp level loading gives a incorrect spawn. Here is an example level: http://www.mediafire.com/?5lmpc9dsno0syuj

You should spawn at 40, 199, 40 in FCraft, however the correct spawn is 2, 159, 38 (you can see this using a MCForge 5.5.0.3 server).

I tried fixing it but I don't know what the problem is. If someone could help that would be great.

Thanks

[fragmer]

I'm looking into it; thanks for taking the time to report a problem

[00pol]

Thanks!

[00pol]

Fixed it. MCForge 5.5.x.x Spawns are saved as X Y Z instead of X Z Y. To fix change line 85 of MapMCSharp.cs to this:

Code: Select all

            // Read in the spawn location
            map.Spawn = new Position {
                X = (short)(bs.ReadInt16() * 32),
                Y = (short)(bs.ReadInt16() * 32),
                Z = (short)(bs.ReadInt16() * 32),
                R = bs.ReadByte(),
                L = bs.ReadByte(),
            };

[fragmer]

I wonder if that's a recent change, or if it's been that way all along. I could've sworn that it functioned properly before...

[BobKare]

I wonder if that's a recent change, or if it's been that way all along. I could've sworn that it functioned properly before...

AFAIK, MCForge has always saved coordinates like that. I had to get used to the other way when I switched to fCraft.

[fragmer]

Fixed in 0.636. Thanks again for reporting.