@inject kdekoliv nebo jen v presenteru?

myslim, ze to nejak jde.. ale nepouzivej to :) ve sluzbach bys mel injectovat jen pres konstruktor

Zdravím,
mám problém v Nette 2.1 s manuálním vytvořením presenteru.

Jedná se o třídu Mailer, ve které bych rád využíval všechny fce a výhody presenteru, stejně jako své fce, které jsem již napsal do BasePresenteru:

<?php
	class MailerPresenter extends BasePresenter {}
?>

Vytvoření instance v BasePresenteru:

<?php
	protected $mailer;

	public function startup()
	{
		$this->mailer = new MailerPresenter();
		// Zde je třeba předat context Presenteru, ale bohužel nevím jak na to
		// Ve staré verzi Nette stačilo Environment::getContext
	}

	public function renderXXX()
	{
		// Ukazka uziti
		$this->mailer->registrationWebuserEmail(array('firstname' => 'Aaaa', 'email' => 'nejaky@email.cz'));
	}
?>

Zkoušel jsem předat context i za pomocí Service:

services:
	- App\FrontModule\Tools\MailerPresenter

A pak v BasePresenteru:

<?php
	/** @var \App\FrontModule\Tools\MailerPresenter @inject */
	public $mailer;
?>

Presenter se vytvoří správně, bohužel se nedaří předat potřebné závislosti. Děkuji za každou radu.

zaachi napsal(a):

Zdravím,
mám problém v Nette 2.1 s manuálním vytvořením presenteru.

Jedná se o třídu Mailer, ve které bych rád využíval všechny fce a výhody presenteru, stejně jako své fce, které jsem již napsal do BasePresenteru:

<?php
	class MailerPresenter extends BasePresenter {}
?>

Vytvoření instance v BasePresenteru:

<?php
	protected $mailer;

	public function startup()
	{
		$this->mailer = new MailerPresenter();
		// Zde je třeba předat context Presenteru, ale bohužel nevím jak na to
		// Ve staré verzi Nette stačilo Environment::getContext
	}

	public function renderXXX()
	{
		// Ukazka uziti
		$this->mailer->registrationWebuserEmail(array('firstname' => 'Aaaa', 'email' => 'nejaky@email.cz'));
	}
?>

Zkoušel jsem předat context i za pomocí Service:

services:
	- App\FrontModule\Tools\MailerPresenter

A pak v BasePresenteru:

<?php
	/** @var \App\FrontModule\Tools\MailerPresenter @inject */
	public $mailer;
?>

Presenter se vytvoří správně, bohužel se nedaří předat potřebné závislosti. Děkuji za každou radu.

v BasePresenter chceš závislost na MailerPresenter? To je dost divný né? Neměla to být spíš nějaká mailer služba než presenter?

@zaachi: v 2.2 by melo byt vytvareni sablon snadnejsi, nez pred tim. Pro preklady si muzes kdekoliv injectnout ITranslator. Problem jsou ty linky..
ale neded presenter, myslim, ze by to mohlo zpusobit i dalsi problemy a vsechno by nemuselo fungovat korektne, pouzij radeji:

class Mailer
{
	protected $application;

	public function __construct(Nette\Application\Application $application)
	{
		$this->application = $application;
	}

	public function send()
	{
		$presenter = $this->application->getPresenter();
		$presenter->createTemplate();
		$presenter->link(...);
		...
	}
}

Mailer registrujes jako sluzbu v neonu a injectnes kam potrebujes – o predani zavislosti se postara nette

matej21 napsal(a):

Mailer registrujes jako sluzbu v neonu a injectnes kam potrebujes – o predani zavislosti se postara nette

Ahoj,
zkusil jsme řešení, které jsi navrhl, ale nefunguje mi zcela optimálně:

<?php
class Mailer
{
	private $application;
	private $presenter;

	public function __construct(\Nette\Application\Application $application)
	{
		$this->application = $application;
		$this->my_presenter = $this->application->getPresenter();
	}

	public function registrationWebuserEmail($webuser)
	{
		$template = $this->my_presenter->createTemplate();
	}
}
?>

Vytvoření služby

	mailer: App\FrontModule\Tools\Mailer

Init

	/** @var \App\FrontModule\Tools\Mailer @inject */
	public $mailer;

Užití

$this->mailer->registrationWebuserEmail(array(''));

Dostanu hlášku:

Call to a member function createTemplate() on a non-object

Pokud to napíšu takhle:

<?php
	$presenter = $this->application->getPresenter();
	$template = $presenter->createTemplate();
?>

Dostanu:

Call to undefined method parent::createTemplate()

Ale například metoda link() funguje správně. Takže to vypadá na problém u templat. Nakonec budu muset asi užít FileTemplate zvlášť.

@Tomáš Votruba: ok, no mě se zdálo divné že musím všude zapnout inject že mě to nechce překládat ;)

@akadlec: Po vášnivé diskuzi na téma @inject bych řekl, že není úplně od věci si zapnout inject všude (třeba pomocí CompilerExtension, ať nemusíš všude psát inject: true) a pak v base třídách používat metody inject*, v konkrétních třídách pak konstruktor. Je to praktické (konstruktor si necháš volný pro potomky) a snad i docela čisté (nemůžu posoudit, viz můj podpis XD ).

Takhle, mám normální komponenty (komponenty a formy) a ty dědí od BaseControl aplikace. A v této BaseControl která zase dědí od Nette se vkládá translator a další pomocné služby (gravatary, obrázky, atd.) a tyto služby se aktuálně předávají přes inject metody. Takže kdybych to z inject přesunul do konstruktoru tak mě tam přibude dalších cca 5 předávaných služeb.

Továrničky mám přes interface tak jak je uvedeno na tvém linku

Editoval akadlec (9. 6. 2014 10:30)

Jop vím, jen sem ukázal jak to řeším já ;) dělat ještě vlastní factory classu mě připadá zase jako tuna kódu navíc ne? Navíc mi pak uniká kde se použije ten interface co je právě v tom postu co si linknul ty. Použije se tedy jako klasický interface te factory classy? Takže se pak vytvoří služba z té factory classy která bude řešit závislosti?

No a jak tedy správě? Udělat si BaseControlFactory a od ní pak dědit v samotných komponentách? Důvodem proč mám BaseControl je ten že v něm mám vytvořenou vlastní render metoru která se chová podle toho co to je za komponenta atd.

;) V podstatě ani ničeho velkého. Jednoduše mám komponentu „Avatar“ ta dědí od BaseControl aplikace a ta zase od Nettí control. A co tady řeším je inject vs constructor ;)

Komponenta avatar dostane přes konstruktor služby které vyžaduje (model, image službu atd.) V BaseControl aplikace je zase přidání služeb typu translator, gravatar, images, atd. Některé jsou napřímo přes injectMetody a něco je přes traity.

Samotná komponenta se pak vytvoří pomocí autmatické továrničky z interface. A co řeším je inject VS constructor. Protože pokud budu volat parent::__constrcut() tak si budu muset předávat vše co je postupně nutné, takže když bude komponenta avatar chtít sama o sobě 3 služby tak k nim budu muset přidat dalších cca 5 co jsou přes inject a už pak ten constructor bobtná.

Hele můžu ale fakt tam nehledej nic extra než sem v předchozím postu popsal ;)

class Control extends IPub\AppModule\Application\UI\Control
{
	/**
	 * @param User $user
	 * @param IFilesManager $filesManager
	 * @param IUserFacade $userFacade
	 */
	public function __construct(
		User $user,
		IFilesManager $filesManager,
		IUserFacade $userFacade
	) {
		parent::__construct();

		// Owner identity
		$this->userEntity	= $user->getIdentity();
		// File manager
		$this->filesManager	= $filesManager;
		// Module facades
		$this->userFacade	= $userFacade;
	}
}

abstract class Control extends \IPub\Application\UI\Control
{
	/**
	 * Implements autowiring extension
	 */
	use \Kdyby\Autowired\AutowireComponentFactories;

	/**
	 * Implement secured links for handle
	 */
	use \Nextras\Application\UI\SecuredLinksPresenterTrait;
}

abstract class Control extends \Nette\Application\UI\Control implements IObservable
{
	/**
	 * @param ITranslator $translator
	 */
	public function injectTranslator(ITranslator $translator)
	{
		$this->translator = $translator;
	}
}

Tady v ukázce vidíš jen jednu inject metodu, ale mám tam ještě 3 traity kde jsou taky inject metody. A o co mě jdě, jak správně ty komponenty vytvářet a předávat ji služby co požaduje.

Editoval akadlec (10. 6. 2014 20:04)

@akadlec: Osobně bych snížil počet závislostí. Dříve jsem měl také velké komponenty, které uměly hodně věcí, kde byl inject potřeba. Např. $translator využíváš jak? Šablona a formulář jej můžou získat sami.

Vypadá to, že to máš komplikované, takže inject je na místě.

Co bys mohl (krok na delší cestě), je používat settery v configu.

Používám Kdyby\Translation, které šablony řeší samo. Tedy v presenteru ani komponentě translator nemám, přesto mám vše přeložené.

Do formu je můžeš předat jednou pomocí továrničky, viz výše. Předávat translator komponentě jen proto, abys ho mohl předat formuláři, by bylo zbytečné.

Díky této sekci to není potřeba. Máš nejnovější verzi?

Přechod doporučuju :) Kdybys chtěl, budu na pixdevday a můžem na to mrknout.

Takže zkouším přechod na 2.2 a zatím zjištuji že ty traity co sem měl pro extension aby dostaly do templaty své služby atd již nebudou potřeba, pokud jsem to pochopil správně? Přes latte makra a helpery šablony které si initnu v DI se ke všemu dostanu, je to správný postup? Udělal jsem si třeba helper getGravatarService() co mi vrátí službu gravataru a pak v makru {gravatar} k němu přistoupím přes:

$template->getGravatarService()

je to správný postup? Takto se zbavím docela dost inject metod.

Mno já takhle tedy přepsal Gravatar ext ale teď teda nevím zda to ponechat a nebo je to špatná cesta? Osobně se mě to líbí, protože se to vše pořeší extension a user nemusí nic dělat.