Tax number on clients is no longer unique and required.

Author: Mario Basic

app/Category.php

@@ -7,6 +7,11 @@
 
 class Category extends Model
 {
+    /**
+     * The attributes that are mass assignable.
+     *
+     * @var array
+     */
     protected $fillable = ['name'];
 
     /**

app/Client.php

@@ -7,8 +7,18 @@
 
 class Client extends Model
 {
+    /**
+     * The attributes that are mass assignable.
+     *
+     * @var array
+     */
     protected $fillable = ['name', 'tax_number', 'street', 'city', 'postal_code'];
 
+    /**
+     * Get services which belong to this client.
+     *
+     * @return [type]
+     */
     public function services()
     {
         return $this->hasMany(Service::class);

app/Http/Controllers/ClientController.php

@@ -39,8 +39,8 @@ public function create()
     public function store(Request $request)
     {
         $this->validate($request, [
-            'name' => 'required|max:255',
-            'tax_number' => 'required|max:255|unique:clients',
+            'name' => 'required|string|max:255',
+            'tax_number' => 'string|max:255|unique:clients',
             'street' => 'string|max:255',
             'city' => 'string|max:255',
             'postal_code' => 'integer',
@@ -84,8 +84,8 @@ public function update(Request $request, $id)
         $client = Client::findOrFail($id);
 
         $this->validate($request, [
-            'name' => 'required|max:255',
-            'tax_number' => 'required|max:255|unique:clients,tax_number,' . $client->id,
+            'name' => 'required|string|max:255',
+            'tax_number' => 'string|max:255|unique:clients,tax_number,' . $client->id,
             'street' => 'string|max:255',
             'city' => 'string|max:255',
             'postal_code' => 'integer',

app/Occurrence.php

@@ -7,25 +7,58 @@
 
 class Occurrence extends Model
 {
+    /**
+     * The attributes that are mass assignable.
+     *
+     * @var array
+     */
     protected $fillable = ['occurs_at', 'offer_sent', 'payment_received', 'receipt_sent'];
 
+    /**
+     * The attributes that should be mutated to dates.
+     *
+     * @var array
+     */
     protected $dates = ['occurs_at'];
 
+    /**
+     * Gets a service that belongs to this occurrence.
+     *
+     * @return [type]
+     */
     public function service()
     {
         return $this->belongsTo(Service::class);
     }
 
+    /**
+     * If the offer_sent is set to true, this
+     * returns false.
+     *
+     * @return [type]
+     */
     public function getFutureOfferState()
     {
         return $this->offer_sent ? 0 : 1;
     }
 
+    /**
+     * If the payment_received is set to true
+     * this returns false.
+     *
+     * @return [type]
+     */
     public function getFuturePaymentState()
     {
         return $this->payment_received ? 0 : 1;
     }
 
+    /**
+     * If the receipt_sent is set to true
+     * this returns false.
+     *
+     * @return [type]
+     */
     public function getFutureReceiptState()
     {
         return $this->receipt_sent ? 0 : 1;

app/Service.php

@@ -13,6 +13,11 @@ class Service extends Model
 {
     use FormAccessible;
 
+    /**
+     * The attributes that are mass assignable.
+     *
+     * @var array
+     */
     protected $fillable = ['title', 'note', 'month', 'day', 'cost', 'currency', 'active', 'exchange_rate'];
 
     /**
@@ -68,6 +73,16 @@ public function getFormattedCostAttribute()
         return number_format($this->cost / 100, 2, ',', '.') . ' ' . strtoupper($this->currency);
     }
 
+    /**
+     * Gets the sum of all services, converted to user's
+     * preferred currency with current exchange rate.
+     *
+     * It is possible to pass a collection of services to this method,
+     * and it will return the sum of that collection.
+     *
+     * @param  Collection|null $services
+     * @return [type]
+     */
     public function getSum(Collection $services = null)
     {
         if(!($services instanceof Collection)) {
@@ -87,6 +102,16 @@ public function getSum(Collection $services = null)
         return number_format($sum, 2, ',', '.') . ' ' . $preferredCurrency;
     }
 
+    /**
+     * Gets the sum of all services, converted to user's
+     * preferred currency with current exchange rate for the entered month.
+     *
+     * By default, all services are included (active and non active).
+     *
+     * @param  int $month
+     * @param  boolean $onlyActive
+     * @return [type]
+     */
     public function getSumForMonth(int $month, $onlyActive = false)
     {
         $services = $this->where('month', $month);

composer.json

@@ -20,7 +20,8 @@
         "laravelcollective/html": "^5.2",
         "laravelista/ekko": "^1.2",
         "laracasts/flash": "^2.0",
-        "florianv/laravel-swap": "^0.4.0"
+        "florianv/laravel-swap": "^0.4.0",
+        "doctrine/dbal": "^2.5"
     },
     "require-dev": {
         "fzaninotto/faker": "~1.4",