PHPStan ITemplate vs Template

před 6 lety
Sitole
Člen | 39
+
+1
-

Ahoj,
při spouštění PHPStanu jsem konkrétně na levelu 2 narazil na problém při kterém se mu nelíbí můj způsob renderování komponent za pomoci $this->template->render('template.latte'), protože v anotacích u Nette\Application\UI\Control se definuje jako ITemplate tak i jako Template. ITemplate nepřijímá do metody render žádné parametry, ale Template přijímá cestu k šabloně. Předpokládám, že to má nějaký důvod, ale přišlo mi hezčí tohle jednořádkové volání namísto ne tak hezkého:

$this->template->setFile(__DIR__ . '/templates/Selector.latte');
$this->template->render();

Z jakého důvodu tomu tak je, popřípadě nešlo by se toho zbavit či přidat nějakou vyjímku třeba do balíčku PHPStan-Nette?

před 6 lety
CZechBoY
Člen | 3608
+
0
-

Zkontroluj, že v $this->template je to co očekáváš a když ne tak vyhoď třeba výjimku.

$template = $this->template;
if (!$template instanceof \Nette\Bridges\ApplicationLatte\Template) {
    throw new Exception('pls Application template');
}

$template->render('template.latte');

Samozřejmě to může být trošku ubíjející to všude psát…
Řešení jsou asi dvě až tři:

  1. v BaseControl anotaci přepsat
  2. udělat si rozšíření reflexe třídy a phpstanu říct že v $this->template je Application template
  3. metoda getTemplate(), která vykoná kod co je nahoře

Editoval CZechBoY (22. 10. 2018 21:47)

před 6 lety
ali
Člen | 342
+
+1
-

Ja to resim takto

/**
 * @property \Nette\Bridges\ApplicationLatte\Template $template
 */
class Component extends \Nette\Application\UI\Control
{
}

Editoval ali (23. 10. 2018 5:54)

před 6 lety
Sitole
Člen | 39
+
0
-

Použití skrze BaseControl bude asi v mém případě nejlepší a zároveň nejhezčí řešení.

Ještě malý dodatek. Jaky to tam má důvod a popřípadě proč se toho nezbavit úplně už na úrovni knihovny?

před 6 lety
MajklNajt
Člen | 498
+
0
-

ja to používam takto:

$this->template->setFile(__DIR__ . "/template.latte")->render();

tých 11 znakov naviac ma nazabije

před 6 lety
Felix
Nette Core | 1245
+
+2
-

@Sitole Nedavno jsem o tom psal clanek.

https://f3l1x.io/…bo-controlu/

před 6 lety
Sitole
Člen | 39
+
0
-

@Felix To vypadá jako fajn řešení, ale stále mi neodpovídá na otázku proč se u Control definuje obojí ještě když Template implementuje ITemplate.

Ale děkuji, použiji.

před 6 lety
Milo
Nette Core | 1283
+
+1
-

Továrna na šablony nemusí vracet Latte šablonu. Jen obecnou ITemplate. Správně by tedy mělo být v anotaci jen to. Jenže všichni používají Latte, tak je tam i druhá anotace, aby IDE nenadávalo při setFile() a podobných. Ze stejného důvodu se tam přidává stdClass, aby IDE nenadávalo při setování proměnných do šablony.

před 6 lety
Tomáš Votruba
Moderator | 1114
+
+1
-

Braní něco opravit toto přímo v Nette? Přijde mi to jako zbytečný pain-points, když se o tom píšou opakované články a je potřeba přidávat traity a upravovat/přidat v každém projektu base control i base presenter.

před 6 lety
CZechBoY
Člen | 3608
+
+3
-

Ono není co opravovat… prostě Nette používá interface aby si mohl použít jakoukoliv jinou implementaci. To že téměř všichni používají Latte je věc jiná.

před 6 lety
Tomáš Votruba
Moderator | 1114
+
0
-

To je ok, jen public metody navíc oproti interface tam nemají co dělat.

před 6 lety
MajklNajt
Člen | 498
+
+1
-

Tomáš Votruba napsal(a):

To je ok, jen public metody navíc oproti interface tam nemají co dělat.

prečo nie? interface mi určuje, aké medóty musia byť implementované, a nelimituje ma, že nemôžem použiť iné public metódy…