[NFR] Name property for Phalcon\Mvc\Router\Group

[moderndeveloperllc]

Allow users to setName() on a group as is done for individual routes. Individual routes should have a groupName attribute that contains this, or have a getGroup() function on Phalcon\Mvc\Router\Route that returns the Phalcon\Mvc\Router\Group object.

This would be useful when doing a getMatchedRoute() to be able to code logic based off of the group the matched route belongs to. I am currently writing a Locale class that does URL rewriting. All of my locale-aware routes are in one group and I want to check if the incoming URL already matches one of those routes our not.

Current Workaround: I'm using setName() on all the routes in the group to name them the same thing.

[ghost]

Implemented in 1.3.0, could you please take a look?

[moderndeveloperllc]

@sjinks Odd, but I'm getting Fatal error: Call to undefined method Phalcon\Mvc\Router\Route::getGroup() even though I've built out the 1.3.0 branch. I also can't see a _group attribute on the Phalcon\Mvc\Router\Route object.

1.3.0 BETA 1
Phalcon\Mvc\Router\Route Object
(
    [_pattern:protected] => /{locale:[a-z]{2,3}([\_\-][[:alnum:]]{1,8})?}/:controller/:action/:params
    [_compiledPattern:protected] => #^/[a-z]{2,3}([\_\-][[:alnum:]]{1,8})?/([a-zA-Z0-9_-]++)/([a-zA-Z0-9_-]++)(/.*+)?+$#
    [_paths:protected] => Array
        (
            [controller] => 2
            [action] => 3
            [params] => 4
            [locale] => 1
        )

    [_methods:protected] => 
    [_hostname:protected] => 
    [_converters:protected] => 
    [_id:protected] => 7
    [_name:protected] => LocaleAwareRoute
    [_beforeMatch:protected] => 
)

[ghost]

Did you build 1.3.0 from build/ or from ext/?

[moderndeveloperllc]

build/ Wrong place? :-)

[moderndeveloperllc]

my bad - forgot to run gen-build.php. I now have the group object, but there is no way to identify if it's the right group. The NFR was also to get a setName()/getName() on the Group object.

[moderndeveloperllc]

@sjinks Should I close this out? I was really hoping that Phalcon\Mvc\Router\Group would get a getName()/setName()