I think the intent of the fix is good - if the "vmfolder" attribute is set to an absolute folder path, it's valid to try and use that. If not, then interpret as a path relative to the DC folder.
We can go 2 ways - enforce a relative path by requiring a "./" prefix, or signal an absolute path with a "/" prefix. The latter seems best.
I think the intent of the fix is good - if the "vmfolder" attribute is set to an absolute folder path, it's valid to try and use that. If not, then interpret as a path relative to the DC folder.
We can go 2 ways - enforce a relative path by requiring a "./" prefix, or signal an absolute path with a "/" prefix. The latter seems best.
We'll get a fix landed.