diff --git a/app/Http/Controllers/SettingsController.php b/app/Http/Controllers/SettingsController.php index 7eecd5476..4aa574a6d 100755 --- a/app/Http/Controllers/SettingsController.php +++ b/app/Http/Controllers/SettingsController.php @@ -645,14 +645,24 @@ class SettingsController extends Controller return redirect()->to('admin')->with('error', trans('admin/settings/message.update.error')); } - $setting->slack_endpoint = $request->input('slack_endpoint'); - $setting->slack_channel = $request->input('slack_channel'); - $setting->slack_botname = $request->input('slack_botname'); + $validatedData = $request->validate([ + 'slack_endpoint' => 'url|required_with:slack_channel|nullable', + 'slack_channel' => 'regex:/(? 'string|nullable', + ]); - if ($setting->save()) { + if ($validatedData) { + + $setting->slack_endpoint = $request->input('slack_endpoint'); + $setting->slack_channel = $request->input('slack_channel'); + $setting->slack_botname = $request->input('slack_botname'); + + $setting->save(); return redirect()->route('settings.index') ->with('success', trans('admin/settings/message.update.success')); + } + return redirect()->back()->withInput()->withErrors($setting->getErrors()); } diff --git a/app/Models/Setting.php b/app/Models/Setting.php index 57f97a95c..4212b1618 100755 --- a/app/Models/Setting.php +++ b/app/Models/Setting.php @@ -20,9 +20,6 @@ class Setting extends Model 'admin_cc_email' => 'email|nullable', 'default_currency' => 'required', 'locale' => 'required', - 'slack_endpoint' => 'url|required_with:slack_channel|nullable', - 'slack_channel' => 'regex:/(? 'string|nullable', 'labels_per_page' => 'numeric|min:1', 'labels_width' => 'numeric', 'labels_height' => 'numeric',