Download as
Browse Code »

Commit 8838b42cbe38a034047ea4af0445c6f0e99299cf

Authored by Rodrigo Souto
1 parent 3371aaf9
Exists in master and in 29 other branches add_member_task_reject_details, admin_visible_profile, article-list-template, community_notifications, contact_admin_translation, event_fixes, fix_comments_pagination, fix_rails4_organization_ratings, fix_sign_up_form, follow_step_fix, forum_topic_creation, mirror_block_improvements, multi_env_on_remote_user, new_security, noosfero_spb_ci, organization_ratings_improvements, organization_ratings_link_to_profile_stable-spb-1.3, organization_ratings_translations_fix, profile_api_improvements, ratings_minor_fixes, remote_user_fix, send_email_to_admins, stable-spb-1.3, stable-spb-1.3-fixes, stable-spb-1.4, stable-spb-1.5, suggest_rejected_value, web_steps_improvements, xss_terminate_custom_options

Fixing users that were supposed to be activated

Showing 3 changed files with 21 additions and 4 deletions   Show diff stats
app/models/user.rb
  Diff comments   View file @8838b42
... ... @@ -21,12 +21,11 @@ class User < ActiveRecord::Base
21 21 end
22 22 end
23 23  
24   - before_create :make_activation_code
25   -
26 24 before_create do |user|
27 25 if user.environment.nil?
28 26 user.environment = Environment.default
29 27 end
  28 + user.send(:make_activation_code) if !user.environment.enabled?('skip_new_user_email_confirmation')
30 29 end
31 30  
32 31 after_create do |user|
... ... @@ -35,7 +34,7 @@ class User < ActiveRecord::Base
35 34 user.person.name ||= user.login
36 35 user.person.visible = false unless user.activated?
37 36 user.person.save!
38   - if user.environment && user.environment.enabled?('skip_new_user_email_confirmation')
  37 + if user.environment.enabled?('skip_new_user_email_confirmation')
39 38 user.activate
40 39 end
41 40 end
... ... @@ -117,7 +116,7 @@ class User < ActiveRecord::Base
117 116 self.activated_at = Time.now.utc
118 117 self.activation_code = nil
119 118 self.person.visible = true
120   - self.person.save! && self.save
  119 + self.person.save! && self.save!
121 120 end
122 121  
123 122 def activated?
... ...
db/migrate/20120813163139_set_activation_code_to_nil_if_already_activated.rb 0 → 100644
  Diff comments   View file @8838b42
... ... @@ -0,0 +1,9 @@
  1 +class SetActivationCodeToNilIfAlreadyActivated < ActiveRecord::Migration
  2 + def self.up
  3 + update("UPDATE users SET activation_code = ? WHERE activated_at IS NOT NULL")
  4 + end
  5 +
  6 + def self.down
  7 + say('Can not be reverted.')
  8 + end
  9 +end
... ...
test/functional/account_controller_test.rb
  Diff comments   View file @8838b42
... ... @@ -610,6 +610,15 @@ class AccountControllerTest &lt; ActionController::TestCase
610 610 assert_equal 'example.com', Person['testuser'].organization
611 611 end
612 612  
  613 + should 'activate user after signup if environment is set to skip confirmation' do
  614 + env = Environment.default
  615 + env.enable('skip_new_user_email_confirmation')
  616 + env.save!
  617 + new_user(:login => 'activated_user')
  618 + user = User.find_by_login('activated_user')
  619 + assert user.activated?
  620 + end
  621 +
613 622 should 'redirect to initial page after logout' do
614 623 login_as :johndoe
615 624 get :logout
... ...