Refactor AboutViewController (#480)

- rename getPaneFor to createPaneFor - rename getViewFor to createViewFor - remove aboutViewController update from ApplicationController
MovingBlocks · Nov 16, 2019 · 1f4c371 · 1f4c371
1 parent a48d817
commit 1f4c371
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 5 deletions.
diff --git a/src/main/java/org/terasology/launcher/gui/javafx/AboutViewController.java b/src/main/java/org/terasology/launcher/gui/javafx/AboutViewController.java
@@ -56,7 +56,7 @@ public void update() {
         Stream.of("README.md", "CHANGELOG.md", "CONTRIBUTING.md", "LICENSE")
                 .map(filename -> BundleUtils.getFXMLUrl(ABOUT, filename))
                 .filter(Objects::nonNull)
-                .map(this::getPaneFor)
+                .map(this::createPaneFor)
                 .filter(Optional::isPresent)
                 .map(Optional::get)
                 .forEach(aboutInfoAccordion.getPanes()::add);
@@ -66,8 +66,8 @@ public void update() {
         }
     }
 
-    private Optional<TitledPane> getPaneFor(URL url) {
-        return getViewFor(url)
+    private Optional<TitledPane> createPaneFor(URL url) {
+        return createViewFor(url)
                 .map(view -> {
                     view.getStylesheets().add(BundleUtils.getFXMLUrl("css_webview").toExternalForm());
                     view.setContextMenuEnabled(false);
@@ -88,7 +88,7 @@ private Optional<TitledPane> getPaneFor(URL url) {
                 });
     }
 
-    private Optional<WebView> getViewFor(URL url) {
+    private Optional<WebView> createViewFor(URL url) {
         switch (Files.getFileExtension(url.getFile().toLowerCase())) {
             case "md":
             case "markdown":

diff --git a/src/main/java/org/terasology/launcher/gui/javafx/ApplicationController.java b/src/main/java/org/terasology/launcher/gui/javafx/ApplicationController.java
@@ -621,7 +621,6 @@ private void updateGui() {
         updateLabels();
         updateTooltipTexts();
         updateChangeLog();
-        aboutViewController.update();
     }
 
     private void updateLabels() {