Make person parameter mandatory.

HostingSubscription_requestRootSoftwareInstance is only used in one place, so, no compatibility problem.

master/bt5/vifib_slap/SkinTemplateItem/portal_skins/vifib_slap/HostingSubscription_requestRootSoftwareInstance.xml

@@ -82,14 +82,6 @@ request_software_instance = hosting_subscription.portal_catalog.getResultValue(\
   root_uid=hosting_subscription.getUid(),\n
 )\n
 \n
-if person is None:\n
-  open_sale_order_line = hosting_subscription.portal_catalog.getResultValue(\n
-    portal_type=\'Open Sale Order Line\', strict_aggregate_uid=hosting_subscription.getUid())\n
-  open_sale_order = open_sale_order_line.getParentValue()\n
-  person_relative_url = open_sale_order.getDestinationSection()\n
-else:\n
-  person_relative_url = person.getRelativeUrl()\n
-\n
 new_one = False\n
 if (request_software_instance is None):\n
   new_one = True\n
@@ -180,7 +172,7 @@ return request_software_instance\n
         </item>
         <item>
             <key> <string>_params</string> </key>
-            <value> <string>tag, person=None</string> </value>
+            <value> <string>tag, person_relative_url</string> </value>
         </item>
         <item>
             <key> <string>id</string> </key>

master/bt5/vifib_slap/WorkflowTemplateItem/portal_workflow/person_slap_interface_workflow/scripts/Person_requestSoftwareInstance.xml

@@ -136,7 +136,7 @@ elif (root_state == "destroyed"):\n
 else:\n
   raise ValueError, "state should be started, stopped or destroyed"\n
 \n
-request_hosting_subscription.HostingSubscription_requestRootSoftwareInstance(tag, person=person)\n
+request_hosting_subscription.HostingSubscription_requestRootSoftwareInstance(tag, person.getRelativeUrl())\n
 \n
 # Change the state at the end to allow to execute updateLocalRoles only once in the transaction\n
 validation_state = request_hosting_subscription.getValidationState()\n

master/bt5/vifib_slap/bt/revision

-716
+717
\ No newline at end of file