credativ team mailing list archive
-
credativ team
-
Mailing list archive
-
Message #01956
Re: [Bug 900266] Re: on_change_product search for the property id instead of account_id
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 <900266@xxxxxxxxxxxxxxxxxx>
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
References