diff --git a/dotCMS/src/main/java/com/dotmarketing/portlets/contentlet/model/ResourceLink.java b/dotCMS/src/main/java/com/dotmarketing/portlets/contentlet/model/ResourceLink.java index a03391ba576b..6415dadadf4a 100644 --- a/dotCMS/src/main/java/com/dotmarketing/portlets/contentlet/model/ResourceLink.java +++ b/dotCMS/src/main/java/com/dotmarketing/portlets/contentlet/model/ResourceLink.java @@ -352,7 +352,10 @@ boolean isDownloadPermissionBasedRestricted(final Contentlet contentlet, final U public String getFileLink(final HttpServletRequest request, final User user, final Contentlet contentlet, final String field) throws DotDataException, DotSecurityException { ResourceLink link = this.build(request, user, contentlet, field); - final Host host = getHost(contentlet.getHost(), user); + Host host = getHost((String)request.getAttribute(HOST_REQUEST_ATTRIBUTE) , user); + if(null == host){ + host = getHost(contentlet.getHost(), user); + } final StringBuilder hostUrlBuilder = new StringBuilder(request.isSecure()? HTTPS_PREFIX:HTTP_PREFIX); diff --git a/dotCMS/src/main/java/com/dotmarketing/portlets/fileassets/business/FileAssetAPIImpl.java b/dotCMS/src/main/java/com/dotmarketing/portlets/fileassets/business/FileAssetAPIImpl.java index 460cf189e2bd..b8401a9e52ac 100644 --- a/dotCMS/src/main/java/com/dotmarketing/portlets/fileassets/business/FileAssetAPIImpl.java +++ b/dotCMS/src/main/java/com/dotmarketing/portlets/fileassets/business/FileAssetAPIImpl.java @@ -344,7 +344,8 @@ private FileAsset transformDotAsset(Contentlet con) { final HttpServletRequest request = HttpServletRequestThreadLocal.INSTANCE.getRequest(); if (request != null) { - final String fileLink = new ResourceLink.ResourceLinkBuilder().getFileLink(request, APILocator.systemUser(), fileAsset, "fileAsset"); + final String fileLink = new ResourceLink.ResourceLinkBuilder().build(request, APILocator.systemUser(), fileAsset, FileAssetAPI.BINARY_FIELD).getConfiguredImageURL(); + fileAsset.getMap().put("fileLink", fileLink); } @@ -479,7 +480,7 @@ private String getRelativeAssetPath(String inode, String fileName) { String path = ""; path = java.io.File.separator + _inode.charAt(0) - + java.io.File.separator + _inode.charAt(1) + java.io.File.separator + _inode + java.io.File.separator + "fileAsset" + java.io.File.separator+ fileName; + + java.io.File.separator + _inode.charAt(1) + java.io.File.separator + _inode + java.io.File.separator + FileAssetAPI.BINARY_FIELD + java.io.File.separator+ fileName; return path; @@ -620,7 +621,7 @@ public String getRealAssetPath(final String inode, final String fileName, final final String fullFileName = UtilMethods.isSet(ext) ? fileName + "." + ext : fileName; final String path = ((!UtilMethods.isSet(realPath)) ? assetPath : realPath) + inode.charAt(0) + java.io.File.separator + inode.charAt(1) - + java.io.File.separator + inode+ java.io.File.separator + "fileAsset" + java.io.File.separator + fullFileName; + + java.io.File.separator + inode+ java.io.File.separator + FileAssetAPI.BINARY_FIELD + java.io.File.separator + fullFileName; if (!UtilMethods.isSet(realPath)) { return FileUtil.getRealPath(path); @@ -680,7 +681,7 @@ public String getRealAssetPath(String inode) { path = ((!UtilMethods.isSet(realPath)) ? assetPath : realPath) + _inode.charAt(0) + java.io.File.separator + _inode.charAt(1) - + java.io.File.separator + _inode+ java.io.File.separator + "fileAsset" + java.io.File.separator; + + java.io.File.separator + _inode+ java.io.File.separator + FileAssetAPI.BINARY_FIELD + java.io.File.separator; if (!UtilMethods.isSet(realPath)) return FileUtil.getRealPath(path); diff --git a/dotCMS/src/main/webapp/html/portlet/ext/workflows/view_workflow_task.jsp b/dotCMS/src/main/webapp/html/portlet/ext/workflows/view_workflow_task.jsp index faa9c3b91348..863e505849f4 100644 --- a/dotCMS/src/main/webapp/html/portlet/ext/workflows/view_workflow_task.jsp +++ b/dotCMS/src/main/webapp/html/portlet/ext/workflows/view_workflow_task.jsp @@ -412,11 +412,14 @@ public String getGravatar(String postedBy){ } %> > - " target="_blank"> - <%= (fileAsset != null && fileAsset.getMap().get("fileLink") != null) ? fileAsset.getUnderlyingFileName() : file.getFileName() %> - + " target="_blank"> + <%= (fileAsset != null && fileAsset.getMap().get("fileLink") != null) ? fileAsset.getUnderlyingFileName() : file.getFileName() %> + + + +