From 2849ddd7db267787e7a6c8c99b4d88593a90937b Mon Sep 17 00:00:00 2001 From: MonaHartdegen Date: Wed, 12 Feb 2020 09:58:50 +0100 Subject: [PATCH 1/5] added fields to order related factories --- .../Sales/src/Database/Factories/OrderFactory.php | 2 ++ .../Sales/src/Database/Factories/OrderItemFactory.php | 1 + .../src/Database/Factories/OrderPaymentFactory.php | 10 ++++++---- 3 files changed, 9 insertions(+), 4 deletions(-) diff --git a/packages/Webkul/Sales/src/Database/Factories/OrderFactory.php b/packages/Webkul/Sales/src/Database/Factories/OrderFactory.php index 78dcae3ba..38ca50906 100644 --- a/packages/Webkul/Sales/src/Database/Factories/OrderFactory.php +++ b/packages/Webkul/Sales/src/Database/Factories/OrderFactory.php @@ -49,6 +49,8 @@ $factory->define(Order::class, function (Faker $faker) { 'channel_id' => 1, 'channel_type' => Channel::class, 'cart_id' => 0, + 'shipping_method' => 'free_free', + 'shipping_title' => 'Free Shipping', ]; }); diff --git a/packages/Webkul/Sales/src/Database/Factories/OrderItemFactory.php b/packages/Webkul/Sales/src/Database/Factories/OrderItemFactory.php index c9a4fc635..3aa99c33f 100644 --- a/packages/Webkul/Sales/src/Database/Factories/OrderItemFactory.php +++ b/packages/Webkul/Sales/src/Database/Factories/OrderItemFactory.php @@ -31,6 +31,7 @@ $factory->define(OrderItem::class, function (Faker $faker) { }, 'created_at' => $now, 'updated_at' => $now, + 'product_type' => Product::class, ]; }); diff --git a/packages/Webkul/Sales/src/Database/Factories/OrderPaymentFactory.php b/packages/Webkul/Sales/src/Database/Factories/OrderPaymentFactory.php index 8a4bb49fe..5acce649d 100644 --- a/packages/Webkul/Sales/src/Database/Factories/OrderPaymentFactory.php +++ b/packages/Webkul/Sales/src/Database/Factories/OrderPaymentFactory.php @@ -6,12 +6,14 @@ use Faker\Generator as Faker; use Illuminate\Support\Facades\DB; use Webkul\Sales\Models\OrderPayment; -$factory->define(OrderPayment::class, function (Faker $faker) { - $now = date("Y-m-d H:i:s"); +$factory->define(OrderPayment::class, function (Faker $faker, array $attributes) { + + if (!array_key_exists('order_id', $attributes)) { + throw new InvalidArgumentException('order_id must be provided.'); + } return [ - 'created_at' => $now, - 'updated_at' => $now, + 'method' => 'cashondelivery', ]; }); From 7a69e4368e8f96ce86fee6c059664209a8807590 Mon Sep 17 00:00:00 2001 From: MonaHartdegen Date: Wed, 12 Feb 2020 10:33:17 +0100 Subject: [PATCH 2/5] modified functional GuestCheckoutCest --- tests/functional/Shop/GuestCheckoutCest.php | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/tests/functional/Shop/GuestCheckoutCest.php b/tests/functional/Shop/GuestCheckoutCest.php index e7610265f..4a62cb411 100644 --- a/tests/functional/Shop/GuestCheckoutCest.php +++ b/tests/functional/Shop/GuestCheckoutCest.php @@ -19,7 +19,7 @@ class GuestCheckoutCest $this->faker = Factory::create(); $pConfigDefault = [ - 'productInventory' => ['qty' => $this->faker->randomNumber(2)], + 'productInventory' => ['qty' => $this->faker->numberBetween(1, 1000)], 'attributeValues' => [ 'status' => true, 'new' => 1, @@ -27,7 +27,7 @@ class GuestCheckoutCest ], ]; $pConfigGuestCheckout = [ - 'productInventory' => ['qty' => $this->faker->randomNumber(2)], + 'productInventory' => ['qty' => $this->faker->numberBetween(1, 1000)], 'attributeValues' => [ 'status' => true, 'new' => 1, From 0219862de0dc625277805a5aef9853f898cc98b6 Mon Sep 17 00:00:00 2001 From: Vivek Sharma Date: Thu, 13 Feb 2020 14:00:59 +0530 Subject: [PATCH 3/5] [Fixed: issue #2440 resolved.] --- .../Admin/ConfigurationController.php | 32 ++++++++++++++----- 1 file changed, 24 insertions(+), 8 deletions(-) diff --git a/packages/Webkul/Velocity/src/Http/Controllers/Admin/ConfigurationController.php b/packages/Webkul/Velocity/src/Http/Controllers/Admin/ConfigurationController.php index cc167f89d..4f98ac4aa 100644 --- a/packages/Webkul/Velocity/src/Http/Controllers/Admin/ConfigurationController.php +++ b/packages/Webkul/Velocity/src/Http/Controllers/Admin/ConfigurationController.php @@ -75,6 +75,16 @@ class ConfigurationController extends Controller foreach ($params['images'] as $index => $images) { $params['advertisement'][$index] = $this->uploadAdvertisementImages($images, $index, $advertisement); } + + if ( $advertisement ) { + foreach ($advertisement as $key => $image_array) { + if (! isset($params['images'][$key])) { + foreach ($advertisement[$key] as $image) { + Storage::delete($image); + } + } + } + } } if (isset($params['product_view_images'])) { @@ -104,21 +114,28 @@ class ConfigurationController extends Controller public function uploadAdvertisementImages($data, $index, $advertisement) { $save_image = []; - + $save_data = $advertisement; + foreach ($data as $imageId => $image) { $file = 'images.' . $index . '.' . $imageId; $dir = 'velocity/images'; - if (Str::contains($imageId, 'image_')) { + if ( Str::contains($imageId, 'image_') ) { if (request()->hasFile($file) && $image) { - Storage::delete($dir . $file); - - $save_image[substr($imageId, 6, 1)] = request()->file($file)->store($dir); + $filter_index = substr($imageId, 6, 1); + if ( isset($data[$filter_index]) ) { + $size = array_key_last($save_data[$index]); + + $save_image[$size + 1] = request()->file($file)->store($dir); + } else { + $save_image[substr($imageId, 6, 1)] = request()->file($file)->store($dir); + } } } else { - if ( isset($advertisement[$index][$imageId]) && $advertisement[$index][$imageId]) { + if ( isset($advertisement[$index][$imageId]) && $advertisement[$index][$imageId] && !request()->hasFile($file)) { $save_image[$imageId] = $advertisement[$index][$imageId]; - unset($advertisement[$index][$imageId]); + + unset($advertisement[$index][$imageId]); } if (request()->hasFile($file) && isset($advertisement[$index][$imageId])) { @@ -132,7 +149,6 @@ class ConfigurationController extends Controller if ( isset($advertisement[$index]) && $advertisement[$index]) { foreach ($advertisement[$index] as $imageId) { Storage::delete($imageId); - $save_image[$imageId] = ''; } } From 9d80f73319d64e62b970641d3fb7caee98a6c897 Mon Sep 17 00:00:00 2001 From: rahul shukla Date: Thu, 13 Feb 2020 14:41:40 +0530 Subject: [PATCH 4/5] Issue #2449 --- .../src/Resources/views/checkout/cart/mini-cart.blade.php | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/packages/Webkul/Shop/src/Resources/views/checkout/cart/mini-cart.blade.php b/packages/Webkul/Shop/src/Resources/views/checkout/cart/mini-cart.blade.php index da7cc9f81..a0e37eb47 100755 --- a/packages/Webkul/Shop/src/Resources/views/checkout/cart/mini-cart.blade.php +++ b/packages/Webkul/Shop/src/Resources/views/checkout/cart/mini-cart.blade.php @@ -53,10 +53,10 @@ {!! view_render_event('bagisto.shop.checkout.cart-mini.item.options.before', ['item' => $item]) !!} - + @if (isset($item->additional['attributes']))
- + @foreach ($item->additional['attributes'] as $attribute) {{ $attribute['attribute_name'] }} : {{ $attribute['option_label'] }}
@endforeach @@ -96,7 +96,7 @@ @else -