Fix removing inactive servers and avoid doing unneeded work.

src/db.go

@@ -42,7 +42,7 @@ func GetOldServers(db *gorm.DB, ts time.Time) []*Server {
 }
 
 func RemoveOldServers(db *gorm.DB, ts time.Time) {
-	db.Where("last_updated < datetime(?, '-7 days')").Delete(Server{})
+	db.Where("last_updated < datetime(?, '-7 days')", ts).Delete(Server{})
 }
 
 func GetServerPopulation(db *gorm.DB, id int, d time.Duration) []*ServerPopulation {

src/updater.go

@@ -39,17 +39,17 @@ func (i *Instance) UpdateServers() {
 		i.update_server(s)
 	}
 
+	if i.Debug {
+		fmt.Println("\nRemoving old servers...")
+	}
+	RemoveOldServers(i.DB, Now())
+
 	if i.Debug {
 		fmt.Println("\nUpdating inactive servers...")
 	}
 	for _, s := range i.get_old_servers() {
 		i.update_server(s)
 	}
-
-	if i.Debug {
-		fmt.Println("\nRemoving old servers...")
-	}
-	RemoveOldServers(i.DB, Now())
 }
 
 func reset() {