]> git.donarmstrong.com Git - infobot.git/blobdiff - src/CommandStubs.pl
* Fix NULL chan being created in %channels
[infobot.git] / src / CommandStubs.pl
index 5cbb9fd2593ac8b3ed7f2dd80eec7a1804fcee8f..2e0bd65253a3265f9b97897e479bae44eb17b604 100644 (file)
@@ -55,7 +55,7 @@ sub parseCmdHook {
     }
 
     if (!defined $cmd) {
-       &WARN("cstubs: cmd == NULL.");
+       &WARN('cstubs: cmd == NULL.');
        return 0;
     }
 
@@ -74,7 +74,7 @@ sub parseCmdHook {
        my %hash = %{ $cmdhooks{$ident} };
 
        if (!scalar keys %hash) {
-           &WARN("CmdHook: hash is NULL?");
+           &WARN('CmdHook: hash is NULL?');
            return 1;
        }
 
@@ -144,7 +144,7 @@ sub parseCmdHook {
            $cmdstats{ $hash{'Cmdstats'} }++;
        }
 
-       &VERB("hooks: End of command.",2);
+       &VERB('hooks: End of command.',2);
 
        $done = 1;
     }
@@ -155,7 +155,7 @@ sub parseCmdHook {
 
 sub Modules {
     if (!defined $message) {
-       &WARN("Modules: message is undefined. should never happen.");
+       &WARN('Modules: message is undefined. should never happen.');
        return;
     }
 
@@ -176,7 +176,7 @@ sub Modules {
     }
 
     # google searching. Simon++
-    my $w3search_regex   = "google";
+    my $w3search_regex   = 'google';
     if ($message =~ /^(?:search\s+)?($w3search_regex)\s+(?:for\s+)?['"]?(.*?)["']?\s*\?*$/i) {
        return unless (&IsChanConfOrWarn('W3Search'));
 
@@ -198,7 +198,7 @@ sub Modules {
        return unless (&IsChanConfOrWarn('Search'));
 
        my $thiscmd     = lc $1;
-       my $args        = $3 || "";
+       my $args        = $3 || '';
 
        $thiscmd        =~ s/^vals$/values/;
        return if ($thiscmd ne 'keys' && $thiscmd ne 'values');
@@ -232,7 +232,7 @@ sub Modules {
        return unless (&IsChanConfOrWarn('Topic'));
 
        my $chan        = $talkchannel;
-       my @args        = split / /, $2 || "";
+       my @args        = split / /, $2 || '';
 
        if (!scalar @args) {
            &msg($who,"Try 'help topic'");
@@ -244,14 +244,14 @@ sub Modules {
 
        # topic over public:
        if ($msgType eq 'public' && $thiscmd =~ /^#/) {
-           &msg($who, "error: channel argument is not required.");
+           &msg($who, 'error: channel argument is not required.');
            &msg($who, "\002Usage\002: topic <CMD>");
            return;
        }
 
        # topic over private:
        if ($msgType eq 'private' && $chan !~ /^#/) {
-           &msg($who, "error: channel argument is required.");
+           &msg($who, 'error: channel argument is required.');
            &msg($who, "\002Usage\002: topic #channel <CMD>");
            return;
        }
@@ -283,7 +283,7 @@ sub Modules {
        my $queue = scalar(keys %wingateToDo);
        if ($queue) {
            $reply .= ".  I have \002$queue\002 hosts in the queue";
-           $reply .= ".  Started the scan ".&Time2String(time() - $wingaterun)." ago";
+           $reply .= '.  Started the scan '.&Time2String(time() - $wingaterun).' ago';
        }
 
        &performStrictReply("$reply.");
@@ -292,7 +292,7 @@ sub Modules {
     }
 
     # do nothing and let the other routines have a go
-    return "CONTINUE";
+    return 'CONTINUE';
 }
 
 # Uptime. xk++
@@ -317,11 +317,11 @@ sub seen {
     $person =~ s/\?*$//;
 
     if (!defined $person or $person =~ /^$/) {
-       &help("seen");
+       &help('seen');
 
-       my $i = &countKeys("seen");
-       &msg($who,"there ". &fixPlural("is",$i) ." \002$i\002 ".
-               "seen ". &fixPlural("entry",$i) ." that I know of.");
+       my $i = &countKeys('seen');
+       &msg($who,'there '. &fixPlural('is',$i) ." \002$i\002 ".
+               'seen '. &fixPlural('entry',$i) .' that I know of.');
 
        return;
     }
@@ -331,11 +331,11 @@ sub seen {
     &seenFlush();      # very evil hack. oh well, better safe than sorry.
 
     # TODO: convert to &sqlSelectRowHash();
-    my $select = "nick,time,channel,host,message";
-    if ($person eq "random") {
-       @seen = &randKey("seen", $select);
+    my $select = 'nick,time,channel,host,message,messagecount';
+    if ($person eq 'random') {
+       @seen = &randKey('seen', $select);
     } else {
-       @seen = &sqlSelect("seen", $select, { nick => $person } );
+       @seen = &sqlSelect('seen', $select, { nick => $person } );
     }
 
     if (scalar @seen < 2) {
@@ -356,18 +356,19 @@ sub seen {
        $reply = "$seen[0] is currently on";
 
        foreach (@chans) {
-           $reply .= " ".$_;
+           $reply .= ' '.$_;
            next unless (exists $userstats{lc $seen[0]}{'Join'});
-           $reply .= " (".&Time2String(time() - $userstats{lc $seen[0]}{'Join'}).")";
+           $reply .= ' ('.&Time2String(time() - $userstats{lc $seen[0]}{'Join'}).')';
        }
 
-       if (&IsChanConf("seenStats") > 0) {
+       if (&IsChanConf('seenStats') > 0) {
            my $i;
-           $i = $userstats{lc $seen[0]}{'Count'};
-           $reply .= ".  Has said a total of \002$i\002 messages" if (defined $i);
+           $i = $seen[6] || $userstats{lc $seen[0]}{'Count'};
+           $reply .= ". Has said a total of \002$i\002 messages" if (defined $i);
            $i = $userstats{lc $seen[0]}{'Time'};
-           $reply .= ".  Is idling for ".&Time2String(time() - $i) if (defined $i);
+           $reply .= '. Is idling for '.&Time2String(time() - $i) if (defined $i);
        }
+       $reply .= ", last said\002:\002 '$seen[4]'.";
     } else {
        my $howlong = &Time2String(time() - $seen[1]);
        $reply = "$seen[0] <$seen[3]> was last seen on IRC ".
@@ -416,11 +417,11 @@ sub cookie {
     ### WILL CHEW TONS OF MEM.
     ### TODO: convert this to a Forker function!
     if ($arg) {
-       my @list = &searchTable("factoids", "factoid_key", "factoid_value", $arg);
+       my @list = &searchTable('factoids', 'factoid_key', 'factoid_value', $arg);
        $key    = &getRandom(@list);
-       $value  = &getFactInfo($key, "factoid_value");
+       $value  = &getFactInfo($key, 'factoid_value');
     } else {
-       ($key,$value) = &randKey("factoids","factoid_key,factoid_value");
+       ($key,$value) = &randKey('factoids','factoid_key,factoid_value');
     }
 
     for ($cookiemsg) {
@@ -447,8 +448,8 @@ sub convert {
     ($from,$to) = ($2,$1) if ($arg =~ /^(.*?) from (.*)$/i);
 
     if (!$to or !$from) {
-       &msg($who, "Invalid format!");
-       &help("convert");
+       &msg($who, 'Invalid format!');
+       &help('convert');
        return;
     }
 
@@ -462,6 +463,7 @@ sub lart {
     my $extra  = 0;
     my $chan   = $talkchannel;
     my ($for);
+    my $mynick = $conn->nick();
 
     if ($msgType eq 'private') {
        if ($target =~ /^($mask{chan})\s+(.*)$/) {
@@ -469,8 +471,8 @@ sub lart {
            $target     = $2;
            $extra      = 1;
        } else {
-           &msg($who, "error: invalid format or missing arguments.");
-           &help("lart");
+           &msg($who, 'error: invalid format or missing arguments.');
+           &help('lart');
            return;
        }
     }
@@ -479,9 +481,9 @@ sub lart {
        $for    = $2;
     }
 
-    my $line = &getRandomLineFromFile($bot_data_dir. "/blootbot.lart");
+    my $line = &getRandomLineFromFile($bot_data_dir. '/infobot.lart');
     if (defined $line) {
-       if ($target =~ /^(me|you|itself|\Q$ident\E)$/i) {
+       if ($target =~ /^(me|you|itself|\Q$mynick\E)$/i) {
            $line =~ s/WHO/$who/g;
        } else {
            $line =~ s/WHO/$target/g;
@@ -491,12 +493,12 @@ sub lart {
 
        &action($chan, $line);
     } else {
-       &status("lart: error reading file?");
+       &status('lart: error reading file?');
     }
 }
 
 sub DebianNew {
-    my $idx   = "debian/Packages-sid.idx";
+    my $idx   = 'debian/Packages-sid.idx';
     my $error = 0;
     my %pkg;
     my @new;
@@ -505,7 +507,7 @@ sub DebianNew {
     $error++ unless ( -e "$idx-old");
 
     if ($error) {
-       $error = "no sid/sid-old index file found.";
+       $error = 'no sid/sid-old index file found.';
        &ERROR("Debian: $error");
        &msg($who, $error);
        return;
@@ -532,14 +534,14 @@ sub DebianNew {
     }
     close IDX1;
 
-    &::performStrictReply( &::formListReply(0, "New debian packages:", @new) );
+    &::performStrictReply( &::formListReply(0, 'New debian packages:', @new) );
 }
 
 sub do_verstats {
     my ($chan) = @_;
 
     if (!defined $chan) {
-       &help("verstats");
+       &help('verstats');
        return;
     }
 
@@ -549,12 +551,12 @@ sub do_verstats {
     }
 
     if (scalar @vernick > scalar(keys %{ $channels{lc $chan}{''} })/4) {
-       &msg($who, "verstats already in progress for someone else.");
+       &msg($who, 'verstats already in progress for someone else.');
        return;
     }
 
     &msg($who, "Sending CTCP VERSION to $chan; results in 60s.");
-    $conn->ctcp("VERSION", $chan);
+    $conn->ctcp('VERSION', $chan);
     $cache{verstats}{chan}     = $chan;
     $cache{verstats}{who}      = $who;
     $cache{verstats}{msgType}  = $msgType;
@@ -588,7 +590,7 @@ sub do_verstats {
        my $unknown     = $total - $vtotal;
        my $perc        = sprintf("%.1f", $unknown * 100 / $total);
        $perc           =~ s/.0$//;
-       $sorted{$perc}{"unknown/cloak"} = "$unknown ($perc%)" if ($unknown);
+       $sorted{$perc}{'unknown/cloak'} = "$unknown ($perc%)" if ($unknown);
 
        foreach (keys %ver) {
            my $count   = scalar keys %{ $ver{$_} };
@@ -624,7 +626,7 @@ sub verstats_flush {
        last unless (scalar @vernicktodo);
 
        my $n = shift(@vernicktodo);
-       $conn->ctcp("VERSION", $n);
+       $conn->ctcp('VERSION', $n);
     }
 
     return unless (scalar @vernicktodo);
@@ -637,16 +639,11 @@ sub do_text_counters {
     $itc =~ s/([^\w\s])/\\$1/g;
     my $z = join '|', split ' ', $itc;
 
-    if ($msgType eq "privmsg" and $message =~ / ($mask{chan})$/) {
+    if ($msgType eq 'privmsg' and $message =~ / ($mask{chan})$/) {
        &DEBUG("ircTC: privmsg detected; chan = $1");
        $chan = $1;
     }
 
-    if ($message =~ /^_stats(\s+(\S+))$/i) {
-       &textstats_main($2);
-       return 1;
-    }
-
     my ($type,$arg);
     if ($message =~ /^($z)stats(\s+(\S+))?$/i) {
        $type = $1;
@@ -655,27 +652,21 @@ sub do_text_counters {
        return 0;
     }
 
-    # even more uglier with channel/time arguments.
-    my $c      = $chan;
-#   my $c      = $chan || "PRIVATE";
-    my $where  = "type=".&sqlQuote($type);
-    if (defined $c) {
-       &DEBUG("c => $c");
-       $where  .= " AND channel=".&sqlQuote($c) if (defined $c);
-    } else {
-       &DEBUG("not using chan arg");
-    }
+    my $c      = $chan || 'PRIVATE';
 
-    my $sum = (&sqlRawReturn("SELECT SUM(counter) FROM stats"
-                       ." WHERE ".$where ))[0];
+    # Define various types of stats in one place.
+    # Note: sqlSelectColHash has built in sqlQuote
+    my $where_chan_type  = { channel => $c, type => $type };
+    my $where_chan_type_nick = { channel => $c, type => $type, nick => $arg};
+
+    my $sum = (&sqlSelect('stats', 'SUM(counter)', $where_chan_type))[0];
 
     if (!defined $arg or $arg =~ /^\s*$/) {
-       # this is way ugly.
 
-       # TODO: convert $where to hash
-       my %hash = &sqlSelectColHash("stats", "nick,counter",
-                       { },
-                       $where." ORDER BY counter DESC LIMIT 3", 1
+       # get top 3 stats of $type in $chan
+       my %hash = &sqlSelectColHash('stats', 'nick,counter',
+                       $where_chan_type,
+                       'ORDER BY counter DESC LIMIT 3', 1
        );
        my $i;
        my @top;
@@ -689,9 +680,9 @@ sub do_text_counters {
                push(@top, "\002$_\002 -- $i ($p%)");
            }
        }
-       my $topstr = "";
+       my $topstr = '';
        if (scalar @top) {
-           $topstr = ".  Top ".scalar(@top).": ".join(', ', @top);
+           $topstr = '.  Top '.scalar(@top).': '.join(', ', @top);
        }
 
        if (defined $sum) {
@@ -700,34 +691,32 @@ sub do_text_counters {
            &performStrictReply("zero counter for \037$type\037.");
        }
     } else {
-       # TODO: convert $where to hash and use a sqlSelect
-       my $x = (&sqlRawReturn("SELECT SUM(counter) FROM stats".
-                       " WHERE $where AND nick=".&sqlQuote($arg) ))[0];
+       my $x = (&sqlSelect('stats', 'SUM(counter)', $where_chan_type_nick))[0];
 
-       if (!defined $x) {      # !defined.
+       if (!defined $x) {      # If no stats were found
            &performStrictReply("$arg has not said $type yet.");
            return 1;
        }
 
-       # defined.
-       # TODO: convert $where to hash
-       my @array = &sqlSelect("stats", "nick", undef,
-                       $where." ORDER BY counter", 1
+       # Get list of all nicks for channel $c and $type
+       my @array = &sqlSelectColArray('stats', 'nick',
+               $where_chan_type,
+               'ORDER BY counter DESC'
        );
-       my $good = 0;
-       my $i = 0;
-       for ($i=0; $i<scalar @array; $i++) {
-           next unless ($array[0] =~ /^\Q$who\E$/);
-           $good++;
+
+       my $total = scalar(@array);
+       my $rank;
+       # Find position of nick $arg in the list
+       for (my $i=0; $i < $total; $i++) {
+           next unless ($array[$i] =~ /^\Q$arg\E$/);
+           $rank = $i + 1;
            last;
        }
-       $i++;
 
-       my $total = scalar(@array);
-       my $xtra = "";
-       if ($total and $good) {
-           my $pct = sprintf("%.01f", 100*(1+$total-$i)/$total);
-           $xtra = ", ranked $i\002/\002$total (percentile: \002$pct\002 %)";
+       my $xtra;
+       if ($total and $rank) {
+           my $pct = sprintf("%.01f", 100*($rank)/$total);
+           $xtra = ", ranked $rank\002/\002$total (percentile: \002$pct\002 %)";
        }
 
        my $pct1 = sprintf("%.01f", 100*$x/$sum);
@@ -737,101 +726,6 @@ sub do_text_counters {
     return 1;
 }
 
-sub textstats_main {
-    my($arg) = @_;
-
-    # even more uglier with channel/time arguments.
-    my $c      = $chan;
-#    my $c     = $chan || "PRIVATE";
-    &DEBUG("not using chan arg") if (!defined $c);
-
-    # example of converting from RawReturn to sqlSelect.
-    my $where_href = (defined $c) ? { channel => $c } : "";
-    my $sum = &sqlSelect("stats", "SUM(counter)", $where_href);
-
-    if (!defined $arg or $arg =~ /^\s*$/) {
-       # this is way ugly.
-       &DEBUG("_stats: !arg");
-
-       my %hash = &sqlSelectColHash("stats", "nick,counter",
-               $where_href,
-               " ORDER BY counter DESC LIMIT 3", 1
-       );
-       my $i;
-       my @top;
-
-       # unfortunately we have to sort it again!
-       my $tp = 0;
-       foreach $i (sort { $b <=> $a } keys %hash) {
-           foreach (keys %{ $hash{$i} }) {
-               my $p   = sprintf("%.01f", 100*$i/$sum);
-               $tp     += $p;
-               push(@top, "\002$_\002 -- $i ($p%)");
-           }
-       }
-
-       my $topstr = "";
-       if (scalar @top) {
-           $topstr = ".  Top ".scalar(@top).": ".join(', ', @top);
-       }
-
-       if (defined $sum) {
-           &performStrictReply("total count of \037$type\037 on \002$c\002: $sum$topstr");
-       } else {
-           &performStrictReply("zero counter for \037$type\037.");
-       }
-
-       return;
-    }
-
-    # TODO: add nick to where_href
-    my %hash = &sqlSelectColHash("stats", "type,counter",
-               $where_href, " AND nick=".&sqlQuote($arg)
-    );
-
-    # this is totally messed up... needs to be fixed... and cleaned up.
-    my $total;
-    my $good;
-    my $ii;
-    my $x;
-
-    foreach (keys %hash) {
-       &DEBUG("_stats: hash{$_} => $hash{$_}");
-       # ranking.
-       # TODO: convert $where to hash
-       my $where = '';
-       my @array = &sqlSelect("stats", "nick", undef, $where." ORDER BY counter", 1);
-       $good = 0;
-       $ii = 0;
-       for(my $i=0; $i<scalar @array; $i++) {
-           next unless ($array[0] =~ /^\Q$who\E$/);
-           $good++;
-           last;
-       }
-       $ii++;
-
-       $total = scalar(@array);
-       &DEBUG("   i => $i, good => $good, total => $total");
-       $x .= " ".$total."blah blah";
-    }
-
-#    return;
-
-    if (!defined $x) { # !defined.
-       &performStrictReply("$arg has not said $type yet.");
-       return;
-    }
-
-    my $xtra = "";
-    if ($total and $good) {
-       my $pct = sprintf("%.01f", 100*(1+$total-$ii)/$total);
-       $xtra = ", ranked $ii\002/\002$total (percentile: \002$pct\002 %)";
-    }
-
-    my $pct1 = sprintf("%.01f", 100*$x/$sum);
-    &performStrictReply("\002$arg\002 has said \037$type\037 \002$x\002 times (\002$pct1\002 %)$xtra");
-}
-
 sub nullski {
     my ($arg) = @_;
     return unless (defined $arg);
@@ -845,63 +739,70 @@ sub nullski {
 ###
 &addCmdHook('(babel(fish)?|x|xlate|translate)', ('CODEREF' => 'babelfish::babelfish', 'Identifier' => 'babelfish', 'Cmdstats' => 'babelfish', 'Forker' => 1, 'Help' => 'babelfish', 'Module' => 'babelfish') );
 &addCmdHook('(botmail|message)', ('CODEREF' => 'botmail::parse', 'Identifier' => 'botmail', 'Cmdstats' => 'botmail') );
-&addCmdHook('bzflist17', ('CODEREF' => 'BZFlag::list17', 'Identifier' => 'BZFlag', 'Cmdstats' => 'BZFlag', 'Forker' => 1) );
-&addCmdHook('bzflist', ('CODEREF' => 'BZFlag::list', 'Identifier' => 'BZFlag', 'Cmdstats' => 'BZFlag', 'Forker' => 1) );
-&addCmdHook('bzfquery', ('CODEREF' => 'BZFlag::query', 'Identifier' => 'BZFlag', 'Cmdstats' => 'BZFlag', 'Forker' => 1) );
+&addCmdHook('bzflist17', ('CODEREF' => 'BZFlag::list17', 'Identifier' => 'BZFlag', 'Cmdstats' => 'BZFlag', 'Forker' => 1, 'Module' => 'BZFlag') );
+&addCmdHook('bzflist', ('CODEREF' => 'BZFlag::list', 'Identifier' => 'BZFlag', 'Cmdstats' => 'BZFlag', 'Forker' => 1, 'Module' => 'BZFlag') );
+&addCmdHook('bzfquery', ('CODEREF' => 'BZFlag::query', 'Identifier' => 'BZFlag', 'Cmdstats' => 'BZFlag', 'Forker' => 1, 'Module' => 'BZFlag') );
 &addCmdHook('chan(stats|info)', ('CODEREF' => 'chaninfo', ) );
 &addCmdHook('cmd(stats|info)', ('CODEREF' => 'cmdstats', ) );
 &addCmdHook('convert', ('CODEREF' => 'convert', 'Forker' => 1, 'Identifier' => 'Units', 'Help' => 'convert') );
 &addCmdHook('(cookie|random)', ('CODEREF' => 'cookie', 'Forker' => 1, 'Identifier' => 'Factoids') );
 &addCmdHook('countdown', ('CODEREF' => 'countdown', 'Module' => 'countdown', 'Identifier' => 'countdown', 'Cmdstats' => 'countdown') );
 &addCmdHook('countrystats', ('CODEREF' => 'countryStats') );
-&addCmdHook('dauthor', ('CODEREF' => 'Debian::searchAuthor', 'Forker' => 1, 'Identifier' => 'Debian', 'Cmdstats' => 'Debian Author Search', 'Help' => "dauthor" ) );
+&addCmdHook('dauthor', ('CODEREF' => 'Debian::searchAuthor', 'Forker' => 1, 'Identifier' => 'Debian', 'Cmdstats' => 'Debian Author Search', 'Help' => 'dauthor' ) );
 &addCmdHook('d?bugs', ('CODEREF' => 'DebianExtra::Parse', 'Forker' => 1, 'Identifier' => 'DebianExtra', 'Cmdstats' => 'Debian Bugs') );
-&addCmdHook('d?contents', ('CODEREF' => 'Debian::searchContents', 'Forker' => 1, 'Identifier' => 'Debian', 'Cmdstats' => 'Debian Contents Search', 'Help' => "contents" ) );
-&addCmdHook('d?find', ('CODEREF' => 'Debian::DebianFind', 'Forker' => 1, 'Identifier' => 'Debian', 'Cmdstats' => 'Debian Search', 'Help' => "find" ) );
+&addCmdHook('d?contents', ('CODEREF' => 'Debian::searchContents', 'Forker' => 1, 'Identifier' => 'Debian', 'Cmdstats' => 'Debian Contents Search', 'Help' => 'contents' ) );
+&addCmdHook('d?find', ('CODEREF' => 'Debian::DebianFind', 'Forker' => 1, 'Identifier' => 'Debian', 'Cmdstats' => 'Debian Search', 'Help' => 'find' ) );
+&addCmdHook('dice', ('CODEREF' => 'dice::dice', 'Identifier' => 'dice', 'Cmdstats' => 'dice', 'Forker' => 1, 'Module' => 'dice') );
 &addCmdHook('Dict', ('CODEREF' => 'Dict::Dict', 'Identifier' => 'Dict', 'Help' => 'dict', 'Forker' => 1, 'Cmdstats' => 'Dict') );
 &addCmdHook('dincoming', ('CODEREF' => 'Debian::generateIncoming', 'Forker' => 1, 'Identifier' => 'Debian' ) );
 &addCmdHook('dnew', ('CODEREF' => 'DebianNew', 'Identifier' => 'Debian' ) );
 &addCmdHook('dns|d?nslookup', ('CODEREF' => 'dns::query', 'Identifier' => 'dns', 'Cmdstats' => 'dns', 'Forker' => 1, 'Help' => 'dns') );
-&addCmdHook('(d|search)desc', ('CODEREF' => 'Debian::searchDescFE', 'Forker' => 1, 'Identifier' => 'Debian', 'Cmdstats' => 'Debian Desc Search', 'Help' => "ddesc" ) );
+&addCmdHook('(d|search)desc', ('CODEREF' => 'Debian::searchDescFE', 'Forker' => 1, 'Identifier' => 'Debian', 'Cmdstats' => 'Debian Desc Search', 'Help' => 'ddesc' ) );
 &addCmdHook('dstats', ('CODEREF' => 'Debian::infoStats', 'Forker' => 1, 'Identifier' => 'Debian', 'Cmdstats' => 'Debian Statistics' ) );
 &addCmdHook('(ex)?change', ('CODEREF' => 'Exchange::query', 'Identifier' => 'Exchange', 'Cmdstats' => 'Exchange', 'Forker' => 1) );
 &addCmdHook('factinfo', ('CODEREF' => 'factinfo', 'Cmdstats' => 'Factoid Info', Module => 'Factoids', ) );
-&addCmdHook('factstats?', ('CODEREF' => 'factstats', 'Cmdstats' => 'Factoid Stats', Help => "factstats", Forker => 1, 'Identifier' => 'Factoids', ) );
+&addCmdHook('factstats?', ('CODEREF' => 'factstats', 'Cmdstats' => 'Factoid Stats', Help => 'factstats', Forker => 1, 'Identifier' => 'Factoids', ) );
 &addCmdHook('help', ('CODEREF' => 'help', 'Cmdstats' => 'Help', ) );
 &addCmdHook('HTTPDtype', ('CODEREF' => 'HTTPDtype::HTTPDtype', 'Identifier' => 'HTTPDtype', 'Cmdstats' => 'HTTPDtype', 'Forker' => 1) );
 &addCmdHook('[ia]?spell', ('CODEREF' => 'spell::query', 'Identifier' => 'spell', 'Cmdstats' => 'spell', 'Forker' => 1, 'Help' => 'spell') );
-&addCmdHook('insult', ('CODEREF' => 'Insult::Insult', 'Forker' => 1, 'Identifier' => 'insult', 'Help' => "insult" ) );
+&addCmdHook('insult', ('CODEREF' => 'Insult::Insult', 'Forker' => 1, 'Identifier' => 'insult', 'Help' => 'insult' ) );
 &addCmdHook('karma', ('CODEREF' => 'karma', ) );
 &addCmdHook('kernel', ('CODEREF' => 'Kernel::Kernel', 'Forker' => 1, 'Identifier' => 'Kernel', 'Cmdstats' => 'Kernel', 'NoArgs' => 1) );
 &addCmdHook('lart', ('CODEREF' => 'lart', 'Identifier' => 'lart', 'Help' => 'lart') );
+&addCmdHook('lc', ('CODEREF' => 'case::lower', 'Identifier' => 'case', 'Cmdstats' => 'case', 'Forker' => 1, 'Module' => 'case') );
 &addCmdHook('listauth', ('CODEREF' => 'CmdListAuth', 'Identifier' => 'Search', Module => 'Factoids', 'Help' => 'listauth') );
 &addCmdHook('md5(sum)?', ('CODEREF' => 'md5::md5', 'Identifier' => 'md5', 'Cmdstats' => 'md5', 'Forker' => 1, 'Module' => 'md5') );
 &addCmdHook('metar', ('CODEREF' => 'Weather::Metar', 'Identifier' => 'Weather', 'Help' => 'weather', 'Cmdstats' => 'Weather', 'Forker' => 1) );
-&addCmdHook('News', ('CODEREF' => 'News::Parse', Module => 'News', 'Cmdstats' => 'News' ) );
+&addCmdHook('News', ('CODEREF' => 'News::Parse', Module => 'News', 'Cmdstats' => 'News', 'Identifier' => 'News' ) );
 &addCmdHook('(?:nick|lame)ometer(?: for)?', ('CODEREF' => 'nickometer::query', 'Identifier' => 'nickometer', 'Cmdstats' => 'nickometer', 'Forker' => 1) );
 &addCmdHook('nullski', ('CODEREF' => 'nullski', ) );
 &addCmdHook('page', ('CODEREF' => 'pager::page', 'Identifier' => 'pager', 'Cmdstats' => 'pager', 'Forker' => 1, 'Help' => 'page') );
 &addCmdHook('piglatin', ('CODEREF' => 'piglatin::piglatin', 'Identifier' => 'piglatin', 'Cmdstats' => 'piglatin', 'Forker' => 1) );
 &addCmdHook('Plug', ('CODEREF' => 'Plug::Plug', 'Identifier' => 'Plug', 'Forker' => 1, 'Cmdstats' => 'Plug') );
 &addCmdHook('quote', ('CODEREF' => 'Quote::Quote', 'Forker' => 1, 'Identifier' => 'Quote', 'Help' => 'quote', 'Cmdstats' => 'Quote') );
+&addCmdHook('reverse', ('CODEREF' => 'reverse::reverse', 'Identifier' => 'reverse', 'Cmdstats' => 'reverse', 'Forker' => 1, 'Module' => 'reverse') );
 &addCmdHook('RootWarn', ('CODEREF' => 'CmdrootWarn', 'Identifier' => 'RootWarn', 'Module' => 'RootWarn') );
+&addCmdHook('OnJoin', ('CODEREF' => 'Cmdonjoin', 'Identifier' => 'OnJoin', 'Module' => 'OnJoin') );
 &addCmdHook('Rss', ('CODEREF' => 'Rss::Rss', 'Identifier' => 'Rss', 'Cmdstats' => 'Rss', 'Forker' => 1, 'Help' => 'rss') );
 &addCmdHook('sched(stats|info)', ('CODEREF' => 'scheduleList', ) );
 &addCmdHook('scramble', ('CODEREF' => 'scramble::scramble', 'Identifier' => 'scramble', 'Cmdstats' => 'scramble', 'Forker' => 1, 'Module' => 'scramble') );
 &addCmdHook('seen', ('CODEREF' => 'seen', 'Identifier' => 'seen') );
 &addCmdHook('slashdot', ('CODEREF' => 'Slashdot::Slashdot', 'Identifier' => 'slashdot', 'Forker' => 1, 'Cmdstats' => 'slashdot') );
 &addCmdHook('tell|explain', ('CODEREF' => 'tell', Help => 'tell', Identifier => 'allowTelling', Cmdstats => 'Tell') );
+&addCmdHook('uc', ('CODEREF' => 'case::upper', 'Identifier' => 'case', 'Cmdstats' => 'case', 'Forker' => 1, 'Module' => 'case') );
 &addCmdHook('Uptime', ('CODEREF' => 'uptime', 'Identifier' => 'Uptime', 'Cmdstats' => 'Uptime') );
 &addCmdHook('u(ser)?info', ('CODEREF' => 'userinfo', 'Identifier' => 'UserInfo', 'Help' => 'userinfo', 'Module' => 'UserInfo') );
 &addCmdHook('verstats', ('CODEREF' => 'do_verstats' ) );
-&addCmdHook('Weather', ('CODEREF' => 'Weather::Weather', 'Identifier' => 'Weather', 'Help' => 'weather', 'Cmdstats' => 'Weather', 'Forker' => 1) );
-&addCmdHook('wiki(pedia)?', ('CODEREF' => 'wikipedia::wikipedia', 'Identifier' => 'wikipedia', 'Cmdstats' => 'wikipedia', 'Forker' => 1, 'Help' => 'wikipedia') );
-&addCmdHook('wtf', ('CODEREF' => 'wtf::query', 'Identifier' => 'wtf', 'Cmdstats' => 'wtf', 'Forker' => 1, 'Help' => 'wtf') );
-&addCmdHook('zfi', ('CODEREF' => 'zfi::query', 'Identifier' => 'zfi', 'Cmdstats' => 'zfi', 'Forker' => 1) );
-&addCmdHook('(zippy|yow)', ('CODEREF' => 'zippy::get', 'Identifier' => 'Zippy', 'Cmdstats' => 'Zippy', 'Forker' => 1) );
-&addCmdHook('zsi', ('CODEREF' => 'zsi::query', 'Identifier' => 'zsi', 'Cmdstats' => 'zsi', 'Forker' => 1) );
+&addCmdHook('Weather', ('CODEREF' => 'Weather::Weather', 'Identifier' => 'Weather', 'Help' => 'weather', 'Cmdstats' => 'Weather', 'Forker' => 1, 'Module' => 'Weather') );
+&addCmdHook('wiki(pedia)?', ('CODEREF' => 'wikipedia::wikipedia', 'Identifier' => 'wikipedia', 'Cmdstats' => 'wikipedia', 'Forker' => 1, 'Help' => 'wikipedia', 'Module' => 'wikipedia') );
+&addCmdHook('wtf', ('CODEREF' => 'wtf::query', 'Identifier' => 'wtf', 'Cmdstats' => 'wtf', 'Forker' => 1, 'Help' => 'wtf', 'Module' => 'wtf') );
+&addCmdHook('zfi', ('CODEREF' => 'zfi::query', 'Identifier' => 'zfi', 'Cmdstats' => 'zfi', 'Forker' => 1, 'Module' => 'zfi') );
+&addCmdHook('(zippy|yow)', ('CODEREF' => 'zippy::get', 'Identifier' => 'Zippy', 'Cmdstats' => 'Zippy', 'Forker' => 1, 'Module' => 'Zippy') );
+&addCmdHook('zsi', ('CODEREF' => 'zsi::query', 'Identifier' => 'zsi', 'Cmdstats' => 'zsi', 'Forker' => 1, 'Module' => 'zsi') );
 ###
 ### END OF ADDING HOOKS.
 ###
-&status("CMD: loaded ".scalar(keys %cmdhooks)." command hooks.");
+
+&status('loaded '.scalar(keys %cmdhooks).' command hooks.');
+
 1;