+1

As Leonardo already said it, class might not be conventional enough. It's
better to grep on _name and _inherit. I don't recall having grepped on a
Class name.

I'm also for a change toward CamelCase


On Fri, Apr 4, 2014 at 11:54 AM, Leonardo Pistone <
[email protected]> wrote:

> Since as Stefan points out, SA is doing the same thing, and since you can
> grep for the _name instead of the  class name, I vote +1.
>
>
> On Fri, Apr 4, 2014 at 11:39 AM, Davide Corio <[email protected]>wrote:
>
>> On 04 Apr 2014, at 11:25, Stefan Rijnhart <[email protected]> wrote:
>> >
>> > +1, especially as OpenERP core code seems to be moving towards this.
>> > From some of the new modules in trunk addons:
>>
>> as Alexander said, this would requires some "grepping habits" changes.
>> i wouldn't like to have to grep twice to search for some classes just
>> because
>> the original one is my_class and the inherited one is MyClass.
>>
>> But if OpenERP is gonna move toward this too you get another +1
>>
>> --
>> Davide Corio
>> [email protected]
>> ERP Consultant
>> L.S. Advanced Software Srl
>>
>>
>>
>> _______________________________________________
>> Mailing list: https://launchpad.net/~openerp-community
>> Post to     : [email protected]
>> Unsubscribe : https://launchpad.net/~openerp-community
>> More help   : https://help.launchpad.net/ListHelp
>>
>
>
> _______________________________________________
> Mailing list: https://launchpad.net/~openerp-community
> Post to     : [email protected]
> Unsubscribe : https://launchpad.net/~openerp-community
> More help   : https://help.launchpad.net/ListHelp
>
>


-- 
Yannick Vaucher
Business Solutions Software Developer

Camptocamp SA
PSE A, CH-1015 Lausanne
Phone: +41 21 619 10 30
Office: +41 21 619 10 10
http://www.camptocamp.com/
_______________________________________________
Mailing list: https://launchpad.net/~openerp-community
Post to     : [email protected]
Unsubscribe : https://launchpad.net/~openerp-community
More help   : https://help.launchpad.net/ListHelp

Reply via email to