[Erp5-report] r13117 - /erp5/trunk/products/ERP5Type/Core/Folder.py

[nobody at svn.erp5.org]

Author: kazuhiko
Date: Thu Mar  1 11:37:05 2007
New Revision: 13117

URL: http://svn.erp5.org?rev=13117&view=rev
Log:
* FolderMixIn.searchFolder(): warn if DomainSelection instance is passed.

Modified:
    erp5/trunk/products/ERP5Type/Core/Folder.py

Modified: erp5/trunk/products/ERP5Type/Core/Folder.py
URL: http://svn.erp5.org/erp5/trunk/products/ERP5Type/Core/Folder.py?rev=13117&r1=13116&r2=13117&view=diff
==============================================================================
--- erp5/trunk/products/ERP5Type/Core/Folder.py (original)
+++ erp5/trunk/products/ERP5Type/Core/Folder.py Thu Mar  1 11:37:05 2007
@@ -52,6 +52,7 @@
 import os
 
 from zLOG import LOG, PROBLEM
+import warnings
 
 # Dummy Functions for update / upgrade
 def dummyFilter(object,REQUEST=None):
@@ -217,10 +218,20 @@
     # We do not have conflicting parent uid values
     delete_parent_uid = 0
     if kw.has_key('selection_domain'):
-      if kw['selection_domain'].asDomainDict().has_key('parent'):
+      if not isinstance(kw['selection_domain'], dict):
+        warnings.warn("To pass a DomainSelection instance is deprecated.\n"
+                      "Please use a domain dict instead.",
+                      DeprecationWarning)
+        kw['selection_domain'] = kw['selection_domain'].asDomainDict()
+      if kw['selection_domain'].has_key('parent'):
         delete_parent_uid = 1
     if kw.has_key('selection_report'):
-      if kw['selection_report'].asDomainDict().has_key('parent'):
+      if not isinstance(kw['selection_report'], dict):
+        warnings.warn("To pass a DomainSelection instance is deprecated.\n"
+                      "Please use a domain dict instead.",
+                      DeprecationWarning)
+        kw['selection_report'] = kw['selection_report'].asDomainDict()
+      if kw['selection_report'].has_key('parent'):
         delete_parent_uid = 1
     if delete_parent_uid:
       del kw['parent_uid']