← Back to team overview

credativ team mailing list archive

[Bug 912631] Re: [stock] product.product method get_product_accounts just check the first level category_id in stock_valuation

 

Hello Amit.

>>You mean if we don't assign a stock valuation account in product category then it will check the account of it's parent category.
Yes this is exactly what Hbto is trying to say.

I think:

If we have:
Category A
--Category A.1

Category A Properties: ACC1 - ACC2 - ACC3 .....
Category A.1 Properties: ACC3 - "False" - "False" - .....

If we have some operation wit a product correspondint to Category A.1 it
will take for operations: ACC3 - ACC2 - ACC3.

Righ now Categories on products are JUST usable for Accounting porpose
nothing more, if we don't have this feature in this way, in
implementations with a lot of products, will be almost imposible detect
the leak of an account, but if you functionally clasified well your
product you can work without problem from an accounting POV.

In Order to explain better:

Now search account properties is in this way:

Account on product > Accounts on Category > Message of Error:

We propose:

Account on product > Accounts on Category > Accounts on Parent Category
[Recursivally] > MEssage of Error:

Do You get our point?

Thanks a lot for your time.

REgards.

-- 
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/912631

Title:
  [stock] product.product method get_product_accounts just check the
  first level category_id in stock_valuation

Status in OpenERP Addons (modules):
  Confirmed

Bug description:
  [stock] product.product method get_product_accounts just check the
  first level category_id

  When trying to get the accounting account for stock_[in/out] variation_account and journal_id 
  from a category_id this method just go up to the first level and does not check upper levels
  which would avoid a lot of problems because those upper level could provide the missing parameters
  lacking to accomplish stock valuation

  the patch provide could avoid that, is simple is easy to use and quite possible would not break any
  backward compatibility.

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


References