diff --git a/resources/views/invoices/edit.blade.php b/resources/views/invoices/edit.blade.php
index 5824f4a24ea5..5458dfc4436b 100644
--- a/resources/views/invoices/edit.blade.php
+++ b/resources/views/invoices/edit.blade.php
@@ -38,6 +38,7 @@
->method($method)
->addClass('warn-on-exit')
->autocomplete('off')
+ ->onsubmit('return onFormSubmit(event)')
->rules(array(
'client' => 'required',
'invoice_number' => 'required',
@@ -52,7 +53,7 @@
-
+
@if ($invoice->id || $data)
@@ -1055,13 +1056,13 @@
}
function onEmailClick() {
- if (!isEmailValid()) {
- alert("{!! trans('texts.provide_email') !!}");
+ if (!NINJA.isRegistered) {
+ alert("{!! trans('texts.registration_required') !!}");
return;
}
- if (!NINJA.isRegistered) {
- alert("{!! trans('texts.registration_required') !!}");
+ if (!isEmailValid()) {
+ alert("{!! trans('texts.provide_email') !!}");
return;
}
@@ -1107,26 +1108,26 @@
}
function submitAction(value) {
- if (!isSaveValid()) {
- model.showClientForm();
- return;
- }
+ $('#action').val(value);
+ $('#submitButton').click();
+ }
+
+ function onFormSubmit(event) {
+ if (!isSaveValid()) {
+ model.showClientForm();
+ return false;
+ }
// check currency matches for expenses
var expenseCurrencyId = model.expense_currency_id();
var clientCurrencyId = model.invoice().client().currency_id() || {{ $account->getCurrencyId() }};
if (expenseCurrencyId && expenseCurrencyId != clientCurrencyId) {
alert("{!! trans('texts.expense_error_mismatch_currencies') !!}");
- return;
+ return false;
}
onPartialChange(true);
- $('#action').val(value);
- $('#submitButton').click();
- }
-
- function onSubmit() {
- console.log('submit');
+
return true;
}