Merge pull request #281 from mrjoel/mrjoel-ticketswithexternalurl

James Moger

2015-07-04 7c4d074a3cc3c204a135ae9aa69b5314980ea8b4

 src/main/java/com/gitblit/wicket/panels/RepositoryUrlPanel.java

@@ -84,7 +84,7 @@
      // grab primary url from the top of the list

      primaryUrl = repositoryUrls.size() == 0 ? null : repositoryUrls.get(0);



      boolean canClone = primaryUrl != null && ((primaryUrl.permission == null) || primaryUrl.permission.atLeast(AccessPermission.CLONE));

      boolean canClone = primaryUrl != null && (!primaryUrl.hasPermission() || primaryUrl.permission.atLeast(AccessPermission.CLONE));



      if (repositoryUrls.size() == 0 || !canClone) {

         // no urls, nothing to show.

@@ -145,7 +145,7 @@
               fragment.add(content);

               item.add(fragment);



               Label permissionLabel = new Label("permission", repoUrl.isExternal() ? externalPermission : repoUrl.permission.toString());

               Label permissionLabel = new Label("permission", repoUrl.hasPermission() ? repoUrl.permission.toString() : externalPermission);

               WicketUtils.setPermissionClass(permissionLabel, repoUrl.permission);

               String tooltip = getProtocolPermissionDescription(repository, repoUrl);

               WicketUtils.setHtmlTooltip(permissionLabel, tooltip);

@@ -201,7 +201,7 @@


      urlPanel.add(new Label("primaryUrl", primaryUrl.url).setRenderBodyOnly(true));



      Label permissionLabel = new Label("primaryUrlPermission", primaryUrl.isExternal() ? externalPermission : primaryUrl.permission.toString());

      Label permissionLabel = new Label("primaryUrlPermission", primaryUrl.hasPermission() ? primaryUrl.permission.toString() : externalPermission);

      String tooltip = getProtocolPermissionDescription(repository, primaryUrl);

      WicketUtils.setHtmlTooltip(permissionLabel, tooltip);

      urlPanel.add(permissionLabel);

@@ -234,8 +234,8 @@
            // filter the urls for the client app

            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.minimumPermission == null || !repoUrl.hasPermission()) {

                  // no minimum permission or untracked permissions, assume it is satisfactory

                  if (clientApp.supportsTransport(repoUrl.url)) {

                     urls.add(repoUrl);

                  }

@@ -339,7 +339,7 @@
   }



   protected Label createPermissionBadge(String wicketId, RepositoryUrl repoUrl) {

      Label permissionLabel = new Label(wicketId, repoUrl.isExternal() ? externalPermission : repoUrl.permission.toString());

      Label permissionLabel = new Label(wicketId, repoUrl.hasPermission() ? repoUrl.permission.toString() : externalPermission);

      WicketUtils.setPermissionClass(permissionLabel, repoUrl.permission);

      String tooltip = getProtocolPermissionDescription(repository, repoUrl);

      WicketUtils.setHtmlTooltip(permissionLabel, tooltip);

@@ -369,18 +369,7 @@
         RepositoryUrl repoUrl) {

      if (!urlPermissionsMap.containsKey(repoUrl.url)) {

         String note;

         if (repoUrl.isExternal()) {

            String protocol;

            int protocolIndex = repoUrl.url.indexOf("://");

            if (protocolIndex > -1) {

               // explicit protocol specified

               protocol = repoUrl.url.substring(0, protocolIndex);

            } else {

               // implicit SSH url

               protocol = "ssh";

            }

            note = MessageFormat.format(getString("gb.externalPermissions"), protocol);

         } else {

         if (repoUrl.hasPermission()) {

            note = null;

            String key;

            switch (repoUrl.permission) {

@@ -411,6 +400,17 @@
               String description = MessageFormat.format(pattern, repoUrl.permission.toString());

               note = description;

            }

         } else {

            String protocol;

            int protocolIndex = repoUrl.url.indexOf("://");

            if (protocolIndex > -1) {

               // explicit protocol specified

               protocol = repoUrl.url.substring(0, protocolIndex);

            } else {

               // implicit SSH url

               protocol = "ssh";

            }

            note = MessageFormat.format(getString("gb.externalPermissions"), protocol);

         }

         urlPermissionsMap.put(repoUrl.url, note);

      }