From: William Morgan <wmorgan-sup@masanjin.net>
Date: Tue, 8 Jan 2008 16:56:13 +0000 (-0800)
Subject: bugfix in reply-to-all
X-Git-Url: https://git.cworth.org/git?a=commitdiff_plain;h=b0028987c0b2bd81729d93b2e2a477970f66084a;p=sup

bugfix in reply-to-all
the "all" option was still showing up even when it wasn't diffferent from
the "sender" option.
---

diff --git a/lib/sup/modes/reply-mode.rb b/lib/sup/modes/reply-mode.rb
index c95ffc0..04018e3 100644
--- a/lib/sup/modes/reply-mode.rb
+++ b/lib/sup/modes/reply-mode.rb
@@ -29,7 +29,7 @@ class ReplyMode < EditMessageMode
         AccountManager.default_account
       end
 
-    ## now, determine to: and cc: addressess. we  ignore reply-to for list
+    ## now, determine to: and cc: addressess. we ignore reply-to for list
     ## messages because it's typically set to the list address, which we
     ## explicitly treat with reply type :list
     to = @m.is_list_message? ? @m.from : (@m.replyto || @m.from)
@@ -56,10 +56,11 @@ class ReplyMode < EditMessageMode
 
     @headers[:user] = {}
 
+    not_me_ccs = cc.select { |p| !AccountManager.is_account?(p) }
     @headers[:all] = {
       "To" => [to.full_address],
-      "Cc" => cc.select { |p| !AccountManager.is_account?(p) }.map { |p| p.full_address },
-    } unless cc.empty?
+      "Cc" => not_me_ccs.map { |p| p.full_address },
+    } unless not_me_ccs.empty?
 
     @headers[:list] = {
       "To" => [@m.list_address.full_address],