Vytvořit jako helper nebo jak nejlépe?

@Etch
Paráda! Rešit tuhle „drobnost“ komponentou mě tedy vůbec nenapadlo!

Můžu k tomu mít pár dotazů?

1. inject do presenteru tímto způsobem nefunguje, resp. musím změnit viditelnost z private na public, je to tak správně?
2. do neonu mi to tímto způsobem zapsat nejde, zapsal jsem to takhle – App\FrontModule\Components\MatchResultFactory je to v pořádku?
3. Kdybych se rozhodl zobrazovat na 1 řádku víc různých typů výsledků (v tvém příkladě Inline i Full), tak se metoda evaluate provede 2× – zbytečně. Kam ji umístit? Do constructoru? Jde mi o to, abych ji vyjal z renderu … –
4. Jelikož mám výpis zápasů jako komponentu, tak mi tato výsledková komponenta nefunguje, napíše to, že neexistuje. Co s tím jak ji předat?

Moje struktura

CompetitionPresenter.php //
 – renderDefault(); // jen vypis soutezi
 – renderDetail($id); // detail souteze

Detail soutěže se skládá pouze z výpisu fází k soutěži a u každé volám komponenty zápasů a tabulek.
Takže to nejdůležitější vypadá takto

competition-detail.latte
===========================
{foreach $stageList as $stage}
 Fáze: {$stage->name}
 Zápasy: {control results, $stage->id}
 Tabulky: {control tables, $stage->id}
{/foreach}

a právě v tech results vypisuji ty zápasy ve kterých potřebuji ještě vložit tu novou komponentu od Tebe a nejde to, viz #4

Díky moc.

Editoval Kcko (29. 11. 2014 13:48)

@Oli díky za odpověd,

1. jasná (to se asi @Etch spletl).,
2. prostě nefunguje

můj neon

services:
	- Cardbook\Helpers\Helpers
	- App\Model\Competition
	- App\Model\Table
	- App\Model\Result
	- App\AclFactory
	authorizator: @App\AclFactory::createAcl
	authenticator: App\Model\UserManager
	#- App\FrontModule\Components\MatchResultFactory
	matchResult: MatchResultFactory # pokud s tím nebudeš dál pracovat tak to nemusíš ani pojmenovávat
	- MatchResultFactory # bez jména

Ale to moje použití jako servisy je špatně? viz zakomentovaný řádek v neonu.

No co bych tedy potřeboval ukázat / vysvělit

3. vyjmout tu metodu evaluate z renderu , (kdybych tam toho měl víc nechci aby se všehno dělo v renderu …)
4. jak tedy předat komponentu do další komponenty?

Mám to takto:

CompetitionPresenter.php

<?php

namespace App\FrontModule\Presenters;

use Nette,
	App,
	App\Model,
	App\FrontModule\Components,
	Nette\Forms\Controls;

class CompetitionPresenter extends BasePresenter
{

	protected $model;
	protected $modelTable;
	protected $modelResult;



	/** @var Components\MatchResultFactory @inject */
	public $matchResultFactory;

	protected function createComponentMatchResult()
	{
	    return $this->matchResultFactory->create();
	}


	public function inject(Model\Competition $model, Model\Table $modelTable, Model\Result $modelResult)
	{
		$this->model       = $model;
		$this->modelTable  = $modelTable;
		$this->modelResult = $modelResult;
	}

	public function beforeRender()
	{
		$this->template->competitions = $this->database->table('competition_season');
	}


	public function renderDefault()
	{

	}





	public function renderDetail($id)
	{
		// info o soutezi
		$this->template->competition_season = $this->database->table('competition_season')
															 ->where('id', $id)->fetch();

		$this->template->stageList = $this->model->getStages(
			$this->template->competition_season->competition_id,
			$this->template->competition_season->season_id
		);
	}


	protected function createComponentResults()
	{

		$control = new Components\Results($this->database, $this->modelResult);
		return $control;
	}


	protected function createComponentTables()
	{

		$control = new Components\Tables($this->database, $this->modelTable);
		return $control;
	}




	protected function createComponentEditForm()
	{

		$form = new Nette\Application\UI\Form;
		$form->addText('goal_home_90', 'Góly domácí po 90 min.');

		$form->addText('goal_away_90', 'Góly hosté po 90 min.');
		$form->addText('goal_home_et', 'Góly domácí ET');
		$form->addText('goal_away_et', 'Góly hosté ET');

		$form->addText('goal_home_penalty', 'Góly domácí penalty');
		$form->addText('goal_away_penalty', 'Góly hosté penalty');

		$form->addCheckbox('played', 'Odehráno');
		$form->addHidden('id');

		$form->addSubmit('btn', 'Uložit');

		$form->onSuccess[] = $this->submitEditForm;


		$renderer = $form->getRenderer();
		$renderer->wrappers['controls']['container'] = NULL;
		$renderer->wrappers['pair']['container'] = 'div class=form-group';
		$renderer->wrappers['pair']['.error'] = 'has-error';
		$renderer->wrappers['control']['container'] = 'div class=col-sm-9';
		$renderer->wrappers['label']['container'] = 'div class="col-sm-3 control-label"';
		$renderer->wrappers['control']['description'] = 'span class=help-block';
		$renderer->wrappers['control']['errorcontainer'] = 'span class=help-block';
		// make form and controls compatible with Twitter Bootstrap
		$form->getElementPrototype()->class('form-horizontal');
		foreach ($form->getControls() as $control) {
			if ($control instanceof Controls\Button) {
				$control->getControlPrototype()->addClass(empty($usedPrimary) ? 'btn btn-primary' : 'btn btn-default');
				$usedPrimary = TRUE;
			} elseif ($control instanceof Controls\TextBase || $control instanceof Controls\SelectBox || $control instanceof Controls\MultiSelectBox) {
				$control->getControlPrototype()->addClass('form-control');
			} elseif ($control instanceof Controls\Checkbox || $control instanceof Controls\CheckboxList || $control instanceof Controls\RadioList) {
				$control->getSeparatorPrototype()->setName('div')->addClass($control->getControlPrototype()->type);
			}
		}


		return $form;
	}

	public function submitEditForm($form)
	{
		$id     = (int) $this->getParameter('id');
		$values = $form->getValues();
		$match  = $this->database->table('match')->where('id', $id)->fetch();
		$match->update($values);

		//\Nette\Diagnostics\Debugger::barDump($values);

		$this->redirect('Competition:detail', $match->competition_id);
	}


	public function actionEditMatch($id)
	{
		$this->template->match = $this->database->table('match')->where('id', $id)->fetch();

		$form = $this['editForm'];

		if (!$form->isSubmitted())
		{

			$defaults = $this->database->table('match')->where('id', $id)->fetch();
			$form->setDefaults($defaults);
		}


	}

}

Competition/detail.php

{var $competName = $competition_season->competition->name}
{var $seasonName = $competition_season->season->name}

{block title} {$competName} - {$seasonName} {/block}
{block content}

        <div class="row">

            <div class="col-lg-8">
                <h1 class="page-header">

                    {$competName} - {$seasonName}
                </h1>


                {foreach $stageList as $id => $stage}

                    {if $stage['row']['has_matches'] || $stage['row']['has_table']}

                        <h4>
                            <span class="glyphicon glyphicon-circle-arrow-right" aria-hidden="true"></span>
                            {$stage['name']}
                        </h4>

                        {if $stage['row']['has_matches']}
                            {control results, $id}
                        {/if}

                        {if $stage['row']['has_table']}
                            {control tables, $id}
                        {/if}

                       {sep} <hr /> {/sep}

                    {/if}

                {/foreach}
            </div>

Components/Results.php

<?

namespace App\FrontModule\Components;

use Nette\Application\UI,
	Nette,
	App;

class Results extends UI\Control
{

	protected $database;
	protected $modelResult;

	public function __construct (Nette\Database\Context $database, App\Model\Result $modelResult)
	{
		$this->database    = $database;
		$this->modelResult = $modelResult;

	}


	public function render($stageId)
	{


		$this->template->setFile(__DIR__ . '/../templates/components/Results/list.latte');
		$this->template->stageId     = $stageId;
		$this->template->modelResult = $this->modelResult;
		$this->template->matches     = $this->database->table('match')->where('stage_list_id', $stageId)->order('id');
		$this->template->render();
	}


}

Results/list.php
Tady potřebuju vyvolat další komponentu – MatchResults

<table class="table table-striped table-bordered table-condensed">
	{foreach $matches as $match}

	{var $homeTeam = $match->ref('team', 'team_home_id')}
	{var $awayTeam = $match->ref('team', 'team_away_id')}


    <tr>
        <td><a href="{plink Competition:editMatch $match->id}">{$match->id}</a></td>
        <td class="text-right" style="width: 42%; padding-right: 2%;">{$homeTeam->name} <img src="/assets/gfx/flags/16/{$homeTeam->logo}" alt="" /></td>
        <td class="text-center">{control matchResult:inline, []}</td>
        <td class="text-left" style="width: 42%; padding-left: 2%;"><img src="/assets/gfx/flags/16/{$awayTeam->logo}" alt="" /> {$awayTeam->name}</td>
    </tr>
    {/foreach}
</table>

Components/MatchResult.php
Je to to samé od @Etch

<?

namespace App\FrontModule\Components;

use Nette\Application\UI,
	Nette,
	App;


interface MatchResultFactory
{
    /** @return MatchResult */
    public function create();
}


class MatchResult extends UI\Control
{

    const TEMPLATE_FULL = 'full';
    const TEMPLATE_INLINE = 'inline';

    public function renderFull($match)
    {
        $this->render($match, self::TEMPLATE_FULL);
    }

    public function renderInline($match)
    {
        $this->render($match, self::TEMPLATE_INLINE);
    }

    private function render($match, $templateName)
    {
        $template = $this->createTemplate();
        //$this->template->setFile(__DIR__ . '/../templates/components/Cards/nice-list.latte');
        $template->setFile(__DIR__.'/../templates/components/MatchResult/'.$templateName.'.latte');
        $template->match = $this->evaluateResult($match);
        $template->render();
    }

    private function evaluateResult($match)
    {
        //@todo -- dodelat, nejake ulozeni do tohoto objektu nebo do vicerozmernho pole
    }


}

Tak snad se v tom vyznáš :)

@Kcko

1. Ano v mém původním příspěvku jsou nějaké drobné chyby (private místo public atd. nahoře opravím, aby to někoho třeba nemátlo)… Ve tři ráno mi to nějak nedocvaklo.
2. Můj původní neon zápis nepředpokládal namespace u daných tříd. Pokud sis tam ten namespace přidal, tak je samozřejmě nutné ho doplnit i v definici v configu. - App\FrontModule\Components\MatchResultFactory je v tvém případě správná definice.
3. V tomto případě dost dobře nejde danou metodu dostat z render metody. Nejjednodušší je v dané metodě zajistit, aby výpočet proběhl pouze jednou.

	private $results = array();

private function render($match, $templateName){
    $template = $this->createTemplate();
    $template->setFile(__DIR__.'/'.$templateName.'.latte');
    $template->result = $this->getResult($match);
    $template->render();
}

	private getResult($match){
		if(!array_key_exists($match['id'], $this->results)){
			$this->results[$match['id']] = $this->evaluateResult($match);
		}

		return $this->results[$match['id']];
	}

4. Pokud danou komponentu potřebuješ v jiné komponentě, tak jí injectuj konstruktorem přímo do dané komponenty:

<?

namespace App\FrontModule\Components;

use Nette\Application\UI,
    Nette,
    App;

class Results extends UI\Control
{

    protected $database;
    protected $modelResult;
	private $matchResultFactory;

    public function __construct (Nette\Database\Context $database, App\Model\Result $modelResult, App\FrontModule\Components\MatchResultFactory $matchResultFactory)
    {
        $this->database    = $database;
        $this->modelResult = $modelResult;
		$this->matchResultFactory = $matchResultFactory;

    }


    public function render($stageId)
    {


        $this->template->setFile(__DIR__ . '/../templates/components/Results/list.latte');
        $this->template->stageId     = $stageId;
        $this->template->modelResult = $this->modelResult;
        $this->template->matches     = $this->database->table('match')->where('stage_list_id', $stageId)->order('id');
        $this->template->render();
    }

	protected function createComponentMatchResult(){
    	return $this->matchResultFactory->create();
	}


}

Editoval Etch (29. 11. 2014 17:47)

@Etch Díky, 3/ je jasná, zatím si tam asi nechám ale to co jsem spatlal já. Na 4/ jsem už taky obdobně přišel.

Vrtá mi hlavou Multiplier.

Dejme tomu, že mám tento formulář:

	protected function createComponentTestForm()
	{


	    return new Nette\Application\UI\Multiplier(function ($itemId) {
	        $form = new Nette\Application\UI\Form;
	        $form->addText('zapas', 'Zápas')->setValue($itemId);

	        $form->addSubmit('send', 'Odeslat');
	        return $form;
	    });
	}

V šabloně

{var $counter = 0}
{foreach $matches as $match}
	{? $counter++}
    {control testForm-$number}
{/foreach}

A já bych v té šabloně tomu formuláři chtěl předat nějaká data, v tomto případě $counter a $match (nejde mi ted o ty data, ale o způsob jak to tam dostat, aby o tom formulář veděl).

A Ještě hypoteticky, kdyby tohle nebyl formulář, ale třeba opět nějaký výpis zápasů a výsledků a potřeboval bych tu kontrolku nějak nastavit přes metodu, můžu to udělat stejně jako v předchozím postu?

{?
    $matchResult = $control['matchResult'];
    $matchResult->evaluateResult($match);  // tímto se zbavím vyhodnocení v renderu
}

// výsledek!
{$matchResult->renderInline()}

Bylo by hezčí, tohle dělat někdě na úrovni Presenteru/Komponenty a ne v šabloně, ale tady to nejde a já potřebuji znát způsob jak dostat data ze šablony do komponenty (v tomto případě přes Multiplier).

Tohle je poslední dotaz.

@Etch

Super, díky moc za pěkný rozbor, vidím, že to co jsem jako poslední napsal, je blbost, i když to funguje. Vypadá to divně a není to moc přehledná komponenta (hlavně v použití v šabloně).

Tady

# Nějaká tabulka

{foreach $matches as $match}
    {control matchResult:full, $match}
{/foreach}

#Nějaký postraní box na stejné stránce

{foreach $matches as $match}
    {control matchResult:inline, $match}
{/foreach}

Tohle je fajn, doupravím to na tento způsob.

Můžu ještě požádat o informaci k tomu co jsem psal ohledně Multiplieru, zkouknul jsem i video J. Tvrdíka o komponentách, ale tam tohle neřeší. Tam sice nastaví interní ID článku, ale v presenteru když ho zná a ne ze šablony, můj post (Viz https://forum.nette.org/…-jak-nejlepe#… , začátek postu).

Třeba chci mít pod každým zápasem formulář a k němu do hidden pole docvaknout ID zápasu, ID souteze, ID sezóny a ještě něco a tyhle údaje budu znát jen ze šablony. (Popsal jsem to v tom prispevku).

Nebo znovu

protected function createComponentTestForm()
{


    return new Nette\Application\UI\Multiplier(function ($matchId) {
        $form = new Nette\Application\UI\Form;
        $form->addHidden('zapas', 'Zápas')->setValue($matchId);
		$form->addHidden('counter', 'Counter')->setValue(???);
		$form->addHidden('competitionId', 'Competition')->setValue(???);
		$form->addHidden('seasonId', 'Season')->setValue(???);
        $form->addSubmit('send', 'Odeslat');
        return $form;
    });
}

V šabloně

{var $counter = 0}
{foreach $matches as $match}
    {? $counter++}
    {control testForm-$number} // jak sem dostanu ten counter a pak soutez a sezonu ktera se nachazi v $match
{/foreach}

Editoval Kcko (29. 11. 2014 20:26)

Místo $counteru se dá použít {$iterator->counter}. K čemu ten counter slouží??

Já vím, že ano. Toto má být jen demonstrace, jak do té komponenty via Multiplier nastavit proměnnou či proměnné zvenčí (ze šablony).

@Etch, jasně, ale mě jde o to i obecně, když budu mít data která chci mít v komponentně možnost předat jen přes šablonu, tak nevím jak na to. Můžeš malou demonstraci a tím bych to uzavřel.

@Etch – koukám a vidím, ten dvojitý multiplier už je ugly. Ted jsem si zkoušel jednoduchý příklad s výpisem knih a nastavil jsem to tam kde se to asi nastavovat má. Tenhle příklad je velmi jednoduchý, v realném by autor byl z jiné tabulky (ref) a vadí mi tady dvojí průchod. Jednou v šabloně, jednou v presenteru.
Jsem zvyklý z NDBT si vracet jen Selection a v šabloně pracovat s aktivním view (tj. používat ref, related ..) a v tomto případě bych si musel model upravit aby mi to vracel všechno najednou seskládané v nějakém poli, aby mě ten dvojí průchod nemrzel.

Tj. něco jako

function getBooks()
{

	$selection =  $this->database->table('book');
	$data = array();
	foreach ($selection as $sel)
	{
		$data['baseData'][] = iterator_to_array($sel);
		$data['autor'][] = $sel->author->name; // ref na jinou tabulku

		// pak treba tagy  -- Related --
		a dalsi struktury
	}


	return $data;
}

Je to tak správně?

	public function actionBook()
	{
		$this->template->books = $this->database->table('book');

		foreach ($this->template->books as $book)
		{
			$this['bookForm-'.$book->id]->setDefaults($book);
		}
	}



	protected function createComponentBookForm()
	{


	    return new Nette\Application\UI\Multiplier(function ($id) {
	        $form = new Nette\Application\UI\Form;
	        $form->addText('id', 'Id')->setValue($id);
	        $form->addText('name', 'Name');
	        $form->addText('author', 'Author');
	        $form->addText('pages', 'Pages');

	        $form->addSubmit('send', 'Odeslat');
	        return $form;
	    });
	}

@Etch
No jenže se mi opět tak nějak vnucuje myšlenka, kdy bych takovouhle funkčnost mohl reálně potřebovat.
No už se mi to stalo :) (nebudu popisovat proč ani jak, byl by to zas elaborát).

Každopádně díky za vyčerpávající komentáře, opět jsem se něco naučil!

Bude nové GW Kcko? :O :D

@Kcko No ona to byla spíš reakce na @Etch

Mám výpis knih a formulář pro „editaci“ každé knihy. V jakou chvíli nastane situace, že potřebuju pro 5O knih vypsat padesát editačních formulářu, které mi jsou stejně k ničemu, protože odeslat mohu reálně jen jeden z nich?

Na to se právě multiplier nehodí, pokud to chceš poslat naráz všechno.

@Kcko: Kozel. Já už tam moc nechodím, ale Lahva by se měl lépe. :))

Jinak super vlákno, hodně mi to dalo. Hlavně ten příspěvek od @Etch je super!

@Etch tak to s omlouvám, došlo k informačnímu šumu. Jsem to četl na mobilu a na počítači odpovídal až zpětně a asi ne uplně aktuálně :-)

Nicméně použil jsem to generování formuláře pomocí multiplikátoru. Bylo to u výpisu článků ke kterým si po kliknutí na ikonku mohl přihlášený uživatel přidat nějakou osobní poznámku. Tam jednoznačně chceš aby se ti otevřel jeden formulář u konkrétního článku. Ale je pravda, že to není zrovna nejčastější požadavek ;-)

@Kcko A nebylo by lepší to vykreslit jako jeden formulář s jedním odesílacím tlačítkem? Nevím jak často se tam mění víc „formulářů“ naráz…

Editoval Oli (1. 12. 2014 18:09)

Přikláněl bych se k bodu 1 a 2.Záleží na použití. Pokud se jedná o výpis dat, která můžou být editována a budou editována stylem „jeden řádek za týden“ tak bych použil variantu 2+4 (inline editace po nějaké akci – jako adminer). Pokud se budou měnit data často tak jednoznačně variantu 1, jinak tě uživatelé roztrhají :-)

@Etch
trochu OT: Zrovna k tomu mazání bych nepoužíval formulář. Někde byla o tom diskuse a je pravda, že formulář k tomu většinou není určenej. Pokud se bavíme o formuláři, kterej má jen tlačítko smazat. Proti CSRF to ošetřuju pomocí Nextras/secured-link.

Kcko napsal(a):

Takže co 1 záznam = 1 malý formulář + submit ⇒ odešle se ajaxem a uživatel přeskočí na další řádek a opět edituje.

Tohle jde řešit snadno a rychle pomocí gridu s možností editovat přímo v něm.
Je jich tady několik – addons.nette.org/