From 575889d33ed2932d32c67a5454758d2d00dcab7b Mon Sep 17 00:00:00 2001
From: "P. Christeas" <p_christ@hol.gr>
Date: Thu, 1 Jul 2010 20:51:33 +0300
Subject: [PATCH] Document API: have equality operators for nodes, context

These will be used for advanced operations (eg. move) of nodes.

Conflicts:

	document/nodes.py

bzr revid: p_christ@hol.gr-20100701175133-gbsdly2fwcwtr0dy
---
 addons/document/nodes.py | 73 ++++++++++++++++++++++++++++++++++++++--
 1 file changed, 70 insertions(+), 3 deletions(-)

diff --git a/addons/document/nodes.py b/addons/document/nodes.py
index acbf4ed8dcd..e4f2a0bf615 100644
--- a/addons/document/nodes.py
+++ b/addons/document/nodes.py
@@ -57,6 +57,22 @@ class node_context(object):
         assert self._dirobj
         self.rootdir = False #self._dirobj._get_root_directory(cr,uid,context)
 
+    def __eq__(self, other):
+        if not type(other) == node_context:
+            return False
+        if self.dbname != other.dbname:
+            return False
+        if self.uid != other.uid:
+            return False
+        if self.context != other.context:
+            return False
+        if self.rootdir != other.rootdir:
+            return False
+        return True
+    
+    def __ne__(self, other):
+        return not self.__eq__(other)
+
     def get_uri(self, cr,  uri):
         """ Although this fn passes back to doc.dir, it is needed since
         it is a potential caching point """            
@@ -68,7 +84,7 @@ class node_context(object):
             duri = duri[1:] 
         return ndir
         
-    def get_dir_node(cr, dbro):
+    def get_dir_node(self, cr, dbro):
         """Create (or locate) a node for a directory
             @param dbro a browse object of document.directory
         """
@@ -81,7 +97,7 @@ class node_context(object):
         else:
             raise ValueError("dir node for %s type", dbro.type)
 
-    def get_file_node(cr, fbro):
+    def get_file_node(self, cr, fbro):
         """ Create or locate a node for a static file
             @param fbro a browse object of an ir.attachment
         """
@@ -152,6 +168,12 @@ class node_class(object):
             self.dctx = parent.dctx.copy()
         self.displayname = 'Object'
     
+    def __eq__(self, other):
+        return NotImplemented
+
+    def __ne__(self, other):
+        return not self.__eq__(other)
+
     def full_path(self):
         """ Return the components of the full path for some
             node. 
@@ -369,7 +391,17 @@ class node_dir(node_database):
                     print e
                     pass
 
-    
+    def __eq__(self, other):
+        if not self.context == other.context:
+            return False
+        if type(self) != type(other):
+            return False
+        # Two directory nodes, for the same document.directory, may have a 
+        # different context! (dynamic folders)
+        if self.dctx != other.dctx:
+            return False
+        return self.dir_id == other.dir_id
+
     def get_data(self, cr):
         res = ''
         for child in self.children(cr):            
@@ -493,6 +525,17 @@ class node_res_dir(node_class):
         for dfld in dirr.dctx_ids:
             self.dctx_dict['dctx_' + dfld.field] = dfld.expr
 
+    def __eq__(self, other):
+        if not self.context == other.context:
+            return False
+        if type(self) != type(other):
+            return False
+        # Two nodes, for the same document.directory, may have a 
+        # different context! (dynamic folders)
+        if self.dctx != other.dctx:
+            return False
+        return self.dir_id == other.dir_id
+
     def children(self, cr, domain=None):
         return self._child_get(cr, domain=domain)
 
@@ -596,6 +639,21 @@ class node_res_obj(node_class):
         else:
             self.res_id = res_id
 
+    def __eq__(self, other):
+        if not self.context == other.context:
+            return False
+        if type(self) != type(other):
+            return False
+        if not self.res_model == other.res_model:
+            return False
+        if not self.res_id == other.res_id:
+            return False
+        if self.domain != other.domain:
+            return False
+        if self.dctx != other.dctx:
+            return False
+        return self.dir_id == other.dir_id
+
     def children(self, cr, domain=None):        
         return self._child_get(cr, domain=domain) + self._file_get(cr)
 
@@ -793,6 +851,15 @@ class node_file(node_class):
                 break
             par = par.parent_id
 
+    def __eq__(self, other):
+        if not self.context == other.context:
+            return False
+        if type(self) != type(other):
+            return False
+        if self.dctx != other.dctx:
+            return False
+        return self.file_id == other.file_id
+
 
     def open_data(self, cr, mode):
         stor = self.storage_id