close #464 Fixed: 0 Money payment in invoice and bill;

Transfer,
Bill, Bill Payment, Bill Total,
Payment,
Invoice, Invoice Payment, Invoice Total,
Revenue.

app/Http/Requests/Banking/Transfer.php

@@ -26,7 +26,7 @@ class Transfer extends Request
         return [
             'from_account_id' => 'required|integer',
             'to_account_id' => 'required|integer',
-            'amount' => 'required',
+            'amount' => 'required|amount',
             'transferred_at' => 'required|date',
             'payment_method' => 'required|string',
         ];

app/Http/Requests/Expense/Bill.php

@@ -39,7 +39,7 @@ class Bill extends Request
             'bill_status_code' => 'required|string',
             'billed_at' => 'required|date',
             'due_at' => 'required|date',
-            'amount' => 'required',
+            'amount' => 'required|amount',
             'currency_code' => 'required|string|currency',
             'currency_rate' => 'required',
             'vendor_id' => 'required|integer',

app/Http/Requests/Expense/BillPayment.php

@@ -27,7 +27,7 @@ class BillPayment extends Request
         return [
             'account_id' => 'required|integer',
             'paid_at' => 'required|date',
-            'amount' => 'required',
+            'amount' => 'required|amount',
             'currency_code' => 'required|string|currency',
             'payment_method' => 'required|string',
             'attachment' => 'mimes:' . setting('general.file_types', 'pdf,jpeg,jpg,png'),

app/Http/Requests/Expense/BillTotal.php

@@ -26,7 +26,7 @@ class BillTotal extends Request
         return [
             'bill_id' => 'required|integer',
             'name' => 'required|string',
-            'amount' => 'required',
+            'amount' => 'required|amount',
             'sort_order' => 'required|integer',
         ];
     }

app/Http/Requests/Expense/Payment.php

@@ -27,7 +27,7 @@ class Payment extends Request
         return [
             'account_id' => 'required|integer',
             'paid_at' => 'required|date',
-            'amount' => 'required',
+            'amount' => 'required|amount',
             'currency_code' => 'required|string|currency',
             'currency_rate' => 'required',
             'vendor_id' => 'nullable|integer',

app/Http/Requests/Income/Invoice.php

@@ -39,7 +39,7 @@ class Invoice extends Request
             'invoice_status_code' => 'required|string',
             'invoiced_at' => 'required|date',
             'due_at' => 'required|date',
-            'amount' => 'required',
+            'amount' => 'required|amount',
             'currency_code' => 'required|string|currency',
             'currency_rate' => 'required',
             'customer_id' => 'required|integer',

app/Http/Requests/Income/InvoicePayment.php

@@ -27,7 +27,7 @@ class InvoicePayment extends Request
         return [
             'account_id' => 'required|integer',
             'paid_at' => 'required|date',
-            'amount' => 'required',
+            'amount' => 'required|amount',
             'currency_code' => 'required|string|currency',
             'payment_method' => 'required|string',
             'attachment' => 'mimes:jpeg,jpg,png,pdf',

app/Http/Requests/Income/InvoiceTotal.php

@@ -26,7 +26,7 @@ class InvoiceTotal extends Request
         return [
             'invoice_id' => 'required|integer',
             'name' => 'required|string',
-            'amount' => 'required',
+            'amount' => 'required|amount',
             'sort_order' => 'required|integer',
         ];
     }

app/Http/Requests/Income/Revenue.php

@@ -27,7 +27,7 @@ class Revenue extends Request
         return [
             'account_id' => 'required|integer',
             'paid_at' => 'required|date',
-            'amount' => 'required',
+            'amount' => 'required|amount',
             'currency_code' => 'required|string|currency',
             'currency_rate' => 'required',
             'customer_id' => 'nullable|integer',

app/Providers/ValidationServiceProvider.php

@@ -18,20 +18,36 @@ class ValidationServiceProvider extends ServiceProvider
         $currency_code = null;
 
         Validator::extend('currency', function ($attribute, $value, $parameters, $validator) use(&$currency_code) {
-                $status = false;
+            $status = false;
 
-                $currencies = Currency::enabled()->pluck('name', 'code')->toArray();
+            $currencies = Currency::enabled()->pluck('name', 'code')->toArray();
 
-                if (array_key_exists($value, $currencies)) {
+            if (array_key_exists($value, $currencies)) {
-                    $status = true;
+                $status = true;
-                }
+            }
 
-                $currency_code = $value;
+            $currency_code = $value;
 
-                return $status;
+            return $status;
-            },
+        },
             trans('validation.custom.invalid_currency', ['attribute' => $currency_code])
         );
+
+        $amount = null;
+
+        Validator::extend('amount', function ($attribute, $value, $parameters, $validator) use (&$amount) {
+            $status = false;
+
+            if ($value > 0) {
+                $status = true;
+            }
+
+            $amount = $value;
+
+            return $status;
+        },
+            trans('validation.custom.invalid_amount', ['attribute' => $amount])
+        );
     }
 
     /**

resources/lang/en-GB/validation.php

@@ -102,6 +102,7 @@ return [
             'rule-name' => 'custom-message',
         ],
         'invalid_currency' => 'The :attribute code is invalid.',
+        'invalid_amount'   => 'The amount :attribute is invalid.',
     ],
 
     /*