From 985047f18db933ecbbe8f6d9b10fbfdcac287b57 Mon Sep 17 00:00:00 2001
From: Tom Hughes <tom@compton.nu>
Date: Fri, 27 Jul 2007 17:46:01 +0000
Subject: [PATCH] User null to indicate the the user's home location is not
 set.

---
 app/views/site/index.rhtml   |  2 +-
 app/views/user/account.rhtml | 14 +++++++-------
 app/views/user/view.rhtml    | 13 ++++++-------
 3 files changed, 14 insertions(+), 15 deletions(-)

diff --git a/app/views/site/index.rhtml b/app/views/site/index.rhtml
index 80746ba07..5b9fd84f6 100644
--- a/app/views/site/index.rhtml
+++ b/app/views/site/index.rhtml
@@ -14,7 +14,7 @@
 <% lon = params['mlon'] %> 
 <% lat = params['mlat'] %>
 <% zoom =  params['zoom'] || '12' %>
-<% elsif @user %> 
+<% elsif @user and !@user.home_lon.nil? and !@user.home_lat.nil? %> 
 <% lon =  @user.home_lon %>
 <% lat =  @user.home_lat %>
 <% zoom = '10' %>
diff --git a/app/views/user/account.rhtml b/app/views/user/account.rhtml
index f17a792fb..4933dc085 100644
--- a/app/views/user/account.rhtml
+++ b/app/views/user/account.rhtml
@@ -24,7 +24,7 @@
 <% end %>
 
 <% nearest_str = "" %>
-<% if @user.home_lat and @user.home_lon %>
+<% if !@user.home_lat.nil? and !@user.home_lon.nil? %>
   <% if !@user.nearby.empty? %>
     <% @user.nearby.each do |nearby| %>
     <% nearest_str += "nearest.push( { 'display_name' : '#{nearby.display_name}', 'home_lat' : #{nearby.home_lat}, 'home_lon' : #{nearby.home_lon} } );\n" %>
@@ -36,17 +36,17 @@
     <%= nearest_str %>
 </script>
 
-<% if @user.home_lat and @user.home_lon %>
+<% if @user.home_lat.nil? or @user.home_lon.nil? %>
+  <% lon =  params['lon'] || '-0.1' %>
+  <% lat =  params['lat'] || '51.5' %>
+  <% zoom =  params['zoom'] || '4' %> 
+<% else %>
   <% marker = true %>
   <% mlon = @user.home_lon %> 
   <% mlat = @user.home_lat %>
   <% lon =  @user.home_lon %>
-  <% lat =  @user.home_lat  %>
+  <% lat =  @user.home_lat %>
   <% zoom =  '12' %>
-<% else %>
-  <% lon =  params['lon'] || '-0.1' %>
-  <% lat =  params['lat'] || '51.5' %>
-  <% zoom =  params['zoom'] || '4' %> 
 <% end %>
 
 <script type="text/javascript" src="/openlayers/OpenLayers.js"></script>
diff --git a/app/views/user/view.rhtml b/app/views/user/view.rhtml
index 15176f375..391523b58 100644
--- a/app/views/user/view.rhtml
+++ b/app/views/user/view.rhtml
@@ -22,7 +22,12 @@
 <% end %>
 
 <h3>Nearby users</h3>
-<% if @this_user.home_lat and @this_user.home_lon %>
+<% if @this_user.home_lat.nil? or @this_user.home_lon.nil? %>
+  No home location has been set.
+  <% if @user and @this_user.id == @user.id %>
+  You can set your home location on your <%= link_to 'settings', :controller => 'user', :action => 'account', :display_name => @user.display_name %> page.
+  <% end %>
+<% else %>
   <% if @this_user.nearby.empty? %>
     There are no users who admit to mapping nearby.
   <% else %>
@@ -35,11 +40,5 @@
     </tr>
     <%end%>
     </table>
-
   <%end%>
-<% else %>
-  No home location has been set.
-  <% if @user and @this_user.id == @user.id %>
-  You can set your home location on your <%= link_to 'settings', :controller => 'user', :action => 'account', :display_name => @user.display_name %> page.
-  <% end %>
 <% end %>
-- 
2.43.2