Vlastni FormRenderer a stale sa pouziva DefaultFormRenderer

Na tomto nic špatného není, jen ty závorky jsou zbytečné. Chyba bude jinde.

Šaman napsal(a):

Na tomto nic špatného není, jen ty závorky jsou zbytečné. Chyba bude jinde.

Novy objekt sa vytvori, ale ani funkcia init() sa nezavola :-(

<?php
/**
 * This file is part of the Nextras community extensions of Nette Framework
 *
 * @license    New BSD License
 * @link       https://github.com/nextras/forms
 * @author     Jan Skrasek
 */
namespace TestUI\Helpers;

use Nette\Forms\Rendering\DefaultFormRenderer;
use Nette\Forms\Controls;
use Nette\Forms\Form;
use Nette;
use Nette\Utils\Html;
/**
 * FormRenderer for Bootstrap 3 framework.
 * @author   Jan Skrasek
 * @author   David Grudl
 */
class Bs3FormRenderer extends DefaultFormRenderer
{
    /** @var Controls\Button */
    public $primaryButton = NULL;
    /** @var bool */
    private $controlsInit = FALSE;
    public function __construct()
    {
        $this->wrappers['controls']['container'] = NULL;
        $this->wrappers['pair']['container'] = 'div class=form-group';
        $this->wrappers['pair']['.error'] = 'has-error';
        $this->wrappers['control']['container'] = 'div class=col-sm-9';
        $this->wrappers['label']['container'] = 'div class="col-sm-3 control-label"';
        $this->wrappers['control']['description'] = 'span class=help-block';
        $this->wrappers['control']['errorcontainer'] = 'span class=help-block';
    }

    public function init() {
        die();
    }



    public function renderBegin()
    {die();
        $this->controlsInit();
        return parent::renderBegin();
    }
    public function renderEnd()
    {die();
        $this->controlsInit();
        return parent::renderEnd();
    }
    public function renderBody()
    {die();
        $this->controlsInit();
        return parent::renderBody();
    }
    public function renderControls($parent)
    {die();
        $this->controlsInit();
        return parent::renderControls($parent);
    }
    public function renderPair(Nette\Forms\IControl $control)
    {die();
        $this->controlsInit();
        return parent::renderPair($control);
    }
    public function renderPairMulti(array $controls)
    {die();
        $this->controlsInit();
        return parent::renderPairMulti($controls);
    }
    public function renderLabel(Nette\Forms\IControl $control)
    {die();
        $this->controlsInit();
        return parent::renderLabel($control);
    }
    public function renderControl(Nette\Forms\IControl $control)
    {die();
        $this->controlsInit();
        return parent::renderControl($control);
    }
    private function controlsInit()
    {die();
        if ($this->controlsInit) {
            return;
        }
        $this->controlsInit = TRUE;
        $this->form->getElementPrototype()->addClass('form-horizontal');
        foreach ($this->form->getControls() as $control) {
            if ($control instanceof Controls\Button) {
                $markAsPrimary = $control === $this->primaryButton || (!isset($this->primary) && empty($usedPrimary) && $control->parent instanceof Form);
                if ($markAsPrimary) {
                    $class = 'btn btn-primary';
                    $usedPrimary = TRUE;
                } else {
                    $class = 'btn btn-default';
                }
                $control->getControlPrototype()->addClass($class);
            } 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) {
                if ($control->getSeparatorPrototype()->getName() !== NULL) {
                    $control->getSeparatorPrototype()->setName('div')->addClass($control->getControlPrototype()->type);
                } else {
                    $control->getItemLabelPrototype()->addClass($control->getControlPrototype()->type . '-inline');
                }
            }
        }
    }
}

A prečo by sa mala volať?

Volá sa metoda render()

A ako to používaš, ono tá časť kódu čo si dal je OK, skús poslať viac, možno celú create component metódu, lebo ako písal @Šaman chyba bude niekde inde.

Nevím, jestli ty tvoje kódy jsou jenom ukázkové, ale název třídy render se neshoduje s tvojí.

Nextras\Forms\Rendering\Bs3FormRenderer a TestUI\Helpers\Bs3FormRenderer

já píšu callback jsko např. $this->BrandFormSuccess a nette/object z toho přes ObjectMixin udělá callback na stejnojmennou metodu.

Presun radek

$form->setRenderer(new \AgileUI\Helpers\Bs3FormRenderer);

Az nakonec toho jak mas nadefinovany formular, tesne nad

	return $form;

nebo callbacky, pisni jestli ti to pomohmo.

jiri.pudil napsal(a):

Komponenta by neměla záviset na tom, že presenter implementuje nějaké konkrétní metody. Já mám zpracování formuláře přímo v té komponentě a stejně tak v ní mám vlastní událost, kterou právě při tom zpracování vyvolám, a obsluhu si na ni navěsím až v presenteru. Viz můj článek

Jj, velmi pekne dakujem za tento pohlad. Sam som nad nim rozmyslal a tiez sa mi nepaci, ze mam v presenteri mnozsto metod na osetrenie formulara. Ale uvazoval som nad tym, ze budem chciet formular pouzit niekolko krat ako standardny formular a niekolko krat ako jaxovy modal window. A v tychto pripadoch budem vzdy redirectovat niekam inam.

Aj ked na druhej strane, target, kam presmerovat by bolo efektnejsie osetrit asi setterom, ci? Napada Ta ine riesenie?

Lexi napsal(a):

Presun radek

$form->setRenderer(new \AgileUI\Helpers\Bs3FormRenderer);

Az nakonec toho jak mas nadefinovany formular, tesne nad

	return $form;

nebo callbacky, pisni jestli ti to pomohmo.

A skusal som aj pred callbacky aj za a nejde. Potom som skusal callbacky dat prec, ale tiez nepomohlo.

Editoval steelbull (27. 6. 2015 19:24)

steelbull napsal(a):

David Matějka napsal(a):

zeptam se znovu, jak ten formular vykreslujes v latte?

Vykreslujem ho takto:

{form form}
{/form}

No, když ho vykresluješ ručně, tak se samozřejmě renderer neprojeví. Můžeš ho nechat buď vykreslit rendererem {control fooForm}, nebo ručně pomocí latte makra {form}{/form}. :D

Zkus to rozchodit na ukázkovém formuláři na čistém sandboxu (nebo spiš web-projectu) a pošli odkaz na zdrojáky. Někde máš zakopanou chybu. A bude se špatně hledat, když nám ukazuješ zdrojáky po kouskách. Chyb tam teda máš víc, třeba komponenta není továrna, ale to jsou věci související s čitelností, ne s funkčností.

A jinak nehledáš teda spíš tohle?

Já mluvil o tomhle class BrandFormFactory extends UI\Control. Tohle není továrna, je to komponenta – potomek Control (která obsahuje továrničku na formulář, ale to už je interní věc té komponenty).

Jinak na tu otázku kam psát obsluhu událostí formuláře, tak do té komponenty.
Ale není problém si udělat opravdu jen továrnu na formulář (bez komponenty) a obsluhu mít také v ní.

Ale těm navěšeným callbackům v presenteru se úplně nevyhneš, protože většinou až presenter umí zpracovat flashmessage a hlavně rozhodnout, kam přesměrovat. Nemluvím teď o triviálním přesměrování na ten samý formulář, ale třeba po registraci chci nejspíš zobrazit jinou stránku – detail nového uživatele, nebo třeba výzvu k potvrzení registrace odkazem v emailu apod. Já mám nejčastěji zpracování dat u formuláře, ale přesměrování a flashmessage navěšuji až v presenteru (nebo komponentě).

Editoval Šaman (28. 6. 2015 9:05)