Ban System/Custom Ban Command

[ikanbuntal12 (FishyDeveloper)]

@xxIamInevitable

if playerToBan then
   if duration then
       if reason then

    end
 end
end

and this to

if playerToBan and duration and reason then
  --code
end

it will check three thing without doing that huge if statement

[nonraymonable (raymond)]

How would I convert the duration variable to days in real time instead of UNIX time? I’m making a gui and having the time a player is banned for would be nice.

[xxIamInevitable (FARKS_GAMING)]

os.time() is in real time though, it uses UTC unlike tick() which is in unix

[nonraymonable (raymond)]

I meant something easily readable like

“You have been banned for 3 days” or something

[xxIamInevitable (FARKS_GAMING)]

Use math.abs to determine the remaining days between the start and the end.

[xxIamInevitable (FARKS_GAMING)]

I separated it so that it could print within SUccess and not success and result

[https_KingPie]

There are plenty of time formatting modules that you can find in #resources:community-resources - just do some research by examining the source code etc. to figure out how to craft your own.

[Ryfiles]

@xxIamInevitable

tostring

@xxIamInevitable

tostring

Your getting the players userid so you will use tonumber

[xxIamInevitable (FARKS_GAMING)]

Not actually, the player’s userid is already an int/number, and we’re converting it into a string because that’s what we’ll be using for the name, and what we’re searching for any key entry/data for the player upon joining.

[nonraymonable (raymond)]

I’m making a UI based off of this, by the way! I need to do a bit more polishing first.

There’s two things wrong with this that my UI fixes:

1. You can’t unban a user without using “/ban name 0 a” and it’s pretty weird.
2. You put the < sign the wrong way here

@xxIamInevitable

if Result.BanStart + Result.BanDuration > os.time() then

My UI also adds a warning message after your ban is over

[xxIamInevitable (FARKS_GAMING)]

Yeah it’s correct. It’s supposed to be < instead of >. Kind of put it in the wrong way with my half-awake self. I’ll be working on an unban soon.

[TheSenorDuck]

Good tutorial. I had a admin system and I was trying to make a ban sytem and this tutorial made it easier and better. Thanks!

[xxIamInevitable (FARKS_GAMING)]

Glad to see I was actually able to help, awesome. (You’re welcome)

Feel free to PM me when you have any inquiries, questions about anything, and all that good stuff. I try to be as active as I can on the DevForums, and I’m looking forward to making some friends on it as well.

[nonraymonable (raymond)]

I’ve uploaded the gui on a seperate post!

ModTools (Ban System Plus) [OUTDATED]

Original tutorial is here I’ve used the tutorial listed above to make a gui for you all! Many already exist but mine includes an extra feature and easy settings. To use it, just place it inside your game! If I release an update, you have to drag the “Settings” ModuleScript out of the original script and put it into the new script. This is only required if you change any settings and would like to keep them, though.

How to use ingame:

To use this ingame, just click the little wrench in the top…

[Rynappel]

I keep getting this weird error?

ServerScriptService.Script:63: invalid argument #2 to ‘sub’ (number expected, got nil)

[vhixloqan (logan)]

What is the format for the command? I cant seem to get the command right

[octav20071 (OctaViano)]

You should change the # thing with 4.

@vhixloqan

What is the format for the command? I cant seem to get the command right

/ban

[Phoenix_Ascended (Phoenix)]

It’s good but the time wont work, if I do for example /ban 100 test it would say banned 00 test and nothing else, no time. so as a result it doesn’t actually ban people, just kicks them

[OnlyJay_WasHere (Jay)]

I actually got the same error too and it was coming from this line:

local duration = string.sub(message, string.find(message, numbers));

Probably because the number in string.find() was nil.

Edit: When you tested it out, did you enter the length of the duration in your message or just the username itself?

[UnrealForUnreal]

Nice tutorial, congratulations.

Just a quick comment/suggestion for anybody (with at least some scripting understanding; you may skip if not) reading this:

@xxIamInevitable

local BanStore = DDS:GetDataStore("BanStore", 2) --scope is optional

@xxIamInevitable

That is what we call a “Scope”, think of it as like a certain folder.

According to the Data Stores official documentation page, the Scopes feature is now a legacy feature and its use is no longer recommended.
According to this Data Stores article’s section, The new recommendation is to follow a more explicit approach to scoping by (in general terms) concatenating the actual scope to the DataStore key. This new approach enhances the use of DataStore:ListKeysAsync()'s first parameter which handles this new approach (and surely more methods yet to be introduced). Besides the fact that you may use multiple scopes with this new approach.
For instance:

Example 1 (simple scoping)

Scope: slot

New scoping:

local slot = "1"
local playerDataStore = DataStoreService:GetDataStore("PlayerDataStore")
local slotDataStoreKey = string.format("%s/slot_%s", userId, slot) -- "userId/slot_1"
-- I use the '/' separator for convenience, nevertheless it may be any character

Legacy scoping:

local slot = "1"
local playerInventoryDataStore = DataStoreService:GetDataStore("PlayerDataStore", slot)
local slotDataStoreKey = tostring(userId) -- "userId"

At this point (in this example), it doesn’t seem terribly different. That’s because simple scoping is supported for both, but the concept multiple scoping is more complex for the legacy format (check out Example 2).

Example 2 (multiple scopes)

Scopes: slot, gold
New scoping:

local playerDataStore = DataStoreService:GetDataStore("PlayerDataStore")
local slot = "1"
local goldDataStoreKey = string.format("%s/slot_%s/gold", userId, slot) -- "userId/slot_1/gold"

Legacy scoping:

Having more than 1 scope is practically impossible without making an extra call to DataStoreService:GetDataStore(), which isn’t desirable in any case because there would be more as minimum 2 separate DataStore calls for the same DataStore, which, in the long run, will overcomplicate everything.

As you may have noticed, a multi-scoped key follows the same format as a simple-scoped key. The new approach’s format is universal for every scope quantity! (Unlike the legacy approach).

---

I’m not saying that the use of the legacy scoping is “wrong”, nevertheless, the use of the new approach is encouraged because new methods may not support legacy approaches. But it’s up to you! You may use whichever approach you feel more comfortable with.

Hope this helps! Happy coding.