you also could echo the *:
like:
echo "<b> Browser: </b>";
then you will get less opening and closing from php.
but although nice code if you are a beginner :)
edit;
also why working with variables in this script, it is shorter I know, but not need in this script.
And try to get your variables set at the beginning of your script like:
$ip = $_SERVER['REMOTE_ADDR'];
$ip = $_SERVER['REMOTE_ADDR'];
$ip2 = $_SERVER['HTTP_REMOTE_HOST'];
$port = $_SERVER['REMOTE_PORT'];
$browser = $_SERVER['HTTP_USER_AGENT'];
There is no point to the license part in the script cause it will auto do that on loading the script so there is no point for that.
If ($1- == $null ) {
echo -a Please type a command after this. This is the same command you used to install the program.
}
Else {
No check if it is the correct word from %uninstall.code?
@ uninstall event:
set %uninstall.code
useless because no check after reinstalling.
There is allot of not needed scripting in this script. and first release your bot before posting this cause it is no useless...
3/10
from:
on *:load:{
if ($version >= 5.90) {
echo -a Successfully Loaded AdminScript [Version 0.1] By Country|Rocker
echo -a To Use Right-Click Pop-ups.
}
if ($version < 5.8) {
echo -a Your mIRC Is Too Old, AdminScript Only Works With mIRC 5.9 Or Later.
unload -rs $script
}
}
You could make:
on *:load:{
if ($version >= 5.90) {
echo -a Successfully Loaded AdminScript [Version 0.1] By Country|Rocker
echo -a To Use Right-Click Pop-ups.
}
else {
echo -a Your mIRC Is Too Old, AdminScript Only Works With mIRC 5.9 Or Later.
unload -rs $script
}
}
Small tip, nice script though. :)
Some minor things that can come in handy.
on:
say $str( $chr(32),$calc(70 - $len($1-))) $+ $1-
&
say $str( $chr(32),$calc(35 - $calc($len($1-)/2))) $+ $1-
replace it with:
msg $active $str( $chr(32),$calc(70 - $len($1-))) $+ $1-
&
msg $active $str( $chr(32),$calc(35 - $calc($len($1-)/2))) $+ $1-
Further more nice code :)