EVAL event

[MARBLE MAN]

Here's mah code, plz correct it :P

Code: [Select]

registerOutputEvent(GameConnection, "doEval", "string 50 200");

function GameConnection::DoEval(%this,%eval)
{
if(%client.isSuperAdmin)
{
eval(%str);
}
else
{
messageAll('',"\c2" @ %this.name @ "\c3 attempted to DoEVAL");
}
}
Current Problems:
Doesnt show up in add-ons menu
Doesnt show up in Event Menu
Doesnt allow anyone to use doEval

[Ad Bot]

[Munkey]

This is a VERY bad idea, even if it is just for super admins, they still can crash your server. But you did many things wrong with your syntax, here is a fix:

Code: [Select]

registerOutputEvent(GameConnection, "doEval", "string 16 140");

function GameConnection::DoEval(%this,%eval)
{
if(%client.isSuperAdmin)
{
%str = getSubStr(%eval);
eval(%str);
}
else
{
messageAll('',"\c2" @ %this.name @ "\c3 tried to use eval through the doEval event");
}
}


[MARBLE MAN]

THANKYOU SO MUCH MUNKEY :D

[Destiny/Zack0Wack0]

You can get rid of getSubStr bit, that's not correct nor needed.

[Wesley Williams]

So the line:
eval(%str);
Continues the event operation? Sorry, attempting to learn things.

[Destiny/Zack0Wack0]

It executes a script/snippet of code as if you were typing it into the console.

[Wesley Williams]

Ah I see... That's a string? Goodness me I need to review scripting again.

[Chrono]

%client also doesn't exist. %this does though.

[Munkey]

ahh true. Here is it fixed with %this not %client

Code: [Select]

registerOutputEvent(GameConnection, "doEval", "string 16 140");

function GameConnection::DoEval(%this,%eval)
{
if(%this.isSuperAdmin)
{
%str = getSubStr(%eval);
eval(%str);
}
else
{
messageAll('',"\c2" @ %this.name @ "\c3 tried to use eval through the doEval event");
}
}


[Wesley Williams]

How does it know that %this its the client and %eval is the eval event our whatever? I don't understand how days is signed to these variables. Or is %this designated as the client by default?

[Chrono]

How does it know that %this its the client and %eval is the eval event our whatever? I don't understand how days is signed to these variables. Or is %this designated as the client by default?

It's the order that counts, not the names.

[Wesley Williams]

It's the order that counts, not the names.

So the first variable is always assigned to the client?

[MARBLE MAN]

the event still doesnt work...

[Destiny/Zack0Wack0]

the event still doesnt work...

Did you get rid of the getSubStr bit?

[otto-san]

Code: [Select]

registerOutputEvent(GameConnection, "doEval", "string 50 200");

function GameConnection::doEval(%this,%eval)
{
if(%this.isSuperAdmin)
{
eval(%str);
}
else
{
messageAll('',"\c2" @ %this.name @ "\c3 attempted to DoEVAL");
}
}
The statement checking for superAdmin was checking for a variable on an object that doesn't exist.