Revision: 4757
          http://sourceforge.net/p/jump-pilot/code/4757
Author:   edso
Date:     2016-01-10 16:27:48 +0000 (Sun, 10 Jan 2016)
Log Message:
-----------
adapt to new Logger

Modified Paths:
--------------
    core/trunk/src/com/vividsolutions/jump/I18N.java
    core/trunk/src/com/vividsolutions/jump/io/datasource/DataSource.java
    core/trunk/src/com/vividsolutions/jump/tools/OverlayEngine.java
    core/trunk/src/com/vividsolutions/jump/util/java2xml/Java2XML.java
    core/trunk/src/com/vividsolutions/jump/util/java2xml/XML2Java.java
    
core/trunk/src/com/vividsolutions/jump/workbench/datasource/AbstractLoadDatasetPlugIn.java
    
core/trunk/src/com/vividsolutions/jump/workbench/datasource/LoadDatasetPlugIn.java
    
core/trunk/src/com/vividsolutions/jump/workbench/imagery/ecw/ECWImageFactory.java
    
core/trunk/src/com/vividsolutions/jump/workbench/imagery/mrsid/MrSIDImageFactory.java
    core/trunk/src/com/vividsolutions/jump/workbench/model/WMSLayer.java
    core/trunk/src/com/vividsolutions/jump/workbench/plugin/AbstractPlugIn.java
    core/trunk/src/com/vividsolutions/jump/workbench/plugin/PlugInManager.java

Modified: core/trunk/src/com/vividsolutions/jump/I18N.java
===================================================================
--- core/trunk/src/com/vividsolutions/jump/I18N.java    2016-01-10 16:20:45 UTC 
(rev 4756)
+++ core/trunk/src/com/vividsolutions/jump/I18N.java    2016-01-10 16:27:48 UTC 
(rev 4757)
@@ -40,14 +40,16 @@
 import java.util.Map;
 import java.util.ResourceBundle;
 
-import org.apache.log4j.Logger;
+import com.vividsolutions.jump.workbench.Logger;
 
+
+
 /**
  * Singleton for the Internationalization (I18N)
  **/
 public final class I18N {
 
-  private static final Logger LOG = Logger.getLogger(I18N.class);
+
   private static final I18N instance = new I18N();
 
   // [Michael Michaud 2007-03-23] plugInsResourceBundle is deactivated because
@@ -159,13 +161,11 @@
       return resourceBundle3.getString(key);
     } catch (java.util.MissingResourceException e) {
       if (!missing.contains(key)) {
-        String msg = getClass().getName()
-            + "\nNo resource bundle or no translation found for''{0}''.\nError 
was:\n{1}";
-        msg = new MessageFormat(msg).format(new String[] { key,
-            e.getLocalizedMessage() });
-        LOG.debug(msg);
-        System.out.println("Missing translation for '" + key
-            + "' in resource bundle '" + this.resourcePath + "'.");
+        String msg = "No translation for key ''{0}'' in bundle ''{1}''.";
+        msg = MessageFormat.format(msg, key, resourcePath);
+
+        Logger.warn(msg, Logger.isDebugEnabled() ? e : null);
+
         missing.add(key);
         // uncomment and add a search string to get staks telling you where the
         // call came from
@@ -300,13 +300,13 @@
     String[] lc = localeCode.split("_");
     Locale locale = Locale.getDefault();
     if (lc.length > 1) {
-      LOG.debug("lang:" + lc[0] + " " + "country:" + lc[1]);
+      Logger.debug("lang:" + lc[0] + " " + "country:" + lc[1]);
       locale = new Locale(lc[0], lc[1]);
     } else if (lc.length > 0) {
-      LOG.debug("lang:" + lc[0]);
+      Logger.debug("lang:" + lc[0]);
       locale = new Locale(lc[0]);
     } else {
-      LOG.debug(localeCode
+      Logger.debug(localeCode
           + " is an illegal argument to define lang [and country]");
     }
 
@@ -430,8 +430,9 @@
       return mformat.format(objects);
     } catch (java.util.MissingResourceException e) {
       final String[] labelpath = label.split("\\.");
-      LOG.warn(e.getMessage() + " no default value, the resource key is used: "
+      Logger.warn(e.getMessage() + " no default value, the resource key is 
used: "
           + labelpath[labelpath.length - 1]);
+
       final MessageFormat mformat = new MessageFormat(
           labelpath[labelpath.length - 1]);
       return mformat.format(objects);

Modified: core/trunk/src/com/vividsolutions/jump/io/datasource/DataSource.java
===================================================================
--- core/trunk/src/com/vividsolutions/jump/io/datasource/DataSource.java        
2016-01-10 16:20:45 UTC (rev 4756)
+++ core/trunk/src/com/vividsolutions/jump/io/datasource/DataSource.java        
2016-01-10 16:27:48 UTC (rev 4757)
@@ -34,8 +34,8 @@
 import com.vividsolutions.jump.coordsys.CoordinateSystem;
 import com.vividsolutions.jump.coordsys.CoordinateSystemRegistry;
 import com.vividsolutions.jump.feature.FeatureCollection;
-import org.apache.log4j.Logger;
 
+
 import java.util.HashMap;
 import java.util.Map;
 /**
@@ -47,8 +47,6 @@
  */
 public abstract class DataSource {
 
-       private static Logger LOG = Logger.getLogger(DataSource.class);
-       
     private Map properties;
 
     /**

Modified: core/trunk/src/com/vividsolutions/jump/tools/OverlayEngine.java
===================================================================
--- core/trunk/src/com/vividsolutions/jump/tools/OverlayEngine.java     
2016-01-10 16:20:45 UTC (rev 4756)
+++ core/trunk/src/com/vividsolutions/jump/tools/OverlayEngine.java     
2016-01-10 16:27:48 UTC (rev 4757)
@@ -36,13 +36,19 @@
 import java.util.Iterator;
 import java.util.List;
 
-import org.apache.log4j.Logger;
-
-import com.vividsolutions.jts.geom.*;
+import com.vividsolutions.jts.geom.Geometry;
+import com.vividsolutions.jts.geom.GeometryCollection;
+import com.vividsolutions.jts.geom.MultiPolygon;
+import com.vividsolutions.jts.geom.Polygon;
 import com.vividsolutions.jts.precision.EnhancedPrecisionOp;
 import com.vividsolutions.jump.I18N;
-import com.vividsolutions.jump.feature.*;
+import com.vividsolutions.jump.feature.BasicFeature;
+import com.vividsolutions.jump.feature.Feature;
+import com.vividsolutions.jump.feature.FeatureCollection;
+import com.vividsolutions.jump.feature.FeatureDataset;
+import com.vividsolutions.jump.feature.IndexedFeatureCollection;
 import com.vividsolutions.jump.task.TaskMonitor;
+import com.vividsolutions.jump.workbench.Logger;
 
 
 /**
@@ -50,7 +56,7 @@
  *  FeatureCollection containing the intersections of all pairs of input 
features.
  */
 public class OverlayEngine {
-       private static Logger LOG = Logger.getLogger(OverlayEngine.class);
+
     private boolean splittingGeometryCollections = true;
     private boolean allowingPolygonsOnly = true;
 
@@ -127,8 +133,8 @@
                     b.getGeometry());
         } catch (Exception ex) {
             monitor.report(ex);
-            LOG.error(a.getGeometry());
-            LOG.error(b.getGeometry());
+            Logger.error(a.getGeometry().toString());
+            Logger.error(b.getGeometry().toString());
         }
 
         if ((intersection == null) || intersection.isEmpty()) {

Modified: core/trunk/src/com/vividsolutions/jump/util/java2xml/Java2XML.java
===================================================================
--- core/trunk/src/com/vividsolutions/jump/util/java2xml/Java2XML.java  
2016-01-10 16:20:45 UTC (rev 4756)
+++ core/trunk/src/com/vividsolutions/jump/util/java2xml/Java2XML.java  
2016-01-10 16:27:48 UTC (rev 4757)
@@ -25,14 +25,12 @@
  * (250)385-6040 www.vividsolutions.com
  */
 package com.vividsolutions.jump.util.java2xml;
-import org.apache.log4j.Logger;
-import org.jdom.Attribute;
-import org.jdom.Document;
-import org.jdom.Element;
-import org.jdom.output.Format;
-import org.jdom.output.XMLOutputter;
 
-import java.io.*;
+import java.io.BufferedWriter;
+import java.io.File;
+import java.io.FileWriter;
+import java.io.StringWriter;
+import java.io.Writer;
 import java.lang.reflect.Method;
 import java.util.ArrayList;
 import java.util.Collection;
@@ -41,8 +39,16 @@
 import java.util.Map;
 
 import javax.xml.namespace.QName;
+
+import org.jdom.Attribute;
+import org.jdom.Document;
+import org.jdom.Element;
+import org.jdom.output.Format;
+import org.jdom.output.XMLOutputter;
+
+import com.vividsolutions.jump.workbench.Logger;
 public class Java2XML extends XMLBinder {
-       private static Logger LOG = Logger.getLogger(Java2XML.class);
+
     public Java2XML() {
     }
     public String write(Object object, String rootTagName) throws Exception {
@@ -115,7 +121,7 @@
                 }
             }, object.getClass());
         } catch (Exception e) {
-               LOG.error("Java2XML: Exception writing "
+               Logger.error("Java2XML: Exception writing "
                     + object.getClass());
             throw e;
         }

Modified: core/trunk/src/com/vividsolutions/jump/util/java2xml/XML2Java.java
===================================================================
--- core/trunk/src/com/vividsolutions/jump/util/java2xml/XML2Java.java  
2016-01-10 16:20:45 UTC (rev 4756)
+++ core/trunk/src/com/vividsolutions/jump/util/java2xml/XML2Java.java  
2016-01-10 16:27:48 UTC (rev 4757)
@@ -25,21 +25,32 @@
  * (250)385-6040 www.vividsolutions.com
  */
 package com.vividsolutions.jump.util.java2xml;
-import com.vividsolutions.jts.util.Assert;
-import com.vividsolutions.jump.util.StringUtil;
-import org.apache.log4j.Logger;
+import java.io.BufferedInputStream;
+import java.io.File;
+import java.io.FileInputStream;
+import java.io.InputStream;
+import java.io.Reader;
+import java.io.StringReader;
+import java.lang.reflect.InvocationTargetException;
+import java.lang.reflect.Method;
+import java.util.ArrayList;
+import java.util.Collection;
+import java.util.Iterator;
+import java.util.List;
+import java.util.Map;
+
+import javax.xml.namespace.QName;
+
 import org.jdom.Element;
 import org.jdom.input.SAXBuilder;
 
-import javax.xml.namespace.QName;
-import java.io.*;
-import java.lang.reflect.InvocationTargetException;
-import java.lang.reflect.Method;
-import java.util.*;
+import com.vividsolutions.jts.util.Assert;
+import com.vividsolutions.jump.util.StringUtil;
+import com.vividsolutions.jump.workbench.Logger;
 public class XML2Java extends XMLBinder {
     private ArrayList listeners = new ArrayList();
     private ClassLoader classLoader = getClass().getClassLoader();
-       private static Logger LOG = Logger.getLogger(XMLBinder.class);
+
     public XML2Java() {
     }
     public XML2Java(ClassLoader classLoader) {
@@ -129,7 +140,7 @@
                     // [sstein 5April2008] replaced XMLB exception by Log
                     // so when a problem with styling appears data are still 
loaded
                     if (tag.getName().equalsIgnoreCase("style")){
-                       LOG.warn(msg);
+                       Logger.warn(msg);
                        System.out.println(msg);
                        return; //return to avoid further messages
                     }
@@ -139,7 +150,7 @@
                         (xmlName.equalsIgnoreCase("editable")   || 
                          xmlName.equalsIgnoreCase("selectable") ||
                          xmlName.equalsIgnoreCase("read-only") ) ) {
-                       LOG.warn(msg);
+                       Logger.warn(msg);
                        System.out.println(msg);
                        return; //return to avoid further messages
                     }
@@ -169,7 +180,7 @@
             try {
                 return read(tag, Class.forName(tag.getAttributeValue("class"), 
true, classLoader));
             } catch (ClassNotFoundException e) {
-                LOG.error("Could not find class for " + tag, e);
+                Logger.error("Could not find class for " + tag, e);
                 System.out.println("Class not found for tag " + tag.getName() 
+ ": " + tag.getAttribute("class").getValue());
                 throw e;
             }

Modified: 
core/trunk/src/com/vividsolutions/jump/workbench/datasource/AbstractLoadDatasetPlugIn.java
===================================================================
--- 
core/trunk/src/com/vividsolutions/jump/workbench/datasource/AbstractLoadDatasetPlugIn.java
  2016-01-10 16:20:45 UTC (rev 4756)
+++ 
core/trunk/src/com/vividsolutions/jump/workbench/datasource/AbstractLoadDatasetPlugIn.java
  2016-01-10 16:27:48 UTC (rev 4757)
@@ -7,8 +7,8 @@
 import java.util.Collection;
 import java.util.Iterator;
 
-import org.apache.log4j.Logger;
 
+
 import com.vividsolutions.jts.util.Assert;
 import com.vividsolutions.jump.I18N;
 import com.vividsolutions.jump.coordsys.CoordinateSystemRegistry;

Modified: 
core/trunk/src/com/vividsolutions/jump/workbench/datasource/LoadDatasetPlugIn.java
===================================================================
--- 
core/trunk/src/com/vividsolutions/jump/workbench/datasource/LoadDatasetPlugIn.java
  2016-01-10 16:20:45 UTC (rev 4756)
+++ 
core/trunk/src/com/vividsolutions/jump/workbench/datasource/LoadDatasetPlugIn.java
  2016-01-10 16:27:48 UTC (rev 4757)
@@ -62,10 +62,7 @@
 import java.util.Collection;
 import java.util.Iterator;
 
-import org.apache.log4j.Logger;
-
 import javax.swing.ImageIcon;
-import javax.swing.SwingUtilities;
 import javax.swing.JFileChooser;
 import java.awt.event.*;
 

Modified: 
core/trunk/src/com/vividsolutions/jump/workbench/imagery/ecw/ECWImageFactory.java
===================================================================
--- 
core/trunk/src/com/vividsolutions/jump/workbench/imagery/ecw/ECWImageFactory.java
   2016-01-10 16:20:45 UTC (rev 4756)
+++ 
core/trunk/src/com/vividsolutions/jump/workbench/imagery/ecw/ECWImageFactory.java
   2016-01-10 16:27:48 UTC (rev 4757)
@@ -42,9 +42,6 @@
 import com.vividsolutions.jump.workbench.imagery.ReferencedImage;
 import com.vividsolutions.jump.workbench.imagery.ReferencedImageFactory;
 
-import org.apache.log4j.Logger;
-import org.openjump.util.UriUtil;
-
 /**
  */
 public class ECWImageFactory implements ReferencedImageFactory {

Modified: 
core/trunk/src/com/vividsolutions/jump/workbench/imagery/mrsid/MrSIDImageFactory.java
===================================================================
--- 
core/trunk/src/com/vividsolutions/jump/workbench/imagery/mrsid/MrSIDImageFactory.java
       2016-01-10 16:20:45 UTC (rev 4756)
+++ 
core/trunk/src/com/vividsolutions/jump/workbench/imagery/mrsid/MrSIDImageFactory.java
       2016-01-10 16:27:48 UTC (rev 4757)
@@ -38,7 +38,6 @@
 import java.net.URI;
 import java.util.HashMap;
 
-import org.apache.log4j.Logger;
 import org.openjump.core.CheckOS;
 import org.openjump.util.UriUtil;
 
@@ -52,9 +51,6 @@
 
 public class MrSIDImageFactory implements ReferencedImageFactory {
 
-  private Logger logger = Logger.getLogger(MrSIDImageFactory.class);
-
-  // --
   public static final String MRSIDDECODE = "mrsidgeodecode";
   public static final String MRSIDINFO = "mrsidgeoinfo";
   private static HashMap<String, String> binariesAvailable = new HashMap();

Modified: core/trunk/src/com/vividsolutions/jump/workbench/model/WMSLayer.java
===================================================================
--- core/trunk/src/com/vividsolutions/jump/workbench/model/WMSLayer.java        
2016-01-10 16:20:45 UTC (rev 4756)
+++ core/trunk/src/com/vividsolutions/jump/workbench/model/WMSLayer.java        
2016-01-10 16:27:48 UTC (rev 4757)
@@ -35,6 +35,9 @@
 import java.awt.Image;
 import java.awt.MediaTracker;
 import java.io.IOException;
+import java.lang.ref.Reference;
+import java.lang.ref.SoftReference;
+import java.net.URL;
 import java.util.ArrayList;
 import java.util.Collections;
 import java.util.List;
@@ -44,30 +47,20 @@
 import com.vividsolutions.jts.geom.Envelope;
 import com.vividsolutions.jts.util.Assert;
 import com.vividsolutions.jump.util.Blackboard;
+import com.vividsolutions.jump.workbench.Logger;
 import com.vividsolutions.jump.workbench.ui.LayerNameRenderer;
 import com.vividsolutions.jump.workbench.ui.LayerViewPanel;
 import com.vividsolutions.jump.workbench.ui.renderer.RenderingManager;
-import com.vividsolutions.wms.AxisOrder;
 import com.vividsolutions.wms.BoundingBox;
 import com.vividsolutions.wms.MapLayer;
 import com.vividsolutions.wms.MapRequest;
 import com.vividsolutions.wms.WMService;
 
-import java.net.URL;
-
-import java.lang.ref.Reference;
-import java.lang.ref.SoftReference;
-import java.util.Iterator;
-
-import org.apache.log4j.Logger;
-
 /**
  * A Layerable that retrieves images from a Web Map Server.
  */
 public class WMSLayer extends AbstractLayerable implements Cloneable {
 
-  private static Logger LOG = Logger.getLogger(WMSLayer.class);
-
   private String format;
 
   private List<String> layerNames = new ArrayList<String>();
@@ -290,8 +283,7 @@
         }
       }
     } catch (IOException e) {
-      LOG.error("Exception caught during WMSLayer envelope calculation: "
-          + e.toString());
+      Logger.error("Exception caught during WMSLayer envelope calculation.", 
e);
     }
     return envelope;
   }

Modified: 
core/trunk/src/com/vividsolutions/jump/workbench/plugin/AbstractPlugIn.java
===================================================================
--- core/trunk/src/com/vividsolutions/jump/workbench/plugin/AbstractPlugIn.java 
2016-01-10 16:20:45 UTC (rev 4756)
+++ core/trunk/src/com/vividsolutions/jump/workbench/plugin/AbstractPlugIn.java 
2016-01-10 16:27:48 UTC (rev 4757)
@@ -48,12 +48,11 @@
 import javax.swing.KeyStroke;
 import javax.swing.undo.UndoableEdit;
 
-import org.apache.log4j.Logger;
-
 import com.vividsolutions.jts.util.Assert;
 import com.vividsolutions.jump.I18N;
 import com.vividsolutions.jump.util.StringUtil;
 import com.vividsolutions.jump.workbench.JUMPWorkbench;
+import com.vividsolutions.jump.workbench.Logger;
 import com.vividsolutions.jump.workbench.WorkbenchContext;
 import com.vividsolutions.jump.workbench.model.CategoryEvent;
 import com.vividsolutions.jump.workbench.model.FeatureEvent;
@@ -74,7 +73,7 @@
  * example), and supporting undo.
  */
 public abstract class AbstractPlugIn implements PlugIn, ShortcutEnabled, 
EnableChecked, Iconified, Recordable {
-  private static Logger LOG = Logger.getLogger(AbstractPlugIn.class);
+
   protected int shortcutModifiers = 0;
   protected int shortcutKeys = 0;
   private String name;
@@ -297,7 +296,7 @@
       return I18N.get(plugInClass.getName());
     } catch (java.util.MissingResourceException e) {
       // No I18N for the PlugIn so log it, but don't stop
-      LOG.error(e.getMessage() + " " + plugInClass.getName());
+      Logger.error(e.getMessage() + " " + plugInClass.getName());
       return StringUtil.toFriendlyName(plugInClass.getName(), "PlugIn");
     }
   }

Modified: 
core/trunk/src/com/vividsolutions/jump/workbench/plugin/PlugInManager.java
===================================================================
--- core/trunk/src/com/vividsolutions/jump/workbench/plugin/PlugInManager.java  
2016-01-10 16:20:45 UTC (rev 4756)
+++ core/trunk/src/com/vividsolutions/jump/workbench/plugin/PlugInManager.java  
2016-01-10 16:27:48 UTC (rev 4757)
@@ -43,12 +43,11 @@
 import java.util.zip.ZipException;
 import java.util.zip.ZipFile;
 
-import org.apache.log4j.Logger;
-
 import com.vividsolutions.jts.util.Assert;
 import com.vividsolutions.jump.I18N;
 import com.vividsolutions.jump.task.TaskMonitor;
 import com.vividsolutions.jump.util.StringUtil;
+import com.vividsolutions.jump.workbench.Logger;
 import com.vividsolutions.jump.workbench.WorkbenchContext;
 import com.vividsolutions.jump.workbench.WorkbenchProperties;
 import com.vividsolutions.jump.workbench.ui.plugin.FeatureInstaller;
@@ -58,8 +57,6 @@
  * depend on, from the plug-in directory.
  */
 public class PlugInManager {
-    private static Logger LOG = Logger.getLogger(PlugInManager.class);
-
     private static final String NOT_INITIALIZED = 
"com.vividsolutions.jump.workbench.plugin.PlugInManager.could-not-be-initialized";
     private static final String LOADING = 
"com.vividsolutions.jump.workbench.plugin.PlugInManager.loading";
     private static final String LOADING_ERROR = 
"com.vividsolutions.jump.workbench.plugin.PlugInManager.throwable-encountered-loading";
@@ -160,7 +157,7 @@
       if (plugInDirectory != null) {
         start = secondsSince(0);
         configurations.addAll(findConfigurations(plugInDirectory));
-        System.out.println("Finding all OJ extensions took "
+        Logger.info("Finding all OJ extensions took "
             + secondsSinceString(start) + "s");
       }
   
@@ -169,7 +166,7 @@
   
       start = secondsSince(0);
       loadConfigurations();
-      System.out.println("Loading all OJ extensions took "
+      Logger.info("Loading all OJ extensions took "
           + secondsSinceString(start) + "s");
       
       // enable autoseparating in installer for plugins possibly installed 
later on
@@ -190,8 +187,7 @@
           // we used the plugin classloader to instantiate extensions already 
above
           configuration.configure(pc);
           
//System.out.println(Arrays.toString(((URLClassLoader)classLoader).getURLs()));
-          System.out
-          .println("Loading Config " + name(configuration) + " "
+          Logger.info("Loading Config " + name(configuration) + " "
               + version(configuration) + " took " + secondsSinceString(start)
               + "s");
         }
@@ -375,7 +371,7 @@
         } catch (ZipException e) {
           // Might not be a zipfile. Eat it. [Jon Aquino]
         }
-        System.out.println("Scanning " + file + " took " + 
secondsSinceString(start)
+        Logger.info("Scanning " + file + " took " + secondsSinceString(start)
             + "s");
       }
   
@@ -457,11 +453,10 @@
                     + ". Refine class name algorithm.");
             return null;
         } catch (Throwable t) {
-            LOG.error(I18N.get(LOADING_ERROR) + " " + className + ":");
+            Logger.error(I18N.get(LOADING_ERROR) + " " + className + ":");
             //e.g. java.lang.VerifyError: class
             // org.apache.xml.serialize.XML11Serializer
             //overrides final method [Jon Aquino]
-            t.printStackTrace(System.out);
             return null;
         }
         return candidate;


------------------------------------------------------------------------------
Site24x7 APM Insight: Get Deep Visibility into Application Performance
APM + Mobile APM + RUM: Monitor 3 App instances at just $35/Month
Monitor end-to-end web transactions and take corrective actions now
Troubleshoot faster and improve end-user experience. Signup Now!
http://pubads.g.doubleclick.net/gampad/clk?id=267308311&iu=/4140
_______________________________________________
Jump-pilot-devel mailing list
Jump-pilot-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/jump-pilot-devel

Reply via email to