[myexperiment-hackers] [2239] trunk/app/controllers/messages_controller.rb: fixed cascading "RE: " when replying to messages

[noreply]

Revision

2239

Author

dgc

Date

2009-07-07 07:22:34 -0400 (Tue, 07 Jul 2009)

Log Message

fixed cascading "RE: " when replying to messages

Modified Paths

• trunk/app/controllers/messages_controller.rb

Diff

Modified: trunk/app/controllers/messages_controller.rb (2238 => 2239)

--- trunk/app/controllers/messages_controller.rb	2009-07-03 15:13:05 UTC (rev 2238)
+++ trunk/app/controllers/messages_controller.rb	2009-07-07 11:22:34 UTC (rev 2239)
@@ -87,9 +87,13 @@
       #  so this is a mere validation - which saves user from typing the message in and learning that
       #  it can't be set because of the limit, which is expired)
       if params[:reply_id]
+
+        subject = @reply.subject
+        subject = "RE: #{subject}" unless subject.starts_with?("RE: ")
+
         @message = Message.new(:to => @reply.from,
                                :reply_id => @reply.id,
-                               :subject => "RE: " + @reply.subject,
+                               :subject => subject,
                                :body => @reply.body.split(/\n/).collect {|line| ">> #{line}"}.join) # there has to be a 'ruby-er' way of doing this?
       else
         @message = Message.new