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