Download as
Browse Code »

Commit 768db9b26e8aeb5bb51cb8b52ceb60645a78951d

Authored by Daniel
1 parent 247ad826
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

profile_editor: added mass-assign support for person's fields 

Added mass-assign support for all person's field that are configurable on
the fields environment page.

(ActionItem3147)
Showing 2 changed files with 15 additions and 6 deletions   Show diff stats
app/models/person.rb
  Diff comments   View file @768db9b
1 1 # A person is the profile of an user holding all relationships with the rest of the system
2 2 class Person < Profile
3 3  
4   - attr_accessible :organization, :contact_information, :sex, :birth_date
  4 + attr_accessible :organization, :contact_information, :sex, :birth_date, :cell_phone, :comercial_phone, :jabber_id, :personal_website, :nationality, :address_reference, :district, :schooling, :schooling_status, :formation, :custom_formation, :area_of_study, :custom_area_of_study, :professional_activity, :organization_website
5 5  
6 6 SEARCH_FILTERS += %w[
7 7 more_popular
... ...
test/functional/profile_editor_controller_test.rb
  Diff comments   View file @768db9b
... ... @@ -6,7 +6,7 @@ class ProfileEditorController; def rescue_action(e) raise e end; end
6 6  
7 7 class ProfileEditorControllerTest < ActionController::TestCase
8 8 all_fixtures
9   -
  9 +
10 10 def setup
11 11 @controller = ProfileEditorController.new
12 12 @request = ActionController::TestRequest.new
... ... @@ -20,11 +20,11 @@ class ProfileEditorControllerTest &lt; ActionController::TestCase
20 20 def test_local_files_reference
21 21 assert_local_files_reference :get, :index, :profile => profile.identifier
22 22 end
23   -
  23 +
24 24 def test_valid_xhtml
25 25 assert_valid_xhtml
26 26 end
27   -
  27 +
28 28 def test_index
29 29 get :index, :profile => profile.identifier
30 30 assert_template 'index'
... ... @@ -50,7 +50,7 @@ class ProfileEditorControllerTest &lt; ActionController::TestCase
50 50 end
51 51  
52 52 should 'saving profile info' do
53   - person = profile
  53 + person = profile
54 54 post :edit, :profile => profile.identifier, :profile_data => { 'name' => 'new person', 'contact_information' => 'new contact information', 'address' => 'new address', 'sex' => 'female' }
55 55 assert_redirected_to :controller => 'profile_editor', :action => 'index'
56 56 person = Person.find(person.id)
... ... @@ -60,6 +60,15 @@ class ProfileEditorControllerTest &lt; ActionController::TestCase
60 60 assert_equal 'female', person.sex
61 61 end
62 62  
  63 + should 'mass assign all environment configurable person fields' do
  64 + person = profile
  65 +
  66 + post :edit, :profile => profile.identifier, :profile_data => { "nickname" => "ze", "description" => "Just a regular ze.", "contact_information" => "What?", "contact_phone" => "+0551133445566", "cell_phone" => "+0551188889999", "comercial_phone" => "+0551144336655", "jabber_id" => "ze1234", "personal_website" => "http://ze.com.br", "sex" => "male", "birth_date" => "2014-06-04", "nationality" => "Brazilian", "country" => "BR", "state" => "DF", "city" => "Brasilia", "zip_code" => "70300-010", "address" => "Palacio do Planalto", "address_reference" => "Praca dos tres poderes", "district" => "DF", "schooling" => "Undergraduate", "schooling_status" => "Concluded", "formation" => "Engineerings", "area_of_study" => "Metallurgy", "professional_activity" => "Metallurgic", "organization" => "Metal Corp.", "organization_website" => "http://metal.com" }
  67 +
  68 + assert_response :redirect
  69 + assert_redirected_to :controller => 'profile_editor', :action => 'index'
  70 + end
  71 +
63 72 should 'not permmit if not logged' do
64 73 logout
65 74 get :index, :profile => profile.identifier
... ... @@ -426,7 +435,7 @@ class ProfileEditorControllerTest &lt; ActionController::TestCase
426 435 get :index, :profile => ent.identifier
427 436 assert_tag :tag => 'a', :attributes => { :href => "/myprofile/#{ent.identifier}/profile_editor/enable" }
428 437 end
429   -
  438 +
430 439 should 'link to disable enterprise' do
431 440 ent = fast_create(Enterprise, :enabled => true)
432 441 get :index, :profile => ent.identifier
... ...