[tryton-dev] Denormalisation of account_move table

2013-04-09 Thread Marc Rechté
Hello, Most queries on the acoount_move table usually require access to the company relation. It is too far (one must go through period.fiscalyear.company), and it may seriously impact performance on such a big table. I suggest to denormalise the table to include the company field. Thanks

Re: [tryton-dev] Denormalisation of account_move table

2013-04-09 Thread Cédric Krier
On 08/04/13 23:58 -0700, Marc Rechté wrote: Hello, Most queries on the acoount_move table usually require access to the company relation. It is too far (one must go through period.fiscalyear.company), and it may seriously impact performance on such a big table. I don't understand where

Re: [tryton-dev] Denormalisation of account_move table

2013-04-09 Thread Marc Rechté
For instance, if we consider the General Ledger report, and the simple table scan on account_move: clause = [ ..., ('period.fiscalyear.company', '=', data['company']), ] return Move.search(clause, order=[('date', 'ASC'), ('id',

Re: [tryton-dev] Denormalisation of account_move table

2013-04-09 Thread Cédric Krier
On 09/04/13 02:44 -0700, Marc Rechté wrote: For instance, if we consider the General Ledger report, and the simple table scan on account_move: clause = [ ..., ('period.fiscalyear.company', '=', data['company']), ] return

[tryton-dev] PATCH: sale_supply_drop_shipment: Fix search and read for moves from drop shipment (issue 778002)

2013-04-09 Thread cedric . krier
Reviewers: , Please review this at http://codereview.tryton.org/778002/ Affected files: M sale.py Index: sale.py === --- a/sale.py +++ b/sale.py @@ -6,6 +6,7 @@ from trytond.pyson import Eval from trytond.pool import Pool,

[tryton-dev] PATCH: account: Fix balance on the balance sheet (issue 775002)

2013-04-09 Thread giedrius . slavinskas
Reviewers: , Please review this at http://codereview.tryton.org/775002/ Affected files: M account.py Index: account.py === --- a/account.py +++ b/account.py @@ -234,9 +234,10 @@ for type_ in childs:

[tryton-dev] account_invoice: avoid exception when sequence is None (issue3134 2.6.4) (issue 777002)

2013-04-09 Thread coinlolcoin
Reviewers: , Please review this at http://codereview.tryton.org/777002/ Affected files: M account.py Index: account.py === --- a/account.py +++ b/account.py @@ -170,6 +170,8 @@ for sequence_name in

[tryton-dev] PATCH: product_cost_fifo: Fix search domain in method get_fifo_move (issue 776002)

2013-04-09 Thread giedrius . slavinskas
Reviewers: , Please review this at http://codereview.tryton.org/776002/ Affected files: M product.py Index: product.py === --- a/product.py +++ b/product.py @@ -46,7 +46,7 @@ while avail_qty 0.0:

[tryton-dev] PATCH: product: Fix Product convert domain for relate clause (issue 777003)

2013-04-09 Thread cedric . krier
Reviewers: , Please review this at http://codereview.tryton.org/777003/ Affected files: M product.py Index: product.py === --- a/product.py +++ b/product.py @@ -206,9 +206,9 @@ and isinstance(arg[1],

[tryton-dev] project: FIX typo project work error message (issue 779004)

2013-04-09 Thread resteve
Reviewers: , Please review this at http://codereview.tryton.org/779004/ Affected files: M work.py Index: work.py === --- a/work.py +++ b/work.py @@ -109,10 +109,10 @@ cls._order.insert(0, ('sequence', 'ASC'))

Re: [tryton-dev] project: FIX typo project work error message (issue 779004)

2013-04-09 Thread Cédric Krier
On 09/04/13 17:38 +, rest...@zikzakmedia.com wrote: Reviewers: , Please review this at http://codereview.tryton.org/779004/ Affected files: M work.py Index: work.py === --- a/work.py +++ b/work.py @@