Merge lp:~stefanor/ibid/coffee-374028 into lp:~ibid-core/ibid/old-trunk-pack-0.92

[Stefano Rivera]

[Stefano Rivera (stefanor)]

Removed russell's coffee hack

[Stefano Rivera (stefanor)]

Removed russell's coffee hack.

Excuse first merge request - had a bad commit

[Michael Gorven (mgorven)]

 review approve

review: Approve

[lp:~stefanor/ibid/coffee-374028]

647. By Stefano Rivera on 2009-05-31

Proper Ibid-style call_later infrastructure for coffee, et al.

648. By Stefano Rivera on 2009-05-31

Unicodeism

649. By Stefano Rivera on 2009-05-31

Return the IDelayedCall instance, so the delayed call can be cancelled

650. By Stefano Rivera on 2009-05-31

Logging typo + comment

651. By Stefano Rivera on 2009-05-31

A wider set of coffee starting responses

652. By Stefano Rivera on 2009-05-31

More "pot on" responses

Unmerged revisions

=== modified file 'ibid/plugins/misc.py'
--- ibid/plugins/misc.py	2009-05-01 12:17:57 +0000
+++ ibid/plugins/misc.py	2009-05-30 22:34:59 +0000
@@ -1,43 +1,61 @@
 from time import sleep
-
+import logging
+
+from twisted.internet import reactor
+
+import ibid
 from ibid.plugins import Processor, match
 from ibid.config import IntOption
 from ibid.utils import ibid_version
 
 help = {}
+log = logging.getLogger('plugins.misc')
 
 help['coffee'] = u"Times coffee brewing and reserves cups for people"
 class Coffee(Processor):
     u"""coffee (on|please)"""
     feature = 'coffee'
 
-    pot = None
+    pots = {}
 
     time = IntOption('coffee_time', u'Brewing time in seconds', 240)
     cups = IntOption('coffee_cups', u'Maximum number of cups', 4)
     
+    def coffee_announce(self, source, channel):
+        try:
+            ibid.dispatcher.send({
+                'reply': u"Coffee's ready for %s!" % u', '.join(self.pots[(source, channel)]),
+                'source': source,
+                'target': channel,
+            })
+            del self.pots[(source, channel)]
+        except:
+            log.exception('Coffee callback')
+
     @match(r'^coffee\s+on$')
     def coffee_on(self, event):
-        # Hi ... race condition.
-        if self.pot:
+        if (event.source, event.channel) in self.pots:
             event.addresponse(u"There's already a pot on")
             return
         
-        self.pot = [event.sender['nick']]
-        sleep(self.time)
-        event.addresponse(u"Coffee's ready for %s!", u', '.join(self.pot))
-        self.pot = None
+        self.pots[(event.source, event.channel)] = [event.sender['nick']]
+        reactor.callLater(float(self.time), self.coffee_announce, event.source, event.channel)
+
+        event.addresponse({'action': True, 'reply': u'flips the salt-timer'})
     
     @match('^coffee\s+(?:please|pls)$')
     def coffee_accept(self, event):
-        if not self.pot:
+        if (event.source, event.channel) not in self.pots:
             event.addresponse(u"There isn't a pot on")
 
-        elif len(self.pot) >= self.cups:
+        elif len(self.pots[(event.source, event.channel)]) >= self.cups:
             event.addresponse(u"Sorry, there aren't any more cups left")
 
+        elif event.sender['nick'] in self.pots[(event.source, event.channel)]:
+            event.addresponse(u"Now now, we don't want anyone getting caffine overdoses")
+
         else:
-            self.pot.append(event.sender['nick'])
+            self.pots[(event.source, event.channel)].append(event.sender['nick'])
             event.addresponse(True)
 
 help['version'] = u"Show the Ibid version currently running"

=== modified file 'scripts/ibid-plugin'
--- scripts/ibid-plugin	2009-05-05 07:58:29 +0000
+++ scripts/ibid-plugin	2009-05-30 22:07:09 +0000
@@ -52,6 +52,7 @@
 class TestSource(dict):
     type = 'test'
     permissions = []
+    supports = []
 
 ibid.sources[u'test_source'] = TestSource()