Nazvy funkci

[MartyIX]

Ahoj,

mel bych dve takove drobnosti.

1. Pokazde, kdyz chci pridat email, tak pisi

$grid->addColumnEmail('email', 'Email');

spravne ma byt:

$grid->addColumnMail('email', 'Email');

I v Nette formularich se pouziva v nazvech Email misto zkraceneho Mail. Neni nazev addColumnEmail lepsi?

2. Funkce setCustomRender mi take prijde spatne pojmenovana. Slovo Render je podle me sloveso a neprijde mi spravne, ze by to slo pouzit takto jako podstatne jmeno.

$grid->addColumnText('enabled', 'Active')
    ->setSortable()
    ->setCustomRender(function ($item) {
            return $item->enabled ? 'Yes' : 'No';
    });

Neni lepsi setCustomRendering?

Hezky vecer!

[o5]

Ahoj,

addColumnEmail() je asi pravda, s tím druhým si nejsem tak jistý :] Ono toho tam bude určitě k nalezení více, chtělo by to vypsat a sjednotit.

Akorát jak to teď řešit. V dalších setinkových verzí bych nechal obě možnosti, v další desetinkové verzi přidal vyhazovaní E_DEPRECATED. Nebo rovnou přidat vyhazovaní výjímek?

[MartyIX]

Ja bych pouzil spis E_DEPRECATED.

Ja si u toho nazvu render take nejsem moc jisty, jen mi to moc nesedelo. Chtelo by to nazor nekoho dalsiho. Kazdopadne moznosti jsou tyto:

• setCustomRender
• setCustomRendering - "nastavit vlastní vykreslování" - to mi prijde, ze odpovida tomu, k cemu se soucasny setCustomRender pouziva
• setCustomRenderer (inspirovane http://api.nette.org/2.1.0/source-Forms.Rendering.DefaultFormRenderer.php.html) - "nastavit vlastní vykreslovač" - to mi sedi mene, protoze si pod tim predstavim nejaky objekt, ktery provadi vykreslovani.

[o5]

Ono se právě už jednou celkem masivní přejmenování dělo a osobně to moc nemám rád (kdo asi má)... Každopádně E_DEPRECATED se mi zdá celkem drsný...no nevím ještě, chtělo by to fakt víc názorů...

OMG github právě nahodil novej design... fůj :-P

[tacoberu]

Mohu-li říct svůj názor, hlasoval bych pro E_DEPRECATED - je to otázka hromadného přejmenování, to tolik nebolí; a pro setCustomRenderer() - je to opravdu věc toho, že tam předávám objekt.

[o5]

Mohu-li říct svůj názor

Ano, každej názor je vítán :)

[MartyIX]

Já tuhle issue beru čistě jako návrh k diskusi a rozhodně bych to neuspěchával.

Jinak teda A E_DEPRECATED je lepší než výjimka.

[o5]

Sry to jsem se špatně vyjádřil, trigger_error() s E_DEPRECATED, vyjímka byla samozřejmě pitomost. Mně šlo ale spíš o to, že by to třeba v další setinkové mělo být zakomentováno (trigger error). Nevím...