From f16b1c4be237dd3953ef39d29088cc1ed0248b2b Mon Sep 17 00:00:00 2001
From: James Moger <james.moger@gitblit.com>
Date: Wed, 28 Nov 2012 07:44:22 -0500
Subject: [PATCH] Revert "Eliminated redundant "view" links (issue 168)"

---
 src/com/gitblit/FileUserService.java |    8 +++++---
 1 files changed, 5 insertions(+), 3 deletions(-)

diff --git a/src/com/gitblit/FileUserService.java b/src/com/gitblit/FileUserService.java
index d411b68..056df82 100644
--- a/src/com/gitblit/FileUserService.java
+++ b/src/com/gitblit/FileUserService.java
@@ -329,8 +329,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 +796,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);

--
Gitblit v1.9.1