From 70e3d195c738cd190c46754d2d1b0c2eed9a4d4b Mon Sep 17 00:00:00 2001
From: James Moger <james.moger@gitblit.com>
Date: Wed, 31 Oct 2012 08:49:37 -0400
Subject: [PATCH] Enable verify committer iff >= push restricted

---
 src/com/gitblit/wicket/pages/EditRepositoryPage.java |   21 +++++++++++++--------
 1 files changed, 13 insertions(+), 8 deletions(-)

diff --git a/src/com/gitblit/wicket/pages/EditRepositoryPage.java b/src/com/gitblit/wicket/pages/EditRepositoryPage.java
index cc0db68..6007617 100644
--- a/src/com/gitblit/wicket/pages/EditRepositoryPage.java
+++ b/src/com/gitblit/wicket/pages/EditRepositoryPage.java
@@ -146,11 +146,10 @@
 			}
 		} else {
 			super.setupPage(getString("gb.edit"), repositoryModel.name);
-			if (repositoryModel.accessRestriction.exceeds(AccessRestrictionType.NONE)) {
-				repositoryUsers.addAll(GitBlit.self().getUserAccessPermissions(repositoryModel));
-				repositoryTeams.addAll(GitBlit.self().getTeamAccessPermissions(repositoryModel));
-				Collections.sort(repositoryUsers);
-			}
+			repositoryUsers.addAll(GitBlit.self().getUserAccessPermissions(repositoryModel));
+			repositoryTeams.addAll(GitBlit.self().getTeamAccessPermissions(repositoryModel));
+			Collections.sort(repositoryUsers);
+			
 			federationSets.addAll(repositoryModel.federationSets);
 			if (!ArrayUtils.isEmpty(repositoryModel.indexedBranches)) {
 				indexedBranches.addAll(repositoryModel.indexedBranches);
@@ -423,8 +422,10 @@
 		final RadioChoice<AuthorizationControl> authorizationControl = new RadioChoice<Constants.AuthorizationControl>(
 				"authorizationControl", acList, new AuthorizationControlRenderer());
 		form.add(authorizationControl);
-				
-		form.add(new CheckBox("verifyCommitter"));
+		
+		final CheckBox verifyCommitter = new CheckBox("verifyCommitter");
+		verifyCommitter.setOutputMarkupId(true);
+		form.add(verifyCommitter);
 
 		form.add(usersPalette);
 		form.add(teamsPalette);
@@ -446,14 +447,16 @@
 			usersPalette.setEnabled(false);
 			teamsPalette.setEnabled(false);
 			authorizationControl.setEnabled(false);
+			verifyCommitter.setEnabled(false);
 		} else {
 			// authenticated something
 			// enable authorization controls
 			authorizationControl.setEnabled(true);
+			verifyCommitter.setEnabled(true);
 			
 			boolean allowFineGrainedControls = repositoryModel.authorizationControl.equals(AuthorizationControl.NAMED);
 			usersPalette.setEnabled(allowFineGrainedControls);
-			teamsPalette.setEnabled(allowFineGrainedControls);			
+			teamsPalette.setEnabled(allowFineGrainedControls);
 		}
 		
 		accessRestriction.add(new AjaxFormComponentUpdatingBehavior("onchange") {
@@ -464,6 +467,7 @@
 				// enable/disable permissions panel based on access restriction
 				boolean allowAuthorizationControl = repositoryModel.accessRestriction.exceeds(AccessRestrictionType.NONE);
 				authorizationControl.setEnabled(allowAuthorizationControl);
+				verifyCommitter.setEnabled(allowAuthorizationControl);
 				
 				boolean allowFineGrainedControls = allowAuthorizationControl && repositoryModel.authorizationControl.equals(AuthorizationControl.NAMED);
 				usersPalette.setEnabled(allowFineGrainedControls);
@@ -474,6 +478,7 @@
 				}
 				
 				target.addComponent(authorizationControl);
+				target.addComponent(verifyCommitter);
 				target.addComponent(usersPalette);
 				target.addComponent(teamsPalette);
 			}

--
Gitblit v1.9.1