SimpleSeoRouter

Ono se to musí kouknout při každém generování odkazu do databáze, jestli tomu dobře rozumim. Zajímalo by mě, jestli je to poznat na rychlosti nebo ne. Jinak se mi to hodně líbí a ty abstraktní metody se dají koneckonců naimplementovat jinak.

Já tohle řešil tak, že jsem na začátku načetl všechny stránky a to hodil do pole.

Jsem první, komu právě došlu, že ten router se dá použít i pro víceúrovňovou strukturu?

Mám ještě dva nápady.

1. Myslím, že by dobré umožnit jinou action než výchozí (adresy /clanky/nazev-clanku?action=edit).

Pak ještě jeden nápad, který možná umožní umazat Simple z názvu :-D

2. Určitě by se šikla jazyková proměnná, třeba ještě s nastavením defaultního jazyku (adresy /en/article a /nazev-clanku zároveň). Bez toho defaultního jazyka by to taky mohlo být jednoduché.

Můžete mně někdo prosím nakopnout, proč mi to dělá to přesměrování?

Když router zpracuje url, tak se ji aplikace pokusí znovu vygenerovat, aby docházelo k přesměrování na jednu variantu adresy.

Jakub Šulák napsal(a):

Vypadá to skvěle, ale nějak se mi nedaří to implementovat.

Když jsem si to zjednodušil:

require_once dirname(__FILE__).'/0.9.php';
Debug::enable();
Debug::enableProfiler();
$application = Environment::getApplication();
$router = $application->getRouter();
abstract class SimpleSeoRouter implements IRouter
...


class CategoriesRouter extends SimpleSeoRouter
{
        protected function getId($url)
        {
                //return $url=='name'?5:NULL;
                return $url=='name'?array('category'=>'5'):NULL;
        }

        protected function getPath($id)
        {
                return $id==5?'name':NULL;
        }
}

$router[] = new CategoriesRouter('katalog/', 'Default', 'default', 'cz');
$router[] = new Route('katalog',array(
       'presenter' => 'Default',
       'action' => 'default',
));


class DefaultPresenter extends Presenter {

	function renderDefault($id)
	{
		Debug::dump($id);
		$this->terminate();
	}
}





$application->run();

Tak to mě adresa http://localhost/Nette-test/katalog/name
přesměrovává na
http://localhost/Nette-test/katalog?id%5Bcategory%5D=5
Když odstraním toto:
return $url=='name'?array('category'=>'5'):NULL;
a nahradím to jen za return $url=='name'?5:NULL; Tak vše funguje zprávně.

Takže se nemůže vracet pole. (Proč ho vlastně potřebuješ vracet?)

Editoval PetrP (19. 5. 2009 14:15)

Honza M. napsal(a):

1. Myslím, že by dobré umožnit jinou action než výchozí (adresy /clanky/nazev-clanku?action=edit).

Dobrý nápad. Hotovo.

<?php
/**
 * Jednoduchý SEO-friendly router
 * ==============================
 *
 * @license     New BSD license
 * @copyright   Copyright (c) 2009 Jan Tvrdík
 * @author      Jan Tvrdík (http://merxes.cz)
 */

/**
 * Jednoduchý SEO router
 *
 * @author      Jan Tvrdík (http://merxes.cz)
 */
abstract class SimpleSeoRouter implements IRouter
{
	/** @var     string      Řetězec vkládaný před před $this->getPath() */
	protected $pathPrefix;

	/** @var     string      Jméno cílového presenteru */
	protected $presenter;

	/** @var     string      Jméno výchozí akce */
	protected $defaultAction;

	/** @var     string      Jméno parametru předeného presenteru */
	protected $identifierName = 'id';

	/** @var     string      Klíč použitý pro akci */
	protected $actionKey = 'action';

	/**
	 * Konstruktor
	 *
	 * @param    string      Řetězec vkládaný před před $this->getPath()
	 * @param    string      Jméno cílového presenteru
	 * @param    string      Jméno cílové akce
	 */
	public function __construct($pathPrefix, $presenter, $defaultAction)
	{
		$this->pathPrefix = $pathPrefix;
		$this->presenter = $presenter;
		$this->defaultAction = $defaultAction;
	}

	/**
	 * Zkusí přeložit http požadavek na PresenterRequest
	 *
	 * @param    IHttpRequest
	 * @return   PresenterRequest|NULL
	 */
	public function match(IHttpRequest $httpRequest)
	{
		$path = $httpRequest->getUri()->relativeUri;

		if (!preg_match('#^' . preg_quote($this->pathPrefix, '#') .  '(.+)$#', $path, $matches)) {
			return NULL;
		}

		$url = & $matches[1];

		$id = $this->getId($url);

		if ($id === NULL) {
			return NULL;
		}

		$params = array();
		$params += $httpRequest->getQuery();


		if (!isset($params[$this->actionKey])) {
			$params['action'] = $this->defaultAction;
		} elseif ($this->actionKey !== 'action') {
			$params['action'] = $params[$this->actionKey];
			unset($params[$this->actionKey]);
		}

		$params[$this->identifierName] = $id;

		return new PresenterRequest(
			$this->presenter,
			$httpRequest->getMethod(),
			$params,
			$httpRequest->getPost(),
			$httpRequest->getFiles(),
			array('secured' => $httpRequest->isSecured())
		);
	}

	/**
	 * Zkusí vygenerovat URL pro daný PresenterRequest
	 *
	 * @param    PresenterRequest
	 * @param    IHttpRequest
	 * @return   string|NULL
	 */
	public function constructUrl(PresenterRequest $request, IHttpRequest $context)
	{
		if ($request->getPresenterName() !== $this->presenter) {
			return NULL;
		}

		$params = $request->getParams();

		if (!isset($params[$this->identifierName])) {
			return NULL;
		}

		$path = $this->getPath($params[$this->identifierName]);

		if ($path === NULL) {
			return NULL;
		}


		if ($params['action'] == $this->defaultAction) {
			unset($params['action']);
		} elseif ($this->actionKey !== 'action') {
			$params[$this->actionKey] = $params['action'];
			unset($params['action']);
		}

		unset($params[$this->identifierName]);

		$uri = new UriScript($context->getUri()->scheme . '://' . $context->getUri()->authority . $context->getUri()->basePath);
		$uri->path .= $this->pathPrefix . $path;
		$uri->query = http_build_query($params, '', '&');

		return $uri->getAbsoluteUri();
	}

	/**
	 * Vrátí identifikátor podle fragmentu URL
	 *
	 * @param    string     Fragment URL
	 * @return   mixed|NULL
	 */
	abstract protected function getId($url);

	/**
	 * Vrátí fragment URL podle identifikátoru
	 *
	 * @param    mixed      Identifikátor
	 * @return   string     Fragment URL
	 */
	abstract protected function getPath($id);
}

2. Určitě by se šikla jazyková proměnná, třeba ještě s nastavením defaultního jazyku (adresy /en/article a /nazev-clanku zároveň). Bez toho defaultního jazyka by to taky mohlo být jednoduché.

Nechceš tuhle představu nějak rozepsat, než se to rozhodnu napsat? Jak to chceš zkombinovat se současnou proměnnou $pathPrefix? Spíš mě napadá, že máš pravdu s tím, že tenhle požadavek odstraní z názvu to Simple, protože to bude dost možná snažší řešit jako abstract class SeoRouter extends Route.

Spíš mam takovej dotaz co to vylepšuje nad použitím překladoveho slovníku?

Route::addStyle('#id');
Route::setStyleProperty('#id', Route::FILTER_IN, 'getId'); // to same co mas v GalleriesRouter
Route::setStyleProperty('#id', Route::FILTER_OUT, 'getPath');

$router[] = new Route('fotogalerie/<id #id>',array(
       'presenter' => 'Front:Galleries',
       'action' => 'view',
));

To totiž dává daleko větší možnosti, možná jsem ale SimpleSeoRouter nepochopil ;]

Hmmm jen takovým jednoduchým zásahem na řádku 207 to vrací NULL

if (($x = call_user_func($meta[self::FILTER_IN], (string) $params[$name]))===NULL)
	return NULL;
$params[$name] = $x;

To samé možná bude potřeba i u FILTER_OUT (ted mi to ale vraci url správně; mozná tam bude ještě nějaká záludnost).
Asi to neni úplně ideální ;] možná to bude mít nějaké skrýté vlastnosti.

Takže když použiju toto:

function getId($url){ return $url=='name'?5:NULL;}
function getPath($id){ return $id==5?'name':NULL;}
function getId2($url){ return $url=='xxx'?4:NULL;}
function getPath2($id){ return $id==4?'xxx':NULL;}
Route::addStyle('#id');
Route::setStyleProperty('#id', Route::FILTER_IN, 'getId');
Route::setStyleProperty('#id', Route::FILTER_OUT, 'getPath');
Route::addStyle('#id2');
Route::setStyleProperty('#id2', Route::FILTER_IN, 'getId2');
Route::setStyleProperty('#id2', Route::FILTER_OUT, 'getPath2');

$router[] = new Route('katalog/<id #id>',array(
       'presenter' => 'Default',
       'action' => 'default',
));

$router[] = new Route('katalog/<id #id2>',array(
       'presenter' => 'Default',
       'action' => 'default',
));

class DefaultPresenter extends Presenter {

	function renderDefault($id)
	{
		Debug::dump($this->link('default',array('id'=>5)));
		Debug::dump($this->link('default',array('id'=>4)));
		Debug::dump($id);
		$this->terminate();
	}
}

Na http://localhost/Nette-test/katalog/xxx vrací

string(24) "/Nette-test/katalog/name"
string(23) "/Nette-test/katalog/xxx"
int(4)

Na http://localhost/Nette-test/katalog/name

string(24) "/Nette-test/katalog/name"
string(23) "/Nette-test/katalog/xxx"
int(5)

Je to očekávané chováni?

Editoval PetrP (19. 5. 2009 18:48)

Jan Tvrdík napsal(a):

Ano, toto bude (pravděpodobně) fungovat identicky, jako SimpleSeoRouter, ale bohužel zásahy tohoto typu nejsou ideálním řešením :)

No identicky to nefunguje protože můzu mít více parametrů v jedné routě ze slovníkem.
Jakou konkrétně nevýhodu to má? Možná jsem už moc unavenej ale nic moc nevydím, nebo se slovník používá k něčemu při čem by to vadilo?

Pěkné. A můžeš mi prozradit ty konkrétní nekompability? Předpokládám že je znáš když o nich píšeš.

Pro to by možná mohl být nějaký příznak při nastavení setStyleProperty, připadně existovat jiný druh filtru. Nerozumim přesně proč by podobné chování mělo být nelogické, jen je to nepatrné rozšíření stávajících překladových slovníků (i ty ti připadají nelogické?). Vyřešila by se s tím poslední věc co routy neumí (původně jsem myslel že s tím byl slovník vyroben; ninejší chování pokládám spíš za chybu). A bylo by to mnohem použitelnější než SimpleSeoRouter (ze vší úctou).

David Grudl napsal(a):

Můžete zkusit ověřit, jestli by pomohla tato úprava v Route.php (od řádku 208)

Neprojde to podmínkou isset($meta['fixity']). Upřímně ani netuším, k čemu má $meta['fixity'] sloužit.

Díky za shrnutí významu fixity, když něco dělám po delší době s třídou Route, tak nad tím vždycky dumám a tohle se mi bude hodit ;) Což byl ostatně i tento případ, správná je tva formuluace s !isset($meta['fixity']).

Přidávám to do Route.

Ještě si říkám, že by se možná mohly výsledky z FILTER_OUT kešovat.

David Grudl napsal(a):

Přidávám to do Route.

Díky moc.

Ještě si říkám, že by se možná mohly výsledky z FILTER_OUT kešovat.

Jen FILTER_OUT tedy při generování adress? a jestli v rámci běhu scriptu tak jo. (dlouhodobější cachování at si každý napíše a invaliduje dle uvážení)

V praxi jsem ho ještě třídou Route nahradit nezkoušel, ale mělo by to bez problémů fungovat. Teď už má SimpleSeoRouter spíš smysl jako jeden z příkladů, jak může vypadat vlastní router.