Merge lp:~camptocamp/account-financial-tools/7.0-batch-validate-log-more-lep into lp:~account-core-editors/account-financial-tools/7.0

[Leonardo Pistone]

[Pedro Manuel Baeza (pedro.baeza)]

Hi, Leonardo, I think it's better a more neutral language on logging messages. Could you put something like "{0} jobs for posting moves will be created"?

Regards.

review: Needs Fixing (code review)

[lp:~camptocamp/account-financial-tools/7.0-batch-validate-log-more-lep]

162. By Leonardo Pistone on 2014-02-27

[imp] account_move_batch_validate: impersonal log messages

[Leonardo Pistone (lepistone)]

There you are Pedro.

[Pedro Manuel Baeza (pedro.baeza)]

Thank you very much, Leonardo.

Regards.

review: Approve (code review)

[Guewen Baconnier @ Camptocamp (gbaconnier-c2c)]

LGTM

review: Approve (code review)

[Nicolas Bessi - Camptocamp (nbessi-c2c-deactivatedaccount)]

LGTM

review: Approve (no test, code review)

=== modified file 'account_move_batch_validate/account.py'
--- account_move_batch_validate/account.py	2014-02-21 13:44:05 +0000
+++ account_move_batch_validate/account.py	2014-02-27 09:56:03 +0000
@@ -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.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 @@
             ('state', '=', 'draft'),
         ], context=context)
 
+        _logger.info(
+            u'{0} jobs for posting moves have been created.'.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 @@
             context = {}
         # For massive amounts of moves, this becomes necessary to avoid
         # MemoryError's
+
+        _logger.info(
+            u'{0} moves marked for posting.'.format(len(move_ids))
+        )
+
         for start in xrange(0, len(move_ids), BLOCK_SIZE):
             self.write(
                 cr,