The attached patch does several things:

  • Adds a "be back logger", which, in addition to a last seen, which parses things like
    • be back in two hours
    • see you later, bb in thirteen hundred hours
    • I'll be back in an hour
  • Based apon a be back message, if someone tries to say something to a user who has set an away status, it will reply with something like "dmitrig01 is exected back in two hours and one minute"
  • If someone comes back before the time they stated, the bot says "Welcome back, $nick"

I've tested the individual parts, but not the thing as a whole, so it should work.
@TODO: make all added function be in the bot_seen namespace. that's the only thing IMO. This patch is the first step to the notes patch, where you can write notes to other people and they recieve them when they please (I am not saying I will do this one :D )

CommentFileSizeAuthor
#4 bot_seen_3.patch8.88 KBShiny
#3 bot_seen_2.patch8.84 KBShiny
#2 bot_seen_bb_1.patch8.39 KBdmitrig01
#1 bot_seen_bb_0.patch8.37 KBdmitrig01
bot_seen_bb.patch7.96 KBdmitrig01

Comments

dmitrig01’s picture

dmitrig01 commented
StatusFileSize
new bot_seen_bb_0.patch8.37 KB

Fixes the mentioned namespace thing.
I forgot to mention great, great, great thanks to kkaefer for wtiting the monster regular expession

dmitrig01’s picture

dmitrig01 commented
StatusFileSize
new bot_seen_bb_1.patch8.39 KB

small bugfix

Shiny’s picture

Shiny commented
StatusFileSize
new bot_seen_2.patch8.84 KB

added bot_seen_update_1
still needs the mysql version added.

Shiny’s picture

Shiny commented
StatusFileSize
new bot_seen_3.patch8.88 KB

made E_ALL compatible :-)

czarphanguye’s picture

czarphanguye commented

I don't get it.. :-?

Seen works pre-patch, then once the patch is applied (without issues) the moment 'nick' says anything db_query("DELETE FROM {bot_seen} WHERE LOWER(nick) = '%s'", drupal_strtolower($data->nick)); goes to work and removed the nick from bot_seen (if added using original bot_seen.module) yet the INSERT never happens.

The last_in tables has been added with both default 0 and -1 (dmitrig01 Vs. Shiny patch), yet no success with either patch.

I missing something?

morbus iff’s picture

morbus iff
they/them
commented
Status: Needs review » Closed (won't fix)

After years of thought (sigh, heh), I think I'm going to wontfix this one. With the existing Tell functionality (mentioned herein as the "notes" functionality), someone can leave a message for folks regardless of when they'll be back. I also don't really see "be back in" phrases commonly used nowadays (one could say "CHICKEN AND EGG MORBUS, JEEZ"), and the infrequency of use versus the amount of code required... well... it's a lot for little massive benefit.