Move Roles to an early step and unify the way roles are generated with consulting.

bt5/erp5_configurator_standard/SkinTemplateItem/portal_skins/erp5_configurator_consulting/BusinessConfiguration_viewConfigurePortalTypeRolesDialog/your_explanation.xml

@@ -221,7 +221,7 @@
 
 Tips: <br>\n
 <br>\n
-<li> You can use this sample file <a href="consulting_configurator_sample_roles_configuration_sheet.ods"> Consulting.Configurator.Sample.Roles.Configuration.Sheet.ods </a></li>
+<li> You can use this sample file <a href="standard_portal_types_roles.ods"> Consulting.Configurator.Sample.Roles.Configuration.Sheet.ods </a></li>
 
 ]]></string> </value>
                 </item>

bt5/erp5_configurator_standard/SkinTemplateItem/portal_skins/erp5_configurator_standard/BusinessConfiguration_afterConfiguration.xml

@@ -67,7 +67,7 @@ if business_template is not None:\n
                     filter=dict(portal_type=portal_type))\n
     for module in module_list:\n
       module.updateLocalRolesOnSecurityGroups()\n
-      print "Updated Role Mappings for: %s(%s) " %(module.getTitle(), module.getPortalType())\n
+      print "Updated Role Mappings for: %s(%s) " % (module.getTitle(), module.getPortalType())\n
 \n
   for preference_to_enable in business_template.getTemplatePreferenceList():\n
     preference = getattr(portal_preferences, preference_to_enable, None)\n

bt5/erp5_configurator_standard/SkinTemplateItem/portal_skins/erp5_configurator_standard/BusinessConfiguration_displayDownload.xml

@@ -70,15 +70,11 @@ configuration_save.addConfigurationItem("Purchase Trade Condition Configurator I
                                          title="General Purchase Trade Condition",\n
                                          reference="PTC-General")\n
 \n
-# setup default security\n
-configuration_save.addConfigurationItem("Role Configurator Item", \n
-                                        filename="standard_portal_types_roles.ods",\n
-                                        group_id = \'group/%s\' %group_id)\n
-\n
 # setup Solvers\n
 for property_dict in context.BusinessConfiguration_getSolverPropertyDict().itervalues():\n
   configuration_save.addConfigurationItem("Solver Configurator Item", **property_dict)\n
 \n
+
 # web site module security\n
 # we will not provide web_site_module for now.\n
 configuration_save.addConfigurationItem("Permission Configurator Item",\n

bt5/erp5_configurator_standard/SkinTemplateItem/portal_skins/erp5_configurator_standard/BusinessConfiguration_setupPortalTypeRole.xml

+<?xml version="1.0"?>
+<ZopeData>
+  <record id="1" aka="AAAAAAAAAAE=">
+    <pickle>
+      <global name="PythonScript" module="Products.PythonScripts.PythonScript"/>
+    </pickle>
+    <pickle>
+      <dictionary>
+        <item>
+            <key> <string>Script_magic</string> </key>
+            <value> <int>3</int> </value>
+        </item>
+        <item>
+            <key> <string>_bind_names</string> </key>
+            <value>
+              <object>
+                <klass>
+                  <global name="NameAssignments" module="Shared.DC.Scripts.Bindings"/>
+                </klass>
+                <tuple/>
+                <state>
+                  <dictionary>
+                    <item>
+                        <key> <string>_asgns</string> </key>
+                        <value>
+                          <dictionary>
+                            <item>
+                                <key> <string>name_container</string> </key>
+                                <value> <string>container</string> </value>
+                            </item>
+                            <item>
+                                <key> <string>name_context</string> </key>
+                                <value> <string>context</string> </value>
+                            </item>
+                            <item>
+                                <key> <string>name_m_self</string> </key>
+                                <value> <string>script</string> </value>
+                            </item>
+                            <item>
+                                <key> <string>name_subpath</string> </key>
+                                <value> <string>traverse_subpath</string> </value>
+                            </item>
+                          </dictionary>
+                        </value>
+                    </item>
+                  </dictionary>
+                </state>
+              </object>
+            </value>
+        </item>
+        <item>
+            <key> <string>_body</string> </key>
+            <value> <string>configuration_save = context.restrictedTraverse(configuration_save_url)\n
+group_id = context.getGlobalConfigurationAttr(\'group_id\')\n
+\n
+context.setGlobalConfigurationAttr(\n
+   portal_type_roles_spreadsheet_configuration_save_relative_url=configuration_save.getRelativeUrl())\n
+\n
+configuration_save.addConfigurationItem("Portal Type Roles Spreadsheet Configurator Item",\n
+                   configuration_spreadsheet_data = getattr(context, "standard_portal_types_roles.ods").data)\n
+</string> </value>
+        </item>
+        <item>
+            <key> <string>_params</string> </key>
+            <value> <string>configuration_save_url=None, **kw</string> </value>
+        </item>
+        <item>
+            <key> <string>id</string> </key>
+            <value> <string>BusinessConfiguration_setupPortalTypeRole</string> </value>
+        </item>
+        <item>
+            <key> <string>title</string> </key>
+            <value> <string>Create a bt5 template that will contain customer configuration</string> </value>
+        </item>
+      </dictionary>
+    </pickle>
+  </record>
+</ZopeData>