[SimBot-commits] CVS: simbot/plugins recap.pl,1.22,1.23 sqlite-logger.pl,1.69,1.70

[Kevin S. <ks...@us...>]

Update of /cvsroot/simbot/simbot/plugins
In directory sc8-pr-cvs1.sourceforge.net:/tmp/cvs-serv28911/plugins

Modified Files:
	recap.pl sqlite-logger.pl 
Log Message:
Rearrange some of the events a little bit to fix some weird problems:
1) Private "out" events were not reporting the destination, rendering them generally useless.
2) The callbacks for action events were being selected from the list of plugins that wanted message events, but called with the correct event, meaning plugins wanting actions but not messages would never get called. This wasn't causing any visible problems because plugins that ask for actions usually want messages also.
3) The arguments to the "in" and "out" events for actions were not the same, causing plugins assuming them to be to lose the first word or duplicate the nickname.  The arguments now match those of the related normal message events.

Index: recap.pl
===================================================================
RCS file: /cvsroot/simbot/simbot/plugins/recap.pl,v
retrieving revision 1.22
retrieving revision 1.23
diff -u -d -p -r1.22 -r1.23
--- recap.pl	10 Aug 2005 11:50:59 -0000	1.22
+++ recap.pl	26 Aug 2005 07:45:38 -0000	1.23
@@ -76,7 +76,7 @@ sub record_recap {
     if ($doing eq 'SAY') {
 		$line .= "<$nick> $content";
     } elsif ($doing eq 'ACTION') {
-		$line .= "* $content";
+		$line .= "* $nick $content";
     } elsif ($doing eq 'KICKED') {
 		$line .= "$target kicked $nick from $channel" . ($content ? " ($content)" : "");
 		$departs{$nick} = 0;

Index: sqlite-logger.pl
===================================================================
RCS file: /cvsroot/simbot/simbot/plugins/sqlite-logger.pl,v
retrieving revision 1.69
retrieving revision 1.70
diff -u -d -p -r1.69 -r1.70
--- sqlite-logger.pl	22 Aug 2005 17:26:53 -0000	1.69
+++ sqlite-logger.pl	26 Aug 2005 07:45:38 -0000	1.70
@@ -225,8 +225,6 @@ sub set_seen {
         # $target will be the arguments for the mode change options
         # let's add them to the content
         $content .= " $target";
-    } elsif ( $doing eq 'ACTION' ) {
-        ( undef, $content ) = split( / /, $content, 2 );
     }
 
     $insert_query->execute( time, $channel_id, $source_nick_id, $target_nick_id,