Injektovanie DB to parent modelu

V potomkovi vůbec konstruktor nepiš. Pokud ho potřebuješ, musíš takto:

<?php
class UserModel extends BaseModel
{
	public function __construct(Nette\Database\Context $db, $somethingElse)
	{
		parent::__construct($db);
		# ...
	}
}
?>

P.S.
Pokud by $somethingElse nešlo autowirovat (chtěl bys ho předávat v configu ručně), tak je dobré napsat ho jako první parametr. V configu bys pak měl:

services:
	- NS\UserModel(%somethingElse%) # databáze jako druhý parametr se aurowiruje sama
	- NS\UserModel(..., %somethingElse%) # teď bys to musel dělat takhle

Editoval Šaman (6. 8. 2014 22:29)

Ne. Jak jsem psal, základní řešení je v potomkovi vůbec konstruktor nepsat.

Anebo přesvědčit Davida, aby inject metody povolil všude. Mě by se to občas hodilo, ale nechci to používat, když by to vyžadovalo hack Nette.

BaseModel je podle mě pěkná prasárna. Pokud to používáš jen proto, že jsi líný autowirovat si databázi vždycky, tak to není dle mě dobrý přístup a znásilňuje to dědičnost + si pak zavařuješ na ten problém s konstruktory.

Base třídy jsou vhodné u presenteru (tam je to skoro až nevyhnutelné) a hodí se i u komponent, ale do modelu bych je teda nedával.

@Šaman
tým „do nich ale už nic dalšího nepridáva“ myslíš, tých potomkov. Teda, že nepotrebujú prepisovať konštruktor.

@JiříNápravník
Podľa mňa to má presne ten istý význam ako v presenteroch. To že je problém s konštruktorom je podľa mňa iná vec.
Asi by to vyriešilo, keby mal BaseModel metódu database()(dá sa injectovať do metódy?).

Editoval Čamo (7. 8. 2014 12:46)

Pokud je to Repository, tak by tam nějaká Base třída být mohla, protože tam pak pravděpodobně budou i nějaký metody jako findBy apod.

Nicméně já to pochopil spíše tak, že to je spíše fasádní/servisní třída a tam tedy base třídy nepatří dle mě.

A čo injektnúť to do toho BaseModelu cez metódu injectDatabase()?
Našiel som v dokumentácii ako sa to robí
ale u seba to neviem rozchodiť. Nerozumiem tej syntaxy v NEONe
U mňa Neon vyzerý takto:

services:
	- App\RouterFactory
	router: @App\RouterFactory::createRouter
	- \App\Model\UserModel
	- \App\Model\GeneralModel
	- \App\Model\BaseModel
	inject: yes	# nefunguje ani s odsadením píše "expects to be callable or stdClass or null, boolean given"

V dokumentácii je zápis:

services:
    service3:
        class: App\Service3
        inject: yes

čo je úplne iná syntax. Prečo tam ja mám pomlčky a prečo sa v docu tie services pomenovávajú… nechápem.

@DavidKudera
Díky, funguje to.

A je to dobrý nápad, takto to injektovať?

EDIT:
omyl nefunguje, len ešte neviem prečo…

Editoval Čamo (7. 8. 2014 15:25)

osobne to riesim tak, ze mam ModelDependencies ktory predavam base modelu a potomkom, ktore prepisuju base konstruktor

na @inject hocikde funguje jednoduchy hack

<?php
class XXX
{
	public function __construct(DI\Container $container) {
		$container->callInjects($this);
	}
}
?>

Editoval japlavaren (7. 8. 2014 16:32)

A kde k tomu $database->table() přistupuješ? zkus si třeba do té inject metody dát dump té databáze, jestli se to v pohodě zavolá. A kdyžtak konstruktor se volá jako první a ne inject, takže jestli máš to volání table v konstruktoru, tak to fungovat určitě nebude

Čamo napsal(a):

Prečo mi nefunguje ten inject?

services:
	-
		class: App\Model\BaseModel
		inject: yes

Takhle to nefunguje! Ty přece nepoužíváš instanci BaseModelu, ale nějakého konkrétního modelu, dejme tomu UserModelu. A právě teď jsi kontejneru řekl, ať vytvoří instanci BaseModel a nainjectuje do ní nějaké závislosti. Ale pak už tuhle instanci nepoužíváš (správně by měla být ta třída abstraktní, takže nevytvořitelná).
Ty tohle musíš napsat k tomu UserModelu (a ke všem dalším potomkům BaseModelu)!
Proto je v tomto případě jednodušší řešit to konstruktorem.

Editoval Šaman (8. 8. 2014 11:27)

@Šaman
Ok tak to nechám v tom konštruktore.
Díky.