Už jsem „did you mean“ dokonce po updatu (asi 10 minut po) využil a
byl jsem za ni rád. Docela mě zajímá jak to vlastně funguje a detekuje.
Musím prostudovat.
Měl bych jeden návrh až skoro prosbu. Nešlo by vizuálně oddělit první
a druhý řádek té chybové hlášky? Neboli to, že je to Notice mě až tak
nezajímá, jako druhý řádek s důležitou informací, která se
v červené ploše ztrácí, nenajdu to zkrátka na první pohled.
Díky a sorry za OT.
#5 Davide Růžičko, No nevím to mi přijde jako docela nebezpečné, co když se Nette
netrefí a změní něco v projektu a odhalení takové chyby může být
náročnější než samotný překlep, který díky takové hlášce odhalím
rychle. Podle mě tohle je ideální stav.
Davide, prosímtě, počkej s vydáním Dibi 2.4.0 do zítřka, mám
rozpracovaný commit pro přidání podpory Offset dle verze MsSQL, tak jak jsem
přidával do PDO driveru.
#1 Petře Nováku, globálně to vypnout v tuto chvíli nejde. Problém je v tom, že
nelze rozlišit mezi opomenutím závorek a jejich záměrným vynecháním,
takže varování se zobrazí jen pokud metoda nemá žádné povinné parametry
a její název začíná na get|is|has, tj. jde o getter – řeší se tím
zmíněné situace, jako je getErrrors nebo isLoggedIn.
Falešná varování jsou samozřejmě nepříjemná, ale mám za to, že
raději jedno falešné, které lze snadno vyřešit (přepisem na klasický
callback, případně zavináčem @), hlavně když to upozorní na vážný
problém.
Pokud bys na falešná varování narazil, dej vědět do tohoto
vlákna a najdeme řešení.
#11 Kuboslave, u tříd je problematické to, že se načítají přes autoloading,
takže jejich kompletní seznam vlastně neexistuje, tudíž není z čeho
vycházet při napovídání.
mohli by ste navrhnut nejaky ide v ktorom funguje doplnanie v sablonach?
pouzivam aktivne phpstorm ale doplnanie mi tam moc nefunguje nette plugin mam
nahraty
před 9 lety
Tento článek byl uzavřen. Už není možné k němu přidávat komentáře.
Komentáře
Petr Novák #1
Je možné to s těma závorkama nějak vypnout, tj. aby
$cb = $obj->getItems fungovalo?
Luki #2
Už jsem „did you mean“ dokonce po updatu (asi 10 minut po) využil a byl jsem za ni rád. Docela mě zajímá jak to vlastně funguje a detekuje. Musím prostudovat.
Filip Hájek #3
Skvělé! Může to být ještě pohodlnější?! Díky!
Lukáš Novotný #4
Měl bych jeden návrh až skoro prosbu. Nešlo by vizuálně oddělit první a druhý řádek té chybové hlášky? Neboli to, že je to Notice mě až tak nezajímá, jako druhý řádek s důležitou informací, která se v červené ploše ztrácí, nenajdu to zkrátka na první pohled.
Díky a sorry za OT.
David Růžička #5
Hezké, ale stále je to na půli cesty. Ať to Nette rovnou opravuje, když ví, kde je chyba 🙂
Luki #6
#5 Davide Růžičko, No nevím to mi přijde jako docela nebezpečné, co když se Nette netrefí a změní něco v projektu a odhalení takové chyby může být náročnější než samotný překlep, který díky takové hlášce odhalím rychle. Podle mě tohle je ideální stav.
Kit #7
#1 Petře Nováku, Co má $obj->getItems dělat? Myslím si, že je lepší psát objektově.
David Růžička #8
#6 Luki, To byl samozřejmě vtip.
bckp #9
Davide, prosímtě, počkej s vydáním Dibi 2.4.0 do zítřka, mám rozpracovaný commit pro přidání podpory Offset dle verze MsSQL, tak jak jsem přidával do PDO driveru.
Ještě to musím doladit a dopsat testy
Aurielle #10
#1 Petře Nováku, Můžeš použít shut-up operátor @.
Kuboslav #11
Když volám třídu z namespace (bez překlepu) a zapomněl jsem použít
use
, upozorní mě to?Luki #12
#8 Davide Růžičko, Tak to pak jo. 🙂
David Grudl #13
#1 Petře Nováku, globálně to vypnout v tuto chvíli nejde. Problém je v tom, že nelze rozlišit mezi opomenutím závorek a jejich záměrným vynecháním, takže varování se zobrazí jen pokud metoda nemá žádné povinné parametry a její název začíná na get|is|has, tj. jde o getter – řeší se tím zmíněné situace, jako je getErrrors nebo isLoggedIn.
Falešná varování jsou samozřejmě nepříjemná, ale mám za to, že raději jedno falešné, které lze snadno vyřešit (přepisem na klasický callback, případně zavináčem @), hlavně když to upozorní na vážný problém.
Pokud bys na falešná varování narazil, dej vědět do tohoto vlákna a najdeme řešení.
#4 Lukáši Novotný, to je dobrý postřeh, zkusím s tím něco udělat
#5 Davide Růžičko, bohužel to možné není, programátorské odbory se proti tomu výslovně postavily
#9 bckpe, Určitě máš pár týdnů čas :)
#11 Kuboslave, u tříd je problematické to, že se načítají přes autoloading, takže jejich kompletní seznam vlastně neexistuje, tudíž není z čeho vycházet při napovídání.
Caine #14
#6 Luki, A co kdyby se tam na localu objevilo např tlačítko „Fix it for me!“ To už by tak nebezpečný nebylo.. :)
tiso #15
#14 Caine, toto by bolo úplne mega
Václav Kraus #16
David ví, jak udělat Tracy ještě přitažlivější. Tohle je totiž úplně boží! :)
Luki #17
#14 Caine, No ale aby za nás Tracy neřešilo všechno a chudák programátor si nevydělá ani na suchej rohlík 😁
Petr #18
#14 Caine, A k tomu ještě zaškrtávátko „na tohle se už přiště neptej a opravuj“ :D
a #19
mohli by ste navrhnut nejaky ide v ktorom funguje doplnanie v sablonach? pouzivam aktivne phpstorm ale doplnanie mi tam moc nefunguje nette plugin mam nahraty
Tento článek byl uzavřen. Už není možné k němu přidávat komentáře.