Parameters are now required.

Do not set any default value anymore.
This should ease compatibility issue (less maintainance).

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

@@ -56,13 +56,24 @@ person = state_change[\'object\']\n
 portal = person.getPortalObject()\n
 # Get required arguments\n
 kwargs = state_change.kwargs\n
-software_release_url_string = state_change.kwargs[\'software_release\']\n
-software_title = kwargs["software_title"]\n
-software_type = kwargs.get("software_type") or "RootSoftwareInstance"\n
-instance_xml = kwargs["instance_xml"]\n
-sla_xml = kwargs.get("sla_xml") or ""\n
-is_slave = kwargs.get("shared", False)\n
-state = kwargs.get(\'state\') or \'started\'\n
+\n
+# Required args\n
+# Raise TypeError if all parameters are not provided\n
+try:\n
+  software_release_url_string = state_change.kwargs[\'software_release\']\n
+  software_title = kwargs["software_title"]\n
+  software_type = kwargs["software_type"]\n
+  instance_xml = kwargs["instance_xml"]\n
+  sla_xml = kwargs["sla_xml"]\n
+  is_slave = kwargs["shared"]\n
+  state = kwargs["state"]\n
+except KeyError:\n
+  raise TypeError, "Person_requestSoftwareInstance takes exactly 7 arguments"\n
+\n
+if state not in ["started", "stopped", "deleted"]:\n
+  raise ValueError, "state should be started, stopped or deleted"\n
+elif is_slave not in [True, False]:\n
+  raise ValueError, "shared should be a boolean"\n
 \n
 hosting_subscription_portal_type = "Hosting Subscription"\n
 \n

master/bt5/vifib_slap/bt/revision

-696
+697
\ No newline at end of file