account-payment-team team mailing list archive

[Raphaël Valyi - http://www.akretion.com <rvalyi@xxxxxxxxx>]

Indeed, I agree with Stefan, the less ugly way is the glue module,
specially considering the future like v7 where now even the sale module
doesn't depend on stock anymore, which is a very good thing for simple
service oriented companies.

Regards.

On Thu, Nov 15, 2012 at 9:26 PM, Stefan Rijnhart (Therp) <stefan@xxxxxxxx>wrote:

> Hi Mikel,
>
> thanks for taking this a step further. However, when I looked at the code
> in the second merge request where you intervene and it seems like payment
> type and fiscal position are actually retrieved in the diff's line 6,
> stock_move._prepare_invoice(), much like the onchange_partner_id method of
> account.invoice does. Looks like I did not give you the best advice after
> all.
>
> If I add things up you may need to override this stock specific method to
> retrieve the payment_type. Things now get ugly in OpenERP, as you probably
> do not want account_payment_extension to depend on stock. Maybe a hidden
> glue module that depends on both stock and account_payment_extension, as
> such a module is supposed to be installed automatically as soon as all
> dependencies are installed.
>
> Cheers,
> Stefan.
> --
>
> https://code.launchpad.net/~mikel-martin/account-payment/group-by/+merge/134433
> Your team OpenERP Committers is subscribed to branch lp:account-payment.
>

-- 
https://code.launchpad.net/~mikel-martin/account-payment/group-by/+merge/134433
Your team Account Payment is subscribed to branch lp:account-payment.