root 5 years ago
parent
commit
ef580c94c4
2 changed files with 54 additions and 15 deletions
  1. 10 15
      app/Http/Controllers/Passport/CommController.php
  2. 44 0
      app/Jobs/SendEmail.php

+ 10 - 15
app/Http/Controllers/Passport/CommController.php

@@ -8,6 +8,7 @@ use App\Http\Controllers\Controller;
 use Illuminate\Http\Exceptions\HttpResponseException;
 use Illuminate\Support\Facades\Mail;
 use Illuminate\Support\Facades\Redis;
+use App\Jobs\SendEmail;
 
 class CommController extends Controller
 {
@@ -34,21 +35,15 @@ class CommController extends Controller
         }
         $code = rand(100000, 999999);
         $subject = config('v2board.app_name', 'V2Board') . '邮箱验证码';
-        Mail::send(
-            'mail.sendEmailVerify', 
-            [
-                'code' => $code,
-                'name' => config('v2board.app_name', 'V2Board')
-            ],
-            function ($message) use($email, $subject) { 
-                $message->to($email)->subject($subject); 
-            }
-        );
-        if (count(Mail::failures()) >= 1) {
-            // 发送失败
-            abort(500, '发送失败');
-        }
-
+    	$this->dispatch(new SendEmail([
+    		'email' => $email,
+    		'template_name' => 'mail.sendEmailVerify',
+    		'template_value' => [
+    			'code' => $code,
+    			'name' => config('v2board.app_name', 'V2Board')
+    		],
+    		'subject' => $subject
+    	]));
         Redis::set($redisKey, $code);
         Redis::expire($redisKey, 600);
         return response([

+ 44 - 0
app/Jobs/SendEmail.php

@@ -0,0 +1,44 @@
+<?php
+
+namespace App\Jobs;
+
+use Illuminate\Bus\Queueable;
+use Illuminate\Contracts\Queue\ShouldQueue;
+use Illuminate\Foundation\Bus\Dispatchable;
+use Illuminate\Queue\InteractsWithQueue;
+use Illuminate\Queue\SerializesModels;
+use Illuminate\Support\Facades\Mail;
+
+class SendEmail implements ShouldQueue
+{
+    use Dispatchable, InteractsWithQueue, Queueable, SerializesModels;
+    protected $params;
+    /**
+     * Create a new job instance.
+     *
+     * @return void
+     */
+    public function __construct($params)
+    {
+        $this->params = $params;
+    }
+
+    /**
+     * Execute the job.
+     *
+     * @return void
+     */
+    public function handle()
+    {
+        $params = $this->params;
+        $email = $params['email'];
+        $subject = $params['subject'];
+        Mail::send(
+            $params['template_name'], 
+            $params['template_value'],
+            function ($message) use($email, $subject) { 
+                $message->to($email)->subject($subject); 
+            }
+        );
+    }
+}