testxhtml debiug

product/ERP5/tests/testXHTML.py

@@ -112,15 +112,22 @@ class TestXHTMLMixin(ERP5TypeTestCase):
           except BrokenProxyField:
             template_field = None
           if template_field is None:
-            # field's parent form can actually exists but not in current skin, check if it can be found in portal_skins
-            found_list_action_list = skins_tool.ZopeFind(skins_tool, obj_ids=[field.form.id], search_sub=1)
-            if found_list_action_list and (found_list_action_list[0][1] != field.form):
+            # check if the field parent form can be used by this skin_name
+            field_form_id = field_path.split('/')[0]
+            if not skins_tool.isFirstInSkin('%s/%s' % (skin_folder_id, field_form_id), skin=skin_name):
+              """
+            found_list_form_list = skins_tool.ZopeFind(skins_tool, obj_ids=[field_form_id], search_sub=1, obj_skinspec=skin_name)
+            if found_list_form_list and (not found_list_form_list[0][0].startswith(skin_folder_id)):"""
+              # The problematic form will not be used by this skin selection
+              # as another object with the same name exists
+              error_list.append(('not critical %s', field_path, 'field_form_id %s' % (field_form_id),
+                                 field.get_value('field_id')))
               continue
             # Base_viewRelatedObjectList (used for proxy listbox ids on
             # relation fields) is an exception, the proxy field has no target
             # by default.
             if field_path != 'Base_viewRelatedObjectList/listbox':
-              error_list.append((skin_name, field_path, field.get_value('form_id'),
+              error_list.append((skin_name, '%s %s !!%s %s' % (found_list_form_list, skin_folder_id, field_form_id, field_path), field.get_value('form_id'),
                                  field.get_value('field_id')))
 
     if error_list: