X-Git-Url: https://git.argeo.org/?a=blobdiff_plain;f=org.argeo.cms.ui.rap%2Fsrc%2Forg%2Fargeo%2Fcms%2Fweb%2FCmsWebApp.java;h=dda02646d973763af7619482a3a94266ea03b4f9;hb=f24f4802611d08a8ef4df79a2939eacc161e6e40;hp=03ac353df248ab5952daa0ad559747d477131532;hpb=94bd4b03ff3ca9d1875bf6424383d648e604b93a;p=lgpl%2Fargeo-commons.git

diff --git a/org.argeo.cms.ui.rap/src/org/argeo/cms/web/CmsWebApp.java b/org.argeo.cms.ui.rap/src/org/argeo/cms/web/CmsWebApp.java
index 03ac353df..dda02646d 100644
--- a/org.argeo.cms.ui.rap/src/org/argeo/cms/web/CmsWebApp.java
+++ b/org.argeo.cms.ui.rap/src/org/argeo/cms/web/CmsWebApp.java
@@ -9,17 +9,21 @@ import org.apache.commons.logging.LogFactory;
 import org.argeo.cms.ui.CmsApp;
 import org.argeo.cms.ui.CmsAppListener;
 import org.argeo.cms.ui.CmsTheme;
+import org.argeo.cms.ui.CmsView;
 import org.argeo.util.LangUtils;
 import org.eclipse.rap.rwt.RWT;
 import org.eclipse.rap.rwt.application.Application;
 import org.eclipse.rap.rwt.application.ApplicationConfiguration;
+import org.eclipse.rap.rwt.application.ExceptionHandler;
+import org.eclipse.rap.rwt.application.Application.OperationMode;
 import org.eclipse.rap.rwt.client.WebClient;
+import org.eclipse.swt.widgets.Display;
 import org.osgi.framework.BundleContext;
 import org.osgi.framework.ServiceRegistration;
 import org.osgi.service.event.EventAdmin;
 
 /** An RWT web app integrating with a {@link CmsApp}. */
-public class CmsWebApp implements ApplicationConfiguration, CmsAppListener {
+public class CmsWebApp implements ApplicationConfiguration, ExceptionHandler, CmsAppListener {
 	private final static Log log = LogFactory.getLog(CmsWebApp.class);
 
 	private BundleContext bundleContext;
@@ -36,7 +40,6 @@ public class CmsWebApp implements ApplicationConfiguration, CmsAppListener {
 		contextName = properties.get(CONTEXT_NAME);
 		if (cmsApp != null)
 			themingUpdated();
-//		registerIfAllThemesAvailable();
 	}
 
 	public void destroy(BundleContext bundleContext, Map<String, String> properties) {
@@ -46,16 +49,31 @@ public class CmsWebApp implements ApplicationConfiguration, CmsAppListener {
 
 	@Override
 	public void configure(Application application) {
+		// TODO make it configurable?
+		// SWT compatibility is required for:
+		// - Browser.execute()
+		// - blocking dialogs
+		application.setOperationMode(OperationMode.SWT_COMPATIBILITY);
 		for (String uiName : cmsApp.getUiNames()) {
 			CmsTheme theme = cmsApp.getTheme(uiName);
 			if (theme != null)
 				WebThemeUtils.apply(application, theme);
 		}
-//		for (CmsTheme theme : themes.values())
-//			WebThemeUtils.apply(application, theme);
 
 		Map<String, String> properties = new HashMap<>();
 		addEntryPoints(application, properties);
+		application.setExceptionHandler(this);
+	}
+
+	@Override
+	public void handleException(Throwable throwable) {
+		Display display = Display.getCurrent();
+		if (display != null && !display.isDisposed()) {
+			CmsView cmsView = CmsView.getCmsView(display.getActiveShell());
+			cmsView.exception(throwable);
+		} else {
+			log.error("Unexpected exception outside an UI thread", throwable);
+		}
 
 	}
 
@@ -68,48 +86,31 @@ public class CmsWebApp implements ApplicationConfiguration, CmsAppListener {
 				properties.put(WebClient.HEAD_HTML, theme.getHtmlHeaders());
 			} else {
 				properties.put(WebClient.THEME_ID, RWT.DEFAULT_THEME_ID);
-//				if (themeId != null)
-//					log.warn("Theme id " + themeId + " was specified but it was not found, using default RWT theme.");
 			}
-			application.addEntryPoint("/" + uiName, () -> {
+			String entryPointName = !uiName.equals("") ? "/" + uiName : "/";
+			application.addEntryPoint(entryPointName, () -> {
 				CmsWebEntryPoint entryPoint = new CmsWebEntryPoint(this, uiName);
 				entryPoint.setEventAdmin(eventAdmin);
 				return entryPoint;
 			}, properties);
 			if (log.isDebugEnabled())
-				log.info("Added web entry point /" + (contextName != null ? contextName : "") + "/" + uiName);
+				log.info("Added web entry point " + (contextName != null ? "/" + contextName : "") + entryPointName);
 		}
-		log.debug("Published CMS web app /" + (contextName != null ? contextName : ""));
+//		if (log.isDebugEnabled())
+//			log.debug("Published CMS web app /" + (contextName != null ? contextName : ""));
 	}
 
-//	private void registerIfAllThemesAvailable() {
-//		boolean themeMissing = false;
-//		uiNames: for (String uiName : cmsApp.getUiNames()) {
-//			String themeId = cmsApp.getThemeId(uiName);
-//			if (RWT.DEFAULT_THEME_ID.equals(themeId))
-//				continue uiNames;
-//			if (!themes.containsKey(themeId)) {
-//				themeMissing = true;
-//				break uiNames;
-//			}
-//		}
-//		if (!themeMissing) {
-//			Dictionary<String, Object> regProps = LangUtils.dict(CONTEXT_NAME, contextName);
-//			if (bundleContext != null) {
-//				rwtAppReg = bundleContext.registerService(ApplicationConfiguration.class, this, regProps);
-//				log.info("Published CMS web app /" + (contextName != null ? contextName : ""));
-//			}
-//		}
-//	}
-
 	CmsApp getCmsApp() {
 		return cmsApp;
 	}
 
+	BundleContext getBundleContext() {
+		return bundleContext;
+	}
+
 	public void setCmsApp(CmsApp cmsApp, Map<String, String> properties) {
 		this.cmsApp = cmsApp;
 		this.cmsApp.addCmsAppListener(this);
-//		registerIfAllThemesAvailable();
 	}
 
 	public void unsetCmsApp(CmsApp cmsApp, Map<String, String> properties) {