Author Topic: /ban command  (Read 4444 times)

0 Members and 1 Guest are viewing this topic.

Offline Furai

  • Administrator
  • Veteran
  • *****
  • Posts: 1908
    • TransHuman Design
/ban command
« on: January 15, 2010, 09:13:08 am »
I still think that this command is a bit too simple. We should be able to give custom ban reasons and lenght. What do you think? And as far as I remember there was way to give custom lenght for bans through arsse, not in-game...
"My senses are so powerful that I can hear the blood pumping through your veins."

Offline EnEsCe

  • Retired Soldat Developer
  • Flamebow Warrior
  • ******
  • Posts: 3101
  • http://enesce.com/
    • [eC] Official Website
Re: /ban command
« Reply #1 on: January 15, 2010, 11:29:51 am »
There is the /tempban command for length bans. It is not an ARSSE feature, it can be done ingame using the aforementioned command and any remote admin.

DarkCrusade

  • Guest
Re: /ban command
« Reply #2 on: January 15, 2010, 11:49:03 am »
But still, don´t you think we admins are way too limited in our powers? Sure, we can kick/ban/whatever, but there is no advanced command system such as Wookash mentioned. I´d like to see Wookashs suggestion and in addition it would be nice if I was able to ban certain nicknames from the server so that I wouldn´t need to have a script that kicks majors automatically. If someone joined with a wrong nickname it would give him the error message >>You can´t join this server with that nickname<<. Would be handy :)
« Last Edit: February 19, 2011, 02:23:20 am by DarkCrusade »

Offline Furai

  • Administrator
  • Veteran
  • *****
  • Posts: 1908
    • TransHuman Design
Re: /ban command
« Reply #3 on: January 15, 2010, 03:06:07 pm »
AFAIK /tempban works only with IP but not with player's ID.
"My senses are so powerful that I can hear the blood pumping through your veins."

Offline Horve

  • Flagrunner
  • ****
  • Posts: 692
  • Vig
Re: /ban command
« Reply #4 on: January 20, 2010, 06:39:45 am »
afaik /ban bans for 30 days, after that, the banned boi is banned no more.
/tempban -1 bans for an eternity, but ya, you need to know the ip >>> arsse

Offline Furai

  • Administrator
  • Veteran
  • *****
  • Posts: 1908
    • TransHuman Design
Re: /ban command
« Reply #5 on: January 20, 2010, 08:29:34 am »
So it should work with player's ID. And we could use syntax like this: 1w 2d 3h.
Where "w" stands for weeks; d -days; h - hours.

f.e.:
 /tempban 4 1w5d4h Offensive language.
This should ban player with ID number 4 for one week 5 days and 4 hours with reason "Offensive language.".
"My senses are so powerful that I can hear the blood pumping through your veins."

Offline Shoozza

  • Retired Soldat Developer
  • Veteran
  • ******
  • Posts: 1632
  • Soldat's Babysitter
    • Website
Re: /ban command
« Reply #6 on: February 19, 2011, 12:25:06 am »
Because I have been linked to this topic several times this month here my opinion:

First: This is not going into soldat 1.5.1.

I noticed that there is a major inconsitency in how the commands work.
/tempban bans by IP
/ban bans by nickname
/banip bans by IP

We should think about how to reimplement them.

My first idea:
/ban => ban forever
/tempban => ban for timeperiod
/bannick => ban nickname forever
/tempbannick => ban nickname for timeperiod

However this seems to be not that easy. We need to support banning:
Ban IP
Ban Hardware ID
Ban Nickname

But we can use different input parameters:
Use IP as param
Use hardware ID as param
Use nickname as param
Use player ID as param

Also timeperiods can be different:
Use minutes as timeperiod
Use hours as timeperiod
Use days as timeperiod
Use month as timeperiod
Use years as timeperiod
Use date as timeperiod

I started thinking about the commands but it's just not right:
/ban <player nickname>
/banhw <player hardware ID>
/banid <player ID>
/banip <player IP>

/bannick <player nickname>
/bannickhw <player hardware ID>
/bannickid <player ID>
/bannickip <player IP>

/tempban <player nickname> <time in minutes>
/tempbanhw <hardware ID> <time in minutes>
/tempbanid <player ID> <time in minutes>
/tempbanip <player IP> <time in minutes>

/tempbannick <player nickname> <time in minutes>
/tempbannickhw <player hardware ID> <time in minutes>
/tempbannickid <player ID> <time in minutes>
/tempbannickip <player IP> <time in minutes>

These are not all commands I could create from the requirements and there are already too many of them.
So tell me how to fix that...
Rules
Tools: ARSSE - SARS - SRB - chatMod

Offline jettlarue

  • Flagrunner
  • ****
  • Posts: 724
Re: /ban command
« Reply #7 on: February 19, 2011, 12:36:19 am »
/ban <name/ipaddress/hardwareid> <timetype> <timeamount>

In-between first <> could be for example-string: "MajorNoob" or "65.181.171.111" or whatever hardwareid is.
Second-string : "minutes", "hours", "days", "months", "forever".
Third-int: "5"
If forever is specified no need to check howlong.

If only /ban <name> make it by default 30 minutes maybe?
If only /ban<name> <timetype> then give each one a default of thirty also maybe?

Tempban could work same way.

Offline Falcon`

  • Flagrunner
  • ****
  • Posts: 792
  • A wanted lagger
Re: /ban command
« Reply #8 on: February 19, 2011, 06:08:55 am »
instead of <timetype> i suggest to just type time like "1d 5h 1m"
If you're not paying for something, you're not the customer; you're the product being sold.
- Andrew Lewis

Always code as if the guy who ends up maintaining your code will be a violent psychopath who knows where you live.

Offline SyavX

  • Soldat Beta Team
  • Camper
  • ******
  • Posts: 338
Re: /ban command
« Reply #9 on: February 19, 2011, 07:54:16 am »
I suggest to use 2 types of ban commands:
ban, tempban     - to ban by nick only
ban#, tempban# - to ban by id/ip/hwid

/ban <nick>
/tempban <nick> [timeperiod]

/ban# <id/ip/hwid>
/tempban# <id/ip/hwid> [timeperiod]

where
[timeperiod] - optional "1w2d3h4m"-formatted string (w, d, h, m should be enough)

Due to different formats of id/ip/hwid  parameters you can easily determine which one was used in the command.

This will help to avoid mess with situations like this: we have 4 players on the server
<id><nick>  <ip>  <hwid>
1Major  127.0.0.1  hwid-1
21  127.0.0.2  hwid-2
3127.0.0.2  127.0.0.3  hwid-3
4hwid-3  127.0.0.4  hwid-4

How to ban a player with nick "1" or "127.0.0.2", or "hwid-3"?
There should be no problem with new ban, ban# commands...
« Last Edit: February 19, 2011, 05:14:26 pm by SyavX »

Offline Furai

  • Administrator
  • Veteran
  • *****
  • Posts: 1908
    • TransHuman Design
Re: /ban command
« Reply #10 on: February 19, 2011, 08:30:47 am »
Permban should be just ban for -1 minutes.

As to other ideas - let's check how it's done in other games, maybe you'll find there inspiration.

And where the hell <reason> got lost? :)
"My senses are so powerful that I can hear the blood pumping through your veins."

Offline L[0ne]R

  • Soldat Beta Team
  • Rainbow Warrior
  • ******
  • Posts: 2079
  • need a life. looking for donors.
Re: /ban command
« Reply #11 on: February 19, 2011, 05:57:05 pm »
Eh, this could be made even simpler:

[/ban] [1] [2] [3]

[1] Who - format:
Nick: "noob"  (with quotes)
ID: #
IP: ###.###.###.###
HW: no clue what it looks like..


[2] Timeperiod - format:
#m - minutes
#h - hours
#d - days
 0 - unban
-1 - permaban
#d#h#m - if you want to be more specific.

Optional parameter, /ban [1] [3] should still work.
If timeperiod is specified - use that (same as /tempban).
If timeperiod is not specified - use default timeperiod (same as /ban).


[3] Reason for kick - format:
"Blah blah string"  (with quotes)

Optional parameter, /ban [1] [2] should still work.
« Last Edit: February 19, 2011, 06:03:14 pm by L[0ne]R »

Offline Shoozza

  • Retired Soldat Developer
  • Veteran
  • ******
  • Posts: 1632
  • Soldat's Babysitter
    • Website
Re: /ban command
« Reply #12 on: February 19, 2011, 06:03:15 pm »
@L[0ne]R
nick and HW could be the same.
or ID and nick could be the same.

unban with 0 is a bit weird
Rules
Tools: ARSSE - SARS - SRB - chatMod

Offline L[0ne]R

  • Soldat Beta Team
  • Rainbow Warrior
  • ******
  • Posts: 2079
  • need a life. looking for donors.
Re: /ban command
« Reply #13 on: February 19, 2011, 06:09:39 pm »
@L[0ne]R
nick and HW could be the same.
or ID and nick could be the same.

unban with 0 is a bit weird

Uh, looks like quotes won't really help since they can be used in nicknames too. The only solution I can think of is restricting certain characters to not be allowed in nicknames.
But wait.. even if someone has a funky nickname, you should still be able to do [/ban #] which will ban by ID and you won't have to use nickname for that (banning by nickname is mostly useless anyway).

As for /ban 0 - was just a different version of /unban, I didn't mean it should completely replace /unban. More like just a little bonus to go with /ban command.
« Last Edit: February 19, 2011, 06:24:35 pm by L[0ne]R »

Offline Deck

  • Major(1)
  • Posts: 4
Re: /ban command
« Reply #14 on: February 19, 2011, 06:12:40 pm »
I think your variants are too difficult. one may to simplify:

/ban        <nick|id|hwid|ip>            [reason]
/tempban <nick|id|hwid|ip> <time>  [reason]
/permban <nick|id|hwid|ip>            [reason]

As SyavX said, you can easy determine parameters.
I dont imagine a better variant than using in nordicbots, except "w" is useless.

Offline Falcon`

  • Flagrunner
  • ****
  • Posts: 792
  • A wanted lagger
Re: /ban command
« Reply #15 on: February 19, 2011, 06:14:00 pm »
oh btw, you could add some /showbans command as well
If you're not paying for something, you're not the customer; you're the product being sold.
- Andrew Lewis

Always code as if the guy who ends up maintaining your code will be a violent psychopath who knows where you live.

Offline L[0ne]R

  • Soldat Beta Team
  • Rainbow Warrior
  • ******
  • Posts: 2079
  • need a life. looking for donors.
Re: /ban command
« Reply #16 on: February 19, 2011, 06:20:32 pm »
I think your variants are too difficult. one may to simplify:

/ban        <nick|id|hwid|ip>            [reason]
/tempban <nick|id|hwid|ip> <time>  [reason]
/permban <nick|id|hwid|ip>            [reason]
Thats pretty much what I wrote, except I was talking about just one command:

/ban     <nick|id|hwid|ip>   <time>   <reason>    = tempban
/ban     <nick|id|hwid|ip>   <time>   <reason>    = ban
/ban     <nick|id|hwid|ip>   <-1>     <reason>    = permaban
« Last Edit: February 19, 2011, 06:23:26 pm by L[0ne]R »

Offline Illuminatus

  • Camper
  • ***
  • Posts: 442
  • ...soldat-freak since 2004...
Re: /ban command
« Reply #17 on: February 19, 2011, 11:29:10 pm »
@L[0ne]R
nick and HW could be the same.
or ID and nick could be the same.
The only solution I can think of at the moment: Remove the possiblity to ban someone by nick.
Advantage: This would solve the problem and make L[0ne]R's suggestion the best imo.
Disadvantage: When you're ingame you need to press F1 to show the players-list everytime you want to ban someone (because you don't know his ID). Maybe you could let players-list show up automatically everytime you write "/ban".
No difference between man and mouse - both end up in pussy.

Offline Deck

  • Major(1)
  • Posts: 4
Re: /ban command
« Reply #18 on: February 20, 2011, 03:49:28 pm »
I thought about that and understood that this is not the solution.
I believe that admins more use auto-complete nick rather than other.
I'd suggest that typing any characters isnt required for autocomplete nick.

Offline Bonecrusher

  • Global Moderator
  • Veteran
  • *****
  • Posts: 1397
  • High above
    • Zabijaka.pl
Re: /ban command
« Reply #19 on: October 13, 2011, 09:19:31 am »
f12 for ban reasons and lenght

/ban id reason lenght  -duration specified by lenght parameter
/ban id reason - 30 days

Im chill like that