Rychlý dotazník pro verzi 2.0

Upozornění: Tohle vlákno je hodně staré a informace nemusí být platné pro současné Nette.
David Grudl
Nette Core | 7823
+
0
-

Měl bych několik rychlých a jednoduchých dotazů na téma „co změnit ve verzi 1“. Ocenil bych vaše stručné odpovědi.

  1. mají se logy výjimek v Debug komprimovat? zrealizováno
  2. defaultní Html::$xhtml má být TRUE nebo FALSE? zůstane TRUE
  3. přejmenovat HttpUploadedFile::getImage() na vhodnější toImage() ? zrealizováno
  4. sjednotit názvy obdobných metod formulářů setEmptyValue() a skipFirst() (asi na setEmptyValue)? bude se řešit
  5. dát u addText() a addPassword() jako třetí parametr $maxLength (nyní je čtvrtý) bude se řešit s větším updatem formulářů
  6. jsou třeba IHttpRequest, IHttpResponse a IUser?
  7. nahradit bool parametry metod za bitové příznaky (např. $user->logout(TRUE) na $user->logout(User::CLEAR_IDENTITY)) bude se nějak řešit, chce to promyslet
  8. změnit Html::render() tak, aby obsah vykreslil element namísto vrátil? ještě to chce promyslet
  9. ponechat ve frameworku Nette\Web\Ftp? přesunuto od doplňků
  10. změnit čtvrtý parametr Permission::allow() z IPermissionAssertion na callback? zrealizováno
  11. zrušit abstraktní třídu AutoLoader?
  12. hodí se k něčemu FreezableObject?
  13. je třeba pro chybějící soubor/adresář disponovat hned třema výjimkama IOException & FileNotFoundException & DirectoryNotFoundException?
  14. je k něčemu ArgumentOutOfRangeException?
  15. podporovat FireLogger? (či nahradit jím FirePHP?)
Honza Marek
Člen | 1664
+
0
-

David Grudl napsal(a):

Měl bych několik rychlých a jednoduchých dotazů na téma „co změnit ve verzi 1“. Ocenil bych vaše stručné odpovědi.

  1. mají se logy výjimek v Debug komprimovat?

Volitelně. Ideální je, pokud při zprovozňování webu se to komprimovat nebude, těch chyb bývá víc a rozbalování otravuje. Při provozu se to klidně komprimovat radši může.

  1. defaultní Html::$xhtml má být TRUE nebo FALSE?

Nevim, asi false. Hlavně ať je to stejné jako ve skeletonech a tak.

  1. přejmenovat HttpUploadedFile::getImage() na vhodnější toImage() ?

Asi bych nechal get.

  1. sjednotit názvy obdobných metod formulářů setEmptyValue() a skipFirst() (asi na setEmptyValue)?

Určitě.

  1. dát u addText() a addPassword() jako třetí parametr $maxLength (nyní je čtvrtý)

Radši ne, těch rozbitých webů…

  1. jsou třeba IHttpRequest, IHttpResponse a IUser?

Nepoužil jsem, ale IUser by se možná někomu hodit mohlo.

  1. nahradit bool parametry metod za bitové příznaky (např. $user->logout(TRUE) na $user->logout(User::CLEAR_IDENTITY))

Těžko říct, je to čitelnější, ale víc psaní.

  1. změnit Html::render() tak, aby obsah vykreslil element namísto vrátil?

Určitě. Normální člověk beztak Html echuje a toStringuje.

  1. ponechat ve frameworku Nette\Web\Ftp?

Extras asi stačí.

  1. změnit čtvrtý parametr Permission::allow() z IPermissionAssertion na callback?

Nevim.

  1. zrušit abstraktní třídu AutoLoader?

Ano.

  1. hodí se k něčemu FreezableObject?

Já z toho dědim v Ormionu recordy a collection.

  1. je třeba pro chybějící soubor/adresář disponovat hned třema výjimkama IOException & FileNotFoundException & DirectoryNotFoundException?

Proč ne.

  1. je k něčemu ArgumentOutOfRangeException?

Tak ve zdrojáku jsou to dva řádky, zrušením týhle výjimky FW moc neodtučníš.

  1. podporovat FireLogger? (či nahradit jím FirePHP?)

Asi jo. FirePHP vlastně ani nevim, jestli mi funguje.

Mikulas Dite
Člen | 756
+
0
-
  1. rozhodně
  2. FALSE
  3. nevím
  4. skipFirst je na první pohled pochopitelnější, nechal bych to
  5. tedy jestli preferovat cols nebo maxlength – radši maxlength, tzn jako třetí
  6. já je nepotřebuju, někdo možná ano?
  7. určitě ne, čitelnosti to nepřidá a je to mnohem delší (navíc najít správnou konstantu je stejně složité, jako si najít co bool parametry znamenají)
  8. render by měl vykreslovat
  9. není potřeba, v extras se bude vyjímat
  10. callback by byl prima, ale nepotřebuju
  11. zrušit abstraktní třídu AutoLoader?
  12. nepoužívám
  13. IOException se teoreticky hodí i mimo soubory, nebo když soubor existuje ale nemá dobrý formát atd., zbylé dvě bych přepsal na FileNotFoundException.
  14. ne, InvalidArgumentException jí zastoupí dobře
  15. tady těžko říct, fireLogger neznám, takže to nemůžu posoudit
nAS
Člen | 277
+
0
-
  1. určitě by to minimálně chtělo možnost komprimaci vypnout.
  2. TRUE. Je to restriktivnější a s nastaveným TRUE je vygenerovaný kód validní i v HTML. Naopak to neplatí.
  3. Je mi jedno.
  4. Jo.
  5. V ideálním finálním stavu by se cols mělo stejně nastavovat v šabloně, takže bych ho klidně přesunul až jako poslední parametr. Ale maxLength je spíš věcí validátoru, takže sem podle mě také tolik nepatří. Takže je mi to jedno.
  6. IUser jo, ostatní já nepotřebuji, ale tím neříkám že by se měly vyhodit.
  7. Asi jo, teď stejně vždycky píšu komentář.
  8. Asi jo.
  9. Do extras s ním.
  10. Spíš jo, ale je mi to celkem jedno, stejně nepoužívám.
  11. Jo.
  12. Mě ne, ale umím si představit využití.
  13. Já bych nechal
  14. Určitě.
  15. Já nepoužívám na vývoj Firefox, takže je mi to jedno.
Honza Marek
Člen | 1664
+
0
-

nAS napsal(a):

V ideálním finálním stavu by se cols mělo stejně nastavovat v šabloně, takže bych ho klidně přesunul až jako poslední parametr.

Což ostatně platí i o labelu políčka, což je druhý parametr.

Ondřej Brejla
Člen | 746
+
0
-
  1. Defaultně zapnuto, možnost vypnout.
  2. Rozhodně TRUE.
  3. Nevím? Asi je mi to jedno.
  4. Ano, to vypadá rozumně.
  5. Spíš ano…
  6. Nepoužívám. Snad se dá možná využít IUser, ale ani to nepoužívám…
  7. Možná, těžko říct. Se současnou implementací problém nemám. (EDIT: asi ano, je to čitelnější)
  8. Ano.
  9. Ve FW netřeba.
  10. Nevím, nepoužívám.
  11. Asi ano.
  12. Nepoužívám.
  13. Možná je to zbytečné. Ale na druhou stranu, proč ne…takže bych je asi ponechal.
  14. Ano, to je poměrně zbytečné. Ale je to podobný problém jako 13. Zásadní problém v tom nevidím.
  15. Nevím…tyto feature moc nepoužívám.
hrach
Člen | 1823
+
0
-

Nevím co je myšleno zrušením IUser; já mám vlastní třídu user odděděnou od User, tzn. nepotřbeuji tam mít interface, ten má určitě User :D ale někde v configu říct, co instancovat potřebuju…

iguana007
Člen | 970
+
0
-
  1. uvital bych prepinac
  2. TRUE
  3. Ano
  4. Ano
  5. Ano
  6. Nepoužívám, ale IUSer se může hodit
  7. proč ne, víc psaní mi nevadí, ruku na srdce, kolikrát se to v aplikaci napíše? :)
  8. Ano
  9. Já jsem pro ponechání, osobně používám v několika aplikacích
  10. nevím, zatím jsem nepoužil
  11. klidně
  12. zatím jsem nepoužil
  13. klidně bych nechal

15. mě zatím stačí FirePHP, ale změně se nebráním

Honza Kuchař
Člen | 1661
+
0
-
  1. mají se logy výjimek v Debug komprimovat?
    • Podle mě ne. Ale možná bych tam nechal možnot to zapnout.
  2. defaultní Html::$xhtml má být TRUE nebo FALSE?
    • používám všude HTML. Tzn. FALSE. Ale pokud je ten XHTML zápis kompatibilní s HTML, tak pak by stálo zauvažovat, zda nebude lepší tam dát TRUE.
  3. přejmenovat HttpUploadedFile::getImage() na vhodnější toImage() ?
    • Ano.
  4. sjednotit názvy obdobných metod formulářů setEmptyValue() a skipFirst() (asi na setEmptyValue)?
    • Ano, setEmptyValue.
  5. dát u addText() a addPassword() jako třetí parametr $maxLength (nyní je čtvrtý)
    • Ne. Proč? Myslím, že je to takhle správně.
  6. jsou třeba IHttpRequest, IHttpResponse a IUser?
    • 1. dva myslím, že ne. IUser myslím, že ano.
  7. nahradit bool parametry metod za bitové příznaky (např. $user->logout(TRUE) na $user->logout(User::CLEAR_IDENTITY))
    • Myslím že ano, bude to přehlednější.
  8. změnit Html::render() tak, aby obsah vykreslil element namísto vrátil?
    • Ano.
  9. ponechat ve frameworku Nette\Web\Ftp?
    • Do extras.
  10. zrušit abstraktní třídu AutoLoader?
    • Nikdy jsem nepoužil, nevím.
  11. hodí se k něčemu FreezableObject?
    • Nepoužil jsem zatím.
  12. je třeba pro chybějící soubor/adresář disponovat hned třema výjimkama IOException & FileNotFoundException & DirectoryNotFoundException?
    • Ne, nechat jen IOException.
  13. je k něčemu ArgumentOutOfRangeException?
    • Občas to používám, takže ano.
  14. podporovat FireLogger? (či nahradit jím FirePHP?)
    • Byl bych rád, protože FirePHP mi na notebooku s Nette vůbec nefunguje.
Foowie
Člen | 268
+
0
-
  1. mají se logy výjimek v Debug komprimovat?
    • Ne
  2. defaultní Html::$xhtml má být TRUE nebo FALSE?
    • True (možná mít toto nastavení v configu?)
  3. přejmenovat HttpUploadedFile::getImage() na vhodnější toImage() ?
    • Ale jo …
  4. sjednotit názvy obdobných metod formulářů setEmptyValue() a skipFirst() (asi na setEmptyValue)?
    • Ano
  5. dát u addText() a addPassword() jako třetí parametr $maxLength (nyní je čtvrtý)
    • Ne
  6. jsou třeba IHttpRequest, IHttpResponse a IUser?
    • N/A
  7. nahradit bool parametry metod za bitové příznaky (např. $user->logout(TRUE) na $user->logout(User::CLEAR_IDENTITY))
    • Hmmmm … delší, ale přehlednější …
  8. změnit Html::render() tak, aby obsah vykreslil element namísto vrátil?
    • Ano
  9. ponechat ve frameworku Nette\Web\Ftp?
    • Klidně v extras
  10. změnit čtvrtý parametr Permission::allow() z IPermissionAssertion na callback?
    • N/A
  11. zrušit abstraktní třídu AutoLoader?
    • N/A
  12. hodí se k něčemu FreezableObject?
    • N/A
  13. je třeba pro chybějící soubor/adresář disponovat hned třema výjimkama IOException & FileNotFoundException & DirectoryNotFoundException?
    • Stačí jedna výjimka
  14. je k něčemu ArgumentOutOfRangeException?
    • N/A
  15. podporovat FireLogger? (či nahradit jím FirePHP?)
    • N/A

Editoval Foowie (7. 6. 2010 11:34)

h4kuna
Backer | 730
+
0
-
  1. nekomprimovat, s moznosti zapnout
  2. TRUE
  3. nechat get
  4. ano
  5. ano
  1. vykreslil
  2. do extras
  1. byl bych pro jednu FileNotFoundException
  2. v SPL je OutOfRangeException co ji timto nahradit?
Petr Motejlek
Člen | 293
+
0
-
  1. mají se logy výjimek v Debug komprimovat?

ne

  1. defaultní Html::$xhtml má být TRUE nebo FALSE?

false

  1. přejmenovat HttpUploadedFile::getImage() na vhodnější toImage() ?

proč ne?

  1. sjednotit názvy obdobných metod formulářů setEmptyValue() a skipFirst() (asi na setEmptyValue)?

proč ne?

  1. dát u addText() a addPassword() jako třetí parametr $maxLength (nyní je čtvrtý)

nemám názor

  1. jsou třeba IHttpRequest, IHttpResponse a IUser?

ne

  1. nahradit bool parametry metod za bitové příznaky (např. $user->logout(TRUE) na $user->logout(User::CLEAR_IDENTITY))

proč ne?

  1. změnit Html::render() tak, aby obsah vykreslil element namísto vrátil?

určitě

  1. ponechat ve frameworku Nette\Web\Ftp?

to je IMHO extras

  1. změnit čtvrtý parametr Permission::allow() z IPermissionAssertion na callback?

nemám názor

  1. zrušit abstraktní třídu AutoLoader?

nemám názor

  1. hodí se k něčemu FreezableObject?

chvíli jsem ho používal, ale pak jsem přestal

  1. je třeba pro chybějící soubor/adresář disponovat hned třema výjimkama IOException & FileNotFoundException & DirectoryNotFoundException?

nemám názor

  1. je k něčemu ArgumentOutOfRangeException?

je

  1. podporovat FireLogger? (či nahradit jím FirePHP?)

nemám názor

sodae
Nette Evangelist | 250
+
0
-
  1. volitelné
  2. false
  3. getImage()
  4. ano
  5. nechat
  6. IUser nechat, další jsem nepoužil..
  7. dal bych jako alias, konstanta je pochopiltelné ale více se sní bude psát :P
  8. ano, máme vedle toho funkci __toString tak jeden return výstup máme
  9. extras/addons
  10. nevim
  11. nepouživam
  12. může se hodit
  13. stačí jedna nebo dvě a u jedne dát jako kod zda je to složka/soubor
  14. nevim
  15. bar mi stačí, ale firephp použivam na ajax..
Jan Tvrdík
Nette guru | 2595
+
0
-
  1. ne nebo volitelně
  2. uvítal bych FALSE, ale je mi to celkem jedno
  3. ano
  4. I když se budou jmenovat stejně, tak se budou chovat odlišně, to může být matoucí. Ale i tak bych to přejmenoval.
  5. ano
  6. IHttpRequest, IHttpResponse – ano (důvod) a IUser – nevím, nepoužil jsem, i když jsem o tom uvažoval. Spíš bych zvážil přejmenování třídy User na UserManager.
  7. ano
  8. ano
  9. ne, přesunout do extras
  10. asi ano, nemám praktickou zkušenost
  11. implementační věc, je mi to fuk
  12. mně ne
  13. není to sice potřeba, ale zachoval bych je
  14. nepoužil jsem
  15. ano, zkusil bych nahradit FirePHP, se kterým byly často problémy
Majkl578
Moderator | 1364
+
0
-
  1. mají se logy výjimek v Debug komprimovat?

Ano i ne, volitelně (ideálně přímo v konfiguráku).

  1. defaultní Html::$xhtml má být TRUE nebo FALSE?

TRUE, všude používám XHTML.

  1. přejmenovat HttpUploadedFile::getImage() na vhodnější toImage() ?

Nechal bych get, ale je mi to jedno.

  1. sjednotit názvy obdobných metod formulářů setEmptyValue() a skipFirst() (asi na setEmptyValue)?

Třeba.

  1. dát u addText() a addPassword() jako třetí parametr $maxLength (nyní je čtvrtý)

Ano, pokud to tam vůbec má být.

  1. jsou třeba IHttpRequest, IHttpResponse a IUser?

IUser by využití mít mohl.

  1. nahradit bool parametry metod za bitové příznaky (např. $user->logout(TRUE) na $user->logout(User::CLEAR_IDENTITY))

Bylo by to asi pěkné, $user->logout($user::CLEAR_IDENTITY) nevypadá zle. :)

  1. změnit Html::render() tak, aby obsah vykreslil element namísto vrátil?

Ano.

  1. ponechat ve frameworku Nette\Web\Ftp?

Nejspíš extras.

  1. hodí se k něčemu FreezableObject?

Nikdy jsem ho nepoužil.

  1. je třeba pro chybějící soubor/adresář disponovat hned třema výjimkama IOException & FileNotFoundException & DirectoryNotFoundException?

Smysl vidím jen u posledních dvou.

  1. je k něčemu ArgumentOutOfRangeException?

Zatím jsem ji nepoužil.

  1. podporovat FireLogger? (či nahradit jím FirePHP?)

Tuhle věc skoro nepoužívám, takže je mi to jedno.

Ondřej Mirtes
Člen | 1536
+
0
-

1. mají se logy výjimek v Debug komprimovat?

Dal bych to jako volitelné nastavení.

2. defaultní Html::$xhtml má být TRUE nebo FALSE?

Určitě FALSE, vše teď směřuje k HTML 5.

3. přejmenovat HttpUploadedFile::getImage() na vhodnější toImage() ?

Nechat getImage(), toImage() spíš říká, že se s tím objektem něco uvnitř stane, což není pravda.

4. sjednotit názvy obdobných metod formulářů setEmptyValue() a skipFirst() (asi na setEmptyValue)?

Řekl bych, že to jsou dvě různé věci, takže nesjednocovat. Spíš bych nějak upravil chování skipFirst() ve spojení s addRule(Form::FILLED), protože se to chová magicky (jedno je potřeba zavolat před druhém, aby to fungovalo) a lidi s tím mají problémy.

5. dát u addText() a addPassword() jako třetí parametr $maxLength (nyní je čtvrtý)

Klidně.

6. jsou třeba IHttpRequest, IHttpResponse a IUser?

Čím víc věci ve frameworku mohu nahradit vlastní implementací, tím lépe. Nechat.

7. nahradit bool parametry metod za bitové příznaky (např. $user->logout(TRUE) na $user->logout(User::CLEAR_IDENTITY))

To dává smysl ve chvíli, kdy by ta metoda mohla přijímat víc takových příznaků. Pokud je jen jeden, jsem jen pro boolean.

8. změnit Html::render() tak, aby obsah vykreslil element namísto vrátil?

Sjednotil bych chování napříč frameworkem, takže ano, vykreslit.

9. ponechat ve frameworku Nette\Web\Ftp?

Za dobu, co dělám v Nette, jsem to nepotřeboval, ale předtím jsem se z PHP několikrát na FTPčko potřeboval připojovat, takže bych to tam ponechal.

10. změnit čtvrtý parametr Permission::allow() z IPermissionAssertion na callback?

Určitě, bude to dávat větší smysl.

11. zrušit abstraktní třídu AutoLoader?

Nedokážu posoudit, jen vidím, že ji využívá jak NetteLoader, tak RobotLoader, tak bych jí asi ponechal.

12. hodí se k něčemu FreezableObject?

Nevím.

13. je třeba pro chybějící soubor/adresář disponovat hned třema výjimkama IOException & FileNotFoundException & DirectoryNotFoundException?

Nechal bych tři výjimky, mají aspoň výmluvné názvy.

14. je k něčemu ArgumentOutOfRangeException?

Může se hodit, pokud je parametr sice správného typu, ale je to např. vyšší nebo nižší číslo, než je dovoleno. Ale zatím jsem ji nikde nevyhodil :)

15. podporovat FireLogger? (či nahradit jím FirePHP?)

Určitě bych neodstraňoval podporu pro FirePHP.

Mikulas Dite
Člen | 756
+
0
-

Hodil sem to do spreadsheetu, ať je jasnější kam míříme. spreadsheets.google.co/0AhLppm… Nebojte se to editovat.

PaBi3
Bronze Partner | 62
+
0
-

1. nie
2. true
3. áno
4. áno
5. áno
7. áno
9. extras
13. áno

Editoval PaBi3 (7. 6. 2010 16:52)

Patrik Votoček
Člen | 2221
+
0
-
  1. Ano i ne. Komprimoval bych balík všech vyjímek + nechal samostatné vyjímky nekomprimované
  2. FALSE
  3. připadá mě vhodnější aktuální stav
  4. nemůžu se rozhodnout
  5. NE
  6. zatím jsem nepotřeboval reimplementovat takže asi ne
  7. ANO
  8. ANO
  9. ANO
  10. ANO příbalně udělat to trochu WTF a podporovat obě varianty
  11. nikdy jsem to nepoužil ani nehodlám
  12. hojně využívám u modelů
  13. NE
  14. mám takové tušení že jsem to už někde použil ale nevzpomenu si kde… zkusím pohledat
  15. Je to v době Nette Debug Baru potřeba?
David Grudl
Nette Core | 7823
+
0
-

Je to v době Nette Debug Baru potřeba?

Je. V tuto chvíli jediný způsob, jak ladit AJAXové požadavky nebo akce které po dokončení přesměrují na jinou stránku.

David Grudl
Nette Core | 7823
+
0
-

Mikulas Dite napsal(a):

Hodil sem to do spreadsheetu, ať je jasnější kam míříme. spreadsheets.google.co/0AhLppm… Nebojte se to editovat.

óó díky!

Honza Kuchař
Člen | 1661
+
0
-

Ty Google Docs budeme používat častěji, je to opravdu fajn věc! :-)

nAS
Člen | 277
+
0
-

Honza Marek napsal(a):

nAS napsal(a):

V ideálním finálním stavu by se cols mělo stejně nastavovat v šabloně, takže bych ho klidně přesunul až jako poslední parametr.

Což ostatně platí i o labelu políčka, což je druhý parametr.

Přesně tak, proto doufám, že se Formy budou ještě trochu koncepčně předělávat, takže mi je v podstatě jedno jak tento bod dopadne, protože doufám, že to nebude na dlouho.

David Grudl napsal(a):

Je to v době Nette Debug Baru potřeba?

Je. V tuto chvíli jediný způsob, jak ladit AJAXové požadavky nebo akce které po dokončení přesměrují na jinou stránku.

Nebo bychom mohli ladící informace ukládat do SESSION a naučit DebugBar je číst (přes nějaký předpřipravený Presenter, který bude odpovídat pouze ve vývojovém prostředí). To by se mi líbilo nejvíce.

Honza Kuchař
Člen | 1661
+
0
-

Nebo bychom mohli ladící informace ukládat do SESSION a naučit DebugBar je číst (přes nějaký předpřipravený Presenter, který bude odpovídat pouze ve vývojovém prostředí). To by se mi líbilo nejvíce.

Zbytečné je znovu vynalézat kolo!

nAS
Člen | 277
+
0
-

Takhle to ale funguje jenom ve Firefoxu a jsi závislý na externím api. Takhle bychom to mohli přizpůsobit zcela požadavkům Nette a mohlo by to být stejně sexy jako zbytek DebugBaru. Navíc implementace je celkem triviální.

Jan Tvrdík
Nette guru | 2595
+
0
-

Hlavní problém FirePHP / FireLoggeru je v tom, že chyby vzniklé po odeslání hlaviček nevidí.

David Grudl
Nette Core | 7823
+
0
-

Doplnil jsem první post o komentáře.