Skip to content
This repository was archived by the owner on Feb 7, 2025. It is now read-only.

In larave 6.6 null parameter is not accepted. #4593

Closed

Conversation

developerdesigner18
Copy link
Contributor

In laravel 6.6 null parameter cleare issue and throw error in /admin/database "Missing required parameters for [Route: voyager.bread.delete] [URI: admin/bread/{id}]" So i change null to '' blank and it works fine now.

In laravel 6.6 null parameter cleare issue and throw error in /admin/database "Missing required parameters for [Route: voyager.bread.delete] [URI: admin/bread/{id}]" So i change null to '' blank and it works fine now.
@MrCrayon
Copy link
Collaborator

As far as I understand it if {id} is required building route it should not be empty.
That's exactly the bug they tried to fix in 6.6 with laravel/framework#30659
So if this is going to be fixed, as I think, empty string is not going to work.

And there is already another PR open #4587

@developerdesigner18
Copy link
Contributor Author

developerdesigner18 commented Nov 29, 2019

Yes but voyager pass null which is not correct too. Passing '' blank works instead on null.

image

@MrCrayon
Copy link
Collaborator

I'm aware, that's why:

And there is already another PR open #4587

@MrCrayon MrCrayon closed this Dec 3, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants