X-Git-Url: http://git.argeo.org/?a=blobdiff_plain;f=org.argeo.cms.ui%2Fsrc%2Forg%2Fargeo%2Fcms%2Fui%2Fwidgets%2FJcrComposite.java;h=eb43438a0f90c76a2442f84f423352f04137c738;hb=4f6cfbd6862813e917883ea93e13ee95edc1c024;hp=0708365bda6f33d25957251df1fafbc060fa0bfd;hpb=fb3182380d9688998a81d853a4eebcf87b6a48c7;p=lgpl%2Fargeo-commons.git

diff --git a/org.argeo.cms.ui/src/org/argeo/cms/ui/widgets/JcrComposite.java b/org.argeo.cms.ui/src/org/argeo/cms/ui/widgets/JcrComposite.java
index 0708365bd..eb43438a0 100644
--- a/org.argeo.cms.ui/src/org/argeo/cms/ui/widgets/JcrComposite.java
+++ b/org.argeo.cms.ui/src/org/argeo/cms/ui/widgets/JcrComposite.java
@@ -1,14 +1,14 @@
 package org.argeo.cms.ui.widgets;
 
 import javax.jcr.Item;
+import javax.jcr.ItemNotFoundException;
 import javax.jcr.Node;
 import javax.jcr.Property;
 import javax.jcr.RepositoryException;
 import javax.jcr.Session;
 
-import org.argeo.cms.CmsException;
 import org.argeo.cms.ui.util.CmsUiUtils;
-import org.eclipse.swt.SWT;
+import org.argeo.jcr.JcrException;
 import org.eclipse.swt.widgets.Composite;
 
 /** A composite which can (optionally) manage a JCR Item. */
@@ -42,20 +42,20 @@ public class JcrComposite extends Composite {
 //					// not even a reference to the item
 //					this.nodeId = null;
 //				} else {
-					Node node;
-					Property property = null;
-					if (item instanceof Node) {
-						node = (Node) item;
-					} else {// Property
-						property = (Property) item;
-						if (property.isMultiple())// TODO manage property index
-							throw new CmsException("Multiple properties not supported yet.");
-						this.property = property.getName();
-						node = property.getParent();
-					}
-					this.nodeId = node.getIdentifier();
-					if (cacheImmediately)
-						this.cache = node;
+				Node node;
+				Property property = null;
+				if (item instanceof Node) {
+					node = (Node) item;
+				} else {// Property
+					property = (Property) item;
+					if (property.isMultiple())// TODO manage property index
+						throw new UnsupportedOperationException("Multiple properties not supported yet.");
+					this.property = property.getName();
+					node = property.getParent();
+				}
+				this.nodeId = node.getIdentifier();
+				if (cacheImmediately)
+					this.cache = node;
 //				}
 				setLayout(CmsUiUtils.noSpaceGridLayout());
 			} catch (RepositoryException e) {
@@ -66,10 +66,10 @@ public class JcrComposite extends Composite {
 	public synchronized Node getNode() {
 		try {
 			if (!itemIsNode())
-				throw new CmsException("Item is not a Node");
+				throw new IllegalStateException("Item is not a Node");
 			return getNodeInternal();
 		} catch (RepositoryException e) {
-			throw new CmsException("Cannot get node " + nodeId, e);
+			throw new JcrException("Cannot get node " + nodeId, e);
 		}
 	}
 
@@ -88,24 +88,40 @@ public class JcrComposite extends Composite {
 	public synchronized Property getProperty() {
 		try {
 			if (itemIsNode())
-				throw new CmsException("Item is not a Property");
+				throw new IllegalStateException("Item is not a Property");
 			Node node = getNodeInternal();
 			if (!node.hasProperty(property))
-				throw new CmsException("Property " + property + " is not set on " + node);
+				throw new IllegalStateException("Property " + property + " is not set on " + node);
 			return node.getProperty(property);
 		} catch (RepositoryException e) {
-			throw new CmsException("Cannot get property " + property + " from node " + nodeId, e);
+			throw new JcrException("Cannot get property " + property + " from node " + nodeId, e);
 		}
 	}
 
-	public synchronized Boolean itemIsNode() {
+	public synchronized boolean itemIsNode() {
 		return property == null;
 	}
 
+	public synchronized boolean itemExists() {
+		if (session == null)
+			return false;
+		try {
+			Node n = session.getNodeByIdentifier(nodeId);
+			if (!itemIsNode())
+				return n.hasProperty(property);
+			else
+				return true;
+		} catch (ItemNotFoundException e) {
+			return false;
+		} catch (RepositoryException e) {
+			throw new JcrException("Cannot check whether node exists", e);
+		}
+	}
+
 	/** Set/update the cache or change the node */
 	public synchronized void setNode(Node node) {
 		if (!itemIsNode())
-			throw new CmsException("Cannot set a Node on a Property");
+			throw new IllegalArgumentException("Cannot set a Node on a Property");
 
 		if (node == null) {// clear cache
 			this.cache = null;
@@ -116,7 +132,7 @@ public class JcrComposite extends Composite {
 //			if (session != null || session != node.getSession())// check session
 //				throw new IllegalArgumentException("Uncompatible session");
 //			if (session == null)
-				session = node.getSession();
+			session = node.getSession();
 			if (nodeId == null || !nodeId.equals(node.getIdentifier())) {
 				nodeId = node.getIdentifier();
 				cache = node;
@@ -132,7 +148,7 @@ public class JcrComposite extends Composite {
 	/** Set/update the cache or change the property */
 	public synchronized void setProperty(Property prop) {
 		if (itemIsNode())
-			throw new CmsException("Cannot set a Property on a Node");
+			throw new IllegalArgumentException("Cannot set a Property on a Node");
 
 		if (prop == null) {// clear cache
 			this.cache = null;