X-Git-Url: https://git.argeo.org/?a=blobdiff_plain;f=org.argeo.app.ui%2Fsrc%2Forg%2Fargeo%2Fapp%2Fui%2FSuiteApp.java;h=a8b9b2e26901c966a4ead4d6b8b4b3ed026516ba;hb=f1b87af2e0de38f1f49ab8fe68f9988018914811;hp=1e71179d4f5f288f6a29384253490fc873cb3285;hpb=b4d2b8bfcb043c9339131954a4ccc88fa18f9ac7;p=gpl%2Fargeo-suite.git

diff --git a/org.argeo.app.ui/src/org/argeo/app/ui/SuiteApp.java b/org.argeo.app.ui/src/org/argeo/app/ui/SuiteApp.java
index 1e71179..a8b9b2e 100644
--- a/org.argeo.app.ui/src/org/argeo/app/ui/SuiteApp.java
+++ b/org.argeo.app.ui/src/org/argeo/app/ui/SuiteApp.java
@@ -80,6 +80,7 @@ public class SuiteApp extends AbstractCmsApp implements CmsEventSubscriber {
 	@Deprecated
 	private String defaultThemeId = "org.argeo.app.theme.default";
 
+	// TODO use QName as key for byType
 	private Map<String, RankedObject<SwtUiProvider>> uiProvidersByPid = Collections.synchronizedMap(new HashMap<>());
 	private Map<String, RankedObject<SwtUiProvider>> uiProvidersByType = Collections.synchronizedMap(new HashMap<>());
 	private Map<String, RankedObject<SuiteLayer>> layersByPid = Collections.synchronizedSortedMap(new TreeMap<>());
@@ -208,6 +209,7 @@ public class SuiteApp extends AbstractCmsApp implements CmsEventSubscriber {
 
 			if (cmsView.isAnonymous() && publicBasePath == null) {// internal app, must login
 				ui.logout();
+				ui.setLoginScreen(true);
 				if (headerUiProvider != null)
 					refreshPart(headerUiProvider, ui.getHeader(), context);
 				ui.refreshBelowHeader(false);
@@ -219,6 +221,12 @@ public class SuiteApp extends AbstractCmsApp implements CmsEventSubscriber {
 			} else {
 				if (LOGIN.equals(state))
 					state = null;
+				if (ui.isLoginScreen()) {
+//					if (state == null)
+//						state = ui.getPostLoginState();
+					ui.setLoginScreen(false);
+//					ui.setPostLoginState(null);
+				}
 				CmsSession cmsSession = cmsView.getCmsSession();
 				if (ui.getUserDirNode() == null) {
 					// FIXME NPE on CMSSession when logging in from anonymous
@@ -264,7 +272,7 @@ public class SuiteApp extends AbstractCmsApp implements CmsEventSubscriber {
 				setState(ui, state != null ? state : defaultLayerPid);
 			}
 		} catch (Exception e) {
-			CmsFeedback.show("Unexpected exception", e);
+			CmsFeedback.error("Unexpected exception", e);
 		}
 	}
 
@@ -362,8 +370,9 @@ public class SuiteApp extends AbstractCmsApp implements CmsEventSubscriber {
 				if (byType.containsKey(type))
 					types.add(type);
 			}
-			if (types.size() == 0)
+			if (types.size() == 0) {
 				throw new IllegalArgumentException("No type found for " + content + " (" + objectClasses + ")");
+			}
 			String type = types.iterator().next();
 			if (!byType.containsKey(type))
 				throw new IllegalArgumentException("No component found for " + content + " with type " + type);
@@ -421,6 +430,11 @@ public class SuiteApp extends AbstractCmsApp implements CmsEventSubscriber {
 			return;
 		}
 		SuiteUi suiteUi = (SuiteUi) cmsUi;
+		if (suiteUi.isLoginScreen()) {
+//			suiteUi.setPostLoginState(state);
+			return;
+		}
+
 		Content node = stateToNode(suiteUi, state);
 		if (node == null) {
 			suiteUi.getCmsView().navigateTo(HOME_STATE);
@@ -549,8 +563,8 @@ public class SuiteApp extends AbstractCmsApp implements CmsEventSubscriber {
 					}
 				}
 			} catch (Exception e) {
-				log.error("Cannot handle event " + event, e);
-//			CmsView.getCmsView(ui).exception(e);
+				CmsFeedback.error("Cannot handle event " + topic + " " + event, e);
+//				log.error("Cannot handle event " + event, e);
 			}
 		});
 	}
@@ -629,8 +643,9 @@ public class SuiteApp extends AbstractCmsApp implements CmsEventSubscriber {
 		}
 		if (properties.containsKey(EntityConstants.TYPE)) {
 			List<String> types = LangUtils.toStringList(properties.get(EntityConstants.TYPE));
-			for (String type : types)
+			for (String type : types) {
 				RankedObject.putIfHigherRank(uiProvidersByType, type, uiProvider, properties);
+			}
 		}
 	}
 
@@ -655,6 +670,32 @@ public class SuiteApp extends AbstractCmsApp implements CmsEventSubscriber {
 		}
 	}
 
+//	public void addLayer(SuiteLayer layer, Map<String, Object> properties) {
+//		if (!properties.containsKey(Constants.SERVICE_PID))
+//			throw new IllegalArgumentException("A layer must have an ID");
+//		String pid = (String) properties.get(Constants.SERVICE_PID);
+//		List<String> types = properties.containsKey(EntityConstants.TYPE)
+//				? LangUtils.toStringList(properties.get(EntityConstants.TYPE))
+//				: new ArrayList<>();
+//		if (types.isEmpty()) {
+//			RankedObject.putIfHigherRank(layersByPid, pid, layer, properties);
+//		} else {
+//			if (layersByPid.containsKey(pid)) {
+//				RankedObject<SuiteLayer> current = layersByPid.get(pid);
+//				List<String> currentTypes = current.getProperties().containsKey(EntityConstants.TYPE)
+//						? LangUtils.toStringList(current.getProperties().get(EntityConstants.TYPE))
+//						: new ArrayList<>();
+//				if (!types.containsAll(currentTypes)) {
+//					throw new IllegalArgumentException("Higher-ranked layer " + pid + " contains only types " + types
+//							+ ", while it must override all " + currentTypes);
+//				}
+//			}
+//			RankedObject.putIfHigherRank(layersByPid, pid, layer, properties);
+//			for (String type : types)
+//				RankedObject.putIfHigherRank(layersByType, type, layer, properties);
+//		}
+//	}
+
 	public void addLayer(SuiteLayer layer, Map<String, Object> properties) {
 		if (properties.containsKey(Constants.SERVICE_PID)) {
 			String pid = (String) properties.get(Constants.SERVICE_PID);