Add a comment about newly added "strict" parameter to discourage from renaming...

Add a comment about newly added "strict" parameter to discourage from renaming it and encourage its deletion.


git-svn-id: https://svn.erp5.org/repos/public/erp5/trunk@22205 20353a03-c40f-0410-a6d1-a30d3c3de9de

product/CMFCategory/CategoryTool.py

@@ -1455,6 +1455,10 @@ class CategoryTool( UniqueObject, Folder, Base ):
     security.declareProtected(Permissions.AccessContentsInformation, 'buildAdvancedSQLSelector')
     def buildAdvancedSQLSelector(self, category_list, query_table='category',
           none_sql_value=None, strict=True, catalog_table_name='catalog'):
+      # XXX: about "strict" parameter: This is a transition parameter,
+      # allowing someone hitting a bug to revert to original behaviour easily.
+      # It is not a correct name, as pointed out by Jerome. But instead of
+      # searching for another name, it would be much better to just remove it.
       """
         Return chunks of SQL to check for category membership.
 

product/ERP5/Tool/DomainTool.py

@@ -61,6 +61,10 @@ class DomainTool(BaseTool):
                             tested_base_category_list=None,
                             filter_method=None, acquired=1,
                             strict=True, **kw):
+      # XXX: about "strict" parameter: This is a transition parameter,
+      # allowing someone hitting a bug to revert to original behaviour easily.
+      # It is not a correct name, as pointed out by Jerome. But instead of
+      # searching for another name, it would be much better to just remove it.
       """
       Search all predicates which corresponds to this particular 
       context.