You are viewing a plain text version of this content. The canonical link for it is here.
Posted to cvs@cocoon.apache.org by un...@apache.org on 2004/09/22 17:02:19 UTC

svn commit: rev 47051 - cocoon/trunk/src/blocks/cron/java/org/apache/cocoon/components/cron

Author: unico
Date: Wed Sep 22 08:02:18 2004
New Revision: 47051

Modified:
   cocoon/trunk/src/blocks/cron/java/org/apache/cocoon/components/cron/QuartzJobExecutor.java
Log:
dispose on correct condition

Modified: cocoon/trunk/src/blocks/cron/java/org/apache/cocoon/components/cron/QuartzJobExecutor.java
==============================================================================
--- cocoon/trunk/src/blocks/cron/java/org/apache/cocoon/components/cron/QuartzJobExecutor.java	(original)
+++ cocoon/trunk/src/blocks/cron/java/org/apache/cocoon/components/cron/QuartzJobExecutor.java	Wed Sep 22 08:02:18 2004
@@ -79,6 +79,7 @@
         String jobrole = null;
 
         boolean release = false;
+        boolean dispose = false;
         try {
             env.startingProcessing();
             EnvironmentHelper.enterProcessor(env.getProcessor(), manager, env);
@@ -90,6 +91,7 @@
                 ContainerUtil.enableLogging(job, logger);
                 ContainerUtil.contextualize(job, appContext);
                 ContainerUtil.service(job, manager);
+                dispose = true;
             } else {
                 job = manager.lookup(jobrole);
                 release = true;
@@ -127,7 +129,7 @@
             if (release && manager != null) {
                 manager.release(job);
             }
-            else if (job != null) {
+            if (dispose) {
             	ContainerUtil.dispose(job);
             }
         }