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
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.
--
Mailing list: https://launchpad.net/~banking-addons-team
Post to : [email protected]
Unsubscribe : https://launchpad.net/~banking-addons-team
More help : https://help.launchpad.net/ListHelp
