Improve JCR UI

[lgpl/argeo-commons.git] / eclipse / runtime / org.argeo.eclipse.ui.jcr / src / main / java / org / argeo / eclipse / ui / jcr / AsyncUiEventListener.java

diff --git a/eclipse/runtime/org.argeo.eclipse.ui.jcr/src/main/java/org/argeo/eclipse/ui/jcr/AsyncUiEventListener.java b/eclipse/runtime/org.argeo.eclipse.ui.jcr/src/main/java/org/argeo/eclipse/ui/jcr/AsyncUiEventListener.java
index 6a83ef3749d3e89d4d66f141fe439fb0732499e4..c152a328acf3ecae9b9ff2d196edbe07384969a5 100644 (file)
--- a/eclipse/runtime/org.argeo.eclipse.ui.jcr/src/main/java/org/argeo/eclipse/ui/jcr/AsyncUiEventListener.java
+++ b/eclipse/runtime/org.argeo.eclipse.ui.jcr/src/main/java/org/argeo/eclipse/ui/jcr/AsyncUiEventListener.java
@@ -1,8 +1,16 @@
 package org.argeo.eclipse.ui.jcr;
 
+import java.util.ArrayList;
+import java.util.List;
+
+import javax.jcr.RepositoryException;
+import javax.jcr.observation.Event;
 import javax.jcr.observation.EventIterator;
 import javax.jcr.observation.EventListener;
 
+import org.apache.commons.logging.Log;
+import org.apache.commons.logging.LogFactory;
+import org.argeo.ArgeoException;
 import org.eclipse.core.runtime.IProgressMonitor;
 import org.eclipse.core.runtime.IStatus;
 import org.eclipse.core.runtime.Status;
@@ -11,36 +19,70 @@ import org.eclipse.swt.widgets.Display;
 
 /** {@link EventListener} which simplifies running actions within the UI thread. */
 public abstract class AsyncUiEventListener implements EventListener {
+       private final static Log logSuper = LogFactory
+                       .getLog(AsyncUiEventListener.class);
+       private final Log logThis = LogFactory.getLog(getClass());
+
        private final Display display;
-       
+
        public AsyncUiEventListener(Display display) {
                super();
                this.display = display;
        }
 
        /** Called asynchronously in the UI thread. */
-       protected abstract void onEventInUiThread(EventIterator events);
+       protected abstract void onEventInUiThread(List<Event> events)
+                       throws RepositoryException;
 
-       public void onEvent(final EventIterator events) {
-               Job job = new Job("JCR Events") {
-                       protected IStatus run(IProgressMonitor monitor) {
-                               //Display display = Display.getCurrent();
-                               //Display display = PlatformUI.getWorkbench().getDisplay();
+       /**
+        * Whether these events should be processed in the UI or skipped with no UI
+        * job created.
+        */
+       protected Boolean willProcessInUiThread(List<Event> events)
+                       throws RepositoryException {
+               return true;
+       }
+
+       protected Log getLog() {
+               return logThis;
+       }
+
+       public final void onEvent(final EventIterator eventIterator) {
+               final List<Event> events = new ArrayList<Event>();
+               while (eventIterator.hasNext())
+                       events.add(eventIterator.nextEvent());
+
+               if (logThis.isDebugEnabled())
+                       logThis.debug("Received " + events.size() + " events");
+
+               try {
+                       if (!willProcessInUiThread(events))
+                               return;
+               } catch (RepositoryException e) {
+                       throw new ArgeoException("Cannot test skip events " + events, e);
+               }
+
+//             Job job = new Job("JCR Events") {
+//                     protected IStatus run(IProgressMonitor monitor) {
+//                             if (display.isDisposed()) {
+//                                     logSuper.warn("Display is disposed cannot update UI");
+//                                     return Status.CANCEL_STATUS;
+//                             }
 
                                display.asyncExec(new Runnable() {
                                        public void run() {
-                                               onEventInUiThread(events);
+                                               try {
+                                                       onEventInUiThread(events);
+                                               } catch (RepositoryException e) {
+                                                       throw new ArgeoException("Cannot process events "
+                                                                       + events, e);
+                                               }
                                        }
                                });
-                               return Status.OK_STATUS;
-                       }
-               };
-               job.schedule();
-
-               // PlatformUI.getWorkbench().getDisplay().asyncExec(new Runnable() {
-               // public void run() {
-               // onEventInUiThread(events);
-               // }
-               // });
+
+//                             return Status.OK_STATUS;
+//                     }
+//             };
+//             job.schedule();
        }
 }