Raggaer / castro

High performance content management system for Open Tibia servers
MIT License
46 stars 19 forks source link

New premium time handling in TFS #109

Closed forgee closed 3 years ago

forgee commented 3 years ago

TFS has changed how they handle premium time and now just use a timestamp in premium_ends_at instead of premdays and lastday in the database. See this PR: https://github.com/otland/forgottenserver/pull/2813

I think it is too much work to keep compatibility with both old and new TFS, so I suggest we create a separate branch or tag a release for older TFS and let master keep compatibility with latest TFS. Are there any other suggestions or opinions before I start working on this?

Raggaer commented 3 years ago

I would just go and update the current code, supporting only newest TFS. But I dont mind having another branch, up to you =)

Thanks

forgee commented 3 years ago

This should be fixed with 8efd60aeff1dbdc391f0f82de83e1a17c4a0977b. Please report any issues.

I decided to just update the master branch because TFS 1.3 has been going for so long and I don't know if castro still worked with the actual TFS 1.2 release anyway.