+ Add Property.NAME support (use getString() ) to the default node label provider

+ Add some more info in the log to ease debug

git-svn-id: https://svn.argeo.org/commons/trunk@6943 4cfe0d0a-d680-48aa-b62c-e0a02a3f76cc

diff --git a/base/runtime/org.argeo.eclipse.ui.jcr/src/main/java/org/argeo/eclipse/ui/jcr/lists/SimpleJcrNodeLabelProvider.java b/base/runtime/org.argeo.eclipse.ui.jcr/src/main/java/org/argeo/eclipse/ui/jcr/lists/SimpleJcrNodeLabelProvider.java
index fa755bad212b3027c3a0f4d7fafff851c70826f8..88585c3438708d28ceee00ecd148b41175990854 100644 (file)
--- a/base/runtime/org.argeo.eclipse.ui.jcr/src/main/java/org/argeo/eclipse/ui/jcr/lists/SimpleJcrNodeLabelProvider.java
+++ b/base/runtime/org.argeo.eclipse.ui.jcr/src/main/java/org/argeo/eclipse/ui/jcr/lists/SimpleJcrNodeLabelProvider.java
@@ -97,9 +97,14 @@ public class SimpleJcrNodeLabelProvider extends ColumnLabelProvider {
                        return numberFormat.format(value.getDecimal());
                case PropertyType.DOUBLE:
                        return numberFormat.format(value.getDouble());
+               case PropertyType.NAME:
+                       return value.getString();
                default:
                        throw new ArgeoException("Unimplemented label provider "
-                                       + "for property type " + value.getType());
+                                       + "for property type " + value.getType()
+                                       + " while getting property " + propertyName + " - value: "
+                                       + value.getString());
+
                }
        }
 

diff --git a/base/runtime/org.argeo.eclipse.ui.jcr/src/main/java/org/argeo/eclipse/ui/jcr/lists/SimpleJcrRowLabelProvider.java b/base/runtime/org.argeo.eclipse.ui.jcr/src/main/java/org/argeo/eclipse/ui/jcr/lists/SimpleJcrRowLabelProvider.java
index c44aa45cf3e625bd25c146e69475274632a39d40..bb55f187d67794b6d098877362f53fbe02f67055 100644 (file)
--- a/base/runtime/org.argeo.eclipse.ui.jcr/src/main/java/org/argeo/eclipse/ui/jcr/lists/SimpleJcrRowLabelProvider.java
+++ b/base/runtime/org.argeo.eclipse.ui.jcr/src/main/java/org/argeo/eclipse/ui/jcr/lists/SimpleJcrRowLabelProvider.java
@@ -10,6 +10,7 @@ import org.argeo.ArgeoException;
  * Base implementation of a label provider for widgets that display JCR Rows.
  */
 public class SimpleJcrRowLabelProvider extends SimpleJcrNodeLabelProvider {
+       private static final long serialVersionUID = -3414654948197181740L;
 
        final private String selectorName;
 
@@ -39,7 +40,8 @@ public class SimpleJcrRowLabelProvider extends SimpleJcrNodeLabelProvider {
                        Node currNode = currRow.getNode(selectorName);
                        return super.getText(currNode);
                } catch (RepositoryException re) {
-                       throw new ArgeoException("Unable to get text from row", re);
+                       throw new ArgeoException("Unable to get Node " + selectorName
+                                       + " from row " + element, re);
                }
        }
 }
\ No newline at end of file