RFC: Komponenty které se samy nevykreslují, ale jen vracejí view

Ta hlavní myšlenka je aby komponenta sama nic nevypisovala na výstup, ale pouze předala ven šablonu, která se má vykreslit.

Pokud se komponenta sama přímo vykreslí tak nejde dobře testovat. A ani mi to nepřijde jako úplně čisté.

To že můžu mít více render metod s tímhle nijak zásadně nesouvisí, stejně to bude fungovat pro všechny render metody.

Editoval mystik (26. 5. 2014 23:08)

enumag napsal(a):

Podle mne by měla vracet něco jiného než šablonu, spíše nějaký ControlView objekt (v původním příspěvku jsem použil pro jednoduchost pole). Tento objekt by se pak předal šabloně k vykreslení. Snažím se tím docílit nezávislosti komponent na šablonách a hlavně na jejich konfiguraci.

Pozn. trochu jsem se inspiroval u symfoních formulářů.

Za mě ještě lepší řešení, ale to už by byl větší zásah do frameworku, který by asi nešel udělat zpětně kompatibilní.

Oproti tvému řešení bych ale nechal komponentě možnost nějak určit šablonu. V jednom jiném projektu (nepoužívá nette) používáme to, že presentery/komponenty vracejí identifikátor šablony, které je až pak následně přeložen na konkrétní šablonu (podle nastavené theme a případně dalších parametrů). Při vykreslování si ale můžeš v případě potřeby předat šablonu jinou.

Editoval mystik (26. 5. 2014 23:32)

no len komponenty sa nemusia vzdy vykreslovat pomocou sablon

Formulář.

@vojtech.dobes, @milo

Ok co v tom případě mít něco jako TextView do kterého přiřadíš jen obsah (content) a on se vykreslí pseudo-šablonou:

{$content}

Komponenta nebude o nic složitější:

public function renderSomething() {
  return new TextView("...");
}

Rozhraní bude identické a v případě potřeby můžu při vykreslování použít jinou šablonu. Např.:

<div class="special-div-needed-in-my-application-to-solve-graphic-glitch">
{$content}
</div>

To stejné pro formuláře.

Editoval mystik (27. 5. 2014 10:52)

Napadlo mě alternativní řešení, které nebude takový zásah do frameworku a bude zpětně kompatibilní.

V latte by se nahradilo

<?php
$control->render()
?>

za

<?php
echo $control->render()
?>

Pro současné komponenty by se nic nezměnilo. Ty by rovnou vypsaly výstup a nic nevracely takže echo by nic nevypsalo.

Nové komponenty by ale mohly vracet string (tím by se mnoho nezměnilo), ale aspoň by se výstup nemusel chytat přes output buffer.

Nebo co je hlavní jakýkoliv objekt s __toString(), který by pak sloužil jako ControlView. To by mohla být přímo Template, ale i nějaký vlastní objekt jehož obsah/typ můžu snadno otestovat a zkontrolovat tak že komponenta dělá to co má.

Jediné změny by bylo přidání toho echo.

PR: https://github.com/…tion/pull/88

Editoval mystik (29. 7. 2015 9:54)

@Felix: To se ale s touto úpravou nevylučuje. ControlView může být právě jedna z věcí, které se z control budou vracet.

akadlec napsal(a):

No a mergne to někdo do nette? Nebo to zase vyšumí? Docela se mě to líbí oproti klasickému echu co tam musí být :(

Jo tak to tezko rict. Bylo by potreba nazoru @DavidGrudl, @FilipProcházka, @JanTvrdík apod..

Problém je, že rozhodnout, zda tohle v Nette chceme nebo ne, je výrazně složitější úloha, než to implementovat.

První, co mě napadá je, že tohle si snadno doimplementuješ sám, ne? V komponentách budeš míst render() používat renderToView, které bude vracet objekt a render() pak zavolá echo $this->renderToView().

Zrovna jsem chtěl napsat, že by nebylo špatné zeptat se autora, proč to navrhl zrovna tak, jak to navrhl, a zda k tomu neměl třeba nějaký rozumný důvod. Nyní již od něj víme, že k tomu důvod měl. Jen škoda, že to více nerozvedl. Zajímalo by mě, o jaký důvod šlo.

Ještě by mohla metoda render() „vracet“ generátor a do něj místo echa yealdovat data postupně. To si ale můžeš naimplementovat sám, nebo v nějakém rozšíření, nemá cenu kvůli tomu zvedat požadavek na php 5.5. Případně ve stylu symfony rovnou vracet nějakou streaming response a do ní to posílat přes callbacky.

Alternativní nápad: Nevracet objekt nad kterým se zavolá echo, ale objekt představující render funkci nad kterým by se zavolalo __invoke. Tím by se postupné vykreslování zachovalo, ale zároveň by se to dobře testovalo.

Pak tam mít něco jako

<?php
if(($view = $control->render()) !== NULL) { view(); }
?>

Pak by ale bylo buď potřeba mít něco jako TempalteView představujícíc render šablony nebo u šablony přidat __invoke volající render.

Editoval mystik (31. 7. 2015 15:52)