From: Tom Hughes <tom@compton.nu>
Date: Sun, 9 May 2010 16:16:43 +0000 (+0100)
Subject: Merge branch 'master' into terms
X-Git-Tag: live~6322^2~6
X-Git-Url: https://git.openstreetmap.org/rails.git/commitdiff_plain/61e51f88cc0242dadd3939cd0bbb5197a21cfb94?hp=-c

Merge branch 'master' into terms
---

61e51f88cc0242dadd3939cd0bbb5197a21cfb94
diff --combined app/controllers/user_controller.rb
index 8fdb89b22,42a50582e..1bab42266
--- a/app/controllers/user_controller.rb
+++ b/app/controllers/user_controller.rb
@@@ -18,29 -18,11 +18,29 @@@ class UserController < ApplicationContr
  
    cache_sweeper :user_sweeper, :only => [:account, :set_status, :delete]
  
 +  def terms
 +    @title = t 'user.new.title'
 +    @user = User.new(params[:user])
 +
 +    @legale = params[:legale] || OSM.IPToCountry(request.remote_ip) || APP_CONFIG['default_legale']
 +    @text = OSM.legal_text_for_country(@legale)
 +
 +    if request.xhr?
 +      render :update do |page|
 +        page.replace_html "contributorTerms", :partial => "terms"
 +      end
 +    elsif @user.invalid?
 +      render :action => 'new'
 +    end
 +  end
 +
    def save
      @title = t 'user.new.title'
  
      if Acl.find_by_address(request.remote_ip, :conditions => {:k => "no_account_creation"})
        render :action => 'new'
 +    elsif params[:decline]
 +      redirect_to t 'user.terms.declined'
      else
        @user = User.new(params[:user])
  
@@@ -49,7 -31,6 +49,7 @@@
        @user.description = "" if @user.description.nil?
        @user.creation_ip = request.remote_ip
        @user.languages = request.user_preferred_languages
 +      @user.terms_agreed = Time.now.getutc
  
        if @user.save
          flash[:notice] = t 'user.new.flash create success message'
@@@ -148,7 -129,7 +148,7 @@@
          if params[:user]
            @user.pass_crypt = params[:user][:pass_crypt]
            @user.pass_crypt_confirmation = params[:user][:pass_crypt_confirmation]
-           @user.status = "active"
+           @user.status = "active" if @user.status == "pending"
            @user.email_valid = true
  
            if @user.save