search

Tharos / LeanQuery

www.leanmapper.com
MIT License
6 stars 5 forks source link

rozlisovat select() a addSelect() #7

Open achtan opened 10 years ago

achtan commented 10 years ago

Ahoj, bolo by fajn ak by metoda sa medota select() premenovala na addSelect() a nova metoda select() by prepisovala select parameter (nie len doplnovala)

je to sice BC ale dost by to zjednodusilo niektore veci a je to aj konvencia...

Tharos commented 10 years ago

Otázkou je, co je konvence. :) Myslím, že Lean Mapper i Lean Query by měly být v první řadě co nejpodobnější dibi, i když chápu, že kdo má zkušenosti i s Doctrine 2, tak tohle ho prostě musí mást. Mně zase mate to, jak je to v Doctrine 2 :). Už se mi nejednou stalo, že jsem v ní namísto rozšíření nechtěně nahradit celý select a bug byl na světě.

Popravdě se mi API Dibi v tomhle líbí víc, a tak bych to asi neměnil. Ale určitě by pak bylo vhodné zavést něco na způsob removeClause.

achtan commented 10 years ago

o nazvoslovie mi az tak nejde... to moze byt aj select() a removeSelect() ako pises alebo nieco ako replaceSelect()