openerp-india team mailing list archive

[David Romero <david.romero@xxxxxxxxxxxxx>]

Public bug reported:

Hola este es el error en el archivo purchase.py, tienen que hacer los siguientes cambios
Purchase.py, lines error 709, 728, 803 text fiscal_position_id for fiscal_position 

David Romero C
TRESCloud
Quito - Ecuador

correct code.
purchase.py, line 700

def onchange_product_uom(self, cr, uid, ids, pricelist_id, product_id, qty, uom_id,
            partner_id, date_order=False, fiscal_position_id=False, date_planned=False,
            name=False, price_unit=False, notes=False, context=None):
        """
        onchange handler of product_uom.
        """
        if not uom_id:
            return {'value': {'price_unit': price_unit or 0.0, 'name': name or '', 'notes': notes or'', 'product_uom' : uom_id or False}}
        return self.onchange_product_id(cr, uid, ids, pricelist_id, product_id, qty, uom_id,
            partner_id, date_order=date_order, fiscal_position=fiscal_position_id, date_planned=date_planned,
            name=name, price_unit=price_unit, notes=notes, context=context)

purchase.py, line 727


    def onchange_product_id(self, cr, uid, ids, pricelist_id, product_id, qty, uom_id,
            partner_id, date_order=False, fiscal_position=False, date_planned=False,
            name=False, price_unit=False, notes=False, context=None):
        """
        onchange handler of product_id.

        :param dict context: 'force_product_uom' key in context override
                             default onchange behaviour to force using the UoM
                             defined on the provided product
        """
        if context is None:
            context = {}
        
        res = {'value': {'price_unit': price_unit or 0.0, 'name': name or '', 'notes': notes or '', 'product_uom' : uom_id or False}}
        if not product_id:
            return res

        product_product = self.pool.get('product.product')
        product_uom = self.pool.get('product.uom')
        res_partner = self.pool.get('res.partner')
        product_supplierinfo = self.pool.get('product.supplierinfo')
        product_pricelist = self.pool.get('product.pricelist')
        account_fiscal_position = self.pool.get('account.fiscal.position')
        account_tax = self.pool.get('account.tax')

        # - check for the presence of partner_id and pricelist_id
        if not pricelist_id:
            raise osv.except_osv(_('No Pricelist !'), _('You have to select a pricelist or a supplier in the purchase form !\nPlease set one before choosing a product.'))
        if not partner_id:
            raise osv.except_osv(_('No Partner!'), _('You have to select a partner in the purchase form !\nPlease set one partner before choosing a product.'))

        # - determine name and notes based on product in partner lang.
        lang = res_partner.browse(cr, uid, partner_id).lang
        context_partner = {'lang': lang, 'partner_id': partner_id}
        product = product_product.browse(cr, uid, product_id, context=context_partner)
        res['value'].update({'name': product.partner_ref, 'notes': notes or product.description_purchase})
        
        # - set a domain on product_uom
        res['domain'] = {'product_uom': [('category_id','=',product.uom_id.category_id.id)]}

        # - check that uom and product uom belong to the same category
        product_uom_po_id = product.uom_po_id.id
        if not uom_id or context.get('force_product_uom'):
            uom_id = product_uom_po_id
        
        if product.uom_id.category_id.id != product_uom.browse(cr, uid, uom_id, context=context).category_id.id:
            res['warning'] = {'title': _('Warning'), 'message': _('Selected UOM does not belong to the same category as the product UOM')}
            uom_id = product_uom_po_id

        res['value'].update({'product_uom': uom_id})

        # - determine product_qty and date_planned based on seller info
        if not date_order:
            date_order = fields.date.context_today(cr,uid,context=context)

        qty = qty or 1.0
        supplierinfo = False
        supplierinfo_ids = product_supplierinfo.search(cr, uid, [('name','=',partner_id),('product_id','=',product.id)])
        if supplierinfo_ids:
            supplierinfo = product_supplierinfo.browse(cr, uid, supplierinfo_ids[0], context=context)
            if supplierinfo.product_uom.id != uom_id:
                res['warning'] = {'title': _('Warning'), 'message': _('The selected supplier only sells this product by %s') % supplierinfo.product_uom.name }
            min_qty = product_uom._compute_qty(cr, uid, supplierinfo.product_uom.id, supplierinfo.min_qty, to_uom_id=uom_id)
            if qty < min_qty: # If the supplier quantity is greater than entered from user, set minimal.
                res['warning'] = {'title': _('Warning'), 'message': _('The selected supplier has a minimal quantity set to %s %s, you should not purchase less.') % (supplierinfo.min_qty, supplierinfo.product_uom.name)}
                qty = min_qty

        dt = self._get_date_planned(cr, uid, supplierinfo, date_order,
context=context).strftime(DEFAULT_SERVER_DATETIME_FORMAT)

        res['value'].update({'date_planned': date_planned or dt,
'product_qty': qty})

        # - determine price_unit and taxes_id
        price = product_pricelist.price_get(cr, uid, [pricelist_id],
                    product.id, qty or 1.0, partner_id, {'uom': uom_id, 'date': date_order})[pricelist_id]
        
        taxes = account_tax.browse(cr, uid, map(lambda x: x.id, product.supplier_taxes_id))
        fpos = fiscal_position and account_fiscal_position.browse(cr, uid, fiscal_position, context=context) or False
        taxes_ids = account_fiscal_position.map_tax(cr, uid, fpos, taxes)
        res['value'].update({'price_unit': price, 'taxes_id': taxes_ids})

        return res

    product_id_change = onchange_product_id
    product_uom_change = onchange_product_uom

** 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/1094071

Title:
  onchange of order line, onchange_product_id (... unexpected keyword
  argument, fiscal_position_id

Status in OpenERP Addons (modules):
  New

Bug description:
  Hola este es el error en el archivo purchase.py, tienen que hacer los siguientes cambios
  Purchase.py, lines error 709, 728, 803 text fiscal_position_id for fiscal_position 

  David Romero C
  TRESCloud
  Quito - Ecuador

  correct code.
  purchase.py, line 700

  def onchange_product_uom(self, cr, uid, ids, pricelist_id, product_id, qty, uom_id,
              partner_id, date_order=False, fiscal_position_id=False, date_planned=False,
              name=False, price_unit=False, notes=False, context=None):
          """
          onchange handler of product_uom.
          """
          if not uom_id:
              return {'value': {'price_unit': price_unit or 0.0, 'name': name or '', 'notes': notes or'', 'product_uom' : uom_id or False}}
          return self.onchange_product_id(cr, uid, ids, pricelist_id, product_id, qty, uom_id,
              partner_id, date_order=date_order, fiscal_position=fiscal_position_id, date_planned=date_planned,
              name=name, price_unit=price_unit, notes=notes, context=context)

  purchase.py, line 727

  
      def onchange_product_id(self, cr, uid, ids, pricelist_id, product_id, qty, uom_id,
              partner_id, date_order=False, fiscal_position=False, date_planned=False,
              name=False, price_unit=False, notes=False, context=None):
          """
          onchange handler of product_id.

          :param dict context: 'force_product_uom' key in context override
                               default onchange behaviour to force using the UoM
                               defined on the provided product
          """
          if context is None:
              context = {}
          
          res = {'value': {'price_unit': price_unit or 0.0, 'name': name or '', 'notes': notes or '', 'product_uom' : uom_id or False}}
          if not product_id:
              return res

          product_product = self.pool.get('product.product')
          product_uom = self.pool.get('product.uom')
          res_partner = self.pool.get('res.partner')
          product_supplierinfo = self.pool.get('product.supplierinfo')
          product_pricelist = self.pool.get('product.pricelist')
          account_fiscal_position = self.pool.get('account.fiscal.position')
          account_tax = self.pool.get('account.tax')

          # - check for the presence of partner_id and pricelist_id
          if not pricelist_id:
              raise osv.except_osv(_('No Pricelist !'), _('You have to select a pricelist or a supplier in the purchase form !\nPlease set one before choosing a product.'))
          if not partner_id:
              raise osv.except_osv(_('No Partner!'), _('You have to select a partner in the purchase form !\nPlease set one partner before choosing a product.'))

          # - determine name and notes based on product in partner lang.
          lang = res_partner.browse(cr, uid, partner_id).lang
          context_partner = {'lang': lang, 'partner_id': partner_id}
          product = product_product.browse(cr, uid, product_id, context=context_partner)
          res['value'].update({'name': product.partner_ref, 'notes': notes or product.description_purchase})
          
          # - set a domain on product_uom
          res['domain'] = {'product_uom': [('category_id','=',product.uom_id.category_id.id)]}

          # - check that uom and product uom belong to the same category
          product_uom_po_id = product.uom_po_id.id
          if not uom_id or context.get('force_product_uom'):
              uom_id = product_uom_po_id
          
          if product.uom_id.category_id.id != product_uom.browse(cr, uid, uom_id, context=context).category_id.id:
              res['warning'] = {'title': _('Warning'), 'message': _('Selected UOM does not belong to the same category as the product UOM')}
              uom_id = product_uom_po_id

          res['value'].update({'product_uom': uom_id})

          # - determine product_qty and date_planned based on seller info
          if not date_order:
              date_order = fields.date.context_today(cr,uid,context=context)

          qty = qty or 1.0
          supplierinfo = False
          supplierinfo_ids = product_supplierinfo.search(cr, uid, [('name','=',partner_id),('product_id','=',product.id)])
          if supplierinfo_ids:
              supplierinfo = product_supplierinfo.browse(cr, uid, supplierinfo_ids[0], context=context)
              if supplierinfo.product_uom.id != uom_id:
                  res['warning'] = {'title': _('Warning'), 'message': _('The selected supplier only sells this product by %s') % supplierinfo.product_uom.name }
              min_qty = product_uom._compute_qty(cr, uid, supplierinfo.product_uom.id, supplierinfo.min_qty, to_uom_id=uom_id)
              if qty < min_qty: # If the supplier quantity is greater than entered from user, set minimal.
                  res['warning'] = {'title': _('Warning'), 'message': _('The selected supplier has a minimal quantity set to %s %s, you should not purchase less.') % (supplierinfo.min_qty, supplierinfo.product_uom.name)}
                  qty = min_qty

          dt = self._get_date_planned(cr, uid, supplierinfo, date_order,
  context=context).strftime(DEFAULT_SERVER_DATETIME_FORMAT)

          res['value'].update({'date_planned': date_planned or dt,
  'product_qty': qty})

          # - determine price_unit and taxes_id
          price = product_pricelist.price_get(cr, uid, [pricelist_id],
                      product.id, qty or 1.0, partner_id, {'uom': uom_id, 'date': date_order})[pricelist_id]
          
          taxes = account_tax.browse(cr, uid, map(lambda x: x.id, product.supplier_taxes_id))
          fpos = fiscal_position and account_fiscal_position.browse(cr, uid, fiscal_position, context=context) or False
          taxes_ids = account_fiscal_position.map_tax(cr, uid, fpos, taxes)
          res['value'].update({'price_unit': price, 'taxes_id': taxes_ids})

          return res

      product_id_change = onchange_product_id
      product_uom_change = onchange_product_uom

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