Fix: Add complete validation rules for internal driver creation #199
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
- Add required validation for name, email, and phone fields - Add validation for drivers_license_number and internal_id - Add validation for photo_uuid and avatar_uuid - Add validation for location fields (latitude, longitude) - Add custom error messages for better UX - Add custom attribute names for clearer validation errors - Ensure email and phone uniqueness checks - Add password minimum length validation (8 characters) This fixes the issue where drivers could be created without required fields through the internal API, and improves the validation error messages shown to users. Resolves issue where onboarding driver creation was failing silently due to incomplete validation rules.
- Check if user exists before calling methods on it - Create new user if user_uuid is provided but user doesn't exist - Prevents 'Call to a member function companies() on null' error - Fixes driver creation failure when invalid user_uuid is provided This resolves the error: Error: Call to a member function companies() on null in DriverController.php on line 172
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Problem
The internal was incomplete and missing validation for required fields like , , and . This caused two issues:
The controller extracts data from
$request->input('driver')but the validation rules didn't properly validate this nested data.Solution
Updated
server/src/Http/Requests/Internal/CreateDriverRequest.phpwith:Required Fields (for creation)
name- Driver name (required, max 255 chars)email- Email address (required, must be valid email, unique)phone- Phone number (required, unique)Optional Fields
password- Minimum 8 charactersdrivers_license_number- Driver's license numberinternal_id- Internal identifiercountry- 2-letter country codecity- City namevehicle- Vehicle assignmentstatus- active/inactivevendor- Vendor assignmentjob- Order assignmentlocation,latitude,longitude- Location dataphoto_uuid,avatar_uuid- Photo uploadsImprovements
Testing
Impact
Related Issues
Fixes driver creation failures in onboarding flow where users could submit empty forms.