Prosím o otestování nový verzí Application & spol

Díky za práci na nových RC.

Pokud mám vlastní form fieldy extendující ty existující, pak to hází Compile Error, protože přibyly return types.

Taky zmizelo PresenterFactory::unformatPresenterClass(), jak už jsem řešil zde. Buď si ji tam teda budu muset přidat sám (hodně volatile), nebo si napsat vlastní presenter mapper.

Editoval dakur (25. 1. 12:05)

@dakur zkusil jsem je dát pryč. Narážíš teď na nějaký konflikt?

@VáclavPávek už si nepamatuju u čeho to konkrétně selhává…

Ten conflict se musí změnit na správnou verzi, žádnou option se bohužel ignorovat nedá.

unformatPresenterClass() taky používám a nikdy jsem na problém nenarazil.
Nehlásil to třeba někdo, kdo měl custom mapping? Třeba librette/presenter-factory mohlo být problém, protože řeší format, ale unformat ne https://github.com/…ormatter.php

@DavidGrudl composer už ok.

Kde to ještě padá:

• TextBase::__construct() – přibylo string|\Stringable|null v prvním parametru
• BaseControl::setDisabled() – přibyl return type static

Pak mi ještě padá vlastní template factory, která dědí z Nette\Bridges\ApplicationLatte\TemplateFactory, konkrétně na return typu createTemplate(), protože tam předtím žádný nebyl a já měl uvedený Nette\Application\UI\Template.

Máme uvedený konstruktor v traitě jako abstract, protože v té traitě máme statický pojmenovaný konstruktor addTo(), který ten hlavní volá. A protože je to traita, tak nezná kontext, abstraktní konstruktor tedy slouží, aby new static($caption) v IDE ukazovalo správně parametry (phpstan by si s tím měl asi poradit i bez toho).

trait AddToFormUsabilityConstructorForTextBase
{

	abstract public function __construct($caption = null);

	public static function addTo(Form $form, string $name, ?string $caption = null, ?string $insertBefore = null): static
	{
		$form->addComponent($input = new static($caption), $name, $insertBefore);
		return $input;
	}

}

final class MyTextInput
{
	use AddToFormUsabilityConstructorForTextBase;

	// ...
}

MyTextInput::addTo($form, 'myTextInput')
	->setDefaultValue('...')
	// ...
;

Asi to není úplně běžné použití, tak záleží, kde leží hrana BC breaku :)

@DavidGrudl: Je nějaký důvod proč musí být final class ControlGroup? Používáme nadstavbu nad formuláři a ControlGroup si rozšiřujeme o parametry navíc pro naše automatické vykreslení přes BS5 (duplikátory, dependendy, whisperery apod.). Ve verzi 3.2 už nefunguje.

Když to zjednoduším na příklad:

class ControlGroup extends \Nette\Forms\ControlGroup
{
	protected ?string $class = null;

	public function setColor(string $color): self
	{
		return $this->setOption('color', $color);
	}
}

// Následná nadstavba

pf renderForm()
if($groupTitle || $this->getTitle())
{
	$groupColor = $group->getOption('color') ? ' ' . $group->getOption('color') : null;

	$carHeader = Html::el('div')
		->class('card-header' . $groupColor)
    	->setHtml($groupTitle ?: $this->getTitle());
}

// Zápis ve formuláři
pf createComponentForm(): Form
{
	...
	$form->setTitle('pes');
	$form->setColor('success');
	...
	//další barevná group apod.
	...
}

Ja bych zase uvital dat final pryc u nette/application/src/Application/LinkGenerator.php a hlavne u metody isLoggedIn() v nette/security/src/Security/User.php kde musim mit patch jenom kvuli tomu final.