Sorry once more

I changed (error) (1)
on_change="product_id_change_c2c_pu(product_id, uos_id, quantity, name, 
parent.type, parent.partner_id, parent.fiscal_position, price_unit_pu, 
parent.address_invoice_id, parent.currency_id, {'company_id': 
parent.company_id},price_unit_id)" 

to (working) (2)
on_change="product_id_change_c2c_pu(product_id, uos_id, quantity, name, 
parent.type, parent.partner_id, parent.fiscal_position, price_unit_pu, 
parent.address_invoice_id, parent.currency_id,  
parent.company_id,price_unit_id)"/>

replacing 
{'company_id': parent.company_id}
by 
parent.company_id

nevertheless
parent.company_id is a non existing field 
- hence the the coding (1) returned an error - which is correct behaviour 
- while coding(2) just ignores a reference to a not existing field - which IMHO 
should not be allowed.


** Changed in: openerp-web
       Status: Fix Released => New

-- 
You received this bug notification because you are a member of C2C
OERPScenario, which is subscribed to OpenERP Project Group.
https://bugs.launchpad.net/bugs/889741

Title:
  literal string parameters to on_change not supported

Status in OpenERP Web:
  New

Bug description:
  Example custom module (sorry didn't find an example in certified modules yet)
  works in GTK

  my blind guess - inheritance problem

  http://bazaar.launchpad.net/~c2c/c2c-rd-
  addons/trunk/files/head:/c2c_product_price_unit/

  purchase_view.xml

       <field name="inherit_id" ref="purchase.purchase_order_line_form"/>
  ....
             <field
                name="price_unit_pu"
                
on_change="onchange_price_unit('price_unit',price_unit_pu,price_unit_id)"/>

  purchase.py

      def onchange_price_unit(self, cr, uid, ids, field_name,price_pu, 
price_unit_id):
          if  price_pu and  price_unit_id:
             coeff = self.pool.get('c2c_product.price_unit').get_coeff(cr, uid, 
price_unit_id)
             price = price_pu / coeff

             return {'value': {field_name : price}}
          return False

To manage notifications about this bug go to:
https://bugs.launchpad.net/openerp-web/+bug/889741/+subscriptions

_______________________________________________
Mailing list: https://launchpad.net/~c2c-oerpscenario
Post to     : [email protected]
Unsubscribe : https://launchpad.net/~c2c-oerpscenario
More help   : https://help.launchpad.net/ListHelp

Reply via email to