Změna implementace makra ifCurrent

Majkl578

Zdravím,
navrhuji změnit implementaci makra ifCurrent tak, aby bylo možné používat wilcardy a bylo tak možné použítí i na moduly, ne jen na presenter/akci.

Nové použití:

{ifCurrent :Modul:Presenter:akce}
{ifCurrent :Modul:Presenter:}
{ifCurrent Presenter:akce}
{ifCurrent Presenter:}
{ifCurrent :Modul:*}
{ifCurrent :Modul:Submodul:*}

První až čtvrtý příklad testuje absolutní/relativní cestu, nejspíš podobně jak je to implementované teď (nelíbí se mi ale, že se kvůli tomu volá metoda Presenter#createRequest(), která je celkem obsáhlá).

Co vy na to? Máte další nápady?

EDIT: David to sliboval už před rokem, ale žádosti mnoha uživatelů stále neměly takovou prioritu, aby se to vyřešilo. :/

Lopata

Také jsem to nedávno potřeboval. Rozhodně souhlasím!

pave.kucera

+1

Patrik Votoček

+1 (předpokládám že odpovědí bude pošly pull)

Aurielle

+1

Majkl578

vrtak-cz napsal(a):

předpokládám že odpovědí bude pošly pull

To mi nebude vadit, klidně to napíšu, jen se s tím nechci dělat pokud už by to David měl.

na1k

+10 (fakt by se to hodilo!)

srigi

Eeeee ludia, taka vec – David pisal, ze ma ten roadmap (60 veci). Co keby nabuduce, ked budete pisat „navrhujem/kcel-by-som/bolo-by-dobre-mat“ nachvilu zastavili, forkli Nette, dopisali to sami a poslali pull rq.

No disrespect Majkl578, ale mali by sme sa ako komunita snazit ulahcit pracu pretazenemu Davidovi.

BTW, tymto prispevkom sa zo mna akoze stal Nette guru, co neni ani nahodou pravda :)

Editoval srigi (1. 10. 2010 14:32)

Majkl578

srigi napsal(a):

No disrespect Majkl578, ale mali by sme sa ako komunita snazit ulahcit pracu pretazenemu Davidovi.

Ona ta absence dlouho slibované road mapy to zas moc neulehčuje nám – nač implementovat a posílat pull request nečeho, co třeba už má hotové (kdo ví)?

iguana007

Jsem rozhodně pro.

Když už by se to upravovalo, vzpomněl jsem si, že jsme nedávno na jabberu řešili vícenásobný ifCurrent, co to při této změně taky implementovat?
tj. např. zápis:

{ifCurrent :Modul:Presenter:akce, :Modul2:*}

jtousek

@Majkl578: +1

@iguana007: +1

Editoval jtousek (1. 10. 2010 17:03)

Bernard Williams

+1

{ifCurrent :Modul:Presenter:akce, :Modul2:*}

+1

Ondřej Mirtes

David kdysi řekl, že tohle makro chce předělat do podoby metody Presenter::isCurrent, takže by se pak v šabloně volalo {if $presenter->isCurrent(':Modul:Presenter:akce')} a daleko líp by se to řetězilo s dalšími podmínkami, univerzálně.

Jak je na tom implementace ovšem nevím.

David Grudl

Přidám do road map :)

Vyki

Vím, že to tu už někdy zaznělo, ale odpověď jsem nenašel… Je někde ten road map k nahlédnutí?

pekelnik

Zatím ne :( Myslím, že spousta lidí se na to celkem třese…

kokoumes

Zdravím, používá už někdo na položky menu něco podobného. Celkem dost by se mi to hodilo a nevim jak jinak to udělat. Předem díky za odpověď.

Filip Procházka

Roadmap

grey

Majkl578, iguana007: +1