chiark / gitweb /
Bugfixes.
[ircbot] / bot.tcl
diff --git a/bot.tcl b/bot.tcl
index 0fd2b75deb6513795f07d4546663c8746c9fda3e..353839714232e556f99b1e0312c46753a809ec83 100755 (executable)
--- a/bot.tcl
+++ b/bot.tcl
@@ -151,7 +151,7 @@ proc tell_event {nl event} {
     # later.  We also support the psuedo-event `none'.  The del msg
     # and new msg events are handled by the command procedures, not here.
     global calling_nick
-    if {[info exists calling_nick] { set save $calling_nick }
+    if {[info exists calling_nick]} { set save $calling_nick }
     switch -exact $event {
        none { }
        talk {
@@ -161,7 +161,7 @@ proc tell_event {nl event} {
        act {
            after 750 [list tell_event_core $nl $event]
        }
-       ident {
+       ident - msgsarrive {
            tell_event_core $nl $event
        }
        default {
@@ -171,31 +171,35 @@ proc tell_event {nl event} {
     if {[info exists save]} { set calling_nick $save }
 }
 
-proc tell_event_core {nl event} {
-    # event is `talk', `act' or `ident'
-    # When user talks we actually get talk now and act later
-FIXME - implement all cmds
-FIXME - implement tells_delete_msgs catch { unset stt }   ?
-FIXME - document the extra param to recordlastseen
-    global calling_nick
-    set calling_nick $nl
-    set iml [msgdb_get $nl inbound]
-    if {![llength $iml]} return
+proc tell_getcstate {} {
+    # uses nl from caller's context
+    # imports telling (as the nick_telling) and u
+    # sets stt, telling_when
+    uplevel 1 {
+       upvar #0 nick_telling($nl) telling
+       upvar #0 nick_unique($nl) u
 
-    upvar #0 nick_telling($nl) telling
-    upvar #0 nick_unique($nl) u
+       if {[info exists telling]} {
+           manyset $telling u_last stt telling_when
+           if {![info exists u] || "$u_last" != "$u"} {
+               unset telling; unset stt; unset telling_when
+           }
+       }
 
-    if {[info exists telling]} {
-       manyset $telling u_last stt telling_when
-       if {![info exists u] || "$u_last" != "$u"} {
-           unset telling; unset stt; unset telling_when
+       if {![info exists stt]} {
+           set stt norecord
+           set telling_when $now
        }
     }
+}
 
-    if {![info exists stt]} {
-       set stt norecord
-       set telling_when $now
-    }
+proc tell_event_core {nl event} {
+    # event is `talk', `act', `ident' or `msgsarrive'
+    # When user talks we actually get talk now and act later
+    global calling_nick
+    set calling_nick $nl
+    set iml [msgsdb_get $nl inbound]
+    if {![llength $iml]} return
 
     set ago [expr {$now - $telling_when}]
 
@@ -215,6 +219,7 @@ FIXME - document the extra param to recordlastseen
     #      t   talk
     #      a   act
     #      i   ident
+    #      m   msgsarrive
     #   security level and timing
     #      ii  Insecure
     #      ss  Secure and soon (before interval)
@@ -266,6 +271,15 @@ FIXME - document the extra param to recordlastseen
            tell_delete_msgs {} $nl
            return
        }
+       pm????? {
+           # oops, messages passed are now out of date
+           catch { unset telling }
+           return
+       }
+       ?m????? {
+           # ignore new msgs if we haven't passed yet
+           return
+       }
        nt????? - mt????? -
        pt???uu - pt???rs - pt???rl - pt???p? {
            # ignore (any other `talk's) - act handles these
@@ -320,6 +334,10 @@ FIXME - document the extra param to recordlastseen
 }
 
 proc recordlastseen_n {n how here} {
+    # here is:
+    #   0 - nick was seen leaving (or changing to another nicks or some such)
+    #   1 - nick was seen doing something else
+    #   2 - nick was seen talking on channel
     global lastseen lookedfor
     set nl [irctolower $n]
     set now [clock seconds]
@@ -1189,18 +1207,72 @@ proc tell_effective_sec {n} {
     return $l
 }
 
-proc tell_peernicks {
--- FIXME HERE
+proc tell_peernicks {text} {
+    set text [irctolower [string trim $text]]
+    set senders [split $text " "]
+    foreach sender $senders {
+       if {[catch { check_nick $sender } emsg]} {
+           error "invalid sender nick `$sender': $emsg" $errorInfo $errorCode
+       }
+    }
+    return $senders
+}
+
+proc msgsdb_set_maydelete {n key l otherkey} {
+    msgsdb_set $n $key $l
+    if {[llength $l]} return
+    if {[llength [msgsdb_get $n $otherkey]]} return
+    msgsdb_delete $n
+}
+
+proc tell_delete_msgs {lsenders lrecip} {
+    set ninbound {}
+    set ndel 0
+    foreach {s t m} [msgsdb_get $recip inbound] {
+       if {[llength $senders]} {
+           if {[lsearch -exact $senders [irctolower $s]] == -1} {
+               lappend ninbound $s $t $m
+               continue
+           }
+       }
+       set rsenders($s) 1
+       incr ndel
+    }
+    msgsdb_set_maydelete $recip inbound $ninbound outbound
+    if {![llength $ninbound]} {
+       upvar #0 nick_telling($lrecip) telling
+       catch { unset telling }
+    }
+    foreach s [array names rsenders] {
+       set noutbound {}
+       foreach {r t c} [msgsdb_get $s outbound] {
+           if {"[irctolower $r]" == "$lrecip"} continue
+           lappend noutbound $r $t $c
+       }
+       msgsdb_set_maydelete $s outbound $noutbound inbound
+    }
+    return $ndel
+}
 
 def_ucmd untell {
     prefix_nick
     check_notonchan
     nick_securitycheck 0
-    set peernicks
+    set recipients [tell_peernicks $text]
+    if {![llength $recipients]} {
+       usererror "You must say which recipients' messages from you to forget."
+    }
+    set ndel 0
+    foreach recip $recipients {
+       incr ndel [tell_delete_msgs [irctolower $n] $recip]
+    }
+    ucmdr "Removed $ndel as yet undelivered message(s)." {}
+}
 
 def_ucmd delmsg {
     global errorInfo errorCode
     prefix_nick
+    set nl [irctolower $n]
     check_notonchan
     manyset [tell_effective_sec $n] sec secwhen
     switch -exact $sec {
@@ -1217,19 +1289,26 @@ def_ucmd delmsg {
            error "delmsg sec $sec"
        }
     }
-    set text [irctolower [string trim $text]]
-    set senders [split $text " "]
-    foreach sender $senders {
-       if {[catch { check_nick $sender } emsg]} {
-           error "invalid sender nick `$sender': $emsg" $errorInfo $errorCode
+    tell_getcstate
+    if {"$stt" != "passed"} {
+       usererror \
+ "There are message(s) you've not yet seen; I'll deliver them to you now.\
+  If you actually want to delete them, just tell me `delmsg' again."
+    }
+    set senders [tell_peernicks $text]
+    set ndel [tell_delete_msgs [irctolower $senders] [irctolower $n]]
+    if {!$ndel} {
+       if {[llength $senders]} {
+           ucmdr "No relevant incoming messages to delete." {}
+       } else {
+           ucmdr "No incoming messages to delete." {}
        }
     }
-    tell_delete_msgs [irctolower $senders] [irctolower $n]
     switch -exact [llength $senders] {
-       0 { ucmdr {} {} "deletes your message(s)." }
-       1 { ucmdr {} {} "deletes your message(s) from $senders." }
+       0 { ucmdr {} {} "deletes your $ndel message(s)." }
+       1 { ucmdr {} {} "deletes your $ndel message(s) from $senders." }
        default {
-           ucmdr {} {} "deletes your message(s) from\
+           ucmdr {} {} "deletes your $ndel message(s) from\
  [lreplace $senders end end] and [lindex $senders end]."
        }
     }
@@ -1288,6 +1367,7 @@ def_ucmd tell {
                }
            }
            append msg .
+           tell_event $ltarget msgsarrive
            ucmdr $msg {}
        }
        mailto {
@@ -1781,7 +1861,7 @@ proc lnick_marktime_start {luser why ms mentiontold} {
     set mt [nickdb_get $luser marktime]
     if {"$mt" == "off"} {
        lnick_marktime_cancel $luser
-       after $ms [list lnick_checktold $luser]
+       if {$mentiontold} { after $ms [list lnick_checktold $luser] }
     } else {
        lnick_marktime_doafter $luser $why $ms $mentiontold
     }