Then you should be able to just add option to use UserId directly instead of having to convert id to name and then having the module convert it back. You can just use typeof() to check if player is a string (name) or a number (userid)
Alright, I will add this soon in the next major update. Until then, if you want, I can make a temporary function that uses UserId instead of Name.
Out of curiosity, though, why couldn’t you just insert the UserId in your browser and just input the name?
I’m talking about situations where someone would want to use your module in a script that uses userid internally rather than player names. Without userid support, this script would have to convert id to name and then your module would just convert it back anyway what would be slower and could occasionally fail.
Obviously you’ll want to have your module as comfortable for users as possible
1 Like
I am showing the basics of how pcalls work, which you would know if you looked back in the discussion. It is better because it logs the error if there was one. Why do I even have to answer this? I feel like my posts explains this itself.
P.S. look who I was responding to for that post mate
Also any call that makes any type of request to the internet like The ones you listed are bound to fail.
Also any call that makes any type of request to the internet like The ones you listed are bound to fail.
Yes they are, this is why I suggest you retry them. Retrying when a pcall fails is good practice, because they mainly fail due to the server not being able to process the request so you have to retry for that.
It is better because it logs the error if there was one. Why do I even have to answer this?
You log the error after you have finished retrying the pcall and the pcall still wasn’t successful. You should know this already…
i think pcalls are easier than xpcalls
Just to let you know, this entire command can be easily bypassed with exploits.
1 Like
Why use Restrictly over EZban and BanIt?
Is there anything special with Restrictly over all the other modules?
4 Likes
I still use xpcalls, I find xpcall and pcall equally easy to do
It’s your choice whether to use it or not on exploiters. However, I’ll try to come up with a better ban option.
Well, over EZBan (unless I’m mistaken), there’s no Server ban/unban option.
It’s probably not meant to be used against exploiters. If you have an exploiter, you’ll usually want to just ban them on sight anyway.
1 Like
Should an option to ban, where you input UserId be implemented into Restrictly.Ban()?
UserId Ban Option
- Yes
- No
0 voters
If Yes, would you like a temporary ban option such as Restrictly.IdBan() whilst it is being implemented?
Temporary UserId Ban
- Yes
- No
0 voters
That’s not a valid point over why people should use this module, the code has major flaws and is highly inefficient, yet alone useless functions.
What do you mean by this? The only ‘useless’ function is the information command.
If you mean the Restrict and Unrestrict, they’re not useless. If you’re seeing this in a scenario where you’re using it against exploiters, you always have the choice not to, as pointed out by @Acu1000
This is a cool module, but why should I use this module over say, adonis or EzBan or BanIt or any other ban/admin system out there? What are the pros of this system?
I think for now it’s still in development so you shouldn’t expect anything too special yet
1 Like
Nothing much for now. It offers I guess a few more features than EZBan (ServerBan/Unban). I might be reworking the module with a colleague soon, though, with more optimized code.
You guys seem to want a new UserId ban option! It’ll be coming soon.
Another poll: Would you guys like a feature where it scans servers every X seconds for a player, so if they’re not in the same server as you they still get banned?
Scan Server Feature
- Yes
- No
0 voters
If Yes, how often should it search?
Server Scan Time (seconds)
- 10
- 15
- 30
- 45
- 60
- Other (Reply with time)
0 voters
Update Log 3.4
Update Log 3.4
-
Restrictly.IdBan()is out! Documentation will be added soon.