rogeraabbccdd / Kento-Rankme

Rankme for CSGO
78 stars 46 forks source link

SM1.9 Fix ? #1

Closed P4R4DiSi4C closed 7 years ago

P4R4DiSi4C commented 7 years ago

Hello ! I saw you didn't support SM 1.9. Do you think to release a fix soon ?

Thank you so much

rogeraabbccdd commented 7 years ago

I didn't test my plugin in SM1.9, this bug is reported by Nachtfrische in this post.

Btw, I'm using SM1.8.0-5947 in my server.

VirtualHeaven commented 7 years ago

I'm running on SM 1.9 without any problems, only one issue with the rankme db being reset after mapchange (using mysql), otherwise everything works just fine for me.

With sqlite db it works too, no resets after mapchange

rogeraabbccdd commented 7 years ago

Thanks for your feedback, I'll checking mysql problem.

rogeraabbccdd commented 7 years ago

@VirtualHeaven Do you have any error log?

Nachtfrische commented 7 years ago

Hey,

as I've mentioned in the thread here, I'm having the same problem as VirtualHeaven. It works fine on 1.9 but the points are not saved after a mapchange. I'm using MySQL as well.

Error logs are: L 11/02/2016 - 23:45:09: [SM] Exception reported: Instruction contained invalid parameter L 11/02/2016 - 23:45:09: [SM] Blaming: rankme_kento.smx L 11/02/2016 - 23:45:09: [SM] Call stack trace: L 11/02/2016 - 23:45:09: [SM] [1] Line 1221, C:\Program Files (x86)\Steam\SteamApps\common\csgo plug\sourcemod-1.7.3-git5280-windows\addons\sourcemod\scripting\rankme_kento.sp::Event_RoundEnd

and

L 11/03/2016 - 23:53:14: [SM] Exception reported: Instruction contained invalid parameter L 11/03/2016 - 23:53:14: [SM] Blaming: rankme_kento.smx L 11/03/2016 - 23:53:14: [SM] Call stack trace: L 11/03/2016 - 23:53:14: [SM] [1] Line 1221, rankme_kento.sp::Event_RoundEnd

The database is filled with players and the default points of 1000, but they are not updated because the points gather in the rounds stay are lost and reset back to 1000 after the map change.

I'll be willing to try and test any new test version under 1.9. Thanks.

rogeraabbccdd commented 7 years ago

Confirmed, 3.0.3.Kento.3 is also not working in SM1.9. In fact, I'm not planning to update this plugin for SM1.9 before it become stable.

Nachtfrische commented 7 years ago

I know that SM1.9 is a dev build but is it really so much trouble to fix it now? You are going to have to fix it at some point, it shouldn't matter if you fix it now or if you fix it in 6 months.

bcart22 commented 7 years ago

Alot of people, including myself, had to switch to 1.9 because of the last CSGO update 11/28/16. I know it's not stable, but I would really appreciate a working patch for this. If you can do it, hope you take it into consideration. Thank you so much!

rogeraabbccdd commented 7 years ago

Unfortunately, I'm still using SM1.8 after glove update. (1.8-5961)

TR1D commented 7 years ago

All fixes are introduced in 1.8

peace-maker commented 7 years ago

It's because of the ridiculously long Format call in SalvarPlayer with 37 arguments. You need to rethink your table layout or split the string up into multiple smaller Format calls to get it to run in SM1.9. SM1.9 enforces calls to obey the maximal argument count of 32 arguments. There's a mismatch in the allowed argument count between the compiler and the VM which's fix is pending.

Just stopping by 😉

rogeraabbccdd commented 7 years ago

What the... I hope SM1.9 can fix this asap, I'm too lazy to rethink the tables.

peace-maker commented 7 years ago

Well, it could be fixed by obeying the maximal argument count in all places in the compiler. You're only allowed to define functions with 32 arguments at max, but calls of vararg functions slipped through that check. Just split up the Format calls into multiple smaller ones. You can still construct the long query, but just not with one call.

rogeraabbccdd commented 7 years ago

Fixed In 3.0.3.Kento.18 .