Merge pull request #376 from prashant-webkul/master

Issues and bug fixes
This commit is contained in:
JItendra Singh 2018-12-21 18:46:39 +05:30 committed by GitHub
commit d7b95eec47
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
15 changed files with 88 additions and 58 deletions

View File

@ -67,7 +67,7 @@ class Order {
$productListener = app(\Webkul\Admin\Listeners\Product::class);
foreach ($order->items as $item) {
$productListener->afterProductCreated($item->product);
$productListener->afterOrderRecieved($item->product->id, $item->qty_ordered);
}
}
}

View File

@ -213,6 +213,20 @@ class Product {
return true;
}
/**
* Updates the product quantity when the order is received
*
* @param $productId
* @param $itemQuantity
*/
public function afterOrderRecieved($productId, $itemQuantity) {
$productGrid = $this->productGrid->findOneByField('product_id', $productId);
$productGrid->update(['quantity' => $productGrid->quantity - $itemQuantity]);
return true;
}
/**
* Manually invoke this function when you have created the products by importing or seeding or factory.
*/

View File

@ -1,8 +1,8 @@
const { mix } = require("laravel-mix");
require("laravel-mix-merge-manifest");
var publicPath = 'publishable/assets';
// var publicPath = "../../../public/vendor/webkul/admin/assets";
// var publicPath = 'publishable/assets';
var publicPath = "../../../public/vendor/webkul/admin/assets";
mix.setPublicPath(publicPath).mergeManifest();
mix.disableNotifications();

View File

@ -639,9 +639,9 @@ class Cart {
}
}
$cart->customer_email = $cart->shipping_address->email;
$cart->customer_first_name = $cart->shipping_address->first_name;
$cart->customer_last_name = $cart->shipping_address->last_name;
$cart->customer_email = $cart->billing_address->email;
$cart->customer_first_name = $cart->billing_address->first_name;
$cart->customer_last_name = $cart->billing_address->last_name;
$cart->save();
return true;

View File

@ -103,6 +103,7 @@ class RegistrationController extends Controller
/**
* Method to verify account
*
* @param string $token
*/
public function verifyAccount($token)
{

View File

@ -26,7 +26,13 @@ return [
'name' => 'Wishlist',
'route' =>'customer.wishlist.index',
'sort' => 4
], [
'key' => 'account.orders',
'name' => 'Orders',
'route' =>'customer.orders.index',
'sort' => 5
]
];
?>

View File

@ -57,7 +57,7 @@ class SliderController extends controller
$this->validate(request(), [
'title' => 'string|required',
'channel_id' => 'required',
'image' => 'required',
'image' => 'required|mimes:jpeg,bmp,png'
]);
$result = $this->slider->save(request()->all());
@ -90,6 +90,7 @@ class SliderController extends controller
$this->validate(request(), [
'title' => 'string|required',
'channel_id' => 'required',
'image' => 'sometimes|mimes:jpeg,bmp,png'
]);
$result = $this->slider->updateItem(request()->all(), $id);

View File

@ -84,9 +84,9 @@ class SubscriptionController extends Controller
$mailSent = true;
try {
session()->flash('success', trans('shop::app.subscription.subscribed'));
Mail::send(new SubscriptionEmail($subscriptionData));
session()->flash('success', trans('shop::app.subscription.subscribed'));
} catch(\Exception $e) {
session()->flash('error', trans('shop::app.subscription.not-subscribed'));

View File

@ -25,6 +25,10 @@ body {
font-family: "Montserrat", sans-serif;
}
textarea {
resize: none;
}
input {
font-family: 'Montserrat', sans-serif;
}

View File

@ -286,7 +286,7 @@ return [
'newest-first' => 'Newest First',
'oldest-first' => 'Oldest First',
'cheapest-first' => 'Cheapest First',
'expensive-first' => 'Expensive First',
'expansive-first' => 'Expensive First',
'show' => 'Show',
'pager-info' => 'Showing :showing of :total Items',
'description' => 'Description',

View File

@ -9,6 +9,7 @@
{{ __('shop::app.search.no-results') }}
@endif
@if($results)
<div class="main mb-30" style="min-height: 27vh;">
@if($results->isEmpty())
<div class="search-result-status">
@ -37,5 +38,5 @@
@include('ui::datagrid.pagination')
@endif
</div>
@endif
@endsection

View File

@ -1,8 +1,8 @@
const { mix } = require("laravel-mix");
require("laravel-mix-merge-manifest");
var publicPath = 'publishable/assets';
// var publicPath = "../../../public/themes/default/assets";
// var publicPath = 'publishable/assets';
var publicPath = "../../../public/themes/default/assets";
mix.setPublicPath(publicPath).mergeManifest();
mix.disableNotifications();

View File

@ -175,16 +175,14 @@ class TaxCategoryController extends Controller
*/
public function destroy($id)
{
if($this->taxCategory->count() == 1) {
session()->flash('error', trans('admin::app.settings.tax-categories.atleast-one'));
} else {
try {
Event::fire('tax.tax_category.delete.before', $id);
$this->taxCategory->delete($id);
Event::fire('tax.tax_category.delete.after', $id);
session()->flash('success', trans('admin::app.settings.tax-categories.delete'));
} catch(Exception $e) {
return redirect()->back();
}
return redirect()->back();

View File

@ -154,18 +154,19 @@ class TaxRateController extends Controller
*/
public function destroy($id)
{
if($this->taxRate->count() == 1) {
session()->flash('error', trans('admin::app.settings.tax-rates.atleast-one'));
} else {
// if($this->taxRate->count() == 1) {
// session()->flash('error', trans('admin::app.settings.tax-rates.atleast-one'));
// } else {
// session()->flash('success', trans('admin::app.settings.tax-rates.delete'));
// }
Event::fire('tax.tax_rate.delete.before', $id);
$this->taxRate->delete($id);
Event::fire('tax.tax_rate.delete.after', $id);
session()->flash('success', trans('admin::app.settings.tax-rates.delete'));
}
return redirect()->back();
}
}

View File

@ -24,6 +24,10 @@ a:active {
color: $brand-color;
}
textarea {
resize: none;
}
ul {
margin: 0;
padding: 0;