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

$this->form->create() pass un-want parameters to Router::match() #321

Closed
Leechael opened this issue Feb 9, 2012 · 3 comments
Closed

$this->form->create() pass un-want parameters to Router::match() #321

Leechael opened this issue Feb 9, 2012 · 3 comments

Comments

@Leechael
Copy link

Leechael commented Feb 9, 2012

recently I create a library with form, I configured route rules like:

Router::connect('/foo/bar', array('controller' => 'foo.Foo', 'action' => 'bar'));

In target template, I call:

$this->form->create()

Then I got 'lithium\net\http\RoutingException' with message 'No parameter match found for URL xxxx'.

The problem cause by pass $request->params to Router::match(), and $request->params have extra field named 'library', which undeclared in route rule configuration.

Possible solution:

When compiling route rule, append library field to Route::$_match or do more check?

@Ciaro
Copy link
Contributor

Ciaro commented Mar 17, 2012

O lol, I missed this one, I posted the same issue (#374) + a 'rough' fix: Ciaro@e20f876

@Leechael
Copy link
Author

@Ciaro Had you sent a pull request to li3 guys?

@Leechael
Copy link
Author

OK, closed this commit for duplicated: #374

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

No branches or pull requests

2 participants