diff --git a/packages/Webkul/Admin/src/Resources/views/catalog/products/edit.blade.php b/packages/Webkul/Admin/src/Resources/views/catalog/products/edit.blade.php index ef383eba2..7e83c7f91 100644 --- a/packages/Webkul/Admin/src/Resources/views/catalog/products/edit.blade.php +++ b/packages/Webkul/Admin/src/Resources/views/catalog/products/edit.blade.php @@ -92,7 +92,7 @@ {{ $attribute->admin_name }} @if ($attribute->type == 'price') - ({{ currency()->symbol(core()->getBaseCurrencyCode()) }}) + ({{ core()->currencySymbol(core()->getBaseCurrencyCode()) }}) @endif validate(request(), [ 'slug' => ['required', 'unique:category_translations,slug', new \Webkul\Core\Contracts\Validations\Slug], - 'name' => 'required' + 'name' => 'required', + 'image.*' => 'mimes:jpeg,jpg,bmp,png' ]); $this->category->create(request()->all()); @@ -116,6 +117,7 @@ class CategoryController extends Controller } }], $locale . '.name' => 'required', + 'image.*' => 'mimes:jpeg,jpg,bmp,png' ]); $this->category->update(request()->all(), $id); diff --git a/packages/Webkul/Core/src/Core.php b/packages/Webkul/Core/src/Core.php index 0e694b17d..f385578fb 100644 --- a/packages/Webkul/Core/src/Core.php +++ b/packages/Webkul/Core/src/Core.php @@ -358,6 +358,21 @@ class Core return currency($this->convertPrice($amount), $currencyCode); } + /** + * Return currency symbol from currency code + * + * @param float $price + * @return string + */ + public function currencySymbol($code) + { + try { + return currency()->symbol($code); + } catch (\Exception $e) { + return $code; + } + } + /** * Format and convert price with currency symbol * diff --git a/packages/Webkul/Core/src/Database/Migrations/2018_12_21_101307_alter_channels_table.php b/packages/Webkul/Core/src/Database/Migrations/2018_12_21_101307_alter_channels_table.php new file mode 100644 index 000000000..cd0a009da --- /dev/null +++ b/packages/Webkul/Core/src/Database/Migrations/2018_12_21_101307_alter_channels_table.php @@ -0,0 +1,33 @@ +dropForeign('channels_default_locale_id_foreign'); + $table->dropForeign('channels_base_currency_id_foreign'); + $table->foreign('default_locale_id')->references('id')->on('locales'); + $table->foreign('base_currency_id')->references('id')->on('currencies'); + }); + } + + /** + * Reverse the migrations. + * + * @return void + */ + public function down() + { + // + } +} \ No newline at end of file diff --git a/packages/Webkul/Core/src/Http/Controllers/ChannelController.php b/packages/Webkul/Core/src/Http/Controllers/ChannelController.php index 6b4646a2b..73d31a776 100644 --- a/packages/Webkul/Core/src/Http/Controllers/ChannelController.php +++ b/packages/Webkul/Core/src/Http/Controllers/ChannelController.php @@ -77,7 +77,9 @@ class ChannelController extends Controller 'locales' => 'required|array|min:1', 'default_locale_id' => 'required', 'currencies' => 'required|array|min:1', - 'base_currency_id' => 'required' + 'base_currency_id' => 'required', + 'logo.*' => 'mimes:jpeg,jpg,bmp,png', + 'favicon.*' => 'mimes:jpeg,jpg,bmp,png' ]); $this->channel->create(request()->all()); @@ -115,7 +117,9 @@ class ChannelController extends Controller 'locales' => 'required|array|min:1', 'default_locale_id' => 'required', 'currencies' => 'required|array|min:1', - 'base_currency_id' => 'required' + 'base_currency_id' => 'required', + 'logo.*' => 'mimes:jpeg,jpg,bmp,png', + 'favicon.*' => 'mimes:jpeg,jpg,bmp,png' ]); $this->channel->update(request()->all(), $id); diff --git a/packages/Webkul/Core/src/Http/Controllers/CurrencyController.php b/packages/Webkul/Core/src/Http/Controllers/CurrencyController.php index 324257403..b87215b7e 100644 --- a/packages/Webkul/Core/src/Http/Controllers/CurrencyController.php +++ b/packages/Webkul/Core/src/Http/Controllers/CurrencyController.php @@ -125,12 +125,16 @@ class CurrencyController extends Controller */ public function destroy($id) { - $result = $this->currency->delete($id); + try { + $result = $this->currency->delete($id); - if($result) - session()->flash('success', 'Currency deleted successfully.'); - else - session()->flash('error', 'At least one currency is required.'); + if($result) + session()->flash('success', 'Currency deleted successfully.'); + else + session()->flash('error', 'At least one currency is required.'); + } catch (\Exception $e) { + session()->flash('error', $e->getMessage()); + } return redirect()->back(); } diff --git a/packages/Webkul/Ui/src/Resources/assets/js/components/image/image-item.vue b/packages/Webkul/Ui/src/Resources/assets/js/components/image/image-item.vue index 8a15cc83d..ff04f1ed4 100644 --- a/packages/Webkul/Ui/src/Resources/assets/js/components/image/image-item.vue +++ b/packages/Webkul/Ui/src/Resources/assets/js/components/image/image-item.vue @@ -2,7 +2,7 @@