From 072b0717e4d7fd69e1d28c5c4c3805c7fb4f563e Mon Sep 17 00:00:00 2001
From: Dan Karran <dan@karran.net>
Date: Sat, 16 Jun 2007 20:38:14 +0000
Subject: [PATCH] Adding titles to some more pages, adding a redirect after
 sent messages and setting correct XML content-type header for RSS feeds.

---
 app/controllers/diary_entry_controller.rb |  2 +-
 app/controllers/message_controller.rb     |  6 ++++++
 app/controllers/user_controller.rb        | 11 ++++++++++-
 3 files changed, 17 insertions(+), 2 deletions(-)

diff --git a/app/controllers/diary_entry_controller.rb b/app/controllers/diary_entry_controller.rb
index 9dba46879..4b94a7071 100644
--- a/app/controllers/diary_entry_controller.rb
+++ b/app/controllers/diary_entry_controller.rb
@@ -32,7 +32,7 @@ class DiaryEntryController < ApplicationController
       rss.add(latitude, longitude, entry.title, url_for({:controller => 'user', :action => 'diary', :id => entry.id, :display_name => entry.user.display_name}), entry.body, entry.created_at)
     end
 
-    response.headers["Content-Type"] = 'application/xml+rss'
+    response.headers["Content-Type"] = 'application/rss+xml'
 
     render :text => rss.to_s
   end
diff --git a/app/controllers/message_controller.rb b/app/controllers/message_controller.rb
index a95ad25b6..0fc5f212b 100644
--- a/app/controllers/message_controller.rb
+++ b/app/controllers/message_controller.rb
@@ -5,6 +5,7 @@ class MessageController < ApplicationController
   before_filter :require_user
 
   def new
+    @title = 'send message'
     if params[:message]
       to_user = User.find(params[:user_id])
       body = params[:message][:body]
@@ -19,6 +20,7 @@ class MessageController < ApplicationController
    
       if message.save
         flash[:notice] = 'Message sent'
+        redirect_to :controller => 'user', :action => 'view', :display_name => @user.display_name
       else
         @message.errors.add("Sending message failed")
       end
@@ -27,12 +29,16 @@ class MessageController < ApplicationController
   end
 
   def read
+    @title = 'read message'
     if params[:message_id]
       id = params[:message_id]
       @message = Message.find_by_id(id)
     end
   end
 
+  def inbox
+  end
+
   def mark
     if params[:message_id]
       id = params[:message_id]
diff --git a/app/controllers/user_controller.rb b/app/controllers/user_controller.rb
index 631da97d0..459058ee5 100644
--- a/app/controllers/user_controller.rb
+++ b/app/controllers/user_controller.rb
@@ -6,6 +6,7 @@ class UserController < ApplicationController
   before_filter :require_user, :only => [:set_home, :account, :go_public, :make_friend]
 
   def save
+    @title = 'create account'
     @user = User.new(params[:user])
     @user.set_defaults
 
@@ -19,6 +20,7 @@ class UserController < ApplicationController
   end
 
   def account
+    @title = 'edit account'
     if params[:user] and params[:user][:display_name] and params[:user][:description]
       home_lat =  params[:user][:home_lat]
       home_lon =  params[:user][:home_lon]
@@ -58,6 +60,7 @@ class UserController < ApplicationController
   end
 
   def lost_password
+    @title = 'lost password'
     if params[:user] and params[:user][:email]
       user = User.find_by_email(params['user']['email'])
       if user
@@ -74,6 +77,7 @@ class UserController < ApplicationController
   end
 
   def reset_password
+    @title = 'reset password'
     if params['token']
       user = User.find_by_token(params['token'])
       if user
@@ -92,9 +96,11 @@ class UserController < ApplicationController
   end
 
   def new
+    @title = 'create account'
   end
 
   def login
+    @title = 'login'
     if params[:user]
       email = params[:user][:email]
       pass = params[:user][:password]
@@ -149,6 +155,7 @@ class UserController < ApplicationController
   end
 
   def preferences
+    @title = 'preferences'
     if request.get?
       render_text @user.preferences
     elsif request.post? or request.put?
@@ -174,10 +181,12 @@ class UserController < ApplicationController
 
   def view
     @this_user = User.find_by_display_name(params[:display_name])
+    @title = @this_user.display_name
   end
 
   def diary
     @this_user = User.find_by_display_name(params[:display_name])
+    @title = @this_user.display_name + "'s diary"
   end
 
   def make_friend
@@ -196,7 +205,7 @@ class UserController < ApplicationController
       else
         flash[:notice] = "You are already friends with #{name}."  
       end
-        redirect_to :controller => 'user', :action => 'view'
+      redirect_to :controller => 'user', :action => 'view'
     end
   end
 
-- 
2.43.2