Complete all tests

.github/workflows/laravel.yml

@@ -13,7 +13,7 @@ jobs:
     steps:
       - uses: shivammathur/setup-php@15c43e89cdef867065b0213be354c2841860869e
         with:
-          php-version: "8.1"
+          php-version: "8.4"
       - uses: actions/checkout@v2
       - name: Copy .env
         run: php -r "file_exists('.env') || copy('.env.example', '.env');"

app/Http/Controllers/ArticleController.php

@@ -13,7 +13,7 @@ public function store(Request $request)
         $request->validate([
             // TASK: create your own validation rule called Uppercase
             // It should check whether title's first letter is uppercase
-            'title' => ['required', new Uppercase()]
+            'title' => ['required', new Uppercase]
         ]);
 
         Article::create(['title' => $request->title]);

app/Http/Controllers/BuildingController.php

@@ -17,7 +17,7 @@ public function create()
     // TASK: Customize the validation error message to say "Please enter the name"
     public function store(StoreBuildingRequest $request)
     {
-        Building::create($validator->validated());
+        Building::create($request->validated());
 
         return 'Success';
     }

app/Http/Controllers/PostController.php

@@ -9,10 +9,9 @@ class PostController extends Controller
 {
     public function store(Request $request)
     {
-        $request->validate(
-            // ... TASK: write validation here so that "title" field
-            //           would be required and unique in the "posts" DB table
-        );
+        $request->validate([
+            'title' => 'string|required|unique:posts'
+        ]);
 
         // Saving the post
         Post::create(['title' => $request->title]);

app/Http/Controllers/ProfileController.php

@@ -9,10 +9,8 @@ class ProfileController extends Controller
     public function update(Request $request)
     {
         $request->validate([
-            // TASK: imagine that in the Blade the fields are
-            // <input name="profile[name]" ... />
-            // <input name="profile[email]" ... />
-            // Write validation rules, so both name and email are required
+            'profile.name' => 'required',
+            'profile.email' => 'required'
         ]);
 
         auth()->user()->update($request->profile ?? []);

app/Http/Controllers/UserController.php

@@ -11,7 +11,7 @@ public function update(User $user, UpdateUserRequest $request)
     {
         // TASK: change this line to not allow is_admin field to be updated
         // Update only the fields that are validated in UpdateUserRequest
-        $user->update($request->all());
+        $user->update($request->validated());
 
         return 'Success';
     }

app/Http/Requests/StoreBuildingRequest.php

@@ -30,4 +30,11 @@ public function rules()
             'name' => 'required'
         ];
     }
+
+    public function messages()
+    {
+        return [
+            'name.required' => 'Please enter the name'
+        ];
+    }
 }

app/Http/Requests/StoreItemRequest.php

@@ -0,0 +1,29 @@
+<?php
+
+namespace App\Http\Requests;
+
+use Illuminate\Foundation\Http\FormRequest;
+
+class StoreItemRequest extends FormRequest
+{
+    /**
+     * Determine if the user is authorized to make this request.
+     */
+    public function authorize(): bool
+    {
+        return true;
+    }
+
+    /**
+     * Get the validation rules that apply to the request.
+     *
+     * @return array<string, \Illuminate\Contracts\Validation\ValidationRule|array<mixed>|string>
+     */
+    public function rules(): array
+    {
+        return [
+            'name' => 'required|max:255',
+            'description' => 'string'
+        ];
+    }
+}

app/Rules/Uppercase.php

@@ -0,0 +1,21 @@
+<?php
+
+namespace App\Rules;
+
+use Closure;
+use Illuminate\Contracts\Validation\ValidationRule;
+
+class Uppercase implements ValidationRule
+{
+    /**
+     * Run the validation rule.
+     *
+     * @param  \Closure(string): \Illuminate\Translation\PotentiallyTranslatedString  $fail
+     */
+    public function validate(string $attribute, mixed $value, Closure $fail): void
+    {
+        if (ucfirst($value) !== $value) {
+            $fail('The title does not start with an uppercased letter');
+        }
+    }
+}