nekiro / TFS-1.5-Downgrades

Alternative forgottenserver versions for older protocols support
72 stars 83 forks source link

Real 8.0 code #78

Closed gesior closed 2 years ago

gesior commented 2 years ago

I did not use bug report format, because I'm not reporting bug. I will make PRs referring to this Issue.

In 8.0 version there is a code for many thing that did not exist in 8.0.

nekiro commented 2 years ago

Datapack is not included with this downgrade, that's why you see stuff like elemental damage, it's there in the code, so people can easily adjust or use tfs newer protocol code without problems. It's left for compatibility. I don't plan to change that nor fix datapack, as it's protocol downgrade, not download and run server.

I don't really care if you want to make datapack better, but removing death, holy, ice is no-go for me. I want them to be there left for compatibility.

kontownik commented 2 years ago

Hey, I saw the same problem on another distro called eAthena (Ragnarok Online), they used ifndef for that. So the idea is to change the packet version (protocol for Tibia) on the compilation level, this way the server will never send a packet that is not supposed to be used for the client (gesior's point). I hope that will make a lot of warnings or crashes during the distro launch, making people clean the data pack by themselves (Nekiro's point).

Not sure how to fix LUA part...

btw. Isn't swimming.lua breaking a single source of truth as it's redefining the conditions on script level?

Code examples: https://github.com/eathena/eathena/blob/master/src/common/mmo.h https://github.com/eathena/eathena/search?q=PACKETVER