[Security][Notifier] Added integration of Login Link with the Notifier component

Author: wouterj

CHANGELOG.md

@@ -10,6 +10,7 @@ CHANGELOG
  * added the `t()` function to easily create `TranslatableMessage` objects
  * Added support for extracting messages from the `t()` function
  * Added `field_*` Twig functions to access string values from Form fields
+ * changed the `importance` context option of `NotificationEmail` to allow `null`
 
 5.0.0
 -----

Mime/NotificationEmail.php

@@ -37,6 +37,7 @@ class NotificationEmail extends TemplatedEmail
         'action_url' => null,
         'markdown' => false,
         'raw' => false,
+        'footer_text' => 'Notification e-mail sent by Symfony',
     ];
 
     public function __construct(Headers $headers = null, AbstractPart $body = null)
@@ -57,6 +58,18 @@ public function __construct(Headers $headers = null, AbstractPart $body = null)
         parent::__construct($headers, $body);
     }
 
+    /**
+     * Creates a NotificationEmail instance that is appropriate to send to normal (non-admin) users.
+     */
+    public static function asPublicEmail(Headers $headers = null, AbstractPart $body = null): self
+    {
+        $email = new static($headers, $body);
+        $email->context['importance'] = null;
+        $email->context['footer_text'] = null;
+
+        return $email;
+    }
+
     /**
      * @return $this
      */
@@ -166,7 +179,9 @@ public function getPreparedHeaders(): Headers
 
         $importance = $this->context['importance'] ?? self::IMPORTANCE_LOW;
         $this->priority($this->determinePriority($importance));
-        $headers->setHeaderBody('Text', 'Subject', sprintf('[%s] %s', strtoupper($importance), $this->getSubject()));
+        if ($this->context['importance']) {
+            $headers->setHeaderBody('Text', 'Subject', sprintf('[%s] %s', strtoupper($importance), $this->getSubject()));
+        }
 
         return $headers;
     }

Resources/views/Email/zurb_2/notification/body.html.twig

@@ -16,7 +16,7 @@
         <row>
             <columns large="12" small="12">
                 {% block lead %}
-                    <small><strong>{{ importance|upper }}</strong></small>
+                    {% if importance is not null %}<small><strong>{{ importance|upper }}</strong></small>{% endif %}
                     <p class="lead">
                         {{ email.subject }}
                     </p>
@@ -49,13 +49,15 @@
         <wrapper class="secondary">
             <spacer size="16"></spacer>
             {% block footer %}
+                {% if footer_text is defined and footer_text is not null %}
                 <row>
                     <columns small="12" large="6">
                         {% block footer_content %}
-                            <p><small>Notification e-mail sent by Symfony</small></p>
+                            <p><small>{{ footer_text }}</small></p>
                         {% endblock %}
                     </columns>
                 </row>
+                {% endif %}
             {% endblock %}
         </wrapper>
     </container>

Tests/Mime/NotificationEmailTest.php

@@ -26,6 +26,7 @@ public function test()
             'markdown' => true,
             'raw' => false,
             'a' => 'b',
+            'footer_text' => 'Notification e-mail sent by Symfony',
         ], $email->getContext());
     }
 
@@ -47,6 +48,7 @@ public function testSerialize()
             'markdown' => false,
             'raw' => true,
             'a' => 'b',
+            'footer_text' => 'Notification e-mail sent by Symfony',
         ], $email->getContext());
     }
 
@@ -63,4 +65,32 @@ public function testSubject()
         $headers = $email->getPreparedHeaders();
         $this->assertSame('[LOW] Foo', $headers->get('Subject')->getValue());
     }
+
+    public function testPublicMail()
+    {
+        $email = NotificationEmail::asPublicEmail()
+            ->markdown('Foo')
+            ->action('Bar', 'http://example.com/')
+            ->context(['a' => 'b'])
+        ;
+
+        $this->assertEquals([
+            'importance' => null,
+            'content' => 'Foo',
+            'exception' => false,
+            'action_text' => 'Bar',
+            'action_url' => 'http://example.com/',
+            'markdown' => true,
+            'raw' => false,
+            'a' => 'b',
+            'footer_text' => null,
+        ], $email->getContext());
+    }
+
+    public function testPublicMailSubject()
+    {
+        $email = NotificationEmail::asPublicEmail()->from('[email protected]')->subject('Foo');
+        $headers = $email->getPreparedHeaders();
+        $this->assertSame('Foo', $headers->get('Subject')->getValue());
+    }
 }