James Moger
2012-10-22 ba54242b0d080475bbfce2ab559b3fcf2a513a5b
Fixed null pointer exceptions Manager create User and Team
2 files modified
18 ■■■■ changed files
src/com/gitblit/client/EditTeamDialog.java 9 ●●●● patch | view | raw | blame | history
src/com/gitblit/client/EditUserDialog.java 9 ●●●● patch | view | raw | blame | history
src/com/gitblit/client/EditTeamDialog.java
@@ -49,6 +49,7 @@
import com.gitblit.models.RepositoryModel;
import com.gitblit.models.ServerSettings;
import com.gitblit.models.TeamModel;
import com.gitblit.utils.ArrayUtils;
import com.gitblit.utils.StringUtils;
public class EditTeamDialog extends JDialog {
@@ -316,8 +317,12 @@
        }
        
        // remove repositories for which team already has a permission
        for (RegistrantAccessPermission rp : permissions) {
            restricted.remove(rp.registrant);
        if (permissions == null) {
            permissions = new ArrayList<RegistrantAccessPermission>();
        } else {
            for (RegistrantAccessPermission rp : permissions) {
                restricted.remove(rp.registrant);
            }
        }
        StringUtils.sortRepositorynames(restricted);
src/com/gitblit/client/EditUserDialog.java
@@ -347,9 +347,14 @@
                restricted.add(repo.name);
            }
        }
        // remove repositories for which user already has a permission
        for (RegistrantAccessPermission rp : permissions) {
            restricted.remove(rp.registrant);
        if (permissions == null) {
            permissions = new ArrayList<RegistrantAccessPermission>();
        } else {
            for (RegistrantAccessPermission rp : permissions) {
                restricted.remove(rp.registrant);
            }
        }
        
        StringUtils.sortRepositorynames(restricted);