Ajaxový odkaz smerujúci na inú action

Moc nerozumím tomu, kde Ti šablona vyhazuje tu chybu, resp. kde přistupuješ k $article->title.
Nebylo by možná lepší udělat AJAXový subrequest, tedy místo actionDelete($id) udělat

public function handleDeleteComment($id)
{
	$this->blogComments->delete((int) $id);
    $this->flashMessage('Komentár bol zmazaný.');

	if ($this->isAjax()) {
		$this->redrawControl('flash');
    	$this->redrawControl('comments');
	}
}

a potom vytvářet odkaz jako n:href="deleteComment! $id"?

EDIT: koukni sem. Možná by se v kapitole o AJAXu mohlo o subrequestech povědět víc, není na první pohled patrné, jak se takovéhle akce mají správně dělat.

Editoval FilipKlimeš (11. 2. 2015 12:25)

Okay, a já přepíšu dokumentaci :)

Akadlec
No a na to slúži dokumetácia, aby to tam našiel človek vysvetlené. V documentácii sú signály spojené s komponentami a preto by bolo dobré ich spomenúť aj pri ajaxe.

no ale chlapci nemíchejte jabka a hrušky…ajax je jakási nádstavba. Primárně máte napsat odkazy tak aby fungovaly bez ajaxu a ajaxifikaci pak jen přidat.

Akadlec
Dnes asi nieje tvoj deň brácho :D
Filip
Ak je to v tvojej kopetencii tak by bolo fajn to doplniť. Neviem ako sa docu píše ale predstavujem si to tak, že sa chytí text doplnia sa 4 vety a commit. Prečo je to taký problém? Samozrejme nechcem zhadzovať to čo je napísané, len ma to mrzí, lebo je to na škodu veci.

@FilipKlimeš:

Toto ale opravdu nesouvisí moc s ajaxem samotným. Tohle je více o „nenastudování životního cyklu presenteru“ (či nenastudování rozdílu mezi signálem a action a prací s parametry), protože tak či tak se lze v dokumentaci „doklikat“ k větě „Signál (aneb subrequest) je komunikace se serverem pod prahem normálního view, tedy akce, které se dějí, aniž by se změnilo view.“ a tedy, že danou funkčnost lze udělat jak pomocí action i pomocí signálu

namespace App\Presenters;

class HomepagePresenter
	extends BasePresenter{

	public function renderDefault($value){
		if(!$this->ajax AND !empty($value)){
			$this->template->value = $value;
		}
	}

	public function actionDelete(){
		$value = $this->action;
		if(!$this->ajax){
			$this->redirect('default', ['value' => $value]);
		}

		$this->view            = 'default';
		$this->template->value = $value;
		$this->redrawControl('snippet');
	}

	public function handleRemove(){
		$value = $this->signal[1];
		if(!$this->ajax){
			$this->redirect('this', ['value' => $value]);
		}

		$this->template->value = $value;
		$this->redrawControl('snippet');
	}
}

{default $value = 'default value'}

{block content}
	<a n:href="default">Reset</a><br>
	<a n:href="remove!" class="ajax">Signal (with Ajax)</a><br>
	<a n:href="delete" class="ajax">Action (with Ajax)</a><br>
	<a n:href="remove!">Signal (without Ajax)</a><br>
	<a n:href="delete">Action (without Ajax)</a><br>
	<br><br>

	{snippet snippet}
		{$value}
	{/snippet}
{/block}

To, že přesune danou operaci z action do signálu neřeší ten problém, že nezná cyklus presenteru a na každém dalším krapet složitějším požadavku se ihned zasekne.

Podlě mě mu ta úvodní věc umře celkem logicky na trying to get property of non object protože v renderDefault bude mít něco na způsob:

	public function renderDefault($id){
		$this->template->article = $this->blogArticles->findOneBy(array('id' => (int)$id)); /** (Nevím jak se přesně používá nette database (nebo co to je :) ) takže je to jen obrazné) */

		/**
		* ...
		* ...
		*/
	}

Výsledek je ten, že si vytvoří link na actionDelete něco ve smyslu:

	<a n:href="delete, id => $comment->id">Smazat</a> // čímž změní ID článku na ID komentáře

což vede na „actionDelete($id)“ kde se provede nějaké mazání (nebo taky klidně nic) a změní se „view“ zpět na „default“ v „renderDefault“ to zase bude chtít načíst článek z DB jenže už ho to bude hledat podle ID komentáře (které si před tím krásně prohodil), což pravděpodobně vrátí „NULL“ no a jsme doma. :)

Tak jako tak to podle mě nemohlo fungovat s ajaxem ani bez něj a pokud to „fungovalo“, tak to byla jen souhra náhod a jen díky tomu, že zrovna byl Merkur v konjunkci s Venuší.

PS: Neříkám, že by se dokumentace ohledně ajaxu neměla upřesnit a třeba tam i dopsat, kdy je většinou vhodnější použít signál a kdy action, ale zrovna v tomhle případě bych neviděl problém v dokumentaci k ajaxu, ale v celkovém nepochopení životního cyklu presenteru a obecně prací s parametry.

Editoval Etch (14. 2. 2015 9:40)

@FilipKlimeš:

Zrovna u toho mazání přes signál mi to přijde v dokumentaci trochu „dvojsečné“.

• Ano, je to krásný příklad.
• Ano, je to cool.
• Ano, je to první věc, která každého napadne.
• Ano, je to v nette věc na pár řádků.

jenže má to jedno podstatné „ale“ a to právě minimálně z pohledu začátečníka.

Muselo by tam být minimálně 3× rudým písmem zdůrazněno, že je to nutno bezpodmínečně ošetřit proti CSRF.

Ovšem i v tom případě si nejsem úplně jist, že by se tím nějaký „začátečník“ zdržoval. :)

Editoval Etch (14. 2. 2015 12:05)

@FilipKlimeš a to je právě ten problém začátečníka, moc toho neumí ale hned bych těl umět udělat eshop (nenarázím na nikoho zde) místo toho aby se prokousal základy a zjistil proč je tohle tak a tak. Jak tady @Etch zmínil, je to o pochopení životního cyklu aplikace.

Jinak třeba to uvedené delete…ve většině případů chce tu akci potvrdit zda uživatel tu akci opravdu chce udělat, takže se dá krásně využít komponenty/extension co to vyřeší a zárovně ošetří proti útoku ;)

@Čamo nextras/secured-links

Prosím vás, tie secured-links nefungujú na action metódy?
Keby niekto aj vedel napísať prečo, tak to bude perfektné.

Potrebné to asi nieje, ale fungovať by mali podľa mňa.