Changeset 6304

Timestamp:

06/17/15 08:12:28 (9 years ago)

Author:

Twan Goosen

Message:

Hierarchy panel hidden for isolated records. Added note to limit number of children shown.
Refs #382

Location:

vlo/trunk/vlo-web-app/src/main/java/eu/clarin/cmdi/vlo/wicket

Files:

• pages/RecordPage.java (modified) (2 diffs)
• panels/record/HierarchyPanel.java (modified) (1 diff)

vlo/trunk/vlo-web-app/src/main/java/eu/clarin/cmdi/vlo/wicket/pages/RecordPage.java

@@ -153 +153 @@
         add(createTechnicalDetailsPanel("technicalProperties"));
 
+        add(createHierarchyPanel("recordtree"));
+
         createSearchLinks("searchlinks");
-
-        add(new HierarchyPanel("recordtree", getModel()));
     }
 
@@ -274 +274 @@
     }
 
+    private HierarchyPanel createHierarchyPanel(String id) {
+        return new HierarchyPanel(id, getModel()) {
+
+            @Override
+            protected void onConfigure() {
+                final SolrDocument document = getModel().getObject();
+                final boolean hasHierarchy // has known parent or children
+                        = null != document.getFieldValue(FacetConstants.FIELD_HAS_PART)
+                        || null != document.getFieldValue(FacetConstants.FIELD_IS_PART_OF);
+
+                // only show hierarchy panel if there's anything to show
+                setVisible(hasHierarchy);
+            }
+
+        };
+    }
+
     @Override
     public void detachModels() {

vlo/trunk/vlo-web-app/src/main/java/eu/clarin/cmdi/vlo/wicket/panels/record/HierarchyPanel.java

@@ -92 +92 @@
 
     private Component createChildrenLinks(String id) {
-        IModel<Collection<String>> partIdsModel = new SolrFieldModel<>(getModel(), FacetConstants.FIELD_HAS_PART);
-        return new ListView<String>(id, new CollectionListModel<>(partIdsModel)) {
+        final IModel<Collection<String>> partIdsModel = new SolrFieldModel<>(getModel(), FacetConstants.FIELD_HAS_PART);
+        //TODO: page or limit (collections can be huge!)
+        return new ListView<String>(id, CollectionListModel.of(partIdsModel)) {
 
             @Override