Ban System - Ban Command with Reason

[Sanchez 2402]

"UPDATE account.account SET status = 'BLOCK', availDt = FROM_UNIXTIME(UNIX_TIMESTAMP(CURRENT_TIMESTAMP()) + %i), reason = '%s' WHERE id = '%d'"

Try this.

[Avalonking 0]

Sorry but i have game 34k, where is "Now open game/cmd.cpp" ??

[Great 16]

fo reason with spaces use

std::string line(argument);
reason = line.substr(line.find(arg2) + 2).c_str();

works:)

[Lehel 14]

Sorry but i have game 34k, where is "Now open game/cmd.cpp" ??

 

I think you can't do that,because the metin2source is 40k .

[Saud bin abdullah 7]

awesome !!

[MrLibya 535]

Nice , Thx

 

But Why You Use AvailDt ? Why You Not Make It Change The State To " BLOCK " ?

[Sanchez 2402]

Because then the player will be banned for forever.

[MrLibya 535]

Can't Make Timer For It ?

[Denis 1454]

Why?There is already availdt to do that...

[MrLibya 535]

I Know . But I Was Just Asking !

[Damixxx0101 6]

ACMD(do_ban)

Now time to add the complete code to ACMD(do_ban):
 
?
// Args
char arg1[256], arg2[256], arg3[256];
 
// Local variables
const char*  szName;
const char*  szReason;
int          iDuration;
 
one_argument(two_arguments(argument, arg1, sizeof(arg1), arg2, sizeof(arg2)), arg3, sizeof(arg3));
 
// Invalid syntax
if (!*arg1 || !*arg2 || !*arg3)
{
    ch->ChatPacket(CHAT_TYPE_INFO, "Invalid Syntax, usage: <player name> <time in hours> <reason> tip: don't use spaces in the reason, use _");
    return;
}
 
szName      = arg1;
iDuration   = atoi(arg2);
szReason    = arg3;
 
if (iDuration <= 0)
{
    ch->ChatPacket(CHAT_TYPE_INFO, "Duration can't be 0 or minus.");
    return;
}
 
LPCHARACTER tch = CHARACTER_MANAGER::instance().FindPC(szName);
 
if (!tch)
{
    ch->ChatPacket(CHAT_TYPE_INFO, "%s is not playing", szName);
    return;
}
 
if (!tch->GetDesc())
{
    ch->ChatPacket(CHAT_TYPE_INFO, "%s don't have desc", szName);
    return;
}
 
if (tch == ch)
{
    ch->ChatPacket(CHAT_TYPE_INFO, "What's wrong with you? Don't ban yourself");
    return;
}
 
if (tch->GetGMLevel() > GM_PLAYER)
{
    ch->ChatPacket(CHAT_TYPE_INFO, "Do not ban GMs");
    return;
}
 
std::auto_ptr<SQLMsg> msg(DBManager::instance().DirectQuery("UPDATE account.account SET availDt = FROM_UNIXTIME(UNIX_TIMESTAMP(CURRENT_TIMESTAMP()) + %i), reason = '%s' WHERE id = %d", iDuration * 3600, szReason, tch->GetDesc()->GetAccountTable().id));
 
tch->GetDesc()->DelayedDisconnect(5);
 
sys_log(0, "%s[%d] banned %s for %i hours with reason: %s", ch->GetName(), ch->GetPlayerID(), szName, iDuration, szReason);
 
ch->ChatPacket(CHAT_TYPE_INFO, "%s has been banned for %i hours with reason: %s", szName, iDuration, szReason);

 not work for me spam fully error i have tab

 

please give me this file cpp i look where is i create wrong

[antrax16 2]

Not work for me this function i am eroor to compile .

What is problem .

[Ken 889]

ACMD(do_ban)
{
	char szName[CHARACTER_NAME_MAX_LEN + 1];
	char szReason[512];
	int iDuration;
	
	one_argument(two_arguments(argument,szName,sizeof(szName),szReason,sizeof(szReason)),iDuration,sizeof(iDuration));
	
	// Invalid syntax
	if (!*arg1 || !*arg2 || !*arg3)
	{
		ch->ChatPacket(CHAT_TYPE_INFO, "Invalid Syntax, usage: <player name> <time in hours> <reason> tip: don't use spaces in the reason, use _");
		return;
	}
	
	str_to_number(iDuration,arg2);
	
	if (iDuration <= 0)
	{
		ch->ChatPacket(CHAT_TYPE_INFO, "Duration can't be 0 or minus.");
		return;		
	}
	
	LPCHARACTER tch = CHARACTER_MANAGER::instance().FindPC(szName);
	
	if (!tch->GetDesc())
	{
		ch->ChatPacket(CHAT_TYPE_INFO, "%s don't have desc", szName);
		return;
	}
	  
	if (tch == ch)
	{
		ch->ChatPacket(CHAT_TYPE_INFO, "What's wrong with you? Don't ban yourself");
		return;
	}
	  
	if (tch->GetGMLevel() > GM_HIGH_WIZARD)
	{
		ch->ChatPacket(CHAT_TYPE_INFO, "Do not ban GMs");
		return;
	}
	
	std::auto_ptr<SQLMsg> msg(DBManager::instance().DirectQuery("UPDATE account.account SET availDt = FROM_UNIXTIME(UNIX_TIMESTAMP(CURRENT_TIMESTAMP()) + %d), reason = '%s' WHERE id = %u", iDuration * 3600, szReason, tch->GetDesc()->GetAccountTable().id));
	
	if(msg.get() && msg->Get()->uiAffectedRows == 0 || msg->Get()->uiAffectedRows == (uint32_t)-1)
	{
		ch->ChatPacket(CHAT_TYPE_INFO,"do_ban failed : can not modify account table");
		return;
	}
	
	tch->GetDesc()->DelayedDisconnect(5);
	  
	sys_log(0, "%s[%u] banned %s for %d hours with reason: %s", ch->GetName(), ch->GetPlayerID(), szName, iDuration, szReason);
	  
	ch->ChatPacket(CHAT_TYPE_INFO, "%s has been banned for %d hours with reason: %s", szName, iDuration, szReason);	
}

Kind Regards

Ken ~ TA

Edited February 17, 2015 by Ken

[Lyrex 6]

Why are you guys still using std::auto_ptr? It's old, deprecated and unsafe. Just use sse std::unique_ptr instead.

[Galet 496]

unique_ptr is used for gcc48+ I guess.

 

However it's use it

[antrax16 2]

ACMD(do_ban)
{
	char szName[CHARACTER_NAME_MAX_LEN + 1];
	char szReason[512];
	int iDuration;
	
	one_argument(two_arguments(argument,arg1,sizeof(arg1),arg2,sizeof(arg2)),arg3,sizeof(arg3));
	
	// Invalid syntax
	if (!*arg1 || !*arg2 || !*arg3)
	{
		ch->ChatPacket(CHAT_TYPE_INFO, "Invalid Syntax, usage: <player name> <time in hours> <reason> tip: don't use spaces in the reason, use _");
		return;
	}
	
	str_to_number(iDuration,arg2);
	
	if (iDuration <= 0)
	{
		ch->ChatPacket(CHAT_TYPE_INFO, "Duration can't be 0 or minus.");
		return;		
	}
	
	LPCHARACTER tch = CHARACTER_MANAGER::instance().FindPC(szName);
	
	if (!tch->GetDesc())
	{
		ch->ChatPacket(CHAT_TYPE_INFO, "%s don't have desc", szName);
		return;
	}
	  
	if (tch == ch)
	{
		ch->ChatPacket(CHAT_TYPE_INFO, "What's wrong with you? Don't ban yourself");
		return;
	}
	  
	if (tch->GetGMLevel() > GM_HIGH_WIZARD)
	{
		ch->ChatPacket(CHAT_TYPE_INFO, "Do not ban GMs");
		return;
	}
	
	std::auto_ptr<SQLMsg> msg(DBManager::instance().DirectQuery("UPDATE account.account SET availDt = FROM_UNIXTIME(UNIX_TIMESTAMP(CURRENT_TIMESTAMP()) + %d), reason = '%s' WHERE id = %u", iDuration * 3600, szReason, tch->GetDesc()->GetAccountTable().id));
	
	if(msg.get() && msg->Get()->uiAffectedRows == 0 || pmsg->Get()->uiAffectedRows == (uint32_t)-1)
	{
		ch->ChatPacket(CHAT_TYPE_INFO,"do_ban failed : can not modify account table");
		return;
	}
	
	tch->GetDesc()->DelayedDisconnect(5);
	  
	sys_log(0, "%s[%u] banned %s for %d hours with reason: %s", ch->GetName(), ch->GetPlayerID(), szName, iDuration, szReason);
	  
	ch->ChatPacket(CHAT_TYPE_INFO, "%s has been banned for %d hours with reason: %s", szName, iDuration, szReason);	
}

Kind Regards

Ken ~ TA

 

I add this code but .

cmd_gm.cpp: In function 'void do_ban(CHARACTER*, const char*, int, int)':
cmd_gm.cpp:4407: error: 'arg1' was not declared in this scope
cmd_gm.cpp:4407: error: 'arg2' was not declared in this scope
cmd_gm.cpp:4407: error: 'arg3' was not declared in this scope
cmd_gm.cpp:4446: error: 'pmsg' was not declared in this scope
Makefile:119: recipe for target 'OBJDIR/cmd_gm.o' failed
gmake: *** [OBJDIR/cmd_gm.o] Error 1
root@metin2:/usr/src/Server/game/src #

[Ken 889]

Sorry. Post is updated again.

 

Kind Regards

Ken ~ TA

[Ken 889]

Why are you guys still using std::auto_ptr? It's old, deprecated and unsafe. Just use sse std::unique_ptr instead.

 

GCC 4.2 is not support that. As galet said. If you wanna use that, you can use it. Meanwhile new command for gcc 4.8+

ACMD(do_ban)
{
    char szName[CHARACTER_NAME_MAX_LEN + 1];
    char szReason[512];
    int iDuration;
     
    one_argument(two_arguments(argument,szName,sizeof(szName),szReason,sizeof(szReason)),iDuration,sizeof(iDuration));
     
    // Invalid syntax
    if (!*arg1 || !*arg2 || !*arg3)
    {
        ch->ChatPacket(CHAT_TYPE_INFO, "Invalid Syntax, usage: <player name> <time in hours> <reason> tip: don't use spaces in the reason, use _");
        return;
    }
     
    str_to_number(iDuration,arg2);
     
    if (iDuration <= 0)
    {
        ch->ChatPacket(CHAT_TYPE_INFO, "Duration can't be 0 or minus.");
        return;     
    }
     
    LPCHARACTER tch = CHARACTER_MANAGER::instance().FindPC(szName);
     
    if (!tch->GetDesc())
    {
        ch->ChatPacket(CHAT_TYPE_INFO, "%s don't have desc", szName);
        return;
    }
       
    if (tch == ch)
    {
        ch->ChatPacket(CHAT_TYPE_INFO, "What's wrong with you? Don't ban yourself");
        return;
    }
       
    if (tch->GetGMLevel() > GM_HIGH_WIZARD)
    {
        ch->ChatPacket(CHAT_TYPE_INFO, "Do not ban GMs");
        return;
    }
     
    std::unique_ptr<SQLMsg> msg(DBManager::instance().DirectQuery("UPDATE account.account SET availDt = FROM_UNIXTIME(UNIX_TIMESTAMP(CURRENT_TIMESTAMP()) + %d), reason = '%s' WHERE id = %u", iDuration * 3600, szReason, tch->GetDesc()->GetAccountTable().id));
     
    if(msg.get() && msg->Get()->uiAffectedRows == 0 || msg->Get()->uiAffectedRows == (uint32_t)-1)
    {
        ch->ChatPacket(CHAT_TYPE_INFO,"do_ban failed : can not modify account table");
        return;
    }
     
    tch->GetDesc()->DelayedDisconnect(5);
       
    sys_log(0, "%s[%u] banned %s for %d hours with reason: %s", ch->GetName(), ch->GetPlayerID(), szName, iDuration, szReason);
       
    ch->ChatPacket(CHAT_TYPE_INFO, "%s has been banned for %d hours with reason: %s", szName, iDuration, szReason);  
}

Kind Regards

Ken ~ TA

Edited February 17, 2015 by Ken

[Shichirojii 8]

edit

[Ken 889]

edit

 

You are removed me and you are say many thing. I can change my nickname and that's too normal. But i am so sorry for you. You are bought this from me also my customer is bought this from me but him system working perfectly

 

what ever #edit

 

Kind Regards

Ken ~ TA

[Damixxx0101 6]

compile cmd_gm.cpp
cmd_gm.cpp: In function 'void do_ban(CHARACTER*, const char*, int, int)':
cmd_gm.cpp:2816: error: only constructors take base initializers
cmd_gm.cpp:2818: error: expected identifier before 'char'
cmd_gm.cpp:2818: error: expected `(' before 'char'
cmd_gm.cpp:2818: error: expected `{' before 'char'
cmd_gm.cpp: At global scope:
cmd_gm.cpp:2825: error: expected constructor, destructor, or type conversion before '(' token
cmd_gm.cpp:2828: error: expected unqualified-id before 'if'
cmd_gm.cpp:2834: error: expected constructor, destructor, or type conversion before '=' token
cmd_gm.cpp:2835: error: expected constructor, destructor, or type conversion before '=' token
cmd_gm.cpp:2836: error: expected constructor, destructor, or type conversion before '=' token
cmd_gm.cpp:2838: error: expected unqualified-id before 'if'
cmd_gm.cpp:2846: error: expected unqualified-id before 'if'
cmd_gm.cpp:2852: error: expected unqualified-id before 'if'
cmd_gm.cpp:2858: error: expected unqualified-id before 'if'
cmd_gm.cpp:2864: error: expected unqualified-id before 'if'
cmd_gm.cpp:2872: error: expected constructor, destructor, or type conversion before '->' token
cmd_gm.cpp:2874: error: expected constructor, destructor, or type conversion before '(' token
cmd_gm.cpp:2876: error: expected constructor, destructor, or type conversion before '->' token
cmd_gm.cpp: In function 'void do_set_stat(CHARACTER*, const char*, int, int)':
cmd_gm.cpp:3997: warning: NULL used in arithmetic
gmake: *** [OBJDIR/cmd_gm.o] Error 1
root@Damian:/usr/src/Source/main/Srcs/Server/game/src #
 

[flygun 298]

Thanks.... i really enjoy reading your topics because you explain every detail

[Galet 496]

Thanks Ken, but "However it's use it" (I don't know why I told this, it's a nonsense)

[JarajTo 40]

How to perm ban?

[Cyclone_DE 9]

How to perm ban?

​Just change the "status" of the account to BLOCK and it will banned for ever, i dont know to make it ingame, if that is what you mean!