only show details pane, when cache is selected

src/main/java/de/geofroggerfx/ui/details/DetailsController.java

@@ -26,10 +26,8 @@
 package de.geofroggerfx.ui.details;
 
 import com.lynden.gmapsfx.GoogleMapView;
-import com.lynden.gmapsfx.MapComponentInitializedListener;
 import com.lynden.gmapsfx.javascript.object.*;
 import de.geofroggerfx.application.SessionContext;
-import de.geofroggerfx.application.SessionContextListener;
 import de.geofroggerfx.model.Attribute;
 import de.geofroggerfx.model.Cache;
 import de.geofroggerfx.model.Log;
@@ -68,6 +66,9 @@ public class DetailsController extends FXMLController {
     @Autowired
     private SessionContext sessionContext;
 
+    @FXML
+    private BorderPane detailsPane;
+
     @FXML
     private Label cacheName;
 
@@ -107,45 +108,38 @@ public class DetailsController extends FXMLController {
 
     @Override
     public void initialize(URL location, ResourceBundle resources) {
+        detailsPane.setVisible(false);
         setCacheListener();
         initializeMap();
         setLogLinkButton();
     }
 
     private void setCacheListener() {
-        sessionContext.addListener(CURRENT_CACHE, new SessionContextListener() {
+        sessionContext.addListener(CURRENT_CACHE, () -> fillContent());
-            @Override
-            public void sessionContextChanged() {
-                fillContent();
-            }
-        });
     }
 
     private void initializeMap() {
-        mapView.addMapInializedListener(new MapComponentInitializedListener() {
+        mapView.addMapInializedListener(() -> {
-            @Override
+            //Set the initial properties of the map.
-            public void mapInitialized() {
+            MapOptions mapOptions = new MapOptions();
-                //Set the initial properties of the map.
+            mapOptions.center(new LatLong(47.6097, -122.3331))
-                MapOptions mapOptions = new MapOptions();
+                    .mapType(MapTypeIdEnum.ROADMAP)
-                mapOptions.center(new LatLong(47.6097, -122.3331))
+                    .overviewMapControl(false)
-                        .mapType(MapTypeIdEnum.ROADMAP)
+                    .panControl(false)
-                        .overviewMapControl(false)
+                    .rotateControl(false)
-                        .panControl(false)
+                    .scaleControl(false)
-                        .rotateControl(false)
+                    .streetViewControl(false)
-                        .scaleControl(false)
+                    .zoomControl(false)
-                        .streetViewControl(false)
+                    .zoom(12);
-                        .zoomControl(false)
-                        .zoom(12);
 
-                map = mapView.createMap(mapOptions);
+            map = mapView.createMap(mapOptions);
 
-                LatLong latLong = new LatLong(49.045458, 9.189835);
+            LatLong latLong = new LatLong(49.045458, 9.189835);
-                MarkerOptions options = new MarkerOptions();
+            MarkerOptions options = new MarkerOptions();
-                options.position(latLong);
+            options.position(latLong);
-                marker = new Marker(options);
+            marker = new Marker(options);
-                map.addMarker(marker);
+            map.addMarker(marker);
-                map.setCenter(latLong);
+            map.setCenter(latLong);
-            }
         });
     }
 
@@ -232,48 +226,54 @@ public class DetailsController extends FXMLController {
 
     private void fillContent() {
         Cache cache = (Cache) sessionContext.getData(CURRENT_CACHE);
-        cacheName.setText(cache.getName());
-        cacheDifficulty.setText(getStarsAsString(cache.getDifficulty()));
-        cacheTerrain.setText(getStarsAsString(cache.getTerrain()));
 
-        String shortDescription = cache.getShortDescription();
+        if (cache != null) {
-        if (!cache.getLongDescriptionHtml()) {
+            detailsPane.setVisible(true);
-            shortDescription = shortDescription.replaceAll("\n","</br>");
+            cacheName.setText(cache.getName());
+            cacheDifficulty.setText(getStarsAsString(cache.getDifficulty()));
+            cacheTerrain.setText(getStarsAsString(cache.getTerrain()));
+
+            String shortDescription = cache.getShortDescription();
+            if (!cache.getLongDescriptionHtml()) {
+                shortDescription = shortDescription.replaceAll("\n", "</br>");
+            }
+
+            String longDescription = cache.getLongDescription();
+            if (!cache.getLongDescriptionHtml()) {
+                longDescription = longDescription.replaceAll("\n", "</br>");
+            }
+
+            String description = "<html><body><p>" + shortDescription + "</p><p>" + longDescription + "</p></body></html>";
+            final WebEngine webEngine = this.description.getEngine();
+            webEngine.loadContent(description);
+
+            attributeList.getChildren().clear();
+            for (Attribute attribute : cache.getAttributes()) {
+                attributeList.getChildren().add(MaterialDesignIconFactory.get().createIcon(GeocachingIcons.getIcon(attribute)));
+            }
+
+            logList.getChildren().clear();
+            int maxRow = Math.min(4, cache.getLogs().size());
+            for (int row = 0; row < maxRow; row++) {
+                setLogIcon(row);
+                setLogText(row, cache.getLogs().get(row));
+            }
+
+            LatLong latLong = new LatLong(cache.getMainWayPoint().getLatitude(), cache.getMainWayPoint().getLongitude());
+            MarkerOptions options = new MarkerOptions();
+            options.position(latLong);
+            options.title(cache.getMainWayPoint().getName());
+
+            map.removeMarker(marker);
+
+            marker.setTitle(cache.getMainWayPoint().getName());
+            marker = new Marker(options);
+            map.addMarker(marker);
+            map.setCenter(latLong);
+            mapView.requestLayout();
+        } else {
+            detailsPane.setVisible(false);
         }
-
-        String longDescription = cache.getLongDescription();
-        if (!cache.getLongDescriptionHtml()) {
-            longDescription = longDescription.replaceAll("\n","</br>");
-        }
-
-        String description = "<html><body><p>"+shortDescription+"</p><p>"+longDescription+"</p></body></html>";
-        final WebEngine webEngine = this.description.getEngine();
-        webEngine.loadContent(description);
-
-        attributeList.getChildren().clear();
-        for (Attribute attribute: cache.getAttributes()) {
-            attributeList.getChildren().add(MaterialDesignIconFactory.get().createIcon(GeocachingIcons.getIcon(attribute)));
-        }
-
-        logList.getChildren().clear();
-        int maxRow = Math.min(4, cache.getLogs().size());
-        for (int row=0; row<maxRow; row++) {
-            setLogIcon(row);
-            setLogText(row, cache.getLogs().get(row));
-        }
-
-        LatLong latLong = new LatLong(cache.getMainWayPoint().getLatitude(), cache.getMainWayPoint().getLongitude());
-        MarkerOptions options = new MarkerOptions();
-        options.position(latLong);
-        options.title(cache.getMainWayPoint().getName());
-
-        map.removeMarker(marker);
-
-        marker.setTitle(cache.getMainWayPoint().getName());
-        marker = new Marker(options);
-        map.addMarker(marker);
-        map.setCenter(latLong);
-        mapView.requestLayout();
     }
 
     private void setLogIcon(int row) {

src/main/resources/de/geofroggerfx/ui/details/details.fxml

@@ -8,7 +8,7 @@
 <?import javafx.scene.layout.*?>
 <?import javafx.scene.web.WebView?>
 <?import java.net.URL?>
-<BorderPane layoutX="10.0" layoutY="10.0" maxHeight="-Infinity" maxWidth="-Infinity" minHeight="600" minWidth="400" style="-fx-background-color: #ffffff;" AnchorPane.bottomAnchor="0.0" AnchorPane.leftAnchor="0.0" AnchorPane.rightAnchor="0.0" AnchorPane.topAnchor="0.0" xmlns="http://javafx.com/javafx/8.0.40" xmlns:fx="http://javafx.com/fxml/1">
+<BorderPane fx:id="detailsPane" layoutX="10.0" layoutY="10.0" maxHeight="-Infinity" maxWidth="-Infinity" minHeight="600" minWidth="400" style="-fx-background-color: #ffffff;" AnchorPane.bottomAnchor="0.0" AnchorPane.leftAnchor="0.0" AnchorPane.rightAnchor="0.0" AnchorPane.topAnchor="0.0" xmlns="http://javafx.com/javafx/8.0.40" xmlns:fx="http://javafx.com/fxml/1">
     <right>
         <VBox minWidth="200.0" prefHeight="200.0" spacing="8.0" BorderPane.alignment="CENTER">
             <children>