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

[nobody at svn.erp5.org]

Author: kazuhiko
Date: Thu Sep  3 15:41:22 2009
New Revision: 28788

URL: http://svn.erp5.org?rev=28788&view=rev
Log:
* add '/view' at the end for non Web Page documents in WebSection_getPermanentURL.
* always use getPermanentURL in WebSection_viewContentListAsRSS.

Modified:
    erp5/trunk/bt5/erp5_web/SkinTemplateItem/portal_skins/erp5_web/WebSection_getPermanentURL.xml
    erp5/trunk/bt5/erp5_web/SkinTemplateItem/portal_skins/erp5_web_widget_library/WebSection_viewContentListAsRSS/listbox_link.xml
    erp5/trunk/bt5/erp5_web/bt/revision

Modified: erp5/trunk/bt5/erp5_web/SkinTemplateItem/portal_skins/erp5_web/WebSection_getPermanentURL.xml
URL: http://svn.erp5.org/erp5/trunk/bt5/erp5_web/SkinTemplateItem/portal_skins/erp5_web/WebSection_getPermanentURL.xml?rev=28788&r1=28787&r2=28788&view=diff
==============================================================================
--- erp5/trunk/bt5/erp5_web/SkinTemplateItem/portal_skins/erp5_web/WebSection_getPermanentURL.xml [utf8] (original)
+++ erp5/trunk/bt5/erp5_web/SkinTemplateItem/portal_skins/erp5_web/WebSection_getPermanentURL.xml [utf8] Thu Sep  3 15:41:22 2009
@@ -99,7 +99,8 @@
 # Return absolute URL if this is not an appropriate portal_type\n
 portal = context.getPortalObject()\n
 valid_portal_type_list = portal.getPortalDocumentTypeList()\n
-if document.getPortalType() not in valid_portal_type_list:\n
+portal_type = document.getPortalType()\n
+if portal_type not in valid_portal_type_list:\n
   return document.absolute_url()\n
 \n
 # Return absolute URL if this is not a \'live\' document\n
@@ -109,7 +110,12 @@
   return document.absolute_url()\n
 \n
 # Return the URL\n
-return "%s/%s" % (context.absolute_url(), reference)\n
+# XXX we need to add \'/view\' in web view\'s permanent URL for non-Web Page documents.\n
+# If we change the behaviour of index_html() to same as view(), we can remove \'/view\'. \n
+if portal_type == \'Web Page\':\n
+  return "%s/%s" % (context.absolute_url(), reference)\n
+else:\n
+  return "%s/%s/view" % (context.absolute_url(), reference)\n
 </string> </value>
         </item>
         <item>
@@ -152,6 +158,7 @@
                             <string>context</string>
                             <string>portal</string>
                             <string>valid_portal_type_list</string>
+                            <string>portal_type</string>
                             <string>validation_state</string>
                           </tuple>
                         </value>

Modified: erp5/trunk/bt5/erp5_web/SkinTemplateItem/portal_skins/erp5_web_widget_library/WebSection_viewContentListAsRSS/listbox_link.xml
URL: http://svn.erp5.org/erp5/trunk/bt5/erp5_web/SkinTemplateItem/portal_skins/erp5_web_widget_library/WebSection_viewContentListAsRSS/listbox_link.xml?rev=28788&r1=28787&r2=28788&view=diff
==============================================================================
--- erp5/trunk/bt5/erp5_web/SkinTemplateItem/portal_skins/erp5_web_widget_library/WebSection_viewContentListAsRSS/listbox_link.xml [utf8] (original)
+++ erp5/trunk/bt5/erp5_web/SkinTemplateItem/portal_skins/erp5_web_widget_library/WebSection_viewContentListAsRSS/listbox_link.xml [utf8] Thu Sep  3 15:41:22 2009
@@ -273,7 +273,7 @@
       <dictionary>
         <item>
             <key> <string>_text</string> </key>
-            <value> <string>python:(cell.getPortalType() == \'Web Page\' and cell.getPermanentURL(cell) or cell.absolute_url() + \'/view\')</string> </value>
+            <value> <string>python:cell.getPermanentURL(cell)</string> </value>
         </item>
       </dictionary>
     </pickle>

Modified: erp5/trunk/bt5/erp5_web/bt/revision
URL: http://svn.erp5.org/erp5/trunk/bt5/erp5_web/bt/revision?rev=28788&r1=28787&r2=28788&view=diff
==============================================================================
--- erp5/trunk/bt5/erp5_web/bt/revision [utf8] (original)
+++ erp5/trunk/bt5/erp5_web/bt/revision [utf8] Thu Sep  3 15:41:22 2009
@@ -1,1 +1,1 @@
-847
+848