]> git.street.me.uk Git - andy/viking.git/blobdiff - src/geonamessearch.c
Add support for acquiring OpenStreetMap Notes as GPX files.
[andy/viking.git] / src / geonamessearch.c
index b146cf29f341fc4d21cdbf7308643c621b0ad3b5..1f72cd3af3b1acb7f825cbd214586b3439d74080 100644 (file)
@@ -1,7 +1,8 @@
 /*
  * viking -- GPS Data and Topo Analyzer, Explorer, and Manager
  *
- * Copyright (C) 2003-2005, Evan Battaglia <gtoevan@gmx.net>
+ * Copyright (C) 2009, Hein Ragas
+ * Copyright (C) 2013, Rob Norris <rw_norris@hotmail.com>
  *
  * This program is free software; you can redistribute it and/or modify
  * it under the terms of the GNU General Public License as published by
@@ -17,7 +18,6 @@
  * along with this program; if not, write to the Free Software
  * Foundation, Inc., 59 Temple Place, Suite 330, Boston, MA  02111-1307  USA
  *
- * Created by Quy Tonthat <qtonthat@gmail.com>
  */
 #ifdef HAVE_CONFIG_H
 #include "config.h"
 #include <glib/gi18n.h>
 
 #include "viking.h"
-#include "curl_download.h"
+#include "util.h"
+#include "geonamessearch.h"
 
-#define GEONAMES_SEARCH_URL_FMT "http://ws.geonames.org/searchJSON?formatted=true&style=medium&maxRows=10&lang=en&q=%s"
-#define GEONAMES_SEARCH_PATTERN_2 "\"lat\": "
-#define GEONAMES_SEARCH_PATTERN_1 "\"lng\": "
-#define GEONAMES_COUNTRY_PATTERN "\"countryName\": \""
+/* Compatibility */
+#if ! GLIB_CHECK_VERSION(2,22,0)
+#define g_mapped_file_unref g_mapped_file_free
+#endif
+
+/**
+ * See http://www.geonames.org/export/wikipedia-webservice.html#wikipediaBoundingBox
+ */
+// Translators may wish to change this setting as appropriate to get Wikipedia articles in that language
+#define GEONAMES_LANG N_("en")
+// TODO - offer configuration of this value somewhere
+//  ATM decided it's not essential enough to warrant putting in the preferences
+#define GEONAMES_MAX_ENTRIES 20
+
+#define GEONAMES_WIKIPEDIA_URL_FMT "http://ws.geonames.org/wikipediaBoundingBoxJSON?formatted=true&north=%s&south=%s&east=%s&west=%s&lang=%s&maxRows=%d"
+
+#define GEONAMES_FEATURE_PATTERN "\"feature\": \""
 #define GEONAMES_LONGITUDE_PATTERN "\"lng\": "
 #define GEONAMES_NAME_PATTERN "\"name\": \""
 #define GEONAMES_LATITUDE_PATTERN "\"lat\": "
+#define GEONAMES_ELEVATION_PATTERN "\"elevation\": "
+#define GEONAMES_TITLE_PATTERN "\"title\": \""
+#define GEONAMES_WIKIPEDIAURL_PATTERN "\"wikipediaUrl\": \""
+#define GEONAMES_THUMBNAILIMG_PATTERN "\"thumbnailImg\": \""
 #define GEONAMES_SEARCH_NOT_FOUND "not understand the location"
 
-static gchar *last_search_str = NULL;
-static VikCoord *last_coord = NULL;
-static gchar *last_successful_search_str = NULL;
+/* found_geoname: Type to contain data returned from GeoNames.org */
 
 typedef struct {
   gchar *name;
-  gchar *country;
+  gchar *feature;
   struct LatLon ll;
+  gdouble elevation;
+  gchar *cmt;
+  gchar *desc;
 } found_geoname;
 
-found_geoname *copy_found_geoname(found_geoname *src)
+static found_geoname *new_found_geoname()
+{
+  found_geoname *ret;
+
+  ret = (found_geoname *)g_malloc(sizeof(found_geoname));
+  ret->name = NULL;
+  ret->feature = NULL;
+  ret->cmt = NULL;
+  ret->desc = NULL;
+  ret->ll.lat = 0.0;
+  ret->ll.lon = 0.0;
+  ret->elevation = VIK_DEFAULT_ALTITUDE;
+  return ret;
+}
+
+static found_geoname *copy_found_geoname(found_geoname *src)
 {
-  found_geoname *dest = (found_geoname *)g_malloc(sizeof(found_geoname));
+  found_geoname *dest = new_found_geoname();
   dest->name = g_strdup(src->name);
-  dest->country = g_strdup(src->country);
+  dest->feature = g_strdup(src->feature);
   dest->ll.lat = src->ll.lat;
   dest->ll.lon = src->ll.lon;
+  dest->elevation = src->elevation;
+  dest->cmt = g_strdup(src->cmt);
+  dest->desc = g_strdup(src->desc);
   return(dest);
 }
 
-gchar * a_geonamessearch_get_search_string_for_this_place(VikWindow *vw)
+static void free_list_geonames(found_geoname *geoname, gpointer userdata)
 {
-  if (!last_coord)
-    return NULL;
-
-  VikViewport *vvp = vik_window_viewport(vw);
-  const VikCoord *cur_center = vik_viewport_get_center(vvp);
-  if (vik_coord_equals(cur_center, last_coord)) {
-    return(last_successful_search_str);
-  }
-  else
-    return NULL;
+  g_free(geoname->name);
+  g_free(geoname->feature);
+  g_free(geoname->cmt);
+  g_free(geoname->desc);
 }
 
-static gboolean prompt_try_again(VikWindow *vw)
+static void free_geoname_list(GList *found_places)
 {
-  GtkWidget *dialog = NULL;
-  gboolean ret = TRUE;
-
-  dialog = gtk_dialog_new_with_buttons ( "", GTK_WINDOW(vw), 0, GTK_STOCK_OK, GTK_RESPONSE_ACCEPT, GTK_STOCK_CANCEL, GTK_RESPONSE_REJECT, NULL );
-  gtk_window_set_title(GTK_WINDOW(dialog), _("Search"));
-
-  GtkWidget *search_label = gtk_label_new(_("I don't know that place. Do you want another search?"));
-  gtk_box_pack_start ( GTK_BOX(GTK_DIALOG(dialog)->vbox), search_label, FALSE, FALSE, 5 );
-  gtk_widget_show_all(dialog);
-
-  if ( gtk_dialog_run ( GTK_DIALOG(dialog) ) != GTK_RESPONSE_ACCEPT )
-    ret = FALSE;
-
-  gtk_widget_destroy(dialog);
-  return ret;
+  g_list_foreach(found_places, (GFunc)free_list_geonames, NULL);
+  g_list_free(found_places);
 }
 
-static gchar *  a_prompt_for_search_string(VikWindow *vw)
+static void none_found(VikWindow *vw)
 {
   GtkWidget *dialog = NULL;
 
-  dialog = gtk_dialog_new_with_buttons ( "", GTK_WINDOW(vw), 0, GTK_STOCK_OK, GTK_RESPONSE_ACCEPT, GTK_STOCK_CANCEL, GTK_RESPONSE_REJECT, NULL );
+  dialog = gtk_dialog_new_with_buttons ( "", GTK_WINDOW(vw), 0, GTK_STOCK_OK, GTK_RESPONSE_ACCEPT, NULL );
   gtk_window_set_title(GTK_WINDOW(dialog), _("Search"));
 
-  GtkWidget *search_label = gtk_label_new(_("Enter address or place name:"));
-  GtkWidget *search_entry = gtk_entry_new();
-  if (last_search_str)
-    gtk_entry_set_text(GTK_ENTRY(search_entry), last_search_str);
-
-  gtk_box_pack_start ( GTK_BOX(GTK_DIALOG(dialog)->vbox), search_label, FALSE, FALSE, 5 );
-  gtk_box_pack_start ( GTK_BOX(GTK_DIALOG(dialog)->vbox), search_entry, FALSE, FALSE, 5 );
+  GtkWidget *search_label = gtk_label_new(_("No entries found!"));
+  gtk_box_pack_start ( GTK_BOX(gtk_dialog_get_content_area(GTK_DIALOG(dialog))), search_label, FALSE, FALSE, 5 );
+  gtk_dialog_set_default_response ( GTK_DIALOG(dialog), GTK_RESPONSE_ACCEPT );
   gtk_widget_show_all(dialog);
 
-  if ( gtk_dialog_run ( GTK_DIALOG(dialog) ) != GTK_RESPONSE_ACCEPT ) {
-    gtk_widget_destroy(dialog);
-    return NULL;
-  }
-
-  gchar *search_str = g_strdup ( gtk_entry_get_text ( GTK_ENTRY(search_entry) ) );
-
+  gtk_dialog_run ( GTK_DIALOG(dialog) );
   gtk_widget_destroy(dialog);
-
-  if (search_str[0] != '\0') {
-    if (last_search_str)
-      g_free(last_search_str);
-    last_search_str = g_strdup(search_str);
-  }
-
-  return(search_str);   /* search_str needs to be freed by caller */
-}
-
-static void free_list_geonames(found_geoname *geoname, gpointer userdata)
-{
-  g_free(geoname->name);
-  g_free(geoname->country);
 }
 
-void buttonToggled(GtkCellRendererToggle* renderer, gchar* pathStr, gpointer data)
-{
-   GtkTreeIter iter;
-   gboolean enabled;
-   GtkTreePath* path = gtk_tree_path_new_from_string(pathStr);
-   gtk_tree_model_get_iter(GTK_TREE_MODEL (data), &iter, path);
-   gtk_tree_model_get(GTK_TREE_MODEL (data), &iter, 0, &enabled, -1);
-   enabled = !enabled;
-   gtk_tree_store_set(GTK_TREE_STORE (data), &iter, 0, enabled, -1);
-}
-
-GList *a_select_geoname_from_list(GtkWindow *parent, GList *geonames, gboolean multiple_selection_allowed, const gchar *title, const gchar *msg)
+static GList *a_select_geoname_from_list(GtkWindow *parent, GList *geonames, gboolean multiple_selection_allowed, const gchar *title, const gchar *msg)
 {
   GtkTreeIter iter;
   GtkCellRenderer *renderer;
-  GtkCellRenderer *toggle_render;
   GtkWidget *view;
   found_geoname *geoname;
   gchar *latlon_string;
@@ -164,87 +146,99 @@ GList *a_select_geoname_from_list(GtkWindow *parent, GList *geonames, gboolean m
                                                   GTK_STOCK_OK,
                                                   GTK_RESPONSE_ACCEPT,
                                                   NULL);
+  /* When something is selected then OK */
+  gtk_dialog_set_default_response ( GTK_DIALOG(dialog), GTK_RESPONSE_ACCEPT );
+  GtkWidget *response_w = NULL;
+#if GTK_CHECK_VERSION (2, 20, 0)
+  /* Default to not apply - as initially nothing is selected! */
+  response_w = gtk_dialog_get_widget_for_response ( GTK_DIALOG(dialog), GTK_RESPONSE_REJECT );
+#endif
   GtkWidget *label = gtk_label_new ( msg );
-  GtkTreeStore *store;
-  if (multiple_selection_allowed)
-  {
-    store = gtk_tree_store_new(4, G_TYPE_BOOLEAN, G_TYPE_STRING, G_TYPE_STRING, G_TYPE_STRING);
-  }
-  else
-  {
-    store = gtk_tree_store_new(3, G_TYPE_STRING, G_TYPE_STRING, G_TYPE_STRING);
-  }
+  GtkTreeStore *store = gtk_tree_store_new(3, G_TYPE_STRING, G_TYPE_STRING, G_TYPE_STRING);
+
   GList *geoname_runner = geonames;
   while (geoname_runner)
   { 
     geoname = (found_geoname *)geoname_runner->data;
     latlon_string = g_strdup_printf("(%f,%f)", geoname->ll.lat, geoname->ll.lon);
     gtk_tree_store_append(store, &iter, NULL);
-    if (multiple_selection_allowed)
-    {
-      gtk_tree_store_set(store, &iter, 0, FALSE, 1, geoname->name, 2, geoname->country, 3, latlon_string, -1);
-    }
-    else
-    {
-      gtk_tree_store_set(store, &iter, 0, geoname->name, 1, geoname->country, 2, latlon_string, -1);
-    }
+    gtk_tree_store_set(store, &iter, 0, geoname->name, 1, geoname->feature, 2, latlon_string, -1);
     geoname_runner = g_list_next(geoname_runner);
     g_free(latlon_string);
   }
+
   view = gtk_tree_view_new();
   renderer = gtk_cell_renderer_text_new();
   column_runner = 0;
-  if (multiple_selection_allowed)
-  {
-    toggle_render = gtk_cell_renderer_toggle_new();
-    g_object_set(toggle_render, "activatable", TRUE, NULL);
-    g_signal_connect(toggle_render, "toggled", (GCallback) buttonToggled, GTK_TREE_MODEL(store));
-    gtk_tree_view_insert_column_with_attributes( GTK_TREE_VIEW(view), -1, "Select", toggle_render, "active", column_runner, NULL);
-    column_runner++;
-  }
-  gtk_tree_view_insert_column_with_attributes( GTK_TREE_VIEW(view), -1, "Name", renderer, "text", column_runner, NULL);
+  GtkTreeViewColumn *column;
+  // NB could allow columns to be shifted around by doing this after each new
+  // gtk_tree_view_column_set_reorderable ( column, TRUE );
+  // However I don't think is that useful, so I haven't put it in
+  column = gtk_tree_view_column_new_with_attributes( _("Name"), renderer, "text", column_runner, NULL);
+  gtk_tree_view_column_set_sort_column_id (column, column_runner);
+  gtk_tree_view_append_column (GTK_TREE_VIEW (view), column);
+
   column_runner++;
-  gtk_tree_view_insert_column_with_attributes( GTK_TREE_VIEW(view), -1, "Country", renderer, "text", column_runner, NULL);
+  column = gtk_tree_view_column_new_with_attributes( _("Feature"), renderer, "text", column_runner, NULL);
+  gtk_tree_view_column_set_sort_column_id (column, column_runner);
+  gtk_tree_view_append_column (GTK_TREE_VIEW (view), column);
+
   column_runner++;
-  gtk_tree_view_insert_column_with_attributes( GTK_TREE_VIEW(view), -1, "Lat/Lon", renderer, "text", column_runner, NULL);
-  gtk_tree_view_set_headers_visible( GTK_TREE_VIEW(view), TRUE);
+  column = gtk_tree_view_column_new_with_attributes( _("Lat/Lon"), renderer, "text", column_runner, NULL);
+  gtk_tree_view_column_set_sort_column_id (column, column_runner);
+  gtk_tree_view_append_column (GTK_TREE_VIEW (view), column);
+
   gtk_tree_view_set_model(GTK_TREE_VIEW(view), GTK_TREE_MODEL(store));
   gtk_tree_selection_set_mode( gtk_tree_view_get_selection(GTK_TREE_VIEW(view)),
       multiple_selection_allowed ? GTK_SELECTION_MULTIPLE : GTK_SELECTION_BROWSE );
   g_object_unref(store);
 
-  gtk_box_pack_start (GTK_BOX(GTK_DIALOG(dialog)->vbox), label, FALSE, FALSE, 0);
-  gtk_widget_show ( label );
-  gtk_box_pack_start (GTK_BOX(GTK_DIALOG(dialog)->vbox), view, FALSE, FALSE, 0);
-  gtk_widget_show ( view );
+  GtkWidget *scrolledwindow = gtk_scrolled_window_new ( NULL, NULL );
+  gtk_scrolled_window_set_policy ( GTK_SCROLLED_WINDOW(scrolledwindow), GTK_POLICY_NEVER, GTK_POLICY_AUTOMATIC );
+  gtk_container_add ( GTK_CONTAINER(scrolledwindow), view );
+
+  gtk_box_pack_start (GTK_BOX(gtk_dialog_get_content_area(GTK_DIALOG(dialog))), label, FALSE, FALSE, 0);
+  gtk_box_pack_start (GTK_BOX(gtk_dialog_get_content_area(GTK_DIALOG(dialog))), scrolledwindow, TRUE, TRUE, 0);
+
+  // Ensure a reasonable number of items are shown, but let the width be automatically sized
+  gtk_widget_set_size_request ( dialog, -1, 400) ;
+  gtk_widget_show_all ( dialog );
+
+  if ( response_w )
+    gtk_widget_grab_focus ( response_w );
+
   while ( gtk_dialog_run (GTK_DIALOG (dialog)) == GTK_RESPONSE_ACCEPT )
   {
     GtkTreeSelection *selection = gtk_tree_view_get_selection(GTK_TREE_VIEW(view));
     GList *selected_geonames = NULL;
 
-    gtk_tree_model_get_iter_first( GTK_TREE_MODEL(store), &iter);
-    geoname_runner = geonames;
-    while (geoname_runner)
-    {
-      if (multiple_selection_allowed)
-      {
-        // nop;
-      }
-      else
-      {
-        if (gtk_tree_selection_iter_is_selected(selection, &iter))
-        {
-          found_geoname *copied = copy_found_geoname(geoname_runner->data);
-          selected_geonames = g_list_prepend(selected_geonames, copied);
+    // Possibily not the fastest method but we don't have thousands of entries to process...
+    if ( gtk_tree_model_get_iter_first( GTK_TREE_MODEL(store), &iter) ) {
+      do {
+        if ( gtk_tree_selection_iter_is_selected ( selection, &iter ) ) {
+          // For every selected item,
+          // compare the name from the displayed view to every geoname entry to find the geoname this selection represents
+          gchar* name;
+          gtk_tree_model_get (GTK_TREE_MODEL(store), &iter, 0, &name, -1 );
+         // I believe the name of these items to be always unique
+          geoname_runner = geonames;
+          while ( geoname_runner ) {
+            if ( !strcmp ( ((found_geoname*)geoname_runner->data)->name, name ) ) {
+              found_geoname *copied = copy_found_geoname(geoname_runner->data);
+              selected_geonames = g_list_prepend(selected_geonames, copied);
+              break;
+            }
+            geoname_runner = g_list_next(geoname_runner);
+          }
         }
       }
-      geoname_runner = g_list_next(geoname_runner);
-      gtk_tree_model_iter_next(GTK_TREE_MODEL(store), &iter);
+      while ( gtk_tree_model_iter_next ( GTK_TREE_MODEL(store), &iter ) );
     }
+
     if (selected_geonames)
     { 
       gtk_widget_destroy ( dialog );
-      return (selected_geonames);
+      return selected_geonames;
     }
     a_dialog_error_msg(parent, _("Nothing was selected"));
   }
@@ -252,14 +246,13 @@ GList *a_select_geoname_from_list(GtkWindow *parent, GList *geonames, gboolean m
   return NULL;
 }
 
-static gboolean parse_file_for_latlon(VikWindow *vw, gchar *file_name, struct LatLon *ll)
+static GList *get_entries_from_file(gchar *file_name)
 {
   gchar *text, *pat;
   GMappedFile *mf;
   gsize len;
   gboolean more = TRUE;
-  gboolean found = TRUE;
-  gchar lat_buf[32], lon_buf[32];
+  gchar lat_buf[32], lon_buf[32], elev_buf[32];
   gchar *s;
   gint fragment_len;
   GList *found_places = NULL;
@@ -267,12 +260,14 @@ static gboolean parse_file_for_latlon(VikWindow *vw, gchar *file_name, struct La
   gchar **found_entries;
   gchar *entry;
   int entry_runner;
+  gchar *wikipedia_url = NULL;
+  gchar *thumbnail_url = NULL;
 
-  lat_buf[0] = lon_buf[0] = '\0';
+  lat_buf[0] = lon_buf[0] = elev_buf[0] = '\0';
 
   if ((mf = g_mapped_file_new(file_name, FALSE, NULL)) == NULL) {
     g_critical(_("couldn't map temp file"));
-    exit(1);
+    return NULL;
   }
   len = g_mapped_file_get_length(mf);
   text = g_mapped_file_get_contents(mf);
@@ -286,19 +281,16 @@ static gboolean parse_file_for_latlon(VikWindow *vw, gchar *file_name, struct La
   while (entry)
   {
     more = TRUE;
-    geoname = (found_geoname *)g_malloc(sizeof(found_geoname));
-    if ((pat = g_strstr_len(entry, strlen(entry), GEONAMES_COUNTRY_PATTERN)) == NULL) {
-      more = FALSE;
-    }
-    else {
-      pat += strlen(GEONAMES_COUNTRY_PATTERN);
+    geoname = new_found_geoname();
+    if ((pat = g_strstr_len(entry, strlen(entry), GEONAMES_FEATURE_PATTERN))) {
+      pat += strlen(GEONAMES_FEATURE_PATTERN);
       fragment_len = 0;
       s = pat;
       while (*pat != '"') {
         fragment_len++;
         pat++;
       }
-      geoname -> country = g_strndup(s, fragment_len);
+      geoname->feature = g_strndup(s, fragment_len);
     }
     if ((pat = g_strstr_len(entry, strlen(entry), GEONAMES_LONGITUDE_PATTERN)) == NULL) {
       more = FALSE;
@@ -317,10 +309,18 @@ static gboolean parse_file_for_latlon(VikWindow *vw, gchar *file_name, struct La
       }
       geoname->ll.lon = g_ascii_strtod(lon_buf, NULL);
     }
-    if ((pat = g_strstr_len(entry, strlen(entry), GEONAMES_NAME_PATTERN)) == NULL) {
-      more = FALSE;
+    if ((pat = g_strstr_len(entry, strlen(entry), GEONAMES_ELEVATION_PATTERN))) {
+      pat += strlen(GEONAMES_ELEVATION_PATTERN);
+      s = elev_buf;
+      if (*pat == '-')
+        *s++ = *pat++;
+      while ((s < (elev_buf + sizeof(elev_buf))) && (pat < (text + len)) &&
+              (g_ascii_isdigit(*pat) || (*pat == '.')))
+        *s++ = *pat++;
+      *s = '\0';
+      geoname->elevation = g_ascii_strtod(elev_buf, NULL);
     }
-    else {
+    if ((pat = g_strstr_len(entry, strlen(entry), GEONAMES_NAME_PATTERN))) {
       pat += strlen(GEONAMES_NAME_PATTERN);
       fragment_len = 0;
       s = pat;
@@ -330,6 +330,36 @@ static gboolean parse_file_for_latlon(VikWindow *vw, gchar *file_name, struct La
       }
       geoname -> name = g_strndup(s, fragment_len);
     }
+    if ((pat = g_strstr_len(entry, strlen(entry), GEONAMES_TITLE_PATTERN))) {
+      pat += strlen(GEONAMES_TITLE_PATTERN);
+      fragment_len = 0;
+      s = pat;
+      while (*pat != '"') {
+        fragment_len++;
+        pat++;
+      }
+      geoname -> name = g_strndup(s, fragment_len);
+    }
+    if ((pat = g_strstr_len(entry, strlen(entry), GEONAMES_WIKIPEDIAURL_PATTERN))) {
+      pat += strlen(GEONAMES_WIKIPEDIAURL_PATTERN);
+      fragment_len = 0;
+      s = pat;
+      while (*pat != '"') {
+        fragment_len++;
+        pat++;
+      }
+      wikipedia_url = g_strndup(s, fragment_len);
+    }
+    if ((pat = g_strstr_len(entry, strlen(entry), GEONAMES_THUMBNAILIMG_PATTERN))) {
+      pat += strlen(GEONAMES_THUMBNAILIMG_PATTERN);
+      fragment_len = 0;
+      s = pat;
+      while (*pat != '"') {
+        fragment_len++;
+        pat++;
+      }
+      thumbnail_url = g_strndup(s, fragment_len);
+    }
     if ((pat = g_strstr_len(entry, strlen(entry), GEONAMES_LATITUDE_PATTERN)) == NULL) {
       more = FALSE;
     }
@@ -353,141 +383,99 @@ static gboolean parse_file_for_latlon(VikWindow *vw, gchar *file_name, struct La
       }
     }
     else {
+      if (wikipedia_url) {
+        // Really we should support the GPX URL tag and then put that in there...
+        geoname->cmt = g_strdup_printf("http://%s", wikipedia_url);
+        if (thumbnail_url) {
+          geoname -> desc = g_strdup_printf("<a href=\"http://%s\" target=\"_blank\"><img src=\"%s\" border=\"0\"/></a>", wikipedia_url, thumbnail_url);
+        }
+        else {
+          geoname -> desc = g_strdup_printf("<a href=\"http://%s\" target=\"_blank\">%s</a>", wikipedia_url, geoname->name);
+        }
+      }
+      if (wikipedia_url) {
+        g_free(wikipedia_url);
+        wikipedia_url = NULL;
+      }
+      if (thumbnail_url) {
+        g_free(thumbnail_url);
+        thumbnail_url = NULL;
+      }
       found_places = g_list_prepend(found_places, geoname);
     }
     entry_runner++;
     entry = found_entries[entry_runner];
   }
   g_strfreev(found_entries);
-  if (g_list_length(found_places) == 1)
-  {
-    geoname = (found_geoname *)found_places->data;
-    ll->lat = geoname->ll.lat;
-    ll->lon = geoname->ll.lon;
-  }
-  else
-  {
-    found_places = g_list_reverse(found_places);
-    GList *selected = a_select_geoname_from_list(VIK_GTK_WINDOW_FROM_WIDGET(vw), found_places, FALSE, "Select place", "Select the place to go to");
-    if (selected)
-    {
-      geoname = (found_geoname *)selected->data;
-      ll->lat = geoname->ll.lat;
-      ll->lon = geoname->ll.lon;
-      g_list_foreach(selected, (GFunc)free_list_geonames, NULL);
-    }
-    else
-    {
-      found = FALSE;
-    }
-  }
-  g_list_foreach(found_places, (GFunc)free_list_geonames, NULL);
-  g_list_free(found_places);
-  g_mapped_file_free(mf);
-  return (found);
+  found_places = g_list_reverse(found_places);
+  g_mapped_file_unref(mf);
+  return(found_places);
 }
 
-gchar *uri_escape(gchar *str)
-{
-  gchar *esc_str = g_malloc(3*strlen(str));
-  gchar *dst = esc_str;
-  gchar *src;
-
-  for (src = str; *src; src++) {
-    if (*src == ' ')
-     *dst++ = '+';
-    else if (g_ascii_isalnum(*src))
-     *dst++ = *src;
-    else {
-      g_sprintf(dst, "%%%02X", *src);
-      dst += 3;
-    }
-  }
-  *dst = '\0';
 
-  return(esc_str);
-}
-
-static int geonames_search_get_coord(VikWindow *vw, VikViewport *vvp, gchar *srch_str, VikCoord *coord)
+void a_geonames_wikipedia_box ( VikWindow *vw, VikTrwLayer *vtl, struct LatLon maxmin[2] )
 {
-  FILE *tmp_file;
-  int tmp_fd;
-  gchar *tmpname;
   gchar *uri;
-  gchar *escaped_srch_str;
-  int ret = 0;  /* OK */
-  struct LatLon ll;
-
-  g_debug("%s: raw search: %s", __FUNCTION__, srch_str);
-
-  escaped_srch_str = uri_escape(srch_str);
-
-  g_debug("%s: escaped search: %s", __FUNCTION__, escaped_srch_str);
-
-  if ((tmp_fd = g_file_open_tmp ("vikgsearch.XXXXXX", &tmpname, NULL)) == -1) {
-    g_critical(_("couldn't open temp file"));
-    exit(1);
+  gchar *tmpname;
+  GList *wiki_places;
+  GList *selected;
+  GList *wp_runner;
+  VikWaypoint *wiki_wp;
+  found_geoname *wiki_geoname;
+
+  /* encode doubles in a C locale */
+  gchar *north = a_coords_dtostr(maxmin[0].lat);
+  gchar *south = a_coords_dtostr(maxmin[1].lat);
+  gchar *east = a_coords_dtostr(maxmin[0].lon);
+  gchar *west = a_coords_dtostr(maxmin[1].lon);
+  uri = g_strdup_printf ( GEONAMES_WIKIPEDIA_URL_FMT, north, south, east, west, GEONAMES_LANG, GEONAMES_MAX_ENTRIES );
+  g_free(north); north = NULL;
+  g_free(south); south = NULL;
+  g_free(east);  east = NULL;
+  g_free(west);  west = NULL;
+  tmpname = a_download_uri_to_tmp_file ( uri, NULL );
+  if (!tmpname) {
+    none_found(vw);
+    return;
   }
-
-  tmp_file = fdopen(tmp_fd, "r+");
-  //uri = g_strdup_printf(GEONAMES_SEARCH_URL_FMT, srch_str);
-  uri = g_strdup_printf(GEONAMES_SEARCH_URL_FMT, escaped_srch_str);
-
-  // TODO: curl may not be available
-  if (curl_download_uri(uri, tmp_file, NULL)) {  // error
-    fclose(tmp_file);
-    tmp_file = NULL;
-    ret = -1;
-    goto done;
+  wiki_places = get_entries_from_file(tmpname);
+  if (g_list_length(wiki_places) == 0) {
+    none_found(vw);
+    return;
   }
-
-  fclose(tmp_file);
-
-  tmp_file = NULL;
-  if (!parse_file_for_latlon(vw, tmpname, &ll)) {
-    ret = -1;
-    goto done;
+  selected = a_select_geoname_from_list(VIK_GTK_WINDOW_FROM_WIDGET(vw), wiki_places, TRUE, "Select articles", "Select the articles you want to add.");
+  wp_runner = selected;
+  while (wp_runner) {
+    wiki_geoname = (found_geoname *)wp_runner->data;
+    wiki_wp = vik_waypoint_new();
+    wiki_wp->visible = TRUE;
+    vik_coord_load_from_latlon(&(wiki_wp->coord), vik_trw_layer_get_coord_mode ( vtl ), &(wiki_geoname->ll));
+    wiki_wp->altitude = wiki_geoname->elevation;
+    vik_waypoint_set_comment(wiki_wp, wiki_geoname->cmt);
+    vik_waypoint_set_description(wiki_wp, wiki_geoname->desc);
+    // Use the featue type to generate a suitable waypoint icon
+    //  http://www.geonames.org/wikipedia/wikipedia_features.html
+    // Only a few values supported as only a few symbols make sense
+    if ( wiki_geoname->feature ) {
+      if ( !strcmp (wiki_geoname->feature, "city") )
+        vik_waypoint_set_symbol(wiki_wp, "city (medium)");
+      if ( !strcmp (wiki_geoname->feature, "edu") )
+        vik_waypoint_set_symbol(wiki_wp, "school");
+      if ( !strcmp (wiki_geoname->feature, "airport") )
+        vik_waypoint_set_symbol(wiki_wp, "airport");
+      if ( !strcmp (wiki_geoname->feature, "mountain") )
+        vik_waypoint_set_symbol(wiki_wp, "summit");
+      if ( !strcmp (wiki_geoname->feature, "forest") )
+        vik_waypoint_set_symbol(wiki_wp, "forest");
+    }
+    vik_trw_layer_filein_add_waypoint ( vtl, wiki_geoname->name, wiki_wp );
+    wp_runner = g_list_next(wp_runner);
   }
-
-  vik_coord_load_from_latlon ( coord, vik_viewport_get_coord_mode(vvp), &ll );
-
-  if (last_coord)
-    g_free(last_coord);
-  last_coord = g_malloc(sizeof(VikCoord));
-  *last_coord = *coord;
-  if (last_successful_search_str)
-    g_free(last_successful_search_str);
-  last_successful_search_str = g_strdup(last_search_str);
-
-done:
-  g_free(escaped_srch_str);
+  free_geoname_list(wiki_places);
+  free_geoname_list(selected);
   g_free(uri);
-  g_remove(tmpname);
-  g_free(tmpname);
-  return ret;
-}
-
-void a_geonames_search(VikWindow *vw, VikLayersPanel *vlp, VikViewport *vvp)
-{
-  VikCoord new_center;
-  gchar *s_str;
-  gboolean more = TRUE;
-
-  do {
-    s_str = a_prompt_for_search_string(vw);
-    if ((!s_str) || (s_str[0] == 0)) {
-      more = FALSE;
-    }
-    else if (!geonames_search_get_coord(vw, vvp, s_str, &new_center)) {
-      vik_viewport_set_center_coord(vvp, &new_center);
-      vik_layers_panel_emit_update(vlp);
-      more = FALSE;
-    }
-/*
-    else if (!prompt_try_again(vw))
-        more = FALSE;
-*/
-    g_free(s_str);
-  } while (more);
+  if (tmpname) {
+    g_remove(tmpname);
+    g_free(tmpname);
+  }
 }
-