fixed #523

Author: paxtonhare

marklogic-data-hub/src/main/resources/ml-modules/root/com.marklogic.hub/mlcp-flow-transform.xqy

@@ -44,9 +44,12 @@ declare function mlcpFlow:transform(
           map:entry($parts[1], $parts[2])
       ))
 
+      let $job-id := (map:get($params, "job-id"), sem:uuid-string())[1]
+      let $entity-name := map:get($params, 'entity-name') ! xdmp:url-decode(.)
+      let $flow-name := map:get($params, 'flow-name') ! xdmp:url-decode(.)
       let $flow := flow:get-flow(
-        map:get($params, 'entity-name'),
-        map:get($params, 'flow-name'),
+        $entity-name,
+        $flow-name,
         $consts:INPUT_FLOW
       )
 
@@ -64,7 +67,7 @@ declare function mlcpFlow:transform(
 
       (: this can throw, but we want MLCP to know about problems, so let it :)
       let $envelope := mlcpFlow:run-flow(
-        map:get($params, "job-id"), $flow, $uri, map:get($content, "value"), $options
+        $job-id, $flow, $uri, map:get($content, "value"), $options
       )
       let $_ := map:put($content, "value", $envelope)
       return

ml-data-hub-plugin/src/main/groovy/com/marklogic/gradle/task/HubTask.groovy

@@ -6,41 +6,51 @@ import com.marklogic.appdeployer.command.CommandContext
 import com.marklogic.client.DatabaseClient
 import com.marklogic.hub.*
 import org.gradle.api.DefaultTask
+import org.gradle.api.tasks.Internal
 
 abstract class HubTask extends DefaultTask {
 
+    @Internal
     HubConfig getHubConfig() {
         getProject().property("hubConfig")
     }
 
+    @Internal
     DataHub getDataHub() {
         getProject().property("dataHub")
     }
 
+    @Internal
     Tracing getTracing() {
         return new Tracing(getStagingClient())
     }
 
+    @Internal
     Debugging getDebugging() {
         return new Debugging(getStagingClient())
     }
 
+    @Internal
     FlowManager getFlowManager() {
         return new FlowManager(getHubConfig())
     }
 
+    @Internal
     DatabaseClient getStagingClient() {
         return getHubConfig().newStagingClient()
     }
 
+    @Internal
     DatabaseClient getFinalClient() {
         return getHubConfig().newFinalClient()
     }
 
+    @Internal
     CommandContext getCommandContext() {
         getProject().property("mlCommandContext")
     }
 
+    @Internal
     boolean isHubInstalled() {
         InstallInfo installInfo = getDataHub().isInstalled();
         return installInfo.isInstalled();

quick-start/src/main/ui/app/mlcp-ui/mlcp-ui.component.ts

@@ -145,7 +145,7 @@ export class MlcpUiComponent implements OnChanges {
             field: 'output_collections',
             type: 'comma-list',
             description: 'A comma separated list of collection URIs. Loaded documents are added to these collections.',
-            value: entityName + ',' + flowName + ',input',
+            value: entityName.replace(new RegExp(' ', 'g'), '') + ',' + flowName.replace(new RegExp(' ', 'g'), '') + ',input',
           },
           {
             label: 'Output Permissions',
@@ -384,7 +384,7 @@ export class MlcpUiComponent implements OnChanges {
             field: 'transform_param',
             type: 'string',
             description: 'Optional extra data to pass through to a custom transformation function. Ignored if -transform_module is not specified.\nDefault: no namespace. For details, see Transforming Content During Ingestion.',
-            value: `entity-name=${entityName},flow-name=${flowName}`,
+            value: `entity-name=${encodeURIComponent(entityName)},flow-name=${encodeURIComponent(flowName)}`,
             readOnly: true,
           },
         ],