From 4ce0324e0f7531327666de7c8c92392988b1e8d6 Mon Sep 17 00:00:00 2001
From: Bruno Sinou <bsinou@argeo.org>
Date: Wed, 14 May 2014 19:02:30 +0000
Subject: [PATCH 1/1] Fix distribution editor after further testing with maven
 produced releases

git-svn-id: https://svn.argeo.org/slc/trunk@6983 4cfe0d0a-d680-48aa-b62c-e0a02a3f76cc
---
 .../ModularDistVersionOverviewPage.java       | 53 +++++++++++++------
 1 file changed, 36 insertions(+), 17 deletions(-)

diff --git a/plugins/org.argeo.slc.client.ui.dist/src/main/java/org/argeo/slc/client/ui/dist/editors/ModularDistVersionOverviewPage.java b/plugins/org.argeo.slc.client.ui.dist/src/main/java/org/argeo/slc/client/ui/dist/editors/ModularDistVersionOverviewPage.java
index 069ce94d5..d2b77497a 100644
--- a/plugins/org.argeo.slc.client.ui.dist/src/main/java/org/argeo/slc/client/ui/dist/editors/ModularDistVersionOverviewPage.java
+++ b/plugins/org.argeo.slc.client.ui.dist/src/main/java/org/argeo/slc/client/ui/dist/editors/ModularDistVersionOverviewPage.java
@@ -44,6 +44,8 @@ import org.argeo.slc.client.ui.dist.DistImages;
 import org.argeo.slc.client.ui.dist.commands.OpenModuleEditor;
 import org.argeo.slc.client.ui.dist.utils.AbstractHyperlinkListener;
 import org.argeo.slc.client.ui.dist.utils.NodeViewerComparator;
+import org.argeo.slc.client.ui.specific.OpenJcrFile;
+import org.argeo.slc.client.ui.specific.OpenJcrFileCmdId;
 import org.argeo.slc.jcr.SlcNames;
 import org.argeo.slc.jcr.SlcTypes;
 import org.argeo.slc.repo.RepoConstants;
@@ -239,8 +241,8 @@ public class ModularDistVersionOverviewPage extends FormPage implements
 			if (!modularDistribution.getSession().nodeExists(srcPath)) {
 				createLT(parent, "Sources", "N/A");
 			} else {
-				Node sourcesNode = modularDistribution.getSession().getNode(
-						srcPath);
+				final Node sourcesNode = modularDistribution.getSession()
+						.getNode(srcPath);
 
 				String srcName = null;
 				if (sourcesNode.hasProperty(SlcNames.SLC_SYMBOLIC_NAME))
@@ -251,19 +253,9 @@ public class ModularDistVersionOverviewPage extends FormPage implements
 				Label label = tk.createLabel(parent, "Sources", SWT.RIGHT);
 				label.setLayoutData(new GridData(SWT.RIGHT, SWT.CENTER, false,
 						false));
-				final Hyperlink link = tk.createHyperlink(parent, srcName,
-						SWT.NONE);
-				link.addHyperlinkListener(new AbstractHyperlinkListener() {
-					@Override
-					public void linkActivated(HyperlinkEvent e) {
-						try {
-							System.out.println("CLICK on Sources link");
-						} catch (Exception ex) {
-							throw new SlcException("error opening browser", ex); //$NON-NLS-1$
-						}
-					}
-				});
-
+				Hyperlink link = tk.createHyperlink(parent, srcName, SWT.NONE);
+				link.addHyperlinkListener(new OpenFileLinkListener(sourcesNode
+						.getPath()));
 			}
 		} catch (RepositoryException e) {
 			throw new SlcException("Unable to configure sources link for "
@@ -271,6 +263,33 @@ public class ModularDistVersionOverviewPage extends FormPage implements
 		}
 	}
 
+	private class OpenFileLinkListener extends AbstractHyperlinkListener {
+		final private String path;
+
+		public OpenFileLinkListener(String path) {
+			this.path = path;
+		}
+
+		@Override
+		public void linkActivated(HyperlinkEvent e) {
+			try {
+				ModuleEditorInput editorInput = (ModuleEditorInput) getEditorInput();
+				Map<String, String> params = new HashMap<String, String>();
+				params.put(OpenJcrFile.PARAM_REPO_NODE_PATH,
+						editorInput.getRepoNodePath());
+				params.put(OpenJcrFile.PARAM_REPO_URI, editorInput.getUri());
+				params.put(OpenJcrFile.PARAM_WORKSPACE_NAME,
+						editorInput.getWorkspaceName());
+				params.put(OpenJcrFile.PARAM_FILE_PATH, path);
+
+				String cmdId = (new OpenJcrFileCmdId()).getCmdId();
+				CommandUtils.callCommand(cmdId, params);
+			} catch (Exception ex) {
+				throw new SlcException("error opening browser", ex); //$NON-NLS-1$
+			}
+		}
+	}
+
 	private void populateModuleTablePart(Composite parent) {
 		GridLayout layout = new GridLayout(1, false);
 		layout.marginWidth = layout.horizontalSpacing = 0;
@@ -421,8 +440,8 @@ public class ModularDistVersionOverviewPage extends FormPage implements
 			String category = JcrUtils.get(moduleCoordinates, SLC_CATEGORY);
 			String name = JcrUtils.get(moduleCoordinates, SLC_NAME);
 			String version = JcrUtils.get(moduleCoordinates, SLC_VERSION);
-			Artifact artifact = new DefaultArtifact(category + ":" + name
-					+ ":" +version);
+			Artifact artifact = new DefaultArtifact(category + ":" + name + ":"
+					+ version);
 			String parentPath = MavenConventionsUtils.artifactParentPath(
 					RepoConstants.DEFAULT_ARTIFACTS_BASE_PATH, artifact);
 
-- 
2.39.2