banking-addons-team team mailing list archive

[Stéphane Bidoul (Acsone) <stephane.bidoul@xxxxxxxxx>]

Yes indeed, they are not equivalent. Getting tired :)

-sbi


On Wed, Oct 2, 2013 at 7:13 PM, Stefan Rijnhart (Therp) <stefan@xxxxxxxx>wrote:

> Thanks for the review!
>
> I would always go for the latter in cases like:
>
>     bar = foo.get('bar')
>     if bar:
>         ...
>
> But as I was going to run a string operation on the result, I didn't want
> to run the risk of sepa_dict containing a key 'TRTP' with value False (even
> if the preceding code currently seems to guarantee that when sepa_dict
> contains a key 'TRTP', the value will always be a string)
> --
>
> https://code.launchpad.net/~therp-nl/banking-addons/7.0-abnamro-sepa_type_case/+merge/188782
> You are reviewing the proposed merge of
> lp:~therp-nl/banking-addons/7.0-abnamro-sepa_type_case into
> lp:banking-addons.
>

-- 
https://code.launchpad.net/~therp-nl/banking-addons/7.0-abnamro-sepa_type_case/+merge/188782
Your team Banking Addons Core Editors is requested to review the proposed merge of lp:~therp-nl/banking-addons/7.0-abnamro-sepa_type_case into lp:banking-addons.