[living computer]

May i ask why you are linking to it instead of just attaching the file here? Thank you for 2.0 btw, i had the same problems before. Hope it works now.

[headline]

Originally Posted by living computer May i ask why you are linking to it instead of just attaching the file here? Thank you for 2.0 btw, i had the same problems before. Hope it works now.

That way if any changes are made to GitHub it will always download the latest version.

[Jillchang]

Does this work with votes? If not, is it compatible with custom vote redux?
https://forums.alliedmods.net/showthread.php?p=2097623

[living computer]

Originally Posted by Headline That way if any changes are made to GitHub it will always download the latest version.

Tested it now, seems to work great. Thank you for finally fixing this, also had that SB problem before.

You are a great guy

[Andrey309a]

If you use sourcebans, the ban does it correctly but without name and ip.

Could be able to get the name of the banned and ip?

Thx

[kshishu]

Originally Posted by Andrey309a If you use sourcebans, the ban does it correctly but without name and ip. Could be able to get the name of the banned and ip? Thx

Can someone fix this? PLEASE

[Byte]

The name and IP don't appear because this plugin wasn't designed for sourcebans in particular.

The name can easily be fixed by checking for the sourcebans library and using the relevant sourcebans include for the currently most active sourcebans project: https://github.com/sbpp/sourcebans-pp/

For the IP, there is no native provided by Sourcebans to do a IP ban so you have to either copy code over or do a manual DB insert. I think however that SBpp should have this feature and I opened an issue here: https://github.com/sbpp/sourcebans-pp/issues/267

I also have a minor suggestions for Headline:
Consider adding aliases sm_dcban and sm_dclist, much nicer in my opinion

[headline]

Originally Posted by Byte The name can easily be fixed by checking for the sourcebans library and using the relevant sourcebans include for the currently most active sourcebans project: https://github.com/sbpp/sourcebans-pp/

Except it can't easily be fixed because those natives uses the client's index.

An old solution to the problem - where the names and ips would show up on sourcebans - seems to have broken over time. I pulled it back to a functional state since that happened.

Old solution:

PHP Code:

BanIdentity(steamid, minutes, BANFLAG_AUTHID, reason, "sm_addban", client); 


[Byte]

Originally Posted by Headline Except it can't easily be fixed because those natives uses the client's index. An old solution to the problem - where the names and ips would show up on sourcebans - seems to have broken over time. I pulled it back to a functional state since that happened. Old solution: PHP Code: BanIdentity(steamid, minutes, BANFLAG_AUTHID, reason, "sm_addban", client);

Oh my mistake, I somehow misread and thought the native accepted a custom name.

The previous solution worked for the GameConnect Sourcebans:
https://github.com/GameConnect/SourceBans

Because it used the OnBanIdentity forward.
However even that could only ban the Steam ID or the IP at once.
So you'd need 2 calls to ban both which is a bit meh.

I'll work on the SB++ feature later myself and submit a pull request so its hopefully added soon.

[`666]

Thanks.