openerp-connector-community team mailing list archive

[Guewen Baconnier <guewen.baconnier@xxxxxxxxxxxxxx>]

On Mon, Jun 30, 2014 at 11:35 AM, Alberto Garcia
<alberto.garcia@xxxxxxxxxxxxxxx> wrote:
> We applied that patch but the problem persist
>
> My code:
> http://pastebin.com/qPDktH0d
>
>
>

Several issues there:
* a ConnectorUnit cannot be registered on a backend if it is not
linked to one/several models
* if you register it on a backend, it should be on a custom backend or
it should 'replace' another one (dangerous), but you can't register 2
concurrent ConnectorUnit on the same backend
* the code remove all None arguments when the last argument is None,
which is different than removing all None trailing arguments

I would simply propose you to propose a merge on the base adapter.