[FIX] minor styling

pingen/ir_attachment.py

@@ -85,7 +85,6 @@ class ir_attachment(orm.Model):
         else:
             if document:
                 if document.state == 'pushed':
-                    # TODO: cancel on pingen.com
                     raise osv.except_osv(
                         _('Error'),
                         _('The attachment %s is already pushed to pingen.com.') % \

pingen/pingen.py

@@ -37,6 +37,7 @@ POST_SENDING_STATUS = {
     400: 'Sending cancelled',
 }
 
+
 class PingenException(RuntimeError):
     """There was an ambiguous exception that occurred while handling your
     request."""
@@ -51,8 +52,7 @@ class APIError(PingenException):
 
 
 class Pingen(object):
-    """ Interface to pingen.com API
-    """
+    """ Interface to the pingen.com API """
 
     def __init__(self, token, staging=True):
         self._token = token

pingen/pingen_document.py

@@ -99,7 +99,7 @@ class pingen_document(orm.Model):
     def _push_to_pingen(self, cr, uid, document, pingen=None, context=None):
         """ Push a document to pingen.com
 
-        :param Pingen pingen: optional pingen object to, reuse session
+        :param Pingen pingen: optional pingen object to reuse session
         """
         attachment_obj = self.pool.get('ir.attachment')
 
@@ -224,6 +224,7 @@ class pingen_document(orm.Model):
                     loc_cr.rollback()
                     raise
 
+                else:
                     loc_cr.commit()
 
         return True
@@ -380,7 +381,7 @@ class pingen_document(orm.Model):
                     _logger.error('Unexcepted error in pingen cron')
                     loc_cr.rollback()
                     raise
-
+                else:
                     loc_cr.commit()
         return True