Ignore:
Timestamp:
2014-11-12T01:51:41+01:00 (10 years ago)
Author:
donvip
Message:

[josm_opendata] global use of Main.info/warn/error

Location:
applications/editors/josm/plugins/opendata/src/org/openstreetmap/josm/plugins/opendata/core
Files:
18 edited

Legend:

Unmodified
Added
Removed

  • applications/editors/josm/plugins/opendata/src/org/openstreetmap/josm/plugins/opendata/core/actions/OpenLinkAction.java

    r30723 r30795  
    1010import javax.swing.Action;
    1111
     12import org.openstreetmap.josm.Main;
    1213import org.openstreetmap.josm.actions.JosmAction;
    1314import org.openstreetmap.josm.plugins.opendata.core.OdConstants;
    … …  
    1516import org.openstreetmap.josm.tools.OpenBrowser;
    1617
    17 @SuppressWarnings("serial")
    1818public class OpenLinkAction extends JosmAction {
    1919
    … …  
    2929    public void actionPerformed(ActionEvent e) {
    3030        try {
    31             System.out.println("Opening "+url);
     31            Main.info("Opening "+url);
    3232            OpenBrowser.displayUrl(url.toURI());
    3333        } catch (URISyntaxException e1) {
    3434            try {
    35                 System.err.println(e1.getLocalizedMessage());
     35                Main.error(e1.getLocalizedMessage());
    3636                int index = e1.getIndex();
    3737                if (index > -1) {
    … …  
    3939                    s = url.toString().replace(s, URLEncoder.encode(s, OdConstants.UTF8));
    4040                    URI uri = new URI(s);
    41                     System.out.println("Opening "+uri);
     41                    Main.info("Opening "+uri);
    4242                    OpenBrowser.displayUrl(uri);
    4343                }
    4444            } catch (Exception e2) {
    45                 e2.printStackTrace();
     45                Main.error(e2);
    4646            }
    4747        }

  • applications/editors/josm/plugins/opendata/src/org/openstreetmap/josm/plugins/opendata/core/io/NeptuneReader.java

    r30738 r30795  
    134134        p.put("ref:neptune", object.getObjectId());
    135135        if (tridentObjects.put(object.getObjectId(), p) != null) {
    136             System.err.println("Trident object duplicated !!! : "+object.getObjectId());
     136            Main.error("Trident object duplicated !!! : "+object.getObjectId());
    137137        }
    138138    }
    … …  
    151151        return n;
    152152    }
    153     /*
    154     private Node createStopPosition(StopPointType stop) {
    155         Node n = createNode(createLatLon(stop));
    156         n.put(OSM_PUBLIC_TRANSPORT, OSM_STOP_POSITION);
    157         linkTridentObjectToOsmPrimitive(stop, n);
    158         n.put("name", stop.getName());
    159         return n;
    160     }
    161 */
     153
    162154    protected Relation createRelation(String type){
    163155        Relation r = new Relation();
    … …  
    199191            r.put(OSM_ROUTE_MASTER, OSM_TROLLEYBUS); break;
    200192        default:
    201             System.err.println("Unsupported transport mode: "+line.getTransportModeName());
     193            Main.warn("Unsupported transport mode: "+line.getTransportModeName());
    202194        }
    203195        r.put("ref", line.getNumber());
    … …  
    265257                        StopArea child = findStopArea(childId);
    266258                        if (child == null) {
    267                             System.err.println("Cannot find StopArea: "+childId);
     259                            Main.warn("Cannot find StopArea: "+childId);
    268260                        } else {
    269261                            if (child.getStopAreaExtension().getAreaType().equals(ChouetteAreaType.BOARDING_POSITION)) {
    … …  
    272264                                        StopPoint grandchild = findStopPoint(grandchildId);
    273265                                        if (grandchild == null) {
    274                                             System.err.println("Cannot find StopPoint: "+grandchildId);
     266                                            Main.warn("Cannot find StopPoint: "+grandchildId);
    275267                                        } else {
    276268                                            if (grandchild.getLongLatType().equals(LongLatTypeType.WGS_84)) {
    … …  
    278270                                                stopArea.addMember(new RelationMember(OSM_PLATFORM, platform));
    279271                                            } else {
    280                                                 System.err.println("Unsupported long/lat type: "+grandchild.getLongLatType());
     272                                                Main.warn("Unsupported long/lat type: "+grandchild.getLongLatType());
    281273                                            }
    282274                                        }
    283275                                    } else {
    284                                         System.err.println("Unsupported grandchild: "+grandchildId);
     276                                        Main.warn("Unsupported grandchild: "+grandchildId);
    285277                                    }
    286278                                }
    … …  
    288280                                AreaCentroid areaCentroid = findAreaCentroid(centroidId);
    289281                                if (areaCentroid == null) {
    290                                     System.err.println("Cannot find AreaCentroid: "+centroidId);
     282                                    Main.warn("Cannot find AreaCentroid: "+centroidId);
    291283                                } else if (!areaCentroid.getLongLatType().equals(LongLatTypeType.WGS_84)) {
    292                                     System.err.println("Unsupported long/lat type: "+areaCentroid.getLongLatType());
     284                                    Main.warn("Unsupported long/lat type: "+areaCentroid.getLongLatType());
    293285                                } else {
    294286                                    for (RelationMember member : stopArea.getMembers()) {
    … …  
    300292                                }
    301293                            } else {
    302                                 System.err.println("Unsupported child type: "+child.getStopAreaExtension().getAreaType());
     294                                Main.warn("Unsupported child type: "+child.getStopAreaExtension().getAreaType());
    303295                            }
    304296                        }
    … …  
    307299                        StopPoint child = findStopPoint(childId);
    308300                        if (child == null) {
    309                             System.err.println("Cannot find StopPoint: "+childId);
     301                            Main.warn("Cannot find StopPoint: "+childId);
    310302                        } else {
    311303                            // TODO
    … …  
    333325                PTLinkType ptlink = findPtLink(id);
    334326                if (ptlink == null) {
    335                     System.err.println("Cannot find PTLinkType: "+id);
     327                    Main.warn("Cannot find PTLinkType: "+id);
    336328                } else {
    337329                    /*StopPoint start = findStopPoint(ptlink.getStartOfLink());
    … …  
    340332                    OsmPrimitive end = tridentObjects.get(ptlink.getEndOfLink());
    341333                    if (start == null) {
    342                         System.err.println("Cannot find start StopPoint: "+ptlink.getStartOfLink());
     334                        Main.warn("Cannot find start StopPoint: "+ptlink.getStartOfLink());
    343335                    } else if (start.get(OSM_PUBLIC_TRANSPORT).equals(OSM_STOP) || start.get(OSM_PUBLIC_TRANSPORT).equals(OSM_PLATFORM)) {
    344336                        addStopToRoute(route, start);
    … …  
    346338
    347339                    if (end == null) {
    348                         System.err.println("Cannot find end StopPoint: "+ptlink.getEndOfLink());
     340                        Main.warn("Cannot find end StopPoint: "+ptlink.getEndOfLink());
    349341                    } else if (end.get(OSM_PUBLIC_TRANSPORT).equals(OSM_STOP) || end.get(OSM_PUBLIC_TRANSPORT).equals(OSM_PLATFORM)) {
    350342                        addStopToRoute(route, end);

  • applications/editors/josm/plugins/opendata/src/org/openstreetmap/josm/plugins/opendata/core/io/NetworkReader.java

    r30723 r30795  
    1111import java.util.regex.Pattern;
    1212
     13import org.openstreetmap.josm.Main;
    1314import org.openstreetmap.josm.data.osm.DataSet;
    1415import org.openstreetmap.josm.gui.progress.ProgressMonitor;
    … …  
    109110            //return CsvReader.class;//FIXME, can also be .tar.gz
    110111        } else {
    111             System.err.println("Unsupported content type: "+contentType);
     112            Main.warn("Unsupported content type: "+contentType);
    112113        }
    113114        return null;

  • applications/editors/josm/plugins/opendata/src/org/openstreetmap/josm/plugins/opendata/core/io/archive/ArchiveReader.java

    r30568 r30795  
    120120            }
    121121        } catch (IllegalArgumentException e) {
    122             System.err.println(e.getMessage());
     122            Main.error(e.getMessage());
    123123        } finally {
    124124            OdUtils.deleteDir(temp);
    … …  
    168168                    from = NeptuneReader.parseDataSet(in, handler, instance);
    169169                } else {
    170                     System.err.println("Unsupported XML file: "+f.getName());
     170                    Main.warn("Unsupported XML file: "+f.getName());
    171171                }
    172                
    173172            } else {
    174                 System.err.println("Unsupported file extension: "+f.getName());
     173                Main.warn("Unsupported file extension: "+f.getName());
    175174            }
    176175            return from;

  • applications/editors/josm/plugins/opendata/src/org/openstreetmap/josm/plugins/opendata/core/io/geographic/AbstractMapInfoReader.java

    r30723 r30795  
    1212import java.util.List;
    1313
     14import org.openstreetmap.josm.Main;
    1415import org.openstreetmap.josm.io.AbstractReader;
    1516import org.openstreetmap.josm.plugins.opendata.core.OdConstants;
    … …  
    6768            return Charset.forName(OdConstants.MAC_ROMAN);
    6869        } else {
    69             System.err.println("Line "+lineNum+". Unknown charset detected: "+line);
     70            Main.error("Line "+lineNum+". Unknown charset detected: "+line);
    7071            return Charset.forName(words[index]);
    7172        }

  • applications/editors/josm/plugins/opendata/src/org/openstreetmap/josm/plugins/opendata/core/io/geographic/DefaultShpHandler.java

    r30723 r30795  
    102102                                            result.add(CRS.findMathTransform(CRS.decode(p.toCode()), targetCRS, lenient));
    103103                                        } catch (FactoryException e) {
    104                                             System.err.println(e.getMessage());
     104                                            Main.error(e.getMessage());
    105105                                        }
    106106                                    }
    … …  
    113113            if (!result.isEmpty()) {
    114114                if (result.size() > 1) {
    115                     System.err.println("Found multiple projections !"); // TODO: something
     115                    Main.warn("Found multiple projections !"); // TODO: something
    116116                }
    117117                return result.get(0);

  • applications/editors/josm/plugins/opendata/src/org/openstreetmap/josm/plugins/opendata/core/io/geographic/GeographicReader.java

    r30723 r30795  
    152152                    w.addNode(createOrGetNode(ls.getPointN(i)));
    153153                } catch (Exception e) {
    154                     System.err.println(e.getMessage());
     154                    Main.error(e.getMessage());
    155155                }
    156156            }
    … …  
    278278                    } catch (FactoryException ex) {
    279279                        // Silently ignore exceptions
    280                         //System.err.println("Warning: cannot decode "+code+". "+ex.getMessage());
    281280                    }
    282281                }
    … …  
    292291                        transform = CRS.findMathTransform(newCRS, wgs84, false);
    293292                    } catch (OperationNotFoundException ex) {
    294                         System.err.println(newCRS.getName()+": "+e.getMessage());
     293                        Main.warn(newCRS.getName()+": "+e.getMessage());
    295294                    }
    296295                }

  • applications/editors/josm/plugins/opendata/src/org/openstreetmap/josm/plugins/opendata/core/io/geographic/ShpReader.java

    r30723 r30795  
    124124                int nGeometries = mp.getNumGeometries();
    125125                if (nGeometries < 1) {
    126                     System.err.println("Error: empty geometry collection found");
     126                    Main.error("empty geometry collection found");
    127127                } else {
    128128                    Relation r = null;
    … …  
    151151                            readNonGeometricAttributes(feature, createOrGetNode((Point) g));
    152152                        } else {
    153                             System.err.println("Error: unsupported geometry : "+g);
     153                            Main.error("unsupported geometry : "+g);
    154154                        }
    155155                    }

  • applications/editors/josm/plugins/opendata/src/org/openstreetmap/josm/plugins/opendata/core/io/geographic/TabReader.java

    r30723 r30795  
    1616import org.geotools.data.shapefile.dbf.DbaseFileReader.Row;
    1717import org.geotools.data.shapefile.files.TabFiles;
     18import org.openstreetmap.josm.Main;
    1819import org.openstreetmap.josm.data.osm.DataSet;
    1920import org.openstreetmap.josm.gui.progress.ProgressMonitor;
    … …  
    8283                    doParse(columns.toArray(new String[0]), instance);
    8384        } catch (IOException e) {
    84             System.err.println(e.getMessage());
     85            Main.error(e.getMessage());
    8586        }
    8687        return ds;
    … …  
    104105            parseColumns(words);
    105106        } else if (!line.isEmpty()) {
    106             System.err.println("Line "+lineNum+". Unknown clause in header: "+line);
     107            Main.warn("Line "+lineNum+". Unknown clause in header: "+line);
    107108        }
    108109    }
    … …  
    117118            datCharset = parseCharset(words, 3);
    118119        } else {
    119             System.err.println("Line "+lineNum+". Unknown Type clause in header: "+line);
     120            Main.warn("Line "+lineNum+". Unknown Type clause in header: "+line);
    120121        }
    121122    }

  • applications/editors/josm/plugins/opendata/src/org/openstreetmap/josm/plugins/opendata/core/io/tabular/SpreadSheetReader.java

    r30723 r30795  
    203203                    }
    204204                } catch (ParseException e) {
    205                     System.err.println("Warning: Parsing error on line "+lineNumber+": "+e.getMessage());
     205                    Main.warn("Parsing error on line "+lineNumber+": "+e.getMessage());
    206206                }
    207207            }
    … …  
    213213                    n.setCoor(c.proj != null && !handlerOK ? c.proj.eastNorth2latlon(en) : handler.getCoor(en, fields));
    214214                } else {
    215                     System.err.println("Warning: Skipping line "+lineNumber+" because no valid coordinates have been found at columns "+c);
     215                    Main.warn("Skipping line "+lineNumber+" because no valid coordinates have been found at columns "+c);
    216216                }
    217217                if (n.getCoor() != null) {

  • applications/editors/josm/plugins/opendata/src/org/openstreetmap/josm/plugins/opendata/core/io/tabular/XlsReader.java

    r30723 r30795  
    4444            Throwable ex = e.getException();
    4545            if (ex != null && ex.getMessage() != null) {
    46                 System.err.println(ex.getClass()+": "+ex.getMessage());
     46                Main.error(ex.getClass()+": "+ex.getMessage());
    4747            }
    4848            throw new IOException(e);

  • applications/editors/josm/plugins/opendata/src/org/openstreetmap/josm/plugins/opendata/core/layers/OdOsmDataLayer.java

    r30723 r30795  
    3838                                nodes = ((Way) p).getNodes();
    3939                            }
    40                             //data.removePrimitive(p);
    4140                            if (nodes != null) {
    4241                                for (Node n : nodes) {
    … …  
    4948                                   
    5049                                    if (refferingAllowedWays.isEmpty()) {
    51                                         //data.removePrimitive(n);
    5250                                        data.addSelected(n);
    5351                                    }
    5452                                }
    5553                            }
    56                             /*for (OsmPrimitive referrer : p.getReferrers()) {
    57                                 System.out.println(referrer);
    58                             }*/
    5954                        }
    6055                    }

  • applications/editors/josm/plugins/opendata/src/org/openstreetmap/josm/plugins/opendata/core/modules/ModuleDownloadTask.java

    r30532 r30795  
    1616import java.util.LinkedList;
    1717
     18import org.openstreetmap.josm.Main;
    1819import org.openstreetmap.josm.data.Version;
    1920import org.openstreetmap.josm.gui.PleaseWaitRunnable;
    … …  
    9495            if (pi.downloadlink == null) {
    9596                String msg = tr("Warning: Cannot download module ''{0}''. Its download link is not known. Skipping download.", pi.name);
    96                 System.err.println(msg);
     97                Main.warn(msg);
    9798                throw new ModuleDownloadException(msg);
    9899            }
    … …  
    116117        } catch(MalformedURLException e) {
    117118            String msg = tr("Warning: Cannot download module ''{0}''. Its download link ''{1}'' is not a valid URL. Skipping download.", pi.name, pi.downloadlink);
    118             System.err.println(msg);
     119            Main.warn(msg);
    119120            throw new ModuleDownloadException(msg);
    120121        } catch (IOException e) {

  • applications/editors/josm/plugins/opendata/src/org/openstreetmap/josm/plugins/opendata/core/modules/ModuleHandler.java

    r30731 r30795  
    134134        if (policy.equals("never")) {
    135135            if ("opendata.modulemanager.time-based-update.policy".equals(togglePreferenceKey)) {
    136                 System.out.println(tr("Skipping module update after elapsed update interval. Automatic update at startup is disabled."));
     136                Main.info(tr("Skipping module update after elapsed update interval. Automatic update at startup is disabled."));
    137137            }
    138138            return false;
    … …  
    141141        if (policy.equals("always")) {
    142142            if ("opendata.modulemanager.time-based-update.policy".equals(togglePreferenceKey)) {
    143                 System.out.println(tr("Running module update after elapsed update interval. Automatic update at startup is disabled."));
     143                Main.info(tr("Running module update after elapsed update interval. Automatic update at startup is disabled."));
    144144            }
    145145            return true;
    … …  
    147147
    148148        if (!policy.equals("ask")) {
    149             System.err.println(tr("Unexpected value ''{0}'' for preference ''{1}''. Assuming value ''ask''.", policy, togglePreferenceKey));
     149            Main.warn(tr("Unexpected value ''{0}'' for preference ''{1}''. Assuming value ''ask''.", policy, togglePreferenceKey));
    150150        }
    151151        int ret = HelpAwareOptionPane.showOptionDialog(
    … …  
    227227            Class<? extends Module> klass = module.loadClass(moduleClassLoader);
    228228            if (klass != null) {
    229                 System.out.println(tr("loading module ''{0}'' (version {1})", module.name, module.localversion));
     229                Main.info(tr("loading module ''{0}'' (version {1})", module.name, module.localversion));
    230230                Module mod = module.load(klass);
    231231                if (moduleList.add(mod)) {
    … …  
    437437                modules = buildListOfModulesToLoad(parent);
    438438            } catch(ExecutionException e) {
    439                 System.out.println(tr("Warning: failed to download module information list"));
     439                Main.warn(tr("Warning: failed to download module information list"));
    440440                e.printStackTrace();
    441441                // don't abort in case of error, continue with downloading modules below
    442442            } catch(InterruptedException e) {
    443                 System.out.println(tr("Warning: failed to download module information list"));
     443                Main.warn(tr("Warning: failed to download module information list"));
    444444                e.printStackTrace();
    445445                // don't abort in case of error, continue with downloading modules below
    … …  
    561561            if (module.exists()) {
    562562                if (!module.delete() && dowarn) {
    563                     System.err.println(tr("Warning: failed to delete outdated module ''{0}''.", module.toString()));
    564                     System.err.println(tr("Warning: failed to install already downloaded module ''{0}''. Skipping installation. JOSM is still going to load the old module version.", moduleName));
     563                    Main.warn(tr("Warning: failed to delete outdated module ''{0}''.", module.toString()));
     564                    Main.warn(tr("Warning: failed to install already downloaded module ''{0}''. Skipping installation. JOSM is still going to load the old module version.", moduleName));
    565565                    continue;
    566566                }
    567567            }
    568568            if (!updatedModule.renameTo(module) && dowarn) {
    569                 System.err.println(tr("Warning: failed to install module ''{0}'' from temporary download file ''{1}''. Renaming failed.", module.toString(), updatedModule.toString()));
    570                 System.err.println(tr("Warning: failed to install already downloaded module ''{0}''. Skipping installation. JOSM is still going to load the old module version.", moduleName));
     569                Main.warn(tr("Warning: failed to install module ''{0}'' from temporary download file ''{1}''. Renaming failed.", module.toString(), updatedModule.toString()));
     570                Main.warn(tr("Warning: failed to install already downloaded module ''{0}''. Skipping installation. JOSM is still going to load the old module version.", moduleName));
    571571            }
    572572        }

  • applications/editors/josm/plugins/opendata/src/org/openstreetmap/josm/plugins/opendata/core/modules/ModuleInformation.java

    r30738 r30795  
    115115     * update site.
    116116     *
    117      * @param other the module information object retrieved from the update
    118      * site
     117     * @param other the module information object retrieved from the update site
    119118     */
    120119    public void updateFromModuleSite(ModuleInformation other) {
    … …  
    149148                URL url = new URL(s);
    150149            } catch (MalformedURLException e) {
    151                 System.out.println(tr("Invalid URL ''{0}'' in module {1}", s, name));
     150                Main.error(tr("Invalid URL ''{0}'' in module {1}", s, name));
    152151                s = null;
    153152            }
    … …  
    155154        link = s;
    156155        s = attr.getValue(lang+"Module-Description");
    157         if(s == null)
    158         {
     156        if(s == null) {
    159157            s = attr.getValue("Module-Description");
    160158            if(s != null) {
    … …  
    173171                icon = extractIcon(iconPath, OdPlugin.getInstance().getPluginInformation().file, true);
    174172            }
     173            if (icon == null) {
     174                Main.error("Unable to load module icon: "+iconPath);
     175            }
    175176        }
    176177

  • applications/editors/josm/plugins/opendata/src/org/openstreetmap/josm/plugins/opendata/core/modules/ReadLocalModuleInformationTask.java

    r30532 r30795  
    1414import java.util.Map;
    1515
     16import org.openstreetmap.josm.Main;
    1617import org.openstreetmap.josm.gui.PleaseWaitRunnable;
    1718import org.openstreetmap.josm.gui.progress.ProgressMonitor;
    … …  
    7879        File[] siteCacheFiles = modulesDirectory.listFiles(
    7980                new FilenameFilter() {
     81                    @Override
    8082                    public boolean accept(File dir, String name) {
    8183                        return name.matches("^([0-9]+-)?site.*\\.txt$");
    … …  
    9395                processLocalModuleInformationFile(f);
    9496            } catch (ModuleListParseException e) {
    95                 System.err.println(tr("Warning: Failed to scan file ''{0}'' for module information. Skipping.", fname));
     97                Main.warn(tr("Warning: Failed to scan file ''{0}'' for module information. Skipping.", fname));
    9698                e.printStackTrace();
    9799            }
    … …  
    103105        File[] siteCacheFiles = modulesDirectory.listFiles(
    104106                new FilenameFilter() {
     107                    @Override
    105108                    public boolean accept(File dir, String name) {
    106109                        return name.matches("^([0-9]+-)?site.*modules-icons\\.zip$");
    … …  
    131134        File[] moduleFiles = modulesDirectory.listFiles(
    132135                new FilenameFilter() {
     136                    @Override
    133137                    public boolean accept(File dir, String name) {
    134138                        return name.endsWith(".jar") || name.endsWith(".jar.new");
    … …  
    152156                }
    153157            } catch(ModuleException e){
    154                 System.err.println(tr("Warning: Failed to scan file ''{0}'' for module information. Skipping.", fname));
     158                Main.warn(tr("Warning: Failed to scan file ''{0}'' for module information. Skipping.", fname));
    155159                e.printStackTrace();
    156160            }

  • applications/editors/josm/plugins/opendata/src/org/openstreetmap/josm/plugins/opendata/core/modules/ReadRemoteModuleInformationTask.java

    r30738 r30795  
    278278            availableModules.addAll(new ModuleListParser().parse(in));
    279279        } catch(UnsupportedEncodingException e) {
    280             System.err.println(tr("Failed to parse module list document from site ''{0}''. Skipping site. Exception was: {1}", site, e.toString()));
     280            Main.error(tr("Failed to parse module list document from site ''{0}''. Skipping site. Exception was: {1}", site, e.toString()));
    281281            e.printStackTrace();
    282282        } catch(ModuleListParseException e) {
    283             System.err.println(tr("Failed to parse module list document from site ''{0}''. Skipping site. Exception was: {1}", site, e.toString()));
     283            Main.error(tr("Failed to parse module list document from site ''{0}''. Skipping site. Exception was: {1}", site, e.toString()));
    284284            e.printStackTrace();
    285285        }

  • applications/editors/josm/plugins/opendata/src/org/openstreetmap/josm/plugins/opendata/core/util/NamesFrUtils.java

    r30738 r30795  
    158158            return label;
    159159        } else {
    160             System.err.println("Warning: unknown street label: "+label);
     160            Main.warn("unknown street label: "+label);
    161161            return label;
    162162        }
Note: See TracChangeset for help on using the changeset viewer.