Pojďme otestovat nette/application 3.1

Někdo se zmiňoval, že (string) $template používá (asi v souvislosti s tvorbou emailů). Pokud je to používané, tak bych stav deprecated zrušil.

Rýpat snad není potřeba, pokládat otázky je zcela ok. Dědit od té třídy se samozřejmě nemusí. Hledal jsem, jak jsi na to narazil, jde asi o formulaci z odkazovaného vlákna? To bylo myšleno jinak, v návaznosti na update, upravím to.

Přesně tak, v odkazovaném vláknu je všude class ArticleTemplate extends Nette\Bridges\ApplicationLatte\Template. Pokud ten objekt nemusí nic dědit ani implementovat tak máš palec nahoru. ;-)

Zatím jsem to zkoušel jen zběžně v mý nejsložitější appce a zdá se, že je všechno ok. Budu ještě zkoušet. Co jsem dneska z toho PoSobota streamu nepochopil je ta DefaultTemplate, můžeš to prosím nějak osvětlit, jak donutit appku, aby použila tu DefaultTemplate, když nemám vytvořenu speciální třídu pro presenter?

Chtěl jsem vytvořit BaseTemplate, kterou bych použil jako rodiče a nemohl jsem přijít na to proč mi nefungují kontroly proměnných, až jsem nakonec zjistil, že tato třída nemůže být abstract.
Je toto chování správné?

// Toto nefunguje
abstract class BaseTemplate extends Template
{
	//...
}
// Toto funguje
class BaseTemplate extends Template
{
	//...
}

Editoval Ages (1. 4. 2020 10:09)

@Ages Nejspíš ti chyběla implementace pro konkrétní template.

@DavidGrudl Davide, ved je tam zretelny BC break: Nette\Bridges\ApplicationLatte\TemplateFactory::createTemplate ma parameter naviac, takze podedena trieda nie je kompatibilna… Co takto pouzit rovnaky „trik“ so skrytym parametrom ako si spravil v Control? Nie ze by som mal rad tieto parametre (presne naopak), ale pre zachovanie kompatibility.

@Martk aby mohla byt final, musela by byt implementacia dostatocne flexibilna, aby neblo nutne kopirovat cely kod sablony kvoli malemu rozsireniu. Pouzivam rozsirenie, kde sa upravuje metoda render sposobom, ktory urcite 99.99% ostatnych uzivatelov nebude potrebovat a pokryvat tento use-case v zakladnej sablone Nette by bolo zbytocne.

@DavidGrudl Dalsi zjavny BC Break je v pripade podedenych sablon – pretoze dedia od Nette\Bridges\ApplicationLatte\Template, a teda nie je dostupna premenna $control, $presenter atd., ktore sa predtym setovali navrdo, v 3.1 sa setuju po kontrole na property_exists.

Takze za mna upravy sablon v 3.1 nie su vobec spatne kompatibilne, pretoze moj balicek rozsirujuci funkcnost sablon po upravach, aby bol kompatibilny s 3.1, nebude spatne kompatibilny s verziou pracujucou s 3.0 == BC Break.

EDIT: David, este nechapem, preco je __toString zrazu reprecated – aky je problem po ~10 rokoch so (string) $template nechapem.

Inak to vyzera, ze to po uprave Template a TemplateFactory slape normalne.

Editoval xr (29. 4. 2020 16:49)

TemplateFactory predsa sluzi na vytvaranie Template …

Mam podedenu (Nette\Bridges\ApplicationLatte) TemplateFactory a pouzivam vlastne sablony, ktore dedia od Nette\Bridges\ApplicationLatte\Template.

UPDATE: nejedna sa o hypoteticke bc breaky – teraz som to vyskusal a pisem len, na co som narazil. Su to minutove zalezitosti, ale nie su spatne kompatibilne. Jediny problem mi sposobuje ta uprava __toString.

Editoval xr (29. 4. 2020 16:56)

Neviem sice ako to pomoze obohatit situaciu, ale mozem popisat.

TemplateFactory mam upravene tak, ze mi na kazdej vytvaranej sablone zaregistruje filtre, premenne, global providers a spusti genericke dekoratory, ktore si mozu vyvojari zaregistrovat v NEONe.
Jednoducha ale velmi sikovna uprava, ktora umoznuje napriklad plosnu registraciu heleprov bez nutnosti riesit, ci su pritomne v sablonach vytvaranych prezentermi alebo v sablonach, ktore sa vytvaraju pre spracovanie mailerom.

V Template mam upravenu render metodu tak, ze s vystupom od parenta este nieco kontrolujem a pripadne korigujem vystup. Toto je dost okrajovy pripad, bohuzial specificka poziadavka od zakaznika. Ak existuje lepsi sposob pre globalny vystupny filter, neviem o nom.

Ako som pisal, je jednoduche pridat parameter a zopar atributov podla vzoru DefaultTemplate. Akurat chcem uviest na spravnu mieru, ze upravy v 3.1 _nie su kompatibilne_.

A s deprecated __toString mam problem preto, ze pouzivam paralelne aj iny sablonovaci system a teraz budem vsade musiet riesit nejaky instanceof alebo nieco podobne. Pritom jednoduchy (string) typecast nic nepokazi, navyse od 7.4 uz je mozne aj pekne vyhadzovat vynimky.

Ano, urcite je to jednym z rieseni. 👍 S velkou pravdepodobnostou je lepsie tu tovaren poskladat kompozicne ako dedit.

Myslel som, ze je dostatocne zrejme, ked uvediem, ze _dedim_ tu tovaren. Dedenie nefinalnych tried predsa nie je nicim vynimocnym. Nechcem tu robit z toho ziaden „flame“ alebo velku vedu, len som na to chcel upozornit, kedze David chcel feedback. Jedine ten typecast ma trapi.

Mozno je vhodnejsie napisat, ze _pouzitie_ je rovnake ako vo verzii 3.0, spatna kompatibilita vo vyzname semantickeho verzovania tam urcite zachovana nie je.

Ptám se jak to používáš, protože prostě rozhoduju, jestli trošku zkomplikuji život tobě teď, nebo v budoucnu sobě.

Vidím to tak, že druhý parametrý parameter createTemplate() asi dám skrytý.

Na __toString už přišly reakce, že se opravdu používá, takže ji rušit nebudu.

Co se týče rozdělení Template / DefaultTemplate, tam je hodně komplikované. Bude výhodné mít pod názvem Nette\Bridges\ApplicationLatte\Template čistou šablonu, od které se teď bude asi dost dědit a ponechat víceméně současné řešení i za cenu BC breaku, který se dá velmi snadno vyřešit. Zvažuju ještě přejmenovat DefaultTemplate na LegacyTemplate.

Verzi rc máme zhruba týden nasazenou na https://www.megapixel.cz a bez problémů. Všechno funguje podle očekávání. Trochu nás zaskočila metoda ->add(), kterou jsme měli použitou a zároveň jsme používali makro {templateType} v šabloně. Metoda ->add() už ve vlastní šabloně není, možná bych to přidal do release notes, společně s ostatními proměnnými jako $user, $presenter, na které byli lidé nejspíš zvyklí

Pokud dnes aktualizuji Nette s tím, že composer vypadá následovně a dojde jen k aktualizaci desetinkových verzí.

"php": "7.3.*",
"nette/application": "^3.0.6",
"nette/bootstrap": "^3",
"nette/caching": "^3",
"nette/database": "^3",
"nette/di": "^3",
 ...

Nainstaluje se php-generator 3.5 a přestane to na PHP7.3 fungovat kvůli pojmenovaným paramterům v DI.

Řešení je snadné, vynutit si verzi generatoru na 3.4.

"nette/php-generator": "3.4.*",

Spíš mi napadá, jestli někde nedošlo k špatnému popsání composer chainu závislostí.

Řekl bych, že to bude tady:

public function afterCompile(Code\ClassType $class)
{
	$init = $class->getMethods()['initialize'];
	$config = $this->validateConfig(['name' => 'addDuplicator', 'container' => \Code\Component\Container::class, 'form' => \Code\Component\Form::class]);

	$init->addBody(Duplicator::class . '::register(?*);', [$config]);
}

A výstup: yntax error, unexpected ':', expecting ')' na řádku 4201: name: 'addDuplicator', – asi jde o tu dvojtečku.

Editoval Pavel Kravčík (4. 11. 2020 8:14)

Super, díky – funguje to. :)

@DavidGrudl Spolu s getContext() jsi deprecatnul i unformatPresenterClass(). Je za to někde nějaká náhrada? Máme v routeru helper, který převádí XyzPresenter::class na AbcModule:Xyz notaci.

getContext() je deprecated, ale zároveň přestal být final, abyste si ji klidně mohli třeba v BasePresenteru přepsat a bude fungovat i nadále. Point je v tom, že třída UI\Presenter nebude mít kontejner jako závislost, protože ho už devět let nepotřebuje.

unformatPresenterClass() je deprecated, protože jsem přišel na to, že to není úplně spolehlivá a nenašel jsem rychle snadný způsob, jak ji napsat spolehlivou a framework se bez ní obejde.

David Grudl napsal(a):

unformatPresenterClass() je deprecated, protože jsem přišel na to, že to není úplně spolehlivá a nenašel jsem rychle snadný způsob, jak ji napsat spolehlivou a framework se bez ní obejde.

Jenže zvenku se nedá dostat k $mapping, nemůžu si tedy funkci zkopírovat do svého kódu. Jak to mám řešit?

Editoval dakur (7. 1. 2021 10:15)

dakur napsal(a):

Jenže zvenku se nedá dostat k $mapping, nemůže si tedy funkci zkopírovat do svého kódu. Jak to mám řešit?

Vlastní PresenterFactory nebo nepada/presenter-mapping.

@Martk fungují takto anotace @function a @filter. Nic jako tvé LatteProperty tam není a ani to nemám v plánu.

Visela tam nějaká stará verze web-projectu, udělal jsem nový tag a už by to mělo fungovat.