Do not store routes with coordinate members

2 files changed, 16 insertions, 3 deletions

diff --git a/src/sidebar.js b/src/sidebar.js
index 7496b225..79fb75fe 100644
--- a/src/sidebar.js
+++ b/src/sidebar.js
@@ -213,9 +213,10 @@ const Sidebar = new Lang.Class({
                     places: places
                 });
 
-                placeStore.addPlace(storedRoute,
-                                    PlaceStore.PlaceType.RECENT_ROUTE);
-
+                if (!storedRoute.containsNull) {
+                    placeStore.addPlace(storedRoute,
+                                        PlaceStore.PlaceType.RECENT_ROUTE);
+                }
                 this._storeRouteTimeoutId = 0;
             }).bind(this));
 
diff --git a/src/storedRoute.js b/src/storedRoute.js
index 16f761d3..05e56de5 100644
--- a/src/storedRoute.js
+++ b/src/storedRoute.js
@@ -102,6 +102,18 @@ const StoredRoute = new Lang.Class({
         return this._containsCurrentLocation;
     },
 
+    get containsNull() {
+        let hasNull = false;
+
+        for (let p = 0; p < this.places.length; p++) {
+            if (!this.places[p].name) {
+                hasNull = true;
+                break;
+            }
+        }
+        return hasNull;
+    },
+
     toJSON: function() {
         let turnPoints = this.route.turnPoints.map(function(turnPoint) {
             let coordinate = { latitude: turnPoint.coordinate.latitude,