Subject: [sup-devel] [PATCH] s/@filename/@path/ in MBox#store_message

From: Adam Lloyd <[email-masked]>
Date: Mon, 16 Aug 2010 21:18:43 -0400

lib/sup/mbox.rb |  4 ++--
 1 files changed, 2 insertions(+), 2 deletions(-)

@filename is undefined in MBox, so trying to store a message to an mbox
source results in a crash.  @path contains the needed information.

This fixes issues 79 and 89, related to crashes on sending mail.

diff --git a/lib/sup/mbox.rb b/lib/sup/mbox.rb
--- a/lib/sup/mbox.rb
+++ b/lib/sup/mbox.rb
@@ -90,8 +90,8 @@
   end
 
   def store_message date, from_email, &block
-    need_blank = File.exists?(@filename) && !File.zero?(@filename)
-    File.open(@filename, "ab") do |f|
+    need_blank = File.exists?(@path) && !File.zero?(@path)
+    File.open(@path, "ab") do |f|
       f.puts if need_blank
       f.puts "From #{from_email} #{date.asctime}"
       yield f
_______________________________________________
Sup-devel mailing list
[email-masked]
http://rubyforge.org/mailman/listinfo/sup-devel