Clean, update comments

git-svn-id: https://svn.argeo.org/slc/trunk@6854 4cfe0d0a-d680-48aa-b62c-e0a02a3f76cc

diff --git a/plugins/org.argeo.slc.client.ui.dist/src/main/java/org/argeo/slc/client/ui/dist/AnonymousDistributionPerspective.java b/plugins/org.argeo.slc.client.ui.dist/src/main/java/org/argeo/slc/client/ui/dist/AnonymousDistributionPerspective.java
index d2b3cfd969a0c4a1c6acfdfa69f7abffecf27c30..8d5a1513dbddc7dddc108e95e17aa600c28a2a8d 100644 (file)
--- a/plugins/org.argeo.slc.client.ui.dist/src/main/java/org/argeo/slc/client/ui/dist/AnonymousDistributionPerspective.java
+++ b/plugins/org.argeo.slc.client.ui.dist/src/main/java/org/argeo/slc/client/ui/dist/AnonymousDistributionPerspective.java
@@ -21,6 +21,7 @@ import org.eclipse.ui.IFolderLayout;
 import org.eclipse.ui.IPageLayout;
 import org.eclipse.ui.IPerspectiveFactory;
 
+/** Simple default perspective that presents public repositories */
 public class AnonymousDistributionPerspective implements IPerspectiveFactory {
 
        public final static String ID = DistPlugin.ID

diff --git a/plugins/org.argeo.slc.client.ui.dist/src/main/java/org/argeo/slc/client/ui/dist/DistConstants.java b/plugins/org.argeo.slc.client.ui.dist/src/main/java/org/argeo/slc/client/ui/dist/DistConstants.java
index 9fed39a021cb2b30b3f80384704f454d97401f4c..52d41c96e8b34b3e81e9441af25f327664f9c65a 100644 (file)
--- a/plugins/org.argeo.slc.client.ui.dist/src/main/java/org/argeo/slc/client/ui/dist/DistConstants.java
+++ b/plugins/org.argeo.slc.client.ui.dist/src/main/java/org/argeo/slc/client/ui/dist/DistConstants.java
@@ -20,15 +20,16 @@ import org.osgi.framework.Constants;
 
 /** Constants used across the application. */
 public interface DistConstants {
+
        /*
         * MISCEALLENEOUS
         */
        public final static String DATE_TIME_FORMAT = "MM/dd/yyyy, HH:mm";
        public final static String DATE_FORMAT = "MM/dd/yyyy";
-       // this should be directly got in JCR
+       // this should be directly retrieved from JCR APIs
        public final static String JCR_IDENTIFIER = "jcr:uuid";
 
-       // FIXME: might be defined in SlcNames
+       // FIXME: should be defined in SlcNames
        public final static String SLC_BUNDLE_NAME = SlcNames.SLC_
                        + Constants.BUNDLE_NAME;
        public final static String SLC_BUNDLE_LICENCE = SlcNames.SLC_

diff --git a/plugins/org.argeo.slc.client.ui.dist/src/main/java/org/argeo/slc/client/ui/dist/DistributionPerspective.java b/plugins/org.argeo.slc.client.ui.dist/src/main/java/org/argeo/slc/client/ui/dist/DistributionPerspective.java
index fa29c8ba895a7eca52e0e4687e6c93392fe6339e..7538e6f37b1078f017e431b5415fe2f7e6880d15 100644 (file)
--- a/plugins/org.argeo.slc.client.ui.dist/src/main/java/org/argeo/slc/client/ui/dist/DistributionPerspective.java
+++ b/plugins/org.argeo.slc.client.ui.dist/src/main/java/org/argeo/slc/client/ui/dist/DistributionPerspective.java
@@ -21,19 +21,13 @@ import org.eclipse.ui.IPageLayout;
 import org.eclipse.ui.IPerspectiveFactory;
 
 /**
- * User interface to manage a set of distributions split into several
- * repositories
+ * Manage a set of software repositories
  */
 public class DistributionPerspective implements IPerspectiveFactory {
 
-//     private RepositoryFactory repositoryFactory;
-//     private Repository nodeRepository;
-
        public final static String ID = DistPlugin.ID + ".distributionPerspective";
 
        public void createInitialLayout(IPageLayout layout) {
-               //initializeModel();
-
                String editorArea = layout.getEditorArea();
                layout.setEditorAreaVisible(true);
                layout.setFixed(false);
@@ -42,51 +36,5 @@ public class DistributionPerspective implements IPerspectiveFactory {
                                0.3f, editorArea);
                main.addView(DistributionsView.ID);
                main.addView("org.eclipse.ui.views.ProgressView");
-
        }
-
-//     private void initializeModel() {
-//             Session nodeSession = null;
-//             try {
-//                     nodeSession = nodeRepository.login();
-//
-//                     Node homeNode = UserJcrUtils.getUserHome(nodeSession);
-//                     if (homeNode == null) // anonymous
-//                             throw new SlcException("User must be authenticated.");
-//
-//                     // make sure base directory is available
-//                     Node repos = JcrUtils.mkdirs(nodeSession, homeNode.getPath()
-//                                     + RepoConstants.REPOSITORIES_BASE_PATH);
-//                     nodeSession.save();
-//
-//                     // register default local java repository
-//                     String alias = RepoConstants.DEFAULT_JAVA_REPOSITORY_ALIAS;
-//                     Repository javaRepository = ArgeoJcrUtils.getRepositoryByAlias(
-//                                     repositoryFactory, alias);
-//                     if (javaRepository != null) {
-//                             if (!repos.hasNode(alias)) {
-//                                     Node repoNode = repos.addNode(alias,
-//                                                     ArgeoTypes.ARGEO_REMOTE_REPOSITORY);
-//                                     repoNode.setProperty(ArgeoNames.ARGEO_URI, "vm:///" + alias);
-//                                     repoNode.addMixin(NodeType.MIX_TITLE);
-//                                     repoNode.setProperty(Property.JCR_TITLE,
-//                                                     RepoConstants.DEFAULT_JAVA_REPOSITORY_LABEL);
-//                                     nodeSession.save();
-//                             }
-//                     }
-//             } catch (RepositoryException e) {
-//                     throw new SlcException("Cannot register repository", e);
-//             } finally {
-//                     JcrUtils.logoutQuietly(nodeSession);
-//             }
-//     }
-
-       // public void setRepositoryFactory(RepositoryFactory repositoryFactory) {
-       // this.repositoryFactory = repositoryFactory;
-       // }
-       //
-       // public void setRepository(Repository nodeRepository) {
-       // this.nodeRepository = nodeRepository;
-       // }
-
 }