On 20/03/10 18:09 +0100, Cédric Krier wrote:
> Hi,
> 
> As we already changed most part of the Function fields, I think we should also
> improve the "search" part.
> 
> Currently, the search function must have this signature:
> 
>     def search_xxx(self, cursor, user, name, args, context=None)
> 
> where args is a list of domain clause.
> 
> But indeed, there is always only one clause like you can see in [1].
> So I propose to change the args list into one clause:
> 
>     def search_xxx(self, cursor, user, name, clause, context=None)
> 
> And this will allow to remove many unnecessary loops all over the code and
> remove the "while" coding style as asked in issue1143 [2].
> 

This changes have been pushed.

-- 
Cédric Krier

B2CK SPRL
Rue de Rotterdam, 4
4000 Liège
Belgium
Tel: +32 472 54 46 59
Email/Jabber: [email protected]
Website: http://www.b2ck.com/

Attachment: pgpoYrYhMg2s4.pgp
Description: PGP signature

Reply via email to