From 2be300b9b9ae5575259b1e1cc39693a731cc17bf Mon Sep 17 00:00:00 2001
From: Gabriel Guzman <gabe@guzman-nunez.com>
Date: Wed, 22 Feb 2012 06:06:44 -0500
Subject: [PATCH 1/2] Add initial support for listing goodbad messages by user.

---
 src/scripts/goodbad.coffee | 15 +++++++++------
 1 file changed, 9 insertions(+), 6 deletions(-)

diff --git a/src/scripts/goodbad.coffee b/src/scripts/goodbad.coffee
index b5eda84f7..82b841aa9 100644
--- a/src/scripts/goodbad.coffee
+++ b/src/scripts/goodbad.coffee
@@ -28,13 +28,16 @@ class GoodBad
     maxBadNum  
   goodlist: -> @goodcache
   badlist: -> @badcache
-  good: (goodString) ->
-    goodthing = {num: @nextGoodNum(), good: goodString}
+  good: (msg) ->
+    goodString = msg.match[2]
+    goodthing = {num: @nextGoodNum(), user: user, good: goodString}
     @goodcache.push goodthing
     @robot.brain.data.good = @goodcache
     goodthing
-  bad: (badString) ->
-    badthing = {num: @nextBadNum(), bad: badString}
+  bad: (msg) ->
+    badString = msg.match[2]
+    user  = msg.message.user.name
+    badthing = {num: @nextBadNum(), user: user, bad: badString}
     @badcache.push badthing
     @robot.brain.data.bad = @badcache
     badthing
@@ -49,11 +52,11 @@ module.exports = (robot) ->
   goodbad = new GoodBad robot
   
   robot.respond /(good) (.+?)$/i, (msg) ->
-    good = goodbad.good msg.match[2]
+    good = goodbad.good msg
     msg.send "The sprint is thriving!"
 
   robot.respond /(bad) (.+?)$/i, (msg) ->
-    bad = goodbad.bad msg.match[2]
+    bad = goodbad.bad msg
     msg.send "The sprint is festering..."
 
   robot.respond /(goodlist)/i, (msg) ->

From 742eb9270d294ee7b658234759db92035e17fd20 Mon Sep 17 00:00:00 2001
From: Gabriel Guzman <gabriel.guzman@gammaemail.com>
Date: Wed, 4 Jul 2012 16:42:52 -0400
Subject: [PATCH 2/2] Latest version of goodbad.

---
 src/scripts/goodbad.coffee | 17 ++++++++---------
 1 file changed, 8 insertions(+), 9 deletions(-)

diff --git a/src/scripts/goodbad.coffee b/src/scripts/goodbad.coffee
index 82b841aa9..7ce6f6598 100644
--- a/src/scripts/goodbad.coffee
+++ b/src/scripts/goodbad.coffee
@@ -28,16 +28,13 @@ class GoodBad
     maxBadNum  
   goodlist: -> @goodcache
   badlist: -> @badcache
-  good: (msg) ->
-    goodString = msg.match[2]
-    goodthing = {num: @nextGoodNum(), user: user, good: goodString}
+  good: (goodString) ->
+    goodthing = {num: @nextGoodNum(), good: goodString}
     @goodcache.push goodthing
     @robot.brain.data.good = @goodcache
     goodthing
-  bad: (msg) ->
-    badString = msg.match[2]
-    user  = msg.message.user.name
-    badthing = {num: @nextBadNum(), user: user, bad: badString}
+  bad: (badString) ->
+    badthing = {num: @nextBadNum(), bad: badString}
     @badcache.push badthing
     @robot.brain.data.bad = @badcache
     badthing
@@ -52,11 +49,13 @@ module.exports = (robot) ->
   goodbad = new GoodBad robot
   
   robot.respond /(good) (.+?)$/i, (msg) ->
-    good = goodbad.good msg
+    message = "#{msg.message.user.name}: #{msg.match[2]}"
+    good = goodbad.good message
     msg.send "The sprint is thriving!"
 
   robot.respond /(bad) (.+?)$/i, (msg) ->
-    bad = goodbad.bad msg
+    message = "#{msg.message.user.name}: #{msg.match[2]}"
+    bad = goodbad.bad message
     msg.send "The sprint is festering..."
 
   robot.respond /(goodlist)/i, (msg) ->