]> git.cworth.org Git - sup/commitdiff
add reply-from hook for user-defined handling of default from in replies
authorMarc Hartstein <marc.hartstein@alum.vassar.edu>
Thu, 12 Jun 2008 16:05:46 +0000 (12:05 -0400)
committerWilliam Morgan <wmorgan-sup@masanjin.net>
Thu, 19 Jun 2008 17:49:46 +0000 (10:49 -0700)
The reply-from hook can now be used to process headers in a different manner
than the default for purposes of generating a default From: header when
replying to an email.

lib/sup/modes/reply-mode.rb

index e7b2929861064e6e039280d4e9cdb5e13c61eca9..d6a7dc68a7fc9f5aa6f2e9fc63fc0d60c07659f1 100644 (file)
@@ -19,6 +19,16 @@ Return value:
   A string containing the text of the quote line (can be multi-line)
 EOS
 
+  HookManager.register "reply-from", <<EOS
+Selects a default address for the From: header of a new reply.
+Variables:
+  message: a message object representing the message being replied to
+    (useful values include message.recipient_email, message.to, and message.cc)
+Return value:
+  A Person to be used as the default for the From: header, or nil to use the
+  default behavior.
+EOS
+
   def initialize message
     @m = message
 
@@ -29,8 +39,19 @@ EOS
 
     ## first, determine the address at which we received this email. this will
     ## become our From: address in the reply.
+    hook_reply_from = HookManager.run "reply-from", :message => @m
+
+    ## sanity check that selection is a Person (or we'll fail below)
+    ## don't check that it's an Account, though; assume they know what they're doing.
+    if hook_reply_from && !(hook_reply_from.is_a? Person)
+        Redwood::log "reply-from returned non-Person, using default from."
+        hook_reply_from = nil
+    end
+
     from =
-      if @m.recipient_email && AccountManager.is_account_email?(@m.recipient_email)
+      if hook_reply_from
+        hook_reply_from
+      elsif @m.recipient_email && AccountManager.is_account_email?(@m.recipient_email)
         PersonManager.person_for(@m.recipient_email)
       elsif(b = (@m.to + @m.cc).find { |p| AccountManager.is_account? p })
         b