← Back to team overview

c2c-oerpscenario team mailing list archive

[Bug 887376] Re: [6.0 & 6.1] [account] def compute needs optimization

 

In fact, there are 2 improvements that I have mentioned.

1) Every time I open the account and subaccount on tree view, recalculates the subaccount amount.
But, the calculate of the account parent depends of account childs, then was calculate before.
This don't need re-calculate in account child.

I mean?

2) The credit is calculated sum(credit) and debit with sum(debit), this is correct. 
But the balance is calculated sum (debit) - sum (credit), should not get here to recalculate the sum (debit) and then sum (credit) when you got them before. 
This need to use the accumulated calculated in a parent view
credit-debit
from ( sum(credit) as credit, sum(debit) as debit .... )

The 2) now is fixed it
The 1) I can't fixed it

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

Title:
  [6.0 & 6.1] [account] def compute needs optimization

Status in OpenERP Addons (modules):
  Confirmed

Bug description:
  The def compute of account module no use parent_right & parent_left fields for calculate sum(debit) & sum(credit)
  and not in use recursive function's

  These is very slow with a big account chart and very much
  account_move_line

  This can optimizate using parent_left & parent_right

  Im working for a propose merge with the solution.

  Meanwhile i put a script sql with my idea

  	SELECT 	MIN(aa_tree_1.code) AS code,
  		SUM(account_move_line.debit) AS debit, SUM(account_move_line.credit) AS credit
  	FROM account_account aa_tree_1
  	INNER JOIN account_account aa_tree_2
  	   ON aa_tree_2.parent_left 
  	      BETWEEN aa_tree_1.parent_left AND aa_tree_1.parent_right
  	INNER JOIN account_move_line
  	  ON account_move_line.account_id = aa_tree_2.id
  	INNER JOIN account_move
  	  ON account_move.id = account_move_line.move_id
  	  AND account_move.state = 'posted'
  	GROUP BY aa_tree_1.id

  
  These script is functionally without consolidate account, but I'm working

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


References