X-Git-Url: https://git.argeo.org/?a=blobdiff_plain;f=org.argeo.cms%2Fsrc%2Forg%2Fargeo%2Fcms%2Finternal%2FImageManagerImpl.java;h=122c24e3554d7886e457a3884c1f69135647dd4d;hb=6ed2ce8f490e2fc020e67f530f8c08e250b7b6dd;hp=b1d27e516092fe63eda29a750660f7227d974a8f;hpb=d546fc1b9e19a86eb0e33ac4be824b52239316fc;p=lgpl%2Fargeo-commons.git

diff --git a/org.argeo.cms/src/org/argeo/cms/internal/ImageManagerImpl.java b/org.argeo.cms/src/org/argeo/cms/internal/ImageManagerImpl.java
index b1d27e516..122c24e35 100644
--- a/org.argeo.cms/src/org/argeo/cms/internal/ImageManagerImpl.java
+++ b/org.argeo.cms/src/org/argeo/cms/internal/ImageManagerImpl.java
@@ -164,24 +164,25 @@ public class ImageManagerImpl implements CmsImageManager, CmsNames {
 	/** @return null if not available */
 	@Override
 	public String getImageUrl(Node node) throws RepositoryException {
-		String name = getResourceName(node);
-		ResourceManager resourceManager = RWT.getResourceManager();
-		if (!resourceManager.isRegistered(name)) {
-			InputStream inputStream = null;
-			Binary binary = getImageBinary(node);
-			if (binary == null)
-				return null;
-			try {
-				inputStream = binary.getStream();
-				resourceManager.register(name, inputStream);
-			} finally {
-				IOUtils.closeQuietly(inputStream);
-				JcrUtils.closeQuietly(binary);
-			}
-			if (log.isDebugEnabled())
-				log.debug("Registered image " + name);
-		}
-		return resourceManager.getLocation(name);
+		return CmsUtils.getDataPath(node);
+		// String name = getResourceName(node);
+		// ResourceManager resourceManager = RWT.getResourceManager();
+		// if (!resourceManager.isRegistered(name)) {
+		// InputStream inputStream = null;
+		// Binary binary = getImageBinary(node);
+		// if (binary == null)
+		// return null;
+		// try {
+		// inputStream = binary.getStream();
+		// resourceManager.register(name, inputStream);
+		// } finally {
+		// IOUtils.closeQuietly(inputStream);
+		// JcrUtils.closeQuietly(binary);
+		// }
+		// if (log.isTraceEnabled())
+		// log.trace("Registered image " + name);
+		// }
+		// return resourceManager.getLocation(name);
 	}
 
 	protected String getResourceName(Node node) throws RepositoryException {
@@ -225,7 +226,7 @@ public class ImageManagerImpl implements CmsImageManager, CmsNames {
 			if (parentNode.hasNode(fileName)) {
 				Node node = parentNode.getNode(fileName);
 				previousResourceName = getResourceName(node);
-				if (node.hasNode(JCR_CONTENT)){
+				if (node.hasNode(JCR_CONTENT)) {
 					node.getNode(JCR_CONTENT).remove();
 					node.addNode(JCR_CONTENT, NT_RESOURCE);
 				}
@@ -245,7 +246,8 @@ public class ImageManagerImpl implements CmsImageManager, CmsNames {
 
 			// reset resource manager
 			ResourceManager resourceManager = RWT.getResourceManager();
-			if (resourceManager.isRegistered(previousResourceName)) {
+			if (previousResourceName != null
+					&& resourceManager.isRegistered(previousResourceName)) {
 				resourceManager.unregister(previousResourceName);
 				if (log.isDebugEnabled())
 					log.debug("Unregistered image " + previousResourceName);