use RoleInformation methods getCategory and getBaseCategory instead of

accessing atributes directly.
add test for missing role information values



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

product/ERP5/Document/BusinessTemplate.py

@@ -2323,11 +2323,8 @@ class PortalTypeRolesTemplateItem(BaseTemplateItem):
         if prop_value:
           type_role_dict['condition'] = prop_value.text
         # multi
-        for property in ('category', 'base_category'):
-          prop_value_list = []
-          for prop_value in getattr(role, property):
-            prop_value_list.append(prop_value)
-          type_role_dict[property] = prop_value_list
+        type_role_dict['category'] = role.getCategory()
+        type_role_dict['base_category'] = role.getBaseCategory()
         type_role_list.append(type_role_dict)
       self._objects[relative_url] = type_role_list
 

product/ERP5/tests/testBusinessTemplate.py

@@ -4935,8 +4935,8 @@ class TestBusinessTemplate(ERP5TypeTestCase, LogInterceptor):
                        name='Geek Role Definition',
                        condition='',
                        category='group/g1\nfunction/f1\n',
-                       base_category_script='',
-                       base_category='',)
+                       base_category_script='Base Category Script',
+                       base_category='group site',)
 
     sequence.edit(portal_type_role='geek_role_definition')
 
@@ -4968,6 +4968,8 @@ class TestBusinessTemplate(ERP5TypeTestCase, LogInterceptor):
     self.assertEquals('Geek Role Definition', role.title)
     self.assertEquals('A definition with non ascii chars éàè', role.description)
     self.assertEquals(('group/g1','function/f1'), role.getCategory())
+    self.assertEquals(('group','site'), role.getBaseCategory())
+    self.assertEquals('Base Category Script', role.getBaseCategoryScript())
 
   def test_36_CheckPortalTypeRoles(self, quiet=quiet, run=run_all_test):
     if not run: return