$presenter->isLinkCurrent() a FILTER_OUT

Ahoj, mne by pomohl zadouci kod a nutny kod, abych to mohl snadno nasimulovat. Slovni kod me odrazuje.

Mimochodem, to samé, pokud je routa definovaná bez masky:

<?php
    $router[] = new Route('<presenter>/<action>/<slug>', array(
      'presenter' => 'Homepage',
      'action' => 'default',
      'slug' => array(
        Route::VALUE => null,
        Route::FILTER_OUT => function($text) {
          return Strings::webalize($text);
        },
      )
    ));
?>

matej21 napsal(a):

budes si muset vymyslet alternativni zpusob kontroly, zda jsou linky stejne. Pri kontrole pomoci isLinkCurrent se to do routeru vubec nedostane

createRequest() jsem studoval – relevantní je spíš řádka 948. Jen jsem doufal, že mi něco uniklo.

Za mě je tohle chyba a divím se, že to dosud nikomu nevadilo.

Ano, potřebuji zvýraznit aktuální stránku.

Nepochopil jsem, co přesně navrhuješ. Mohl bys nastřelit v pseudo-kódu, jak by vypadala tahle řádka?
<a n:href=„Homepage:default $nazev“ n:class=„$presenter->linkCurrent ? funguje“>odkaz jinam</a>

deprecated addStyle: No to je paráda. Netušíš, kde bych se dočetl, jak to dělat ponovu? Dokumentace mlčí, ve fóru to nikdo zatím neprobíral. Chtělo by to zavést pravidlo, že se do Nette nesmí nic přidat, pokud autor současně nedoplní dokumentaci.

deprecated addStyle: zatim je to deprecated jen v dev (a bude asi v pristi stable). Ja jsem to nikdy nepouzil – vzdy nastavuji filtry konkretni route

jinak imho to vyresit nepujde. Filtry by teoreticky mely byt obousmerne – tedy FILTER_OUT to predela do url podoby a FILTER_IN do toho, co chce aplikacni request, potazmo presenter. tak to resim i ja – v presenterech/sablonach vsude posilam entitu a routa si z toho vezme slug/id nebo co potrebuje. Pri matchovani requestu zas dle id/slugu vybere entitu.

jak potom vybiras dle slugu, kdyz mas v databazi normalni verzi a pro where mas webalized verzi? jak resis unikatnost?

Editoval matej21 (26. 8. 2014 12:53)

ano, jsou na sobe nezavisle, ale z principu by presenter, sablony atd. nemely o konkretnich pravidlech routovani vedet (a tedy ze si routovani nejak prevadi vstupy/vystupy).

Kdyz bych tedy do constructUrl poslal nejakej request, to mi vratilo url a tu url poslal do match, mel bych dostat zpet stejny request.

Editoval matej21 (26. 8. 2014 13:02)

@petr.pavel aha, trochu prasarnicka, no :) jako mas tam problem s tim, ze url nejsou persistentni (pri zmene nazvu je to na jine url) apod.

ale kdybys pridal filter_in a vstupni slug vymenil za ne-webalizovany nazev, tak by to melo vyresit tvuj problem s isLinkCurrent

ale kdybys pridal filter_in a vstupni slug vymenil za ne-webalizovany nazev, tak by to melo vyresit tvuj problem s isLinkCurrent

Kdybych tam neměl ještě ten další parametr, který ve FILTER_OUT zpracovávám, tak bys měl pravdu.
(Parametr sezóna, který za určitých okolností NULLuji.)

Proč je „správné“ mít FILTER_IN i FILTER_OUT:
Asi jsme se nepochopili. Nepotřebuji, aby presenter něco věděl o routování. Nevidím ale důvod, proč by mělo platit, co říkáš o request1 → url → match → request2 == request1.

Kdyby ti router ve FILTER_IN vyhazoval stejny request, se kterym pracujes v presenteru, tak by nikde problem nebyl

Jednak si nejsem jistý, jestli máš pravdu, ale i kdyby: považuju to za workaround (vochcávku), ne za řešení.

Editoval petr.pavel (26. 8. 2014 13:36)

Máš recht, že to vyhledávání v PHP ve smyčce v presenteru není košér. Problém s porovnávání url chápu, máš pravdu, takhle by to nešlo.

Pojďme probrat ten druhý parametr.

<?php
    Route::addStyle('season');
    Route::setStyleProperty('season', Route::PATTERN, '(\d{4}|)');  // match a year or NULL
    Route::setStyleProperty('season', Route::FILTER_OUT, function($season) use ($seasonModel) {
      return $seasonModel->getCurrentSeason() == $season ? null : $season;
    });
    $router[] = new Route('[<season>/]events/<slug>', 'Tournament:detail');
?>

Tahle konstrukce mi zaručí, že se /2014/events/slug přesměruje na /events/slug a že parametr $season má hodnotu jen v případě, že se jedná o archiv. V tomhle použití filtru je taky nějaký problém?

Jak zajistíš to přesměrovávání, aniž bys duplikoval routy? Je jich hodně, různé presentery.

Edit: Asi bych měl přesněji formulovat otázku:
Jak doplnit FILTER_IN, aby se splnilo tvoje pravidlo o zrcadlové funkci FILTER_IN a FILTER_OUT, a současně se pro aktuální sezónu rok v url neobjevil.

Editoval petr.pavel (27. 8. 2014 0:50)

Ano. Co to je kanonizace v prezenteru? Znám jen kanonizaci v routeru.

Považuji tohle za otázku routování, takže jak jsme se shodli :-), presenter by se o to starat neměl.

přesměrování na kanonickou url:
Tak to mluvíme o tom samém. Sice to vykonává metoda presenteru ale podle definic v routeru. Jen jsem si chtěl být jistý, že @matej21 nenavrhoval něco jako if ($season == $default) $this->redirect().

Takže pořád zůstává otázka, jak řešit přesměrovávání při aktuální sezóně, aby mi fungoval $isCurrentLink a aniž bych něco musel prasit v presenterech.

Aby bylo jasno: mě to přesměrování funguje (díky nullování ve FILTER_OUT), jen nefunguje ten $isCurrentLink (tj. nastavení $current v $presenter->createRequest()).

OT: Měli bychom spíš měli říkat kanonikace (canonicalization) než kanonizace (canonization). Aby si to někdo nepletl se svatořečením.

Edit: Koukám, že pár lidí používá kanonikalizace, což dává větší smysl než kanonizace. Kdyby tak existovalo něco pěkně českého.

Editoval petr.pavel (27. 8. 2014 14:26)

@petr.pavel no i kdyz.. wiki mluvi o kanonizaci: https://cs.wikipedia.org/wiki/Kanonizace_(informatika)

Editoval matej21 (27. 8. 2014 14:41)

Škoda, že už to není otázka, ale táhlá diskuse. Teď pro nově příchozí není jasné, jak situaci řešit a budou základát nová témata.

Jestli jste k něčemu došli, mohl bys to Petře nějak zvýraznit (editovat první příspěvek/ztučnit…)?

Ano, stále nefunguje, ale aspoň už pouze kvůli slugu. Výchozí hodnota pro sezónu funguje a zdá se mi čistší než moje původní úprava ve filtru.

Těžko se mi vyhodnocuje, co je osobní zvyk, dojem programátora, a co záměr autorů frameworku. V dokumentaci se nepíše, že by musely být filtry vždy zrcadlové.
Chápu, že to působí pěkně vyrovnaně, což někoho může vést k dojmu, že to je jediný správný způsob použití. Jestli to tak ale opravdu je, to nevím.

Souhlasím, že můj způsob řešení slugu v případě, který jsem uvedl, není nejčistší. Mám ale i jiné slugy, třeba profil uživatele:

<?php
    $router[] = new Route('players/<id [0-9]+>/<firstNameSlug>-<lastNameSlug>', 'Player:profile');
?>

V presenteru vyhledávám pouze podle id, slugy tam jsou kvůli lidem a SEO. A je pohodlné vyrábět odkaz pomocí křestního a příjmení v původní podobě a nestarat se, do čeho jsou převáděny kvůli adrese.
Netvrďte mi, že „správné řešení“ ™ je zde extra položka v databázi, kterou aktualizuji při přejmenování uživatele, a FILTER_IN, který podle slugů vrací křestní a příjmení, které nepotřebuji.

V tomto případě naštěstí nepotřebuji zvýrazňovat aktuální stránku, takže debata je čistě akademická.

Editoval petr.pavel (29. 8. 2014 11:25)

Ono v těchto případech je lepší nepředávat do presenteru ani ID ani slug ale rovnou celou entitu. Tzn. celé to <id [0-9]+>/<firstNameSlug>-<lastNameSlug> by byl jeden parametr. Ve FILTER_IN by sis z toho vyříznul ty čísla na začátku a vytáhl z db entitu. Ve FILTER_OUT bys pak z té entity ty slugy snadno poskládal.

EDIT: A ano, reálně to používám jen to nedělám přes ty filtry v routeru ale trochu jinde (což není až tak podstatné).

Editoval enumag (1. 9. 2014 16:11)

petr.pavel napsal(a):

Kanonizace v tomto významu je široce používaný pojem, což ale nemusí vypovídat o jeho správnosti.
Kanonikaci nikdo nepoužívá (Google nic nenajde), pár lidí (i tady na fóru) používá kanonikalizace.
Zítra zavolám do Ústavu pro jazyk český a dám vědět.

Jinak díky, pánové, za asistenci a trpělivost.

Volal jsem, výsledek najdete v diskusi ke stránce na Wikipedii. Tímto vás tam zvu a těším se na připomínky.

Mimochodem, dřív se tu angažoval i náš David, tak můžete kráčet v jeho šlépějích.