remove unique on update #214

app/Http/Requests/CreateAirportRequest.php

@@ -5,18 +5,29 @@ namespace App\Http\Requests;
 use App\Models\Airport;
 use Illuminate\Foundation\Http\FormRequest;
 
+/**
+ * Class CreateAirportRequest
+ * @package App\Http\Requests
+ */
 class CreateAirportRequest extends FormRequest
 {
-    public function authorize()
+    /**
+     * Determine if the user is authorized to make this request.
+     * @return bool
+     */
+    public function authorize(): bool
     {
         return true;
     }
 
-    public function rules()
+    /**
+     * Get the validation rules that apply to the request.
+     * @return array
+     */
+    public function rules(): array
     {
         $rules = Airport::$rules;
         $rules['icao'] .= '|unique:airports';
-
         return $rules;
     }
 }

app/Http/Requests/CreateFareRequest.php

@@ -9,21 +9,21 @@ class CreateFareRequest extends FormRequest
 {
     /**
      * Determine if the user is authorized to make this request.
-     *
      * @return bool
      */
-    public function authorize()
+    public function authorize(): bool
     {
         return true;
     }
 
     /**
      * Get the validation rules that apply to the request.
-     *
      * @return array
      */
-    public function rules()
+    public function rules(): array
     {
-        return Fare::$rules;
+        $rules = Fare::$rules;
+        $rules['code'] .= '|unique:fares';
+        return $rules;
     }
 }

app/Http/Requests/CreateSubfleetRequest.php

@@ -9,21 +9,21 @@ class CreateSubfleetRequest extends FormRequest
 {
     /**
      * Determine if the user is authorized to make this request.
-     *
      * @return bool
      */
-    public function authorize()
+    public function authorize(): bool
     {
         return true;
     }
 
     /**
      * Get the validation rules that apply to the request.
-     *
      * @return array
      */
-    public function rules()
+    public function rules(): array
     {
-        return Subfleet::$rules;
+        $rules = Subfleet::$rules;
+        $rules['type'] .= '|unique:subfleets';
+        return $rules;
     }
 }

app/Http/Requests/CreateUserRequest.php

@@ -9,24 +9,21 @@ class CreateUserRequest extends FormRequest
 {
     /**
      * Determine if the user is authorized to make this request.
-     *
      * @return bool
      */
-    public function authorize()
+    public function authorize(): bool
     {
         return true;
     }
 
     /**
      * Get the validation rules that apply to the request.
-     *
      * @return array
      */
-    public function rules()
+    public function rules(): array
     {
         $rules = User::$rules;
         $rules['email'] .= '|unique:users,email';
-
         return $rules;
     }
 }

app/Models/Fare.php

@@ -33,7 +33,7 @@ class Fare extends Model
     ];
 
     public static $rules = [
-        'code' => 'required|unique:fares',
+        'code' => 'required',
         'name' => 'required',
     ];
 

app/Models/Subfleet.php

@@ -40,7 +40,7 @@ class Subfleet extends Model
     ];
 
     public static $rules = [
-        'type'                       => 'required|unique:subfleets',
+        'type'                       => 'required',
         'name'                       => 'required',
         'ground_handling_multiplier' => 'nullable|numeric',
     ];
@@ -50,9 +50,7 @@ class Subfleet extends Model
      */
     public function setTypeAttribute($type)
     {
-        $type = str_replace(' ', '-', $type);
-        $type = str_replace(',', '', $type);
-
+        $type = str_replace([' ', ','], array('-', ''), $type);
         $this->attributes['type'] = $type;
     }