Validating forms

[dreamy canopy]

idk whats happening while validating forms it just does not work

code

    public function store(Request $request): RedirectResponse
    {
        $request->validate([
            'name' => ['required', 'string', 'max:255'],
            'email' => ['required', 'string', 'lowercase', 'email', 'max:255', 'unique:'.User::class],
            'password' => ['required', 'confirmed', Rules\Password::defaults()],
        ]);

        $user = User::create([
            'name' => $request->name,
            'email' => $request->email,
            'password' => Hash::make($request->password),
        ]);

     event(new Registered($user));

    return redirect()->route('users');
}

this is my store function

in this part of I enter data manually in the controller it works

$user = User::create([
            'name' => $request->name,
            'email' => $request->email,
            'password' => Hash::make($request->password),
        ]);

but not working when submitting

[nocturne wolf]

What does your form look like?

[dreamy canopy]

like code or ui ?

[nocturne wolf]

The code

[dreamy canopy]

oki

[dreamy canopy]

https://codeshare.io/Q8O8nL

[nocturne wolf]

Can you try dd($request) at the top of your store function to see if it even reaches the handler

[dreamy canopy]

and web.php

<?php

use App\Http\Controllers\ProfileController;
use App\Http\Controllers\UserController;
use Illuminate\Support\Facades\Route;

Route::get('/', function () {
    return view('welcome');
});
Route::get('/users', [UserController::class, 'index'])->name('users');
Route::get('/create-user', [UserController::class, 'create']);
Route::post('/create-user', [UserController::class, 'store'])->name('new-user');

Route::get('/dashboard', function () {
    return view('dashboard');
})->middleware(['auth', 'verified'])->name('dashboard');

Route::middleware('auth')->group(function () {
    Route::get('/profile', [ProfileController::class, 'edit'])->name('profile.edit');
    Route::patch('/profile', [ProfileController::class, 'update'])->name('profile.update');
    Route::delete('/profile', [ProfileController::class, 'destroy'])->name('profile.destroy');
});

require __DIR__.'/auth.php';

[grave basin]

how are you saving? like this?

 $product = Product::create($request->validated());

[dreamy canopy]

using events

[grave basin]

$request->validated()

that part matters.. it will pass the validated array...

[dreamy canopy]

shall I create a new route ?

[nocturne wolf]

no just dd the request on your current route

see if it even reaches

[dreamy canopy]

smthng like this

?

$data = $request->validate([
            'name' => ['required', 'string', 'max:255'],
            'email' => ['required', 'string', 'lowercase', 'email', 'max:255', 'unique:'.User::class],
            'password' => ['required', 'confirmed', Rules\Password::defaults()],
        ]);
        dd($data);

[stark copper]

I'd advice you to run php artisan make:request StoreUserRequest and move your validation logic there. Then replace the Request in your store() with StoreUserRequest $request. Then calling $request->validated() will return you validation passed data.

[dreamy canopy]

oki

[dreamy canopy]

If you dont find can you say little briefly like in which function to move it rules/authorize ?

[grave basin]

rules

[dreamy canopy]

k

[stark copper]

Use:
php artisan make:request StoreUserRequest

navigate to StoreUserRequest.php in app/Http/Requests

use Illuminate\Foundation\Http\FormRequest;
use Illuminate\Validation\Rules;

class StoreUserRequest extends FormRequest
{
    public function authorize()
    {
        return true; // Modify this according to your authentication logic
    }

    public function rules()
    {
        return [
            'name' => ['required', 'string', 'max:255'],
            'email' => ['required', 'string', 'lowercase', 'email', 'max:255', 'unique:users'],
            'password' => ['required', 'confirmed', Rules\Password::defaults()],
        ];
    }
}```

Then just use it in your Controller
```use App\Http\Requests\StoreUserRequest;
use App\Models\User;
use Illuminate\Http\RedirectResponse;
use Illuminate\Support\Facades\Hash;
use Illuminate\Auth\Events\Registered;

public function store(StoreUserRequest $request): RedirectResponse
{
    $user = User::create([
        'name' => $request->name,
        'email' => $request->email,
        'password' => Hash::make($request->password),
    ]);

    event(new Registered($user));

    return redirect()->route('users');
}```

Idk what is Rules\Password::defaults() tho.

So it could throw an error.

[dreamy canopy]

k

let me try

returns error
Class "App\Http\Controllers\StoreUserRequest" does not exist

[stark copper]

Also I'd avoid to calling $request->name etc, best is to call for validated data either by using ->validated() or ->safe(), you can read more about them in Laravel docs. They'd assure you are processing trully validated data only.

[stark copper]

Did you imported it in your controller?
use App\Http\Requests\StoreUserRequest;

[dreamy canopy]

oh yeah in controller right ?

now

403
THIS ACTION IS UNAUTHORIZED.

[stark copper]

Did you modified the authorize method in request file as provided in my example?

You have to change it to return true;

[dreamy canopy]

yep

[stark copper]

Interesting, can you send a snippet?

of both the request and controller?

[dreamy canopy]

oki

req

<?php

namespace App\Http\Requests;

use Illuminate\Foundation\Http\FormRequest;

class StoreUserRequest 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', 'string', 'max:255'],
            'email' => ['required', 'string', 'lowercase', 'email', 'max:255', 'unique:'.User::class],
            'password' => ['required', 'confirmed'],
        ];
    }
}

Controller

public function store(StoreUserRequest $request): RedirectResponse
    {
        // $data = $request->validate([
        //     'name' => ['required', 'string', 'max:255'],
        //     'email' => ['required', 'string', 'lowercase', 'email', 'max:255', 'unique:'.User::class],
        //     'password' => ['required', 'confirmed', Rules\Password::defaults()],
        // ]);
        // dd($data);

        $user = User::create([
            'name' => $request->name,
            'email' => $request->email,
            'password' => Hash::make($request->password),
        ]);

     event(new Registered($user));

    return redirect()->route('users');
}

i just done some thing it fixed but returns

[stark copper]

That would be caused by the unique rule you have there.

[dreamy canopy]

there is no user with that mail

[stark copper]

No, it's a select error.

This is wrong

there should be a name of the table and a column provided if I am right. I always use Rule::unique() tho instead of this string definition.

User::class returns "App\Http\Requests\User" which should be "App\Models\User" but that doesn't matter right now. You have to put your users table there.

Seems 'unique:users' should be sufficient.

[dreamy canopy]

oki

[grave basin]

the controller also won't work as posted..

[dreamy canopy]

another err occurred while getting all users

Class "App\Http\Controllers\DB" not found

but controller contains

use App\Http\Controllers\DB;

[grave basin]

@stark copper mentioned you need to use $request->validated() or safe() then will work also in the docs on user you 'may' use the model directly if you want
'email' => 'unique:App\Models\User,email_address'

what is that? Controllers\DB?

[dreamy canopy]

is it Illuminate/Support/Facades/DB ?

[stark copper]

Oh, you can provide class there? Interesting. Well, I always used the Rule facade for this to avoid writing string like this. 😄

[dreamy canopy]

ignore slash

[stark copper]

There is not DB class in Controllers

What program are you using?

[dreamy canopy]

sorry ?

[grave basin]

if you're using that but in your routes you have Route::get('/users', [UserController::class, 'index'])->name('users');

I bet you have that Facade in the usersController but you don't need it really

$users = Users::all();

[grave basin]

Yeah i also prefer that.. just another option I guess..

[dreamy canopy]

still the issue is not fixed of creating user

like validating form

@stark copper please help I'm try to do this since 2days

.

Ohhhhhhh

Finlly fixed it

the isssue was coming here 'password' => ['required', 'confirmed'],

idh any confirm field

so i removed it it works

Thnx guys

[dreamy canopy]

for that I get this error
Class "App\Http\Controllers\Users" not found

So I was just using DB::select()

[grave basin]

Exactly - when you type Users normally you're IDE will give you the option to import the model - otherwise just import the model

use App\Models\User;

Imported in the controller?

[dreamy canopy]

model is imported

ig

use App\Models\User; right?

yes

[grave basin]

correct then if you dd($users) you should have them..

[dreamy canopy]

my index function

public function index()
    {
        $data = Users::all();
        return view('crud.users', compact('data'));
    }

[grave basin]

hmm and still not found?

[dreamy canopy]

i dont get u

[dreamy canopy]

where is the issue ?

[grave basin]

I mean after you refresh and go again you still get the error Class "App\Http\Controllers\Users" not found

[dreamy canopy]

ohk

[stark copper]

Sorry I was away, does it wark? 😄

[dreamy canopy]

let me check

[grave basin]

i'm lost now which problem you still have 🙂

[dreamy canopy]

ye

[dreamy canopy]

getting users with $users = Users::all()

[stark copper]

You need confirmed, it's for the "second" try for a password so user won't mistype it.

Basically in your form you need to have password_confirmation field

[dreamy canopy]

Like to create user from admin side

[dreamy canopy]

when user create acc noramlly it is available any ways i am using breeze

[dreamy canopy]

Vel bro help me out with this

[dreamy canopy]

still same

[grave basin]

have you cached anything (please don't if you haven't)

[dreamy canopy]

nah

[stark copper]

I think it would be best for you to sort of open LaraCast and run through a base courses so you'd understand how it works.

[dreamy canopy]

oki

[stark copper]

Because I think you are missing a lot of knowledge you are supposed to have trying to achieve this.

Don't get me wrong, it could take you only a few hours of courses, so you'd be able to program this even today. If you'll not rush it.

[dreamy canopy]

I mostly work with pterodactyl(run on laravel) so few time I need to fixes few issues by my self so I'm learning

[dreamy canopy]

thats ok bro np

[dreamy canopy]

anyways can you help with this last thing

[stark copper]

I started with Laracasts a long time ago and it helped me understand a lot of things.

And what is the "last" issue again?

[dreamy canopy]

this one 👆

[stark copper]

You should get users with User::get()

[dreamy canopy]

ohk

Just a question

I am fimilar with react so should I use react or blade ?

[stark copper]

React is a javascript framework, blade is just html templating. I mean, you can use whatever suits you and needs of your project best.

[dreamy canopy]

Finally done

Thnx bro

[dreamy canopy]

oki

[grave basin]

hang on are you saying it worked with ::get() and not all() ?

[dreamy canopy]

actually bros

[grave basin]

I'm using ::all() all over the place in many many projects and files with no issues..

[dreamy canopy]

the issue was Users it should be User

[grave basin]

also a tip - if you want to use react you're best when you start a new project installing the breeze/react scaffold

[dreamy canopy]

oki

[grave basin]

ahhhh - really sorry that was a Typo from me 🤦‍♂️

[dreamy canopy]

shall I go with this
https://laracasts.com/series/30-days-to-learn-laravel-11

[dreamy canopy]

aww np

[dreamy canopy]

i just saw few laravel+react templates, they use react-dom in which I'm really bad

[stark copper]

doesn't all return an array?

Instead of elloquent collection?

[dreamy canopy]

Illuminate\Database\Eloquent\Collection {#1022 ▼ // app\Http\Controllers\UserController.php:25
  #items: array:3 [▼
    0 => 
App\Models
\
User {#1288 ▶}
    1 => 
App\Models
\
User {#1289 ▶}
    2 => 
App\Models
\
User {#1290 ▶}
  ]
  #escapeWhenCastingToString: false
}

dd() works with any type of route ?

[grave basin]

dd stands for dump and die so you can use it in your backend to help with debugging problems - @stark copper I did a quick test on all() vs get() and in both cases there is a collection of models returned..

[dreamy canopy]

ig this is what it returns

Thnx guys bye ❤️

[grave basin]

correct but the question was about using get() vs all() and it turns out both methods return the same in this case..

@dreamy canopy good luck - don't forget to mark as solved 🙂

[dreamy canopy]

thnx

done

[stark copper]

So what is the difference then? o.O

[grave basin]

I can't really find one - i 'suspect' all() is just a shortcut, meaning if you use model::where(something) then you need to use get() or first() in that case get() still returns a collection and first() returns the model object, I learned something new today though, i think for consistency using get() with no args can be a bit cleaner, unless I miss something...interestingly all() has a dock block but my IDE (vsCode) doesn't pick that up for get()

[dreamy canopy]

@grave basin Hello

I just have a confusion

How does the StoreUserRequest know to only captue data of a single route which we haven't defined in StoreUserRequest.php

[stark copper]

It doesn’t unless you tell it to. You can reuse on multiple routes.

[dreamy canopy]

How do I tell or define it