[Erp5-report] r28905 - /erp5/trunk/products/ERP5Type/patches/CMFCoreSkinsTool.py

[nobody at svn.erp5.org]

Author: kazuhiko
Date: Thu Sep 10 18:05:17 2009
New Revision: 28905

URL: http://svn.erp5.org?rev=28905&view=rev
Log:
fix the wrong condition and code. this bug was found thanks to a horrible name; portal_skins/erp5_dms/RSS (Script Python) whose id exists in one of skin selection name...

Modified:
    erp5/trunk/products/ERP5Type/patches/CMFCoreSkinsTool.py

Modified: erp5/trunk/products/ERP5Type/patches/CMFCoreSkinsTool.py
URL: http://svn.erp5.org/erp5/trunk/products/ERP5Type/patches/CMFCoreSkinsTool.py?rev=28905&r1=28904&r2=28905&view=diff
==============================================================================
--- erp5/trunk/products/ERP5Type/patches/CMFCoreSkinsTool.py [utf8] (original)
+++ erp5/trunk/products/ERP5Type/patches/CMFCoreSkinsTool.py [utf8] Thu Sep 10 18:05:17 2009
@@ -60,8 +60,8 @@
       if container_id in skin_folder_id_list:
         skin_folder_id_list.reverse()
         this_folder_index = skin_folder_id_list.index(container_id)
-        if skin_location_list.has_key(object_id):
-          existing_folder_index = skin_folder_id_list.index(skin_location_list[object_id])
+        if skin_location_list[selection_name].has_key(object_id):
+          existing_folder_index = skin_folder_id_list.index(skin_location_list[selection_name][object_id])
         else:
           existing_folder_index = this_folder_index + 1
         if existing_folder_index > this_folder_index: