fetching emails for invitation done in background

* Added message if user wants to fetch from hotmail * Added <url> to message if user removed it * Refactored invite_controller * Added ContactList * Added get_email_contacts_job * Added invitation_job (ActionItem1640)

fetching emails for invitation done in background
* Added message if user wants to fetch from hotmail * Added <url> to message if user removed it * Refactored invite_controller * Added ContactList * Added get_email_contacts_job * Added invitation_job (ActionItem1640)
Daniela Feitosa · Antonio Terceiro
1 parent f9cb2dd4
Showing 27 changed files with 566 additions and 223 deletions Show diff stats
app/controllers/public/invite_controller.rb
app/models/contact_list.rb
app/models/invitation.rb
app/views/friends/index.rhtml
app/views/invite/_contact_list.rhtml
app/views/invite/_dialog_wait_loading.rhtml
app/views/invite/friends.rhtml
app/views/invite/select_address_book.rhtml
app/views/invite/select_friends.rhtml
app/views/profile/friends.rhtml
app/views/profile/members.rhtml
app/views/profile_members/index.rhtml
config/initializers/delayed_job_config.rb
db/migrate/20100823190348_create_contact_lists.rb
db/schema.rb
features/invitation.feature
lib/get_email_contacts_job.rb
lib/invitation_job.rb
public/javascripts/application.js
public/stylesheets/application.css
@@ -2,42 +2,57 @@ class InviteController &lt; PublicController
  
   needs_profile
   before_filter :login_required
-  before_filter :check_permissions_to_invite, :only => 'friends'
+  before_filter :check_permissions_to_invite
  
-  def friends
-    step = params[:step]
+  def select_address_book
+    @import_from = params[:import_from] || "manual"
     if request.post?
-      if step == '1'
-        begin
-          @contacts = Invitation.get_contacts(params[:import_from], params[:login], params[:password])
-        rescue
-          @login = params[:login]
-          flash.now[:notice] = _('There was an error while looking for your contact list. Did you enter correct login and password?')
-        end
-      elsif step == '2'
-        manual_import_addresses = params[:manual_import_addresses]
-        webmail_import_addresses = params[:webmail_import_addresses]
-        contacts_to_invite = Invitation.join_contacts(manual_import_addresses, webmail_import_addresses)
-        if !params[:mail_template].match(/<url>/)
-          flash.now[:notice] = _('&lt;url&gt; is needed in invitation mail.')
-        elsif !contacts_to_invite.empty?
-          Delayed::Job.enqueue InvitationJob.new(user.id, contacts_to_invite, params[:mail_template], profile.id)
-          session[:notice] = _('Your invitations are being sent.')
-          if profile.person?
-            redirect_to :controller => 'friends'
-          else
-            redirect_to :controller => 'profile_members'
-          end
+      contact_list = ContactList.create
+      Delayed::Job.enqueue GetEmailContactsJob.new(@import_from, params[:login], params[:password], contact_list.id) if @import_from != 'manual'
+      redirect_to :action => 'select_friends', :contact_list => contact_list.id, :import_from => @import_from
+    end
+  end
+
+  def select_friends
+    @contact_list = ContactList.find(params[:contact_list])
+    @mail_template = params[:mail_template] || environment.invitation_mail_template(profile)
+    @import_from = params[:import_from] || "manual"
+    if request.post?
+      manual_import_addresses = params[:manual_import_addresses]
+      webmail_import_addresses = params[:webmail_import_addresses]
+      contacts_to_invite = Invitation.join_contacts(manual_import_addresses, webmail_import_addresses)
+      if !contacts_to_invite.empty?
+        Delayed::Job.enqueue InvitationJob.new(current_user.person.id, contacts_to_invite, params[:mail_template], profile.id, @contact_list.id)
+        session[:notice] = _('Your invitations are being sent.')
+        if profile.person?
+          redirect_to :controller => 'profile', :action => 'friends'
         else
-          flash.now[:notice] = _('Please enter a valid email address.')
+          redirect_to :controller => 'profile', :action => 'members'
         end
-        @contacts = params[:webmail_friends] ? params[:webmail_friends].map {|e| YAML.load(e)} : []
-        @manual_import_addresses = manual_import_addresses || ""
-        @webmail_import_addresses = webmail_import_addresses || []
+        return
+      else
+        session[:notice] = _('Please enter a valid email address.')
       end
+      @manual_import_addresses = manual_import_addresses || ""
+      @webmail_import_addresses = webmail_import_addresses || []
     end
-    @import_from = params[:import_from] || "manual"
-    @mail_template = params[:mail_template] || environment.invitation_mail_template(profile)
+  end
+
+  def invitation_data
+    contact_list = ContactList.find(params[:contact_list])
+    render :text => contact_list.data.to_json, :layout => false, :content_type => "application/javascript"
+  end
+
+  def add_contact_list
+    contact_list = ContactList.find(params[:contact_list])
+    contacts = contact_list.list
+    render :partial => 'invite/contact_list', :locals => {:contacts => contacts}
+  end
+
+  def cancel_fetching_emails
+    contact_list = ContactList.find(params[:contact_list])
+    contact_list.destroy
+    redirect_to :action => 'select_address_book'
   end
  
   protected
@@ -0,0 +1,27 @@
+class ContactList < ActiveRecord::Base
+
+  serialize :list, Array
+
+  def list
+    self[:list] || []
+  end
+
+  def data
+    if self.fetched
+      { "fetched" => true, "contact_list" => self.id, "error" => self.error_fetching }
+    else
+      {}
+    end
+  end
+
+  def register_auth_error
+    msg = _('There was an error while authenticating. Did you enter correct login and password?')
+    self.update_attributes(:fetched => true, :error_fetching => msg)
+  end
+
+  def register_error
+    msg = _('There was an error while looking for your contact list. Please, try again')
+    self.update_attributes(:fetched => true, :error_fetching => msg)
+  end
+
+end
@@ -11,7 +11,6 @@ class Invitation &lt; Task
   validates_format_of :friend_email, :with => Noosfero::Constants::EMAIL_FORMAT, :if => Proc.new{|invite| invite.target_id.blank?}
  
   validates_presence_of :message, :if => Proc.new{|invite| invite.target_id.blank?}
-  validates_format_of :message, :with => /<url>/, :if => Proc.new{|invite| invite.target_id.blank?}
  
   alias :person :requestor
   alias :person= :requestor=
@@ -19,6 +18,12 @@ class Invitation &lt; Task
   alias :friend :target
   alias :friend= :target=
  
+  before_create do |task|
+    if task.message && !task.message.match(/<url>/)
+      task.message += task.message_to_accept_invitation
+    end
+  end
+
   after_create do |task|
     TaskMailer.deliver_invitation_notification(task) unless task.friend
   end
@@ -72,21 +77,26 @@ class Invitation &lt; Task
     end
   end
  
-  def self.get_contacts(source, login, password)
-    contacts = []
+  def self.get_contacts(source, login, password, contact_list_id)
+    contact_list = ContactList.find(contact_list_id)
     case source
     when "gmail"
-      contacts = Contacts::Gmail.new(login, password).contacts
+      email_service = Contacts::Gmail.new(login, password)
     when "yahoo"
-      contacts = Contacts::Yahoo.new(login, password).contacts
+      email_service = Contacts::Yahoo.new(login, password)
     when "hotmail"
-      contacts = Contacts::Hotmail.new(login, password).contacts
+      email_service = Contacts::Hotmail.new(login, password)
     when "manual"
       #do nothing
     else
       raise NotImplementedError, 'Unknown source to get contacts'
     end
-    contacts.map { |contact| contact + ["#{contact[0]} <#{contact[1]}>"] }
+    if email_service
+      contact_list.list = email_service.contacts.map { |contact| contact + ["#{contact[0]} <#{contact[1]}>"] }
+      contact_list.fetched = true
+      contact_list.save
+    end
+    contact_list.list
   end
  
   def self.join_contacts(manual_import_addresses, webmail_import_addresses)
@@ -112,4 +122,7 @@ class Invitation &lt; Task
     raise 'You should implement mail_template in a subclass'
   end
  
+  def message_to_accept_invitation
+    '<p>' + _('To accept invitation, please follow this link: <url>') + '</p>'
+  end
 end
@@ -14,7 +14,7 @@
     <% button_bar do %>
       <%= button(:back, _('Back to control panel'), :controller => 'profile_editor') %>
       <%= button(:search, _('Find people'), :controller => 'search', :action => 'assets', :asset => 'people') %>
-      <%= button(:search, _('Invite people from my e-mail contacts'), :controller => 'invite', :action => 'friends') %>
+      <%= button(:search, _('Invite people from my e-mail contacts'), :controller => 'invite', :action => 'select_address_book') %>
     <% end %>
   <% end %>
  
@@ -40,11 +40,11 @@
     <%= pagination_links @friends, :param_name => 'npage' %>
   </div>
  
-<% button_bar do %>
-  <%= button(:back, _('Back to control panel'), :controller => 'profile_editor') %>
-  <%= button(:search, _('Find people'), :controller => 'search', :action => 'assets', :asset => 'people') %>
-  <%= button(:search, _('Invite people from my e-mail contacts'), :controller => 'invite', :action => 'friends') %>
-<% end %>
+  <% button_bar do %>
+    <%= button(:back, _('Back to control panel'), :controller => 'profile_editor') %>
+    <%= button(:search, _('Find people'), :controller => 'search', :action => 'assets', :asset => 'people') %>
+    <%= button(:search, _('Invite people from my e-mail contacts'), :controller => 'invite', :action => 'select_address_book') %>
+  <% end %>
 <% end %>
  
 </div><!-- end id="manage_friends" -->
@@ -0,0 +1,12 @@
+<% friend_pos = 0 %>
+<% contacts.each do |contact| %>
+  <% friend_pos += 1 %>
+  <p>
+    <%= check_box_tag("webmail_import_addresses[]", contact[2],
+          (!@webmail_import_addresses || @webmail_import_addresses.include?(contact[2])),
+          :id => "contacts_to_invite_#{friend_pos}",
+          :class => "contact_to_invite" )
+    %>
+    <label for="<%= "contacts_to_invite_#{friend_pos}" %>"><%= "#{contact[0]} (#{contact[1]})" %></label>
+  </p>
+<% end %>
@@ -0,0 +1,23 @@
+<% javascript_tag do %>
+  jQuery(function($) {
+    $("#loading-dialog").dialog({
+      height: 160,
+      width: 450,
+      modal: true,
+      resizable: false,
+      title: "<%= ui_icon('ui-icon-info') + _('Please, wait...') %>",
+      open: check_contact_list('<%= contact_list.to_s %>'),
+     });
+   });
+<% end %>
+
+
+<div id="loading-dialog">
+  <p><%= _('Your e-mails contacts are being fetched') %></p>
+  <p><%=  _('If it takes too long, you will be redirected to the previous page to try again. Be sure to fill in the fields with your correct login and password.') %>
+  <span class='loading-message'/>
+  <%= link_to(_('Verify contact list'), {:action => 'invitation_data', :contact_list => @contact_list}, :id => "verify-contact-list", :style => 'display:none') %>
+  <%= link_to(_('Add contact list'), {:action => 'add_contact_list', :contact_list => @contact_list}, :id => "add-contact-list", :style => 'display:none') %>
+  <%= link_to(_('Cancel fetching e-mails'), {:action => 'cancel_fetching_emails', :contact_list => @contact_list}, :id => "cancel-fetching-emails", :style => 'display:none') %>
+</div>
+
@@ -1,93 +0,0 @@
-<% if profile.person? %>
-  <h1><%= _('Invite your friends') %></h1>
-<% else %>
-  <h1><%= _('Invite your friends to join %s') % profile.name %></h1>
-<% end %>
-
-<% unless @contacts %>
-
-  <h2><%= _('Step 1 of 2: Select address book') %></h2>
-
-  <% form_tag do %>
-    <%= hidden_field_tag(:step, 1) %>
-
-    <%= [
-          radio_button_tag(:import_from, "manual", @import_from == "manual", :onclick => 'hide_invite_friend_login_password()') + content_tag('label', _('Manually (empty field)'), :for => "import_from_manual"),
-          radio_button_tag(:import_from, "gmail", @import_from == "gmail", :onclick => 'show_invite_friend_login_password()') + content_tag('label', 'Gmail', :for => 'import_from_gmail'),
-          radio_button_tag(:import_from, "yahoo", @import_from == "yahoo", :onclick => 'show_invite_friend_login_password()') + content_tag('label', 'Yahoo', :for => "import_from_yahoo"),
-          radio_button_tag(:import_from, "hotmail", @import_from == "hotmail", :onclick => 'show_invite_friend_login_password()') + content_tag('label', 'Hotmail', :for => "import_from_hotmail")
-        ].join("\n<br/>\n") %>
-
-    <script type="text/javascript">
-      function hide_invite_friend_login_password() {
-        $('invite-friends-login-password').hide();
-      }
-      function show_invite_friend_login_password() {
-        $('invite-friends-login-password').show();
-        $('login').focus();
-      }
-    </script>
-    <div id='invite-friends-login-password' <%= "style='display: none;'" if (@import_from == 'manual') %>>
-    <%= labelled_form_field(_("Username") + ":", text_field_tag(:login, @login)) %>
-    <%= labelled_form_field(_("Password") + ":", password_field_tag(:password)) %>
-    </div>
-
-    <% button_bar do %>
-      <%= submit_button(:forward, _("Next")) %>
-    <% end %>
-    <p><%= _("We won't store your password or contact anyone without your permission.") %></p>
-  <% end %>
-
-<% else %>
-
-  <h2><%= _('Step 2 of 2: Selecting Friends') %></h2>
-  <p>
-  <%= _('Indicate which friends you want to invite.') %>
-  </p>
-
-  <% form_tag do %>
-    <%= hidden_field_tag(:step, 2) %>
-    <%= hidden_field_tag(:import_from, @import_from) %>
-
-    <div>
-    <%= labelled_form_field(_('Enter one e-mail address per line:'), text_area_tag(:manual_import_addresses, (@manual_import_addresses || ''), :cols => 72, :rows => 5)) %>
-    </div>
-    <% if @import_from != 'manual' %>
-      <div>
-        <%= link_to_function _('Check all'), "$$('input.contact_to_invite').each(function(checkbox) { checkbox.checked = true; });" %>
-        <%= link_to_function _('Uncheck all'), "$$('input.contact_to_invite').each(function(checkbox) { checkbox.checked = false; });" %>
-        <% friend_pos = 0 %>
-        <div id='contacts-list'>
-          <% @contacts.each do |contact| %>
-            <% friend_pos += 1 %>
-            <p>
-              <%= hidden_field_tag("webmail_friends[]", contact.to_yaml) %>
-              <%= check_box_tag("webmail_import_addresses[]", contact[2],
-                    (!@webmail_import_addresses || @webmail_import_addresses.include?(contact[2])),
-                    :id => "contacts_to_invite_#{friend_pos}",
-                    :class => "contact_to_invite" )
-              %>
-              <label for="<%= "contacts_to_invite_#{friend_pos}" %>"><%= "#{contact[0]} (#{contact[1]})" %></label>
-            </p>
-          <% end %>
-        </div>
-      </div>
-    <% end -%>
-
-    <br/>
-
-    <%= link_to_function(_('Personalize invitation mail'), nil) do |page|
-      page['invitation-mail_template'].show
-    end %>
-
-    <div id='invitation-mail_template' style='display:none'>
-      <%= h _("Now enter an invitation message. You must keep the <url> code in your invitation text. When your friends receive the invitation e-mail, <url> will be replaced by a link that they need to click to activate their account. <user> and <friend> codes will be replaced by your name and friend name, but they are optional.") %>
-      <%= labelled_form_field(_('Invitation text:'), text_area_tag(:mail_template, @mail_template, :cols => 72, :rows => 8)) %>
-    </div>
-
-    <% button_bar do %>
-      <%= submit_button(:ok, _("Invite my friends!")) %>
-    <% end %>
-  <% end %>
-
-<% end %>
@@ -0,0 +1,51 @@
+<% if profile.person? %>
+  <h1><%= _('Invite your friends') %></h1>
+<% else %>
+  <h1><%= _('Invite your friends to join %s') % profile.name %></h1>
+<% end %>
+
+<h2><%= _('Step 1 of 2: Select address book') %></h2>
+
+<% form_tag do %>
+
+  <%= [
+        radio_button_tag(:import_from, "manual", @import_from == "manual", :onclick => 'hide_invite_friend_login_password()') + content_tag('label', _('Manually (empty field)'), :for => "import_from_manual"),
+        radio_button_tag(:import_from, "gmail", @import_from == "gmail", :onclick => 'show_invite_friend_login_password(this.value)') + content_tag('label', 'Gmail', :for => 'import_from_gmail'),
+        radio_button_tag(:import_from, "yahoo", @import_from == "yahoo", :onclick => 'show_invite_friend_login_password(this.value)') + content_tag('label', 'Yahoo', :for => "import_from_yahoo"),
+        radio_button_tag(:import_from, "hotmail", @import_from == "hotmail", :onclick => 'show_invite_friend_login_password(this.value)') + content_tag('label', 'Hotmail', :for => "import_from_hotmail")
+      ].join("\n<br/>\n") %>
+
+  <script type="text/javascript">
+    function hide_invite_friend_login_password() {
+      $('invite-friends-login-password').hide();
+    }
+    function show_invite_friend_login_password(option) {
+      if (option == 'hotmail') {
+        $('hotmail_username_tip').show();
+      } else {
+        $('hotmail_username_tip').hide();
+      }
+      $('invite-friends-login-password').show();
+      $('login').focus();
+    }
+  </script>
+  <div id='invite-friends-login-password' <%= "style='display: none;'" if (@import_from == 'manual') %>>
+    <div id='hotmail_username_tip'>
+      <%= ui_icon('ui-icon-alert') %>
+      <%= _('Please type your username in the format yourname@example.com') %>
+    </div>
+
+    <%= labelled_form_field(_("Username") + ":", text_field_tag(:login, @login)) %>
+    <%= labelled_form_field(_("Password") + ":", password_field_tag(:password)) %>
+  </div>
+
+  <% button_bar do %>
+    <%= submit_button(:forward, _("Next")) %>
+  <% end %>
+  <p><%= _("We won't store your password or contact anyone without your permission.") %></p>
+<% end %>
+
+<div id="loadingScreen"></div>
+
+
+
@@ -0,0 +1,47 @@
+<%= render :partial => 'invite/dialog_wait_loading', :locals => {:contact_list => @contact_list.id } if @import_from != 'manual' %>
+
+<% if profile.person? %>
+  <h1><%= _('Invite your friends') %></h1>
+<% else %>
+  <h1><%= _('Invite your friends to join %s') % profile.name %></h1>
+<% end %>
+
+
+<h2><%= _('Step 2 of 2: Selecting Friends') %></h2>
+
+<%= button(:back, _('Back'), { :action => 'select_address_book' }, :id => 'invitation_back_button') %>
+
+<p>
+<%= _('Indicate which friends you want to invite.') %>
+</p>
+
+<% form_tag do %>
+  <%= hidden_field_tag(:import_from, @import_from) %>
+  <%= hidden_field_tag(:contact_list, @contact_list.id) %>
+
+  <div>
+  <%= labelled_form_field(_('Enter one e-mail address per line:'), text_area_tag(:manual_import_addresses, (@manual_import_addresses || ''), :cols => 72, :rows => 5)) %>
+  </div>
+  <% if @import_from != 'manual' %>
+    <div>
+      <%= link_to_function _('Check all'), "$$('input.contact_to_invite').each(function(checkbox) { checkbox.checked = true; });" %>
+      <%= link_to_function _('Uncheck all'), "$$('input.contact_to_invite').each(function(checkbox) { checkbox.checked = false; });" %>
+      <div id='contacts-list'></div>
+    </div>
+  <% end -%>
+
+  <br/>
+
+  <%= link_to_function(_('Personalize invitation mail'), nil) do |page|
+    page['invitation-mail_template'].show
+  end %>
+
+  <div id='invitation-mail_template' style='display:none'>
+    <%= h _("Now enter an invitation message. You must keep the <url> code in your invitation text. When your friends receive the invitation e-mail, <url> will be replaced by a link that they need to click to activate their account. <user> and <friend> codes will be replaced by your name and friend name, but they are optional.") %>
+    <%= labelled_form_field(_('Invitation text:'), text_area_tag(:mail_template, @mail_template, :cols => 72, :rows => 8)) %>
+  </div>
+
+  <% button_bar do %>
+    <%= submit_button(:ok, _("Invite my friends!")) %>
+  <% end %>
+<% end %>
@@ -18,7 +18,7 @@
   <%= button :back, _('Go back'), { :controller => 'profile' } %>
   <% if user == profile %>
     <%= button :edit, _('Manage my friends'), :controller => 'friends', :action => 'index', :profile => profile.identifier %>
-    <%= button(:search, _('Invite people from my e-mail contacts'), :controller => 'invite', :action => 'friends') %>
+    <%= button(:search, _('Invite people from my e-mail contacts'), :controller => 'invite', :action => 'select_address_book') %>
   <% end %>
 <% end %>
  
@@ -17,7 +17,7 @@
 <% button_bar do %>
   <%= button :back, _('Go back'), { :controller => 'profile' } %>
   <% if profile.community? and user and user.has_permission?(:invite_members, profile) %>
-    <%= button :search, _('Invite your friends to join %s') % profile.name, :controller => 'invite', :action => 'friends' %>
+    <%= button :search, _('Invite your friends to join %s') % profile.name, :controller => 'invite', :action => 'selects_address_book' %>
   <% end %>
 <% end %>
  
@@ -6,6 +6,6 @@
   <%= button :back, _('Back'), :controller => 'profile_editor' %>
   <%= button :add, _('Add members'), :action => 'add_members' if profile.enterprise? %>
   <% if profile.community? and user.has_permission?(:invite_members, profile) %>
-    <%= button :search, _('Invite your friends to join %s') % profile.name, :controller => 'invite', :action => 'friends' %>
+    <%= button :search, _('Invite your friends to join %s') % profile.name, :controller => 'invite', :action => 'select_address_book' %>
   <% end %>
 <% end %>
@@ -0,0 +1,2 @@
+Delayed::Worker.max_attempts = 2
+Delayed::Worker.max_run_time = 10.minutes
@@ -0,0 +1,14 @@
+class CreateContactLists < ActiveRecord::Migration
+  def self.up
+    create_table :contact_lists do |t|
+      t.text :list
+      t.string :error_fetching
+      t.boolean :fetched, :default => false
+      t.timestamps
+    end
+  end
+
+  def self.down
+    drop_table :contact_lists
+  end
+end
@@ -9,7 +9,7 @@
 #
 # It's strongly recommended to check this file into your version control system.
  
-ActiveRecord::Schema.define(:version => 20100822034415) do
+ActiveRecord::Schema.define(:version => 20100823190348) do
  
   create_table "article_versions", :force => true do |t|
     t.integer  "article_id"
@@ -145,9 +145,9 @@ ActiveRecord::Schema.define(:version =&gt; 20100822034415) do
     t.string   "name",           :null => false
     t.string   "description"
     t.string   "link"
+    t.integer  "environment_id"
     t.datetime "created_at"
     t.datetime "updated_at"
-    t.integer  "environment_id"
   end
  
   create_table "comments", :force => true do |t|
@@ -160,6 +160,14 @@ ActiveRecord::Schema.define(:version =&gt; 20100822034415) do
     t.datetime "created_at"
   end
  
+  create_table "contact_lists", :force => true do |t|
+    t.text     "list"
+    t.string   "error_fetching"
+    t.boolean  "fetched",        :default => false
+    t.datetime "created_at"
+    t.datetime "updated_at"
+  end
+
   create_table "delayed_jobs", :force => true do |t|
     t.integer  "priority",   :default => 0
     t.integer  "attempts",   :default => 0
@@ -351,9 +359,9 @@ ActiveRecord::Schema.define(:version =&gt; 20100822034415) do
  
   create_table "roles", :force => true do |t|
     t.string  "name"
+    t.text    "permissions"
     t.string  "key"
     t.boolean "system",         :default => false
-    t.text    "permissions"
     t.integer "environment_id"
   end
  
@@ -32,15 +32,14 @@ Feature: invitation
     When I am on /profile/josesilva/invite/friends
     Then I should see "Invite your friends"
  
-  # why not work?
-  Scenario: back to manage friends after invite friends
+  Scenario: back to friends after invite friends
     Given I am on /myprofile/josesilva/friends
     And I follow "Invite people from my e-mail contacts"
     And I press "Next"
     And I fill in "manual_import_addresses" with "misfits@devil.doll"
     And I fill in "mail_template" with "Follow this link <url>"
     When I press "Invite my friends!"
-    Then I should be on /myprofile/josesilva/friends
+    Then I should be on /profile/josesilva/friends
  
   Scenario: see link to invite members to community
     When I am on /profile/26-bsslines/members
@@ -75,14 +74,14 @@ Feature: invitation
     Given I am on Beatles For Sale's members management
     Then I should not see "Invite your friends to join Beatles For Sale" link
  
-  Scenario: back to manage members after invite friends
+  Scenario: back to members after invite friends to join a community
     Given I am on 26 Bsslines's members management
     And I follow "Invite your friends to join 26 Bsslines"
     And I press "Next"
     And I fill in "manual_import_addresses" with "misfits@devil.doll"
     And I fill in "mail_template" with "Follow this link <url>"
     When I press "Invite my friends!"
-    Then I should be on /myprofile/26-bsslines/profile_members
+    Then I should be on /profile/26-bsslines/members
  
   Scenario: noosfero user receives a task when a user invites to join a community
     Given I am on 26 Bsslines's members management
@@ -0,0 +1,11 @@
+class GetEmailContactsJob < Struct.new(:import_from, :login, :password, :contact_list_id)
+  def perform
+    begin
+      Invitation.get_contacts(import_from, login, password, contact_list_id)
+    rescue Contacts::AuthenticationError => ex
+      ContactList.find(contact_list_id).register_auth_error
+    rescue Exception => ex
+      ContactList.find(contact_list_id).register_error
+    end
+  end
+end
-class InvitationJob < Struct.new(:person_id, :contacts_to_invite, :message, :profile_id)
+class InvitationJob < Struct.new(:person_id, :contacts_to_invite, :message, :profile_id, :contact_list_id)
   def perform
     begin
-      person = Person.find(person_id)
-      profile = Profile.find(profile_id)
-      Invitation.invite(person, contacts_to_invite, message, profile)
+     person = Person.find(person_id)
+     profile = Profile.find(profile_id)
+     Invitation.invite(person, contacts_to_invite, message, profile)
+     ContactList.find(contact_list_id).destroy
     rescue ActiveRecord::NotFound => e
-      # ...
+      #...
     end
   end
 end
@@ -455,9 +455,46 @@ jQuery(function($) {
       $('#user .not-logged-in, .login-block .not-logged-user').fadeIn();
     }
     if (data.notice) {
-      var $noticeBox = $('<div id="notice"></div>').html(data.notice).appendTo('body').fadeTo('fast', 0.8);
-      $noticeBox.click(function() { $(this).hide(); });
-      setTimeout(function() { $noticeBox.fadeOut('fast'); }, 5000);
+      display_notice(data.notice);
     }
   });
 });
+
+// controls the display of contact list
+function check_contact_list(contact_list) {
+  jQuery(function($) {
+    var verify_url = $('#verify-contact-list').attr('href');
+    var add_contacts_url = $('#add-contact-list').attr('href');
+    var cancel_contacts_fetching_url = $('#cancel-fetching-emails').attr('href');
+    var interval = setInterval(function() {
+      $.getJSON(verify_url, function(data) {
+        if (data.fetched) {
+          clearInterval(interval);
+          if (data.error) {
+            $("#loading-dialog").dialog('close');
+            $.get(cancel_contacts_fetching_url);
+            redirect_to($('#invitation_back_button').attr('href'));
+            display_notice(data.error);
+          } else {
+            $.get(add_contacts_url, function(data){
+              $("#contacts-list").html(data);
+            });
+          };
+          $("#loading-dialog").dialog('close');
+        }
+      });
+    }, 5000);
+    setTimeout(function() {
+      clearInterval(interval);
+      $("#loading-dialog").dialog('close');
+      $.get(cancel_contacts_fetching_url);
+      redirect_to($('#invitation_back_button').attr('href'));
+    }, 600000);
+  });
+}
+
+function display_notice(message) {
+   var $noticeBox = jQuery('<div id="notice"></div>').html(message).appendTo('body').fadeTo('fast', 0.8);
+   $noticeBox.click(function() { $(this).hide(); });
+   setTimeout(function() { $noticeBox.fadeOut('fast'); }, 5000);
+}
@@ -3189,6 +3189,26 @@ h1#agenda-title {
   margin: 5px 0px;
 }
  
+#loading-dialog .loading-message {
+  height: 32px;
+  width: 32px;
+  left: 209px;
+  bottom: 5px;
+  position: absolute;
+  display: block;
+  background: transparent url(/images/loading-dark.gif) center center no-repeat;
+}
+
+/* hide the close x on the loading screen */
+.controller-invite .ui-dialog-titlebar-close {
+  display: none;
+}
+
+#hotmail_username_tip {
+  color: red;
+  margin-top: 10px;
+}
+
 /* ==> public/stylesheets/controller_manage_products.css <== */
  
 #category_form {
@@ -6,28 +6,9 @@
 # The role of this script is to just start/stop the daemon, write a PID file,
 # etc. The actual feed update logic is DelayedJob plugin.
  
+require File.expand_path(File.join(File.dirname(__FILE__), '..', 'config', 'environment'))
 require 'daemons'
+require 'delayed/command'
  
-NOOSFERO_ROOT = File.expand_path(File.dirname(__FILE__) + '/../')
-
-options = {
-  :dir_mode  => :normal,
-  :dir       => File.dirname(__FILE__) + '/../tmp/pids',
-  :multiple  => false,
-  :backtrace => true,
-  :monitor   => true,
-}
-
-Daemons.run_proc('job_runner', options) do
-  if ARGV.include?('--')
-    ARGV.slice! 0..ARGV.index('--')
-  else
-    ARGV.clear
-  end
-
-  Dir.chdir NOOSFERO_ROOT
-  RAILS_ENV = ARGV.first || ENV['RAILS_ENV'] || 'development'
-  require NOOSFERO_ROOT + '/config/environment'
-
-  Delayed::Worker.new.start
-end
+ENV['RAILS_ENV'] ||= "production"
+Delayed::Command.new(ARGV).daemonize
@@ -20,14 +20,14 @@ do_start() {
   clear_cache
   ./script/ferret_server -e $RAILS_ENV start
   ./script/feed-updater start
-  ./script/delayed_job start
+#  ./script/delayed_job start -n 4
   mongrel_rails cluster::start
 }
  
 do_stop() {
   mongrel_rails cluster::stop
   ./script/delayed_job stop
-  ./script/feed-updater stop
+#  ./script/feed-updater stop
   ./script/ferret_server -e $RAILS_ENV stop
 }
  
@@ -11,9 +11,10 @@ class InviteControllerTest &lt; ActionController::TestCase
   attr_accessor :profile, :friend, :community
  
   should 'add manually invitation of an added address with friend object on a queue and process it later' do
+    contact_list = ContactList.create
     assert_difference Delayed::Job, :count, 1 do
-      post :friends, :profile => profile.identifier, :manual_import_addresses => "#{friend.name} <#{friend.email}>", :import_from => "manual", :mail_template => "click: <url>", :step => 2
-      assert_redirected_to :controller => 'friends'
+      post :select_friends, :profile => profile.identifier, :manual_import_addresses => "#{friend.name} <#{friend.email}>", :import_from => "manual", :mail_template => "click: <url>", :contact_list => contact_list.id
+      assert_redirected_to :controller => 'profile', :action => 'friends'
     end
  
     assert_difference InviteFriend, :count, 1 do
@@ -22,9 +23,10 @@ class InviteControllerTest &lt; ActionController::TestCase
   end
  
   should 'add manually invitation of an added address with only email on a queue and process it later' do
+    contact_list = ContactList.create
     assert_difference Delayed::Job, :count, 1 do
-      post :friends, :profile => profile.identifier, :manual_import_addresses => "test@test.com", :import_from => "manual", :mail_template => "click: <url>", :step => 2
-      assert_redirected_to :controller => 'friends'
+      post :select_friends, :profile => profile.identifier, :manual_import_addresses => "test@test.com", :import_from => "manual", :mail_template => "click: <url>", :contact_list => contact_list.id
+      assert_redirected_to :controller => 'profile', :action => 'friends'
     end
  
     assert_difference InviteFriend, :count, 1 do
@@ -33,9 +35,10 @@ class InviteControllerTest &lt; ActionController::TestCase
   end
  
   should 'add manually invitation of an added address with email and other format on a queue and process it later' do
+    contact_list = ContactList.create
     assert_difference Delayed::Job, :count, 1 do
-      post :friends, :profile => profile.identifier, :manual_import_addresses => "test@test.cz.com", :import_from => "manual", :mail_template => "click: <url>", :step => 2
-      assert_redirected_to :controller => 'friends'
+      post :select_friends, :profile => profile.identifier, :manual_import_addresses => "test@test.cz.com", :import_from => "manual", :mail_template => "click: <url>", :contact_list => contact_list.id
+      assert_redirected_to :controller => 'profile', :action => 'friends'
     end
  
     assert_difference InviteFriend, :count, 1 do
@@ -44,9 +47,10 @@ class InviteControllerTest &lt; ActionController::TestCase
   end
  
   should 'add manually invitation of more than one added address on a queue and process it later' do
+    contact_list = ContactList.create
     assert_difference Delayed::Job, :count, 1 do
-      post :friends, :profile => profile.identifier, :manual_import_addresses => "Some Friend <somefriend@email.com>\r\notherperson@bleble.net\r\n", :import_from => "manual", :mail_template => "click: <url>", :step => 2
-      assert_redirected_to :controller => 'friends'
+      post :select_friends, :profile => profile.identifier, :manual_import_addresses => "Some Friend <somefriend@email.com>\r\notherperson@bleble.net\r\n", :import_from => "manual", :mail_template => "click: <url>", :contact_list => contact_list.id
+      assert_redirected_to :controller => 'profile', :action => 'friends'
     end
  
     assert_difference InviteFriend, :count, 2 do
@@ -55,9 +59,10 @@ class InviteControllerTest &lt; ActionController::TestCase
   end
  
   should 'add manually invitation of an added address with name and e-mail on a queue and process it later' do
+    contact_list = ContactList.create
     assert_difference Delayed::Job, :count, 1 do
-      post :friends, :profile => profile.identifier, :manual_import_addresses => "Test Name <test@test.com>", :import_from => "manual", :mail_template => "click: <url>", :step => 2
-      assert_redirected_to :controller => 'friends'
+      post :select_friends, :profile => profile.identifier, :manual_import_addresses => "Test Name <test@test.com>", :import_from => "manual", :mail_template => "click: <url>", :contact_list => contact_list.id
+      assert_redirected_to :controller => 'profile', :action => 'friends'
     end
  
     assert_difference InviteFriend, :count, 1 do
@@ -66,9 +71,10 @@ class InviteControllerTest &lt; ActionController::TestCase
   end
  
   should 'add invitation of yourself on a queue and not process it later' do
+    contact_list = ContactList.create
     assert_difference Delayed::Job, :count, 1 do
-      post :friends, :profile => profile.identifier, :manual_import_addresses => "#{profile.name} <#{profile.user.email}>", :import_from => "manual", :mail_template => "click: <url>", :step => 2
-      assert_redirected_to :controller => 'friends'
+      post :select_friends, :profile => profile.identifier, :manual_import_addresses => "#{profile.name} <#{profile.user.email}>", :import_from => "manual", :mail_template => "click: <url>", :contact_list => contact_list.id
+      assert_redirected_to :controller => 'profile', :action => 'friends'
     end
  
     assert_no_difference InviteFriend, :count do
@@ -80,9 +86,10 @@ class InviteControllerTest &lt; ActionController::TestCase
     friend = create_user('testfriend', :email => 'friend@noosfero.org')
     friend.person.add_friend(profile)
  
+    contact_list = ContactList.create
     assert_difference Delayed::Job, :count, 1 do
-      post :friends, :profile => profile.identifier, :manual_import_addresses => "#{friend.name} <#{friend.email}>", :import_from => "manual", :mail_template => "click: <url>", :step => 2
-      assert_redirected_to :controller => 'friends'
+      post :select_friends, :profile => profile.identifier, :manual_import_addresses => "#{friend.name} <#{friend.email}>", :import_from => "manual", :mail_template => "click: <url>", :contact_list => contact_list.id
+      assert_redirected_to :controller => 'profile', :action => 'friends'
     end
  
     assert_no_difference InviteFriend, :count do
@@ -91,42 +98,131 @@ class InviteControllerTest &lt; ActionController::TestCase
   end
  
   should 'display invitation page' do
-    get :friends, :profile => profile.identifier
+    get :select_address_book, :profile => profile.identifier
     assert_response :success
     assert_tag :tag => 'h1', :content => 'Invite your friends'
   end
  
   should 'get mail template to invite members' do
     community.add_admin(profile)
-    get :friends, :profile => community.identifier
+    contact_list = ContactList.create
+    get :select_friends, :profile => community.identifier, :contact_list => contact_list.id
     assert_equal InviteMember.mail_template, assigns(:mail_template)
   end
  
   should 'get mail template to invite friends' do
     community.add_admin(profile)
-    get :friends, :profile => profile.identifier
+    contact_list = ContactList.create
+    get :select_friends, :profile => profile.identifier, :contact_list => contact_list.id
     assert_equal InviteFriend.mail_template, assigns(:mail_template)
   end
  
-  should 'deny if user has no rights to invite members' do
-    get :friends, :profile => community.identifier
+  should 'deny select_address_book f user has no rights to invite members' do
+    get :select_address_book, :profile => community.identifier
     assert_response 403 # forbidden
   end
  
-  should 'deny access when trying to invite friends to another user' do
-    get :friends, :profile => friend.identifier
+  should 'deny select_friends if user has no rights to invite members' do
+    get :select_friends, :profile => community.identifier
     assert_response 403 # forbidden
   end
  
-  should 'redirect to friends after invitation if profile is a person' do
-    post :friends, :profile => profile.identifier, :manual_import_addresses => "#{friend.name} <#{friend.email}>", :import_from => "manual", :mail_template => "click: <url>", :step => 2
-    assert_redirected_to :controller => 'friends'
+  should 'deny select_address_book access when trying to invite friends to another user' do
+    get :select_address_book, :profile => friend.identifier
+    assert_response 403 # forbidden
+  end
+
+  should 'deny select_friends access when trying to invite friends to another user' do
+    get :select_address_book, :profile => friend.identifier
+    assert_response 403 # forbidden
+  end
+
+  should 'redirect to profile after invitation if profile is a person' do
+    contact_list = ContactList.create
+    post :select_friends, :profile => profile.identifier, :manual_import_addresses => "#{friend.name} <#{friend.email}>", :import_from => "manual", :mail_template => "click: <url>", :contact_list => contact_list.id
+    assert_redirected_to :controller => 'profile', :action => 'friends'
   end
  
-  should 'redirect to friends after invitation if profile is not a person' do
+  should 'redirect to profile after invitation if profile is not a person' do
     community.add_admin(profile)
-    post :friends, :profile => community.identifier, :manual_import_addresses => "#{friend.name} <#{friend.email}>", :import_from => "manual", :mail_template => "click: <url>", :step => 2
-    assert_redirected_to :controller => 'profile_members'
+    contact_list = ContactList.create
+    post :select_friends, :profile => community.identifier, :manual_import_addresses => "#{friend.name} <#{friend.email}>", :import_from => "manual", :mail_template => "click: <url>", :contact_list => contact_list.id
+    assert_redirected_to :controller => 'profile', :action => 'members'
+  end
+
+  should 'create a job to get emails after choose address book' do
+    community.add_admin(profile)
+    contact_list = ContactList.create
+    assert_difference Delayed::Job, :count, 1 do
+      post :select_address_book, :profile => community.identifier, :contact_list => contact_list.id, :import_from => 'gmail'
+      assert_redirected_to :action => 'select_friends'
+    end
+  end
+
+  should 'destroy contact_list after invitation when import is manual' do
+    contact_list = ContactList.create
+    post :select_friends, :profile => profile.identifier, :manual_import_addresses => "#{friend.name} <#{friend.email}>", :import_from => "manual", :mail_template => "click: <url>", :contact_list => contact_list.id
+
+    assert ContactList.exists?(contact_list.id)
+    Delayed::Worker.new.work_off
+    assert !ContactList.exists?(contact_list.id)
+  end
+
+  should 'destroy contact_list after invitation when import is not manual' do
+    contact_list = ContactList.create
+    post :select_friends, :profile => profile.identifier, :manual_import_addresses => "#{friend.name} <#{friend.email}>", :import_from => "not_manual", :mail_template => "click: <url>", :contact_list => contact_list.id
+
+    assert ContactList.exists?(contact_list.id)
+    Delayed::Worker.new.work_off
+    assert !ContactList.exists?(contact_list.id)
+  end
+
+  should 'return empty hash as invitation data if contact list was not fetched' do
+    contact_list = ContactList.create
+    get :invitation_data, :profile => profile.identifier, :contact_list => contact_list.id
+
+    assert_equal 'application/javascript', @response.content_type
+    assert_equal '{}', @response.body
+  end
+
+  should 'return hash as invitation data if contact list was fetched' do
+    contact_list = ContactList.create(:fetched => true)
+    get :invitation_data, :profile => profile.identifier, :contact_list => contact_list.id
+
+    assert_equal 'application/javascript', @response.content_type
+    assert_equal "{\"fetched\": true, \"contact_list\": #{contact_list.id}, \"error\": null}", @response.body
+  end
+
+  should 'render empty list of contacts' do
+    contact_list = ContactList.create(:fetched => true)
+    get :add_contact_list, :profile => profile.identifier, :contact_list => contact_list.id
+
+    assert_response :success
+    assert_template '_contact_list'
+    assert_no_tag(:tag => 'input', :attributes => { :type => 'checkbox', :name => 'webmail_import_addresses[]'})
+  end
+
+  should 'render list of contacts' do
+    contact_list = ContactList.create(:fetched => true, :list => ['email1@noosfero.org', 'email2@noosfero.org'])
+    get :add_contact_list, :profile => profile.identifier, :contact_list => contact_list.id
+
+    assert_response :success
+    assert_template '_contact_list'
+
+    i = 0
+    contact_list.list.each do |contact|
+      i += 1
+      assert_tag(:tag => 'input', :attributes => { :type => 'checkbox', :name => 'webmail_import_addresses[]', :id => "contacts_to_invite_#{i}", :value => contact[2]})
+    end
+  end
+
+  should 'destroy contact_list when cancel_fetching_emails' do
+    contact_list = ContactList.create
+
+    assert_difference ContactList, :count, -1 do
+      get :cancel_fetching_emails, :profile => profile.identifier, :contact_list => contact_list.id
+    end
+    assert_redirected_to :action => 'select_address_book'
   end
  
 end
@@ -0,0 +1,41 @@
+require File.dirname(__FILE__) + '/../test_helper'
+
+class ContactListTest < ActiveSupport::TestCase
+
+  should 'have list as an array' do
+    assert_equal [], ContactList.create.list
+  end
+
+  should 'display list' do
+    contact_list = ContactList.create(:list => ['email1@noosfero.org', 'email2@noosfero.org'])
+
+    assert_equal ['email1@noosfero.org', 'email2@noosfero.org'], contact_list.list
+  end
+
+  should 'return empty hash if contact list was not fetched' do
+    contact_list = ContactList.create
+    assert_equal({}, contact_list.data)
+  end
+
+  should 'return hash if contact list was fetched' do
+    contact_list = ContactList.create(:fetched => true)
+    assert_equal({"fetched" => true, "contact_list" => contact_list.id, "error" => contact_list.error_fetching}, contact_list.data)
+  end
+
+  should 'update fetched and error_fetching when register auth error' do
+    contact_list = ContactList.create
+    assert_equal({}, contact_list.data)
+
+    contact_list.register_error
+    assert_equal({"fetched" => true, "contact_list" => contact_list.id, "error" => 'There was an error while looking for your contact list. Please, try again'}, contact_list.data)
+  end
+
+  should 'update fetched and error_fetching when register error' do
+    contact_list = ContactList.create
+    assert_equal({}, contact_list.data)
+
+    contact_list.register_auth_error
+    assert_equal({"fetched" => true, "contact_list" => contact_list.id, "error" => 'There was an error while authenticating. Did you enter correct login and password?'}, contact_list.data)
+  end
+
+end
@@ -0,0 +1,27 @@
+require File.dirname(__FILE__) + '/../test_helper'
+
+class GetEmailContactsJobTest < ActiveSupport::TestCase
+
+  should 'register error' do
+    contact_list = ContactList.create
+    Invitation.expects(:get_contacts).with('from-email', 'mylogin', 'mypassword', contact_list.id).raises(Exception.new("crash"))
+
+    job = GetEmailContactsJob.new('from-email', 'mylogin', 'mypassword', contact_list.id)
+    job.perform
+
+    assert ContactList.find(contact_list).fetched
+    assert_equal 'There was an error while looking for your contact list. Please, try again', ContactList.find(contact_list).error_fetching
+  end
+
+  should 'register auth error' do
+    contact_list = ContactList.create
+    Invitation.expects(:get_contacts).with('from-email', 'mylogin', 'wrongpassword', contact_list.id).raises(Contacts::AuthenticationError)
+
+    job = GetEmailContactsJob.new('from-email', 'mylogin', 'wrongpassword', contact_list.id)
+    job.perform
+
+    assert ContactList.find(contact_list).fetched
+    assert_equal 'There was an error while authenticating. Did you enter correct login and password?', ContactList.find(contact_list).error_fetching
+  end
+
+end
@@ -26,8 +26,9 @@ class InvitationTest &lt; ActiveSupport::TestCase
   end
  
   should 'raises when try get contacts from unknown source' do
+    contact_list = ContactList.create
     assert_raise NotImplementedError do
-      Invitation.get_contacts('ze', 'ze12', 'bli-mail')
+      Invitation.get_contacts('ze', 'ze12', 'bli-mail', contact_list.id)
     end
   end
  
@@ -71,4 +72,29 @@ class InvitationTest &lt; ActiveSupport::TestCase
     end
   end
  
+  should 'add url on message if user removed it' do
+    person = create_user('testuser1').person
+    friend = create_user('testuser2').person
+    invitation = Invitation.create!(
+      :person => person,
+      :friend => friend,
+      :message => 'Hi <friend>, <user> is inviting you!'
+    )
+    assert_equal "Hi <friend>, <user> is inviting you!#{invitation.message_to_accept_invitation}", invitation.message
+  end
+
+  should 'do nothing with message if user added url' do
+    person = create_user('testuser1').person
+    friend = create_user('testuser2').person
+    invitation = Invitation.create!(
+      :person => person,
+      :friend => friend,
+      :message => 'Hi <friend>, <user> is inviting you to be his friend on <url>!'
+    )
+    assert_equal "Hi <friend>, <user> is inviting you to be his friend on <url>!", invitation.message
+  end
+
+  should 'have a message with url' do
+    assert_equal "<p>To accept invitation, please follow this link: <url></p>", Invitation.new.message_to_accept_invitation
+  end
 end
@@ -70,21 +70,6 @@ class InviteFriendTest &lt; ActiveSupport::TestCase
     ok('must validate with no target') { !task.errors.invalid?(:target_id) }
   end
  
-  should 'require message with <url> tag if no target given' do
-    task = InviteFriend.new
-    task.valid?
-
-    ok('must not validate with no message') { task.errors.invalid?(:message) }
-
-    task.message = 'a simple message'
-    task.valid?
-    ok('must not validate with no <url> tag in message') { task.errors.invalid?(:message) }
-
-    task.message = 'a simple message with <url>'
-    task.valid?
-    ok('must validate when message is given with <url> tag') { !task.errors.invalid?(:message)}
-  end
-
   should 'dont require message if target given (person being invited)' do
     task = InviteFriend.new(:target => create_user('testuser2').person)
     task.valid?
...	...	@@ -2,42 +2,57 @@ class InviteController < PublicController
2	2
3	3	needs_profile
4	4	before_filter :login_required
5		- before_filter :check_permissions_to_invite, :only => 'friends'
	5	+ before_filter :check_permissions_to_invite
6	6
7		- def friends
8		- step = params[:step]
	7	+ def select_address_book
	8	+ @import_from = params[:import_from] \|\| "manual"
9	9	if request.post?
10		- if step == '1'
11		- begin
12		- @contacts = Invitation.get_contacts(params[:import_from], params[:login], params[:password])
13		- rescue
14		- @login = params[:login]
15		- flash.now[:notice] = _('There was an error while looking for your contact list. Did you enter correct login and password?')
16		- end
17		- elsif step == '2'
18		- manual_import_addresses = params[:manual_import_addresses]
19		- webmail_import_addresses = params[:webmail_import_addresses]
20		- contacts_to_invite = Invitation.join_contacts(manual_import_addresses, webmail_import_addresses)
21		- if !params[:mail_template].match(/<url>/)
22		- flash.now[:notice] = _('<url> is needed in invitation mail.')
23		- elsif !contacts_to_invite.empty?
24		- Delayed::Job.enqueue InvitationJob.new(user.id, contacts_to_invite, params[:mail_template], profile.id)
25		- session[:notice] = _('Your invitations are being sent.')
26		- if profile.person?
27		- redirect_to :controller => 'friends'
28		- else
29		- redirect_to :controller => 'profile_members'
30		- end
	10	+ contact_list = ContactList.create
	11	+ Delayed::Job.enqueue GetEmailContactsJob.new(@import_from, params[:login], params[:password], contact_list.id) if @import_from != 'manual'
	12	+ redirect_to :action => 'select_friends', :contact_list => contact_list.id, :import_from => @import_from
	13	+ end
	14	+ end
	15	+
	16	+ def select_friends
	17	+ @contact_list = ContactList.find(params[:contact_list])
	18	+ @mail_template = params[:mail_template] \|\| environment.invitation_mail_template(profile)
	19	+ @import_from = params[:import_from] \|\| "manual"
	20	+ if request.post?
	21	+ manual_import_addresses = params[:manual_import_addresses]
	22	+ webmail_import_addresses = params[:webmail_import_addresses]
	23	+ contacts_to_invite = Invitation.join_contacts(manual_import_addresses, webmail_import_addresses)
	24	+ if !contacts_to_invite.empty?
	25	+ Delayed::Job.enqueue InvitationJob.new(current_user.person.id, contacts_to_invite, params[:mail_template], profile.id, @contact_list.id)
	26	+ session[:notice] = _('Your invitations are being sent.')
	27	+ if profile.person?
	28	+ redirect_to :controller => 'profile', :action => 'friends'
31	29	else
32		- flash.now[:notice] = _('Please enter a valid email address.')
	30	+ redirect_to :controller => 'profile', :action => 'members'
33	31	end
34		- @contacts = params[:webmail_friends] ? params[:webmail_friends].map {\|e\| YAML.load(e)} : []
35		- @manual_import_addresses = manual_import_addresses \|\| ""
36		- @webmail_import_addresses = webmail_import_addresses \|\| []
	32	+ return
	33	+ else
	34	+ session[:notice] = _('Please enter a valid email address.')
37	35	end
	36	+ @manual_import_addresses = manual_import_addresses \|\| ""
	37	+ @webmail_import_addresses = webmail_import_addresses \|\| []
38	38	end
39		- @import_from = params[:import_from] \|\| "manual"
40		- @mail_template = params[:mail_template] \|\| environment.invitation_mail_template(profile)
	39	+ end
	40	+
	41	+ def invitation_data
	42	+ contact_list = ContactList.find(params[:contact_list])
	43	+ render :text => contact_list.data.to_json, :layout => false, :content_type => "application/javascript"
	44	+ end
	45	+
	46	+ def add_contact_list
	47	+ contact_list = ContactList.find(params[:contact_list])
	48	+ contacts = contact_list.list
	49	+ render :partial => 'invite/contact_list', :locals => {:contacts => contacts}
	50	+ end
	51	+
	52	+ def cancel_fetching_emails
	53	+ contact_list = ContactList.find(params[:contact_list])
	54	+ contact_list.destroy
	55	+ redirect_to :action => 'select_address_book'
41	56	end
42	57
43	58	protected
...	...
...	...	@@ -0,0 +1,27 @@
	1	+class ContactList < ActiveRecord::Base
	2	+
	3	+ serialize :list, Array
	4	+
	5	+ def list
	6	+ self[:list] \|\| []
	7	+ end
	8	+
	9	+ def data
	10	+ if self.fetched
	11	+ { "fetched" => true, "contact_list" => self.id, "error" => self.error_fetching }
	12	+ else
	13	+ {}
	14	+ end
	15	+ end
	16	+
	17	+ def register_auth_error
	18	+ msg = _('There was an error while authenticating. Did you enter correct login and password?')
	19	+ self.update_attributes(:fetched => true, :error_fetching => msg)
	20	+ end
	21	+
	22	+ def register_error
	23	+ msg = _('There was an error while looking for your contact list. Please, try again')
	24	+ self.update_attributes(:fetched => true, :error_fetching => msg)
	25	+ end
	26	+
	27	+end
...	...
...	...	@@ -11,7 +11,6 @@ class Invitation < Task
11	11	validates_format_of :friend_email, :with => Noosfero::Constants::EMAIL_FORMAT, :if => Proc.new{\|invite\| invite.target_id.blank?}
12	12
13	13	validates_presence_of :message, :if => Proc.new{\|invite\| invite.target_id.blank?}
14		- validates_format_of :message, :with => /<url>/, :if => Proc.new{\|invite\| invite.target_id.blank?}
15	14
16	15	alias :person :requestor
17	16	alias :person= :requestor=
...	...	@@ -19,6 +18,12 @@ class Invitation < Task
19	18	alias :friend :target
20	19	alias :friend= :target=
21	20
	21	+ before_create do \|task\|
	22	+ if task.message && !task.message.match(/<url>/)
	23	+ task.message += task.message_to_accept_invitation
	24	+ end
	25	+ end
	26	+
22	27	after_create do \|task\|
23	28	TaskMailer.deliver_invitation_notification(task) unless task.friend
24	29	end
...	...	@@ -72,21 +77,26 @@ class Invitation < Task
72	77	end
73	78	end
74	79
75		- def self.get_contacts(source, login, password)
76		- contacts = []
	80	+ def self.get_contacts(source, login, password, contact_list_id)
	81	+ contact_list = ContactList.find(contact_list_id)
77	82	case source
78	83	when "gmail"
79		- contacts = Contacts::Gmail.new(login, password).contacts
	84	+ email_service = Contacts::Gmail.new(login, password)
80	85	when "yahoo"
81		- contacts = Contacts::Yahoo.new(login, password).contacts
	86	+ email_service = Contacts::Yahoo.new(login, password)
82	87	when "hotmail"
83		- contacts = Contacts::Hotmail.new(login, password).contacts
	88	+ email_service = Contacts::Hotmail.new(login, password)
84	89	when "manual"
85	90	#do nothing
86	91	else
87	92	raise NotImplementedError, 'Unknown source to get contacts'
88	93	end
89		- contacts.map { \|contact\| contact + ["#{contact[0]} <#{contact[1]}>"] }
	94	+ if email_service
	95	+ contact_list.list = email_service.contacts.map { \|contact\| contact + ["#{contact[0]} <#{contact[1]}>"] }
	96	+ contact_list.fetched = true
	97	+ contact_list.save
	98	+ end
	99	+ contact_list.list
90	100	end
91	101
92	102	def self.join_contacts(manual_import_addresses, webmail_import_addresses)
...	...	@@ -112,4 +122,7 @@ class Invitation < Task
112	122	raise 'You should implement mail_template in a subclass'
113	123	end
114	124
	125	+ def message_to_accept_invitation
	126	+ '<p>' + _('To accept invitation, please follow this link: <url>') + '</p>'
	127	+ end
115	128	end
...	...
...	...	@@ -14,7 +14,7 @@
14	14	<% button_bar do %>
15	15	<%= button(:back, _('Back to control panel'), :controller => 'profile_editor') %>
16	16	<%= button(:search, _('Find people'), :controller => 'search', :action => 'assets', :asset => 'people') %>
17		- <%= button(:search, _('Invite people from my e-mail contacts'), :controller => 'invite', :action => 'friends') %>
	17	+ <%= button(:search, _('Invite people from my e-mail contacts'), :controller => 'invite', :action => 'select_address_book') %>
18	18	<% end %>
19	19	<% end %>
20	20
...	...	@@ -40,11 +40,11 @@
40	40	<%= pagination_links @friends, :param_name => 'npage' %>
41	41	</div>
42	42
43		-<% button_bar do %>
44		- <%= button(:back, _('Back to control panel'), :controller => 'profile_editor') %>
45		- <%= button(:search, _('Find people'), :controller => 'search', :action => 'assets', :asset => 'people') %>
46		- <%= button(:search, _('Invite people from my e-mail contacts'), :controller => 'invite', :action => 'friends') %>
47		-<% end %>
	43	+ <% button_bar do %>
	44	+ <%= button(:back, _('Back to control panel'), :controller => 'profile_editor') %>
	45	+ <%= button(:search, _('Find people'), :controller => 'search', :action => 'assets', :asset => 'people') %>
	46	+ <%= button(:search, _('Invite people from my e-mail contacts'), :controller => 'invite', :action => 'select_address_book') %>
	47	+ <% end %>
48	48	<% end %>
49	49
50	50	</div><!-- end id="manage_friends" -->
...	...
...	...	@@ -0,0 +1,12 @@
	1	+<% friend_pos = 0 %>
	2	+<% contacts.each do \|contact\| %>
	3	+ <% friend_pos += 1 %>
	4	+ <p>
	5	+ <%= check_box_tag("webmail_import_addresses[]", contact[2],
	6	+ (!@webmail_import_addresses \|\| @webmail_import_addresses.include?(contact[2])),
	7	+ :id => "contacts_to_invite_#{friend_pos}",
	8	+ :class => "contact_to_invite" )
	9	+ %>
	10	+ <label for="<%= "contacts_to_invite_#{friend_pos}" %>"><%= "#{contact[0]} (#{contact[1]})" %></label>
	11	+ </p>
	12	+<% end %>
...	...
...	...	@@ -0,0 +1,23 @@
	1	+<% javascript_tag do %>
	2	+ jQuery(function($) {
	3	+ $("#loading-dialog").dialog({
	4	+ height: 160,
	5	+ width: 450,
	6	+ modal: true,
	7	+ resizable: false,
	8	+ title: "<%= ui_icon('ui-icon-info') + _('Please, wait...') %>",
	9	+ open: check_contact_list('<%= contact_list.to_s %>'),
	10	+ });
	11	+ });
	12	+<% end %>
	13	+
	14	+
	15	+<div id="loading-dialog">
	16	+ <p><%= _('Your e-mails contacts are being fetched') %></p>
	17	+ <p><%= _('If it takes too long, you will be redirected to the previous page to try again. Be sure to fill in the fields with your correct login and password.') %>
	18	+ <span class='loading-message'/>
	19	+ <%= link_to(_('Verify contact list'), {:action => 'invitation_data', :contact_list => @contact_list}, :id => "verify-contact-list", :style => 'display:none') %>
	20	+ <%= link_to(_('Add contact list'), {:action => 'add_contact_list', :contact_list => @contact_list}, :id => "add-contact-list", :style => 'display:none') %>
	21	+ <%= link_to(_('Cancel fetching e-mails'), {:action => 'cancel_fetching_emails', :contact_list => @contact_list}, :id => "cancel-fetching-emails", :style => 'display:none') %>
	22	+</div>
	23	+
...	...
...	...	@@ -1,93 +0,0 @@
1		-<% if profile.person? %>
2		- <h1><%= _('Invite your friends') %></h1>
3		-<% else %>
4		- <h1><%= _('Invite your friends to join %s') % profile.name %></h1>
5		-<% end %>
6		-
7		-<% unless @contacts %>
8		-
9		- <h2><%= _('Step 1 of 2: Select address book') %></h2>
10		-
11		- <% form_tag do %>
12		- <%= hidden_field_tag(:step, 1) %>
13		-
14		- <%= [
15		- radio_button_tag(:import_from, "manual", @import_from == "manual", :onclick => 'hide_invite_friend_login_password()') + content_tag('label', _('Manually (empty field)'), :for => "import_from_manual"),
16		- radio_button_tag(:import_from, "gmail", @import_from == "gmail", :onclick => 'show_invite_friend_login_password()') + content_tag('label', 'Gmail', :for => 'import_from_gmail'),
17		- radio_button_tag(:import_from, "yahoo", @import_from == "yahoo", :onclick => 'show_invite_friend_login_password()') + content_tag('label', 'Yahoo', :for => "import_from_yahoo"),
18		- radio_button_tag(:import_from, "hotmail", @import_from == "hotmail", :onclick => 'show_invite_friend_login_password()') + content_tag('label', 'Hotmail', :for => "import_from_hotmail")
19		- ].join("\n<br/>\n") %>
20		-
21		- <script type="text/javascript">
22		- function hide_invite_friend_login_password() {
23		- $('invite-friends-login-password').hide();
24		- }
25		- function show_invite_friend_login_password() {
26		- $('invite-friends-login-password').show();
27		- $('login').focus();
28		- }
29		- </script>
30		- <div id='invite-friends-login-password' <%= "style='display: none;'" if (@import_from == 'manual') %>>
31		- <%= labelled_form_field(_("Username") + ":", text_field_tag(:login, @login)) %>
32		- <%= labelled_form_field(_("Password") + ":", password_field_tag(:password)) %>
33		- </div>
34		-
35		- <% button_bar do %>
36		- <%= submit_button(:forward, _("Next")) %>
37		- <% end %>
38		- <p><%= _("We won't store your password or contact anyone without your permission.") %></p>
39		- <% end %>
40		-
41		-<% else %>
42		-
43		- <h2><%= _('Step 2 of 2: Selecting Friends') %></h2>
44		- <p>
45		- <%= _('Indicate which friends you want to invite.') %>
46		- </p>
47		-
48		- <% form_tag do %>
49		- <%= hidden_field_tag(:step, 2) %>
50		- <%= hidden_field_tag(:import_from, @import_from) %>
51		-
52		- <div>
53		- <%= labelled_form_field(_('Enter one e-mail address per line:'), text_area_tag(:manual_import_addresses, (@manual_import_addresses \|\| ''), :cols => 72, :rows => 5)) %>
54		- </div>
55		- <% if @import_from != 'manual' %>
56		- <div>
57		- <%= link_to_function _('Check all'), "$$('input.contact_to_invite').each(function(checkbox) { checkbox.checked = true; });" %>
58		- <%= link_to_function _('Uncheck all'), "$$('input.contact_to_invite').each(function(checkbox) { checkbox.checked = false; });" %>
59		- <% friend_pos = 0 %>
60		- <div id='contacts-list'>
61		- <% @contacts.each do \|contact\| %>
62		- <% friend_pos += 1 %>
63		- <p>
64		- <%= hidden_field_tag("webmail_friends[]", contact.to_yaml) %>
65		- <%= check_box_tag("webmail_import_addresses[]", contact[2],
66		- (!@webmail_import_addresses \|\| @webmail_import_addresses.include?(contact[2])),
67		- :id => "contacts_to_invite_#{friend_pos}",
68		- :class => "contact_to_invite" )
69		- %>
70		- <label for="<%= "contacts_to_invite_#{friend_pos}" %>"><%= "#{contact[0]} (#{contact[1]})" %></label>
71		- </p>
72		- <% end %>
73		- </div>
74		- </div>
75		- <% end -%>
76		-
77		- <br/>
78		-
79		- <%= link_to_function(_('Personalize invitation mail'), nil) do \|page\|
80		- page['invitation-mail_template'].show
81		- end %>
82		-
83		- <div id='invitation-mail_template' style='display:none'>
84		- <%= h _("Now enter an invitation message. You must keep the <url> code in your invitation text. When your friends receive the invitation e-mail, <url> will be replaced by a link that they need to click to activate their account. <user> and <friend> codes will be replaced by your name and friend name, but they are optional.") %>
85		- <%= labelled_form_field(_('Invitation text:'), text_area_tag(:mail_template, @mail_template, :cols => 72, :rows => 8)) %>
86		- </div>
87		-
88		- <% button_bar do %>
89		- <%= submit_button(:ok, _("Invite my friends!")) %>
90		- <% end %>
91		- <% end %>
92		-
93		-<% end %>
...	...	@@ -0,0 +1,51 @@
	1	+<% if profile.person? %>
	2	+ <h1><%= _('Invite your friends') %></h1>
	3	+<% else %>
	4	+ <h1><%= _('Invite your friends to join %s') % profile.name %></h1>
	5	+<% end %>
	6	+
	7	+<h2><%= _('Step 1 of 2: Select address book') %></h2>
	8	+
	9	+<% form_tag do %>
	10	+
	11	+ <%= [
	12	+ radio_button_tag(:import_from, "manual", @import_from == "manual", :onclick => 'hide_invite_friend_login_password()') + content_tag('label', _('Manually (empty field)'), :for => "import_from_manual"),
	13	+ radio_button_tag(:import_from, "gmail", @import_from == "gmail", :onclick => 'show_invite_friend_login_password(this.value)') + content_tag('label', 'Gmail', :for => 'import_from_gmail'),
	14	+ radio_button_tag(:import_from, "yahoo", @import_from == "yahoo", :onclick => 'show_invite_friend_login_password(this.value)') + content_tag('label', 'Yahoo', :for => "import_from_yahoo"),
	15	+ radio_button_tag(:import_from, "hotmail", @import_from == "hotmail", :onclick => 'show_invite_friend_login_password(this.value)') + content_tag('label', 'Hotmail', :for => "import_from_hotmail")
	16	+ ].join("\n<br/>\n") %>
	17	+
	18	+ <script type="text/javascript">
	19	+ function hide_invite_friend_login_password() {
	20	+ $('invite-friends-login-password').hide();
	21	+ }
	22	+ function show_invite_friend_login_password(option) {
	23	+ if (option == 'hotmail') {
	24	+ $('hotmail_username_tip').show();
	25	+ } else {
	26	+ $('hotmail_username_tip').hide();
	27	+ }
	28	+ $('invite-friends-login-password').show();
	29	+ $('login').focus();
	30	+ }
	31	+ </script>
	32	+ <div id='invite-friends-login-password' <%= "style='display: none;'" if (@import_from == 'manual') %>>
	33	+ <div id='hotmail_username_tip'>
	34	+ <%= ui_icon('ui-icon-alert') %>
	35	+ <%= _('Please type your username in the format yourname@example.com') %>
	36	+ </div>
	37	+
	38	+ <%= labelled_form_field(_("Username") + ":", text_field_tag(:login, @login)) %>
	39	+ <%= labelled_form_field(_("Password") + ":", password_field_tag(:password)) %>
	40	+ </div>
	41	+
	42	+ <% button_bar do %>
	43	+ <%= submit_button(:forward, _("Next")) %>
	44	+ <% end %>
	45	+ <p><%= _("We won't store your password or contact anyone without your permission.") %></p>
	46	+<% end %>
	47	+
	48	+<div id="loadingScreen"></div>
	49	+
	50	+
	51	+
...	...
...	...	@@ -0,0 +1,47 @@
	1	+<%= render :partial => 'invite/dialog_wait_loading', :locals => {:contact_list => @contact_list.id } if @import_from != 'manual' %>
	2	+
	3	+<% if profile.person? %>
	4	+ <h1><%= _('Invite your friends') %></h1>
	5	+<% else %>
	6	+ <h1><%= _('Invite your friends to join %s') % profile.name %></h1>
	7	+<% end %>
	8	+
	9	+
	10	+<h2><%= _('Step 2 of 2: Selecting Friends') %></h2>
	11	+
	12	+<%= button(:back, _('Back'), { :action => 'select_address_book' }, :id => 'invitation_back_button') %>
	13	+
	14	+<p>
	15	+<%= _('Indicate which friends you want to invite.') %>
	16	+</p>
	17	+
	18	+<% form_tag do %>
	19	+ <%= hidden_field_tag(:import_from, @import_from) %>
	20	+ <%= hidden_field_tag(:contact_list, @contact_list.id) %>
	21	+
	22	+ <div>
	23	+ <%= labelled_form_field(_('Enter one e-mail address per line:'), text_area_tag(:manual_import_addresses, (@manual_import_addresses \|\| ''), :cols => 72, :rows => 5)) %>
	24	+ </div>
	25	+ <% if @import_from != 'manual' %>
	26	+ <div>
	27	+ <%= link_to_function _('Check all'), "$$('input.contact_to_invite').each(function(checkbox) { checkbox.checked = true; });" %>
	28	+ <%= link_to_function _('Uncheck all'), "$$('input.contact_to_invite').each(function(checkbox) { checkbox.checked = false; });" %>
	29	+ <div id='contacts-list'></div>
	30	+ </div>
	31	+ <% end -%>
	32	+
	33	+ <br/>
	34	+
	35	+ <%= link_to_function(_('Personalize invitation mail'), nil) do \|page\|
	36	+ page['invitation-mail_template'].show
	37	+ end %>
	38	+
	39	+ <div id='invitation-mail_template' style='display:none'>
	40	+ <%= h _("Now enter an invitation message. You must keep the <url> code in your invitation text. When your friends receive the invitation e-mail, <url> will be replaced by a link that they need to click to activate their account. <user> and <friend> codes will be replaced by your name and friend name, but they are optional.") %>
	41	+ <%= labelled_form_field(_('Invitation text:'), text_area_tag(:mail_template, @mail_template, :cols => 72, :rows => 8)) %>
	42	+ </div>
	43	+
	44	+ <% button_bar do %>
	45	+ <%= submit_button(:ok, _("Invite my friends!")) %>
	46	+ <% end %>
	47	+<% end %>
...	...
...	...	@@ -18,7 +18,7 @@
18	18	<%= button :back, _('Go back'), { :controller => 'profile' } %>
19	19	<% if user == profile %>
20	20	<%= button :edit, _('Manage my friends'), :controller => 'friends', :action => 'index', :profile => profile.identifier %>
21		- <%= button(:search, _('Invite people from my e-mail contacts'), :controller => 'invite', :action => 'friends') %>
	21	+ <%= button(:search, _('Invite people from my e-mail contacts'), :controller => 'invite', :action => 'select_address_book') %>
22	22	<% end %>
23	23	<% end %>
24	24
...	...