openerp-community team mailing list archive

[Alexandre Fayolle <alexandre.fayolle@xxxxxxxxxxxxxx>]

On 04/04/2014 11:05, Sebastien Beau wrote:
> Hi All,
> I always hesitate to move or not on CamelCase for every class in OCA
> module.
> I prefer CamelCase but a lot of code do not use it.
> And sometime not using CamelCase can introduce bug and you have to
> rename the class in something different like
> here https://code.launchpad.net/~camptocamp/carriers-deliveries/7.0-change-class-name-to-avoid-error_rde/+merge/213881
> <https://code.launchpad.net/%7Ecamptocamp/carriers-deliveries/7.0-change-class-name-to-avoid-error_rde/+merge/213881>
>
> Maybe it's time to move on CamelCase and update the great Nicolas
> Bessi tools to check the class name
> (https://github.com/nbessi/openerp-conventions)?
>
> What do you think? 
>
>
>
> _______________________________________________
> Mailing list: https://launchpad.net/~openerp-community
> Post to     : openerp-community@xxxxxxxxxxxxxxxxxxx
> Unsubscribe : https://launchpad.net/~openerp-community
> More help   : https://help.launchpad.net/ListHelp

People will have to change their grepping habits, and it means a
significant change in coding style between OCA addons and the official
addons.

-0 for me

-- 
Alexandre Fayolle
Chef de Projet
Tel : + 33 (0)4 79 26 57 94

Camptocamp France SAS
Savoie Technolac, BP 352
73377 Le Bourget du Lac Cedex
http://www.camptocamp.com