[Erp5-report] r22468 - in /erp5/trunk/bt5/erp5_web: SkinTemplateItem/portal_skins/erp5_web/...

[nobody at svn.erp5.org]

Author: rafael
Date: Tue Jul 15 11:52:08 2008
New Revision: 22468

URL: http://svn.erp5.org?rev=22468&view=rev
Log:
Added advise comment

Modified:
    erp5/trunk/bt5/erp5_web/SkinTemplateItem/portal_skins/erp5_web/WebSection_zGetDocumentValueList.xml
    erp5/trunk/bt5/erp5_web/bt/revision

Modified: erp5/trunk/bt5/erp5_web/SkinTemplateItem/portal_skins/erp5_web/WebSection_zGetDocumentValueList.xml
URL: http://svn.erp5.org/erp5/trunk/bt5/erp5_web/SkinTemplateItem/portal_skins/erp5_web/WebSection_zGetDocumentValueList.xml?rev=22468&r1=22467&r2=22468&view=diff
==============================================================================
--- erp5/trunk/bt5/erp5_web/SkinTemplateItem/portal_skins/erp5_web/WebSection_zGetDocumentValueList.xml (original)
+++ erp5/trunk/bt5/erp5_web/SkinTemplateItem/portal_skins/erp5_web/WebSection_zGetDocumentValueList.xml Tue Jul 15 11:52:08 2008
@@ -153,7 +153,11 @@
 <dtml-let query="buildSQLQuery(query=portal_catalog.getSecurityQuery(**kw), validation_state=validation_state, **kw)">\n
 \n
 <dtml-comment>\n
-   the subquery is named catalog to prevent use another LEFT JOIN.\n
+  Currently, there is no other choice to implement this method as an SQL catalog until SQLCatalog \n
+  can support more features which are needed here. Once SQLCatalog supports those feature, \n
+  this method should be refactored to use catalog only.\n
+  \n
+   The subquery is named catalog to prevent use another LEFT JOIN.\n
 </dtml-comment>\n
 \n
 SELECT\n
@@ -262,7 +266,11 @@
 <dtml-let query="buildSQLQuery(query=portal_catalog.getSecurityQuery(**kw), validation_state=validation_state, **kw)">\n
 \n
 <dtml-comment>\n
-   the subquery is named catalog to prevent use another LEFT JOIN.\n
+  Currently, there is no other choice to implement this method as an SQL catalog until SQLCatalog \n
+  can support more features which are needed here. Once SQLCatalog supports those feature, \n
+  this method should be refactored to use catalog only.\n
+  \n
+   The subquery is named catalog to prevent use another LEFT JOIN.\n
 </dtml-comment>\n
 \n
 SELECT\n

Modified: erp5/trunk/bt5/erp5_web/bt/revision
URL: http://svn.erp5.org/erp5/trunk/bt5/erp5_web/bt/revision?rev=22468&r1=22467&r2=22468&view=diff
==============================================================================
--- erp5/trunk/bt5/erp5_web/bt/revision (original)
+++ erp5/trunk/bt5/erp5_web/bt/revision Tue Jul 15 11:52:08 2008
@@ -1,1 +1,1 @@
-682
+683