Fixed Race Map Bug

java/com.sap.sailing.gwt.ui/src/main/java/com/sap/sailing/gwt/ui/adminconsole/RacesListBoxPanel.java

@@ -50,7 +50,7 @@ public class RacesListBoxPanel extends FormPanel implements RaceSelectionProvide
                 Triple<EventDAO, RegattaDAO, RaceDAO> selectedRace = getSelectedRace();
                 List<Triple<EventDAO, RegattaDAO, RaceDAO>> selectedRacesCollection = Collections.emptyList();
                 if (selectedRace != null) {
-                    Collections.singletonList(selectedRace);
+                    selectedRacesCollection = Collections.singletonList(selectedRace);
                 }
                 fireRaceSelectionChanged(selectedRacesCollection);
             }

java/com.sap.sailing.gwt.ui/src/main/java/com/sap/sailing/gwt/ui/shared/EventDAO.java

@@ -33,6 +33,7 @@ public class EventDAO extends NamedDAO implements IsSerializable {
     public String getLocationAsString() {
         //TODO Finish the format of the location string
         if (locations == null) {
+            locations = "";
             for (RegattaDAO regattaDAO : regattas) {
                 for (RaceDAO raceDAO : regattaDAO.races) {
                     RacePlaceOrder placeOrder = raceDAO.racePlaces;

java/com.sap.sailing.gwt.ui/src/main/java/com/sap/sailing/gwt/ui/spectator/OverviewEventPanel.java

@@ -165,7 +165,6 @@ public class OverviewEventPanel extends AbstractEventPanel {
         functionPanelEvents.add(btnShowLeaderboards);
 
         // Create event table
-        // TODO
         {
             eventTable = new CellTable<EventDAO>();
             eventTable.setWidth("100%");
@@ -178,6 +177,7 @@ public class OverviewEventPanel extends AbstractEventPanel {
                     return locations != null ? locations : stringConstants.locationNotAvailable();
                 }
             };
+            locationColumn.setSortable(true);
             //Creating event name column
             TextColumn<EventDAO> nameColumn = new TextColumn<EventDAO>() {
                 @Override
@@ -185,14 +185,16 @@ public class OverviewEventPanel extends AbstractEventPanel {
                     return eventDAO.name;
                 }
             };
+            nameColumn.setSortable(true);
             //Creating start date column
             TextColumn<EventDAO> startDateColumn = new TextColumn<EventDAO>() {
                 @Override
                 public String getValue(EventDAO eventDAO) {
                     Date start = eventDAO.regattas.get(0).races.get(0).startOfRace;
-                    return start != null ? start.toString() : stringConstants.startDateNotAvailable();
+                    return start != null ? dateFormatter.render(start) : stringConstants.startDateNotAvailable();
                 }
             };
+            startDateColumn.setSortable(true);
             
             eventTable.addColumn(locationColumn, stringConstants.location());
             eventTable.addColumn(nameColumn, stringConstants.eventName());
@@ -243,7 +245,7 @@ public class OverviewEventPanel extends AbstractEventPanel {
         functionPanelLeaderboards.add(btnHideLeaderboards);
 
         // Create leaderboard list
-        //  TODO
+        // TODO
         
         loadEvents();
         
@@ -254,30 +256,6 @@ public class OverviewEventPanel extends AbstractEventPanel {
         //Until here
     }
 
-    private ListHandler<EventDAO> getEventSortHandler(List<EventDAO> list, TextColumn<EventDAO> locationColumn,
-            TextColumn<EventDAO> nameColumn, TextColumn<EventDAO> startDateColumn) {
-        ListHandler<EventDAO> sortHandler = new ListHandler<EventDAO>(list);
-        sortHandler.setComparator(locationColumn, new Comparator<EventDAO>() {
-            @Override
-            public int compare(EventDAO e1, EventDAO e2) {
-                return e1.getLocationAsString().compareTo(e2.getLocationAsString());
-            }
-        });
-        sortHandler.setComparator(nameColumn, new Comparator<EventDAO>() {
-            @Override
-            public int compare(EventDAO e1, EventDAO e2) {
-                return e1.name.compareTo(e2.name);
-            }
-        });
-        sortHandler.setComparator(startDateColumn, new Comparator<EventDAO>() {
-            @Override
-            public int compare(EventDAO e1, EventDAO e2) {
-                return e1.getStartDate().compareTo(e2.getStartDate());
-            }
-        });
-        return sortHandler;
-    }
-
     private void loadEvents(final Runnable actionAfterLoading) {
         sailingService.listEvents(true, new AsyncCallback<List<EventDAO>>() {
 
@@ -344,4 +322,29 @@ public class OverviewEventPanel extends AbstractEventPanel {
         
     }
 
+    private ListHandler<EventDAO> getEventSortHandler(List<EventDAO> list, TextColumn<EventDAO> locationColumn,
+            TextColumn<EventDAO> nameColumn, TextColumn<EventDAO> startDateColumn) {
+        //TODO Check if comparators are correct
+        ListHandler<EventDAO> sortHandler = new ListHandler<EventDAO>(list);
+        sortHandler.setComparator(locationColumn, new Comparator<EventDAO>() {
+            @Override
+            public int compare(EventDAO e1, EventDAO e2) {
+                return e1.getLocationAsString().compareTo(e2.getLocationAsString());
+            }
+        });
+        sortHandler.setComparator(nameColumn, new Comparator<EventDAO>() {
+            @Override
+            public int compare(EventDAO e1, EventDAO e2) {
+                return e1.name.compareTo(e2.name);
+            }
+        });
+        sortHandler.setComparator(startDateColumn, new Comparator<EventDAO>() {
+            @Override
+            public int compare(EventDAO e1, EventDAO e2) {
+                return e1.getStartDate().compareTo(e2.getStartDate());
+            }
+        });
+        return sortHandler;
+    }
+
 }