diff --git a/app/controllers/public/account_controller.rb b/app/controllers/public/account_controller.rb index c7d8b18..4b5483a 100644 --- a/app/controllers/public/account_controller.rb +++ b/app/controllers/public/account_controller.rb @@ -60,10 +60,6 @@ class AccountController < ApplicationController def signup @invitation_code = params[:invitation_code] begin - if params[:user] - params[:user].delete(:password_confirmation_clear) - params[:user].delete(:password_clear) - end @user = User.new(params[:user]) @user.terms_of_use = environment.terms_of_use @user.environment = environment @@ -213,26 +209,16 @@ class AccountController < ApplicationController @identifier = params[:identifier] valid = Person.is_available?(@identifier, environment) if valid - @status = _('This login name is available') + @status = _('Available!') @status_class = 'available' else - @status = _('This login name is unavailable') + @status = _('Unavailable!') @status_class = 'unavailable' end + @url = environment.top_url + '/' + @identifier render :partial => 'identifier_status' end - def check_email - if User.find_by_email_and_environment_id(params[:address], environment.id).nil? - @status = _('This e-mail address is available') - @status_class = 'available' - else - @status = _('This e-mail address is taken') - @status_class = 'unavailable' - end - render :partial => 'email_status' - end - def user_data user_data = if logged_in? diff --git a/app/models/person.rb b/app/models/person.rb index f7b0356..6fee0f5 100644 --- a/app/models/person.rb +++ b/app/models/person.rb @@ -116,7 +116,6 @@ class Person < Profile contact_phone contact_information description - image ] validates_multiparameter_assignments diff --git a/app/views/account/_email_status.rhtml b/app/views/account/_email_status.rhtml deleted file mode 100644 index 533e388..0000000 --- a/app/views/account/_email_status.rhtml +++ /dev/null @@ -1,7 +0,0 @@ -
<%= @status %>
- -<%= @status %>
- +<%= @url %> <%= @status %>
#{_('Checking if e-mail address is already taken...')}
');", - :complete => "jQuery('#user_email').removeClass('checking')", - :before => "if (!( jQuery('#user_email').valid() )) { - jQuery('#user_email').removeClass().addClass('unavailable'); - jQuery('#email-check').html('#{_('This e-mail address is not valid')}
'); - return false; - }" - %> - -
#{_('Checking availability of login name...')}
');", - :complete => "jQuery('#user_login').removeClass('checking')" - %> +<%= required_fields_message %> +- <%= submit_button('save', _('Sign up')) %> -
- +<%= required f.password_field(:password_confirmation) %> +<%= content_tag(:small,_('To confirm, repeat your password.')) %> + +<% labelled_fields_for :profile_data, @person do |f| %> + <%= render :partial => 'profile_editor/person_form', :locals => {:f => f} %> +<% end %> + +<% unless @terms_of_use.blank? %> +<%= labelled_check_box(environment.terms_of_use_acceptance_text.blank? ? _('I accept the terms of use') : environment.terms_of_use_acceptance_text, 'user[terms_accepted]') %>
+<%= link_to(_('Go to the homepage'), '/') %>
diff --git a/app/views/profile_editor/_person.rhtml b/app/views/profile_editor/_person.rhtml index 4b2e3da..7d1f357 100644 --- a/app/views/profile_editor/_person.rhtml +++ b/app/views/profile_editor/_person.rhtml @@ -2,8 +2,6 @@ <%= required_fields_message %> - <%= required f.text_field(:name) %> - - <%= required f.text_field(:email) %> + <%= required f.text_field(:email)%> <%= render :partial => 'person_form', :locals => {:f => f} %> diff --git a/app/views/profile_editor/_person_form.rhtml b/app/views/profile_editor/_person_form.rhtml index c84d99a..9d23436 100644 --- a/app/views/profile_editor/_person_form.rhtml +++ b/app/views/profile_editor/_person_form.rhtml @@ -1,26 +1,28 @@ <% @person ||= @profile %> +<%= required f.text_field(:name) %> + <% optional_field(@person, 'nickname') do %> - <%= f.text_field(:nickname, :maxlength => 16, :size => 30, :rel => _('Nickname')) %> + <%= f.text_field(:nickname, :maxlength => 16, :size => 30) %>