James Moger
2011-05-11 511554c6bdd7daa080506fb62400bf6d295ab9e2
Renamed User to UserModel.
1 files renamed
8 files modified
86 ■■■■ changed files
src/com/gitblit/GitBlit.java 18 ●●●● patch | view | raw | blame | history
src/com/gitblit/ILoginService.java 12 ●●●● patch | view | raw | blame | history
src/com/gitblit/JettyLoginService.java 16 ●●●● patch | view | raw | blame | history
src/com/gitblit/wicket/AuthorizationStrategy.java 4 ●●●● patch | view | raw | blame | history
src/com/gitblit/wicket/GitBlitWebSession.java 8 ●●●● patch | view | raw | blame | history
src/com/gitblit/wicket/LoginPage.java 8 ●●●● patch | view | raw | blame | history
src/com/gitblit/wicket/models/UserModel.java 4 ●●●● patch | view | raw | blame | history
src/com/gitblit/wicket/pages/EditUserPage.java 12 ●●●● patch | view | raw | blame | history
src/com/gitblit/wicket/pages/RepositoriesPage.java 4 ●●●● patch | view | raw | blame | history
src/com/gitblit/GitBlit.java
@@ -22,7 +22,7 @@
import com.gitblit.Constants.AccessRestrictionType;
import com.gitblit.utils.JGitUtils;
import com.gitblit.wicket.models.RepositoryModel;
import com.gitblit.wicket.models.User;
import com.gitblit.wicket.models.UserModel;
public class GitBlit implements ServletContextListener {
@@ -67,14 +67,14 @@
        this.loginService = loginService;
    }
    public User authenticate(String username, char[] password) {
    public UserModel authenticate(String username, char[] password) {
        if (loginService == null) {
            return null;
        }
        return loginService.authenticate(username, password);
    }
    public User authenticate(Cookie[] cookies) {
    public UserModel authenticate(Cookie[] cookies) {
        if (loginService == null) {
            return null;
        }
@@ -89,19 +89,19 @@
        return null;
    }
    public void setCookie(WebResponse response, User user) {
    public void setCookie(WebResponse response, UserModel user) {
        Cookie userCookie = new Cookie(Constants.NAME, user.getCookie());
        userCookie.setMaxAge(Integer.MAX_VALUE);
        userCookie.setPath("/");
        response.addCookie(userCookie);
    }
    public User getUser(String username) {
        User user = loginService.getUserModel(username);
    public UserModel getUser(String username) {
        UserModel user = loginService.getUserModel(username);
        return user;
    }
    public void editUserModel(User user, boolean isCreate) throws GitBlitException {
    public void editUserModel(UserModel user, boolean isCreate) throws GitBlitException {
        if (!loginService.updateUserModel(user)) {
            throw new GitBlitException(isCreate ? "Failed to add user!" : "Failed to update user!");
        }
@@ -125,7 +125,7 @@
        return r;
    }
    public List<RepositoryModel> getRepositoryModels(User user) {
    public List<RepositoryModel> getRepositoryModels(UserModel user) {
        List<String> list = getRepositoryList();
        List<RepositoryModel> repositories = new ArrayList<RepositoryModel>();
        for (String repo : list) {
@@ -137,7 +137,7 @@
        return repositories;
    }
    
    public RepositoryModel getRepositoryModel(User user, String repositoryName) {
    public RepositoryModel getRepositoryModel(UserModel user, String repositoryName) {
        RepositoryModel model = getRepositoryModel(repositoryName);
        if (model.accessRestriction.atLeast(AccessRestrictionType.VIEW)) {
            if (user != null && user.canView(model)) {
src/com/gitblit/ILoginService.java
@@ -1,17 +1,17 @@
package com.gitblit;
import com.gitblit.wicket.models.User;
import com.gitblit.wicket.models.UserModel;
public interface ILoginService {
    User authenticate(String username, char[] password);
    UserModel authenticate(String username, char[] password);
    User authenticate(char[] cookie);
    UserModel authenticate(char[] cookie);
    
    User getUserModel(String username);
    UserModel getUserModel(String username);
    
    boolean updateUserModel(User model);
    boolean updateUserModel(UserModel model);
    
    boolean deleteUserModel(User model);
    boolean deleteUserModel(UserModel model);
    
}
src/com/gitblit/JettyLoginService.java
@@ -18,7 +18,7 @@
import org.eclipse.jetty.util.log.Log;
import com.gitblit.utils.StringUtils;
import com.gitblit.wicket.models.User;
import com.gitblit.wicket.models.UserModel;
public class JettyLoginService extends MappedLoginService implements ILoginService {
@@ -31,12 +31,12 @@
    }
    @Override
    public User authenticate(String username, char[] password) {
    public UserModel authenticate(String username, char[] password) {
        UserIdentity identity = login(username, new String(password));
        if (identity == null || identity.equals(UserIdentity.UNAUTHENTICATED_IDENTITY)) {
            return null;
        }
        User user = new User(username);
        UserModel user = new UserModel(username);
        user.setCookie(StringUtils.getSHA1((Constants.NAME + username + new String(password))));
        user.canAdmin(identity.isUserInRole(Constants.ADMIN_ROLE, null));
@@ -53,14 +53,14 @@
    }
    @Override
    public User authenticate(char[] cookie) {
    public UserModel authenticate(char[] cookie) {
        // TODO cookie login
        return null;
    }
    @Override
    public User getUserModel(String username) {
        User model = new User(username);
    public UserModel getUserModel(String username) {
        UserModel model = new UserModel(username);
        UserIdentity identity = _users.get(username);
        Subject subject = identity.getSubject();
        for (Principal principal : subject.getPrincipals()) {
@@ -83,7 +83,7 @@
    }
    @Override
    public boolean updateUserModel(User model) {
    public boolean updateUserModel(UserModel model) {
        try {
            Properties properties = new Properties();
            FileReader reader = new FileReader(realmFile);
@@ -129,7 +129,7 @@
    }
    @Override
    public boolean deleteUserModel(User model) {
    public boolean deleteUserModel(UserModel model) {
        try {
            // Read realm file
            Properties properties = new Properties();
src/com/gitblit/wicket/AuthorizationStrategy.java
@@ -7,7 +7,7 @@
import com.gitblit.GitBlit;
import com.gitblit.Keys;
import com.gitblit.wicket.models.User;
import com.gitblit.wicket.models.UserModel;
import com.gitblit.wicket.pages.RepositoriesPage;
public class AuthorizationStrategy extends AbstractPageAuthorizationStrategy implements IUnauthorizedComponentInstantiationListener {
@@ -29,7 +29,7 @@
                return false;
            }
            
            User user = session.getUser();
            UserModel user = session.getUser();
            if (pageClass.isAnnotationPresent(AdminPage.class)) {
                // admin page
                if (allowAdmin) {
src/com/gitblit/wicket/GitBlitWebSession.java
@@ -7,7 +7,7 @@
import org.apache.wicket.protocol.http.WebSession;
import org.apache.wicket.protocol.http.request.WebClientInfo;
import com.gitblit.wicket.models.User;
import com.gitblit.wicket.models.UserModel;
public final class GitBlitWebSession extends WebSession {
@@ -15,7 +15,7 @@
    protected TimeZone timezone = null;
    private User user = null;
    private UserModel user = null;
    
    private String errorMessage = null;
@@ -39,11 +39,11 @@
        return user.canAdmin();
    }
    public User getUser() {
    public UserModel getUser() {
        return user;
    }
    public void setUser(User user) {
    public void setUser(UserModel user) {
        this.user = user;
    }
src/com/gitblit/wicket/LoginPage.java
@@ -18,7 +18,7 @@
import com.gitblit.Constants;
import com.gitblit.GitBlit;
import com.gitblit.Keys;
import com.gitblit.wicket.models.User;
import com.gitblit.wicket.models.UserModel;
public class LoginPage extends WebPage {
@@ -61,7 +61,7 @@
            String username = LoginPage.this.username.getObject();
            char[] password = LoginPage.this.password.getObject().toCharArray();
            User user = GitBlit.self().authenticate(username, password);
            UserModel user = GitBlit.self().authenticate(username, password);
            if (user == null)
                error("Invalid username or password!");
            else
@@ -70,7 +70,7 @@
    }
    private void tryAutomaticLogin() {
        User user = null;
        UserModel user = null;
        // Grab cookie from Browser Session
        Cookie[] cookies = ((WebRequest) getRequestCycle().getRequest()).getCookies();
@@ -82,7 +82,7 @@
        loginUser(user);
    }
    private void loginUser(User user) {
    private void loginUser(UserModel user) {
        if (user != null) {
            // Set the user into the session
            GitBlitWebSession.get().setUser(user);
src/com/gitblit/wicket/models/UserModel.java
File was renamed from src/com/gitblit/wicket/models/User.java
@@ -6,7 +6,7 @@
import com.gitblit.Constants.AccessRestrictionType;
public class User implements Serializable {
public class UserModel implements Serializable {
    private static final long serialVersionUID = 1L;
@@ -16,7 +16,7 @@
    private boolean canAdmin = false;
    private List<String> repositories = new ArrayList<String>();
    public User(String username) {
    public UserModel(String username) {
        this.username = username;
    }
src/com/gitblit/wicket/pages/EditUserPage.java
@@ -22,7 +22,7 @@
import com.gitblit.wicket.AdminPage;
import com.gitblit.wicket.BasePage;
import com.gitblit.wicket.WicketUtils;
import com.gitblit.wicket.models.User;
import com.gitblit.wicket.models.UserModel;
@AdminPage
public class EditUserPage extends BasePage {
@@ -33,7 +33,7 @@
        // create constructor
        super();
        isCreate = true;
        setupPage(new User(""));
        setupPage(new UserModel(""));
    }
    public EditUserPage(PageParameters params) {
@@ -41,23 +41,23 @@
        super(params);
        isCreate = false;
        String name = WicketUtils.getUsername(params);
        User model = GitBlit.self().getUser(name);
        UserModel model = GitBlit.self().getUser(name);
        setupPage(model);
    }
    protected void setupPage(final User userModel) {
    protected void setupPage(final UserModel userModel) {
        if (isCreate) {
            super.setupPage("", getString("gb.newUser"));
        } else {
            super.setupPage("", getString("gb.edit"));
        }
        final Model<String> confirmPassword = new Model<String>();
        CompoundPropertyModel<User> model = new CompoundPropertyModel<User>(userModel);
        CompoundPropertyModel<UserModel> model = new CompoundPropertyModel<UserModel>(userModel);
        List<String> repos = GitBlit.self().getRepositoryList();
        repos.add(0, "*"); // all repositories wildcard
        final Palette<String> repositories = new Palette<String>("repositories", new ListModel<String>(userModel.getRepositories()), new CollectionModel<String>(repos), new ChoiceRenderer<String>("", ""), 10, false);        
        Form<User> form = new Form<User>("editForm", model) {
        Form<UserModel> form = new Form<UserModel>("editForm", model) {
            private static final long serialVersionUID = 1L;
src/com/gitblit/wicket/pages/RepositoriesPage.java
@@ -33,7 +33,7 @@
import com.gitblit.wicket.LinkPanel;
import com.gitblit.wicket.WicketUtils;
import com.gitblit.wicket.models.RepositoryModel;
import com.gitblit.wicket.models.User;
import com.gitblit.wicket.models.UserModel;
public class RepositoriesPage extends BasePage {
@@ -99,7 +99,7 @@
        }
        add(repositoriesMessage);
        User user = GitBlitWebSession.get().getUser();
        UserModel user = GitBlitWebSession.get().getUser();
        List<RepositoryModel> rows = GitBlit.self().getRepositoryModels(user);
        DataProvider dp = new DataProvider(rows);
        DataView<RepositoryModel> dataView = new DataView<RepositoryModel>("repository", dp) {