fix(CLI): Limited user can not operate without errors

[fabrizzio-dotCMS]

Proposed Changes

• FileFetcher is class used by PullService to traverse folders remotely. The flag fail was hard coded to true therefore when a limited user would run into a permission denied response the exceptions would no longer let the pull process complete.
• WeAssetHelper class had issues. It needs to be transactional since it does two operations. It creates the file and then it publishes it (if needed).
• Now we're validating the publish permissions before attempting the operation.
• Another problem that needs attention is that RemoteFolderTraversalTask when instructed to continue instead of fast-failing, can return A Null TreeNode accompanied with a List of Exceptions. But this can lead to an NPE. We need to make small refactoring to make the code more semantic and force revising the state of the returned TreeNode

[dotcms-sonarqube[bot]]

Quality Gate passed

Issues
0 New issues
4 Fixed issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube