RSS[2010–04–13] přejmenování metod User::authenticate, signOut, isAuthenticated → login, logout, isLoggedIn
#1 před 2 lety
- David Grudl
- Administrator
- Registrovaný: 8. 2. 2005
- Příspěvky: 5835
[2010–04–13] přejmenování metod User::authenticate, signOut, isAuthenticated → login, logout, isLoggedIn
Na základě hlasování komunity došlo k přejmenování metod třídy Nette\Web\User souvisejících s autentifikací.
- authenticate() → login()
- signOut() → logout()
- isAuthenticated() → isLoggedIn()
- getSignOutReason() → getLogoutReason()
- $onAuthenticated → $onLoggedIn
- $onSignedOut → $onLoggedOut
Změna se týká verzí 0.9 i 1.0. Ve verzi 0.9.4 lze spíše než o přejmenování hovořit o vytvoření aliasů, protože původní názvy jsou stále funkční a nevyhazují žádné chybové hlášky.
#2 před 2 lety
- Patrik Votoček
- Nette Evangelist
- Registrovaný: 8. 3. 2008
- Příspěvky: 2122
Re: [2010–04–13] přejmenování metod User::authenticate, signOut, isAuthenticated → login, logout, isLoggedIn
Můžu mít dotaz proč jsou aliasi i v 1.0?
Sorry my bad english.
Vrtak-CZ | Nella Project | Osobní Blog | Twitter | LinkedIn | GitHub
„Nastala chyba která neměla nastat“ aneb „Když se chce
všechno jde.“
#3 před 2 lety
#4 před 2 lety
- Honza Kuchař
- Moderator
- Registrovaný: 12. 8. 2007
- Příspěvky: 1588
Re: [2010–04–13] přejmenování metod User::authenticate, signOut, isAuthenticated → login, logout, isLoggedIn
Neměla by být verze 1.0 úplně bez aliasů? Prostě nezatížená minulostí?
Editoval honzakuchar (14. 4. 2010 10:35)
#5 před 2 lety
- Petr Motejlek
- Nette guru
- Registrovaný: 9. 1. 2009
- Příspěvky: 325
Re: [2010–04–13] přejmenování metod User::authenticate, signOut, isAuthenticated → login, logout, isLoggedIn
honzakuchar napsal(a):
Neměla by být verze 1.0 úplně bez aliasů? Prostě nezatížená minulostí?
S tím plně souhlasím. Nevím, proč máme pořád nutkání být zpětně kompatibilní – podívejte se na MSIE, ten je tak zpětně kompatibilní, že si dokáže 15 let s sebou táhnout jednu chybu… Nová verze je od toho, že je nová ;).
#6 před 2 lety
- Honza Marek
- Moderator
- Registrovaný: 31. 3. 2007
- Příspěvky: 1632
Re: [2010–04–13] přejmenování metod User::authenticate, signOut, isAuthenticated → login, logout, isLoggedIn
Cifro napsal(a):
A nemalo by byť zmenené
authenticate()nalogin()aj v interfaceIAuthenticator?
Jedině pokud by se přejmenoval i ten interface.
#7 před 2 lety
- Honza Kuchař
- Moderator
- Registrovaný: 12. 8. 2007
- Příspěvky: 1588
Re: [2010–04–13] přejmenování metod User::authenticate, signOut, isAuthenticated → login, logout, isLoggedIn
Když nová verze, ať je nová. Když je změna, tak ať je úplná. Ve verzi 1.0 bych přejmenoval úplně všechno co se toho týká. Žádná zpětná kompatibilita. V 1.0 bude kotrmelec a všichni s tím počítají.
Já to už docela dlouho dělám tak, že i nové weby vyvíjím na stabilních verzích nette a framework neaktualizuji, protože tím můžu klidně otevřít nějakou bezpečnostní díru. (teda kromě nějakých bezpečnostních záplat, samozřejmě)
#8 před 2 lety
- David Grudl
- Administrator
- Registrovaný: 8. 2. 2005
- Příspěvky: 5835
Re: [2010–04–13] přejmenování metod User::authenticate, signOut, isAuthenticated → login, logout, isLoggedIn
IAuthenticator a autenticate() je naprosto v pořádku, to je prostě vykonavatel autentifikačního procesu, nikoliv procesu přihlášení.
Do verze 1.0 samozřejmě plánuju odříznou zpětnou (ne)kompatibilitu, ale rovnou zrušit takto používané metody by bylo drsné. Pro začátek přidám nějakou E_WARNING.
#9 před 2 lety
- veena
- Člen
- Registrovaný: 12. 2. 2008
- Příspěvky: 129
Re: [2010–04–13] přejmenování metod User::authenticate, signOut, isAuthenticated → login, logout, isLoggedIn
honzakuchar napsal(a):
Když nová verze, ať je nová. Když je změna, tak ať je úplná. Ve verzi 1.0 bych přejmenoval úplně všechno co se toho týká. Žádná zpětná kompatibilita. V 1.0 bude kotrmelec a všichni s tím počítají.
Já to už docela dlouho dělám tak, že i nové weby vyvíjím na stabilních verzích nette a framework neaktualizuji, protože tím můžu klidně otevřít nějakou bezpečnostní díru. (teda kromě nějakých bezpečnostních záplat, samozřejmě)
Můj souhlasný názor:
Dokonalá release policy pro vývoj Open Source
http://www.facebook.com/note.php?…
Případnou diskuzi k tomu nechť případný diskutující případně založí v případném jiném threadu.
The Web is our DNA!
#10 před 2 lety
- David Grudl
- Administrator
- Registrovaný: 8. 2. 2005
- Příspěvky: 5835
Re: [2010–04–13] přejmenování metod User::authenticate, signOut, isAuthenticated → login, logout, isLoggedIn
Už to prosím nepitvejme. O (ne)kompatibilitě mezi 0.9 a 1.0 jsem se bavili dříve, stejně tak je compatibility-policy v Nette historicky poměrně zřejmá, ke všemu vede vždy jen jedna cesta.
Je pravda, že jsem to napsal v úvodním příspěvku blbě – opravil jsem jej. Chtěl jsem vydat stabilní verzi a neunáhlit se s razantní změnou vzniklou na poslední chvíli.
V současných dev verzích se už vyhazuje E_WARNING, s větším časovým odstupem je z 1.0 úplně odstraním.
#11 před 2 lety
- hrach
- Nette guru
- Registrovaný: 11. 3. 2007
- Příspěvky: 519
Re: [2010–04–13] přejmenování metod User::authenticate, signOut, isAuthenticated → login, logout, isLoggedIn
Z jakého důvodu nebyla přejmenována metoda
setAuthenticated() ?
#12 před 2 lety
- Jan Tvrdík
- Nette guru
- Registrovaný: 13. 4. 2008
- Příspěvky: 1262
Re: [2010–04–13] přejmenování metod User::authenticate, signOut, isAuthenticated → login, logout, isLoggedIn
Pokusil jsem se aktualizovat dokumentaci, pokud narazíte na opomenuté místo, tak ho prosím opravte sami nebo mi dejte vědět a já to spravím.
#13 před 2 lety
- lkrocek
- Nový člen
- Registrovaný: 19. 3. 2010
- Příspěvky: 8
Re: [2010–04–13] přejmenování metod User::authenticate, signOut, isAuthenticated → login, logout, isLoggedIn
Ahoj, dneska jsem si stahnul Nette Framework 1.0 alpha – PHP 5.2 – prefixed
a pri logovani mi to hlasi:
PHP Warning: NUser::authenticate() is deprecated; use login() instead. in
…\libs\Nette\Web\User.php:436
asi se na tuto verzi zapomnelo
mmj. koukam ze to ted uz ze stranky download zmizelo, mam prejit spis na klasicke Nette bez prefixu, nebo to uteklo nejakou nahodou? :-)
Editoval lkrocek (28. 5. 2010 14:11)
#14 před 2 lety
- gmvasek
- Nette guru
- Registrovaný: 19. 3. 2009
- Příspěvky: 1015
Re: [2010–04–13] přejmenování metod User::authenticate, signOut, isAuthenticated → login, logout, isLoggedIn
A co máš ve svém kódu? authenticate nebo login?
#15 před 2 lety
- lkrocek
- Nový člen
- Registrovaný: 19. 3. 2010
- Příspěvky: 8
Re: [2010–04–13] přejmenování metod User::authenticate, signOut, isAuthenticated → login, logout, isLoggedIn
gmvasek napsal(a):
A co máš ve svém kódu? authenticate nebo login?
mas pravdu mam tam chybny authenticate, nejak jsem ho prehledl a k tomu me spletla ta cesta „…\libs\Nette\Web\User.php:436“ a ja si najivne myslel ze je chyba v tom User skriptu