From 46e6ab7c5a665d99e9181bbbcc9215595646f548 Mon Sep 17 00:00:00 2001 From: Leonardo Pistone Date: Wed, 26 Feb 2014 17:16:05 +0100 Subject: [PATCH 1/2] [imp] account_move_batch_validate: log more when creating jobs --- account_move_batch_validate/account.py | 15 +++++++++++++++ 1 file changed, 15 insertions(+) diff --git a/account_move_batch_validate/account.py b/account_move_batch_validate/account.py index ea0620418..9f1bbd823 100644 --- a/account_move_batch_validate/account.py +++ b/account_move_batch_validate/account.py @@ -20,6 +20,8 @@ ############################################################################### """Accounting customisation for delayed posting.""" +import logging + from openerp.osv import fields, orm from openerp.tools.translate import _ @@ -27,6 +29,8 @@ from openerp.addons.connector.queue.job import job from openerp.addons.connector.session import ConnectorSession from openerp.addons.connector.queue.job import OpenERPJobStorage +_logger = logging.getLogger(__name__) + # do a massive write on account moves BLOCK_SIZE at a time BLOCK_SIZE = 1000 @@ -67,6 +71,12 @@ class account_move(orm.Model): ('state', '=', 'draft'), ], context=context) + _logger.info( + u'I will now create {0} jobs for posting moves.'.format( + len(move_ids) + ) + ) + for move_id in move_ids: job_uuid = validate_one_move.delay(session, self._name, move_id, eta=eta) @@ -107,6 +117,11 @@ class account_move(orm.Model): context = {} # For massive amounts of moves, this becomes necessary to avoid # MemoryError's + + _logger.info( + u'I will now mark {0} moves for posting.'.format(len(move_ids)) + ) + for start in xrange(0, len(move_ids), BLOCK_SIZE): self.write( cr, From 024ea5e2dad0f4b1365655854865abf43733ec8c Mon Sep 17 00:00:00 2001 From: Leonardo Pistone Date: Thu, 27 Feb 2014 10:55:26 +0100 Subject: [PATCH 2/2] [imp] account_move_batch_validate: impersonal log messages --- account_move_batch_validate/account.py | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/account_move_batch_validate/account.py b/account_move_batch_validate/account.py index 9f1bbd823..307c9919a 100644 --- a/account_move_batch_validate/account.py +++ b/account_move_batch_validate/account.py @@ -72,7 +72,7 @@ class account_move(orm.Model): ], context=context) _logger.info( - u'I will now create {0} jobs for posting moves.'.format( + u'{0} jobs for posting moves have been created.'.format( len(move_ids) ) ) @@ -119,7 +119,7 @@ class account_move(orm.Model): # MemoryError's _logger.info( - u'I will now mark {0} moves for posting.'.format(len(move_ids)) + u'{0} moves marked for posting.'.format(len(move_ids)) ) for start in xrange(0, len(move_ids), BLOCK_SIZE):