You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@servicemix.apache.org by gn...@apache.org on 2008/09/10 18:52:53 UTC
svn commit: r693893 - in
/servicemix/smx4/kernel/branches/kernel-1.0.x/gshell/gshell-features/src/main/java/org/apache/servicemix/gshell/features:
Feature.java internal/FeatureImpl.java internal/FeaturesServiceImpl.java
Author: gnodet
Date: Wed Sep 10 09:52:52 2008
New Revision: 693893
URL: http://svn.apache.org/viewvc?rev=693893&view=rev
Log:
SMX4KNL-82: The installation state of a feature is not displayed correctly after a restart when using the "features list" command
Modified:
servicemix/smx4/kernel/branches/kernel-1.0.x/gshell/gshell-features/src/main/java/org/apache/servicemix/gshell/features/Feature.java
servicemix/smx4/kernel/branches/kernel-1.0.x/gshell/gshell-features/src/main/java/org/apache/servicemix/gshell/features/internal/FeatureImpl.java
servicemix/smx4/kernel/branches/kernel-1.0.x/gshell/gshell-features/src/main/java/org/apache/servicemix/gshell/features/internal/FeaturesServiceImpl.java
Modified: servicemix/smx4/kernel/branches/kernel-1.0.x/gshell/gshell-features/src/main/java/org/apache/servicemix/gshell/features/Feature.java
URL: http://svn.apache.org/viewvc/servicemix/smx4/kernel/branches/kernel-1.0.x/gshell/gshell-features/src/main/java/org/apache/servicemix/gshell/features/Feature.java?rev=693893&r1=693892&r2=693893&view=diff
==============================================================================
--- servicemix/smx4/kernel/branches/kernel-1.0.x/gshell/gshell-features/src/main/java/org/apache/servicemix/gshell/features/Feature.java (original)
+++ servicemix/smx4/kernel/branches/kernel-1.0.x/gshell/gshell-features/src/main/java/org/apache/servicemix/gshell/features/Feature.java Wed Sep 10 09:52:52 2008
@@ -32,6 +32,4 @@
Map<String, Map<String, String>> getConfigurations();
- boolean isInstalled();
-
}
Modified: servicemix/smx4/kernel/branches/kernel-1.0.x/gshell/gshell-features/src/main/java/org/apache/servicemix/gshell/features/internal/FeatureImpl.java
URL: http://svn.apache.org/viewvc/servicemix/smx4/kernel/branches/kernel-1.0.x/gshell/gshell-features/src/main/java/org/apache/servicemix/gshell/features/internal/FeatureImpl.java?rev=693893&r1=693892&r2=693893&view=diff
==============================================================================
--- servicemix/smx4/kernel/branches/kernel-1.0.x/gshell/gshell-features/src/main/java/org/apache/servicemix/gshell/features/internal/FeatureImpl.java (original)
+++ servicemix/smx4/kernel/branches/kernel-1.0.x/gshell/gshell-features/src/main/java/org/apache/servicemix/gshell/features/internal/FeatureImpl.java Wed Sep 10 09:52:52 2008
@@ -32,8 +32,6 @@
private List<String> dependencies = new ArrayList<String>();
private List<String> bundles = new ArrayList<String>();
private Map<String, Map<String,String>> configs = new HashMap<String, Map<String,String>>();
- private boolean installed = false;
-
public FeatureImpl(String name) {
this.name = name;
@@ -67,12 +65,4 @@
configs.put(name, properties);
}
- public boolean isInstalled() {
- return installed;
- }
-
- public void setInstalled(boolean installed) {
- this.installed = installed;
- }
-
}
Modified: servicemix/smx4/kernel/branches/kernel-1.0.x/gshell/gshell-features/src/main/java/org/apache/servicemix/gshell/features/internal/FeaturesServiceImpl.java
URL: http://svn.apache.org/viewvc/servicemix/smx4/kernel/branches/kernel-1.0.x/gshell/gshell-features/src/main/java/org/apache/servicemix/gshell/features/internal/FeaturesServiceImpl.java?rev=693893&r1=693892&r2=693893&view=diff
==============================================================================
--- servicemix/smx4/kernel/branches/kernel-1.0.x/gshell/gshell-features/src/main/java/org/apache/servicemix/gshell/features/internal/FeaturesServiceImpl.java (original)
+++ servicemix/smx4/kernel/branches/kernel-1.0.x/gshell/gshell-features/src/main/java/org/apache/servicemix/gshell/features/internal/FeaturesServiceImpl.java Wed Sep 10 09:52:52 2008
@@ -137,7 +137,7 @@
}
public void installFeature(String name) throws Exception {
- FeatureImpl f = (FeatureImpl) getFeature(name);
+ Feature f = getFeature(name);
if (f == null) {
throw new Exception("No feature named '" + name + "' available");
}
@@ -163,7 +163,6 @@
bundles.add(b.getBundleId());
}
installed.put(name, bundles);
- f.setInstalled(true);
saveState();
}
@@ -221,7 +220,7 @@
Collection<String> features = new ArrayList<String>();
for (Repository repo : repositories.values()) {
for (Feature f : repo.getFeatures()) {
- String installStatus = f.isInstalled() ? "installed " : "uninstalled";
+ String installStatus = installed.containsKey(f.getName()) ? "installed " : "uninstalled";
features.add("[" + installStatus + "] " + f.getName());
}
}