NockyCZ / CS2_BombsitesRestrict

This plugin restricts the bombsites of each round if there are less than X players in teams.
20 stars 4 forks source link

[BUG] v1.0.4 Server crash #7

Closed jinnixyz closed 9 months ago

jinnixyz commented 11 months ago

i I updated the plugin version and testet it : If you place the bomb on a blocked spot with the latest version, the server crashes. (The plugin is running and shows the spawn message but does not interrupt the plant and the bomb is not thrown away when plant, then it crashs) Server goes offline and this command appears in console : SERVER IS OFFLINE, exit with code: 139

My Config:

"Minimum players": 5, "Disable plant message": "Less then 5 CTs, bomb can only be planted on {SITE}", "Warning message chat": "You cant plant the bomb on this site!", "Warning message center": "You cant plant the bomb on this site!", "Count bots as players": true, "Message type": 2, "Disabled site": 2, //0 - Random , 1 - A , 2 - B "Which team count as players": 1, //0 - Both teams , 1 - Only CT , 2 - Only T "ConfigVersion": 1

EDIT: i try the plugin with the default config, server also crashed.

asdfxD commented 11 months ago

Plugin works for me, the only thing is on a blocked bombsite while try to plant, the player drops the bomb. No crash.

jinnixyz commented 11 months ago

Plugin works for me, the only thing is on a blocked bombsite while try to plant, the player drops the bomb. No crash.

Whats your CSS Version? idk, i tryed with default config my server crashs everytime i want try to plant the bomb on a blocked bombesite. he dont even throw the bomb away, just plant and crash

asdfxD commented 11 months ago

"Minimum players": 4, "Disable plant message": "Less then 5 CTs, bomb can only be planted on {SITE}", "Warning message chat": "You cant plant the bomb on this site!", "Warning message center": "You cant plant the bomb on this site!", "Count bots as players": true, "Message type": 2, "Disabled site": 2, "Which team count as players": 1, "ConfigVersion": 1

Debian 11 own kernel Pterodactyl SteamRT3 CSS v116

NockyCZ commented 11 months ago

I tried it on the 105 version and it also works fine. I will try it later on the latest version

jinnixyz commented 11 months ago

I tried it on the 105 version and it also works fine. I will try it later on the latest version

I found the mistake. The plugin version does not work in combination with this plugin: https://github.com/K4ryuu/K4-System Is it possible to adapt the plugin so that you can use both plugins in combination? I'm not sure which author has to do something, I noted it as a bug for the other one too. The previous version worked without any problems with the other plugin. If it's not part of your duties then I'm sorry.

jinnixyz commented 11 months ago

i tryed this plugin too : https://github.com/partiusfabaa/cs2-ranks and this crashs the server too. i think its because when you plant the bomb you raise exp points, but when you try to plant your plugin drops the bomb and then the server crashs.

jinnixyz commented 11 months ago

@NockyCZ Can you check for the error please? It has something to do with the fact that he drops the bomb when he places it. Because in the version before where he jumps up while laying the bomb everything worked. I definitely need your plugin but the rank plugin is also important :)

NockyCZ commented 11 months ago

I'll look into it tomorrow :)

jinnixyz commented 11 months ago

i have an idea to fix this problem: can you add an option where you can choose whether he throws the bomb away or, like in the old version, just jumps to prevent him from planting the bomb? or add to disable site option: "Disabled site": , //0 - Random , 1 - A , 2 - B 3 - OFF that only the message is displayed as a notice or do you already have checked the problem? it still doesn't work together even though the Rank plugin released a new version :/

NockyCZ commented 11 months ago

I haven't had time to look at it yet, maybe over the weekend :)

jinnixyz commented 10 months ago

@NockyCZ are you still working with the plugin? I'm currently using the previous version, where you jump as soon as you try to plant the bomb. It's okay, but I would need the setting: It allows setting which team will be considered as the 'Minimum players'. 0 Possible to add? The latest version does not work with a Rankme plugin due to the bomb drop

NockyCZ commented 10 months ago

I already have a new version ready, I'll probably release it this weekend.

NockyCZ commented 10 months ago

@jinnixyz , Try it in v1.0.5

jinnixyz commented 10 months ago

@NockyCZ no change still crashes.... I would use the version before where he doesn't throw the bomb away but just jumps. can you add there this feature maybe? It allows setting which team will be considered as the 'Minimum players'. (0 - Both teams , 1 - Only CT , 2 - Only T)

jinnixyz commented 9 months ago

@NockyCZ :/

NockyCZ commented 9 months ago

@jinnixyz Try it in version 1.0.6

jinnixyz commented 9 months ago

now it works thank you. A very good idea to simply “remove” the other bomb site completely. i close <3