Hello, I checked the source version 6.0.3 (in Aug.). Damien is exactly!
Pls check the newest code! Thanks Hifly On Wed, Dec 21, 2011 at 9:28 PM, Damien <[email protected]> wrote: > Hello, > > I looked at the last version of the code on > http://bazaar.launchpad.net/~openerp/openobject- > addons/6.0/view/head:/account/invoice.py > > At the line 1332 and following (-> 1337), the variable app_acc_in is > defined, and defined correctly. > > What I said is that the variable app_acc_exp should be defined like that > too (see line 1346). > > Because the browse search to the account with the id exp_pro_id, but > exp_pro_id contains the id of the property, not of the account (see line > 1320 or 1322: "property_obj.search(cr, uid, [..."). > > Do you agree with me? > > -- > You received this bug notification because you are a member of OpenERP > Framework Experts, which is subscribed to OpenERP Addons. > https://bugs.launchpad.net/bugs/900266 > > Title: > on_change_product search for the property id instead of account_id > > Status in OpenERP Addons (modules): > Incomplete > > Bug description: > Hello, > > In addons/account/invoice.py, you'll see this lines (1345): > app_acc_exp = account_obj.browse(cr, uid, exp_pro_id, > context=context)[0] > But, if you search at exp_pro_id, you'll see that it is define in line > 1319: > > exp_pro_id = property_obj.search(cr, uid, > [('name','=','property_account_expense'), > 'res_id','=','product.template,'+str(res.product_tmpl_id.id > )+''),('company_id','=',company_id)]) > > or 1321: > > exp_pro_id = property_obj.search(cr, uid, > > > [('name','=','property_account_expense_categ'),('res_id','=','product.template,'+str( > res.categ_id.id)+''),('company_id','=',company_id)]) > > exp_pro_id is the id of a property (table ir.property), not of an > account. This causes errors. > > To resolve the problem, you should use the same function that the in > account. > > # Get the fields from the ir.property record > my_value = > property_obj.read(cr,uid,in_pro_id,['name','value_reference','res_id']) > # Parse the value_reference field to get the ID of the > account.account record > account_id = int (my_value[0]["value_reference"].split(",")[1]) > > I use OpenERP 6.0.3 (the release we can download from the download > page in the openerp website) > > To manage notifications about this bug go to: > https://bugs.launchpad.net/openobject-addons/+bug/900266/+subscriptions > -- You received this bug notification because you are a member of OpenERP Framework Experts, which is subscribed to OpenERP Addons. https://bugs.launchpad.net/bugs/900266 Title: on_change_product search for the property id instead of account_id Status in OpenERP Addons (modules): Incomplete Bug description: Hello, In addons/account/invoice.py, you'll see this lines (1345): app_acc_exp = account_obj.browse(cr, uid, exp_pro_id, context=context)[0] But, if you search at exp_pro_id, you'll see that it is define in line 1319: exp_pro_id = property_obj.search(cr, uid, [('name','=','property_account_expense'), 'res_id','=','product.template,'+str(res.product_tmpl_id.id)+''),('company_id','=',company_id)]) or 1321: exp_pro_id = property_obj.search(cr, uid, [('name','=','property_account_expense_categ'),('res_id','=','product.template,'+str(res.categ_id.id)+''),('company_id','=',company_id)]) exp_pro_id is the id of a property (table ir.property), not of an account. This causes errors. To resolve the problem, you should use the same function that the in account. # Get the fields from the ir.property record my_value = property_obj.read(cr,uid,in_pro_id,['name','value_reference','res_id']) # Parse the value_reference field to get the ID of the account.account record account_id = int (my_value[0]["value_reference"].split(",")[1]) I use OpenERP 6.0.3 (the release we can download from the download page in the openerp website) To manage notifications about this bug go to: https://bugs.launchpad.net/openobject-addons/+bug/900266/+subscriptions -- Mailing list: https://launchpad.net/~credativ Post to : [email protected] Unsubscribe : https://launchpad.net/~credativ More help : https://help.launchpad.net/ListHelp
