From 8c99a7929b48b4c4a33c80d0b6646f6e6bb563b5 Mon Sep 17 00:00:00 2001 From: James Moger <james.moger@gitblit.com> Date: Wed, 03 Jul 2013 21:35:02 -0400 Subject: [PATCH] Documentation --- src/main/java/com/gitblit/wicket/panels/RepositoryUrlPanel.java | 26 ++++++++++---------------- 1 files changed, 10 insertions(+), 16 deletions(-) diff --git a/src/main/java/com/gitblit/wicket/panels/RepositoryUrlPanel.java b/src/main/java/com/gitblit/wicket/panels/RepositoryUrlPanel.java index b28599f..3711ca9 100644 --- a/src/main/java/com/gitblit/wicket/panels/RepositoryUrlPanel.java +++ b/src/main/java/com/gitblit/wicket/panels/RepositoryUrlPanel.java @@ -162,7 +162,7 @@ } // access restriction icon and tooltip - if (isGitblitServingRepositories()) { + if (GitBlit.isServingRepositories()) { switch (repository.accessRestriction) { case NONE: urlPanel.add(WicketUtils.newClearPixel("accessRestrictionIcon").setVisible(false)); @@ -228,18 +228,16 @@ final GitClientApplication clientApp = item.getModelObject(); // filter the urls for the client app - List<RepositoryUrl> urls; - if (clientApp.minimumPermission == null) { - // client app does not specify minimum access permission - urls = repositoryUrls; - } else { - urls = new ArrayList<RepositoryUrl>(); - for (RepositoryUrl repoUrl : repositoryUrls) { - if (repoUrl.permission == null) { - // external permissions, assume it is satisfactory + List<RepositoryUrl> urls = new ArrayList<RepositoryUrl>(); + for (RepositoryUrl repoUrl : repositoryUrls) { + if (clientApp.minimumPermission == null || repoUrl.permission == null) { + // no minimum permission or external permissions, assume it is satisfactory + if (clientApp.supportsTransport(repoUrl.url)) { urls.add(repoUrl); - } else if (repoUrl.permission.atLeast(clientApp.minimumPermission)) { - // repo url meets minimum permission requirement + } + } else if (repoUrl.permission.atLeast(clientApp.minimumPermission)) { + // repo url meets minimum permission requirement + if (clientApp.supportsTransport(repoUrl.url)) { urls.add(repoUrl); } } @@ -333,10 +331,6 @@ protected String substitute(String pattern, String repoUrl, String baseUrl) { return pattern.replace("${repoUrl}", repoUrl).replace("${baseUrl}", baseUrl); - } - - protected boolean isGitblitServingRepositories() { - return GitBlit.getBoolean(Keys.git.enableGitServlet, true) || (GitBlit.getInteger(Keys.git.daemonPort, 0) > 0); } protected Label createPermissionBadge(String wicketId, RepositoryUrl repoUrl) { -- Gitblit v1.9.1