svn commit: r728536 - in /ofbiz/trunk/framework: base/src/org/ofbiz/base/util/ service/entitydef/ service/src/org/ofbiz/service/ service/src/org/ofbiz/service/job/ webapp/src/org/ofbiz/webapp/event/ webtools/webapp/webtools/WEB-INF/actions/log/ webtool...

Previous Topic Next Topic
 
classic Classic list List threaded Threaded
1 message Options
Reply | Threaded
Open this post in threaded view
|

svn commit: r728536 - in /ofbiz/trunk/framework: base/src/org/ofbiz/base/util/ service/entitydef/ service/src/org/ofbiz/service/ service/src/org/ofbiz/service/job/ webapp/src/org/ofbiz/webapp/event/ webtools/webapp/webtools/WEB-INF/actions/log/ webtool...

jleroux@apache.org
Author: jleroux
Date: Sun Dec 21 14:25:22 2008
New Revision: 728536

URL: http://svn.apache.org/viewvc?rev=728536&view=rev
Log:
Reverted on Adam's guidances

Modified:
    ofbiz/trunk/framework/base/src/org/ofbiz/base/util/Debug.java
    ofbiz/trunk/framework/service/entitydef/entitymodel.xml
    ofbiz/trunk/framework/service/src/org/ofbiz/service/GenericAbstractDispatcher.java
    ofbiz/trunk/framework/service/src/org/ofbiz/service/LocalDispatcher.java
    ofbiz/trunk/framework/service/src/org/ofbiz/service/ServiceUtil.java
    ofbiz/trunk/framework/service/src/org/ofbiz/service/job/GenericServiceJob.java
    ofbiz/trunk/framework/service/src/org/ofbiz/service/job/JobInvoker.java
    ofbiz/trunk/framework/service/src/org/ofbiz/service/job/JobManager.java
    ofbiz/trunk/framework/service/src/org/ofbiz/service/job/PersistedServiceJob.java
    ofbiz/trunk/framework/webapp/src/org/ofbiz/webapp/event/CoreEvents.java
    ofbiz/trunk/framework/webtools/webapp/webtools/WEB-INF/actions/log/LogView.groovy
    ofbiz/trunk/framework/webtools/webapp/webtools/service/ServiceForms.xml

Modified: ofbiz/trunk/framework/base/src/org/ofbiz/base/util/Debug.java
URL: http://svn.apache.org/viewvc/ofbiz/trunk/framework/base/src/org/ofbiz/base/util/Debug.java?rev=728536&r1=728535&r2=728536&view=diff
==============================================================================
--- ofbiz/trunk/framework/base/src/org/ofbiz/base/util/Debug.java (original)
+++ ofbiz/trunk/framework/base/src/org/ofbiz/base/util/Debug.java Sun Dec 21 14:25:22 2008
@@ -25,7 +25,6 @@
 import java.util.Enumeration;
 import java.util.HashMap;
 import java.util.Map;
-import java.util.Vector;
 
 import org.apache.avalon.util.exception.ExceptionHelper;
 import org.apache.log4j.Level;
@@ -71,8 +70,6 @@
     protected static final boolean useLevelOnCache = true;
     
     protected static Logger root = Logger.getRootLogger();
-    
-    private static Map<String, Vector<Appender>> activeThreadGroupLoggerMap = new HashMap<String, Vector<Appender>>();
 
     static {
         levelStringMap.put("verbose", Debug.VERBOSE);
@@ -131,108 +128,6 @@
             return root;
         }
     }
-    
-    /**
-     * Checks if a logger exists and if it is in activeThreadGroupLoggerMap
-     * @param threadGroupId a thread group id
-     * @return
-     */
-    private static boolean hasActiveThreadLogger(String threadGroupId) {
-        //used so entries are not doubled in rootAppenders
-        Logger threadGroupLogger = org.apache.log4j.LogManager
-                .exists(threadGroupId);
-        return threadGroupLogger != null
-                && (activeThreadGroupLoggerMap.get(threadGroupId) != null && activeThreadGroupLoggerMap
-                        .get(threadGroupId).size() > 0);
-    }
-
-    private static Appender getAppender(String threadGroupId,
-            String appenderName) {
-        Vector<Appender> appenders = activeThreadGroupLoggerMap
-                .get(threadGroupId);
-        if (appenders != null) {
-            for (Appender appender : appenders) {
-                if (appender != null && appender.getName().equals(appenderName)) {
-                    return appender;
-                }
-            }
-        }
-        return null;
-    }
-
-    private static void addAppenderToThreadGroupMap(String threadGroupId,
-            Appender appender) {
-        Vector<Appender> appenders = activeThreadGroupLoggerMap.get(threadGroupId);
-        if (appenders == null) {
-            appenders = new Vector<Appender>();
-        }
-        appenders.add(appender);
-        activeThreadGroupLoggerMap.put(threadGroupId, appenders);
-    }
-
-    private static void removeAppenderFromThreadGroupMap(String threadGroupId,
-            Appender appender) {
-        Vector<Appender> appenders = activeThreadGroupLoggerMap.get(threadGroupId);
-        if (appenders != null && appenders.contains(appender)) {
-            appenders.remove(appender);
-            activeThreadGroupLoggerMap.put(threadGroupId, appenders);
-        }
-    }
-
-    private static String currentThreadGroupId() {
-        return "" + Thread.currentThread().getThreadGroup().hashCode();
-    }
-
-    public static void registerCurrentThreadGroupLogger(String logFile, String appenderName) {
-        String pattern = "<div class=%p>%d (%t) [%24F:%-3L:%-5p]%x %m </div>%n";
-        registerThreadAppender(getNewFileAppender(appenderName,
-                logFile, 0, 0, pattern));
-    }
-
-    public static void registerThreadAppender(Appender appender) {
-        String threadGroupId = currentThreadGroupId();
-        
-        if (threadGroupId != null && threadGroupId.length() > 0) {
-            Logger theLogger = getLogger(threadGroupId);
-            if (theLogger != null) {
-                theLogger.setAdditivity(false);
-                theLogger.addAppender(appender);
-                addAppenderToThreadGroupMap(threadGroupId, appender);
-            }
-        }
-    }
-
-    public static void unregisterCurrentThreadGroupLogger(String appenderName) {
-        String threadGroupId = currentThreadGroupId();
-        Appender foundAppender = getAppender(threadGroupId, appenderName);
-        unregisterThreadAppender(foundAppender);
-    }
-
-    public static void unregisterThreadAppender(Appender appender) {
-        String threadGroupId = currentThreadGroupId();
-        if (threadGroupId != null && threadGroupId.length() > 0
-                && appender != null) {
-            Logger theLogger = getLogger(threadGroupId);
-            theLogger.removeAppender(appender);
-            removeAppenderFromThreadGroupMap(threadGroupId, appender);
-        }
-    }
-
-    /**
-     * Thread-specific logging
-     */
-    private static void logThreadGroup(int level, Throwable t, String msg, String module,
-            String callingClass) {
-        String threadGroupId = currentThreadGroupId();
-        if (hasActiveThreadLogger(threadGroupId)) {
-            Logger grplogger = getLogger(threadGroupId + "." + module);
-            if (SYS_DEBUG != null) {
-                grplogger.setLevel(Level.DEBUG);
-            }
-            grplogger.log(callingClass, levelObjs[level], msg, t);
-        }
-    }
-    
 
     /** Gets an Integer representing the level number from a String representing the level name; will return null if not found */
     public static Integer getLevelFromString(String levelName) {
@@ -269,7 +164,6 @@
                     logger.setLevel(Level.DEBUG);
                 }
                 logger.log(callingClass, levelObjs[level], msg, t);
-                logThreadGroup(level, t, msg, module, callingClass);
             } else {
                 StringBuilder prefixBuf = new StringBuilder();
 

Modified: ofbiz/trunk/framework/service/entitydef/entitymodel.xml
URL: http://svn.apache.org/viewvc/ofbiz/trunk/framework/service/entitydef/entitymodel.xml?rev=728536&r1=728535&r2=728536&view=diff
==============================================================================
--- ofbiz/trunk/framework/service/entitydef/entitymodel.xml (original)
+++ ofbiz/trunk/framework/service/entitydef/entitymodel.xml Sun Dec 21 14:25:22 2008
@@ -64,8 +64,6 @@
         <field name="startDateTime" type="date-time"></field>
         <field name="finishDateTime" type="date-time"></field>
         <field name="cancelDateTime" type="date-time"></field>
-        <field name="ownLogfile" type="indicator"></field>
-        <field name="logLocation" type="long-varchar"></field>
         <prim-key field="jobId"/>
         <relation type="one" fk-name="JOB_SNDBX_RECINFO" rel-entity-name="RecurrenceInfo">
             <key-map field-name="recurrenceInfoId"/>

Modified: ofbiz/trunk/framework/service/src/org/ofbiz/service/GenericAbstractDispatcher.java
URL: http://svn.apache.org/viewvc/ofbiz/trunk/framework/service/src/org/ofbiz/service/GenericAbstractDispatcher.java?rev=728536&r1=728535&r2=728536&view=diff
==============================================================================
--- ofbiz/trunk/framework/service/src/org/ofbiz/service/GenericAbstractDispatcher.java (original)
+++ ofbiz/trunk/framework/service/src/org/ofbiz/service/GenericAbstractDispatcher.java Sun Dec 21 14:25:22 2008
@@ -59,18 +59,8 @@
      * @see org.ofbiz.service.LocalDispatcher#schedule(java.lang.String, java.lang.String, java.lang.String, java.util.Map, long, int, int, int, long, int)
      */
     public void schedule(String jobName, String poolName, String serviceName, Map<String, ? extends Object> context, long startTime, int frequency, int interval, int count, long endTime, int maxRetry) throws GenericServiceException {
-        schedule(jobName, poolName, serviceName, context, startTime, frequency, interval, count, endTime, maxRetry, false);
-    }
-    public void schedule(String jobName, String poolName, String serviceName, long startTime, int frequency, int interval, int count, long endTime, int maxRetry, Object... context) throws GenericServiceException {
-        schedule(jobName, poolName, serviceName, ServiceUtil.makeContext(context), startTime, frequency, interval, count, endTime, maxRetry);
-    }
-    
-    /**
-     * @see org.ofbiz.service.LocalDispatcher#schedule(java.lang.String, java.lang.String, java.lang.String, java.util.Map, long, int, int, int, long, int, boolean)
-     */
-    public void schedule(String jobName, String poolName, String serviceName, Map<String, ? extends Object> context, long startTime, int frequency, int interval, int count, long endTime, int maxRetry, boolean ownLogfile) throws GenericServiceException {
         try {
-            getJobManager().schedule(jobName, poolName, serviceName, context, startTime, frequency, interval, count, endTime, maxRetry, ownLogfile);
+            getJobManager().schedule(jobName, poolName, serviceName, context, startTime, frequency, interval, count, endTime, maxRetry);
                 
             if (Debug.verboseOn()) {
                 Debug.logVerbose("[LocalDispatcher.schedule] : Current time : " + (new Date()).getTime(), module);
@@ -80,16 +70,15 @@
                 Debug.logVerbose("[LocalDispatcher.schedule] : Count        : " + count, module);
                 Debug.logVerbose("[LocalDispatcher.schedule] : EndTime      : " + endTime, module);
                 Debug.logVerbose("[LocalDispatcher.schedule] : MazRetry     : " + maxRetry, module);
-                Debug.logVerbose("[LocalDispatcher.schedule] : OwnLogfile     : " + ownLogfile, module);
             }
             
         } catch (JobManagerException e) {
             throw new GenericServiceException(e.getMessage(), e);
         }
     }
-    
-    public void schedule(String jobName, String poolName, String serviceName, long startTime, int frequency, int interval, int count, long endTime, int maxRetry, boolean ownLogfile, Object... context) throws GenericServiceException {
-        schedule(jobName, poolName, serviceName, ServiceUtil.makeContext(context), startTime, frequency, interval, count, endTime, maxRetry, ownLogfile);
+
+    public void schedule(String jobName, String poolName, String serviceName, long startTime, int frequency, int interval, int count, long endTime, int maxRetry, Object... context) throws GenericServiceException {
+        schedule(jobName, poolName, serviceName, ServiceUtil.makeContext(context), startTime, frequency, interval, count, endTime, maxRetry);
     }
 
     /**

Modified: ofbiz/trunk/framework/service/src/org/ofbiz/service/LocalDispatcher.java
URL: http://svn.apache.org/viewvc/ofbiz/trunk/framework/service/src/org/ofbiz/service/LocalDispatcher.java?rev=728536&r1=728535&r2=728536&view=diff
==============================================================================
--- ofbiz/trunk/framework/service/src/org/ofbiz/service/LocalDispatcher.java (original)
+++ ofbiz/trunk/framework/service/src/org/ofbiz/service/LocalDispatcher.java Sun Dec 21 14:25:22 2008
@@ -228,25 +228,6 @@
     public void schedule(String jobName, String poolName, String serviceName, Map<String, ? extends Object> context, long startTime, int frequency, int interval, int count, long endTime, int maxRetry) throws GenericServiceException;
     public void schedule(String jobName, String poolName, String serviceName, long startTime, int frequency, int interval, int count, long endTime, int maxRetry, Object... context) throws GenericServiceException;
 
-    /**
-     * Schedule a service to run asynchronously at a specific start time.
-     * @param jobName Name of the job
-     * @param poolName Name of the service pool to send to.
-     * @param serviceName Name of the service to invoke.
-     * @param context The name/value pairs composing the context.
-     * @param startTime The time to run this service.
-     * @param frequency The frequency of the recurrence (RecurrenceRule.DAILY, etc).
-     * @param interval The interval of the frequency recurrence.
-     * @param count The number of times to repeat.
-     * @param endTime The time in milliseconds the service should expire
-     * @param maxRetry The number of times we should retry on failure
-     * @param ownLogfile Indicator whether this job uses it's own logfile
-     * @throws ServiceAuthException
-     * @throws ServiceValidationException
-     * @throws GenericServiceException
-     */
-    public void schedule(String jobName, String poolName, String serviceName, Map<String, ? extends Object> context, long startTime, int frequency, int interval, int count, long endTime, int maxRetry, boolean ownLogfile) throws GenericServiceException;
-    public void schedule(String jobName, String poolName, String serviceName, long startTime, int frequency, int interval, int count, long endTime, int maxRetry, boolean ownLogfile, Object... context) throws GenericServiceException;
 
     /**
      * Schedule a service to run asynchronously at a specific start time.

Modified: ofbiz/trunk/framework/service/src/org/ofbiz/service/ServiceUtil.java
URL: http://svn.apache.org/viewvc/ofbiz/trunk/framework/service/src/org/ofbiz/service/ServiceUtil.java?rev=728536&r1=728535&r2=728536&view=diff
==============================================================================
--- ofbiz/trunk/framework/service/src/org/ofbiz/service/ServiceUtil.java (original)
+++ ofbiz/trunk/framework/service/src/org/ofbiz/service/ServiceUtil.java Sun Dec 21 14:25:22 2008
@@ -38,8 +38,6 @@
 
 import javax.servlet.http.HttpServletRequest;
 import javax.transaction.Transaction;
-
-import java.io.File;
 import java.sql.Timestamp;
 import java.util.*;
 
@@ -422,14 +420,7 @@
                         boolean beganTx2 = false;
                         try {
                             beganTx2 = TransactionUtil.begin();
-                            String logLocation = job.getString("logLocation");
-                            
                             job.remove();
-                            
-                            if (logLocation != null) {
-                                File logfile = new File(job.getString("logLocation"));
-                                logfile.delete();
-                            }
                             runtimeToDelete.add(runtimeId);
                         } catch (GenericEntityException e) {
                             Debug.logInfo("Cannot remove job data for ID: " + jobId, module);

Modified: ofbiz/trunk/framework/service/src/org/ofbiz/service/job/GenericServiceJob.java
URL: http://svn.apache.org/viewvc/ofbiz/trunk/framework/service/src/org/ofbiz/service/job/GenericServiceJob.java?rev=728536&r1=728535&r2=728536&view=diff
==============================================================================
--- ofbiz/trunk/framework/service/src/org/ofbiz/service/job/GenericServiceJob.java (original)
+++ ofbiz/trunk/framework/service/src/org/ofbiz/service/job/GenericServiceJob.java Sun Dec 21 14:25:22 2008
@@ -38,9 +38,7 @@
 
     private String service = null;
     private Map<String, Object> context = null;
-    
-    private String logLocation = null;
-        
+
     public GenericServiceJob(DispatchContext dctx, String jobId, String jobName, String service, Map<String, Object> context, GenericRequester req) {
         super(jobId, jobName);
         this.dctx = dctx;
@@ -63,18 +61,12 @@
      */
     public void exec() throws InvalidJobException {
         init();
-        String appenderName = module;
+
         // no transaction is necessary since runSync handles this
         try {
             // get the dispatcher and invoke the service via runSync -- will run all ECAs
             LocalDispatcher dispatcher = dctx.getDispatcher();
-            
-            if (this.logLocation != null) {
-                Debug
-                .registerCurrentThreadGroupLogger(this.logLocation,
-                        appenderName);
-            }
-            Map result = dispatcher.runSync(getServiceName(), getContext());
+            Map<String, Object> result = dispatcher.runSync(getServiceName(), getContext());
 
             // check for a failure
             boolean isError = ModelService.RESPOND_ERROR.equals(result.get(ModelService.RESPONSE_MESSAGE));
@@ -95,8 +87,6 @@
 
             // call the failed method
             this.failed(t);
-        }finally{
-            Debug.unregisterCurrentThreadGroupLogger(appenderName);
         }
 
         // call the finish method
@@ -141,13 +131,5 @@
      */
     protected String getServiceName() throws InvalidJobException {
         return service;
-    }
-    
-    /**
-     * Sets the logfile name
-     * @param logLocation
-     */
-    protected void setLogLocation(String logLocation) {
-        this.logLocation = logLocation;
-    }
+    }        
 }

Modified: ofbiz/trunk/framework/service/src/org/ofbiz/service/job/JobInvoker.java
URL: http://svn.apache.org/viewvc/ofbiz/trunk/framework/service/src/org/ofbiz/service/job/JobInvoker.java?rev=728536&r1=728535&r2=728536&view=diff
==============================================================================
--- ofbiz/trunk/framework/service/src/org/ofbiz/service/job/JobInvoker.java (original)
+++ ofbiz/trunk/framework/service/src/org/ofbiz/service/job/JobInvoker.java Sun Dec 21 14:25:22 2008
@@ -19,7 +19,6 @@
 package org.ofbiz.service.job;
 
 import java.util.Date;
-import java.util.Random;
 
 import org.ofbiz.service.config.ServiceConfigUtil;
 import org.ofbiz.base.util.Debug;
@@ -61,11 +60,11 @@
 
         // service dispatcher delegator name (for thread name)
         String delegatorName = jp.getManager().getDelegator().getDelegatorName();
-        
+
         // get a new thread
-        this.thread = new Thread(new ThreadGroup("JobInvoker" + this.hashCode()), this);
+        this.thread = new Thread(this);
         this.name = delegatorName + "-invoker-" + this.thread.getName();
-        
+
         this.thread.setDaemon(false);
         this.thread.setName(this.name);
 

Modified: ofbiz/trunk/framework/service/src/org/ofbiz/service/job/JobManager.java
URL: http://svn.apache.org/viewvc/ofbiz/trunk/framework/service/src/org/ofbiz/service/job/JobManager.java?rev=728536&r1=728535&r2=728536&view=diff
==============================================================================
--- ofbiz/trunk/framework/service/src/org/ofbiz/service/job/JobManager.java (original)
+++ ofbiz/trunk/framework/service/src/org/ofbiz/service/job/JobManager.java Sun Dec 21 14:25:22 2008
@@ -28,7 +28,6 @@
 import javolution.util.FastMap;
 
 import org.ofbiz.base.util.Debug;
-import org.ofbiz.base.util.FileUtil;
 import org.ofbiz.base.util.GeneralRuntimeException;
 import org.ofbiz.base.util.UtilDateTime;
 import org.ofbiz.base.util.UtilMisc;
@@ -311,7 +310,7 @@
      *@param endTime The time in milliseconds the service should expire
      */
     public void schedule(String poolName, String serviceName, Map<String, ? extends Object> context, long startTime, int frequency, int interval, int count, long endTime) throws JobManagerException {
-        schedule(null, null, serviceName, context, startTime, frequency, interval, count, endTime, -1, false);
+        schedule(null, null, serviceName, context, startTime, frequency, interval, count, endTime, -1);
     }
 
     /**
@@ -325,10 +324,9 @@
      *@param interval The interval of the frequency recurrence
      *@param count The number of times to repeat
      *@param endTime The time in milliseconds the service should expire
-     *@param ownLogfile Indicator whether this job uses it's own logfile
      *@param maxRetry The max number of retries on failure (-1 for no max)
      */
-    public void schedule(String jobName, String poolName, String serviceName, Map<String, ? extends Object> context, long startTime, int frequency, int interval, int count, long endTime, int maxRetry, boolean ownLogfile) throws JobManagerException {
+    public void schedule(String jobName, String poolName, String serviceName, Map<String, ? extends Object> context, long startTime, int frequency, int interval, int count, long endTime, int maxRetry) throws JobManagerException {
         if (delegator == null) {
             Debug.logWarning("No delegator referenced; cannot schedule job.", module);
             return;
@@ -350,7 +348,7 @@
         }
 
         // schedule the job
-        schedule(jobName, poolName, serviceName, dataId, startTime, frequency, interval, count, endTime, maxRetry, ownLogfile);
+        schedule(jobName, poolName, serviceName, dataId, startTime, frequency, interval, count, endTime, maxRetry);
     }
 
     /**
@@ -361,7 +359,7 @@
      *@param startTime The time in milliseconds the service should run
      */
     public void schedule(String poolName, String serviceName, String dataId, long startTime) throws JobManagerException {
-        schedule(null, poolName, serviceName, dataId, startTime, -1, 0, 1, 0, -1, false);
+        schedule(null, poolName, serviceName, dataId, startTime, -1, 0, 1, 0, -1);
     }
 
     /**
@@ -376,9 +374,8 @@
      *@param count The number of times to repeat
      *@param endTime The time in milliseconds the service should expire
      *@param maxRetry The max number of retries on failure (-1 for no max)
-     *@param ownLogfile Indicator whether this job uses it's own logfile
      */
-    public void schedule(String jobName, String poolName, String serviceName, String dataId, long startTime, int frequency, int interval, int count, long endTime, int maxRetry, boolean ownLogfile) throws JobManagerException {
+    public void schedule(String jobName, String poolName, String serviceName, String dataId, long startTime, int frequency, int interval, int count, long endTime, int maxRetry) throws JobManagerException {
         if (delegator == null) {
             Debug.logWarning("No delegator referenced; cannot schedule job.", module);
             return;
@@ -414,12 +411,6 @@
 
         // set the max retry
         jFields.put("maxRetry", Long.valueOf(maxRetry));
-        
-        if (ownLogfile) {
-            jFields.put("ownLogfile", "Y");
-        } else {
-            jFields.put("ownLogfile", "N");
-        }
 
         // create the value and store
         GenericValue jobV;

Modified: ofbiz/trunk/framework/service/src/org/ofbiz/service/job/PersistedServiceJob.java
URL: http://svn.apache.org/viewvc/ofbiz/trunk/framework/service/src/org/ofbiz/service/job/PersistedServiceJob.java?rev=728536&r1=728535&r2=728536&view=diff
==============================================================================
--- ofbiz/trunk/framework/service/src/org/ofbiz/service/job/PersistedServiceJob.java (original)
+++ ofbiz/trunk/framework/service/src/org/ofbiz/service/job/PersistedServiceJob.java Sun Dec 21 14:25:22 2008
@@ -174,24 +174,6 @@
             throw new RuntimeException(e.getMessage());
         }
         if (Debug.infoOn()) Debug.logInfo(this.toString() + "[" + getJobId() + "] -- Next runtime: " + new Date(nextRecurrence), module);
-    
-        //set the location of the logfile if this was desired
-        if ("Y".equals(job.get("ownLogfile"))) {
-            String logLocation = System.getProperty("ofbiz.log.dir", "runtime/logs") + "/"
-            + getServiceName()
-            + "_"
-            + UtilDateTime.getTimestamp(System.currentTimeMillis())
-                    .toString().trim().replace(" ", "_") + ".html";
-            this.setLogLocation(logLocation);
-            job.set("logLocation", logLocation);
-            try {
-                job.store();
-            } catch (GenericEntityException e) {
-                Debug.logError(e, "Cannot update the job [" + getJobId() + "] sandbox", module);
-            }
-        }
-        
-    
     }
 
     private void createRecurrence(GenericValue job, long next) throws GenericEntityException {

Modified: ofbiz/trunk/framework/webapp/src/org/ofbiz/webapp/event/CoreEvents.java
URL: http://svn.apache.org/viewvc/ofbiz/trunk/framework/webapp/src/org/ofbiz/webapp/event/CoreEvents.java?rev=728536&r1=728535&r2=728536&view=diff
==============================================================================
--- ofbiz/trunk/framework/webapp/src/org/ofbiz/webapp/event/CoreEvents.java (original)
+++ ofbiz/trunk/framework/webapp/src/org/ofbiz/webapp/event/CoreEvents.java Sun Dec 21 14:25:22 2008
@@ -216,11 +216,6 @@
         String serviceIntr = (String) params.remove("SERVICE_INTERVAL");
         String serviceCnt = (String) params.remove("SERVICE_COUNT");
         String retryCnt = (String) params.remove("SERVICE_MAXRETRY");
-        
-        boolean ownLogfile = false;
-        if ("Y".equals(params.get("OWN_LOGFILE"))) {
-            ownLogfile = true;
-        }
 
         // the frequency map
         Map<String, Integer> freqMap = FastMap.newInstance();
@@ -414,7 +409,7 @@
             if(null!=request.getParameter("_RUN_SYNC_") && request.getParameter("_RUN_SYNC_").equals("Y")){
                 syncServiceResult = dispatcher.runSync(serviceName, serviceContext);
             }else{
-                dispatcher.schedule(jobName, poolName, serviceName, serviceContext, startTime, frequency, interval, count, endTime, maxRetry, ownLogfile);
+                dispatcher.schedule(jobName, poolName, serviceName, serviceContext, startTime, frequency, interval, count, endTime, maxRetry);
             }
         } catch (GenericServiceException e) {
             String errMsg = UtilProperties.getMessage(CoreEvents.err_resource, "coreEvents.service_dispatcher_exception", locale);

Modified: ofbiz/trunk/framework/webtools/webapp/webtools/WEB-INF/actions/log/LogView.groovy
URL: http://svn.apache.org/viewvc/ofbiz/trunk/framework/webtools/webapp/webtools/WEB-INF/actions/log/LogView.groovy?rev=728536&r1=728535&r2=728536&view=diff
==============================================================================
--- ofbiz/trunk/framework/webtools/webapp/webtools/WEB-INF/actions/log/LogView.groovy (original)
+++ ofbiz/trunk/framework/webtools/webapp/webtools/WEB-INF/actions/log/LogView.groovy Sun Dec 21 14:25:22 2008
@@ -19,13 +19,6 @@
 
 import org.ofbiz.base.util.FileUtil;
 
-if (parameters.jobId!=null) {
-    value = delegator.findByPrimaryKey("JobSandbox", [jobId:parameters.jobId]);
-    if (value.getString("logLocation") != null) {
-        context.logFileName = value.getString("logLocation");
-        logFileName = value.getString("logLocation");
-    }
-}
 sb = null;
 try {
     sb = FileUtil.readTextFile(logFileName, true);

Modified: ofbiz/trunk/framework/webtools/webapp/webtools/service/ServiceForms.xml
URL: http://svn.apache.org/viewvc/ofbiz/trunk/framework/webtools/webapp/webtools/service/ServiceForms.xml?rev=728536&r1=728535&r2=728536&view=diff
==============================================================================
--- ofbiz/trunk/framework/webtools/webapp/webtools/service/ServiceForms.xml (original)
+++ ofbiz/trunk/framework/webtools/webapp/webtools/service/ServiceForms.xml Sun Dec 21 14:25:22 2008
@@ -43,7 +43,6 @@
         <field name="SERVICE_INTERVAL" title="${uiLabelMap.WebtoolsInterval}" tooltip="${uiLabelMap.WebtoolsMessage8}"><text/></field>
         <field name="SERVICE_COUNT" title="${uiLabelMap.WebtoolsCount}" tooltip="${uiLabelMap.WebtoolsMessage9}"><text default-value="1"/></field>
         <field name="SERVICE_MAXRETRY" title="${uiLabelMap.WebtoolsMaxRetry}" tooltip="${uiLabelMap.WebtoolsMessage10}"><text/></field>
-        <field name="OWN_LOGFILE" title="${uiLabelMap.WebtoolsOwnLogfile}"><check/></field>
         <field name="submitButton" title="${uiLabelMap.CommonSubmit}"><submit button-type="button"/></field>
     </form>
 
@@ -73,9 +72,6 @@
     <form name="JobList" title="" target="" type="list" list-name="jobs"
         paginate-target="jobList" override-list-size="${jobListSize}"
         odd-row-style="alternate-row" default-table-style="basic-table hover-bar">
-        <row-actions>
-            <set field="ownLogfile" from-field="ownLogfile" default-value="N"/>
-        </row-actions>
         <field name="jobName" title="${uiLabelMap.WebtoolsJob}"><display/></field>
         <field name="jobId" title="${uiLabelMap.CommonId}"><display/></field>
         <field name="poolId" title="${uiLabelMap.WebtoolsPool}"><display/></field>
@@ -86,11 +82,8 @@
         </field>
         <field name="statusId" title="${uiLabelMap.CommonStatus}"><display-entity entity-name="StatusItem" description="${description}"/></field>
         <field name="cancelDateTime" title="${uiLabelMap.CommonEndDateTime}"><display/></field>
-        <field name="ownLogfile" title="${uiLabelMap.WebtoolsOwnLogfile}"><display/></field>
-        <field name="logFile" use-when="logLocation != null" title="${uiLabelMap.WebtoolsJobLog}"><hyperlink target="LogView?jobId=${jobId}" description="${uiLabelMap.WebtoolsJobLog}"/></field>
-        <field name="logFile" use-when="logLocation == null" title="${uiLabelMap.WebtoolsJobLog}"><display/></field>
         <field name="cancelButton" title="${uiLabelMap.CommonEmptyHeader}" use-when="startDateTime==null&amp;&amp;finishDateTime==null&amp;&amp;cancelDateTime==null" widget-style="buttontext">
             <hyperlink also-hidden="false" description="${uiLabelMap.WebtoolsCancelJob}" target="cancelJob?jobId=${jobId}"/>
         </field>
     </form>
-</forms>
+</forms>
\ No newline at end of file