Vydání Nette 2.1 do konce roku s maximálním zachováním BC

Šlo mi jen o automatické volání inject metod, které se vždy dá zapnout v konfigu pomocí inject: true nebo setup: [injectMethod]. Týká se to teda i property s anotací @inject.

U presenterů by to fungovalo automaticky nadále.

David Grudl napsal(a):

• vypustil zkrácené {#block}

preco? v dev verzii to ostane?

@DG: Důkladně jsem si prošel svůj seznam co bych v Nette chtěl a vyfiltroval co bych ve 2.1 rád viděl vyřešené. Většinu věcí jsem ale i tak vypustil protože buď nejsou tak horké nebo je na ně pozdě. Zbylo mi tohle:

DIC factories

Generované továrny jsou sice fajn, ale potřebovaly by vyřešit dva drobné neduhy abych je považoval za připravené pro stable verzi:

Automatické přidávání parametrů – https://github.com/…e/issues/873.

Zohlednění use statements – Uvádění celého typu včetně namespacu v anotaci @return mi dost vadí, zvykl jsem si používat use na všechno (stejně jako to dělá symfony). Navíc mi to IDE furt opravuje a pak to hází výjimky. Pokud jde o implementaci tak doctrine/annotations to umí.

Autorizátor a identita

(Chápu že jsem s tím asi už otravný.) Přemýšlel jsem nad tím a tvůj koncept imho nedává úplně smysl dokud v Nette není nativně možnost oddělené autentizace pro různé sekce (typicky moduly), existuje jen nějaké extension MultiAuthenticator a moc se mi nelíbí. Když bych to potřeboval tak bych zřejmě vytvořil druhou službu typu Nette\Security\User a Nette\Http\UserStorage (s jinak nastaveným setNamespace) + samozřejmě druhý Authenticator a Authorizator. (<OT>Je už v extensions funkční a někde zdokumentovaný lokální autowiring?</OT>)

Pro tuto chvíli mi dává smysl ta identita s tím že se později přepíše autentizace i autorizace nebo alespoň přidá lepší podpora pro více authenticatorů. Jsi-li ochotný ten můj PR mergnout, dej vědět ale ještě s tím počkej. Jsou tam dvě drobnosti které bych musel fixnout.

Editoval enumag (6. 11. 2013 22:13)

enumag napsal(a):
Zohlednění use statements – Uvádění celého typu včetně namespacu v anotaci @return mi dost vadí, zvykl jsem si používat use na všechno (stejně jako to dělá symfony). Navíc mi to IDE furt opravuje a pak to hází výjimky. Pokud jde o implementaci tak doctrine/annotations to umí.

Také jsem potřeboval a v nadšení jsem to implementoval jako samostatnou třídu AliasExpander. Po čase jsem samozřejmě zjistil, že to řešil kde kdo a v principu všichni stejně.

Ale nasazení je hodně o BC. Představte si kód:

namespace Test;

use MyApp\MyForm;

class Foo
{
	/**
	 * @return Nette\Forms\Form
	 * nebo
	 * @return MyForm
	 */
	public function createForm()
	{
	}
}

Když se „zapne“ uplatnění namespace a aliasu v anotacích, co tedy metoda vrací?

• Test\Nette\Forms\Form nebo Nette\Forms\Form?
• MyForm, Test\MyForm nebo MyApp\MyForm?

aneb těžko odhadnout co autor myslel. Musela by se zavést nějaká konvence, například:

• absolutní název třídy v anotaci vždy začína \ (backslash), zbytek se expanduje
• expanduje se pouze ClassName::class (à la PHP 5.5), na zbytek se nesahá
• nebo něco jiného

Podle mého, tohle do stable nepatří.

Zohlednění use statements: Rozhodně jsem proti. Podle mne tam patří třída včetně namespaces (tzv. celý název):

• @return \Nette\Utils\Html – ANO
• @return Html – NE

@enumag: Pravdou je, že továrny jsem použil párkrát a IDE používám zřídka. A ani nevím jak moc správně lidé anotace píšou. Chtěl jsem jen poukázat na možný BC break, v praxi půjde zřejmě jen o to zpětné lomítko na začátku.

@bazo: A jak vypadá konfigurace té factory v neonu? ;-)

enumag napsal(a):

@bazo: A jak vypadá konfigurace té factory v neonu? ;-)

tak tam uz su zamozrejme cele namespacy, ale to nie je problem, kedze je to copy paste a autocomplete. ale neviem ako by to mohlo inak fungovat

attributeForm:
	class: App\Components\AttributeForm
	implement: App\Components\Factories\AttributeFormFactory

tak takto to je, potom suhlasim.

potom by sa mohlo rovno pisat

factories:
	attributeForm: App\Components\Factories\AttributeFormFactory

ako pri sluzbach

Editoval bazo (7. 11. 2013 14:01)

Na use jsem zrovna narazil min. tyden a chtěl jsem to napsat do requirements ;-)

Milo wrote: Moc rád bych uvolnění stable verze klidně bez všech fičur co jsi vypsal.

Souhlas, přesně o to jsem se snažil před rokem.

Možná by nebylo od věci odkázat na https://doc.nette.org/…tions/to-2-1, když už to bylo před časem sepsáno.

Takže diskuse, co z masteru odebrat, aby se mohla vydat 2.1, se zvrhla v diskusi co ještě všechno přidat :-) Přidávat se bude pořád, dokud bude Nette existovat. To teď neřešme.

Připravil jsem Nette Framework 2.1.0 RC (Github, distribuce).

Kompletní popis změn zatím neexistuje (nejblíž k němu má https://doc.nette.org/…tions/to-2-1), nicméně tato verze slouží spíš k připomínkování pro insidery.

Do finální verze bych chtěl ještě přejmenovat Nette\Database\SelectionFactory. Ale nechtěl bych už přidávat nic, co by odkládalo vydání.

Ugh, ten revert změn okolo formulářů (zejména validace) teda bolí…

Že jsem tak smělý, v té 2.1 bude jaké preferované injectování servis do presenteru? @inject, injectBlabla(…) nebo oboje? Trochu se v tom ztrácim.

v presenterech pude stale pouzivat inject* metody i @inject anotace, viz https://github.com/…rFactory.php#L66

vypnuto bylo pouze injectovani timhle zpusobem do sluzeb

edit: k tomu co je preferovano: samozrejme, nejlepsi je konstruktor injection. pri ruznem dedeni atd. by ale pekne rychle vznik depedency hell. pokud si dobre vzpominam, tak David rikal, ze preferovanym zpusobem budou @inject anotace, inject metody vsak zustanou stale podporovane. alternativne muzes pouzit kdyby/autowired coz jsou chytrejsi @inject anotace

Editoval matej21 (20. 11. 2013 14:27)

Označil a vydal jsem 2.1.0 RC. Více info https://forum.nette.org/…-rc-released

Venku je RC2. Asi krom jediné věci (UI\Control::createTemplate) jsem s ní spokojen a vidím v ní podstatný pokrok oproti 2.0. Přitom s přechodem z 2.0 by neměl být problém. Stačí v podstatě změnit:

• Nette\Config\Configurator → Nette\Configurator (zvažuji, zda nedat do fw obě třídy)
• Nette\Database\Connection → Nette\Database\Context
• Message::send() → Mailer::send($message)
• při manuálním vykreslování formuláře dát pozor na Checkbox a nezapomenout na chybové zprávy u jednotlivých inputů
• Nette\Utils\PhpGenerator → Nette\PhpGenerator
• Nette\Config\CompilerExtension → Nette\DI\CompilerExtension

@Jan Tvrdík++

David Grudl napsal(a):

Stačí v podstatě změnit:

• Nette\Config\CompilerExtension → Nette\DI\CompilerExtension

Takže přestanou fungovat všechny doplňky, které mají extension. Dík.

Ohledne te NDB, neni to tak optimisticke, jak si, Davide, napsal. Treba backjoin api je nove :)

Tak co jsem si ted udelal diff na database slozkou, tak to vypada na zakladni oblasti:

• backjoin api
• kompozice vs. dedicnost
• context, a rename puvodnich metod z pdo
• selection::group() ma nyni selection::having()

@motorcb: To znamená, že verze 2.2 (master, @dev) je vyvíjena paralelně s přípravou verze 2.1.0.

@dg: Backjoin API: dvojtečka se píše před název tabulky místo mezi název tabulky a název sloupce. Metoda group měla dříve dva argumenty $group a $having, nyní má jen jeden + přibyla metoda having.

Editoval enumag (17. 12. 2013 23:32)

U NDB jste ještě zapomněli na docela zásadní BC break, a to, že u ActiveRow nelze nastavit nové hodnoty záznamu a pak dát jen update():

2.0:

$row->some_column = 'some new value';
$row->update(); //note: provede se jen update

2.1:

$row->update(array('some_column' => 'some new value')); //note: provede update a vzapeti hned select *