Merge branch 'stoa' of gitlab.com:colivre/noosfero into stoa

Rodrigo Souto
2 parents d8c8e9cf 57ae66f8
Showing 4 changed files with 15 additions and 8 deletions Show diff stats
app/helpers/search_term_helper.rb
app/views/home/welcome.html.erb
test/functional/enterprise_registration_controller_test.rb
test/functional/profile_editor_controller_test.rb
@@ -12,6 +12,7 @@ module SearchTermHelper
   #TODO Think smarter criteria to normalize search terms properly
   def normalize_term(search_term)
+    search_term ||= ''
     search_term.downcase
   end
 end
@@ -17,8 +17,11 @@
         <p><%= _("You won't appear as %s until your account is approved.") % link_to(_('user'), {:controller => :search, :action => :people, :filter => 'more_recent'}, :target => '_blank') %></p>
       <% end %>
       <h4><%= _("What to do next?") %></h4>
-      <p><%= _("Access your %s and see your face on the network!") % link_to(_('Profile'), {:controller => 'profile', :profile => user.identifier}, :target => '_blank') %></p>
-      <p><%= _("Explore your %s to customize your profile. Here are some %s on what you can do there.") % [link_to(_('Control Panel'), {:controller => 'profile_editor', :profile => user.identifier}, :target => '_blank'), link_to(_('tips'), {:controller => 'doc', :action => 'topic', :section => 'user', :topic => 'editing-person-info'}, :target => '_blank')] %></p>
+      <p><%= _("Access your %s and see your face on the network!") %
+        (user.present? ? link_to(_('Profile'), {:controller => 'profile', :profile => user.identifier}, :target => '_blank') : 'Profile') %>
+      <%= _("You can also explore your %s to customize your profile. Here are some %s on what you can do there.") %
+        [user.present? ? link_to(_('Control Panel'), {:controller => 'profile_editor', :profile => user.identifier}, :target => '_blank') : 'Control Panel',
+          link_to(_('tips'), {:controller => 'doc', :action => 'topic', :section => 'user', :topic => 'editing-person-info'}, :target => '_blank')] %></p>
       <p><%= _("%s your Gmail, Yahoo and Hotmail contacts!") % link_to(_('Invite and find'), {:controller => 'doc', :action => 'topic', :section => 'user', :topic => 'invite-contacts'}, :target => '_blank') %></p>
       <p><%= _("Learn the guidelines. Read the %s for more details on how to use this social network!") % link_to(_('Documentation'), {:controller => 'doc'}, :target => '_blank') %></p>
       <p><%= _("Start exploring and have fun!") %></p>
@@ -66,7 +66,7 @@ class EnterpriseRegistrationControllerTest &lt; ActionController::TestCase
     template.welcome_page = welcome_page
     template.save!
-    data = { :name => 'My new enterprise', :identifier => 'mynew', :region => region, :template_id => template.id }
+    data = { :name => 'My new enterprise', :identifier => 'mynew', :region_id => region.id, :template_id => template.id }
     create_enterprise = CreateEnterprise.new(data)
     post :index, :create_enterprise => data
@@ -880,14 +880,14 @@ class ProfileEditorControllerTest &lt; ActionController::TestCase
     end
   end
-  should 'have welcome_page only for person template' do
+  should 'have welcome_page only for template' do
     organization = fast_create(Organization, :is_template => false)
     @controller.stubs(:profile).returns(organization)
     assert !@controller.send(:has_welcome_page)
     organization = fast_create(Organization, :is_template => true)
     @controller.stubs(:profile).returns(organization)
-    assert !@controller.send(:has_welcome_page)
+    assert @controller.send(:has_welcome_page)
     person = fast_create(Person, :is_template => false)
     @controller.stubs(:profile).returns(person)
@@ -901,11 +901,11 @@ class ProfileEditorControllerTest &lt; ActionController::TestCase
   should 'display welcome_page button only if profile has_welcome_page' do
     @controller.stubs(:has_welcome_page).returns(true)
     get :index, :profile => fast_create(Profile).identifier
-    assert_tag :tag => 'a', :content => 'Edit Welcome Page'
+    assert_tag :tag => 'a', :content => 'Edit welcome page'
     @controller.stubs(:has_welcome_page).returns(false)
     get :index, :profile => fast_create(Profile).identifier
-    assert_no_tag :tag => 'a', :content => 'Edit Welcome Page'
+    assert_no_tag :tag => 'a', :content => 'Edit welcome page'
   end
   should 'not be able to access welcome_page if profile does not has_welcome_page' do
@@ -920,11 +920,14 @@ class ProfileEditorControllerTest &lt; ActionController::TestCase
   end
   should 'update welcome page and redirect to index' do
-    welcome_page = fast_create(TinyMceArticle, :body => 'Initial welcome page')
     person_template = create_user('person_template').person
     person_template.is_template = true
+
+    welcome_page = fast_create(TinyMceArticle, :body => 'Initial welcome page')
     person_template.welcome_page = welcome_page
     person_template.save!
+    welcome_page.profile = person_template
+    welcome_page.save!
     new_content = 'New welcome page'
     post :welcome_page, :profile => person_template.identifier, :welcome_page => {:body => new_content}
	@@ -12,6 +12,7 @@ module SearchTermHelper		@@ -12,6 +12,7 @@ module SearchTermHelper
12		12
13	#TODO Think smarter criteria to normalize search terms properly	13	#TODO Think smarter criteria to normalize search terms properly
14	def normalize_term(search_term)	14	def normalize_term(search_term)
		15	+ search_term \|\|= ''
15	search_term.downcase	16	search_term.downcase
16	end	17	end
17	end	18	end
	@@ -66,7 +66,7 @@ class EnterpriseRegistrationControllerTest < ActionController::TestCase		@@ -66,7 +66,7 @@ class EnterpriseRegistrationControllerTest < ActionController::TestCase
66	template.welcome_page = welcome_page	66	template.welcome_page = welcome_page
67	template.save!	67	template.save!
68		68
69	- data = { :name => 'My new enterprise', :identifier => 'mynew', :region => region, :template_id => template.id }	69	+ data = { :name => 'My new enterprise', :identifier => 'mynew', :region_id => region.id, :template_id => template.id }
70	create_enterprise = CreateEnterprise.new(data)	70	create_enterprise = CreateEnterprise.new(data)
71		71
72	post :index, :create_enterprise => data	72	post :index, :create_enterprise => data