S klasickym Route nefunguje FILTER_TABLE

flame-org / Modules

Nette modules on the Steroids

http://flame-org.github.io/Modules/

GNU Lesser General Public License v3.0

13 stars 7 forks source link

S klasickym Route nefunguje FILTER_TABLE #17

Closed jirinapravnik closed 9 years ago

jirinapravnik commented 10 years ago

Pokud použiju RouteMock v definici extension jako tohle

$frontList[] = new NetteRouteMock('//' . $serverMainUrl . '/<articleUrl [a-z\-]+/[0-9]+-[a-z0-9\-]+>/forum[/podle-<view>]', [
    'presenter' => 'Discussion',
    'action' => 'article',
    'view' => [
        Route::FILTER_TABLE => [
            'casu' => 'time',
            'vlaken' => 'thread',
        ],
        Route::FILTER_STRICT => TRUE,
    ]
);

tak bez problemu funguje jak očekáváme.

Pokud ovšem udělám:

$frontList[] = new Route('//' . $serverMainUrl . '/<articleUrl [a-z\-]+/[0-9]+-[a-z0-9\-]+>/forum[/podle-<view>]', [
    'presenter' => 'Discussion',
    'action' => 'article',
    'view' => [
        Route::FILTER_TABLE => [
                'casu' => 'time',
            'vlaken' => 'thread',
        ],
        Route::FILTER_STRICT => TRUE,
           ]
    ]
);

tak se uplne ignoruje to co jsme nastavil pro parametr. Problem je nejspise v tomhle: https://github.com/flame-org/Modules/blob/master/Flame/Modules/DI/ModulesExtension.php#L252 getDefaults() totiz nevraci vsechna metadata

TomasVotruba commented 10 years ago

Píšeš "tak se uplne ignoruje to co jsme nastavil pro parametr". Jaký parametr máš na mysli?

Toto je spíše Nette issue. Co je potřeba upravit v Route, aby fungovalo dle očekávání?

jirinapravnik commented 10 years ago

Asi by mělo stačit jen upravit getDefaults(), ci nejak jinak umoznit vytahnout metadata, protože tady se sahá na getDefaults() a to vrací jen nějak osekané parametry, třeba v mém případě vůbec nic a když mrknu do DIC vygenerované, tak mam u view => NULL

TomasVotruba commented 10 years ago

Nevidím část textu za jen: "Asi by mělo stačit jen , protože". Můžeš to upravit prosím?

Popiš prosím bug a návrh řešení v issues k Nette\Routing

jirinapravnik commented 10 years ago

Doplnil jsem text.

No ja nevim hlavne jestli je to bug, treba nechteji schvalne vyhazovat vsechna metadata ven a taky me navrh reseni nenapada, krom klasickeho getteru:-)

TomasVotruba commented 10 years ago

Otevři issue a uvidíme, jak se k tomu ostatní vyjádří.