Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Allow for segmented parameters as proposed in #176 #252

Closed
wants to merge 1 commit into from

Conversation

JoostK
Copy link
Contributor

@JoostK JoostK commented Feb 4, 2013

See #176 for a detailed discussion about this feature. It allows for combining multiple route parameters into one actual input parameter, which is particularly useful when more than one parameter is required, e.g. for the new Route::bind.

@taylorotwell
Copy link
Member

This proposal was never approved. I think it's a little too complicated honestly and not really necessary.

@JoostK
Copy link
Contributor Author

JoostK commented Feb 5, 2013

What I'm really hoping for is a way to combine multiple parameters into one input variable, especially for the new binding feature.

This is what is allowed by this PR. However I understand that you don't want to merge it in since the syntax and clearness is not ideal. I thought of another way which is much less complicated and easier to follow:

Route::get('users/[{firstname}/{lastname}]', function($user)
{
    // $user['firstname'] and $user['lastname']
});

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants