Start/Stop Parking

Hi Povilas, question regarding your code under start method of ParkingController:

$parking->load('vehicle', 'zone');

Is it still necessary to use the load function to eager load vehicle and zone? i believe there wont be any N+1 problem knowing that there will only be one vehicle and one zone per parking.

In start method, when we fire this endpoint more one time it will store duplicate records at same parking and zone, so I suggest set a validation when same parking exists if stop time null.

I think you put the wrong screenshot "We launch this endpoint with success!" (after this sentence)

you forgot to add "id" in ParkingResource.php

return [ 'id' => $this->id, ...

Is there difference if we register an observer in AppServiceProvider rather than EventServiceProvider (as in Laravel docs) like you did for Vehicle and Parking observers?

Regarding

Route::get('parkings/{parking}', [ParkingController::class, 'show']);
Route::put('parkings/{parking}', [ParkingController::class, 'stop']);

Would it be better it we add these constraints for these routes?? Like this

Route::get('parkings/{parking}', [ParkingController::class, 'show'])->whereNumber('parking')
Route::put('parkings/{parking}', [ParkingController::class, 'stop'])->whereNumber('parking')

hi, After creating this : 'exists:vehicles,id,deleted_at,NULL,user_id,'.auth()->id(), for validation Rule, when runing postman , I have this issue: Column not found: 1054 Unknown column 'deleted_at' in 'where clause' (SQL: select count(*) as aggregate from `vehicles`
I need to create deleted_at colum in the parking migration ?

Hi Povilas, I think 'stop_time' => $this->stop_time?->ToDateTimeString(), the ? dont fixe my problem about parse null toDateTime.. this is what postman said: "message": "Call to a member function ToDateTimeString() on null",

but it work on database!

It looks like you're using the local scope Parking::active() when checking if there's an active parking in the controller but it's not been added to the model in any previous steps.

Hi Povilas, The syntax

'exists:vehicles,id,deleted_at,NULL,user_id,'.auth()->id(),

applied in ParkingController.php validation calls my attention and i need some explanation.

Why can I use so many parameters separated by commas?, in the documentation is not clear to me.

I have edit the

if (Parking::active()->where('vehicle_id', $request->vehicle_id)->exists())

to

if (Parking::active()->where('vehicle_id', $request->vehicle_id)->whereNull('stop_time')->exists())

to allow the user to park with the same vehicle more than once only after checking out.

Why do we are checking for authentication in the observers if they're only executed on context where there's always an authenticated user? (Parking creation only happens in this context: accessing the endpoint '/parking/start' unauthenticated results in authentication error).