From: Nick Black <nickb@svn.openstreetmap.org>
Date: Wed, 25 Apr 2007 08:12:20 +0000 (+0000)
Subject: Fixed up name search and postcode search
X-Git-Tag: live~8498
X-Git-Url: https://git.openstreetmap.org/rails.git/commitdiff_plain/61ae5e7af0dd759ddb1acfcea10adcbede9da58c

Fixed up name search and postcode search
---

diff --git a/app/controllers/geocoder_controller.rb b/app/controllers/geocoder_controller.rb
index c12c0525b..6955579ed 100644
--- a/app/controllers/geocoder_controller.rb
+++ b/app/controllers/geocoder_controller.rb
@@ -7,7 +7,7 @@ class GeocoderController < ApplicationController
   def search
     @postcode_arr = []
 
-    if params[:query][:postcode] 
+    unless params[:query][:postcode].empty? 
       postcode = params[:query][:postcode].upcase
       escaped_postcode = postcode.sub(/\s/,'%20')
 
@@ -68,7 +68,7 @@ class GeocoderController < ApplicationController
         end
         redirect_to "/index.html?lat=#{lat}&lon=#{lon}&zoom=14"
         #redirect_to "/index.html?error=unknown_postcode_or_zip"
-      else
+      elsif
         # Some other postcode / zip code
         # Throw it at geonames, and see if they have any luck with it
         Net::HTTP.start('ws.geonames.org') do |http|
@@ -85,11 +85,11 @@ class GeocoderController < ApplicationController
           lon = data_lon.split(/[<>]/)[1]
           redirect_to "/index.html?lat=#{lat}&lon=#{lon}&zoom=14"
         end
+      else
+        # Some other postcode / zip file
+        redirect_to "/index.html?error=unknown_postcode_or_zip"
+        return
       end
-    else
-      # Some other postcode / zip file
-      redirect_to "/index.html?error=unknown_postcode_or_zip"
-      return
     end
 
     if params[:query][:place_name]  
diff --git a/app/controllers/user_controller.rb b/app/controllers/user_controller.rb
index 3837d0976..b2ac3072c 100644
--- a/app/controllers/user_controller.rb
+++ b/app/controllers/user_controller.rb
@@ -29,6 +29,18 @@ class UserController < ApplicationController
     end
   end
 
+  def set_home
+    if params[:user][:home_lat] and params[:user][:home_lon]
+    lat = params[:user][:home_lat]
+    lon = params[:user][:home_lon]
+    unless 
+      #check the lat and lon
+    end
+    #make an api request to insert a new node
+    #get the onde id
+    end
+    end
+
   def go_public
     @user.data_public = true
     @user.save
diff --git a/app/views/user/account.rhtml b/app/views/user/account.rhtml
index f8047d3d8..257e8c47a 100644
--- a/app/views/user/account.rhtml
+++ b/app/views/user/account.rhtml
@@ -4,6 +4,7 @@
   <tr><td>email</td><td><%= @user.email %></td></tr>
   <tr><td>display name</td><td><%= @user.display_name %> (<%= link_to 'change...', :controller => 'user', :action => 'edit' %>)</td></tr>
   <tr><td>account created</td><td><%= @user.creation_time %> (<%= time_ago_in_words(@user.creation_time) %> ago)</td></tr>
+  <tr><td>user home</td><td><%= @user.home_node_id %>  (<%= link_to 'change...', :controller => 'user', :action => 'edit' %>) </td></tr>
   <tr><td valign="top">description</td><td><%= simple_format(@user.description) %>  (<%= link_to 'change...', :controller => 'user', :action => 'edit' %>)</td></tr>
 </table>
 
diff --git a/app/views/user/edit.rhtml b/app/views/user/edit.rhtml
index 6c30ef0b3..a78b9e173 100644
--- a/app/views/user/edit.rhtml
+++ b/app/views/user/edit.rhtml
@@ -7,3 +7,8 @@
 </table>
 <%= submit_tag 'Change' %>
 <%= end_form_tag %>
+
+<%= start_form_tag :controller => 'user', :action => 'set_home' %>
+  <tr><td>home location</td><td><%= text_field :user, :home  %></td></tr>
+<%= submit_tag 'Change Home' %>
+<%= end_form_tag %>
diff --git a/db/migrate.sql b/db/migrate.sql
index dc81e5d0c..0f78b9dd3 100644
--- a/db/migrate.sql
+++ b/db/migrate.sql
@@ -39,5 +39,5 @@ create table diary_entries(id bigint not null auto_increment, user_id bigint not
 alter table diary_entries add created_at datetime;
 alter table diary_entries add updated_at datetime;
 
-
+alter table users add column (home_node_id bigint(64) default NULL);