Merge pull request #141 from paxtonhare/140_fix_vet

fixed #140 - vet plugins not working correctly

Author: dmcassel

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

@@ -612,16 +612,41 @@ declare function flow:validate-entities()
     let $_ :=
       for $plugin in $flow/hub:plugins/hub:plugin
       let $destination := $plugin/@dest
+      let $module-uri := $plugin/@module
+      let $module-name := hul:get-module-name($module-uri)
+      let $filename as xs:string := hul:get-file-from-uri($module-uri)
+      let $type := flow:get-type($filename)
+      let $ns :=
+        if ($type eq $flow:TYPE-JAVASCRIPT) then ()
+        else
+          $flow:PLUGIN-NS || fn:lower-case($module-name)
+      let $func-name :=
+        if ($type eq $flow:TYPE-JAVASCRIPT) then
+          "create" || functx:capitalize-first($destination)
+        else
+          "create-" || $destination
       return
+        (:
+         : Note that we are static checking the files.
+         : This is because there is no reasonable way to actually
+         : call the plugins and pass in data that will work for all plugins.
+         :
+         : The disadvantage to static checking is that we will not catch typos
+         : like ctsdoc <- (missing period) because Javascript is dynamically
+         : typed. Static checking will only catch syntax errors in sjs.
+         :)
         try {
-          flow:run-plugin(
-            $plugin,
-            $data-format,
-            "123",
-            (),
-            (),
-            (),
-            $options)
+          if ($type eq $flow:TYPE-XQUERY) then
+            xdmp:eval(
+              'import module namespace x = "' || $ns || '" at "' || $module-uri || '"; ' ||
+              '()',
+              map:new(map:entry("staticCheck", fn:true()))
+            )
+          else
+            xdmp:javascript-eval(
+              'var x = require("' || $module-uri || '");',
+              map:new(map:entry("staticCheck", fn:true()))
+            )
         }
         catch($ex) {
           json:array-push($errors, flow:make-error-json($ex))