Skip to content

Commit 95ecefd

Browse files
author
Yasin Dilekçi
committed
Fix retrieval of bundles
1 parent d8281f9 commit 95ecefd

File tree

3 files changed

+7
-6
lines changed

3 files changed

+7
-6
lines changed

src/main/java/nl/isaac/dotcms/plugin/configuration/ConfigurationService.java

+3-3
Original file line numberDiff line numberDiff line change
@@ -18,8 +18,8 @@
1818
import org.apache.commons.configuration.DefaultConfigurationBuilder.ConfigurationProvider;
1919
import org.apache.commons.configuration.FileConfiguration;
2020
import org.apache.commons.lang.text.StrLookup;
21+
import org.apache.felix.framework.OSGIUtil;
2122
import org.osgi.framework.Bundle;
22-
import org.osgi.framework.FrameworkUtil;
2323

2424
import com.dotmarketing.util.Logger;
2525

@@ -225,7 +225,7 @@ private static CustomConfiguration retrieveFromCacheOrCreateConfiguration(String
225225

226226
if (splitPluginName.length > 2) {
227227
long bundleId = Long.valueOf(splitPluginName[2]);
228-
bundle = FrameworkUtil.getBundle(ConfigurationService.class).getBundleContext().getBundle(bundleId);
228+
bundle = OSGIUtil.getInstance().getBundle(bundleId);
229229
} else {
230230
// If we didn't get a bundleId we need to search for it, somewhat slower, but the result is cached
231231
bundle = searchBundles(splitPluginName[1]);
@@ -286,7 +286,7 @@ private static CustomConfiguration retrieveFromCacheOrCreateConfiguration(String
286286
return conf;
287287
}
288288
private static Bundle searchBundles(String pluginName) {
289-
for (Bundle bundle : FrameworkUtil.getBundle(ConfigurationService.class).getBundleContext().getBundles()) {
289+
for (Bundle bundle : OSGIUtil.getInstance().getBundles()) {
290290
if (pluginName.equals(bundle.getHeaders().get("Bundle-Name"))) {
291291
return bundle;
292292
}

src/main/java/nl/isaac/dotcms/plugin/configuration/viewtool/portlet/admin/ConfigurationAdminViewTool.java

+3-2
Original file line numberDiff line numberDiff line change
@@ -14,9 +14,9 @@
1414
import javax.servlet.http.HttpServletRequest;
1515

1616
import org.apache.commons.configuration.ConfigurationException;
17+
import org.apache.felix.framework.OSGIUtil;
1718
import org.apache.velocity.tools.view.tools.ViewTool;
1819
import org.osgi.framework.Bundle;
19-
import org.osgi.framework.FrameworkUtil;
2020

2121
import com.dotmarketing.beans.Host;
2222
import com.dotmarketing.business.APILocator;
@@ -45,14 +45,15 @@ public List<String> getAllHosts() throws DotDataException, DotSecurityException
4545

4646
public List<String> getAllPlugins(String hostName, HttpServletRequest request) {
4747
List<String> plugins = APILocator.getPluginAPI().getDeployedPluginOrder();
48+
4849
List<String> pluginNames = new ArrayList<>();
4950
for (String pluginName : plugins) {
5051
if (null != tryGetPluginConfiguration(hostName, pluginName, request)) {
5152
pluginNames.add(pluginName);
5253
}
5354
}
5455

55-
for(Bundle bundle: FrameworkUtil.getBundle(getClass()).getBundleContext().getBundles()) {
56+
for(Bundle bundle: OSGIUtil.getInstance().getBundles()) {
5657
String bundleName = bundle.getHeaders().get("Bundle-Name");
5758
if(!bundleName.contains(" ")) {
5859
//Do not show dotCMS' own plugins like "Apache Felix Bundle Repository" etc

src/main/java/nl/isaac/dotcms/util/osgi/MonitoringServlet.java

+1-1
Original file line numberDiff line numberDiff line change
@@ -11,10 +11,10 @@
1111
import javax.servlet.http.HttpServletRequest;
1212
import javax.servlet.http.HttpServletResponse;
1313

14+
import org.apache.commons.io.IOUtils;
1415
import org.apache.velocity.app.VelocityEngine;
1516
import org.apache.velocity.tools.view.context.ChainedContext;
1617

17-
import com.dotcms.repackage.org.apache.commons.io.IOUtils;
1818
import com.dotmarketing.util.VelocityUtil;
1919

2020
public class MonitoringServlet extends HttpServlet {

0 commit comments

Comments
 (0)