]> git.argeo.org Git - lgpl/argeo-commons.git/blobdiff - org.argeo.cms/src/org/argeo/cms/AbstractCmsEntryPoint.java
Do not time out demo.
[lgpl/argeo-commons.git] / org.argeo.cms / src / org / argeo / cms / AbstractCmsEntryPoint.java
index 2ff9aff58e9275a043dc28b636a9c453bac53f42..743a142a4430c4e4e7e2253ce012439a3c04ce0c 100644 (file)
@@ -5,7 +5,7 @@ import java.util.HashMap;
 import java.util.Map;
 
 import javax.jcr.Node;
-import javax.jcr.NodeIterator;
+import javax.jcr.PathNotFoundException;
 import javax.jcr.Property;
 import javax.jcr.Repository;
 import javax.jcr.RepositoryException;
@@ -19,10 +19,8 @@ import javax.servlet.http.HttpServletRequest;
 
 import org.apache.commons.logging.Log;
 import org.apache.commons.logging.LogFactory;
-import org.argeo.ArgeoException;
 import org.argeo.cms.auth.AuthConstants;
 import org.argeo.cms.auth.HttpRequestCallbackHandler;
-import org.argeo.cms.util.CmsUtils;
 import org.argeo.eclipse.ui.specific.UiContext;
 import org.argeo.jcr.JcrUtils;
 import org.eclipse.rap.rwt.RWT;
@@ -37,8 +35,7 @@ import org.eclipse.swt.widgets.Display;
 import org.eclipse.swt.widgets.Shell;
 
 /** Manages history and navigation */
-public abstract class AbstractCmsEntryPoint extends AbstractEntryPoint
-               implements CmsView {
+public abstract class AbstractCmsEntryPoint extends AbstractEntryPoint implements CmsView {
        private final Log log = LogFactory.getLog(AbstractCmsEntryPoint.class);
 
        private final Subject subject;
@@ -52,6 +49,7 @@ public abstract class AbstractCmsEntryPoint extends AbstractEntryPoint
        // Current state
        private Session session;
        private Node node;
+       private String nodePath;// useful when changing auth
        private String state;
        private String page;
        private Throwable exception;
@@ -60,8 +58,8 @@ public abstract class AbstractCmsEntryPoint extends AbstractEntryPoint
        private final JavaScriptExecutor jsExecutor;
        private final BrowserNavigation browserNavigation;
 
-       public AbstractCmsEntryPoint(Repository repository, String workspace,
-                       String defaultPath, Map<String, String> factoryProperties) {
+       public AbstractCmsEntryPoint(Repository repository, String workspace, String defaultPath,
+                       Map<String, String> factoryProperties) {
                this.repository = repository;
                this.workspace = workspace;
                this.defaultPath = defaultPath;
@@ -70,28 +68,25 @@ public abstract class AbstractCmsEntryPoint extends AbstractEntryPoint
 
                // Initial login
                try {
-                       loginContext = new LoginContext(AuthConstants.LOGIN_CONTEXT_USER,
-                                       subject, new HttpRequestCallbackHandler(
-                                                       UiContext.getHttpRequest()));
+                       loginContext = new LoginContext(AuthConstants.LOGIN_CONTEXT_USER, subject,
+                                       new HttpRequestCallbackHandler(UiContext.getHttpRequest()));
                        loginContext.login();
                } catch (CredentialNotFoundException e) {
                        try {
-                               loginContext = new LoginContext(
-                                               AuthConstants.LOGIN_CONTEXT_ANONYMOUS, subject);
+                               loginContext = new LoginContext(AuthConstants.LOGIN_CONTEXT_ANONYMOUS, subject);
                                loginContext.login();
                        } catch (LoginException e1) {
-                               throw new ArgeoException("Cannot log as anonymous", e);
+                               throw new CmsException("Cannot log as anonymous", e);
                        }
                } catch (LoginException e) {
-                       throw new ArgeoException("Cannot initialize subject", e);
+                       throw new CmsException("Cannot initialize subject", e);
                }
                authChange(loginContext);
 
                jsExecutor = RWT.getClient().getService(JavaScriptExecutor.class);
                browserNavigation = RWT.getClient().getService(BrowserNavigation.class);
                if (browserNavigation != null)
-                       browserNavigation
-                                       .addBrowserNavigationListener(new CmsNavigationListener());
+                       browserNavigation.addBrowserNavigationListener(new CmsNavigationListener());
        }
 
        @Override
@@ -119,8 +114,7 @@ public abstract class AbstractCmsEntryPoint extends AbstractEntryPoint
                                try {
                                        initUi(parent);
                                } catch (Exception e) {
-                                       throw new CmsException("Cannot create entrypoint contents",
-                                                       e);
+                                       throw new CmsException("Cannot create entrypoint contents", e);
                                }
                                return null;
                        }
@@ -161,18 +155,17 @@ public abstract class AbstractCmsEntryPoint extends AbstractEntryPoint
        }
 
        @Override
-       public Subject getSubject() {
+       public synchronized Subject getSubject() {
                return subject;
        }
 
        @Override
-       public void logout() {
+       public synchronized void logout() {
                if (loginContext == null)
                        throw new CmsException("Login context should not be null");
                try {
                        loginContext.logout();
-                       LoginContext anonymousLc = new LoginContext(
-                                       AuthConstants.LOGIN_CONTEXT_ANONYMOUS, subject);
+                       LoginContext anonymousLc = new LoginContext(AuthConstants.LOGIN_CONTEXT_ANONYMOUS, subject);
                        anonymousLc.login();
                        authChange(anonymousLc);
                } catch (LoginException e) {
@@ -181,29 +174,25 @@ public abstract class AbstractCmsEntryPoint extends AbstractEntryPoint
        }
 
        @Override
-       public void authChange(LoginContext loginContext) {
+       public synchronized void authChange(LoginContext loginContext) {
                if (loginContext == null)
                        throw new CmsException("Login context cannot be null");
                this.loginContext = loginContext;
-               Subject.doAs(subject, new PrivilegedAction<Void>() {
+               Subject.doAs(loginContext.getSubject(), new PrivilegedAction<Void>() {
 
                        @Override
                        public Void run() {
                                try {
-                                       String currentPath = null;
-                                       if (node != null)
-                                               currentPath = node.getPath();
                                        JcrUtils.logoutQuietly(session);
-
                                        session = repository.login(workspace);
-                                       if (currentPath != null)
+                                       if (nodePath != null)
                                                try {
-                                                       node = session.getNode(currentPath);
-                                               } catch (Exception e) {
-                                                       logout();
-                                                       session = repository.login(workspace);
+                                                       node = session.getNode(nodePath);
+                                               } catch (PathNotFoundException e) {
+                                                       // logout();
+                                                       // session = repository.login(workspace);
                                                        navigateTo("~");
-                                                       throw e;
+                                                       // throw e;
                                                }
 
                                        // refresh UI
@@ -225,7 +214,7 @@ public abstract class AbstractCmsEntryPoint extends AbstractEntryPoint
                doRefresh();
        }
 
-       protected void doRefresh() {
+       protected synchronized void doRefresh() {
                Subject.doAs(subject, new PrivilegedAction<Void>() {
                        @Override
                        public Void run() {
@@ -239,7 +228,7 @@ public abstract class AbstractCmsEntryPoint extends AbstractEntryPoint
        protected synchronized String setState(String newState) {
                String previousState = this.state;
 
-               node = null;
+               Node node = null;
                page = null;
                this.state = newState;
                if (newState.equals("~"))
@@ -265,21 +254,11 @@ public abstract class AbstractCmsEntryPoint extends AbstractEntryPoint
                                node = getDefaultNode(session);
                                page = state;
                        }
-
-                       // Title
-                       String title;
-                       if (node.isNodeType(NodeType.MIX_TITLE)
-                                       && node.hasProperty(Property.JCR_TITLE))
-                               title = node.getProperty(Property.JCR_TITLE).getString()
-                                               + " - " + getBaseTitle();
-                       else
-                               title = getBaseTitle();
-
-                       publishMetaData(title);
+                       setNode(node);
+                       String title = publishMetaData(node);
 
                        if (log.isTraceEnabled())
-                               log.trace("node=" + node + ", state=" + state + " (page="
-                                               + page + ", title=" + title + ")");
+                               log.trace("node=" + node + ", state=" + state + " (page=" + page + ")");
 
                        return title;
                } catch (Exception e) {
@@ -287,46 +266,44 @@ public abstract class AbstractCmsEntryPoint extends AbstractEntryPoint
                        if (previousState.equals(""))
                                previousState = "~";
                        navigateTo(previousState);
-                       throw new CmsException("Unexpected issue when accessing #"
-                                       + newState, e);
+                       throw new CmsException("Unexpected issue when accessing #" + newState, e);
                }
        }
 
-       private void publishMetaData(String title) throws RepositoryException {
+       private String publishMetaData(Node node) throws RepositoryException {
+               // Title
+               String title;
+               if (node.isNodeType(NodeType.MIX_TITLE) && node.hasProperty(Property.JCR_TITLE))
+                       title = node.getProperty(Property.JCR_TITLE).getString() + " - " + getBaseTitle();
+               else
+                       title = getBaseTitle();
+
                HttpServletRequest request = UiContext.getHttpRequest();
                if (request == null)
-                       return;
-               String url = CmsUtils.getCanonicalUrl(node, request);
-               String imgUrl = null;
-               for (NodeIterator it = node.getNodes(); it.hasNext();) {
-                       Node child = it.nextNode();
-                       if (child.isNodeType(CmsTypes.CMS_IMAGE))
-                               imgUrl = CmsUtils.getDataUrl(child, request);
-               }
+                       return null;
 
                StringBuilder js = new StringBuilder();
+               title = title.replace("'", "\\'");// sanitize
                js.append("document.title = '" + title + "';");
-               js.append("var metas = document.getElementsByTagName('meta');");
-               js.append("for (var i=0; i<metas.length; i++) {");
-               js.append("     if (metas[i].getAttribute('property'))");
-               js.append("      if(metas[i].getAttribute('property')=='og:title')");
-               js.append("       metas[i].setAttribute('content','" + title + "');");
-               js.append("      else if(metas[i].getAttribute('property')=='og:url')");
-               js.append("       metas[i].setAttribute('content','" + url + "');");
-               if (imgUrl != null) {
-                       js.append("      else if(metas[i].getAttribute('property')=='og:image')");
-                       js.append("       metas[i].setAttribute('content','" + imgUrl + "');");
-               } else {
-                       // TODO reset default image
-               }
-               js.append("     };");
                jsExecutor.execute(js.toString());
+               return title;
        }
 
-       protected Node getNode() {
+       // Simply remove some illegal character
+       // private String clean(String stringToClean) {
+       // return stringToClean.replaceAll("'", "").replaceAll("\\n", "")
+       // .replaceAll("\\t", "");
+       // }
+
+       protected synchronized Node getNode() {
                return node;
        }
 
+       private synchronized void setNode(Node node) throws RepositoryException {
+               this.node = node;
+               this.nodePath = node == null ? null : node.getPath();
+       }
+
        protected String getState() {
                return state;
        }