Skip to content
This repository has been archived by the owner on Sep 16, 2024. It is now read-only.

MlReloadModules is not picking up modules from paths containing spaces #130

Closed
josvanroosmalen opened this issue Mar 13, 2020 · 1 comment

Comments

@josvanroosmalen
Copy link

ml(Re)loadmodules is not picking up modules from paths containing spaces. It is important to get this fixed, because we not always control this (e.g. data stored is user directories containing names).

According to @rjrudin this is related to twice escaping the path names.

12:27:20.894 [INFO] [com.marklogic.hub.deploy.commands.LoadUserModulesCommand] Loading modules from paths: [/Users/jvanroos/clientname/Informatie%20Management/datahub/src/main/ml-modules]
12:27:20.894 [DEBUG] [com.marklogic.client.ext.modulesloader.impl.DefaultModulesLoader] Loading modules from paths: [/Users/jvanroos/clientname/Informatie%20Management/datahub/src/main/ml-modules]
12:27:20.895 [DEBUG] [com.marklogic.client.ext.modulesloader.impl.DefaultModulesFinder] Finding modules in baseDir: /Users/jvanroos/clientname/Informatie%20Management/datahub/src/main/ml-modules
12:27:20.895 [DEBUG] [com.marklogic.client.ext.modulesloader.impl.DefaultModulesFinder] Finding modules in baseDir, which was modified to be: file:/Users/jvanroos/clientname/Informatie%2520Management/datahub/src/main/ml-modules/
12:27:20.896 [DEBUG] [com.marklogic.client.ext.modulesloader.impl.DefaultModulesFinder] Finding non-REST module directories at path: file:/Users/jvanroos/clientname/Informatie%2520Management/datahub/src/main/ml-modules/*
12:27:20.897 [DEBUG] [com.marklogic.client.ext.modulesloader.impl.DefaultModulesFinder] Finding non-REST module directories at path: file:/Users/jvanroos/clientname/Informatie%2520Management/datahub/src/main/ml-modules/*/
@rjrudin rjrudin transferred this issue from marklogic/ml-gradle May 14, 2020
@rjrudin rjrudin changed the title MlReloadModules is not picking up modules from directories MlReloadModules is not picking up modules from paths containing spaces Jul 17, 2020
hansenmc added a commit to hansenmc/ml-javaclient-util that referenced this issue Jan 3, 2021
@hansenmc
Copy link
Contributor

hansenmc commented Jan 4, 2021

It seems that this may have already been resolved?

If I rename the /src/test/resources/path with spaces/root/example/example.xqy file to example file with spaces.xqy and run the pathWithSpaces() test after changing the path to the resource:

String moduleXml = modulesClient.newXMLDocumentManager().read("/example/example file with spaces.xqy", new StringHandle()).get();

it works fine and passes.

hansenmc added a commit to hansenmc/ml-javaclient-util that referenced this issue Mar 21, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants