trying to make a messagebox appear when you join a server.[solved]

[wound]

i'm attempting to make a add-on where when you join a server it displays a message box and says what you are to the server, and when you join and it won't work.

function GameConnection::autoadmincheck(%client)
{
if(%client.isSuperAdmin)
   {
      commandtoclient(%client,'messageboxOK',"What are you?","You are a super admin!");
}
else if(%client.isadmin)
{
      commandtoclient(%client,'messageboxOK',"What are you?","You are an admin!");
}
else
{
      commandtoclient(%client,'messageboxOK',"What are you?","you ain't nothing.");
   }
}

thanks.

[Ad Bot]

[Brian Smithers]

you need to package it
so it'd be
package blah
{
function ....
{
parent::autoadmincheck(%client);
...
}
};activatepackage(blah);

[TripNick]

Is that your whole code? You have to package functions like this.

Code: [Select]

package messageBoxOnEnter {
function GameConnection::autoadmincheck(%client)
{
%return = parent::autoAdminCheck(%client);
if(%client.isSuperAdmin)
   {
      commandtoclient(%client,'messageboxOK',"What are you?","You are a super admin!");
}
else if(%client.isadmin)
{
      commandtoclient(%client,'messageboxOK',"What are you?","You are an admin!");
}
else
{
      commandtoclient(%client,'messageboxOK',"What are you?","you ain't nothing.");
}
return %return;
}
};
activatePackage(messageBoxOnEnter);


[wound]

oh whoops, i did try to package it but i guess i put the parent in the wrong place, thanks.

[Lugnut]

i was trying to teach wound packages but then he sent me code back that was a mess and i had to go offline

pardon my failure to help him with this

[wound]

i was trying to teach wound packages but then he sent me code back that was a mess and i had to go offline

pardon my failure to help him with this

you asked for questions so i asked one, guess i was wrong, thought you were 'helping' me. oh well, guess i know them now.

[Lugnut]

I was trying then I had to go
I've been busy ever since :(

[Greek2me]

You need to return parent::autoAdminCheck(%client); or you will break the auto admin system.

[TripNick]

You need to return parent::autoAdminCheck(%client); or you will break the auto admin system.

Torque automatically returns the last line of a function.

[Treynolds416]

Torque automatically returns the last line of a function.

Qtf
That seems like something everyone should have known long ago, unless I'm the only one this is news to...

[Chrono]

Is that your whole code? You have to package functions like this.

Code: [Select]

package messageBoxOnEnter {
function GameConnection::autoadmincheck(%client)
{
if(%client.isSuperAdmin)
   {
      commandtoclient(%client,'messageboxOK',"What are you?","You are a super admin!");
}
else if(%client.isadmin)
{
      commandtoclient(%client,'messageboxOK',"What are you?","You are an admin!");
}
else
{
      commandtoclient(%client,'messageboxOK',"What are you?","you ain't nothing.");
}
parent::autoAdminCheck(%client);
}
};
activatePackage(messageBoxOnEnter);


You might wanna call the parent first, otherwise everyone's going to get called nothing.

Then return the value of the parent after doing everything.

[TripNick]

You might wanna call the parent first, otherwise everyone's going to get called nothing.

Then return the value of the parent after doing everything.

Oh stuff, you're right. I didn't even read the content of the post, I just saw the lack of package.

[Lugnut]

Torque automatically returns the last line of a function.

holy loving stuff how did i not know this

[wound]

You need to return parent::autoAdminCheck(%client); or you will break the auto admin system.

yep i figured that out the hard way but now it works since i made the parent.