X-Git-Url: https://git.argeo.org/?a=blobdiff_plain;f=org.argeo.cms%2Fsrc%2Forg%2Fargeo%2Fcms%2Finternal%2Fkernel%2FDataModels.java;h=19117a892b2d5bb414e83e664a38f92ab1bf16f2;hb=1b09095425a7c067c097eb91bed5b4a4a33f0dc8;hp=48b6b7fc75f3476caab9b7a9be218feca546e8cf;hpb=3a3cc97dfb568ba7ebfcb03e9816e628fd1fdf1f;p=lgpl%2Fargeo-commons.git

diff --git a/org.argeo.cms/src/org/argeo/cms/internal/kernel/DataModels.java b/org.argeo.cms/src/org/argeo/cms/internal/kernel/DataModels.java
index 48b6b7fc7..19117a892 100644
--- a/org.argeo.cms/src/org/argeo/cms/internal/kernel/DataModels.java
+++ b/org.argeo.cms/src/org/argeo/cms/internal/kernel/DataModels.java
@@ -27,7 +27,7 @@ class DataModels implements BundleListener {
 
 	public DataModels(BundleContext bc) {
 		for (Bundle bundle : bc.getBundles())
-			processBundle(bundle);
+			processBundle(bundle, null);
 		bc.addBundleListener(this);
 	}
 
@@ -45,7 +45,7 @@ class DataModels implements BundleListener {
 	@Override
 	public void bundleChanged(BundleEvent event) {
 		if (event.getType() == Bundle.RESOLVED) {
-			processBundle(event.getBundle());
+			processBundle(event.getBundle(), null);
 		} else if (event.getType() == Bundle.UNINSTALLED) {
 			BundleWiring wiring = event.getBundle().adapt(BundleWiring.class);
 			List<BundleCapability> providedDataModels = wiring.getCapabilities(CMS_DATA_MODEL_NAMESPACE);
@@ -58,7 +58,9 @@ class DataModels implements BundleListener {
 
 	}
 
-	protected void processBundle(Bundle bundle) {
+	protected void processBundle(Bundle bundle, List<Bundle> scannedBundles) {
+		if (scannedBundles != null && scannedBundles.contains(bundle))
+			throw new IllegalStateException("Cycle in CMS data model requirements for " + bundle);
 		BundleWiring wiring = bundle.adapt(BundleWiring.class);
 		if (wiring == null) {
 			int bundleState = bundle.getState();
@@ -77,7 +79,12 @@ class DataModels implements BundleListener {
 		List<BundleWire> requiredDataModels = wiring.getRequiredWires(CMS_DATA_MODEL_NAMESPACE);
 		// process requirements first
 		for (BundleWire bundleWire : requiredDataModels) {
-			processBundle(bundleWire.getProvider().getBundle());
+			List<Bundle> nextScannedBundles = new ArrayList<>();
+			if (scannedBundles != null)
+				nextScannedBundles.addAll(scannedBundles);
+			nextScannedBundles.add(bundle);
+			Bundle providerBundle = bundleWire.getProvider().getBundle();
+			processBundle(providerBundle, nextScannedBundles);
 		}
 		for (BundleCapability bundleCapability : providedDataModels) {
 			String name = (String) bundleCapability.getAttributes().get(DataModelNamespace.NAME);