Make email not required for customer/vendor
This commit is contained in:
@@ -243,6 +243,7 @@ class Users extends Controller
|
||||
public function autocomplete(ARequest $request)
|
||||
{
|
||||
$user = false;
|
||||
$data = false;
|
||||
|
||||
$column = $request['column'];
|
||||
$value = $request['value'];
|
||||
@@ -250,7 +251,7 @@ class Users extends Controller
|
||||
if (!empty($column) && !empty($value)) {
|
||||
switch ($column) {
|
||||
case 'id':
|
||||
$user = User::find();
|
||||
$user = User::find((int) $value);
|
||||
break;
|
||||
case 'email':
|
||||
$user = User::where('email', $value)->first();
|
||||
@@ -258,12 +259,16 @@ class Users extends Controller
|
||||
default:
|
||||
$user = User::where($column, $value)->first();
|
||||
}
|
||||
|
||||
$data = $user;
|
||||
} elseif (!empty($column) && empty($value)) {
|
||||
$data = trans('validation.required', ['attribute' => $column]);
|
||||
}
|
||||
|
||||
return response()->json([
|
||||
'errors' => ($user) ? false: true,
|
||||
'success' => ($user) ? true: false,
|
||||
'data' => $user
|
||||
'errors' => ($user) ? false : true,
|
||||
'success' => ($user) ? true : false,
|
||||
'data' => $data
|
||||
]);
|
||||
}
|
||||
}
|
||||
|
||||
@@ -44,6 +44,10 @@ class Vendors extends Controller
|
||||
*/
|
||||
public function store(Request $request)
|
||||
{
|
||||
if (empty($request['email'])) {
|
||||
$request['email'] = '';
|
||||
}
|
||||
|
||||
Vendor::create($request->all());
|
||||
|
||||
$message = trans('messages.success.added', ['type' => trans_choice('general.vendors', 1)]);
|
||||
@@ -120,6 +124,10 @@ class Vendors extends Controller
|
||||
*/
|
||||
public function update(Vendor $vendor, Request $request)
|
||||
{
|
||||
if (empty($request['email'])) {
|
||||
$request['email'] = '';
|
||||
}
|
||||
|
||||
$vendor->update($request->all());
|
||||
|
||||
$message = trans('messages.success.updated', ['type' => trans_choice('general.vendors', 1)]);
|
||||
@@ -169,6 +177,10 @@ class Vendors extends Controller
|
||||
|
||||
public function vendor(Request $request)
|
||||
{
|
||||
if (empty($request['email'])) {
|
||||
$request['email'] = '';
|
||||
}
|
||||
|
||||
$vendor = Vendor::create($request->all());
|
||||
|
||||
return response()->json($vendor);
|
||||
|
||||
@@ -46,6 +46,10 @@ class Customers extends Controller
|
||||
public function store(Request $request)
|
||||
{
|
||||
if (empty($request->input('create_user'))) {
|
||||
if (empty($request['email'])) {
|
||||
$request['email'] = '';
|
||||
}
|
||||
|
||||
Customer::create($request->all());
|
||||
} else {
|
||||
// Check if user exist
|
||||
@@ -112,6 +116,11 @@ class Customers extends Controller
|
||||
|
||||
foreach ($rows as $row) {
|
||||
$data = $row->toArray();
|
||||
|
||||
if (empty($data['email'])) {
|
||||
$data['email'] = '';
|
||||
}
|
||||
|
||||
$data['company_id'] = session('company_id');
|
||||
|
||||
Customer::create($data);
|
||||
@@ -149,6 +158,10 @@ class Customers extends Controller
|
||||
public function update(Customer $customer, Request $request)
|
||||
{
|
||||
if (empty($request->input('create_user'))) {
|
||||
if (empty($request['email'])) {
|
||||
$request['email'] = '';
|
||||
}
|
||||
|
||||
$customer->update($request->all());
|
||||
} else {
|
||||
// Check if user exist
|
||||
@@ -220,6 +233,10 @@ class Customers extends Controller
|
||||
|
||||
public function customer(Request $request)
|
||||
{
|
||||
if (empty($request['email'])) {
|
||||
$request['email'] = '';
|
||||
}
|
||||
|
||||
$customer = Customer::create($request->all());
|
||||
|
||||
return response()->json($customer);
|
||||
|
||||
@@ -489,6 +489,10 @@ class Invoices extends Controller
|
||||
*/
|
||||
public function emailInvoice(Invoice $invoice)
|
||||
{
|
||||
if (empty($invoice->customer_email)) {
|
||||
return redirect()->back();
|
||||
}
|
||||
|
||||
$invoice = $this->prepareInvoice($invoice);
|
||||
|
||||
$logo = $this->getLogo();
|
||||
|
||||
@@ -23,6 +23,11 @@ class Vendor extends Request
|
||||
*/
|
||||
public function rules()
|
||||
{
|
||||
$email = '';
|
||||
|
||||
// Get company id
|
||||
$company_id = $this->request->get('company_id');
|
||||
|
||||
// Check if store or update
|
||||
if ($this->getMethod() == 'PATCH') {
|
||||
$id = $this->vendor->getAttribute('id');
|
||||
@@ -30,12 +35,13 @@ class Vendor extends Request
|
||||
$id = null;
|
||||
}
|
||||
|
||||
// Get company id
|
||||
$company_id = $this->request->get('company_id');
|
||||
if (!empty($this->request->get('email'))) {
|
||||
$email = 'email|unique:vendors,NULL,' . $id . ',id,company_id,' . $company_id . ',deleted_at,NULL';
|
||||
}
|
||||
|
||||
return [
|
||||
'name' => 'required|string',
|
||||
'email' => 'required|email|unique:vendors,NULL,' . $id . ',id,company_id,' . $company_id . ',deleted_at,NULL',
|
||||
'email' => $email,
|
||||
'currency_code' => 'required|string',
|
||||
];
|
||||
}
|
||||
|
||||
@@ -23,8 +23,12 @@ class Customer extends Request
|
||||
*/
|
||||
public function rules()
|
||||
{
|
||||
$email = '';
|
||||
$required = '';
|
||||
|
||||
// Get company id
|
||||
$company_id = $this->request->get('company_id');
|
||||
|
||||
// Check if store or update
|
||||
if ($this->getMethod() == 'PATCH') {
|
||||
$id = $this->customer->getAttribute('id');
|
||||
@@ -36,12 +40,13 @@ class Customer extends Request
|
||||
$required = 'required|';
|
||||
}
|
||||
|
||||
// Get company id
|
||||
$company_id = $this->request->get('company_id');
|
||||
if (!empty($this->request->get('email'))) {
|
||||
$email = 'email|unique:customers,NULL,' . $id . ',id,company_id,' . $company_id . ',deleted_at,NULL';
|
||||
}
|
||||
|
||||
return [
|
||||
'name' => 'required|string',
|
||||
'email' => 'required|email|unique:customers,NULL,' . $id . ',id,company_id,' . $company_id . ',deleted_at,NULL',
|
||||
'email' => $email,
|
||||
'currency_code' => 'required|string',
|
||||
'password' => $required . 'confirmed',
|
||||
];
|
||||
|
||||
Reference in New Issue
Block a user