openerp-india team mailing list archive

["Xavier Fernandez http://www.smile.fr" <887701@xxxxxxxxxxxxxxxxxx>]

Public bug reported:

file wizard/pos_box_out.py

The field product_id links to a product.product instance (check function _get_expense_product in same file)
'product_id': fields.selection(_get_expense_product, "Operation", required=True),


but in the function get_out we have:

        product_obj = self.pool.get('product.template')
        productp_obj = self.pool.get('product.product')
        res_obj = self.pool.get('res.users')
        for data in  self.read(cr, uid, ids, context=context):
##############################
some code
##############################
            amount_check = productp_obj.browse(cr, uid, data['product_id'], context=context).am_out or False
##############################
some other code
##############################
            acc_id = product_obj.browse(cr, uid, data['product_id'], context=context).property_account_income

so data['product_id'] is sometimes used as a product.product id and
sometimes like a product.template.

(but this is supposed to be a product.product id)

** Affects: openobject-addons
     Importance: Undecided
         Status: New

-- 
You received this bug notification because you are a member of OpenERP
Indian Team, which is subscribed to OpenERP Addons.
https://bugs.launchpad.net/bugs/887701

Title:
  module point of sale wizard pos.box.out

Status in OpenERP Addons (modules):
  New

Bug description:
  file wizard/pos_box_out.py

  The field product_id links to a product.product instance (check function _get_expense_product in same file)
  'product_id': fields.selection(_get_expense_product, "Operation", required=True),

  
  but in the function get_out we have:

          product_obj = self.pool.get('product.template')
          productp_obj = self.pool.get('product.product')
          res_obj = self.pool.get('res.users')
          for data in  self.read(cr, uid, ids, context=context):
  ##############################
  some code
  ##############################
              amount_check = productp_obj.browse(cr, uid, data['product_id'], context=context).am_out or False
  ##############################
  some other code
  ##############################
              acc_id = product_obj.browse(cr, uid, data['product_id'], context=context).property_account_income

  so data['product_id'] is sometimes used as a product.product id and
  sometimes like a product.template.

  (but this is supposed to be a product.product id)

To manage notifications about this bug go to:
https://bugs.launchpad.net/openobject-addons/+bug/887701/+subscriptions