TYPO3.Flow

Gnah, could somebody with the required permissions please fix the formatting?

The difference is that with defaults, these will be set if the value is not set at all. Therefore, all requests where no package/subpackage is set in the request will go to MyPackage\Api, which is not what I desire. I want it the other way round - only the requests to this subpackage have to match this route, no other requests. For them, I have a different URL scheme.

Andreas Wolf wrote:

I want it the other way round - only the requests to this subpackage have to match this route, no other requests.

Not sure whether I got this now, let me rephrase:
In your example you want the route to match only if the request path is "api/v1/<someController>/<someAction>" AND some other request argument (e.g. POST) is array('@package' => 'MyPackage', 'subpackage' => 'Api') ?
Or how would you specify those in your example?

Bastian Waidelich wrote:

Andreas Wolf wrote:

I want it the other way round - only the requests to this subpackage have to match this route, no other requests.

Not sure whether I got this now, let me rephrase: In your example you want the route to match only if the request path is "api/v1/<someController>/<someAction>" AND some other request argument (e.g. POST) is array('@package' => 'MyPackage', 'subpackage' => 'Api') ?

No, I want all requests to /api/v1/.* to go to the Api subpackage. But I only want links to this subpackage to have a URL /api/v1/... This does not work if I set defaults, as all view helpers etc. where the package and/or subpackage is not set will then have these defaults applied - leading to errors because the controller is not found in the API subpackage.

Andreas Wolf wrote:

In your example you want the route to match only if the request path is "api/v1/<someController>/<someAction>" AND some other request argument (e.g. POST) is array('@package' => 'MyPackage', 'subpackage' => 'Api') ?

No, I want all requests to /api/v1/.* to go to the Api subpackage. But I only want links to this subpackage to have a URL /api/v1/... This does not work if I set defaults, as all view helpers etc. where the package and/or subpackage is not set will then have these defaults applied - leading to errors because the controller is not found in the API subpackage.

Thanks for your patience, now I'm with you!
I didn't get it at first, because the behavior you describe was meant to be the default.. That is: when creating an URI you have to set all values that are defined in the Routes default (except for @format which is a bit special)..

I don't know when this changed or if it was broken form the start, but right now you can omit default values that are not part of the uriPattern..
Changing Route::compareAndRemoveMatchingDefaultValues() from

 1 protected function compareAndRemoveMatchingDefaultValues(array $defaults, array &$routeValues) {
 2     foreach ($defaults as $key => $defaultValue) {
 3         if (isset($routeValues[$key])) {
 4             if (is_array($defaultValue)) {
 5                 if (!is_array($routeValues[$key])) {
 6                     return FALSE;
 7                 }
 8                 return $this->compareAndRemoveMatchingDefaultValues($defaultValue, $routeValues[$key]);
 9             } elseif (is_array($routeValues[$key])) {
10                 return FALSE;
11             }
12             if (strtolower($routeValues[$key]) !== strtolower($defaultValue)) {
13                 return FALSE;
14             }
15             unset($routeValues[$key]);
16         }
17     }
18     return TRUE;
19 }

to

 1 protected function compareAndRemoveMatchingDefaultValues(array $defaults, array &$routeValues) {
 2     foreach ($defaults as $key => $defaultValue) {
 3         if (!isset($routeValues[$key])) {
 4             if ($defaultValue === '' || ($key === '@format' && $defaultValue === 'html')) {
 5                 continue;
 6             }
 7             return FALSE;
 8         }
 9         if (is_array($defaultValue)) {
10             if (!is_array($routeValues[$key])) {
11                 return FALSE;
12             }
13             return $this->compareAndRemoveMatchingDefaultValues($defaultValue, $routeValues[$key]);
14         } elseif (is_array($routeValues[$key])) {
15             return FALSE;
16         }
17         if (strtolower($routeValues[$key]) !== strtolower($defaultValue)) {
18             return FALSE;
19         }
20         unset($routeValues[$key]);
21     }
22     return TRUE;
23 }

fixes this for me, but it would be quite a breaking change probably..
could you please verify if that works for you?

BTW: To make debugging with fluid easier you can replace line 263 of AbstractViewHelper from throw $exception; to return $exception->getMessage(); – we'll fix this in fluid asap

BTW: With

./flow routing:list

you can see in what order the routes are processed. Try including your "MyPackageB" routes before the "MyPackageA" routes or refine the routes to make them more specific