Merge pull request #3294 from anthonybocci/setup-requires-username-3244

Set the mail username optional in setup
cachethq · Dec 29, 2018 · e857143 · e857143
2 parents 59df070 + cecf894
commit e857143
Showing 1 changed file with 5 additions and 1 deletion.
diff --git a/app/Http/Controllers/SetupController.php b/app/Http/Controllers/SetupController.php
@@ -194,10 +194,14 @@ public function postStep1()
             return $input->env['mail_driver'] === 'smtp';
         });
 
-        $v->sometimes(['env.mail_address', 'env.mail_username', 'env.mail_password'], 'required', function ($input) {
+        $v->sometimes(['env.mail_address', 'env.mail_password'], 'required', function ($input) {
             return !in_array($input->env['mail_driver'], ['log', 'smtp']);
         });
 
+        $v->sometimes(['env.mail_username'], 'required', function($input) {
+            return !in_array($input->env['mail_username'], ['sendmail']);
+        });
+
         if ($v->passes()) {
             return Response::json(['status' => 1]);
         }