From 85f6fc93f3f187e44186d6e5c2ea0dc765262585 Mon Sep 17 00:00:00 2001
From: fmittag <fmittag@gmx.net>
Date: Thu, 1 Aug 2013 16:14:35 +0200
Subject: [PATCH] some name cleanup in MultiLeaderboardPanel

---
 .../ui/leaderboard/MetaLeaderboardViewer.java |  2 +-
 .../ui/leaderboard/MultiLeaderboardPanel.java | 64 +++++++++----------
 2 files changed, 33 insertions(+), 33 deletions(-)

diff --git a/java/com.sap.sailing.gwt.ui/src/main/java/com/sap/sailing/gwt/ui/leaderboard/MetaLeaderboardViewer.java b/java/com.sap.sailing.gwt.ui/src/main/java/com/sap/sailing/gwt/ui/leaderboard/MetaLeaderboardViewer.java
index 8a001458b3d..df7dd0b2c3d 100644
--- a/java/com.sap.sailing.gwt.ui/src/main/java/com/sap/sailing/gwt/ui/leaderboard/MetaLeaderboardViewer.java
+++ b/java/com.sap.sailing.gwt.ui/src/main/java/com/sap/sailing/gwt/ui/leaderboard/MetaLeaderboardViewer.java
@@ -108,7 +108,7 @@ public class MetaLeaderboardViewer extends SimplePanel {
             
             @Override
             public void onSuccess(List<Pair<String, String>> leaderboardNamesAndDisplayNames) {
-                multiLeaderboardPanel.setLeaderboardNames(leaderboardNamesAndDisplayNames);
+                multiLeaderboardPanel.setActLeaderboardNames(leaderboardNamesAndDisplayNames);
             }
             
             @Override
diff --git a/java/com.sap.sailing.gwt.ui/src/main/java/com/sap/sailing/gwt/ui/leaderboard/MultiLeaderboardPanel.java b/java/com.sap.sailing.gwt.ui/src/main/java/com/sap/sailing/gwt/ui/leaderboard/MultiLeaderboardPanel.java
index dcb3e733463..4a1f8a4ff45 100644
--- a/java/com.sap.sailing.gwt.ui/src/main/java/com/sap/sailing/gwt/ui/leaderboard/MultiLeaderboardPanel.java
+++ b/java/com.sap.sailing.gwt.ui/src/main/java/com/sap/sailing/gwt/ui/leaderboard/MultiLeaderboardPanel.java
@@ -28,14 +28,13 @@ import com.sap.sailing.gwt.ui.client.shared.components.SettingsDialogComponent;
  */
 public class MultiLeaderboardPanel extends AbstractLazyComponent<LeaderboardSettings> implements TimeListener {
 
-    private LeaderboardPanel selectedLeaderboardPanel;
+    private LeaderboardPanel selectedActLeaderboardPanel;
 
     private final StringMessages stringMessages;
     private final ErrorReporter errorReporter;
     private final SailingServiceAsync sailingService;
 
-    private final String preselectedLeaderboardName;
-    private String selectedLeaderboardName;
+    private String selectedActLeaderboardName;
     
     private final AsyncActionsExecutor asyncActionsExecutor;
     private final UserAgentDetails userAgent;
@@ -43,13 +42,13 @@ public class MultiLeaderboardPanel extends AbstractLazyComponent<LeaderboardSett
     private final Timer timer;
 
     private VerticalPanel mainPanel;
-    private final List<Pair<String, String>> leaderboardNamesAndDisplayNames;
+    private final List<Pair<String, String>> actLeaderboardNamesAndDisplayNames;
     private ListBox leaderboardSelectionListBox;
 
-    private LeaderboardSettings selectedLeaderboardSettings; 
+    private LeaderboardSettings selectedActLeaderboardSettings; 
     
     public MultiLeaderboardPanel(SailingServiceAsync sailingService, AsyncActionsExecutor asyncActionsExecutor, Timer timer,
-            LeaderboardSettings leaderboardSettings, String preselectedLeaderboardName, RaceIdentifier preselectedRace, 
+            LeaderboardSettings leaderboardSettings, String preselectedActLeaderboardName, RaceIdentifier preselectedRace, 
             ErrorReporter errorReporter, StringMessages stringMessages,
             UserAgentDetails userAgent, boolean showRaceDetails, boolean autoExpandLastRaceColumn) {
         this.stringMessages = stringMessages;
@@ -59,10 +58,10 @@ public class MultiLeaderboardPanel extends AbstractLazyComponent<LeaderboardSett
         this.userAgent = userAgent;
         this.showRaceDetails = showRaceDetails;
         this.timer = timer;
-        this.preselectedLeaderboardName = preselectedLeaderboardName;
+        this.selectedActLeaderboardName = preselectedActLeaderboardName;
         
-        leaderboardNamesAndDisplayNames = new ArrayList<Pair<String, String>>();
-        selectedLeaderboardSettings = LeaderboardSettingsFactory.getInstance().createNewDefaultSettings(null, null, null, false);
+        actLeaderboardNamesAndDisplayNames = new ArrayList<Pair<String, String>>();
+        selectedActLeaderboardSettings = LeaderboardSettingsFactory.getInstance().createNewDefaultSettings(null, null, null, false);
     }
 
     @Override
@@ -83,14 +82,14 @@ public class MultiLeaderboardPanel extends AbstractLazyComponent<LeaderboardSett
         leaderboardSelectionListBox.setVisible(false);
         mainPanel.add(leaderboardSelectionListBox);
         
-        updateLeaderboardSelection();
+        updateActLeaderboardSelection();
 
         return mainPanel;
     }
 
     @Override
     public String getLocalizedShortName() {
-        return "Leaderboards";
+        return stringMessages.leaderboards();
     }
 
     @Override
@@ -100,62 +99,63 @@ public class MultiLeaderboardPanel extends AbstractLazyComponent<LeaderboardSett
 
     @Override
     public SettingsDialogComponent<LeaderboardSettings> getSettingsDialogComponent() {
-        return selectedLeaderboardPanel.getSettingsDialogComponent();
+        return selectedActLeaderboardPanel.getSettingsDialogComponent();
     }
 
     @Override
     public void updateSettings(LeaderboardSettings newSettings) {
-        selectedLeaderboardPanel.updateSettings(newSettings);
+        selectedActLeaderboardPanel.updateSettings(newSettings);
     }
 
-    public void setLeaderboardNames(List<Pair<String, String>> newLeaderboardNamesAndDisplayNames) {
-        leaderboardNamesAndDisplayNames.clear();
-        leaderboardNamesAndDisplayNames.addAll(newLeaderboardNamesAndDisplayNames);
+    public void setActLeaderboardNames(List<Pair<String, String>> newLeaderboardNamesAndDisplayNames) {
+        actLeaderboardNamesAndDisplayNames.clear();
+        actLeaderboardNamesAndDisplayNames.addAll(newLeaderboardNamesAndDisplayNames);
         
-        updateLeaderboardSelection();
+        updateActLeaderboardSelection();
     }
 
-    private void updateLeaderboardSelection() {
+    private void updateActLeaderboardSelection() {
         if(leaderboardSelectionListBox != null) {
             leaderboardSelectionListBox.clear();
             
             int index = 0;
-            for (Pair<String, String> leaderboardNameAndDisplayName : leaderboardNamesAndDisplayNames) {
+            for (Pair<String, String> leaderboardNameAndDisplayName : actLeaderboardNamesAndDisplayNames) {
                 leaderboardSelectionListBox.addItem(leaderboardNameAndDisplayName.getB(), leaderboardNameAndDisplayName.getA());
-                if(selectedLeaderboardName != null && selectedLeaderboardName.equals(leaderboardNameAndDisplayName.getA())) {
+
+                if(selectedActLeaderboardName != null && selectedActLeaderboardName.equals(leaderboardNameAndDisplayName.getA())) {
                     leaderboardSelectionListBox.setSelectedIndex(index);
                 }
                 index++;
             }
             
-            leaderboardSelectionListBox.setVisible(leaderboardNamesAndDisplayNames.size() > 0);
+            leaderboardSelectionListBox.setVisible(actLeaderboardNamesAndDisplayNames.size() > 0);
         }
     }
 
     private void updateSelectedLeaderboard(String selectedLeaderboardName) {
         if(selectedLeaderboardName != null) {
-            if(selectedLeaderboardPanel != null) {
-                mainPanel.remove(selectedLeaderboardPanel);
-                selectedLeaderboardPanel = null;
+            if(selectedActLeaderboardPanel != null) {
+                mainPanel.remove(selectedActLeaderboardPanel);
+                selectedActLeaderboardPanel = null;
             }
             
-            selectedLeaderboardPanel = new LeaderboardPanel(sailingService, asyncActionsExecutor,
-                    selectedLeaderboardSettings, /* preselectedRace*/ null, new CompetitorSelectionModel(true), timer,
+            selectedActLeaderboardPanel = new LeaderboardPanel(sailingService, asyncActionsExecutor,
+                    selectedActLeaderboardSettings, /* preselectedRace*/ null, new CompetitorSelectionModel(true), timer,
                     null, selectedLeaderboardName, errorReporter, stringMessages, userAgent,
                     showRaceDetails, /* raceTimesInfoProvider */null, false,  /* adjustTimerDelay */ true);
-            mainPanel.add(selectedLeaderboardPanel);
+            mainPanel.add(selectedActLeaderboardPanel);
         } else {
-            if(selectedLeaderboardPanel != null) {
-                mainPanel.remove(selectedLeaderboardPanel);
-                selectedLeaderboardPanel = null;
+            if(selectedActLeaderboardPanel != null) {
+                mainPanel.remove(selectedActLeaderboardPanel);
+                selectedActLeaderboardPanel = null;
             }
         }
     }
 
     @Override
     public void timeChanged(Date date) {
-        if(selectedLeaderboardPanel != null) {
-            selectedLeaderboardPanel.timeChanged(date);
+        if(selectedActLeaderboardPanel != null) {
+            selectedActLeaderboardPanel.timeChanged(date);
         }
     }