[MERGE] latest trunk

bzr revid: abo@openerp.com-20120917114455-nygv16c93qwcwnsx

1 files changed, 9 insertions, 11 deletions

diff --git a/addons/document/document_directory.py b/addons/document/document_directory.py
index 03b2dcc47bf..1ead23ab402 100644
--- a/addons/document/document_directory.py
+++ b/addons/document/document_directory.py
@@ -69,7 +69,7 @@ class document_directory(osv.osv):
     }
 
 
-    def _get_root_directory(self, cr,uid, context=None):
+    def _get_root_directory(self, cr, uid, context=None):
         objid=self.pool.get('ir.model.data')
         try:
             mid = objid._get_id(cr, uid, 'document', 'dir_root')
@@ -77,9 +77,7 @@ class document_directory(osv.osv):
                 return False
             root_id = objid.read(cr, uid, mid, ['res_id'])['res_id']
             return root_id
-        except Exception, e:
-            
-            _logger.warning('Cannot set directory root:'+ str(e))
+        except Exception:
             return False
         return objid.browse(cr, uid, mid, context=context).res_id
 
@@ -108,7 +106,7 @@ class document_directory(osv.osv):
     _sql_constraints = [
         ('dirname_uniq', 'unique (name,parent_id,ressource_id,ressource_parent_type_id)', 'The directory name must be unique !'),
         ('no_selfparent', 'check(parent_id <> id)', 'Directory cannot be parent of itself!'),
-        ('dir_parented', 'check(parent_id IS NOT NULL OR storage_id IS NOT NULL)', 'Directory must have a parent or a storage')
+        ('dir_parented', 'check(parent_id IS NOT NULL OR storage_id IS NOT NULL)', 'Directory must have a parent or a storage.')
     ]
     def name_get(self, cr, uid, ids, context=None):
         res = []
@@ -153,7 +151,7 @@ class document_directory(osv.osv):
         return True
 
     _constraints = [
-        (_check_recursion, 'Error! You can not create recursive Directories.', ['parent_id'])
+        (_check_recursion, 'Error! You cannot create recursive directories.', ['parent_id'])
     ]
 
     def __init__(self, *args, **kwargs):
@@ -179,7 +177,7 @@ class document_directory(osv.osv):
 
     def get_node_class(self, cr, uid, ids, dbro=None, dynamic=False, context=None):
         """Retrieve the class of nodes for this directory
-           
+
            This function can be overriden by inherited classes ;)
            @param dbro The browse object, if caller already has it
         """
@@ -193,17 +191,17 @@ class document_directory(osv.osv):
         elif dbro.type == 'ressource':
             return nodes.node_res_dir
         else:
-            raise ValueError("dir node for %s type", dbro.type)
+            raise ValueError("dir node for %s type.", dbro.type)
 
     def _prepare_context(self, cr, uid, nctx, context=None):
         """ Fill nctx with properties for this database
         @param nctx instance of nodes.node_context, to be filled
         @param context ORM context (dict) for us
-        
-        Note that this function is called *without* a list of ids, 
+
+        Note that this function is called *without* a list of ids,
         it should behave the same for the whole database (based on the
         ORM instance of document.directory).
-        
+
         Some databases may override this and attach properties to the
         node_context. See WebDAV, CalDAV.
         """