SONAR-6433 Stop supporting the concept of "group/ecosystem" of plugins

diff --git a/pom.xml b/pom.xml
index 6ec2d1c6be87ffbb62d799418794f2045a8e002d..264fa7aee2fbe4387dba067b2176c7525d56de20 100644 (file)
--- a/pom.xml
+++ b/pom.xml
@@ -65,7 +65,7 @@
   </prerequisites>
 
   <properties>
-    <sonarUpdateCenter.version>1.11</sonarUpdateCenter.version>
+    <sonarUpdateCenter.version>1.13-SNAPSHOT</sonarUpdateCenter.version>
     <sonarJava.version>3.0</sonarJava.version>
     <sonarGit.version>1.0</sonarGit.version>
     <sonarSvn.version>1.0</sonarSvn.version>

diff --git a/server/sonar-server/src/main/java/org/sonar/server/plugins/PluginReferentialMetadataConverter.java b/server/sonar-server/src/main/java/org/sonar/server/plugins/PluginReferentialMetadataConverter.java
index 77e243e33775eb9ead9bbe752f147f9fb2e1bab4..30646c02a953d08db0e6ab759f84c4e1f23071b1 100644 (file)
--- a/server/sonar-server/src/main/java/org/sonar/server/plugins/PluginReferentialMetadataConverter.java
+++ b/server/sonar-server/src/main/java/org/sonar/server/plugins/PluginReferentialMetadataConverter.java
@@ -63,7 +63,6 @@ public class PluginReferentialMetadataConverter {
     pluginManifest.setHomepage(metadata.getHomepage());
     pluginManifest.setIssueTrackerUrl(metadata.getIssueTrackerUrl());
     pluginManifest.setBasePlugin(metadata.getBasePlugin());
-    pluginManifest.setParent(metadata.getParent());
     pluginManifest.setRequirePlugins(metadata.getRequiredPlugins().toArray(new String []{}));
     return pluginManifest;
   }

diff --git a/server/sonar-web/src/main/webapp/WEB-INF/app/views/updatecenter/available.html.erb b/server/sonar-web/src/main/webapp/WEB-INF/app/views/updatecenter/available.html.erb
index b3f462b6b02ed77e48cbfe065186daff611434e0..3ffd2be120595882e8cafe78e69cb301e86c9f0b 100644 (file)
--- a/server/sonar-web/src/main/webapp/WEB-INF/app/views/updatecenter/available.html.erb
+++ b/server/sonar-web/src/main/webapp/WEB-INF/app/views/updatecenter/available.html.erb
@@ -50,21 +50,6 @@
                 <div id="detail-<%= plugin.getKey() -%>" style="display:none">
                   <table class="spaced width100">
 
-                    <% index = 0 %>
-                    <% release.children.each do |child|
-                      child_plugin = child.artifact
-                    %>
-                      <tr>
-                        <% if index == 0 %>
-                          <td class="thin nowrap"><b>Includes:</b></td>
-                        <% else %>
-                          <td>&nbsp;</td>
-                        <% end %>
-                        <td><span><%= child_plugin.name -%></span> : <span><%= child_plugin.description -%></span></td>
-                      </tr>
-                      <% index = index + 1 %>
-                    <% end %>
-
                     <% release.outgoingDependencies.each_with_index do |outgoing_release, index| %>
                       <tr>
                         <% if index == 0 %>

diff --git a/server/sonar-web/src/main/webapp/WEB-INF/app/views/updatecenter/index.html.erb b/server/sonar-web/src/main/webapp/WEB-INF/app/views/updatecenter/index.html.erb
index 4b45615e0914b1bc2b2b8aedc0070db7444a8fba..aaf243c8278386ab5b116c1cce88b5978b650843 100644 (file)
--- a/server/sonar-web/src/main/webapp/WEB-INF/app/views/updatecenter/index.html.erb
+++ b/server/sonar-web/src/main/webapp/WEB-INF/app/views/updatecenter/index.html.erb
@@ -41,21 +41,6 @@
 
                 <table class="spaced width100">
 
-                  <% index = 0 %>
-                  <% release.children.each do |child|
-                    child_plugin = @installed_plugin_referential.findPlugin(child.key)
-                  %>
-                    <tr>
-                      <% if index == 0 %>
-                        <td class="thin nowrap"><b>Includes:</b></td>
-                      <% else %>
-                        <td>&nbsp;</td>
-                      <% end %>
-                      <td><span><%= child_plugin.name -%></span> : <span><%= child_plugin.description -%></span></td>
-                    </tr>
-                    <% index = index + 1 %>
-                  <% end %>
-
                   <% release.outgoingDependencies.each_with_index do |outgoing_release, index| %>
                     <tr>
                       <% if index == 0 %>

diff --git a/sonar-core/src/main/java/org/sonar/core/plugins/DefaultPluginMetadata.java b/sonar-core/src/main/java/org/sonar/core/plugins/DefaultPluginMetadata.java
index 35cef375c332defdaa9f2a8046bde3ab4a39f4e0..9005f7a09174672f7366b72508e3faf4605bda23 100644 (file)
--- a/sonar-core/src/main/java/org/sonar/core/plugins/DefaultPluginMetadata.java
+++ b/sonar-core/src/main/java/org/sonar/core/plugins/DefaultPluginMetadata.java
@@ -50,7 +50,6 @@ public class DefaultPluginMetadata implements PluginMetadata, Comparable<PluginM
   private String basePlugin;
   private boolean core;
   private String implementationBuild;
-  private String parent;
   private List<String> requiredPlugins;
 
   private DefaultPluginMetadata() {
@@ -185,16 +184,6 @@ public class DefaultPluginMetadata implements PluginMetadata, Comparable<PluginM
     return this;
   }
 
-  @Override
-  public String getParent() {
-    return parent;
-  }
-
-  public DefaultPluginMetadata setParent(String parent) {
-    this.parent = parent;
-    return this;
-  }
-
   @Override
   public List<String> getRequiredPlugins() {
     return ImmutableList.copyOf(requiredPlugins);
@@ -283,6 +272,11 @@ public class DefaultPluginMetadata implements PluginMetadata, Comparable<PluginM
     return this;
   }
 
+  @Override
+  public String getParent() {
+    return null;
+  }
+
   @Override
   public boolean equals(Object o) {
     if (this == o) {

diff --git a/sonar-core/src/main/java/org/sonar/core/plugins/PluginJarInstaller.java b/sonar-core/src/main/java/org/sonar/core/plugins/PluginJarInstaller.java
index 0f9a06d8e22dac1d71ba7408515faaaf60d0af9b..301410df7751af0778e74f2cf451587925dd5d6a 100644 (file)
--- a/sonar-core/src/main/java/org/sonar/core/plugins/PluginJarInstaller.java
+++ b/sonar-core/src/main/java/org/sonar/core/plugins/PluginJarInstaller.java
@@ -78,7 +78,6 @@ public abstract class PluginJarInstaller implements BatchComponent, ServerCompon
       metadata.setUseChildFirstClassLoader(manifest.isUseChildFirstClassLoader());
       metadata.setBasePlugin(manifest.getBasePlugin());
       metadata.setImplementationBuild(manifest.getImplementationBuild());
-      metadata.setParent(manifest.getParent());
       metadata.setRequiredPlugins(Arrays.asList(manifest.getRequirePlugins()));
       metadata.setCore(isCore);
       return metadata;

diff --git a/sonar-core/src/test/java/org/sonar/core/plugins/DefaultPluginMetadataTest.java b/sonar-core/src/test/java/org/sonar/core/plugins/DefaultPluginMetadataTest.java
index f738ada38c72acb4f9e02b207ba7758b6a5708b2..98cdf2698b6d6b1700e2574aee0e6d0dc3da4b19 100644 (file)
--- a/sonar-core/src/test/java/org/sonar/core/plugins/DefaultPluginMetadataTest.java
+++ b/sonar-core/src/test/java/org/sonar/core/plugins/DefaultPluginMetadataTest.java
@@ -36,7 +36,6 @@ public class DefaultPluginMetadataTest {
   public void testGettersAndSetters() {
     DefaultPluginMetadata metadata = DefaultPluginMetadata.create(new File("sonar-checkstyle-plugin.jar"));
     metadata.setKey("checkstyle")
-      .setParent("java")
       .setLicense("LGPL")
       .setDescription("description")
       .setHomepage("http://home")
@@ -51,7 +50,7 @@ public class DefaultPluginMetadataTest {
       .setImplementationBuild("abcdef");
 
     assertThat(metadata.getKey()).isEqualTo("checkstyle");
-    assertThat(metadata.getParent()).isEqualTo("java");
+    assertThat(metadata.getParent()).isNull();
     assertThat(metadata.getLicense()).isEqualTo("LGPL");
     assertThat(metadata.getDescription()).isEqualTo("description");
     assertThat(metadata.getHomepage()).isEqualTo("http://home");

diff --git a/sonar-core/src/test/java/org/sonar/core/plugins/PluginJarInstallerTest.java b/sonar-core/src/test/java/org/sonar/core/plugins/PluginJarInstallerTest.java
index 3fc177f99d42c63957e21cf07297f77fc5756759..b66bf864085412a6f35e705a87afaad261e298c3 100644 (file)
--- a/sonar-core/src/test/java/org/sonar/core/plugins/PluginJarInstallerTest.java
+++ b/sonar-core/src/test/java/org/sonar/core/plugins/PluginJarInstallerTest.java
@@ -81,14 +81,6 @@ public class PluginJarInstallerTest {
     assertThat(metadata.getBasePlugin()).isEqualTo("checkstyle");
   }
 
-  @Test
-  public void should_extract_parent_information() throws IOException {
-    DefaultPluginMetadata metadata = extractor.extractMetadata(getFileFromCache("fake1bis-plugin-1.0.jar"), true);
-
-    assertThat(metadata.getKey()).isEqualTo("fake1bis");
-    assertThat(metadata.getParent()).isEqualTo("fake1");
-  }
-
   @Test
   public void should_extract_requires_plugin_information() throws IOException {
     DefaultPluginMetadata metadata = extractor.extractMetadata(getFileFromCache("fake2-plugin-1.1.jar"), true);

diff --git a/sonar-core/src/test/resources/org/sonar/core/plugins/fake1bis-plugin-1.0.jar b/sonar-core/src/test/resources/org/sonar/core/plugins/fake1bis-plugin-1.0.jar
deleted file mode 100644 (file)
index b2323cc..0000000
Binary files a/sonar-core/src/test/resources/org/sonar/core/plugins/fake1bis-plugin-1.0.jar and /dev/null differ

diff --git a/sonar-plugin-api/src/main/java/org/sonar/api/platform/PluginMetadata.java b/sonar-plugin-api/src/main/java/org/sonar/api/platform/PluginMetadata.java
index 29a84f84a531b44ed5fbc57453bdd598d2fa5076..2afd2312c3c22bd5e9984b5ef93639041837e52c 100644 (file)
--- a/sonar-plugin-api/src/main/java/org/sonar/api/platform/PluginMetadata.java
+++ b/sonar-plugin-api/src/main/java/org/sonar/api/platform/PluginMetadata.java
@@ -19,6 +19,7 @@
  */
 package org.sonar.api.platform;
 
+import javax.annotation.CheckForNull;
 import java.io.File;
 import java.util.List;
 
@@ -57,6 +58,12 @@ public interface PluginMetadata {
 
   String getBasePlugin();
 
+  /**
+   * Always return <code>null</code> since version 5.2
+   * @deprecated in 5.2. Concept of parent relationship is removed. See https://jira.codehaus.org/browse/SONAR-6433
+   */
+  @Deprecated
+  @CheckForNull
   String getParent();
 
   List<String> getRequiredPlugins();