From 74283f1aa24c26315292b2c3e7029e23b2c2d2c8 Mon Sep 17 00:00:00 2001 From: Seyed AmirHossein Adhami MirHosseini Date: Mon, 18 Jul 2022 21:01:51 +0430 Subject: [PATCH] bug fix - at least one shipping method must be enabled --- .../Http/Controllers/ConfigurationController.php | 14 +++++++++----- 1 file changed, 9 insertions(+), 5 deletions(-) diff --git a/packages/Webkul/Admin/src/Http/Controllers/ConfigurationController.php b/packages/Webkul/Admin/src/Http/Controllers/ConfigurationController.php index 9385bcef8..366857504 100755 --- a/packages/Webkul/Admin/src/Http/Controllers/ConfigurationController.php +++ b/packages/Webkul/Admin/src/Http/Controllers/ConfigurationController.php @@ -97,16 +97,20 @@ class ConfigurationController extends Controller * Store a newly created resource in storage. * * @param \Webkul\Admin\Http\Requests\ConfigurationForm $request - * @return \Illuminate\Http\Response + * @return \Illuminate\Http\RedirectResponse */ public function store(ConfigurationForm $request) { $data = $request->request->all(); if (isset($data['sales']['carriers'])) { - $freeShipping = $data['sales']['carriers']['free']['active']; - $flatrateShipping = $data['sales']['carriers']['flatrate']['active']; - - if (! $freeShipping && ! $flatrateShipping) { + $at_least_one_carrier_enabled = false; + foreach ($data['sales']['carriers'] as $carrier) { + if ($carrier['active']) { + $at_least_one_carrier_enabled = true; + break; + } + } + if (! $at_least_one_carrier_enabled) { session()->flash('error', trans('admin::app.configuration.enable-atleast-one-shipping')); return redirect()->back(); }