Merge 14394:14533 from trunk.
authorTom Hughes <tom@compton.nu>
Thu, 16 Apr 2009 21:11:12 +0000 (21:11 +0000)
committerTom Hughes <tom@compton.nu>
Thu, 16 Apr 2009 21:11:12 +0000 (21:11 +0000)
16 files changed:
1  2 
app/controllers/amf_controller.rb
app/controllers/api_controller.rb
app/controllers/application.rb
app/controllers/browse_controller.rb
app/controllers/diary_entry_controller.rb
app/controllers/message_controller.rb
app/controllers/node_controller.rb
app/controllers/old_node_controller.rb
app/controllers/old_way_controller.rb
app/controllers/relation_controller.rb
app/controllers/trace_controller.rb
app/controllers/user_controller.rb
app/controllers/way_controller.rb
app/views/layouts/site.rhtml
app/views/site/edit.rhtml
config/environment.rb

@@@ -37,11 -28,8 +37,11 @@@ class AmfController < ApplicationContro
  
    include Potlatch
  
 +  # Help methods for checking boundary sanity and area size
 +  include MapBoundary
 +
    session :off
-   before_filter :check_write_availability
+   before_filter :check_api_writable
  
    # Main AMF handlers: process the raw AMF string (using AMF library) and
    # calls each action (private method) accordingly.
Simple merge
Simple merge
Simple merge
@@@ -3,19 -3,16 +3,21 @@@ class MessageController < ApplicationCo
  
    before_filter :authorize_web
    before_filter :require_user
+   before_filter :check_database_readable
+   before_filter :check_database_writable, :only => [:new, :reply, :mark]
  
 +  # Allow the user to write a new message to another user. This action also 
 +  # deals with the sending of that message to the other user when the user
 +  # clicks send.
 +  # The user_id param is the id of the user that the message is being sent to.
    def new
      @title = 'send message'
 +    @to_user = User.find(params[:user_id])
      if params[:message]
        @message = Message.new(params[:message])
 -      @message.to_user_id = params[:user_id]
 +      @message.to_user_id = @to_user.id
        @message.from_user_id = @user.id
 -      @message.sent_on = Time.now
 +      @message.sent_on = Time.now.getutc
     
        if @message.save
          flash[:notice] = 'Message sent'
Simple merge
Simple merge
@@@ -1,11 -1,13 +1,13 @@@
  class TraceController < ApplicationController
    layout 'site'
  
 -  before_filter :authorize_web  
 -  before_filter :require_user, :only => [:mine, :edit, :delete, :make_public]
 +  before_filter :authorize_web
 +  before_filter :require_user, :only => [:mine, :create, :edit, :delete, :make_public]
    before_filter :authorize, :only => [:api_details, :api_data, :api_create]
-   before_filter :check_database_availability, :except => [:api_details, :api_data, :api_create]
-   before_filter :check_read_availability, :only => [:api_details, :api_data, :api_create]
+   before_filter :check_database_readable, :except => [:api_details, :api_data, :api_create]
+   before_filter :check_database_writable, :only => [:create, :edit, :delete, :make_public]
+   before_filter :check_api_readable, :only => [:api_details, :api_data]
+   before_filter :check_api_writable, :only => [:api_create]
   
    # Counts and selects pages of GPX traces for various criteria (by user, tags, public etc.).
    #  target_user - if set, specifies the user to fetch traces for.  if not set will fetch all traces
Simple merge
Simple merge
Simple merge
Simple merge
Simple merge