Merge lp:~domsense/domsense-agilebg-addons/ea_import_catching_value_error into lp:~domsense/domsense-agilebg-addons/ea_import

[Alex Comba - Agile BG]

[ADD] ValueError catching and logging for template_line_type

Added ValueError catching and logging for template_line_type

=== modified file 'ea_import/ea_import_template_line.py'
--- ea_import/ea_import_template_line.py	2012-12-28 14:01:09 +0000
+++ ea_import/ea_import_template_line.py	2013-02-15 11:52:24 +0000
@@ -23,7 +23,9 @@
 import datetime
 from tools.translate import _
 import base64
+import logging
 
+_logger = logging.getLogger('ea_import')
 
 class ea_import_template_line(osv.osv):
 
@@ -177,33 +179,48 @@
                 if template_line_type == 'binary':
                     return base64.b32encode(target_string)
                 elif template_line_type == 'integer':
-                    return int(target_string)
+                    try:
+                        return int(target_string)
+                    except ValueError, e:
+                        _logger.warning('message: %s target_string: %s' % (e.message,target_string))
+                        return None
                 elif template_line_type == 'float':
-                    return float(target_string)
+                    try:
+                        return float(target_string)
+                    except ValueError, e:
+                        _logger.warning('message: %s target_string: %s' % (e.message,target_string))
+                        return None
                 elif template_line_type == 'boolean':
                     if template_line.boolean_field_ids:
                         return template_line_boolean_field_pool.get_value(cr, uid,
                                                                           [boolean_field.id for boolean_field in template_line.boolean_field_ids],
                                                                           target_string, context=context)
                     else:
-                        return bool(target_string)
+                        try:
+                            return bool(target_string)
+                        except ValueError, e:
+                            _logger.warning('message: %s target_string: %s' % (e.message,target_string))
+                            return None
                 elif template_line_type == 'date':
                     try:
                         target_time = datetime.datetime.strptime(target_string, template_line.time_format)
                         return target_time.strftime("%Y-%m-%d")
                     except ValueError:
+                        _logger.warning('message: %s target_string: %s' % (e.message,target_string))
                         return None
                 elif template_line_type == 'datetime':
                     try:
                         target_time = datetime.datetime.strptime(target_string, template_line.time_format)
                         return target_time.strftime("%Y-%m-%d %H:%M:%S")
                     except ValueError:
+                        _logger.warning('message: %s target_string: %s' % (e.message,target_string))
                         return None
                 elif template_line_type == 'time':
                     try:
                         target_time = datetime.datetime.strptime(target_string, template_line.time_format)
                         return target_time.strftime("%H:%M:%S")
                     except ValueError:
+                        _logger.warning('message: %s target_string: %s' % (e.message,target_string))
                         return None
                 elif template_line_type == 'many2one':
                     target_obj_pool = self.pool.get(template_line.target_field.relation)