Přejmenovat getAjaxDriver()

Upozornění: Tohle vlákno je hodně staré a informace nemusí být platné pro současné Nette.
David Grudl
Nette Core | 8218
+
0
-

Během vývoje Nette\Application\AjaxDriver změnil svou funkci a kromě driveru se stal především úložištěm pro AJAXem předávaná data. Úložiště dostupné přes $presenter->getAjaxDriver() resp. $presenter->ajaxDriver zavání špatnou terminologií. Navrhuji změnu (samozřejmě zpětně kompatibilní), vyčlenení úložiště pod novou metodu.

Co třeba

  • getAjaxData()
  • getAjaxStorage()
  • getAjaxPayload()

Pokud bychom se nechtěli omezit čistě na AJAX, ale třeba přijmout i AMF, bylo by vhodné asi

  • getPayload()

Termín payload se používá běžně ve spojitosti s XML/RPC a SOAP, označuje vlastně ty samotné přenášené data.

Co myslíte?

Honza Marek
Člen | 1664
+
0
-

No mně připadá AjaxDriver docela driverovitej, kvůli metodám open a close. Tam si určuje, jak se bude chovat, například v metodě close určí i výstupní formát.

David Grudl
Nette Core | 8218
+
0
-

Já jsem se špatně vyjádřil a post jsem doplnil. Driver by nadále zůstal, jen to úložiště by bylo dostupné pod jinou metodou.

Honza Marek
Člen | 1664
+
0
-

Tak to mi vůbec nevadí…

S názvem neporadím, je mi to v zásadě jedno. Osobně bych zvolil název AjaxDataStorage a pak bych si nadával, že je moc dlouhej :-D

Tomik
Nette Evangelist | 485
+
0
-

Přejmenování metodu pro vracení dat mi přijde docela dobrý nápad, bude to pak míň matoucí… Já bych byl pro getAjaxPayload… vcelku to to vystihuje…

David Grudl
Nette Core | 8218
+
0
-

Potřeboval jsem to dořešit kvůli aktuálnímu článku na Zdrojáku, tak jsem zvolil (technologicky neutrální) getPayload(). Ladí to s template:

$this->template->var = 'data';
$this->payload->var2 = 'data2';