Comments (5)
Could you create a test for this in a separate branch?
from rest.
For the both cases the route didn't match. I greated the both tests on my fork and both tests fail:
function test_optional_parameter_in_class_routes(){
$r = new Router();
$r->any('/optional/*', 'MyOptionalParamRoute');
$response = $r->dispatch('get', '/optional')->response();
$this->assertEquals('John Doe', (string) $response);
}
function test_optional_parameter_in_function_routes(){
$r = new Router();
$r->any('/optional/*', function($user=null){
return $user ?: 'John Doe';
});
$response = $r->dispatch('get', '/optional')->response();
$this->assertEquals('John Doe', (string) $response);
}
from rest.
This is actually a known issue. I've discussed this with @augustohp a couple of weeks ago. If you declare more than one parameter (like /optional/*/*
), the second and subsequent ones can be optional, but the first can't.
I'm not sure if we should drop optional parameters at all (it is easy to achieve the same result declaring more routes) or fix this. The fix may be complex 'cause this part of the implementation is very sensitive (regex translation). Documentation also needs to be improved.
I'm gonna work on this today or tomorrow depending on my time windows.
from rest.
@pablolmiranda: In the mean time, there still a way to work with this.
Register two routes to the same class/closure, like this:
- /optional
- /optional/*
This works since we already use that in some enviroments. Let us know if this works for you. ;)
from rest.
I believe this is fixed for now! @pablolmiranda I've pulled your test cases from your repo and commited them. It should be working for any route with any number of optional parameters (as long as they're at the end of the route).
I'm closing the issue, if the problem persists please let me know.
from rest.
Related Issues (20)
- Bind specific controller methods to route names HOT 2
- Composer package is outdated HOT 1
- Code 404, no matter what
- Make 404 a exception instead of a response HOT 5
- Route with direct instances and specific method of instance HOT 2
- Can I Somehow Nest Routers? HOT 2
- Allow Router as Target HOT 2
- 'sortRoutesByComplexity' is broken HOT 4
- OPTIONS requests is not handled correctly
- file_exists doesn't work? HOT 4
- Explode as an array when the post parameter is a raw HOT 1
- Fatal error: Method Respect\Rest\Router::__toString() HOT 2
- Route matching precedence HOT 1
- ->accept() IgnorableFileExtension should be configurable HOT 1
- PHP7.3: preg_replace(): Compilation failed: invalid range in character class at offset 8 HOT 2
- No updates for 4 years HOT 14
- Need to fix build error on master. HOT 3
- OPTIONS request and Access-Control-Allow-Headers HOT 4
- New Roadmap HOT 1
- Optimize error handling HOT 1
Recommend Projects
-
React
A declarative, efficient, and flexible JavaScript library for building user interfaces.
-
Vue.js
🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.
-
Typescript
TypeScript is a superset of JavaScript that compiles to clean JavaScript output.
-
TensorFlow
An Open Source Machine Learning Framework for Everyone
-
Django
The Web framework for perfectionists with deadlines.
-
Laravel
A PHP framework for web artisans
-
D3
Bring data to life with SVG, Canvas and HTML. 📊📈🎉
-
Recommend Topics
-
javascript
JavaScript (JS) is a lightweight interpreted programming language with first-class functions.
-
web
Some thing interesting about web. New door for the world.
-
server
A server is a program made to process requests and deliver data to clients.
-
Machine learning
Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.
-
Visualization
Some thing interesting about visualization, use data art
-
Game
Some thing interesting about game, make everyone happy.
Recommend Org
-
Facebook
We are working to build community through open source technology. NB: members must have two-factor auth.
-
Microsoft
Open source projects and samples from Microsoft.
-
Google
Google ❤️ Open Source for everyone.
-
Alibaba
Alibaba Open Source for everyone
-
D3
Data-Driven Documents codes.
-
Tencent
China tencent open source team.
from rest.