    def create(self, cr, uid, vals, context=None):
        context = context or {}
        sol = self.pool.get('sale.order.line')  
        if vals.get('name','/')=='/':
            vals['name'] = self.pool.get('ir.sequence').get(cr, uid, 'sale.order') or '/'
        res = super(sale_order, self).create(cr, uid, vals, context=context)
 # res is the id of the created sale order, and we create 2 lines with product_1 and product_2. Here I would like ti add tax_id: 
        self.write(cr, uid, [res], {'order_line': [[0, 0, {'product_id': 1,'name': 'Service','price_unit': 50}], [0, 0, {'product_id': 4,'name': 'transport','price_unit': 15}]]})
 return res

Cyril Gaspard (GEM)

Best Answer

Hi,

call the onchange product with good order pricelist, fiscal position, date order, partner_id order .... and you will have all good values like if you will do this manually in the view.

UPDATE : code written without testing

1) just do operation in create sale.order :

def create(self, cr, uid, vals, context=None):

context = context or {}

sol = self.pool.get('sale.order.line')

if vals.get('name','/')=='/':

vals['name'] = self.pool.get('ir.sequence').get(cr, uid, 'sale.order') or '/'

pricelist_id = vals.get('pricelist_id') or False

fiscal_position_id = vals.get('fiscal_position') or False

partner_id = vals.get('partner_id') or False

date_order = vals.get('date_order') or False

order_line = False

# In onchange, I put quantity = 1 because not specified, important to calculate correctly pricelist sometimes give different price_unit depending of quantity ... (specify you real quantity by product). You can loop directly on onchange I think ...

for product_id, qty in [(1, 1), (4, 1)]:

onchange = sol.product_id_change(cr, uid, [], pricelist_id, product_id, qty=qty, uom=False, qty_uos=qty, uos=False, name='', partner_id=partner_id, lang=False, update_tax=True, date_order=date_order, packaging=False, fiscal_position=fiscal_position_id , flag=False, context=context)

onchange = onchange.get('value') or False

if onchange:

tax_id = onchange.get('tax_id') or []

if tax_id:

tax_id = [[6, 0, tax_id]]

onchange['tax_id'] = tax_id

onchange['product_id'] = product_id

onchange['qty'] = qty

order_line += [(0, 0, onchange)]

if order_line:

vals['order_line'] += order_line

res = super(sale_order, self).create(cr, uid, vals, context=context)

2) to get tax (see in sol function product_id_change):

context = context or {}

lang = lang or context.get('lang', False)

if not partner_id:

raise osv.except_osv(_('No Customer Defined!'), _('Before choosing a product,\n select a customer in the sales form.'))

partner_obj = self.pool.get('res.partner')

product_obj = self.pool.get('product.product')

context = {'lang': lang, 'partner_id': partner_id}

partner = partner_obj.browse(cr, uid, partner_id)

lang = partner.lang

context_partner = {'lang': lang, 'partner_id': partner_id}

fpos = False

if not fiscal_position:

fpos = partner.property_account_position or False

else:

fpos = self.pool.get('account.fiscal.position').browse(cr, uid, fiscal_position)

if update_tax: #The quantity only have changed

result['tax_id'] = self.pool.get('account.fiscal.position').map_tax(cr, uid, fpos, product_obj.taxes_id)

Bye

Selverine

Avtor

Hi Cyril. Thank you for your help. Is it possible to call the onchange product from create? Because one is on sale.order ( create function) the other one is in sale.order.line (onchange product). So how can I find the parameters ids for onchange product? Or need I to do it in order function? Thank you. Selverine

Selverine

Avtor

Hi Cyril.

Thank you again a lot for your help.

I try the first solution but even if I don't have any erros, nothing happen.

pricelist_id,partner_id and date_order are correct.
fiscal_position_id is egual to False (fiscal_position_id = vals.get('fiscal_position') or False)

            onchange = sol.product_id_change(cr, uid, [], pricelist_id, product_id, qty=qty, uom=False, qty_uos=5, uos=False, name='', partner_id=partner_id, lang=False, update_tax=True, date_order=date_order, packaging=False, fiscal_position=fiscal_position_id , flag=False, context=context)

            onchange = onchange.get('value') and onchange['value'].get('result') or False

            if onchange:
                tax_id = onchange.get('tax_id') or []

I have nothing for tax_id so I suppose that onchange didn't works.

Do you have any idea please?

Update3:

The output:

-> res = super(sale_order, self).create(cr, uid, vals, context=context)
(Pdb) n
IndexError: 'list index out of range'
> /opt/OpenERP/openerp/HorizonGroup/openerp/addons/sale/sale.py(519)create()
-> res = super(sale_order, self).create(cr, uid, vals, context=context)
(Pdb) print vals
{'origin': False, 'message_follower_ids': False, 'bat_sent_to_Ed': False, 'categ_ids': [[6, False, []]], 'order_line': [(0, 0, {'product_uos_qty': 1, 'name': u'Service', 'product_uom': 5, 'order_id': 168L, 'price_unit': 75.0, 'price_list': u'Service', 'delay': 7.0, 'product_packaging': False, 'product_id': 1, 'th_weight': 0.0, 'product_uos': False, 'type': u'make_to_order', 'purchase_price': 30.0, 'condition': u' ', 'tax_id': [6, 0, [2]]}), (0, 0, {'product_uos_qty': 1, 'name': u'Transport', 'product_uom': 1, 'price_unit': 1.0, 'price_list': u'Transport', 'delay': 7.0, 'product_packaging': False, 'product_id': 4, 'th_weight': 0.0, 'product_uos': False, 'type': u'make_to_order', 'purchase_price': 0.0, 'condition': u' ', 'tax_id': [6, 0, [2]]})], 'picking_policy': 'direct', 'order_policy': 'manual', 'image': '/9j/4AAQSkZJRgABAQAAAQABAAD/4QcqRXhpZgAASUkqAAgAAAAIABIBCQABAAAAAQAAABoBCQABAAAASAAAABsBCQABAAAASAAAACgBCQABAAAAAgAAADEBAgAOAAAAbgAAADIBAgAUAAAAfAAAABMCCQABAAAAAQAAAGmHBAABAAAAkAAAAN4AAABnVGh1bWIgMi4xNC4zADIwMTU6MDI6MDEgMTk6NDM6NDEABgAAkAcABAAAADAyMjEBkQcABAAAAAECAwAAoAcABAAAADAxMDABoAkAAQAAAAEAAAACoAkAAQAAAGQAAAADoAkAAQAAADMAAAAAAAAABgADAQMAAQAAAAYAAAAaAQkAAQAAAEgAAAAbAQkAAQAAAEgAAAAoAQkAAQAAAAIAAAABAgQAAQAAACwBAAACAgQAAQAAAPUFAAAAAAAA/9j/4AAQSkZJRgABAQAAAQABAAD/2wBDAAgGBgcGBQgHBwcJCQgKDBQNDAsLDBkSEw8UHRofHh0aHBwgJC4nICIsIxwcKDcpLDAxNDQ0Hyc5PTgyPC4zNDL/2wBDAQkJCQwLDBgNDRgyIRwhMjIyMjIyMjIyMjIyMjIyMjIyMjIyMjIyMjIyMjIyMjIyMjIyMjIyMjIyMjIyMjIyMjL/wAARCAAzAGQDASIAAhEBAxEB/8QAHwAAAQUBAQEBAQEAAAAAAAAAAAECAwQFBgcICQoL/8QAtRAAAgEDAwIEAwUFBAQAAAF9AQIDAAQRBRIhMUEGE1FhByJxFDKBkaEII0KxwRVS0fAkM2JyggkKFhcYGRolJicoKSo0NTY3ODk6Q0RFRkdISUpTVFVWV1hZWmNkZWZnaGlqc3R1dnd4eXqDhIWGh4iJipKTlJWWl5iZmqKjpKWmp6ipqrKztLW2t7i5usLDxMXGx8jJytLT1NXW19jZ2uHi4+Tl5ufo6erx8vP09fb3+Pn6/8QAHwEAAwEBAQEBAQEBAQAAAAAAAAECAwQFBgcICQoL/8QAtREAAgECBAQDBAcFBAQAAQJ3AAECAxEEBSExBhJBUQdhcRMiMoEIFEKRobHBCSMzUvAVYnLRChYkNOEl8RcYGRomJygpKjU2Nzg5OkNERUZHSElKU1RVVldYWVpjZGVmZ2hpanN0dXZ3eHl6goOEhYaHiImKkpOUlZaXmJmaoqOkpaanqKmqsrO0tba3uLm6wsPExcbHyMnK0tPU1dbX2Nna4uPk5ebn6Onq8vP09fb3+Pn6/9oADAMBAAIRAxEAPwD3+iisTVfElvp7GKJfOmHUA8Csq1enQjz1HZFQhKbtFG3SM6oMswA9Sa8/uvE2p3BIWbylPaMY/XrWXLPPOd0srufVmJrxauf0o6U4t+un+Z2QwM38TselvqVjH9+7gH/bQVEdb0wdb2H/AL6rzXBo21yPiCr0gvxNlgI9z0oa3ph/5fYf++qlTUrGQ4S7gJ/3xXmGDmjBHehcQVesEJ4CPRnq6urjKsCPY06vK45pojmORlPqDitO18Q6lbkDzzIo7Sc110s+py/iRa/EylgZLZnoNFYWm+JYbtliuF8qQ8A5+U/4Vu17NDEU68eam7o45wlB2kFFFFbEkVwWW2lKnDBCQffFeaTRl5C+ck9a9OlG6Jx6qRXmkuRnBAx614GeK6h8zuwbs3YgEbMcAE1dh0a+nwUtpMepGP51BBfz2sm+JirjuOa1IvF99H/rBG+P7y4P6V4NClhpP99Jr0X9fkds51V8CCPwtqD9UVfq4qUeEb3u8Q/4Ef8ACpk8an+O1Q/R8f0qX/hNYsf8ehz/ANdP/rV6UcPlVtZv8f8AI53UxXYqHwlfDo8X/fRqF/C+oqOI1b6MKvt42TtZ8+8n/wBaq8njaZuI7eJfqSaU6GVdJP8Ar5DU8V2MyXSL235ktpAPXbkVW8tlOCOavzeK9Tk4WVUz/dUVlXF9c3L75pGZj6mvNrQoJ/uW36pf1+B0QdR/EkT7ljOc8+leh6XK82l20jnLNGMmvMlBZsV6fpsflaZbIDnEa9fpXtZFfnl2sceNWiLVFFFfSHnhXGa1oE8MkktuhkgbJwvJWuzorlxeEhiYcsjSlVdN3R5S6EZBz0x6VHghl4J44NeoXOm2d3nz7dGPrjB/Osubwlp8hJRpYz7HIr5+rkleL9xp/gd8MbD7SOAJ5OT3/u4pCO205x1xXaP4MQ/cvGx7r/8AXqI+C3zkXg/75rkeVYxfY/Ff5myxdLucfjcOnWgDC/SuyXwWe94PXhP/AK9TJ4Mth/rLmRvoAKqOU4t/Z/FA8XS7nEDGcmpYoZJWCxxszHsOSa9Ah8L6ZCQTE0h/22/wrTgtLe2GIYUj/wB1cV10sjqv+JJL8TGeOj9lHI6R4XmkdJb0eXEOfL7t/hXZgBQABgDpS0V72FwlPDR5YHBVqyqO8gooorqMwooooAKKKKACiiigAooooAKKKKACiiigAooooA//2QD/2wBDAAUDBAQEAwUEBAQFBQUGBwwIBwcHBw8LCwkMEQ8SEhEPERETFhwXExQaFRERGCEYGh0dHx8fExciJCIeJBweHx7/2wBDAQUFBQcGBw4ICA4eFBEUHh4eHh4eHh4eHh4eHh4eHh4eHh4eHh4eHh4eHh4eHh4eHh4eHh4eHh4eHh4eHh4eHh7/wAARCAAzAGQDASIAAhEBAxEB/8QAHAABAAICAwEAAAAAAAAAAAAAAAQHBQYBAggD/8QAORAAAQMCAwUFBQUJAAAAAAAAAQACAwQRBQYSByExQVEUQmGh0SJykcHhE2KBgpIXIzNDU3Fzk7H/xAAbAQADAQADAQAAAAAAAAAAAAAABAUGAQIDB//EADIRAAEDAgIGCAYDAAAAAAAAAAEAAgMEEQUGEjFxgaHBEyEiMkFCsdEjJDNhkfBDUVL/2gAMAwEAAhEDEQA/APZaItTzVnahweR1NTs7VVN3EA2a0+JStXWwUcfSTOsF6RRPldosFytsXWSRkbdT3taOpNlTOKZ4zDWEhlV2Zh7sLdPnx81r9TVVlU7XU1M0rur3knzWRqc8U7DaGMu2m3uq0WCyO77gFfE2N4PCbS4pRtPQzN9VHdmjLzeOL0n61RJaTv3rjSpbs9VJPZibxTIwNni4q925oy87hi9J+tSIcbweY2ixOjeegmb6qgS03QBw4EhDM81PmibxQcEZ4OK9GRyRyN1Rva8dWm67LzxT1NVTuDoZ5I3dWuIK2DCs5Y/RuDe2OnYO7N7V/wAeKqU2doHm00ZbsN/ZKyYNI3uuurnRahlvO9JiEjKetjFLM42DgbsJ+S29a2jroK2PpIHXClSwvidovFkRETa818MRc9lBUPjNnticWnobFUPVwukndLqLid56q+6luumlZ1YR5Kh6q7dViBbmeCw2c23EV/vyVjCSQXWUZkL3mzWEnwCylHlnGaqxiw+ctPMtsPiVEocXraCf7WmeWSN5tF1sFLtHxiIWmFPLbjrjsfKyxdHTUDz81I4bAPW/JWJZagfTaDvXEGQsdk3uijj96QfJSG7O8XI3y0o/OfRSodqB/mYbE73ZSPkV9/2oU9t+Fuv/AJvotBHQ5at1yu335NSLp8R/yP3esc/Z3jABtLSn859FGmyHj0Yu2COT3ZAsu/ajEB7OF7/Gb6KHUbUKt1xT0NMw/ecXei6S0eWwOzI7dfmFyyXETraP3esDV5dxekuZ8PnaBz0Ej4hQPsHtdpc0g9LLMVe0HMM/sx1EcN+TIx87la9X4riFbKZqueSR5B4neoFXDRMPyznHaB6g8k/E6d31ABsUovZCb6hq6dFduVp5anLtBPMbyPgaSeu5UFE1z5LWueu/x4q/8tQiDL9BECTpp2cf7Ba/Jml0sltVh+bqVi4s1t9ayCIi+gKEh3hVbnTKNZTTzVNDE+eleSbMF3Mvyt0VpIpuJ4XFiMXRydVtR/pMU9S6B2k1edpoi0uBBvp0m+4r4WcJG7iRbc4m5K9AYlgmFYiD2uhhkce9ps74jeteq9nmCTEuhfUQE8g4ED4hYWpydWMPwnBw/B9uKtRYvER2wQqbLiSdTr2POOx+K6vby0HVa+q3krVm2ZxH+Fislujo7/NR3bMZtVxirf8AWfVTDlnFG/xcR7poYnTHzeqrHSHNvYXJsCR4LlrbMtbgbE/grOZswPexUDff2YvqpcGzLD2n9/iE8nUNaB6ruzLGJu1x23j3XLsUph5uBVTi17m/DdZSKWmnqHiOGGSR7uAaCSVc1FkPL1OQXU0k5H9R/pZZ+iw+hom6aSkhhH3GAKpTZMqXm8zw0fbrPJKy4zGO40lVrlDIdXNNHU4szs9O0giHvP8AA9P+q0mNDWhrQAALADkuUW2w3C4MOj0IvHWTrKiVFS+odpPRERUkuiIiEIiIhCIiIQiIiEIiIhCIiIQiIiEL/9k=', 'order_line4': [], 'shop_id': 1, 'client_order_ref': False, 'order_line5': [], 'date_order': '2015-04-04', 'partner_id': 27, 'message_ids': False, 'fiscal_position': False, 'user_id': 1, 'sale_BAT3': [], 'payment_term': 4, 'company_id': 8, 'texte_bat': 1, 'pricelist_id': 1, 'project_id': False, 'incoterm': False, 'ecart': 0, 'section_id': False, 'link_purchase': False, 'partner_invoice_id': 27, 'description_BAT': False, 'order_line3': [], 'name': u'SO182', 'partner_shipping_id': 27, 'description_Quotation': False, 'invoice_quantity': 'order', 'devis_accept_by_client': False}
(Pdb) print order_line
[(0, 0, {'product_uos_qty': 1, 'name': u'Service', 'product_uom': 5, 'order_id': 168L, 'price_unit': 75.0, 'price_list': u'Service', 'delay': 7.0, 'product_packaging': False, 'product_id': 1, 'th_weight': 0.0, 'product_uos': False, 'type': u'make_to_order', 'purchase_price': 30.0, 'condition': u' ', 'tax_id': [6, 0, [2]]}), (0, 0, {'product_uos_qty': 1, 'name': u'Transport', 'product_uom': 1, 'price_unit': 1.0, 'price_list': u'Transport', 'delay': 7.0, 'product_packaging': False, 'product_id': 4, 'th_weight': 0.0, 'product_uos': False, 'type': u'make_to_order', 'purchase_price': 0.0, 'condition': u' ', 'tax_id': [6, 0, [2]]})]

Best regards,

Selverine

PS: name_BAT was a text field. I changed my code and now it is working better ;-)

Selverine

Avtor

Hi Cyril. I updated my answer. Than you !

Cyril Gaspard (GEM)

hi, sorry for delay, alot of work, I updated my code, you should now enter in the condition if onchange, but in the dict 'value' you returned with pdb, what is the field 'name_BAT': u'ServiceBAT[]1' ? Type of field and value ? I think the value will give an error, stand answer. bye

Selverine

Avtor

Hi Cyril. I am really sorry to answer just now...some problems too :-(. I updated my answer with the new output. Thank you a lot, for your time. It is really nice ! Selverine

Cyril Gaspard (GEM)

Hi, oups, I forgot to set calculated value in vals for field order line, just add after the loop for => if order_line: vals['order_line'] = order_line, I updated my answer with this, should work now (if no tax displayed, save the sale order, and verify if taxes are displayed), bye

Selverine

Avtor

I updated my answer to ;-). Now I have an error: IndexError: 'list index out of range'. Selverine

Cyril Gaspard (GEM)

oups 2, tax_id should be a list of list : tax_id = [[6, 0, tax_id]], code updated.

Selverine

Avtor

You are my god, It works! Thank you a lot! I just changed this: vals['order_line'] = order_line to vals['order_line'] += order_line in order keep the others order_line.

Cyril Gaspard (GEM)

Hi, right for +=, good news now it works, good continuation, bye

Enjoying the discussion? Don't just read, join in!

Create an account today to enjoy exclusive features and engage with our awesome community!

Prijavi

Related Posts	Odgovori	Prikazi
Create custom Group Tax with custom module Solved tax tax_id account.tax	1 maj 18	6919
Advance customer payment and its VAT reflection tax	2 nov. 25	3087
POS Tax tax	1 jul. 25	8338
Federal Unemployment Tax (FUTA) calculates at 10X in Payroll App tax	1 mar. 25	2327
Tax rate on shipping cost Germany/ Austria tax	2 avg. 24	2974

This question has been flagged

Enjoying the discussion? Don't just read, join in!