ActionItem1032: allowing login with custom domain

Antonio Terceiro
1 parent f7d6ffda
Showing 29 changed files with 211 additions and 98 deletions Show diff stats
app/controllers/my_profile/profile_editor_controller.rb
app/helpers/application_helper.rb
app/helpers/profile_editor_helper.rb
app/models/domain.rb
app/models/environment.rb
app/models/person.rb
app/models/profile.rb
app/models/profile_list_block.rb
app/views/account/index.rhtml
app/views/blocks/profile_info.rhtml
app/views/cms/view.rhtml
app/views/content_viewer/_article.rhtml
app/views/content_viewer/_uploaded_file.rhtml
app/views/content_viewer/view_page.rhtml
app/views/layouts/application.rhtml
app/views/manage_products/index.rhtml
app/views/profile_editor/_person_form.rhtml
app/views/profile_editor/edit.rhtml
app/views/shared/user_menu.rhtml
config/routes.rb
@@ -11,6 +11,7 @@ class ProfileEditorController &lt; MyProfileController
   # edits the profile info (posts back)
   def edit
     @profile_data = profile
+    @possible_domains = profile.possible_domains
     if request.post?
       if profile.update_attributes(params[:profile_data])
         redirect_to :action => 'index'
@@ -130,21 +130,6 @@ module ApplicationHelper
     link_to text, p.url, options
   end
-  def link_to_myprofile(text, url = {}, profile = nil, options = {})
-    p = if profile
-          Profile[profile]
-        else
-          user
-        end
-    link_to text, p.admin_url.merge(url), options
-  end
-
-  def link_to_document(doc, text = nil, html_options = {}, url_options = {})
-    text ||= doc.title
-    path = doc.path.split(/\//)
-    link_to text, homepage_path({:profile => doc.profile.identifier , :page => path, :view => true}.merge(url_options)), html_options
-  end
-
   def link_if_permitted(link, permission = nil, target = nil)
     if permission.nil? || current_user.person.has_permission?(permission, target)
       link
@@ -736,9 +721,7 @@ module ApplicationHelper
   end
   def login_url
-    options = { :controller => 'account', :action => 'login' }
-    options.merge!(:protocol => 'https://', :host => request.host) unless ENV['RAILS_ENV'] == 'development' || environment.disable_ssl
-    url_for(options)
+    url_for(Noosfero.url_options.merge({ :controller => 'account', :action => 'login', :host => request.host }))
   end
   def base_url
@@ -117,4 +117,20 @@ module ProfileEditorHelper
     select(object, method, [[_('[Select ...]'), nil]] + ProfileEditorHelper::SCHOOLING_STATUS.map{|s| [gettext(s), s]}, {}, options)
   end
+  def select_preferred_domain(object)
+    profile = instance_variable_get("@#{object}")
+    domains = []
+    if profile
+      if profile.preferred_domain
+        # FIXME should be able to change
+        return ''
+      else
+        domains = profile.possible_domains
+      end
+    else
+      domains = environment.domains
+    end
+    labelled_form_field(__('Preferred domain name:'), select(object, :preferred_domain_id, domains.map {|item| [item.name, item.id]}, :prompt => '&lt;' + _('Select domain') + '&gt;'))
+  end
+
 end
@@ -63,4 +63,25 @@ class Domain &lt; ActiveRecord::Base
     end
   end
+  @hosting = {}
+
+  # Tells whether the given domain name is hosting a profile or not.
+  #
+  # Since this is going to be called a lot, The queries are cached so the
+  # database is hit only for the first query for a given domain name. This way,
+  # transfering a domain from a profile to an environment of vice-versa
+  # requires restarting the application.
+  def self.hosting_profile_at(domainname)
+    @hosting[domainname] ||=
+      begin
+        domain = Domain.find_by_name(domainname)
+        !domain.nil? && (domain.owner_type == 'Profile')
+      end
+  end
+
+  # clears the cache of hosted domains. Used for testing.
+  def self.clear_cache
+    @hosting = {}
+  end
+
 end
@@ -45,6 +45,7 @@ class Environment &lt; ActiveRecord::Base
       'warn_obsolete_browser' => _('Enable warning of obsolete browser'),
       'wysiwyg_editor_for_environment_home' => _('Use WYSIWYG editor to edit environment home page'),
       'media_panel' => _('Media panel in WYSIWYG editor'),
+      'select_preferred_domain' => _('Select preferred domains per profile'),
     }
   end
@@ -42,6 +42,7 @@ class Person &lt; Profile
   end
   FIELDS = %w[
+  preferred_domain
   nickname
   sex
   address
@@ -94,6 +94,7 @@ class Profile &lt; ActiveRecord::Base
   belongs_to :user
   has_many :domains, :as => :owner
+  belongs_to :preferred_domain, :class_name => 'Domain', :foreign_key => 'preferred_domain_id'
   belongs_to :environment
   has_many :articles, :dependent => :destroy
@@ -327,19 +328,15 @@ class Profile &lt; ActiveRecord::Base
   end
   def url
-    @url ||= if self.domains.empty?
-      generate_url(:controller => 'content_viewer', :action => 'view_page', :page => [])
-    else
-      Noosfero.url_options.merge({ :host => self.domains.first.name, :controller => 'content_viewer', :action => 'view_page', :page => []})
-    end
+    @url ||= generate_url(:controller => 'content_viewer', :action => 'view_page', :page => [])
   end
   def admin_url
-    generate_url(:controller => 'profile_editor', :action => 'index')
+    { :profile => identifier, :controller => 'profile_editor', :action => 'index' }
   end
   def public_profile_url
-    generate_url(:controller => 'profile', :action => 'index')
+    generate_url(:profile => identifier, :controller => 'profile', :action => 'index')
   end
   def generate_url(options)
@@ -347,7 +344,25 @@ class Profile &lt; ActiveRecord::Base
   end
   def url_options
-    Noosfero.url_options.merge({ :host => self.environment.default_hostname, :profile => self.identifier})
+    options = { :host => default_hostname, :profile => (hostname ? nil : self.identifier) }
+    Noosfero.url_options.merge(options)
+  end
+
+  def default_hostname
+    @default_hostname ||= (hostname || environment.default_hostname)
+  end
+
+  def hostname
+    if preferred_domain
+      return preferred_domain.name
+    else
+      domain = self.domains.first
+      domain ? domain.name : nil
+    end
+  end
+
+  def possible_domains
+    environment.domains + domains
   end
   # FIXME this can be SLOW
@@ -64,12 +64,10 @@ class ProfileListBlock &lt; Block
     title = self.title
     nl = "\n"
     lambda do
-      #list = profiles.map {|item| content_tag( 'li', profile_image_link(item) ) }.join("\n  ")
       count=0
       list = profiles.map {|item|
                count+=1
                profile_image_link( item ) #+
-               #( ( count%2 == 0 && count < profiles.size ) ? nl+'</tr><tr>' : '' )
              }.join("\n  ")
       if list.empty?
         list = '<div class="common-profile-list-block-none">'+ _('None') +'</div>'
@@ -6,7 +6,7 @@
 </p>
 <p>
-<%= link_to_myprofile _('Edit Personal details.'), :controller => 'profile_editor' %>
+<%= link_to _('Edit Personal details.'), :controller => 'profile_editor', :profile => user.identifier, :action => 'index' %>
 <%= _('You can change your personal details.') %>
 </p>
@@ -16,7 +16,7 @@
 </p>
 <p>
-<%= link_to_myprofile(_('Manage content.'), :controller => 'cms') %>
+<%= link_to(_('Manage content.'), :controller => 'cms', :profile => user.identifier, :action => 'index') %>
 <%= _('Manage your content.') %>
 </p>
@@ -19,7 +19,7 @@
   <li><%= link_to _('View profile'), block.owner.public_profile_url %></li>
   <li><%= link_to(_('Products/Services'), :controller => 'catalog', :profile => block.owner.identifier) if block.owner.enterprise? && !block.owner.environment.enabled?('disable_products_for_enterprises') %></li>
   <% if !user.nil? and user.has_permission?('edit_profile', profile) %>
-    <li><%= link_to _('Control panel'), :controller => 'profile_editor' %></li>
+    <li><%= link_to _('Control panel'), block.owner.admin_url %></li>
   <% end %>
   <% if profile.person? %>
     <li><%= _('Since %{year}/%{month}') % { :year => block.owner.created_at.year, :month => block.owner.created_at.month } %></li>
@@ -7,7 +7,7 @@
   <h1 id='article-full-path'>
     <%= icon('cms') %>
     <%= link_to profile.identifier, :action => 'index' %>
-    <%= @article.hierarchy.map {|item| " / " + ((item == @article) ? item.name : link_to_document(item)) } %>
+    <%= @article.hierarchy.map {|item| " / " + ((item == @article) ? item.name : link_to(item.slug, :id => item.id)) } %>
   </h1>
 <% end %>
@@ -42,7 +42,7 @@
       </td>
       <td>
         <%= link_to _('Properties'), :action => 'edit', :id => folder.id %>
-        <%= link_to_document(folder, _('Public view')) %>
+        <%= link_to _('Public view'), folder.url %>
         <%= link_to _('Use as homepage'), { :action => 'set_home_page', :id => folder.id }, :method => :post %>
         <%= link_to _('Delete'), { :action => 'destroy', :id => folder.id }, :method => :post, :confirm => _('Are you sure that you want to remove this folder? Note that all the items inside it will also be removed!') %>
       </td>
@@ -68,7 +68,7 @@
       </td>
       <td>
         <%= link_to _('Edit'), :action => 'edit', :id => item.id %>
-        <%= link_to_document(item, _('Public view')) %>
+        <%= link_to _('Public view'), item.url %>
         <%= link_to _('Spread this'), :action => 'publish', :id => item.id %>
         <%= link_to _('Use as homepage'), { :action => 'set_home_page', :id => item.id }, :method => :post %>
         <%= link_to _('Delete'), { :action => 'destroy', :id => item.id }, :method => :post, :confirm => _('Are you sure that you want to remove this item?') %>
-<%= link_to_document(article, '', :class => article.css_class_name) %>
-<span><%= link_to_document(article) %></span>
+<%= link_to '', article.url, '', :class => article.css_class_name) %>
+<span><%= link_to article.title, article.view_url %></span>
 <% if uploaded_file.image? %>
-  <div> <%= link_to_document(uploaded_file, image_tag(uploaded_file.public_filename(:thumb))) %> </div>
+  <div> <%= link_to image_tag(uploaded_file.public_filename(:thumb)), uploaded_file.view_url %> </div>
 <% else %>
   <%= render :partial => 'article', :object => uploaded_file %>
 <% end %>
@@ -34,24 +34,24 @@
     <div id="article-actions" class="hidden">
       <% unless @page.blog? %>
       <%= link_to content_tag( 'span', label_for_edit_article(@page) ),
-          { :controller => 'cms', :action => 'edit', :id => @page },
+          profile.admin_url.merge({ :controller => 'cms', :action => 'edit', :id => @page.id }),
           :class => 'button with-text icon-edit' %>
       <% end %>
       <% if !(profile.kind_of?(Enterprise) && environment.enabled?('disable_cms')) %>
         <% if @page != profile.home_page %>
           <%= link_to content_tag( 'span', _('Delete') ),
-            { :controller => 'cms', :action => 'destroy', :id => @page },
+            profile.admin_url.merge({ :controller => 'cms', :action => 'destroy', :id => @page }),
             :class => 'button with-text icon-delete' %>
         <% end %>
         <% if profile.kind_of?(Person) && !environment.enabled?('disable_cms') %>
           <%= link_to content_tag( 'span', _('Spread this') ),
-              { :controller => 'cms', :action => 'publish', :id => @page },
+              profile.admin_url.merge({ :controller => 'cms', :action => 'publish', :id => @page }),
               :class => 'button with-text icon-spread' %>
         <% end %>
         <% if @page.display_as_gallery? %>
-          <%= button('upload-file', _('Upload files'), :controller => 'cms', :action => 'upload_files', :parent_id => (@page.folder? ? @page : @page.parent)) %>
+          <%= button('upload-file', _('Upload files'), profile.admin_url.merge(:controller => 'cms', :action => 'upload_files', :parent_id => (@page.folder? ? @page : @page.parent))) %>
         <% else %>
-          <%= lightbox_button(:new, label_for_new_article(@page), :controller => 'cms', :action => 'new', :parent_id => (@page.folder? ? @page : (@page.parent.nil? ? nil : @page.parent))) %>
+          <%= lightbox_button(:new, label_for_new_article(@page), profile.admin_url.merge(:controller => 'cms', :action => 'new', :parent_id => (@page.folder? ? @page : (@page.parent.nil? ? nil : @page.parent)))) %>
         <% end %>
       <% end %>
     </div>
@@ -73,7 +73,7 @@
 <% if @page.parent && !@page.parent.path.blank? %>
 <div id="article-parent">
-  <%= link_to_document(@page.parent, _('Go back')) %>
+  <%= link_to _('Go back'), @page.parent.url %>
 </div>
 <% end %>
@@ -49,7 +49,6 @@
     <%= icon_theme_stylesheet_tag %>
     <%= stylesheet_link_tag 'iepngfix/iepngfix.css' %>
-    <base href="<%= base_url %>"/>
   </head>
   <body class='noosfero category<%= category_color %><%=
@@ -110,7 +109,7 @@
         <div id="navigation_bar">
           <%= link_to "<span>"+ @environment.name() +"</span>",
-                      { :controller=>"home" },
+                      @environment.top_url,
                         :id=>"menu_link_to_envhome",
                         :title=>@environment.name  %>
           <% unless environment.enabled?(:disable_categories) %>
@@ -47,6 +47,6 @@
 <%= button :add, _('Add raw material'), :action => 'new_consumption' %>
 <p>
-<%= link_to_myprofile( '<span>'+ _('Back') +'</span>', {}, @profile.identifier, :class => 'button with-text icon-back' ) %>
+<%= link_to( '<span>'+ _('Back') +'</span>', { :controller => 'profile_editor', :profile => @profile.identifier, :action => 'index' }, :class => 'button with-text icon-back' ) %>
 </p>
@@ -9,6 +9,7 @@
   </div>
 <% end %>
+<%= optional_field(@person, 'preferred_domain', select_preferred_domain(:profile_data)) %>
 <%= optional_field(@person, 'contact_information', f.text_field(:contact_information)) %>
 <%= optional_field(@person, 'contact_phone', labelled_form_field(_('Home phone'), text_field(:profile_data, :contact_phone))) %>
 <%= optional_field(@person, 'cell_phone', f.text_field(:cell_phone)) %>
@@ -13,6 +13,7 @@
     <% end %>
   </div>
+
   <h2><%= _('Privacy options') %></h2>
   <p>
     <%= _('This profile is:') %>
@@ -27,10 +27,9 @@
           </li>
         <% end %>
-        <li><%= link_to_myprofile( '<span class="icon-menu-ctrl-panel"></span>'+ _('Control panel'),
-                                     {}, nil, :id => 'link_edit_profile',
-                                     :help => _('Control panel: change your picture, edit your personal information, create content or change the way your home page looks.')
-                                 ) %></li>
+        <li>
+        <%= link_to('<span class="icon-menu-ctrl-panel"></span>'+ _('Control panel'), { :controller => 'profile_editor', :action => 'index', :profile => user.identifier }, :id => 'link_edit_profile', :help => _('Control panel: change your picture, edit your personal information, create content or change the way your home page looks.')) %>
+        </li>
         <% if user.is_admin?(environment) %>
           <li><%= link_to( '<span class="icon-menu-"></span>'+ _('Admin'),
@@ -15,18 +15,11 @@ ActionController::Routing::Routes.draw do |map|
   ## Public controllers
   ######################################################
-  # You can have the root of your site routed by hooking up ''
-  hosted_domain_matcher = lambda do |env|
-    domain = Domain.find_by_name(env[:host])
-    domain && (domain.owner_type == 'Profile')
-  end
-  map.connect 'profile/:profile/:action/:id', :controller => 'profile', :id => /.*/, :profile => /#{Noosfero.identifier_format}/, :conditions => { :if => hosted_domain_matcher }
-  map.connect '*page', :controller => 'content_viewer', :action => 'view_page', :conditions => { :if => hosted_domain_matcher }
-
   map.connect 'test/:controller/:action/:id'  , :controller => /.*test.*/
   # -- just remember to delete public/index.html.
-  map.connect '', :controller => "home"
+  # You can have the root of your site routed by hooking up ''
+  map.connect '', :controller => "home", :conditions => { :if => lambda { |env| !Domain.hosting_profile_at(env[:host]) } }
   # user account controller
   map.connect 'account/new_password/:code', :controller => 'account', :action => 'new_password'
@@ -83,7 +76,9 @@ ActionController::Routing::Routes.draw do |map|
   # cache stuff - hack
   map.cache 'public/:action/:id', :controller => 'public'
-  # *content viewing*
+  # match requests for content in domains hosted for profiles
+  map.connect '*page', :controller => 'content_viewer', :action => 'view_page', :conditions => { :if => lambda { |env| Domain.hosting_profile_at(env[:host])} }
+
   # XXX this route must come last so other routes have priority over it.
   map.homepage ':profile/*page', :controller => 'content_viewer', :action => 'view_page', :profile => /#{Noosfero.identifier_format}/
@@ -9,7 +9,7 @@
 #
 # It's strongly recommended to check this file into your version control system.
-ActiveRecord::Schema.define(:version => 65) do
+ActiveRecord::Schema.define(:version => 66) do
   create_table "article_versions", :force => true do |t|
     t.integer  "article_id"
@@ -87,8 +87,8 @@ ActiveRecord::Schema.define(:version =&gt; 65) do
     t.boolean "virtual",     :default => false
   end
-  add_index "articles_categories", ["article_id"], :name => "index_articles_categories_on_article_id"
   add_index "articles_categories", ["category_id"], :name => "index_articles_categories_on_category_id"
+  add_index "articles_categories", ["article_id"], :name => "index_articles_categories_on_article_id"
   create_table "blocks", :force => true do |t|
     t.string  "title"
@@ -128,8 +128,8 @@ ActiveRecord::Schema.define(:version =&gt; 65) do
     t.boolean "virtual",     :default => false
   end
-  add_index "categories_profiles", ["profile_id"], :name => "index_categories_profiles_on_profile_id"
   add_index "categories_profiles", ["category_id"], :name => "index_categories_profiles_on_category_id"
+  add_index "categories_profiles", ["profile_id"], :name => "index_categories_profiles_on_profile_id"
   create_table "comments", :force => true do |t|
     t.string   "title"
@@ -207,8 +207,8 @@ ActiveRecord::Schema.define(:version =&gt; 65) do
     t.datetime "updated_at"
   end
-  add_index "product_categorizations", ["product_id"], :name => "index_product_categorizations_on_product_id"
   add_index "product_categorizations", ["category_id"], :name => "index_product_categorizations_on_category_id"
+  add_index "product_categorizations", ["product_id"], :name => "index_product_categorizations_on_product_id"
   create_table "products", :force => true do |t|
     t.integer  "enterprise_id"
@@ -230,7 +230,7 @@ ActiveRecord::Schema.define(:version =&gt; 65) do
     t.string   "type"
     t.string   "identifier"
     t.integer  "environment_id"
-    t.boolean  "active",                          :default => true
+    t.boolean  "active",                            :default => true
     t.string   "address"
     t.string   "contact_phone"
     t.integer  "home_page_id"
@@ -241,13 +241,14 @@ ActiveRecord::Schema.define(:version =&gt; 65) do
     t.float    "lat"
     t.float    "lng"
     t.integer  "geocode_precision"
-    t.boolean  "enabled",                         :default => true
-    t.string   "nickname",          :limit => 16
+    t.boolean  "enabled",                           :default => true
+    t.string   "nickname",            :limit => 16
     t.text     "custom_header"
     t.text     "custom_footer"
     t.string   "theme"
-    t.boolean  "public_profile",                  :default => true
+    t.boolean  "public_profile",                    :default => true
     t.date     "birth_date"
+    t.integer  "preferred_domain_id"
   end
   add_index "profiles", ["environment_id"], :name => "index_profiles_on_environment_id"
@@ -285,8 +286,8 @@ ActiveRecord::Schema.define(:version =&gt; 65) do
     t.datetime "created_at"
   end
-  add_index "taggings", ["tag_id"], :name => "index_taggings_on_tag_id"
   add_index "taggings", ["taggable_id", "taggable_type"], :name => "index_taggings_on_taggable_id_and_taggable_type"
+  add_index "taggings", ["tag_id"], :name => "index_taggings_on_tag_id"
   create_table "tags", :force => true do |t|
     t.string  "name"
@@ -65,6 +65,7 @@ class Zen3Terminology &lt; Noosfero::Terminology::Custom
       'Disable activation of enterprises' => N_('Disable activation of organizations'),
       'Manage community fields' => N_('Manage group fields'),
       'Create a new community' => N_('Create a new group'),
+      'Preferred domain name:' => N_('Choose your host community:'),
     })
   end
@@ -600,6 +600,7 @@ class AccountControllerTest &lt; Test::Unit::TestCase
   end
   should 'not point to SSL URL in login popup when in development mode' do
+    @request.stubs(:ssl?).returns(false)
     ENV.expects(:[]).with('RAILS_ENV').returns('development').at_least_once
     get :login_popup
     assert_no_tag :tag => 'form', :attributes => { :action => /^https:\/\// }
@@ -234,32 +234,6 @@ class ApplicationControllerTest &lt; Test::Unit::TestCase
     assert_no_tag :tag => 'div', :attributes => { :id => 'user_box' }, :descendant => { :tag => 'a', :attributes => { :href => 'http://web.mail/' } }
   end
-  should 'use environment top_url as base' do
-    Environment.any_instance.expects(:top_url).returns('http://www.lala.net')
-    get :index
-    assert_tag :tag => 'base', :attributes => { :href => 'http://www.lala.net' }
-  end
-
-  should 'request environment top_url with ssl when under ssl for base' do
-    Environment.any_instance.expects(:top_url).with(true).returns('https://www.lala.net/')
-    @request.expects(:ssl?).returns(true).at_least_once
-    get :index
-    assert_tag :tag => 'base', :attributes => { :href => 'https://www.lala.net/' }
-  end
-
-  should 'use correct environment for base tag' do
-    default = Environment.default
-    Environment.stubs(:default).returns(default)
-    default.stubs(:top_url).returns('http://default.com/')
-
-    current = Environment.create!(:name => 'test environment')
-    current.domains.create!(:name => 'example.com')
-    
-    @request.expects(:host).returns('example.com').at_least_once
-    get :index
-    assert_tag :tag => 'base', :attributes => { :href => 'http://example.com' }
-  end
-
   should 'display theme test panel when testing theme' do
     @request.session[:theme] = 'my-test-theme'
     theme = mock
@@ -632,4 +632,39 @@ class ProfileEditorControllerTest &lt; Test::Unit::TestCase
     assert_tag :tag => 'a', :attributes => { :href => "/myprofile/default_user/cms/edit/#{profile.blog.id}" }
   end
+  should 'not show select preferred domain if not enabled in environment' do
+    profile.environment.custom_person_fields = {}; profile.environment.save!
+
+    get :edit, :profile => profile.identifier
+    assert_no_tag :tag => 'select', :attributes => { :name => 'profile_data[preferred_domain_id]' }
+  end
+
+  should 'be able to choose preferred domain' do
+    profile.environment.custom_person_fields = {'preferred_domain' => {'required' => 'true', 'active' => 'true'} }; profile.environment.save!
+
+    profile.domains << Domain.new(:name => 'myowndomain.net')
+    profile.environment.domains << Domain.new(:name => 'myenv.net')
+
+    get :edit, :profile => profile.identifier
+    assert_tag :tag => 'select', :attributes => { :name => 'profile_data[preferred_domain_id]' }, :descendant => { :tag => "option", :content => 'myowndomain.net' }
+    assert_tag :tag => 'select', :attributes => { :name => 'profile_data[preferred_domain_id]' }, :descendant => { :tag => "option", :content => 'myenv.net' }
+
+    post :edit, :profile => profile.identifier, :profile_data => { :preferred_domain_id => profile.domains.first.id.to_s }
+
+    assert_equal 'myowndomain.net', Profile.find(profile.id).preferred_domain.name
+  end
+
+  should 'be able to set no preferred domain at all' do
+    profile.environment.custom_person_fields = {'preferred_domain' => {'required' => 'true', 'active' => 'true'} }; profile.environment.save!
+
+    profile.domains << Domain.new(:name => 'myowndomain.net')
+    profile.environment.domains << Domain.new(:name => 'myenv.net')
+
+    get :edit, :profile => profile.identifier
+    assert_tag :tag => "select", :attributes => { :name => 'profile_data[preferred_domain_id]'}, :descendant => { :tag => 'option', :content => '&lt;Select domain&gt;', :attributes => { :value => '' } }
+
+    post :edit, :profile => profile.identifier, :profile_data => { :preferred_domain_id => '' }
+    assert_nil Profile.find(profile.id).preferred_domain
+  end
+
 end
@@ -2,6 +2,10 @@ require &quot;#{File.dirname(__FILE__)}/../test_helper&quot;
 class RoutingTest < ActionController::IntegrationTest
+  def setup
+    Domain.clear_cache
+  end
+
   # home page
   ################################################################
   def test_homepage
@@ -165,7 +169,6 @@ class RoutingTest &lt; ActionController::IntegrationTest
   end
   def test_hosted_domain_routing
-
     user = create_user('testuser').person
     domain = Domain.create!(:name => 'example.com', :owner => user)
@@ -174,6 +177,15 @@ class RoutingTest &lt; ActionController::IntegrationTest
     assert_routing('/work/free-software', :controller => 'content_viewer', :action =>  'view_page', :page => [ 'work', 'free-software'] )
   end
+  def test_root_of_hosted_domain
+    user = create_user('testuser').person
+    domain = Domain.create!(:name => 'example.com', :owner => user)
+
+    ActionController::TestRequest.any_instance.expects(:host).returns('www.example.com')
+
+    assert_routing('', :controller => 'content_viewer', :action =>  'view_page', :page => [])
+  end
+
   def test_profile_under_hosted_domain
     community = Community.create!(:identifier => 'testcomm', :name => "test community")
     domain = Domain.create!(:name => 'example.com', :owner => community)
@@ -202,6 +202,9 @@ class ApplicationHelperTest &lt; Test::Unit::TestCase
     environment = mock
     environment.expects(:disable_ssl).returns(true).at_least_once
     stubs(:environment).returns(environment)
+    request = mock
+    request.stubs(:host).returns('localhost')
+    stubs(:request).returns(request)
     expects(:url_for).with(has_entries(:protocol => 'https://')).never
     expects(:url_for).with(has_key(:controller)).returns("LALALA")
@@ -3,6 +3,10 @@ require File.dirname(__FILE__) + &#39;/../test_helper&#39;
 class DomainTest < Test::Unit::TestCase
   fixtures :domains, :environments, :profiles, :users
+  def setup
+    Domain.clear_cache
+  end
+
   # Replace this with your real tests.
   def test_domain_name_format
     c = Domain.new
@@ -83,4 +87,17 @@ class DomainTest &lt; Test::Unit::TestCase
     assert_nil Domain.find_by_name('colivre.net').profile
   end
+  def test_hosted_domain
+    assert_equal false, Domain.hosting_profile_at('example.com')
+
+    profile = create_user('hosted_user').person
+    Domain.create!(:name => 'example.com', :owner => profile)
+    assert_equal true, Domain.hosting_profile_at('example.com')
+  end
+
+  def test_not_report_profile_hosted_for_environment_domains
+    Domain.create!(:name => 'example.com', :owner => Environment.default)
+    assert_equal false, Domain.hosting_profile_at('example.com')
+  end
+
 end
 require File.dirname(__FILE__) + '/../test_helper'
 class ProfileTest < Test::Unit::TestCase
-  fixtures :profiles, :environments, :users, :roles
+  fixtures :profiles, :environments, :users, :roles, :domains
   def test_identifier_validation
     p = Profile.new
@@ -284,7 +284,7 @@ class ProfileTest &lt; Test::Unit::TestCase
   should "use own domain name instead of environment's for home page url" do
     profile = Profile.create!(:name => "Test Profile", :identifier => 'testprofile', :environment_id => create_environment('mycolivre.net').id)
     profile.domains << Domain.new(:name => 'micojones.net')
-    assert_equal({:host => 'micojones.net', :controller => 'content_viewer', :action => 'view_page', :page => []}, profile.url)
+    assert_equal({:host => 'micojones.net', :profile => nil, :controller => 'content_viewer', :action => 'view_page', :page => []}, profile.url)
   end
   should 'help developers by adding a suitable port to url options' do
@@ -1258,6 +1258,44 @@ class ProfileTest &lt; Test::Unit::TestCase
     assert_nil person.find_in_all_tasks(task2.id)
   end
+  should 'use environment hostname by default' do
+    profile = Profile.new
+    env = mock
+    env.stubs(:default_hostname).returns('myenvironment.net')
+    profile.stubs(:environment).returns(env)
+    assert_equal 'myenvironment.net', profile.default_hostname
+  end
+
+  should 'use its first domain hostname name if available' do
+    profile = create_user('testuser').person
+    profile.domains << Domain.new(:name => 'myowndomain.net')
+    assert_equal 'myowndomain.net', profile.default_hostname
+  end
+
+  should 'have a preferred domain name' do
+    person = create_user('testuser').person
+    domain = Domain.create!(:name => 'myowndomain.net', :owner => person)
+    person.preferred_domain = domain
+    person.save!
+
+    assert_equal domain, Person.find(person.id).preferred_domain(true)
+  end
+
+  should 'use preferred domain for hostname' do
+    profile = Profile.new
+    profile.stubs(:preferred_domain).returns(Domain.new(:name => 'preferred.net'))
+    assert_equal 'preferred.net', profile.url[:host]
+  end
+
+  should 'provide a list of possible preferred domain names' do
+    profile = create_user('testuser').person
+    domain1 = Domain.create!(:name => 'envdomain.net', :owner => profile.environment)
+    domain2 = Domain.create!(:name => 'profiledomain.net', :owner => profile)
+
+    assert_includes profile.possible_domains, domain1
+    assert_includes profile.possible_domains, domain2
+  end
+
   private
   def assert_invalid_identifier(id)
	@@ -130,21 +130,6 @@ module ApplicationHelper		@@ -130,21 +130,6 @@ module ApplicationHelper
130	link_to text, p.url, options	130	link_to text, p.url, options
131	end	131	end
132		132
133	- def link_to_myprofile(text, url = {}, profile = nil, options = {})
134	- p = if profile
135	- Profile[profile]
136	- else
137	- user
138	- end
139	- link_to text, p.admin_url.merge(url), options
140	- end
141	-
142	- def link_to_document(doc, text = nil, html_options = {}, url_options = {})
143	- text \|\|= doc.title
144	- path = doc.path.split(/\//)
145	- link_to text, homepage_path({:profile => doc.profile.identifier , :page => path, :view => true}.merge(url_options)), html_options
146	- end
147	-
148	def link_if_permitted(link, permission = nil, target = nil)	133	def link_if_permitted(link, permission = nil, target = nil)
149	if permission.nil? \|\| current_user.person.has_permission?(permission, target)	134	if permission.nil? \|\| current_user.person.has_permission?(permission, target)
150	link	135	link
	@@ -736,9 +721,7 @@ module ApplicationHelper		@@ -736,9 +721,7 @@ module ApplicationHelper
736	end	721	end
737		722
738	def login_url	723	def login_url
739	- options = { :controller => 'account', :action => 'login' }
740	- options.merge!(:protocol => 'https://', :host => request.host) unless ENV['RAILS_ENV'] == 'development' \|\| environment.disable_ssl
741	- url_for(options)	724	+ url_for(Noosfero.url_options.merge({ :controller => 'account', :action => 'login', :host => request.host }))
742	end	725	end
743		726
744	def base_url	727	def base_url
	@@ -117,4 +117,20 @@ module ProfileEditorHelper		@@ -117,4 +117,20 @@ module ProfileEditorHelper
117	select(object, method, [[_('[Select ...]'), nil]] + ProfileEditorHelper::SCHOOLING_STATUS.map{\|s\| [gettext(s), s]}, {}, options)	117	select(object, method, [[_('[Select ...]'), nil]] + ProfileEditorHelper::SCHOOLING_STATUS.map{\|s\| [gettext(s), s]}, {}, options)
118	end	118	end
119		119
		120	+ def select_preferred_domain(object)
		121	+ profile = instance_variable_get("@#{object}")
		122	+ domains = []
		123	+ if profile
		124	+ if profile.preferred_domain
		125	+ # FIXME should be able to change
		126	+ return ''
		127	+ else
		128	+ domains = profile.possible_domains
		129	+ end
		130	+ else
		131	+ domains = environment.domains
		132	+ end
		133	+ labelled_form_field(__('Preferred domain name:'), select(object, :preferred_domain_id, domains.map {\|item\| [item.name, item.id]}, :prompt => '<' + _('Select domain') + '>'))
		134	+ end
		135	+
120	end	136	end
	@@ -63,4 +63,25 @@ class Domain < ActiveRecord::Base		@@ -63,4 +63,25 @@ class Domain < ActiveRecord::Base
63	end	63	end
64	end	64	end
65		65
		66	+ @hosting = {}
		67	+
		68	+ # Tells whether the given domain name is hosting a profile or not.
		69	+ #
		70	+ # Since this is going to be called a lot, The queries are cached so the
		71	+ # database is hit only for the first query for a given domain name. This way,
		72	+ # transfering a domain from a profile to an environment of vice-versa
		73	+ # requires restarting the application.
		74	+ def self.hosting_profile_at(domainname)
		75	+ @hosting[domainname] \|\|=
		76	+ begin
		77	+ domain = Domain.find_by_name(domainname)
		78	+ !domain.nil? && (domain.owner_type == 'Profile')
		79	+ end
		80	+ end
		81	+
		82	+ # clears the cache of hosted domains. Used for testing.
		83	+ def self.clear_cache
		84	+ @hosting = {}
		85	+ end
		86	+
66	end	87	end
	@@ -45,6 +45,7 @@ class Environment < ActiveRecord::Base		@@ -45,6 +45,7 @@ class Environment < ActiveRecord::Base
45	'warn_obsolete_browser' => _('Enable warning of obsolete browser'),	45	'warn_obsolete_browser' => _('Enable warning of obsolete browser'),
46	'wysiwyg_editor_for_environment_home' => _('Use WYSIWYG editor to edit environment home page'),	46	'wysiwyg_editor_for_environment_home' => _('Use WYSIWYG editor to edit environment home page'),
47	'media_panel' => _('Media panel in WYSIWYG editor'),	47	'media_panel' => _('Media panel in WYSIWYG editor'),
		48	+ 'select_preferred_domain' => _('Select preferred domains per profile'),
48	}	49	}
49	end	50	end
50		51
	@@ -42,6 +42,7 @@ class Person < Profile		@@ -42,6 +42,7 @@ class Person < Profile
42	end	42	end
43		43
44	FIELDS = %w[	44	FIELDS = %w[
		45	+ preferred_domain
45	nickname	46	nickname
46	sex	47	sex
47	address	48	address
	@@ -6,7 +6,7 @@		@@ -6,7 +6,7 @@
6	</p>	6	</p>
7		7
8	<p>	8	<p>
9	-<%= link_to_myprofile _('Edit Personal details.'), :controller => 'profile_editor' %>	9	+<%= link_to _('Edit Personal details.'), :controller => 'profile_editor', :profile => user.identifier, :action => 'index' %>
10	<%= _('You can change your personal details.') %>	10	<%= _('You can change your personal details.') %>
11	</p>	11	</p>
12		12
	@@ -16,7 +16,7 @@		@@ -16,7 +16,7 @@
16	</p>	16	</p>
17		17
18	<p>	18	<p>
19	-<%= link_to_myprofile(_('Manage content.'), :controller => 'cms') %>	19	+<%= link_to(_('Manage content.'), :controller => 'cms', :profile => user.identifier, :action => 'index') %>
20	<%= _('Manage your content.') %>	20	<%= _('Manage your content.') %>
21	</p>	21	</p>
22		22
	@@ -19,7 +19,7 @@		@@ -19,7 +19,7 @@
19	<li><%= link_to _('View profile'), block.owner.public_profile_url %></li>	19	<li><%= link_to _('View profile'), block.owner.public_profile_url %></li>
20	<li><%= link_to(_('Products/Services'), :controller => 'catalog', :profile => block.owner.identifier) if block.owner.enterprise? && !block.owner.environment.enabled?('disable_products_for_enterprises') %></li>	20	<li><%= link_to(_('Products/Services'), :controller => 'catalog', :profile => block.owner.identifier) if block.owner.enterprise? && !block.owner.environment.enabled?('disable_products_for_enterprises') %></li>
21	<% if !user.nil? and user.has_permission?('edit_profile', profile) %>	21	<% if !user.nil? and user.has_permission?('edit_profile', profile) %>
22	- <li><%= link_to _('Control panel'), :controller => 'profile_editor' %></li>	22	+ <li><%= link_to _('Control panel'), block.owner.admin_url %></li>
23	<% end %>	23	<% end %>
24	<% if profile.person? %>	24	<% if profile.person? %>
25	<li><%= _('Since %{year}/%{month}') % { :year => block.owner.created_at.year, :month => block.owner.created_at.month } %></li>	25	<li><%= _('Since %{year}/%{month}') % { :year => block.owner.created_at.year, :month => block.owner.created_at.month } %></li>
	@@ -7,7 +7,7 @@		@@ -7,7 +7,7 @@
7	<h1 id='article-full-path'>	7	<h1 id='article-full-path'>
8	<%= icon('cms') %>	8	<%= icon('cms') %>
9	<%= link_to profile.identifier, :action => 'index' %>	9	<%= link_to profile.identifier, :action => 'index' %>
10	- <%= @article.hierarchy.map {\|item\| " / " + ((item == @article) ? item.name : link_to_document(item)) } %>	10	+ <%= @article.hierarchy.map {\|item\| " / " + ((item == @article) ? item.name : link_to(item.slug, :id => item.id)) } %>
11	</h1>	11	</h1>
12	<% end %>	12	<% end %>
13		13
	@@ -42,7 +42,7 @@		@@ -42,7 +42,7 @@
42	</td>	42	</td>
43	<td>	43	<td>
44	<%= link_to _('Properties'), :action => 'edit', :id => folder.id %>	44	<%= link_to _('Properties'), :action => 'edit', :id => folder.id %>
45	- <%= link_to_document(folder, _('Public view')) %>	45	+ <%= link_to _('Public view'), folder.url %>
46	<%= link_to _('Use as homepage'), { :action => 'set_home_page', :id => folder.id }, :method => :post %>	46	<%= link_to _('Use as homepage'), { :action => 'set_home_page', :id => folder.id }, :method => :post %>
47	<%= link_to _('Delete'), { :action => 'destroy', :id => folder.id }, :method => :post, :confirm => _('Are you sure that you want to remove this folder? Note that all the items inside it will also be removed!') %>	47	<%= link_to _('Delete'), { :action => 'destroy', :id => folder.id }, :method => :post, :confirm => _('Are you sure that you want to remove this folder? Note that all the items inside it will also be removed!') %>
48	</td>	48	</td>
	@@ -68,7 +68,7 @@		@@ -68,7 +68,7 @@
68	</td>	68	</td>
69	<td>	69	<td>
70	<%= link_to _('Edit'), :action => 'edit', :id => item.id %>	70	<%= link_to _('Edit'), :action => 'edit', :id => item.id %>
71	- <%= link_to_document(item, _('Public view')) %>	71	+ <%= link_to _('Public view'), item.url %>
72	<%= link_to _('Spread this'), :action => 'publish', :id => item.id %>	72	<%= link_to _('Spread this'), :action => 'publish', :id => item.id %>
73	<%= link_to _('Use as homepage'), { :action => 'set_home_page', :id => item.id }, :method => :post %>	73	<%= link_to _('Use as homepage'), { :action => 'set_home_page', :id => item.id }, :method => :post %>
74	<%= link_to _('Delete'), { :action => 'destroy', :id => item.id }, :method => :post, :confirm => _('Are you sure that you want to remove this item?') %>	74	<%= link_to _('Delete'), { :action => 'destroy', :id => item.id }, :method => :post, :confirm => _('Are you sure that you want to remove this item?') %>
1	-<%= link_to_document(article, '', :class => article.css_class_name) %>
2	-<span><%= link_to_document(article) %></span>	1	+<%= link_to '', article.url, '', :class => article.css_class_name) %>
		2	+<span><%= link_to article.title, article.view_url %></span>
1	<% if uploaded_file.image? %>	1	<% if uploaded_file.image? %>
2	- <div> <%= link_to_document(uploaded_file, image_tag(uploaded_file.public_filename(:thumb))) %> </div>	2	+ <div> <%= link_to image_tag(uploaded_file.public_filename(:thumb)), uploaded_file.view_url %> </div>
3	<% else %>	3	<% else %>
4	<%= render :partial => 'article', :object => uploaded_file %>	4	<%= render :partial => 'article', :object => uploaded_file %>
5	<% end %>	5	<% end %>