[6.0RC1] stock real-time evaluation -company settings

[Ferdinand]

I am currently working on a stock_accounting module (more or less ready)

in stock/stock.py

    def _create_product_valuation_moves(self, cr, uid, move, context=None):
        """
        Generate the appropriate accounting moves if the product being moves is subject
        to real_time valuation tracking, and the source or destination location is
        a transit location or is outside of the company.
        """
        if move.product_id.valuation == 'real_time': # FIXME: product valuation should perhaps be a property?

this is very risky to leave it to users to set this at product level
no one will be able to explan the difference between stock and accounting
IMHO this is a company wide setting !?
pls think of multi-company

I think this should read
        if move.company_id.valuation == 'real_time':

and move the valuation field from product_product to res_company.

[Vinay Rana (OpenERP) (vra-openerp)]

We Let the addons2 team decide.

Thanks.