Issue with negative map ids

[cerus]

Hello,

I've noticed that you're negating negative map ids when a map packet is sent and I'm wondering why exactly you're doing that. Is there a specific reason for that? It's breaking other plugins and potentially Vanilla behavior as well.

https://github.com/SBDPlugins/MapReflectionAPI/blob/master/src/main/java/tech/sbdevelopment/mapreflectionapi/listeners/PacketListener.java#L68-L71

Thank you!

[stijnb1234]

That's done to detect map ids created by MapReflectionAPI. In the packet listener, the event is called and the number is swapped again (negative to positive).

[cerus]

That unfortunately breaks other plugins that also use negative map ids. Is the negative -> positive conversion needed? The Minecraft client will accept negative ids, that shouldn't be a problem.

[stijnb1234]

Do you have some example plugins broken?

[cerus]

I only have one example: cerus/maps.
maps also uses negative ids for clientside maps. The id counter starts at Integer.MIN_VALUE and counts up whenever a new clientside map is created. The map id is also used to send item frame packets that contain these maps. Changing the id will make the maps appear without any drawings to the client, because the item frame packets still contain the negative id.

[stijnb1234]

I see. This plugin is a fork, so I have to look into that further to see what's the exact reason it's used.

[cerus]

I appreciate the help, let me know if you need anything else