From 2038cd52c65158a9b31299c62e4beb72c61a18b7 Mon Sep 17 00:00:00 2001
From: Tom Hughes <tom@compton.nu>
Date: Tue, 29 Jun 2010 09:43:30 +0100
Subject: [PATCH] Include an [OpenStreetMap] prefix on the subject of message
 notifications

Put back the [OpenStreetMap] prefix to the subject line of email
notifications of messages, but strip it again from any incoming
email replies.
---
 app/models/notifier.rb | 2 +-
 config/locales/en.yml  | 1 +
 script/deliver-message | 3 ++-
 3 files changed, 4 insertions(+), 2 deletions(-)

diff --git a/app/models/notifier.rb b/app/models/notifier.rb
index de0ad21d6..f63d08b67 100644
--- a/app/models/notifier.rb
+++ b/app/models/notifier.rb
@@ -48,7 +48,7 @@ class Notifier < ActionMailer::Base
   def message_notification(message)
     common_headers message.recipient
     from_header message.sender.display_name, "m", message.id, message.digest
-    subject message.title
+    subject I18n.t('notifier.message_notification.subject_header', :user => message.title, :locale => locale)
     body :to_user => message.recipient.display_name,
          :from_user => message.sender.display_name,
          :body => message.body,
diff --git a/config/locales/en.yml b/config/locales/en.yml
index f79a3d9e5..e3d8803ed 100644
--- a/config/locales/en.yml
+++ b/config/locales/en.yml
@@ -1062,6 +1062,7 @@ en:
       header: "{{from_user}} has commented on your recent OpenStreetMap diary entry with the subject {{subject}}:"
       footer: "You can also read the comment at {{readurl}} and you can comment at {{commenturl}} or reply at {{replyurl}}"
     message_notification:
+      subject_header: "[OpenStreetMap] {{subject}}"
       hi: "Hi {{to_user}},"
       header: "{{from_user}} has sent you a message through OpenStreetMap with the subject {{subject}}:"
       footer1: "You can also read the message at {{readurl}}"
diff --git a/script/deliver-message b/script/deliver-message
index 3df09f85f..58afa3451 100755
--- a/script/deliver-message
+++ b/script/deliver-message
@@ -15,6 +15,7 @@ else
   digest = message.digest
   from = message.recipient
   to = message.sender
+  subject = message.subject.sub(/\[OpenStreetMap\] */, "")
 end
 
 exit 0 unless recipient[3] == digest[0,6]
@@ -23,7 +24,7 @@ mail = TMail::Mail.parse(STDIN.readlines.join)
 
 message = Message.new(:sender => from, :recipient => to,
                       :sent_on => Time.now.getutc,
-                      :title => mail.subject,
+                      :title => subject,
                       :body => mail.body)
 message.save!
 
-- 
2.43.2