Merge lp://staging/~savoirfairelinux-openerp/openerp-hr/department_sequence_concatination_NoneType into lp://staging/openerp-hr

LGTM. You can also make one liner with:

..., name)] + args or [],

Regards.

Thank you for the review.

I could make it a one liner, but I think it's easier to read this way.
The line is long enough as it is :)

How about this?
   args = args or []

Just trying to have constant style with

if context is None:
    context = context

Could also use args = args or list(), args = [] in the signature.

 if context is None:
- context = context
+ context = {}

> How about this?
> args = args or []

With this one, you will build a new list when args is an empty list, which is unnecessary.

Hello,

looks good. Tests run fine.
I think, since you made a fix you should update the module's version number.

Regards.

>> How about this?
>> args = args or []
>
> With this one, you will build a new list when args is an empty list, which is unnecessary.

Indeed, but personally I appreciate it's brevity, compared with a 2-line 'if' statement.

Just occurred to me this one line alternative, but it's still not as neat:
args = [] if args is None else args