Název článku/postu v URL z databáze

Model AdRepository.php

<?php
namespace Model;

class AdRepository extends Repository {

	public function getAll() {
		return $this->database->table('ad');
	}

	public function getDetail($id) {
		return $this->database->table('ad')->where('idAd = ?', $id);
	}

	public function getIdByUrl() {

	}

	public function getTitleById($id) {
		return $this->database->query('SELECT title FROM ad WHERE idAd = ?', $id); //získání title
	}
}
?>

RouterFactory.php

<?php

namespace App;

use Nette,
	Nette\Application\Routers\RouteList,
	Nette\Application\Routers\Route,
	Nette\Application\Routers\SimpleRouter,
	Nette\Utils\Strings;


/**
 * Router factory.
 */
class RouterFactory
{

	public $adRepository;

	public function inject(Model\AdRepository $adRepository) {
		$this->adRepository = $adRepository;
	}

	/**
	 * @return \Nette\Application\IRouter
	 */
	public function createRouter()
	{
		$router = new RouteList();
		$router[] = new Route('<presenter>/<action>[/<id>]', 'Homepage:default');
		$router[] = new Route('<presenter>/<action>/<id>', array(
		  'presenter' => 'Homepage',
		  'action' => 'detail',
		  'id' => array(
		    'Route::VALUE' => 0, // default value
		    'Route::FILTER_OUT' => callback('adRepository::getTitleById'),
		  ),
		));
		return $router;
	}

}
?>

Nic se neděje. URL pořád ve tvaru localhost/homepage/detail/0.

jiri.pudil napsal(a):

1. Ten callback stejně vytváříš statický. Musíš ho navázat na tu instanci, kterou si injektuješ.
2. Route::VALUE a Route::FILTER_OUT jsou konstanty třídy Route. Apostrofy okolo nich nemají co dělat.

Ergo nějak takhle by to mělo fungovat:

'id' => array(
	Route::VALUE => 0,
	Route::FILTER_OUT => array($this->adRepository, 'getTitleById'),
),

A jedno doporučení na závěr: v čemkoliv, co není presenter, je nejvhodnější použít injektování přes konstruktor, tzn. metodu inject přejmenuj na __construct (a proměnná $adRepository by měla být private).

Děkuji za rychlý feedback a tipy, ale teď mi zase vyhazuje laděnka chybu.

Pokud nechám metodu inject(), tak laděnka vyhodí error.

call_user_func() expects parameter 1 to be a valid callback, first array member is not a valid class name or object

Pokud použiji metodu __construct, tak vyhodí zase jiný error.

Nette\DI\ServiceCreationException

Service ‚22_App_RouterFactory‘: No service of type App\Model\AdRepository found. Make sure the type hint in App\RouterFactory::__construct() is written correctly and service of this type is registered.

Přitom službu Model\AdRepository mám zaregistrovanou v config.neon.

EIFEL napsal(a):

Checkpoint napsal(a):

jiri.pudil napsal(a):

1. Ten callback stejně vytváříš statický. Musíš ho navázat na tu instanci, kterou si injektuješ.
2. Route::VALUE a Route::FILTER_OUT jsou konstanty třídy Route. Apostrofy okolo nich nemají co dělat.

Ergo nějak takhle by to mělo fungovat:

'id' => array(
	Route::VALUE => 0,
	Route::FILTER_OUT => array($this->adRepository, 'getTitleById'),
),

A jedno doporučení na závěr: v čemkoliv, co není presenter, je nejvhodnější použít injektování přes konstruktor, tzn. metodu inject přejmenuj na __construct (a proměnná $adRepository by měla být private).

Děkuji za rychlý feedback a tipy, ale teď mi zase vyhazuje laděnka chybu.

Pokud nechám metodu inject(), tak laděnka vyhodí error.

call_user_func() expects parameter 1 to be a valid callback, first array member is not a valid class name or object

Pokud použiji metodu __construct, tak vyhodí zase jiný error.

Nette\DI\ServiceCreationException

Service ‚22_App_RouterFactory‘: No service of type App\Model\AdRepository found. Make sure the type hint in App\RouterFactory::__construct() is written correctly and service of this type is registered.

Přitom službu Model\AdRepository mám zaregistrovanou v config.neon.

Dej si pozor na Namespace… v erroru to máš i napsaný
Router má namespace App; takže pokud napíšeš

namespace App;
__construct(Model\AdRepository $repository) // = App\Model\AdRepository

ale ty to máš mít jen Model\AdRepository takže musíš uvést \Model\AdRepository … to \ je tam důležité

Už jsem na to došel, akorát jsem chtěl upravit předchozí příspěvěk. :)
Každopádně neustále přetrvává první chybka. :X

Tak jsem teda dal na Vaši radu a injektoval jsem přes konstruktor.
Všechno funguje, ale pořád je URL ve stejném tvaru. Po názvu postu z DB ani stopy.

Checkpoint napsal(a):

Tak jsem teda dal na Vaši radu a injektoval jsem přes konstruktor.
Všechno funguje, ale pořád je URL ve stejném tvaru. Po názvu postu z DB ani stopy.

Routy se čtou v pořadí jak je tam vkládáš. To znamená, že se veme první routa která vyhovuje a další se už ignorují (v debug panelu jsou označeny: may).

Nejdřív si dej na první místo tvoje specifické routy a všeobecné dej až na konec jako poslední možnost.

Tak jsem se s tím chvíli hrál, až jsem se dostal do fáze, kdy mi laděnka vypisuje

rawurldecode() expects parameter 1 to be string, object given

Funkce rawurldecode() očekává parametr 1 jako string, ale já dodávám objekt? wtf

Je to protože tvoje repository vraci ResultSet (to co vrací query), musíš v té metodě vrátit string :) tzn. uprav metodu getTitleById()

asi tam posilas id, ktere neexistuje. dej si tam nejakou kontrolu

@**Checkpoint**: Po query() nemůžeš mít ->fetch().
Pořádně si přečti https://doc.nette.org/cs/database

Zde je screen laděnky.
https://www.dropbox.com/…r_nette_.jpg

Přikládám ještě zdrojové kódy

AdRepository.php

<?php
namespace Model;

class AdRepository extends Repository {

	public function getAll() {
		return $this->database->table('ad');
	}

	public function getDetail($id) {
		return $this->database->table('ad')->where('idAd = ?', $id);
	}

	public function getIdByUrl() {

	}

	public function getTitleById($id) {
		return $this->database->query('SELECT slug FROM ad WHERE idAd = ?', $id)->fetch()->slug;
	}
}
?>

RouterFactory.php

<?php

namespace App;
use Model;
use Nette,
	Nette\Application\Routers\RouteList,
	Nette\Application\Routers\Route,
	Nette\Application\Routers\SimpleRouter;


/**
 * Router factory.
 */
class RouterFactory
{

	private $adRepository;

	public function __construct(Model\AdRepository $adRepository) {
		$this->adRepository = $adRepository;
	}

	/**
	 * @return \Nette\Application\IRouter
	 */
	public function createRouter()
	{
		$router = new RouteList();
		$router[] = new Route('vsechny-obory/', array(
			'presenter' => 'Obor',
			'action' => 'default',
			'id' => NULL,
		));
		$router[] = new Route('vsechny-obory/obor/<id>', array(
			'presenter' => 'Obor',
			'action' => 'detail',
			'id' => NULL,
		));

		$router[] = new Route('inzerat/detail-inzeratu/<id>', array(
			'presenter' => 'Homepage',
			'action' => 'detail',
			'id' => array(
				Route::VALUE => 0,
				Route::FILTER_OUT => array($this->adRepository, 'getTitleById'),
			),
		));

		$router[] = new Route('[<presenter>/]<action>[/<id>]', 'Homepage:default');
		return $router;
	}

}
?>

Editoval Checkpoint (11. 2. 2014 11:04)

V odkazu už se ukazuje titulek, super.

Ale snaží se přejít na DetailPresenter, ne na akci ‚detail‘.

https://www.dropbox.com/…r_action.jpg

Zde je URL.
<a n:href=„Homepage:detail $posledni[‚idAd‘]“>

Už to frčí, super.
Pro zajímavost, kdyby někdo hledal, tak jsem vytvořil metodu getIdByURL().
A pomocí FILTER_IN přeložil Slug na ID.

Zde je metoda v AdRepository.php

<?php
public function getIdByUrl($slug) {
$query = $this->database->query('SELECT idAd FROM ad WHERE slug = ?', $slug);
if($row = $query->fetch()) {
	return $row->idAd;
}
    	return NULL;
}
?>

Zde je routa v RouterFactory.php

<?php
...
$router[] = new Route('inzerat/detail-inzeratu/<id>', array(
	'presenter' => 'Homepage',
	'action' => 'detail',
	'id' => array(
		Route::FILTER_IN => array($this->adRepository, 'getIdByUrl'),
		Route::FILTER_OUT => array($this->adRepository, 'getTitleById'),
	),
));

...
?>

Díky všem zůčastněným za pomoc. :)