Latte 3: největší vývojový skok v dějinách Nette

Hlásím první bug.

<a href="{$link->url|formatUrl}">

Property ->url obsahuje object Url a s tím počítá i filter formatUrl. Bohužel aktuálně se vygeneruje toto:

echo LR\Filters::escapeHtmlAttr(($this->filters->formatUrl)(LR\Filters::safeUrl($link->url)))

Tedy nejprve se to prožene přes safeUrl, které z toho udělá string a tím formatUrl zhavaruje.

Slevomat\Url\UrlFormatter::format(): Argument #1 ($url) must be of type Slevomat\Url\Url, string given

Myslím, že safeUrl má být poslední.

Mělo by to být opravené.

David Grudl napsal(a):

Mělo by to být opravené.

Potvrzuji. Na Slevomatu se mi povedlo aktuálně zkompilovat všech asi 2000 šablon a mám zelený build :) Pokud by vyšly u všech 4 repositářů nějaké RC verze, tak bych to i zkusil na produkci.

Dokazete mi poradit jak L3 nainstalovat ?

    "require": {
        "php": ">= 8.0",
        "nette/application": "^3.1",
        "nette/bootstrap": "^3.1",
        "nette/caching": "^3.1",
        "nette/di": "^3.0",
        "nette/finder": "^2.5",
        "nette/forms": "^3.1",
        "nette/http": "^3.1",
        "nette/mail": "^3.1",
        "nette/robot-loader": "^3.4",
        "nette/security": "^3.1",
        "nette/utils": "^3.2",
        "latte/latte": "3.*",
        "tracy/tracy": "^2.8",
        "texy/texy": "^3.1",
        "nextras/orm": "^4.0",
        "nextras/migrations": "^3.1",
        "nextras/static-router": "^2.0.1",
        "contributte/console": "^0.9"
    },
    "minimum-stability": "beta"

 {dev} .../KreativniLaborator/posobota.cz$ composer upgrade -W
Loading composer repositories with package information
Updating dependencies
Your requirements could not be resolved to an installable set of packages.

  Problem 1
    - nette/forms[v3.1.0, ..., v3.1.6] conflict with latte/latte v3.0.0-beta2.
    - Root composer.json requires latte/latte 3.* -> satisfiable by latte/latte[v3.0.0-beta1, v3.0.0-beta2].
    - Root composer.json requires nette/forms ^3.1 -> satisfiable by nette/forms[v3.1.0, ..., v3.1.6].

Use the option --with-all-dependencies (-W) to allow upgrades, downgrades and removals for packages currently locked to specific versions.

Tak minimalni verze „dev“ pomohlo, a kdyz jsem odstranil dve komponenty (contributte/mailing a nextras/mail-panel) tak vse funguje Jdu psat @Felix a @Merxes o podporu pro 3.0-beta 🐣

"require": {
        "php": ">= 8.0",
        "nette/application": "^3.1",
        "nette/bootstrap": "^3.1",
        "nette/caching": "^3.1",
        "nette/di": "^3.0",
        "nette/finder": "^2.5",
        "nette/forms": "^3.1",
        "nette/http": "^3.1",
        "nette/mail": "^3.1",
        "nette/robot-loader": "^3.4",
        "nette/security": "^3.1",
        "nette/utils": "^3.2",
        "latte/latte": "^3.0",
        "tracy/tracy": "^2.8",
        "texy/texy": "^3.1",
        "nextras/orm": "^4.0",
        "nextras/migrations": "^3.1",
        "nextras/static-router": "^2.0.1",
        "contributte/console": "^0.9"

    },
    "minimum-stability": "dev"

@DavidGrudl Na latte 2.11.2:

{* původně *}
{* výsledek: Put variables in curly brackets, replace 'timeAgo-$feedback->encodedId' with 'timeAgo-{$feedback->encodedId}' (or wrap whole expression in double quotes) *}
{control timeAgo-{$feedback->encodedId}:withTooltip, $feedback->postedAt}

{* jak jsem myslel, že to bude fungovat *}
{* výsledek: Component name must be non-empty alphanumeric string, 'ZDFhNmNmN2ItNjczZC00MTZhLWI0MjctY2MxODM3NTA0Yjc1:withTooltip' given. *}
{control 'timeAgo-' . $feedback->encodedId . ':withTooltip', $feedback->postedAt}

{* jak to teda funguje *}
{control timeAgo-{$feedback->encodedId}:withTooltip, $feedback->postedAt}

Pokud tam ale není ten render param, tak běžný concatenation funguje: {control 'xyz-' . $var}

Mít curly braces v curly braces mi však přijde divný, nejde to nějak jen s uvozovkami?

Editoval dakur (2. 5. 2022 12:58)

Ještě zvažuju jednu interní úpravu, a sice že bych opakující se

/** @var array<ArgumentNode|VariadicPlaceholderNode> */
public array $args = [];

obalil třídou a tak zaměnil na public ArgumentListNode $args

Kašlu na to :)

@dakur dvojtečka nepatří do uvozovek, to je oddělovač dvou parametrů, podobně jako čárka.

Ještě jsem narazil na jeden problém s (explode) chováním. V šabloně používám cca

"{plink nejaka_akce (explode)$presenter->getRequest()->getParameters()}":null

ale v nové verzi končím na

Cannot unpack array with string keys

Taky mám ještě dotaz zda-li půjde vlastní makra přidávat přes config.neon, tak jak to je teď, nebo se to bude muset dít přes addExtension v base presenteru?

David Grudl napsal(a):

Latte beta 4: ještě jsem

• vylepšil sandbox
• přidal možnost určovat prioritu n:attributů
• mírně vylepšil nody v Latte\Compiler\Nodes\Php

Na TODO listu mám poslední věc a to nějak se zbavit Latte\Runtime\Filters::$dateFormat.

Paráda, vše mi funguje dobře na mém IS.

David Grudl napsal(a):

@dakur dvojtečka nepatří do uvozovek, to je oddělovač dvou parametrů, podobně jako čárka.

@DavidGrudl Když zkusím kteroukoliv z těchto možností, dostanu syntax error, unexpected token ":"

{control 'timeAgo-' . $feedback->encodedId:withTooltip, $feedback->postedAt}
{control "timeAgo-" . $feedback->encodedId:withTooltip, $feedback->postedAt}
{control 'timeAgo-' . $feedback->encodedId:'withTooltip', $feedback->postedAt}
{control "timeAgo-" . $feedback->encodedId:"withTooltip", $feedback->postedAt}
{control 'timeAgo-' . $feedback->encodedId . '':'withTooltip', $feedback->postedAt}
{control "timeAgo-" . $feedback->encodedId . "":"withTooltip", $feedback->postedAt}

Tak to asi nejde.

Editoval dakur (5. 5. 2022 16:09)

@DavidGrudl Narazil jsem na to, že Latte nezvládá PascalCase konstanty, interpretuje celou expression jako string. S UPPER_CASE je vše v pořádku.

Toto hlásí Expression 'OriCMF\Admin\Presenter\BaseAdminPresenter::LayoutPath' should be wrapped in double quotes.

{layout OriCMF\Admin\Presenter\BaseAdminPresenter::LayoutPath}

Toto nehlásí nic

{layout OriCMF\Admin\Presenter\BaseAdminPresenter::LAYOUT_PATH}

Editoval Marek Bartoš (6. 5. 2022 23:29)

Ahoj, hlásím bug https://github.com/…ailPanel.php#L121:

Latte\CompileException: Unexpected tag {link} (in '.../src/MailPanel.latte' on line 61 at column 16) in /var/www/html/vendor/latte/latte/src/Latte/Compiler/TemplateParser.php:317
Stack trace:

@DavidGrudl

Mám tento kód:

$code[] = $context->format(
	<<<'CODE'
		printf(
			'<link href="%s" rel="stylesheet" media="all" integrity="%s" crossorigin="anonymous">',
			%0.raw::escapeHtmlAttr(($this->filters->staticUrl)($compiledCss->getPath())),
			%0.raw::escapeHtmlAttr($this->global->compiledAssetContentManager->getDigest($compiledCss->getPath())),
		);
	CODE,
	Filters::class,
);

V betách to fungovalo, v RC to havaruje:

Thrown exception 'Unhandled match case 's''

Vygeneruje si to tento kus kódu:

printf(
    '<link href="%0.s" rel="stylesheet" media="all" integrity="%1.s" crossorigin="anonymous">',
    %0.raw::escapeHtmlAttr(($this->filters->stati … h())),
);'

Vlastně to dává smysl, ale nějakým způsobem bych potřeboval zapsat %s. Zkoušel jsem %%s, ale to nezabralo. Je nějaký způsob?

@kukulich ono to předtím ignorovalo neznámé placeholdery jako např. %s, ale došlo mi, že bych tím zavřel cestu k možnému přidání nového placeholderu bez BC breaku, kdyby bylo potřeba. Escapování tam doplním, třeba jako to %%s. Byť teda zrovna v tomhle případě by šlo použít

$code[] = $context->format(
	<<<'CODE'
		printf(
			%raw,
			%raw::escapeHtmlAttr(($this->filters->staticUrl)($compiledCss->getPath())),
			%raw::escapeHtmlAttr($this->global->compiledAssetContentManager->getDigest($compiledCss->getPath())),
		);
	CODE,
	'<link href="%s" rel="stylesheet" media="all" integrity="%s" crossorigin="anonymous">',
	Filters::class,
);

@dakur to mě fakt nenapadlo, že ve vlákně o Latte 3 řešíš Latte 2.

David Grudl napsal(a):

@kukulich nebo mě napadá ignorovat jednoznakové placeholdery…

Se bojím, že to nebude stačit pro zápisy typu %1$s.

David Grudl napsal(a):

Bralo by to %[a-z]{3,} a %\d+\.[a-z]{3,}

To vypadá asi dobře. Nebo je asi taky možnost, že by to bylo třeba %latte.raw Ale jasně je to více psaní. Ale zas jen pro lidi, co píšou tyhle „interní“ věci.

@dakur ten můstek je jednosměrný, v2 nemusí rozumět tomu, čemu rozumí v3. Jak souvisí Latte 3 s routingem?

David Grudl napsal(a):

@kukulich ještě tam doplním %node čistě pro Nody, to %raw mi prostě nesedí…

To bych řekl, že je skvělý. To mi při prvním testování trvalo, než jsem pochopil.

@DavidGrudl hlásí mi to tuto chybu:

Error: Class "Latte\Macros\MacroSet" not found  in Bridges\ApplicationLatte\UIMacros.php(27)

Marek Bartoš napsal(a):

@IJVo Musíš si nainstalovat verze knihoven kompatibilní s novým Latte, MacroSet v Latte 3 už neexistuje

https://forum.nette.org/…jinach-nette?p=2

Díky za radu, jenže v nette/application v3.1.6-RC1 pořád v souboru Bridges\ApplicationLatte\UIMacros.php odkaz na ten MacroSet je.

	"require": {
		"nette/application": "^3.1-RC",

Mi pořád instaluje poslední nette/application (3.1.1) a umírá na Latte\Compiler, a funguje to až když jsem to změnil vysloveně na:

"nette/application": "^v3.1.6-RC1",

nightfish napsal(a):

mkoula napsal(a):
Mi pořád instaluje poslední nette/application (3.1.1) a umírá na Latte\Compiler, a funguje to až když jsem to změnil vysloveně na:

"nette/application": "^v3.1.6-RC1",

@mkoula
Nemáš v composer.json nastaveno "prefer-stable": true? https://getcomposer.org/…04-schema.md#…

Mám, ale pokud to přepnu, tak mi to hodí do dev úplně všechny balíčky a pak už nefunguje vůbec nic :-D
Každopádně díky za pomoc :-)

David Grudl napsal(a):

Jak souvisí Latte 3 s routingem?

Hm, myslel jsem, že nette/application už v té nové RC verzi vyžaduje i nový nette/routing, ale špatně jsem si přečetl/domyslel error z composeru..

Editoval dakur (11. 5. 2022 12:43)