Merge lp:~agilebg/server-env-tools/binary_field_pep8 into lp:~akretion-team/server-env-tools/server-env-tools

[Lorenzo Battistini]

=== modified file 'binary_field/__init__.py'
--- binary_field/__init__.py	2014-05-01 21:58:16 +0000
+++ binary_field/__init__.py	2014-06-20 14:15:36 +0000
@@ -1,7 +1,7 @@
 # -*- coding: utf-8 -*-
 ###############################################################################
 #
-#   Module for OpenERP 
+#   Module for OpenERP
 #   Copyright (C) 2013 Akretion (http://www.akretion.com).
 #   @author Sébastien BEAU <[e-mail]>
 #
 
=== modified file 'binary_field/__openerp__.py'
--- binary_field/__openerp__.py	2014-06-08 21:45:23 +0000
+++ binary_field/__openerp__.py	2014-06-20 14:15:36 +0000
@@ -1,7 +1,7 @@
 # -*- coding: utf-8 -*-
 ###############################################################################
 #
-#   Module for OpenERP 
+#   Module for OpenERP
 #   Copyright (C) 2013 Akretion (http://www.akretion.com).
 #   @author Sébastien BEAU <[e-mail]>
 #
@@ -33,13 +33,14 @@
 - ImageRezise
 
 All of this fields will be store on the file system by default and not in the
-database. If you want to store it on an other support (database, S3, ftp, SFTP...)
-Then you should create your own 'storage class' and use your custom 'storage 
+database. If you want to store it on an other support (database, S3, ftp,
+SFTP...)
+Then you should create your own 'storage class' and use your custom 'storage
 class' instead
 
-The default Storage class will store the field on the file system and build 
+The default Storage class will store the field on the file system and build
 the path like that
- 
+
  BASE_LOCATION/DB_NAME/MODEL-FIELD/XXX/YYYYY
 
 with
@@ -75,7 +76,3 @@
  'installable': True,
  'application': True,
 }
-
-
-
-
 
=== modified file 'binary_field/fields.py'
--- binary_field/fields.py	2014-06-08 21:45:23 +0000
+++ binary_field/fields.py	2014-06-20 14:15:36 +0000
@@ -40,7 +40,8 @@
         if config and config.get('field_key'):
             self.field_key = config['field_key']
         else:
-            self.field_key = ("%s-%s" % (record._name, field_name)).replace('.', '')
+            self.field_key = (
+                "%s-%s" % (record._name, field_name)).replace('.', '')
         if config and config.get('base_location'):
             self.base_location = config['base_location']
         else:
@@ -103,8 +104,8 @@
         self.config = config
         super(BinaryField, self).__init__(**new_kwargs)
 
-    #No postprocess are needed
-    #we already take care of bin_size option in the context
+    # No postprocess are needed
+    # we already take care of bin_size option in the context
     def postprocess(self, cr, uid, obj, field, value=None, context=None):
         return value
 
@@ -126,7 +127,8 @@
                 res = storage.update(binary_uid, value)
             else:
                 res = storage.add(value)
-            vals = self._prepare_binary_meta(cr, uid, field_name, res, context=context)
+            vals = self._prepare_binary_meta(
+                cr, uid, field_name, res, context=context)
             record.write(vals)
         return True
 
@@ -137,8 +139,10 @@
                                        config=self.config)
             binary_uid = record['%s_uid' % field_name]
             if binary_uid:
-                #Compatibility with existing binary field
-                if context.get('bin_size_%s' % field_name, context.get('bin_size')):
+                # Compatibility with existing binary field
+                if context.get(
+                    'bin_size_%s' % field_name, context.get('bin_size')
+                ):
                     size = record['%s_file_size' % field_name]
                     result[record.id] = tools.human_size(long(size))
                 else:
@@ -194,8 +198,8 @@
         if not isinstance(ids, (list, tuple)):
             ids = [ids]
         for record_id in ids:
-            _logger.debug('Refreshing Image Cache from the field %s of object %s '
-                          'id : %s' % (field_name, obj._name, record_id))
+            _logger.debug('Refreshing Image Cache from the field %s of object '
+                          '%s id : %s' % (field_name, obj._name, record_id))
             field = obj._columns[field_name]
             record = obj.browse(cr, uid, record_id, context=context)
             original_image = record[field.related_field]
@@ -237,10 +241,11 @@
                     '%s_uid' % field:
                         fields.char('%s UID' % self._columns[field].string),
                     '%s_file_size' % field:
-                        fields.integer('%s File Size' % self._columns[field].string),
+                        fields.integer(
+                            '%s File Size' % self._columns[field].string),
                     })
 
-            #Inject the store invalidation function for ImageResize
+            # Inject the store invalidation function for ImageResize
             if isinstance(self._columns[field], ImageResizeField):
                 self._columns[field].store = {
                     self._name: (
@@ -261,7 +266,7 @@
         # Hack for passing the field_key in the full path
         # For now I prefer to use this hack and to reuse
         # the ir.attachment code
-        # An alternative way will to copy/paste and 
+        # An alternative way will to copy/paste and
         # adapt the ir.attachment code
         if isinstance(location, tuple):
             base_location, field_key = location