Revert changes to MailMessage (#33816)

Author: driesvints

src/Illuminate/Notifications/Channels/MailChannel.php

@@ -89,11 +89,8 @@ protected function messageBuilder($notifiable, $notification, $message)
      */
     protected function buildView($message)
     {
-        if ($message->view || $message->textView) {
-            return [
-                'html' => $message->view,
-                'text' => $message->textView,
-            ];
+        if ($message->view) {
+            return $message->view;
         }
 
         if (property_exists($message, 'theme') && ! is_null($message->theme)) {

src/Illuminate/Notifications/Messages/MailMessage.php

@@ -17,13 +17,6 @@ class MailMessage extends SimpleMessage implements Renderable
      */
     public $view;
 
-    /**
-     * The plain text view to use for the message.
-     *
-     * @var string
-     */
-    public $textView;
-
     /**
      * The view data for the message.
      *
@@ -111,28 +104,13 @@ class MailMessage extends SimpleMessage implements Renderable
     public function view($view, array $data = [])
     {
         $this->view = $view;
-        $this->viewData = array_merge($this->viewData, $data);
+        $this->viewData = $data;
 
         $this->markdown = null;
 
         return $this;
     }
 
-    /**
-     * Set the plain text view for the message.
-     *
-     * @param  string  $textView
-     * @param  array  $data
-     * @return $this
-     */
-    public function text($textView, array $data = [])
-    {
-        $this->textView = $textView;
-        $this->viewData = array_merge($this->viewData, $data);
-
-        return $this;
-    }
-
     /**
      * Set the Markdown template for the notification.
      *
@@ -331,13 +309,9 @@ protected function arrayOfAddresses($address)
      */
     public function render()
     {
-        if (isset($this->view) || isset($this->textView)) {
+        if (isset($this->view)) {
             return Container::getInstance()->make('mailer')->render(
-                [
-                    'html' => $this->view,
-                    'text' => $this->textView,
-                ],
-                $this->data()
+                $this->view, $this->data()
             );
         }
 

tests/Integration/Notifications/SendingMailNotificationsTest.php

@@ -259,7 +259,7 @@ public function testMailIsSentUsingMailMessageWithHtmlAndPlain()
         ]);
 
         $this->mailer->shouldReceive('send')->once()->with(
-            ['html' => 'html', 'text' => 'plain'],
+            ['html', 'plain'],
             array_merge($notification->toMail($user)->toArray(), [
                 '__laravel_notification_id' => $notification->id,
                 '__laravel_notification' => get_class($notification),
@@ -291,7 +291,7 @@ public function testMailIsSentUsingMailMessageWithHtmlOnly()
         ]);
 
         $this->mailer->shouldReceive('send')->once()->with(
-            ['html' => 'html', 'text' => null],
+            'html',
             array_merge($notification->toMail($user)->toArray(), [
                 '__laravel_notification_id' => $notification->id,
                 '__laravel_notification' => get_class($notification),
@@ -323,7 +323,7 @@ public function testMailIsSentUsingMailMessageWithPlainOnly()
         ]);
 
         $this->mailer->shouldReceive('send')->once()->with(
-            ['html' => null, 'text' => 'plain'],
+            [null, 'plain'],
             array_merge($notification->toMail($user)->toArray(), [
                 '__laravel_notification_id' => $notification->id,
                 '__laravel_notification' => get_class($notification),
@@ -438,8 +438,7 @@ public function via($notifiable)
     public function toMail($notifiable)
     {
         return (new MailMessage)
-            ->view('html')
-            ->text('plain');
+            ->view(['html', 'plain']);
     }
 }
 
@@ -453,8 +452,7 @@ public function via($notifiable)
     public function toMail($notifiable)
     {
         return (new MailMessage)
-            ->view('html')
-            ->text(null);
+            ->view('html');
     }
 }
 
@@ -468,7 +466,6 @@ public function via($notifiable)
     public function toMail($notifiable)
     {
         return (new MailMessage)
-            ->view(null)
-            ->text('plain');
+            ->view([null, 'plain']);
     }
 }

tests/Notifications/NotificationMailMessageTest.php

@@ -18,11 +18,27 @@ public function testTemplate()
         $this->assertSame('notifications::foo', $message->markdown);
     }
 
+    public function testHtmlAndPlainView()
+    {
+        $message = new MailMessage;
+
+        $this->assertNull($message->view);
+        $this->assertSame([], $message->viewData);
+
+        $message->view(['notifications::foo', 'notifications::bar'], [
+            'foo' => 'bar',
+        ]);
+
+        $this->assertSame('notifications::foo', $message->view[0]);
+        $this->assertSame('notifications::bar', $message->view[1]);
+        $this->assertSame(['foo' => 'bar'], $message->viewData);
+    }
+
     public function testHtmlView()
     {
         $message = new MailMessage;
 
-        $this->assertSame(null, $message->view);
+        $this->assertNull($message->view);
         $this->assertSame([], $message->viewData);
 
         $message->view('notifications::foo', [
@@ -37,14 +53,14 @@ public function testPlainView()
     {
         $message = new MailMessage;
 
-        $this->assertSame(null, $message->textView);
+        $this->assertNull($message->view);
         $this->assertSame([], $message->viewData);
 
-        $message->text('notifications::foo', [
+        $message->view([null, 'notifications::foo'], [
             'foo' => 'bar',
         ]);
 
-        $this->assertSame('notifications::foo', $message->textView);
+        $this->assertSame('notifications::foo', $message->view[1]);
         $this->assertSame(['foo' => 'bar'], $message->viewData);
     }
 

tests/Notifications/NotificationSendQueuedNotificationTest.php

@@ -3,11 +3,12 @@
 namespace Illuminate\Tests\Notifications;
 
 use Illuminate\Contracts\Database\ModelIdentifier;
+use Illuminate\Database\Eloquent\Model;
 use Illuminate\Notifications\AnonymousNotifiable;
 use Illuminate\Notifications\ChannelManager;
+use Illuminate\Notifications\Notifiable;
 use Illuminate\Notifications\SendQueuedNotifications;
 use Illuminate\Support\Collection;
-use Illuminate\Tests\Integration\Notifications\NotifiableUser;
 use Mockery as m;
 use PHPUnit\Framework\TestCase;
 
@@ -52,3 +53,11 @@ public function testSerializationOfNormalNotifiable()
         $this->assertStringContainsString($serializedNotifiable, $serialized);
     }
 }
+
+class NotifiableUser extends Model
+{
+    use Notifiable;
+
+    public $table = 'users';
+    public $timestamps = false;
+}