Laurens Vrijnsen
2013-03-22 662fc186b14b25019ecc9ae05ce0894360d44393
src/com/gitblit/FileUserService.java
@@ -20,6 +20,7 @@
import java.io.IOException;
import java.text.MessageFormat;
import java.util.ArrayList;
import java.util.Collection;
import java.util.Collections;
import java.util.HashSet;
import java.util.List;
@@ -270,12 +271,12 @@
   /**
    * Updates/writes all specified user objects.
    * 
    * @param model a list of user models
    * @param models a list of user models
    * @return true if update is successful
    * @since 1.2.0
    */
   @Override
   public boolean updateUserModels(List<UserModel> models) {
   public boolean updateUserModels(Collection<UserModel> models) {
      try {         
         Properties allUsers = read();
         for (UserModel model : models) {
@@ -329,8 +330,7 @@
         UserModel oldUser = getUserModel(username);
         List<String> roles;
         if (model.permissions == null) {
            // legacy, use repository list
            roles = new ArrayList<String>(model.repositories);
            roles = new ArrayList<String>();
         } else {
            // discrete repository permissions
            roles = new ArrayList<String>();
@@ -797,7 +797,10 @@
                     repositories.add(role);
                  }
               }
               team.addRepositoryPermissions(repositories);
               if (!team.canAdmin) {
                  // only read permissions for non-admin teams
                  team.addRepositoryPermissions(repositories);
               }
               team.addUsers(users);
               team.addMailingLists(mailingLists);
               team.preReceiveScripts.addAll(preReceive);
@@ -995,7 +998,7 @@
    * @return true if update is successful
    * @since 1.2.0
    */
   public boolean updateTeamModels(List<TeamModel> models) {
   public boolean updateTeamModels(Collection<TeamModel> models) {
      try {
         Properties allUsers = read();
         for (TeamModel model : models) {