On 02/11/12 12:59 +0000, [email protected] wrote:
> Reviewers: ,
> 
> 
> 
> Please review this at http://codereview.tryton.org/581004/
> 
> Affected files:
>   M tax.py
> 
> 
> Index: tax.py
> ===================================================================
> 
> --- a/tax.py
> +++ b/tax.py
> @@ -942,10 +942,11 @@
>          pattern['group'] = tax.group.id if tax and tax.group else None
>          pattern['origin_tax'] = tax.id if tax else None
> 
> +        taxes = []
>          for line in self.lines:
>              if line.match(pattern):
> -                return line.get_taxes()
> -        return tax and [tax.id] or None
> +                taxes.extend(line.get_taxes())
> +        return taxes if taxes else tax and [tax.id] or None
> 
>      def update_rule(self, template2rule=None):

Please explain the rational of this change?

-- 
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: pgpxctyXwWzYn.pgp
Description: PGP signature

Reply via email to