From fa60f25185d5a454ddb325235aa2441d45e0186d Mon Sep 17 00:00:00 2001
From: Thorsten Eckel <te@znuny.com>
Date: Wed, 14 Nov 2018 16:14:25 +0100
Subject: [PATCH] Fixed informal/formal mix of language in customer facing
 notification templates texts to formal language - as discussed in #2348 and
 #2350. Thanks to @Charburner who did the actual change!

---
 app/views/mailer/password_change/de.html.erb         | 10 +++++-----
 app/views/mailer/password_reset/de.html.erb          | 12 ++++++------
 .../notification_factory_mailer_template_test.rb     | 12 ++++++------
 3 files changed, 17 insertions(+), 17 deletions(-)

diff --git a/app/views/mailer/password_change/de.html.erb b/app/views/mailer/password_change/de.html.erb
index 24cf7e9b8..e16032535 100644
--- a/app/views/mailer/password_change/de.html.erb
+++ b/app/views/mailer/password_change/de.html.erb
@@ -1,9 +1,9 @@
-Dein #{config.product_name} Passwort wurde geändert
+Ihr #{config.product_name} Passwort wurde geändert
 
-<div>Hallo #{user.firstname},</div>
+<div>Hallo #{user.firstname} #{user.lastname},</div>
 <br>
-<div>das Passwort für Dein #{config.product_name} Account <b>#{user.login}</b> wurde kürzlich geändert.</div>
+<div>das Passwort für Ihren #{config.product_name} Account <b>#{user.login}</b> wurde kürzlich geändert.</div>
 <br>
-<div>Diese Aktivität ist Dir nicht bekannt? In diesen Fall kontaktiere Deinen System-Administrator.</div>
+<div>Diese Aktivität ist Ihnen nicht bekannt? In diesem Fall kontaktieren Sie Ihren Systemadministrator.</div>
 <br>
-<div>Dein #{config.product_name} Team</div>
+<div>Ihr #{config.product_name} Team</div>
diff --git a/app/views/mailer/password_reset/de.html.erb b/app/views/mailer/password_reset/de.html.erb
index fd03cff03..ea325b563 100644
--- a/app/views/mailer/password_reset/de.html.erb
+++ b/app/views/mailer/password_reset/de.html.erb
@@ -1,15 +1,15 @@
-Zurücksetzen Deines #{config.product_name} Passworts
+Zurücksetzen Ihres #{config.product_name} Passworts
 
-<div>Hallo #{user.firstname},</div>
+<div>Hallo #{user.firstname} #{user.lastname},</div>
 <br>
-<div>wir haben eine Anfrage zum Zurücksetzen des Passworts für #{config.product_name} Account <b>#{user.login}</b> erhalten.</div>
+<div>wir haben eine Anfrage zum Zurücksetzen des Passworts für Ihren #{config.product_name} Account <b>#{user.login}</b> erhalten.</div>
 <br>
-<div>Wenn Sie Ihr Passwort zurückzusetzen wollen, klicken Sie auf den unten stehenden Link (oder kopieren Sie die URL in den Browser einfügen):</div>
+<div>Wenn Sie Ihr Passwort zurücksetzen wollen, klicken Sie auf den unten stehenden Link (oder kopieren Sie die URL in Ihren Browser):</div>
 <br>
 <div><a href="#{config.http_type}://#{config.fqdn}/#password_reset_verify/#{token.name}">#{config.http_type}://#{config.fqdn}/#password_reset_verify/#{token.name}</a></div>
 <br>
 <div>Dieser Link führt Sie zu einer Seite, auf der Sie Ihr Passwort ändern können.</div>
 <br>
-<div>Wenn Sie Ihr Passwort nicht zurücksetzen wollen, ignorieren Sie diese Meldung. Das Passwort bleibt unverändert.</div>
+<div>Wenn Sie Ihr Passwort nicht zurücksetzen wollen, ignorieren Sie diese Meldung. In diesem Fall bleibt das Passwort unverändert.</div>
 <br>
-<div>Dein #{config.product_name} Team</div>
+<div>Ihr #{config.product_name} Team</div>
diff --git a/test/unit/notification_factory_mailer_template_test.rb b/test/unit/notification_factory_mailer_template_test.rb
index ed0010914..b03bf8366 100644
--- a/test/unit/notification_factory_mailer_template_test.rb
+++ b/test/unit/notification_factory_mailer_template_test.rb
@@ -47,10 +47,10 @@ class NotificationFactoryMailerTemplateTest < ActiveSupport::TestCase
         user: agent1,
       },
     )
-    assert_match('Zurücksetzen Deines', result[:subject])
+    assert_match('Zurücksetzen Ihres', result[:subject])
     assert_match('wir haben eine Anfrage zum Zurücksetzen', result[:body])
-    assert_match('Dein', result[:body])
-    assert_match('Dein', result[:body])
+    assert_match('Ihr', result[:body])
+    assert_match('Ihr', result[:body])
     assert_match('Notification&lt;b&gt;xxx&lt;/b&gt;', result[:body])
     assert_no_match('Your', result[:body])
 
@@ -61,9 +61,9 @@ class NotificationFactoryMailerTemplateTest < ActiveSupport::TestCase
         user: agent1,
       },
     )
-    assert_match('Zurücksetzen Deines', result[:subject])
+    assert_match('Zurücksetzen Ihres', result[:subject])
     assert_match('wir haben eine Anfrage zum Zurücksetzen', result[:body])
-    assert_match('Dein', result[:body])
+    assert_match('Ihr', result[:body])
     assert_match('Notification&lt;b&gt;xxx&lt;/b&gt;', result[:body])
     assert_no_match('Your', result[:body])
 
@@ -78,7 +78,7 @@ class NotificationFactoryMailerTemplateTest < ActiveSupport::TestCase
     assert_match('We received a request to reset the password', result[:body])
     assert_match('Your', result[:body])
     assert_match('Notification&lt;b&gt;xxx&lt;/b&gt;', result[:body])
-    assert_no_match('Dein', result[:body])
+    assert_no_match('Ihr', result[:body])
 
     ticket = Ticket.create(
       group_id: Group.lookup(name: 'Users').id,