Cinch-Tell 0.0.4 → 0.0.5

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: a56bede95d10981553c8b63a7eaadbeb41be3dd7
-  data.tar.gz: 6c4754c10182a6b27ec64263727abeb428899d1c
+  metadata.gz: 35e8cea7fd8e9036ac37189981efd9e655b7a4e4
+  data.tar.gz: d50246c1b4b1a593395c741f10d8a5ce1b77ade1
 SHA512:
-  metadata.gz: eef02982dfd295b89c84810d26f283c8100a8148b432429fbdbe9f143fe2d8b5959067747536a98a8e621e725cd6105bd23bedd2b335efc56ea7dc672124cb1f
-  data.tar.gz: 4993775663ba8a4eb5cb163520b8d48dd6f5c3cb6412dce03aab68b506d2e81c8268541f14c9bcaec0d5fe1c44b5c04aa32265217e5d6762fa99076d13a20b69
+  metadata.gz: e51c31376a35d3627ba0c1d18629a1f2ca7b57595a061ec47d8777ac398309db3cecb66e54a9c3fdfe3e472cfc724e6de0ad841abaabcef578ae4a440e1fa14b
+  data.tar.gz: 5f74ac931e1eec12ca5de38874e632cc4b997feb9e02a8bc8f6a13d58861bfda31cc35b4f67b4537af4946c831270dea88e60ad01d8433029250fb4a837a8381

data/LICENSE

@@ -1,6 +1,6 @@
 The MIT License (MIT)
 
-Copyright (c) 2016 Lily Jónsdottír / Lily Seki
+Copyright (c) 2016 Lily Jónsdóttir / Lily Seki
 
 Permission is hereby granted, free of charge, to any person obtaining a copy
 of this software and associated documentation files (the "Software"), to deal

data/lib/cinch/plugins/tell/version.rb

@@ -1,5 +1,5 @@
 module Cinch
   module Tell
-    VERSION = '0.0.4'.freeze
+    VERSION = '0.0.5'.freeze
   end
 end

data/lib/cinch/plugins/tell.rb

@@ -18,10 +18,14 @@ module Cinch
       listen_to :join, method: :check_tells
       listen_to :nick, method: :check_tells
 
+      # I don't actually know if this does anything with cinch
+      # but I keep anything that's not an actual command private
       private
 
       def initialize(*args)
         super
+        # TODO: make database name based on bot nick
+        # or configuration variable
         @db = Sequel.sqlite('riria.db')
         # Create the tables if they don't exist
         @db.create_table? :tells do
@@ -34,11 +38,8 @@ module Cinch
         @db = purge_ancients(@db)
       end
 
-      def header(title)
-        puts '=' * 5 + " #{title} " + '=' * 5
-      end
-
       def read_tells(nick)
+        # read the database and return array of tells for specified nick
         tells = @db[:tells]
         ret = tells.where(to_user: nick).all
         return [] if ret.empty?
@@ -47,7 +48,6 @@ module Cinch
 
       def parse_tells(tell)
         # ready tell for telling
-        to_nick = tell[:to_user]
         from_nick = tell[:from_user]
         msg = tell[:message]
         msg_when = tell[:time]
@@ -65,17 +65,18 @@ module Cinch
         ago_str << 'ago.' unless ago_str.empty?
         ago_str = 'an undetermined amount of time ago.' if ago_str.empty?
         output << ago_str
-        # feels odd without "return output" but I guess that's Ruby
       end
 
       def purge_ancients(db)
         # purge any messages older than x date
+        # TODO: make this a config variable
         purge_date = Time.now.shift(-6, :months)
         db[:tells].where('time < ?', purge_date.to_f).delete
         db
       end
 
       def add_tell(nick, msg, from)
+        # add a tell to the database
         t = Time.now.to_f
         @db[:tells].insert(to_user: nick, from_user: from, message: msg, time: t)
       end
@@ -83,6 +84,11 @@ module Cinch
       public
 
       def check_tells(m)
+        # Listen to join and nick changes, and check if user has any waiting
+        # memos in the database. Might listen to all messages and check if
+        # speaking user has any waiting memos, for people who go away
+        # but don't change their nick, but I'm not sure if I want to query
+        # the database at every IRC message in every channel
         tells = read_tells(m.user.nick.downcase)
         return if tells.empty?
         r = parse_tells(tells.first)
@@ -94,7 +100,7 @@ module Cinch
             User(i[:to_user]).send(parse_tells(i))
           end
         end
-        @db[:tells].where(to_user: m.user.nick).delete
+        @db[:tells].where(to_user: m.user.nick.downcase).delete
       end
 
       match(/tell(?: (.+))?/, method: :irc_tell)

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: Cinch-Tell
 version: !ruby/object:Gem::Version
-  version: 0.0.4
+  version: 0.0.5
 platform: ruby
 authors:
 - Lily Jónsdóttir