From c015bc6c5e9fd539ec3d0e79f1ae1eb10d34d9d6 Mon Sep 17 00:00:00 2001
From: Neil Bartlett <njbartlett@gmail.com>
Date: Sat, 26 Mar 2016 20:42:52 +0000
Subject: [PATCH] [template] Show help for Github repos

Signed-off-by: Neil Bartlett <njbartlett@gmail.com>
---
 .../shared/TemplateSelectionWizardPage.java   | 10 ++++++++
 bndtools.repository.base/_plugin.xml          |  2 +-
 .../templating/jgit/GitCloneTemplate.java     |  2 +-
 .../jgit/GitCloneTemplateParams.java          |  1 +
 .../jgit/GitHubWorkspaceTemplateLoader.java   | 23 +++++++++++++++----
 5 files changed, 32 insertions(+), 6 deletions(-)

diff --git a/bndtools.core/src/org/bndtools/core/ui/wizards/shared/TemplateSelectionWizardPage.java b/bndtools.core/src/org/bndtools/core/ui/wizards/shared/TemplateSelectionWizardPage.java
index 0a452df91..07025f2f8 100644
--- a/bndtools.core/src/org/bndtools/core/ui/wizards/shared/TemplateSelectionWizardPage.java
+++ b/bndtools.core/src/org/bndtools/core/ui/wizards/shared/TemplateSelectionWizardPage.java
@@ -236,6 +236,16 @@ public void linkActivated(HyperlinkEvent ev) {
                 }
             }
         });
+        txtDescription.getFormText().addHyperlinkListener(new HyperlinkAdapter() {
+            @Override
+            public void linkActivated(HyperlinkEvent ev) {
+                try {
+                    PlatformUI.getWorkbench().getBrowserSupport().getExternalBrowser().openURL(new URL((String) ev.getHref()));
+                } catch (Exception ex) {
+                    log.log(new Status(IStatus.ERROR, Plugin.PLUGIN_ID, 0, "Browser open error", ex));
+                }
+            }
+        });
     }
 
     private class LoadTemplatesJob implements IRunnableWithProgress {
diff --git a/bndtools.repository.base/_plugin.xml b/bndtools.repository.base/_plugin.xml
index dc8542e52..71d350f83 100644
--- a/bndtools.repository.base/_plugin.xml
+++ b/bndtools.repository.base/_plugin.xml
@@ -5,7 +5,7 @@
    <extension point="bndtools.core.workspaceTemplates">
        <template
            name="Bndtools Bundle Hub"
-           description="Built-in Template – Deprecated"
+           description="Built-in Template – DEPRECATED"
            category="Legacy Templates"
            class="org.bndtools.wizards.workspace.legacy.ExtensionWorkspaceTemplate"
            icon="icons/bndtools-logo-16x16.png"
diff --git a/org.bndtools.templating.gitrepo/src/org/bndtools/templating/jgit/GitCloneTemplate.java b/org.bndtools.templating.gitrepo/src/org/bndtools/templating/jgit/GitCloneTemplate.java
index fa70f4021..f74078e9b 100644
--- a/org.bndtools.templating.gitrepo/src/org/bndtools/templating/jgit/GitCloneTemplate.java
+++ b/org.bndtools.templating.gitrepo/src/org/bndtools/templating/jgit/GitCloneTemplate.java
@@ -145,7 +145,7 @@ public URI getIcon() {
 
     @Override
     public URI getHelpContent() {
-        return null;
+        return params.helpUri;
     }
 
     private static ResourceMap toResourceMap(File baseDir, FileFilter filter) {
diff --git a/org.bndtools.templating.gitrepo/src/org/bndtools/templating/jgit/GitCloneTemplateParams.java b/org.bndtools.templating.gitrepo/src/org/bndtools/templating/jgit/GitCloneTemplateParams.java
index 8846c57b1..2a8c890e4 100644
--- a/org.bndtools.templating.gitrepo/src/org/bndtools/templating/jgit/GitCloneTemplateParams.java
+++ b/org.bndtools.templating.gitrepo/src/org/bndtools/templating/jgit/GitCloneTemplateParams.java
@@ -11,5 +11,6 @@ public class GitCloneTemplateParams {
     String name;
     String category;
     URI iconUri;
+    URI helpUri;
 
 }
diff --git a/org.bndtools.templating.gitrepo/src/org/bndtools/templating/jgit/GitHubWorkspaceTemplateLoader.java b/org.bndtools.templating.gitrepo/src/org/bndtools/templating/jgit/GitHubWorkspaceTemplateLoader.java
index 50f47a213..d753cf8fc 100644
--- a/org.bndtools.templating.gitrepo/src/org/bndtools/templating/jgit/GitHubWorkspaceTemplateLoader.java
+++ b/org.bndtools.templating.gitrepo/src/org/bndtools/templating/jgit/GitHubWorkspaceTemplateLoader.java
@@ -22,6 +22,7 @@
 
 import aQute.bnd.header.Attrs;
 import aQute.bnd.header.Parameters;
+import aQute.lib.base64.Base64;
 import aQute.service.reporter.Reporter;
 
 @Component(name = "org.bndtools.core.templating.workspace.github", property = {
@@ -59,7 +60,7 @@ public Promise<List<Template>> findTemplates(String type, Reporter reporter) {
             final Attrs attribs = entry.getValue();
 
             try {
-                GitHub gitHub = new GitHub(cache, executor);
+                final GitHub gitHub = new GitHub(cache, executor);
                 promises.add(gitHub.loadRepoDetails(repo).then(new Success<GithubRepoDetailsDTO,Template>() {
                     @Override
                     public Promise<Template> call(Promise<GithubRepoDetailsDTO> resolved) throws Exception {
@@ -77,7 +78,7 @@ public Promise<Template> call(Promise<GithubRepoDetailsDTO> resolved) throws Exc
                         if (name == null)
                             name = repo;
                         String branch = attribs.get("branch");
-                        GitCloneTemplateParams params = new GitCloneTemplateParams();
+                        final GitCloneTemplateParams params = new GitCloneTemplateParams();
                         params.cloneUrl = detailsDTO.clone_url;
                         if (branch != null)
                             params.branch = branch;
@@ -87,9 +88,13 @@ public Promise<Template> call(Promise<GithubRepoDetailsDTO> resolved) throws Exc
                         params.category = "GitHub";
                         params.iconUri = avatarUri;
 
-                        Template template = new GitCloneTemplate(params);
-                        return Promises.resolved(template);
+                        if (detailsDTO.html_url != null) {
+                            params.helpUri = createHelpUri(repo, detailsDTO.html_url);
+                        }
+
+                        return Promises.<Template> resolved(new GitCloneTemplate(params));
                     }
+
                 }));
             } catch (Exception e) {
                 reporter.exception(e, "Error loading template from Github repository %s", repo);
@@ -99,4 +104,14 @@ public Promise<Template> call(Promise<GithubRepoDetailsDTO> resolved) throws Exc
         return Promises.all(promises);
     }
 
+    private static URI createHelpUri(String repoName, String linkUri) {
+        try {
+            String formText = String.format("<form><p>This is a GitHub template using the repository %s. See the <a href='%s'>project homepage</a> for more information.</p></form>", repoName, new URI(linkUri));
+            String encodedFormText = Base64.encodeBase64(formText.getBytes("UTF-8"));
+            return new URI("data:text/xml;base64," + encodedFormText);
+        } catch (Exception e) {
+            return null;
+        }
+    }
+
 }