From 7812236b90f39fd92699017f376f1b50c4ea24cb Mon Sep 17 00:00:00 2001
From: James Moger <james.moger@gitblit.com>
Date: Mon, 16 May 2011 16:45:51 -0400
Subject: [PATCH] Move admin links. Drop extra GitBlit link. Line numbers on Blob page.

---
 src/com/gitblit/GitBlit.java |   63 +++++++++++++++++++++++++------
 1 files changed, 50 insertions(+), 13 deletions(-)

diff --git a/src/com/gitblit/GitBlit.java b/src/com/gitblit/GitBlit.java
index c633f6e..62ff55e 100644
--- a/src/com/gitblit/GitBlit.java
+++ b/src/com/gitblit/GitBlit.java
@@ -19,9 +19,10 @@
 import org.slf4j.Logger;
 import org.slf4j.LoggerFactory;
 
+import com.gitblit.Constants.AccessRestrictionType;
 import com.gitblit.utils.JGitUtils;
-import com.gitblit.wicket.User;
 import com.gitblit.wicket.models.RepositoryModel;
+import com.gitblit.wicket.models.UserModel;
 
 public class GitBlit implements ServletContextListener {
 
@@ -66,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;
 		}
@@ -88,13 +89,36 @@
 		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 List<String> getAllUsernames() {
+		return loginService.getAllUsernames();
+	}
+
+	public UserModel getUserModel(String username) {
+		UserModel user = loginService.getUserModel(username);
+		return user;
+	}
+	
+	public List<String> getRepositoryUsers(RepositoryModel repository) {
+		return loginService.getUsernamesForRole(repository.name);
+	}
+	
+	public boolean setRepositoryUsers(RepositoryModel repository, List<String> repositoryUsers) {
+		return loginService.setUsernamesForRole(repository.name, repositoryUsers);
+	}
+
+	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!");
+		}
+	}
+
 	public List<String> getRepositoryList() {
 		return JGitUtils.getRepositoryList(repositoriesFolder, exportAll, storedSettings.getBoolean(Keys.git.nestedRepositories, true));
 	}
@@ -112,17 +136,31 @@
 		}
 		return r;
 	}
-	
-	public List<RepositoryModel> getRepositoryModels() {
+
+	public List<RepositoryModel> getRepositoryModels(UserModel user) {
 		List<String> list = getRepositoryList();
 		List<RepositoryModel> repositories = new ArrayList<RepositoryModel>();
 		for (String repo : list) {
-			RepositoryModel model = getRepositoryModel(repo);
-			repositories.add(model);
+			RepositoryModel model = getRepositoryModel(user, repo);
+			if (model != null) {
+				repositories.add(model);
+			}
 		}
 		return repositories;
 	}
 	
+	public RepositoryModel getRepositoryModel(UserModel user, String repositoryName) {
+		RepositoryModel model = getRepositoryModel(repositoryName);
+		if (model.accessRestriction.atLeast(AccessRestrictionType.VIEW)) {
+			if (user != null && user.canAccessRepository(model.name)) {
+				return model;
+			}
+			return null;
+		} else {
+			return model;
+		}
+	}
+
 	public RepositoryModel getRepositoryModel(String repositoryName) {
 		Repository r = getRepository(repositoryName);
 		RepositoryModel model = new RepositoryModel();
@@ -133,10 +171,9 @@
 		if (config != null) {
 			model.description = config.getString("gitblit", null, "description");
 			model.owner = config.getString("gitblit", null, "owner");
-			model.group = config.getString("gitblit", null, "group");
 			model.useTickets = config.getBoolean("gitblit", "useTickets", false);
 			model.useDocs = config.getBoolean("gitblit", "useDocs", false);
-			model.useRestrictedAccess = config.getBoolean("gitblit", "restrictedAccess", false);
+			model.accessRestriction = AccessRestrictionType.fromName(config.getString("gitblit", null, "accessRestriction"));
 			model.showRemoteBranches = config.getBoolean("gitblit", "showRemoteBranches", false);
 		}
 		r.close();
@@ -149,7 +186,7 @@
 			if (new File(repositoriesFolder, repository.name).exists()) {
 				throw new GitBlitException(MessageFormat.format("Can not create repository {0} because it already exists.", repository.name));
 			}
-			// create repository			
+			// create repository
 			logger.info("create repository " + repository.name);
 			r = JGitUtils.createRepository(repositoriesFolder, repository.name, true);
 		} else {
@@ -170,7 +207,7 @@
 		config.setString("gitblit", null, "owner", repository.owner);
 		config.setBoolean("gitblit", null, "useTickets", repository.useTickets);
 		config.setBoolean("gitblit", null, "useDocs", repository.useDocs);
-		config.setBoolean("gitblit", null, "restrictedAccess", repository.useRestrictedAccess);
+		config.setString("gitblit", null, "accessRestriction", repository.accessRestriction.name());
 		config.setBoolean("gitblit", null, "showRemoteBranches", repository.showRemoteBranches);
 		try {
 			config.save();
@@ -181,7 +218,7 @@
 	}
 
 	public void configureContext(IStoredSettings settings) {
-		logger.info("Configure GitBlit from " + settings.toString());
+		logger.info("Using configuration from " + settings.toString());
 		this.storedSettings = settings;
 		repositoriesFolder = new File(settings.getString(Keys.git.repositoriesFolder, "repos"));
 		exportAll = settings.getBoolean(Keys.git.exportAll, true);

--
Gitblit v1.9.1