Pojďte otestovat Nette 2.4 RC

Úplně správně by to mělo být try_files $uri $uri/ /index.php$is_args$args;

A je to jako best practice někde u quickstartu nebo obecně v dokumentaci? Nebo v nějakym ukázkovym konfigu jako součást repozitáře? (Nenašel jsem, předpokládám, že není. Commitnul som)

Informaci o té změně a potenciální nefunkčnosti bych pak taky rád viděl v nějakých poznámkách k vydání, prosím.

mrtnzlml napsal(a):

Tak zatím nevím o co s tou rychlostí jde, ale podle Tracy se stránka načítá normálně rychle. Jen TTFB je třeba 15 sekund. Nedávno řešil někdo něco podobného, pokud si dobře vzpomínám. Je tomu tak?

Nemas zapnuty Session panel? Nam stranku hrozne zpomalovala Tracy data, ktera se v nem vykreslovala. Chtel jsem poslat pull request, aby se ta data ignorovala, ale jeste jsem to nestihl :(

@DavidGrudl Metoda Form::beforeRender() mi zní jako dobrý nápad. Zrovna minulý týden jsme ve Slevomatu řešili, jak do všech formulářů hromadně v některých situacích přidat nějaké prvky. Tohle by nám pomohlo.

Jestli to pomůže tomu replicatoru, se nějak nemůžu rozhodnout. Zdrojáky jsem viděl poprvé, když jsem se ho snažil minulý týden patchnout :) Primární problém byl v tom, že se moc brzo volalo receiveHttpData(), takže se nevykonal kód v if ($action instanceof Link) { a formuláři se pak vygenerovala nesmyslná action.

Ajax Tracy je úplne skvelý nástroj, ale je možné ju pre niektoré dotazy pozastaviť Tracy?
Periodicky cez JS kontrolujem notifikácie a zakaždým sa nanovo generuje aj Tracy.

Nevím, jestli někde nedělám chybu, ale dynamické filtry nefungují při aplikaci na block?

services:
	latte.latteFactory:
		setup:
			- addFilter( null, 'Filters::common' )

class Filters
{
	function loader($filter, $value)
	{
		if (method_exists($this, $filter)) {
			return call_user_func_array([$this, $filter], array_slice(func_get_args(), 1));
		}
	}

	function my_filter($s)
	{
		return $s;
	}
}

{block |my_filter}Test{/block}

Končí výjimkou LogicException: Filter |my_filter is not defined..

Nasadil jsem poslední RC na Slevomat a všechno funguje. A potvrzuju, že i kdyby/replicator nyní funguje bez jakéhokoliv patche.

Jen commit https://github.com/…a6df11826142 mi teda dal trochu zabrat, ale aspoň jsem náš kód zase trochu vyčistil :)

Editoval kukulich (7. 6. 2016 22:53)

@kukulich a kvůli něčemu jinému, než filtrem nad $template? Toho jsem měl v šablonách taky mraky a vyřešil jsem to regulárem \$template->(\w+)\(([^,)]+)(,[^)]+)?\) → ($2|$1$3)

Složitější byly případy, když jsem si dával do šablony „jakože filtry“, např. $template->rss, což samozřejmě lze jednoduše přepsat na ('feed.xml'|rss), ale vypadá to blbě.

Samozřejmě se to dá řešit vytvořením metody na presenteru $presenter->rss(), ale to není přenosné do více šablon. Nebo předáním callbacku do proměnné $rss a pak voláním $rss('feed.xml'). Ale to nelze nastavit globálně na TemplateFactory. Nebo přes provider, který lze nastavit globálně, a volat $this->global->rss('feed.xml'). A nebo mergnout podporu pro vlastní funkce, takže by se callback nastavil přes $latte->addFunction('rss', ...) a používal jako rss('feed.xml'), ale fakt si nejsem jist, jestli je tohle úplně potřebné.

@DavidGrudl Rozdíl je v tom, že u {$var} můžeš explicitně zapnout manuální mód, pomocí |noescape, což u {include} nefunguje.

Je tady ještě jedna věc, kterou se snažím vyřešit od 2.1, a tou jsou povinné a nepovinné prvky formuláře. V podstatě jde o to, že

$form->addText('email')->addRule($form::EMAIL);

vytváří implicitně povinné políčko, protože pokud není vyplněné, validace neprojde kvůli tomu, že prázdný řetězec není platná emailová adresa. Ale zároveň nemá nastaveno setRequired() a z toho vyplývá, že chybí například třída v HTML atd.

…pokračování

@DavidGrudl Ještě k těm vlastním funkcím. Osobně mi to potřebné nepřijde. My pravidelně používáme callbacky do proměnné, když je to něco, co se týká jednoho presenteru. Když to má být znovupoužitelné, tak prostě máme filtr (klidně jako ('feed.xml'|rss)) a nebo si napíšeme vlastní makro {rss 'feed.xml'}.

Ahoj, ještě bych chtěl otevřít otázku přidání konfliktů do composer.json.

Jeden příklad za všechny je vztah latte – application. Každý z těchto balíčků je sice možné používat samostatně bez toho druhého, tedy není mezi nimi žádná tvrdá závsilost. Je ale prakticky nemožné(*) používat dohromady třeba latte 2.4 a application 2.3 – tahle skutečnost není nikde v composer.json podchycena, a v případě, že to zkusíte, tak aplikace vyhazuje podivné chyby bez zjevné příčiny (konec konců i přímo v tomto vlákně na tom pár lidí na tom pohořelo).

Řešení je celkem jednoduché – do composer.json se přidají konflikty pro nekompatibilní verze. Otázkou je, kde všude by se měly přidat? Dvojice latte-application je myslím celkem jasná, ale co zbytek?
Existují případy, kdy používáte oba balíčky, ale jejich integraci mezi sebou si zajišťujete sami (nebo vůbec nepoužíváte)? Třeba latte a forms?

(*) Museli byste re-implementovat v podstatě celý Bridge znova.

Editoval xificurk (13. 6. 2016 17:43)

David Grudl napsal(a):

@xificurk Řekl bych, že když nette/application 2.4 vyloučí latte, nette/forms a nette/di ve verzi < 2.4, tak by to mohlo být dostačující.

Tohle imho nestačí, protože to nevyloučí instalaci např. latte 2.4 a application 2.3.

@DavidGrudl V podstatě by měly být konflikty symetrické na obou stranách, tj. i v di a forms přidat konflikt s application < 2.4.
Podobný problém, ale existuje i třeba u vazby forms-latte (formulářová marka nově používají provider z latte 2.4)… Ale tady bych se už trochu bál, že někdo bude chtít tyto dva balíčky používat bez vzájemné integrace a nevím, jestli je konflikt úplně košér.

Nette\DI\Statement::setEntity is deprecated, change Statement object itself.
doctrine to hlasi dal som „minimum-stability“: „dev“

@enumag pises ze do starsej vetve.. ako si teda mam aktualizovat na tu najnovsiu ?

erikbalog napsal(a):

@enumag pises ze do starsej vetve.. ako si teda mam aktualizovat na tu najnovsiu ?

kukni sem:http://zlml.cz/…th-nette-2-4

@TomášJacík a latte kod vypadá jak? :-)

@TomášJacík asi za to muze nextras/forms https://github.com/…utMacros.php#L30, ktere registruje ten label bez flagu jako nette/forms

Alternativa k ~$form::PATTERN někdy bude nebo mám použít negative lookahead?

Antik napsal(a):

Alternativa k ~$form::PATTERN někdy bude nebo mám použít negative lookahead?

dá se pořád do kladného $form::PATTERN zapsat záporný regulární výraz (třeba používám na zakázaná uživatelská jména již při registraci takovýto regexp a všechno se pak validuje už v prohlížeči u klienta):

pattern="^(?!(admin|administrator|servis|…)$).*"

(atribut input elementu)

Jenom hlasim, ze componette s konfiguraci:

Funguje v pohode.

"nette/application": "dev-master",
"nette/caching": "dev-master",
"latte/latte": "dev-master",

S timhle, hlasi chybu.

"nette/application": "~2.4.0",
"nette/caching": "~2.4.0",
"latte/latte": "~2.4.0",

Editoval Felix (30. 6. 2016 11:27)

@blaztar, @DavidGrudl

Dekuji, pomohlo. Mozna by chtel zase pridat konflikt, ze latte koliduje s nette/caching 2.4? Ale to necham Davide na tobe, moc tomu jeste nerozumim.. :-)

Dříve jsem měl ve formuláři

$this->addText('identifier', 'Identifier')
  ->addRule(self::PATTERN, 'Položka "%label" musí obsahovat 7 číslic.', '([0-9]\s*){7}');

Takže při nevyplnění nebo špatném vyplnění uživatel dostal jednu chybovou hlášku, ze které bylo ihned zřejmé, co je špatně.

Teď ale musím psát

$this->addText('identifier', 'Identifier')
  ->setRequired('Položka "%label" musí být vyplněná.')
  ->addRule(self::PATTERN, 'Položka "%label" musí obsahovat 7 číslic.', '([0-9]\s*){7}');

čímž se zobrazí zbytečně hlášky dvě. Není to škoda?