comparison app/controllers/members_controller.rb @ 931:ec1c49528f36 cannam_integration

Merge from branch "redmine-1.3"

     members = []
     if params[:member] && request.post?
       attrs = params[:member].dup
       if (user_ids = attrs.delete(:user_ids))
         user_ids.each do |user_id|
-          @new_member = Member.new(attrs.merge(:user_id => user_id))
+          @new_member = Member.new(:role_ids => params[:member][:role_ids], :user_id => user_id)
           members << @new_member
 
           # send notification to member
           Mailer.deliver_added_to_project(@new_member, @project)
 
         end
       else
-        @new_member = Member.new(attrs)
+        @new_member = Member.new(:role_ids => params[:member][:role_ids], :user_id => params[:member][:user_id])
         members << @new_member
         
         # send notification to member
         Mailer.deliver_added_to_project(@new_member, @project)
         

       end
     end
   end
 
   def edit
-    if request.post? and @member.update_attributes(params[:member])
+    if params[:member]
+      @member.role_ids = params[:member][:role_ids]
+    end
+    if request.post? and @member.save
   	 respond_to do |format|
         format.html { redirect_to :action => 'index', :project_id => @project }
         format.js {
           render(:update) {|page|
             page.replace_html "memberlist", :partial => 'editlist'

       }
     end
   end
 
   def autocomplete_for_member
-    @principals = Principal.active.like(params[:q]).find(:all, :limit => 100) - @project.principals
+    @principals = Principal.active.not_member_of(@project).like(params[:q]).all(:limit => 100)
     logger.debug "Query for #{params[:q]} returned #{@principals.size} results"
     render :layout => false
   end
 
 end