Changeset 32545 in osm for applications/editors


Ignore:
Timestamp:
2016-07-03T22:13:58+02:00 (8 years ago)
Author:
donvip
Message:

checkstyle

Location:
applications/editors/josm/plugins/opendata
Files:
1 added
85 edited

Legend:

Unmodified
Added
Removed

  • applications/editors/josm/plugins/opendata/.checkstyle

    r31695 r32545  
    88    <file-match-pattern match-pattern="." include-pattern="true"/>
    99  </fileset>
     10  <filter name="DerivedFiles" enabled="true"/>
     11  <filter name="FilesFromPackage" enabled="true">
     12    <filter-data value="src/org/geotools"/>
     13    <filter-data value="data"/>
     14    <filter-data value="images"/>
     15    <filter-data value="includes"/>
     16    <filter-data value="styles"/>
     17    <filter-data value="resources"/>
     18    <filter-data value="scripts"/>
     19  </filter>
    1020</fileset-config>

  • applications/editors/josm/plugins/opendata/.project

    r32286 r32545  
    2626                        </arguments>
    2727                </buildCommand>
     28                <buildCommand>
     29                        <name>net.sf.eclipsecs.core.CheckstyleBuilder</name>
     30                        <arguments>
     31                        </arguments>
     32                </buildCommand>
    2833        </buildSpec>
    2934        <natures>
    … …  
    3136                <nature>org.eclipse.jdt.core.javanature</nature>
    3237                <nature>org.eclipse.wst.common.project.facet.core.nature</nature>
     38                <nature>net.sf.eclipsecs.core.CheckstyleNature</nature>
    3339        </natures>
    3440</projectDescription>

  • applications/editors/josm/plugins/opendata/.settings/org.eclipse.jdt.ui.prefs

    r28696 r32545  
    11eclipse.preferences.version=1
     2editor_save_participant_org.eclipse.jdt.ui.postsavelistener.cleanup=true
    23formatter_profile=_JOSM
    34formatter_settings_version=12
     5sp_cleanup.add_default_serial_version_id=true
     6sp_cleanup.add_generated_serial_version_id=false
     7sp_cleanup.add_missing_annotations=true
     8sp_cleanup.add_missing_deprecated_annotations=true
     9sp_cleanup.add_missing_methods=false
     10sp_cleanup.add_missing_nls_tags=false
     11sp_cleanup.add_missing_override_annotations=true
     12sp_cleanup.add_missing_override_annotations_interface_methods=true
     13sp_cleanup.add_serial_version_id=false
     14sp_cleanup.always_use_blocks=true
     15sp_cleanup.always_use_parentheses_in_expressions=false
     16sp_cleanup.always_use_this_for_non_static_field_access=false
     17sp_cleanup.always_use_this_for_non_static_method_access=false
     18sp_cleanup.convert_functional_interfaces=false
     19sp_cleanup.convert_to_enhanced_for_loop=false
     20sp_cleanup.correct_indentation=false
     21sp_cleanup.format_source_code=false
     22sp_cleanup.format_source_code_changes_only=false
     23sp_cleanup.insert_inferred_type_arguments=false
     24sp_cleanup.make_local_variable_final=true
     25sp_cleanup.make_parameters_final=false
     26sp_cleanup.make_private_fields_final=true
     27sp_cleanup.make_type_abstract_if_missing_method=false
     28sp_cleanup.make_variable_declarations_final=false
     29sp_cleanup.never_use_blocks=false
     30sp_cleanup.never_use_parentheses_in_expressions=true
     31sp_cleanup.on_save_use_additional_actions=true
     32sp_cleanup.organize_imports=true
     33sp_cleanup.qualify_static_field_accesses_with_declaring_class=false
     34sp_cleanup.qualify_static_member_accesses_through_instances_with_declaring_class=true
     35sp_cleanup.qualify_static_member_accesses_through_subtypes_with_declaring_class=true
     36sp_cleanup.qualify_static_member_accesses_with_declaring_class=false
     37sp_cleanup.qualify_static_method_accesses_with_declaring_class=false
     38sp_cleanup.remove_private_constructors=true
     39sp_cleanup.remove_redundant_type_arguments=true
     40sp_cleanup.remove_trailing_whitespaces=true
     41sp_cleanup.remove_trailing_whitespaces_all=true
     42sp_cleanup.remove_trailing_whitespaces_ignore_empty=false
     43sp_cleanup.remove_unnecessary_casts=true
     44sp_cleanup.remove_unnecessary_nls_tags=false
     45sp_cleanup.remove_unused_imports=true
     46sp_cleanup.remove_unused_local_variables=false
     47sp_cleanup.remove_unused_private_fields=true
     48sp_cleanup.remove_unused_private_members=false
     49sp_cleanup.remove_unused_private_methods=true
     50sp_cleanup.remove_unused_private_types=true
     51sp_cleanup.sort_members=false
     52sp_cleanup.sort_members_all=false
     53sp_cleanup.use_anonymous_class_creation=false
     54sp_cleanup.use_blocks=false
     55sp_cleanup.use_blocks_only_for_return_and_throw=false
     56sp_cleanup.use_lambda=true
     57sp_cleanup.use_parentheses_in_expressions=false
     58sp_cleanup.use_this_for_non_static_field_access=false
     59sp_cleanup.use_this_for_non_static_field_access_only_if_necessary=true
     60sp_cleanup.use_this_for_non_static_method_access=false
     61sp_cleanup.use_this_for_non_static_method_access_only_if_necessary=true
     62sp_cleanup.use_type_arguments=false

  • applications/editors/josm/plugins/opendata/build.xml

    r32287 r32545  
    11<?xml version="1.0" encoding="utf-8"?>
    22<project name="opendata" default="dist" basedir=".">
    3     <property name="plugin.main.version" value="10407"/>
     3    <property name="plugin.main.version" value="10420"/>
    44    <property name="plugin.author" value="Don-vip"/>
    55    <property name="plugin.class" value="org.openstreetmap.josm.plugins.opendata.OdPlugin"/>

  • applications/editors/josm/plugins/opendata/src/org/openstreetmap/josm/plugins/opendata/OdPlugin.java

    r32287 r32545  
    9898    }
    9999
    100     public static final OdPlugin getInstance() {
     100    public static OdPlugin getInstance() {
    101101        return instance;
    102102    }
    … …  
    191191    }
    192192
    193     private final void loadModules() {
     193    private void loadModules() {
    194194        List<ModuleInformation> modulesToLoad = ModuleHandler.buildListOfModulesToLoad(Main.parent);
    195195        if (!modulesToLoad.isEmpty() && ModuleHandler.checkAndConfirmModuleUpdate(Main.parent)) {
    … …  
    201201    }
    202202
    203     private final File getSubDirectory(String name) {
     203    private File getSubDirectory(String name) {
    204204        File dir = new File(getPluginDir()+File.separator+name);
    205205        if (!dir.exists()) {
    … …  
    209209    }
    210210
    211     public final File getModulesDirectory() {
     211    public File getModulesDirectory() {
    212212        return getSubDirectory("modules");
    213213    }
    214214
    215     public final File getResourcesDirectory() {
     215    public File getResourcesDirectory() {
    216216        return getSubDirectory("resources");
    217217    }

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

    r30723 r32545  
    1010
    1111public abstract class OdConstants {
    12    
     12
    1313    /**
    1414     * Encodings
    … …  
    2020    public static final String CP1252 = "Cp1252";
    2121    public static final String MAC_ROMAN = "MacRoman";
    22    
     22
    2323    /**
    2424     * Patterns
    2525     */
    2626    public static final String PATTERN_LANG = "{lang}";
    27    
     27
    2828    /**
    2929     * Preferences
    … …  
    3535    public static final String PREF_OAPI = "opendata.oapi";
    3636    public static final String DEFAULT_OAPI = "http://www.overpass-api.de/api/interpreter?";
    37    
     37
    3838    public static final String PREF_XAPI = "opendata.xapi";
    3939    public static final String DEFAULT_XAPI = "http://www.overpass-api.de/api/xapi?";
    … …  
    5050    public static final String PREF_MODULES = "opendata.modules";
    5151    public static final String PREF_MODULES_SITES = "opendata.modules.sites";
    52     public final static String OSM_SITE = "http://svn.openstreetmap.org/applications/editors/josm/plugins/opendata/";
    53     public final static String[] DEFAULT_MODULE_SITES = {OSM_SITE + "modules.txt%<?modules=>"};
     52    public static final String OSM_SITE = "http://svn.openstreetmap.org/applications/editors/josm/plugins/opendata/";
     53    public static final String[] DEFAULT_MODULE_SITES = {OSM_SITE + "modules.txt%<?modules=>"};
    5454
    5555    public static final String PREF_CRS_COMPARISON_TOLERANCE = "opendata.crs.comparison_tolerance";
    … …  
    6464    public static final String ICON_CORE_24 = "o24.png";
    6565    public static final String ICON_CORE_48 = "o48.png";
    66    
     66
    6767    public static final String ICON_OSM_16 = "osm16.png";
    6868    public static final String ICON_OSM_24 = "osm24.png";
    … …  
    9090    public static final String JAR_EXT = "jar";
    9191    public static final String XML_EXT = "xml";
    92    
     92
    9393    /**
    9494     * Protocols
    … …  
    9696    public static final String PROTO_FILE = "file://";
    9797    public static final String PROTO_RSRC = "resource://";
    98    
     98
    9999    /**
    100100     * Coordinates fields
    … …  
    102102    public static final String X_STRING = "X|LON|LONGI|.*LONGITUDE.*|EASTING";
    103103    public static final String Y_STRING = "Y|LAT|LATI|.*LATITUDE.*|NORTHING";
    104    
     104
    105105    // The list of all ProjectionPatterns (filled at each constructor call)
    106106    public static final Collection<ProjectionPatterns> PROJECTIONS = new ArrayList<>();
    107    
     107
     108    // CHECKSTYLE.OFF: LineLength
    108109    public static final ProjectionPatterns PRJ_WGS84 = new ProjectionPatterns("GPS|WGS84|°décimaux", Projections.getProjectionByCode("EPSG:4326"));
    109110    public static final ProjectionPatterns PRJ_LAMBERT_93 = new ProjectionPatterns("LAMB93|L93", Projections.getProjectionByCode("EPSG:2154"));
    … …  
    112113    public static final ProjectionPatterns PRJ_LAMBERT_1972 = new ProjectionPatterns("LAMB72|LAMB1972", Projections.getProjectionByCode("EPSG:31370"));
    113114    public static final ProjectionPatterns PRJ_LAMBERT_2008 = new ProjectionPatterns("LAMB08|LAMB2008", Projections.getProjectionByCode("EPSG:3812"));
     115    // CHECKSTYLE.ON: LineLength
    114116
    115117    // Must always be declared last

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

    r30723 r32545  
    2828
    2929    private AbstractDataSetHandler handler;
    30    
     30
    3131    @Override
    3232    public Future<?> download(boolean newLayer, Bounds downloadArea, ProgressMonitor progressMonitor) {
    … …  
    6363        return false;
    6464    }
    65    
     65
    6666    @Override
    6767    public String[] getPatterns() {
    … …  
    8989        @Override
    9090        protected OsmDataLayer createNewLayer(String layerName) {
    91             File associatedFile = ((NetworkReader)reader).getReadFile();
    92             String filename = ((NetworkReader)reader).getReadFileName();
     91            File associatedFile = ((NetworkReader) reader).getReadFile();
     92            String filename = ((NetworkReader) reader).getReadFileName();
    9393            if (layerName == null || layerName.isEmpty()) {
    9494                if (associatedFile != null) {
    … …  
    104104        }
    105105    }
    106    
     106
    107107    /**
    108108     * returns true if the user accepts the license, false if they refuse
    … …  
    114114        try {
    115115            return new AskLicenseAgreementDialog(license).showDialog().getValue() == 1;
    116            
     116
    117117        } catch (IOException e) {
    118118            JOptionPane.showMessageDialog(Main.parent, tr("License URL not available: {0}", license.toString()));

  • applications/editors/josm/plugins/opendata/src/org/openstreetmap/josm/plugins/opendata/core/datasets/AbstractDataSetHandler.java

    r32125 r32545  
    2121import org.openstreetmap.josm.io.AbstractReader;
    2222import org.openstreetmap.josm.plugins.opendata.core.OdConstants;
     23import org.openstreetmap.josm.plugins.opendata.core.io.archive.ArchiveHandler;
    2324import org.openstreetmap.josm.plugins.opendata.core.io.archive.DefaultArchiveHandler;
    24 import org.openstreetmap.josm.plugins.opendata.core.io.archive.ArchiveHandler;
    2525import org.openstreetmap.josm.plugins.opendata.core.io.geographic.DefaultGmlHandler;
    2626import org.openstreetmap.josm.plugins.opendata.core.io.geographic.DefaultShpHandler;
    … …  
    3737
    3838public abstract class AbstractDataSetHandler {
    39    
     39
    4040    public abstract boolean acceptsFilename(String filename);
    41    
     41
    4242    public boolean acceptsFile(File file) {
    4343        return acceptsFilename(file.getName());
    4444    }
    45    
     45
    4646    public abstract void updateDataSet(DataSet ds);
    4747
    … …  
    6060        }
    6161    }
    62    
     62
    6363    public void checkNames(DataSet ds) {
    6464        if (ds != null) {
    … …  
    8383        setGmlHandler(new DefaultGmlHandler());
    8484    }
    85    
    86     private final boolean acceptsFilename(String filename, String[] expected, String ... extensions ) {
     85
     86    private boolean acceptsFilename(String filename, String[] expected, String ... extensions) {
    8787        if (filename != null) {
    8888            for (String name : expected) {
    8989                for (String ext : extensions) {
    9090                    if (Pattern.compile(name+"\\."+ext, Pattern.CASE_INSENSITIVE).matcher(filename).matches()) {
    91                     //if (filename.equalsIgnoreCase(name+"."+ext)) {
     91                        //if (filename.equalsIgnoreCase(name+"."+ext)) {
    9292                        return true;
    9393                    }
    … …  
    9797        return false;
    9898    }
    99    
     99
    100100    protected final boolean acceptsCsvFilename(String filename, String ... expected) {
    101101        return acceptsFilename(filename, expected, OdConstants.CSV_EXT);
    … …  
    157157        return acceptsFilename(filename, expected, OdConstants.CSV_EXT, OdConstants.KMZ_EXT, OdConstants.TAB_EXT);
    158158    }
    159        
     159
    160160    protected final boolean acceptsCsvXlsFilename(String filename, String ... expected) {
    161161        return acceptsFilename(filename, expected, OdConstants.CSV_EXT, OdConstants.XLS_EXT);
    162162    }
    163    
     163
    164164    // -------------------- License --------------------
    165    
     165
    166166    private License license;
    167    
     167
    168168    public License getLicense() {
    169169        return license;
    … …  
    175175
    176176    // --------------------- URLs ---------------------
    177    
     177
    178178    private URL dataURL;
    179179    private URL wikiURL;
    180180    private URL localPortalURL;
    181181    private URL nationalPortalURL;
    182    
     182
    183183    public URL getDataURL() {
    184184        return dataURL;
    … …  
    229229    }
    230230
    231     public List<Pair<String,URL>> getDataURLs() {return null;}
    232 
    233     public AbstractReader getReaderForUrl(String url) {return null;}
    234    
     231    public List<Pair<String, URL>> getDataURLs() {
     232        return null;
     233    }
     234
     235    public AbstractReader getReaderForUrl(String url) {
     236        return null;
     237    }
     238
    235239    private boolean hasLicenseToBeAccepted = true;
    236240
    … …  
    252256
    253257    public final Collection<String> getOsmXapiRequests(Bounds bounds) {
    254         return getOsmXapiRequests(
    255                 LatLon.roundToOsmPrecision(bounds.getMin().lon())+","+
    256                 LatLon.roundToOsmPrecision(bounds.getMin().lat())+","+
    257                 LatLon.roundToOsmPrecision(bounds.getMax().lon())+","+
    258                 LatLon.roundToOsmPrecision(bounds.getMax().lat()));
    259     }
    260    
    261     protected Collection<String> getOsmXapiRequests(String bbox) {return null;}
    262    
     258        return getOsmXapiRequests(LatLon.roundToOsmPrecision(bounds.getMin().lon())+","+
     259                                  LatLon.roundToOsmPrecision(bounds.getMin().lat())+","+
     260                                  LatLon.roundToOsmPrecision(bounds.getMax().lon())+","+
     261                                  LatLon.roundToOsmPrecision(bounds.getMax().lat()));
     262    }
     263
     264    protected Collection<String> getOsmXapiRequests(String bbox) {
     265        return null;
     266    }
     267
    263268    public final String getOverpassApiRequest(Bounds bounds) {
    264         return getOverpassApiRequest(
    265                 "w=\""+LatLon.roundToOsmPrecision(bounds.getMin().lon())+"\" "+
    266                 "s=\""+LatLon.roundToOsmPrecision(bounds.getMin().lat())+"\" "+
    267                 "e=\""+LatLon.roundToOsmPrecision(bounds.getMax().lon())+"\" "+
    268                 "n=\""+LatLon.roundToOsmPrecision(bounds.getMax().lat())+"\"");
    269     }
    270 
    271     protected String getOverpassApiRequest(String bbox) {return null;}
    272 
    273     public boolean equals(IPrimitive p1, IPrimitive p2) {return false;}
    274    
    275     public boolean isRelevant(IPrimitive p) {return false;}
    276    
     269        return getOverpassApiRequest("w=\""+LatLon.roundToOsmPrecision(bounds.getMin().lon())+"\" "+
     270                                     "s=\""+LatLon.roundToOsmPrecision(bounds.getMin().lat())+"\" "+
     271                                     "e=\""+LatLon.roundToOsmPrecision(bounds.getMax().lon())+"\" "+
     272                                     "n=\""+LatLon.roundToOsmPrecision(bounds.getMax().lat())+"\"");
     273    }
     274
     275    protected String getOverpassApiRequest(String bbox) {
     276        return null;
     277    }
     278
     279    public boolean equals(IPrimitive p1, IPrimitive p2) {
     280        return false;
     281    }
     282
     283    public boolean isRelevant(IPrimitive p) {
     284        return false;
     285    }
     286
    277287    public final Collection<IPrimitive> extractRelevantPrimitives(DataSet ds) {
    278288        ArrayList<IPrimitive> result = new ArrayList<>();
    … …  
    284294        return result;
    285295    }
    286    
    287     public boolean isForbidden(IPrimitive p) {return false;}
    288    
    289     public boolean hasForbiddenTags() {return false;}
    290    
     296
     297    public boolean isForbidden(IPrimitive p) {
     298        return false;
     299    }
     300
     301    public boolean hasForbiddenTags() {
     302        return false;
     303    }
     304
    291305    public interface ValueReplacer {
    292         public String replace(String value);
    293     }
    294    
     306        String replace(String value);
     307    }
     308
    295309    protected final void replace(IPrimitive p, String dataKey, String osmKey) {
    296310        addOrReplace(p, dataKey, osmKey, null, null, null, true);
    … …  
    304318        addOrReplace(p, dataKey, osmKey, dataValues, osmValues, null, true);
    305319    }
    306    
     320
    307321    protected final void add(IPrimitive p, String dataKey, String osmKey, ValueReplacer replacer) {
    308322        addOrReplace(p, dataKey, osmKey, null, null, replacer, false);
    … …  
    312326        addOrReplace(p, dataKey, osmKey, dataValues, osmValues, null, false);
    313327    }
    314    
    315     private final void addOrReplace(IPrimitive p, String dataKey, String osmKey, String[] dataValues, String[] osmValues, ValueReplacer replacer, boolean replace) {
     328
     329    private void addOrReplace(IPrimitive p, String dataKey, String osmKey, String[] dataValues, String[] osmValues,
     330            ValueReplacer replacer, boolean replace) {
    316331        String value = p.get(dataKey);
    317332        if (value != null) {
    … …  
    334349        }
    335350    }
    336    
    337     private final void doAddReplace(IPrimitive p, String dataKey, String osmKey, String osmValue, boolean replace) {
     351
     352    private void doAddReplace(IPrimitive p, String dataKey, String osmKey, String osmValue, boolean replace) {
    338353        if (replace) {
    339354            p.remove(dataKey);
    … …  
    345360        return null;
    346361    }
    347        
     362
    348363    public final String getSourceDate() {
    349364        return sourceDate;
    350365    }
    351    
     366
    352367    public final void setSourceDate(String sourceDate) {
    353368        this.sourceDate = sourceDate;
    … …  
    357372        return name;
    358373    }
    359    
     374
    360375    public final void setName(String name) {
    361376        this.name = name;
    … …  
    365380        return OdConstants.ICON_CORE_24;
    366381    }
    367        
     382
    368383    public String getNationalPortalIconName() {
    369384        return OdConstants.ICON_CORE_24;
    370385    }
    371        
     386
    372387    public String getDataLayerIconName() {
    373388        return OdConstants.ICON_CORE_16;
    374389    }
    375    
     390
    376391    public ExtendedSourceEntry getMapPaintStyle() {
    377392        return null;
    … …  
    387402
    388403    protected final ExtendedSourceEntry getMapPaintStyle(String displayName, String fileNameWithoutExtension) {
    389         return new ExtendedSourceEntry(displayName,    OdConstants.PROTO_RSRC+//"/"+
     404        return new ExtendedSourceEntry(displayName, OdConstants.PROTO_RSRC+//"/"+
    390405                this.getClass().getPackage().getName().replace(".", "/")+"/"+
    391406                fileNameWithoutExtension+"."+OdConstants.MAPCSS_EXT);
    392407    }
    393    
     408
    394409    public final ImageIcon getMenuIcon() {
    395410        return menuIcon;
    … …  
    411426        return this.associatedFile;
    412427    }
    413    
     428
    414429    public boolean acceptsUrl(String url) {
    415430        URL dataURL = getDataURL();
    … …  
    427442        return false;
    428443    }
    429    
     444
    430445    // --------- Shapefile handling ---------
    431    
     446
    432447    private ShpHandler shpHandler;
    433448
    … …  
    435450        shpHandler = handler;
    436451    }
    437    
     452
    438453    public final ShpHandler getShpHandler() {
    439454        return shpHandler;
    … …  
    441456
    442457    // --------- MIF handling ---------
    443    
     458
    444459    private MifHandler mifHandler;
    445460
    … …  
    447462        mifHandler = handler;
    448463    }
    449    
     464
    450465    public final MifHandler getMifHandler() {
    451466        return mifHandler;
    … …  
    453468
    454469    // --------- GML handling ---------
    455    
     470
    456471    private GmlHandler gmlHandler;
    457472
    … …  
    459474        gmlHandler = handler;
    460475    }
    461    
     476
    462477    public final GmlHandler getGmlHandler() {
    463478        return gmlHandler;
    … …  
    465480
    466481    // ------------ Archive handling ------------
    467    
     482
    468483    private ArchiveHandler archiveHandler;
    469484
    … …  
    471486        archiveHandler = handler;
    472487    }
    473    
     488
    474489    public ArchiveHandler getArchiveHandler() {
    475490        return archiveHandler;
    476491    }
    477    
     492
    478493    // ------ Spreadsheet handling ----------
    479494
    … …  
    483498        ssHandler = handler;
    484499    }
    485    
     500
    486501    public final SpreadSheetHandler getSpreadSheetHandler() {
    487502        return ssHandler;
    … …  
    491506        setSpreadSheetHandler(handler);
    492507    }
    493    
     508
    494509    public final CsvHandler getCsvHandler() {
    495510        if (ssHandler instanceof CsvHandler) {
    … …  
    499514        }
    500515    }
    501    
     516
    502517    // Tools
    503518
    504519    private final Collection<JosmAction> tools = new ArrayList<>();
    505    
     520
    506521    public final Collection<JosmAction> getTools() {
    507522        return tools;
    508523    }
    509    
     524
    510525    public final boolean addTool(JosmAction tool) {
    511526        return tool != null ? tools.add(tool) : false;
    512527    }
    513    
     528
    514529    public final boolean removeTool(JosmAction tool) {
    515530        return tool != null ? tools.remove(tool) : false;

  • applications/editors/josm/plugins/opendata/src/org/openstreetmap/josm/plugins/opendata/core/datasets/SimpleDataSetHandler.java

    r30723 r32545  
    2828    private final List<Tag> relevantTags = new ArrayList<>();
    2929    private final List<Tag> forbiddenTags = new ArrayList<>();
    30    
     30
    3131    private final boolean relevantUnion;
    32    
     32
    3333    public SimpleDataSetHandler() {
    3434        this.relevantUnion = false;
    3535    }
    36            
     36
    3737    public SimpleDataSetHandler(String relevantTag) {
    3838        addRelevantTag(relevantTag);
    … …  
    4747        setMenuIcon(MapPaintStyles.getNodeIcon(tag));
    4848    }
    49    
     49
    5050    public SimpleDataSetHandler(boolean relevantUnion, String ... relevantTags) {
    5151        addRelevantTag(relevantTags);
    … …  
    6161        this.relevantUnion = false;
    6262    }
    63    
     63
    6464    public SimpleDataSetHandler(boolean relevantUnion, Tag ... relevantTags) {
    6565        addRelevantTag(relevantTags);
    … …  
    8686        addTags(this.forbiddenTags, forbiddenTags);
    8787    }
    88    
    89     private final void addTags(final List<Tag> list, String ... tags) {
     88
     89    private void addTags(final List<Tag> list, String ... tags) {
    9090        if (tags != null) {
    9191            for (String tag : tags) {
    … …  
    102102    }
    103103
    104     private final void addTags(final List<Tag> list, Tag ... tags) {
     104    private void addTags(final List<Tag> list, Tag ... tags) {
    105105        if (tags != null) {
    106106            for (Tag tag : tags) {
    … …  
    111111        }
    112112    }
    113    
     113
    114114    @Override
    115115    public boolean equals(IPrimitive p1, IPrimitive p2) {
    … …  
    135135        return true;
    136136    }
    137    
     137
    138138    @Override
    139139    public boolean isForbidden(IPrimitive p) {
    … …  
    163163        String[] mpconditions = new String[conditions.length+1];
    164164        mpconditions[0] = OverpassApi.hasKey("type", "multipolygon");
    165         for (int i=0; i<conditions.length; i++) {
     165        for (int i = 0; i < conditions.length; i++) {
    166166            mpconditions[i+1] = conditions[i];
    167167        }
    168         return OverpassApi.query(bbox, NODE, conditions) + "\n" + // Nodes 
    169             OverpassApi.recurse(NODE_RELATION, RELATION_WAY, WAY_NODE) + "\n" +
    170             OverpassApi.query(bbox, WAY, conditions) + "\n" + // Full ways and their full relations
    171             OverpassApi.recurse(WAY_NODE, "nodes") + "\n" +
    172             OverpassApi.recurse(WAY_RELATION, RELATION_WAY, WAY_NODE) + "\n" +
    173             OverpassApi.query(bbox, RELATION, mpconditions) + "\n" + // Full multipolygons
    174             OverpassApi.recurse(RELATION_WAY, WAY_NODE);
    175     }
    176    
     168        return OverpassApi.query(bbox, NODE, conditions) + "\n" + // Nodes
     169        OverpassApi.recurse(NODE_RELATION, RELATION_WAY, WAY_NODE) + "\n" +
     170        OverpassApi.query(bbox, WAY, conditions) + "\n" + // Full ways and their full relations
     171        OverpassApi.recurse(WAY_NODE, "nodes") + "\n" +
     172        OverpassApi.recurse(WAY_RELATION, RELATION_WAY, WAY_NODE) + "\n" +
     173        OverpassApi.query(bbox, RELATION, mpconditions) + "\n" + // Full multipolygons
     174        OverpassApi.recurse(RELATION_WAY, WAY_NODE);
     175    }
     176
    177177    @Override
    178178    protected String getOverpassApiRequest(String bbox) {
    … …  
    180180        if (this.relevantUnion) {
    181181            for (Tag tag : this.relevantTags) {
    182                 result += getOverpassApiQueries(bbox, OverpassApi.hasKey(tag.getKey(), tag.getValue())); 
     182                result += getOverpassApiQueries(bbox, OverpassApi.hasKey(tag.getKey(), tag.getValue()));
    183183            }
    184184            result = OverpassApi.union(result);

  • applications/editors/josm/plugins/opendata/src/org/openstreetmap/josm/plugins/opendata/core/datasets/WayCombiner.java

    r31655 r32545  
    2626
    2727// FIXME: Try to refactor CombineWayAction instead of using this class
    28 public class WayCombiner {
    29    
     28public final class WayCombiner {
     29
     30    private WayCombiner() {
     31        // Hide default constructor for utilities classes
     32    }
     33
    3034    protected static Way getTargetWay(Collection<Way> combinedWays) {
    3135        // init with an arbitrary way
    … …  
    4246        return targetWay;
    4347    }
    44    
    45     /**
    46      * @param ways
    47      * @throws UserCancelException
    48      */
     48
    4949    public static void combineWays(Collection<Way> ways) throws UserCancelException {
    5050
    … …  
    103103                ReverseWayTagCorrector reverseWayTagCorrector = new ReverseWayTagCorrector();
    104104                List<Way> reversedTagWays = new ArrayList<>();
    105                 Collection<Command> changePropertyCommands =  null;
     105                Collection<Command> changePropertyCommands = null;
    106106                for (Way w : reversedWays) {
    107107                    Way wnew = new Way(w);
    … …  
    138138                parentRelations,
    139139                ways
    140         );
     140                );
    141141        dialog.prepareDefaultDecisions();
    142142
    … …  
    146146            dialog.setVisible(true);
    147147            //if (dialog.isCanceled()) // FIXME
    148                 throw new UserCancelException();
     148            throw new UserCancelException();
    149149        }
    150150

  • applications/editors/josm/plugins/opendata/src/org/openstreetmap/josm/plugins/opendata/core/datasets/at/AustrianConstants.java

    r30723 r32545  
    11// License: GPL. For details, see LICENSE file.
    22package org.openstreetmap.josm.plugins.opendata.core.datasets.at;
    3 
    43
    54public interface AustrianConstants {
    … …  
    87     * Icon
    98     */
    10     public static final String ICON_AT_24 = "at24.png";
     9    String ICON_AT_24 = "at24.png";
    1110}

  • applications/editors/josm/plugins/opendata/src/org/openstreetmap/josm/plugins/opendata/core/datasets/at/AustrianGmlHandler.java

    r30723 r32545  
    1313    public CoordinateReferenceSystem getCrsFor(String crsName)
    1414            throws NoSuchAuthorityCodeException, FactoryException {
    15        
     15
    1616        // See http://www.esri-austria.at/downloads/coords_at.html
    17        
     17
    1818        if (crsName != null && crsName.startsWith("AUT")) {
     19            // CHECKSTYLE.OFF: LineLength
    1920            if (crsName.equalsIgnoreCase("AUT-GK28")) {             // Gauß-Krüger, Meridianstreifen M28
    2021                return CRS.decode("EPSG:31281");
    21            
     22
    2223            } else if (crsName.equalsIgnoreCase("AUT-GK31")) {      // Gauß-Krüger, Meridianstreifen M31
    2324                return CRS.decode("EPSG:31282");
    24                
     25
    2526            } else if (crsName.equalsIgnoreCase("AUT-GK34")) {      // Gauß-Krüger, Meridianstreifen M34
    2627                return CRS.decode("EPSG:31283");
    27                
     28
    2829            } else if (crsName.equalsIgnoreCase("AUT-GK28-5")) {    // Gauß-Krüger, Meridianstreifen M28, ohne 5 Mio. im Hochwert
    2930                return CRS.decode("EPSG:31251");
    30                
     31
    3132            } else if (crsName.equalsIgnoreCase("AUT-GK31-5")) {    // Gauß-Krüger, Meridianstreifen M31, ohne 5 Mio. im Hochwert
    3233                return CRS.decode("EPSG:31252");
    33                
     34
    3435            } else if (crsName.equalsIgnoreCase("AUT-GK34-5")) {    // Gauß-Krüger, Meridianstreifen M34, ohne 5 Mio. im Hochwert
    3536                return CRS.decode("EPSG:31253");
    36                
     37
    3738            } else if (crsName.equalsIgnoreCase("AUT-BM28")) {      // Bundesmeldenetz, Meridianstreifen M28
    3839                return CRS.decode("EPSG:31288");
    39                
     40
    4041            } else if (crsName.equalsIgnoreCase("AUT-BM31")) {      // Bundesmeldenetz, Meridianstreifen M31
    4142                return CRS.decode("EPSG:31289");
    42                
     43
    4344            } else if (crsName.equalsIgnoreCase("AUT-BM34")) {      // Bundesmeldenetz, Meridianstreifen M34
    4445                return CRS.decode("EPSG:31290");
    45                
     46
    4647            } else if (crsName.equalsIgnoreCase("AUT-LM")) {        // Lambertsche Kegelprojektion (geogr. Breite des Koo.Ursprungs = 47°30')
    4748                return CRS.decode("EPSG:31287");
    48                
     49
    4950            } else if (crsName.equalsIgnoreCase("AUT-LL-BESSEL")) { // Geographische Koordinaten auf dem Bessel-Ellipsoid, Längenzählung nach Greenwich
    5051                // See http://josebatiz.com/granvision/Almap/Install/Data1/_B5694C166D6A4B5390B1E547C6A1FAF6
    5152                // FIXME
    5253            }
     54            // CHECKSTYLE.ON: LineLength
    5355        }
    5456        return super.getCrsFor(crsName);

  • applications/editors/josm/plugins/opendata/src/org/openstreetmap/josm/plugins/opendata/core/datasets/au/AustralianConstants.java

    r30723 r32545  
    11// License: GPL. For details, see LICENSE file.
    22package org.openstreetmap.josm.plugins.opendata.core.datasets.au;
    3 
    43
    54public interface AustralianConstants {
    … …  
    87     * Portal
    98     */
    10     public static final String AUSTRALIAN_PORTAL = "http://data.gov.au/dataset/";
    11    
     9    String AUSTRALIAN_PORTAL = "http://data.gov.au/dataset/";
     10
    1211    /**
    1312     * Icon
    1413     */
    15     public static final String ICON_AU_24 = "au24.png";
     14    String ICON_AU_24 = "au24.png";
    1615}

  • applications/editors/josm/plugins/opendata/src/org/openstreetmap/josm/plugins/opendata/core/datasets/be/BelgianConstants.java

    r30723 r32545  
    99     * Portal
    1010     */
    11     public static final String BELGIAN_PORTAL = "http://data.gov.be/"+OdConstants.PATTERN_LANG+"/dataset/";
    12    
     11    String BELGIAN_PORTAL = "http://data.gov.be/"+OdConstants.PATTERN_LANG+"/dataset/";
     12
    1313    /**
    1414     * Icon
    1515     */
    16     public static final String ICON_BE_24 = "be24.png";
     16    String ICON_BE_24 = "be24.png";
    1717}

  • applications/editors/josm/plugins/opendata/src/org/openstreetmap/josm/plugins/opendata/core/datasets/be/BelgianDataSetHandler.java

    r30723 r32545  
    2626    protected static final Projection lambert1972 = OdConstants.PRJ_LAMBERT_1972.getProjection();
    2727    protected static final Projection lambert2008 = OdConstants.PRJ_LAMBERT_2008.getProjection();
    28    
     28
    2929    protected static final Projection[] projections = new Projection[]{
    30         lambert1972,
    31         lambert2008
     30            lambert1972,
     31            lambert2008
    3232    };
    33    
     33
    3434    protected class InternalCsvHandler extends DefaultCsvHandler {
    3535        /*@Override
    … …  
    4141            }
    4242        }*/
    43        
     43
    4444        @Override
    4545        public LatLon getCoor(EastNorth en, String[] fields) {
    … …  
    7070        init();
    7171    }
    72    
     72
    7373    private void init() {
    7474        setCsvHandler(new InternalCsvHandler());
    7575    }
    7676
    77     protected final void setNationalPortalPath(String nationalPortalPathDe, String nationalPortalPathEn, String nationalPortalPathFr, String nationalPortalPathNl) {
     77    protected final void setNationalPortalPath(String nationalPortalPathDe, String nationalPortalPathEn,
     78            String nationalPortalPathFr, String nationalPortalPathNl) {
    7879        this.nationalPortalPathDe = nationalPortalPathDe;
    7980        this.nationalPortalPathEn = nationalPortalPathEn;
    … …  
    9596                lang = Locale.getDefault().toString();
    9697            }
    97                
     98
    9899            if (lang.startsWith("de") && nationalPortalPathDe != null) {
    99100                nationalPortalPath = nationalPortalPathDe;
    … …  
    105106                nationalPortalPath = nationalPortalPathEn;
    106107            }
    107             return new URL(BELGIAN_PORTAL.replace(OdConstants.PATTERN_LANG, lang.substring(0, 2))+nationalPortalPath);//FIXME
     108            return new URL(BELGIAN_PORTAL.replace(OdConstants.PATTERN_LANG, lang.substring(0, 2))+nationalPortalPath); //FIXME
    108109        } catch (MalformedURLException e) {
    109110            e.printStackTrace();

  • applications/editors/josm/plugins/opendata/src/org/openstreetmap/josm/plugins/opendata/core/datasets/ca/CanadianConstants.java

    r30723 r32545  
    11// License: GPL. For details, see LICENSE file.
    22package org.openstreetmap.josm.plugins.opendata.core.datasets.ca;
    3 
    43
    54public interface CanadianConstants {
    … …  
    87     * Portal
    98     */
    10     public static final String CANADIAN_PORTAL = "http://www.data.gc.ca/geo/";
    11    
     9    String CANADIAN_PORTAL = "http://www.data.gc.ca/geo/";
     10
    1211    /**
    1312     * Icon
    1413     */
    15     public static final String ICON_CA_24 = "ca24.png";
     14    String ICON_CA_24 = "ca24.png";
    1615}

  • applications/editors/josm/plugins/opendata/src/org/openstreetmap/josm/plugins/opendata/core/datasets/cl/ChileanConstants.java

    r30723 r32545  
    11// License: GPL. For details, see LICENSE file.
    22package org.openstreetmap.josm.plugins.opendata.core.datasets.cl;
    3 
    43
    54public interface ChileanConstants {
    … …  
    87     * Portal
    98     */
    10     public static final String CHILEAN_PORTAL = "http://datos.gob.cl/datasets/";
    11    
     9    String CHILEAN_PORTAL = "http://datos.gob.cl/datasets/";
     10
    1211    /**
    1312     * Icon
    1413     */
    15     public static final String ICON_CL_24 = "cl24.png";
     14    String ICON_CL_24 = "cl24.png";
    1615}

  • applications/editors/josm/plugins/opendata/src/org/openstreetmap/josm/plugins/opendata/core/datasets/es/SpanishConstants.java

    r30723 r32545  
    11// License: GPL. For details, see LICENSE file.
    22package org.openstreetmap.josm.plugins.opendata.core.datasets.es;
    3 
    43
    54public interface SpanishConstants {
    … …  
    87     * Portal
    98     */
    10     public static final String SPANISH_PORTAL = "http://datos.gob.es/datos/";
    11    
     9    String SPANISH_PORTAL = "http://datos.gob.es/datos/";
     10
    1211    /**
    1312     * Icon
    1413     */
    15     public static final String ICON_ES_24 = "es24.png";
     14    String ICON_ES_24 = "es24.png";
    1615}

  • applications/editors/josm/plugins/opendata/src/org/openstreetmap/josm/plugins/opendata/core/datasets/fr/FrenchConstants.java

    r30723 r32545  
    11// License: GPL. For details, see LICENSE file.
    22package org.openstreetmap.josm.plugins.opendata.core.datasets.fr;
    3 
    43
    54public interface FrenchConstants {
    … …  
    87     * Portal
    98     */
    10     public static final String FRENCH_PORTAL = "http://www.data.gouv.fr/";
    11    
     9    String FRENCH_PORTAL = "http://www.data.gouv.fr/";
     10
    1211    /**
    1312     * Icon
    1413     */
    15     public static final String ICON_FR_24 = "fr24.png";
    16    
     14    String ICON_FR_24 = "fr24.png";
     15
    1716    /**
    1817     * NEPTUNE XML Schema
    1918     */
    20     public static final String NEPTUNE_XSD = "/resources/neptune/neptune.xsd";
     19    String NEPTUNE_XSD = "/resources/neptune/neptune.xsd";
    2120}

  • applications/editors/josm/plugins/opendata/src/org/openstreetmap/josm/plugins/opendata/core/datasets/fr/FrenchDataSetHandler.java

    r30723 r32545  
    3030    protected static final Projection utm38 = Projections.getProjectionByCode("EPSG:32738"); // UTM 38 S - Mayotte
    3131    protected static final Projection utm40 = Projections.getProjectionByCode("EPSG:32740"); // UTM 40 S - Reunion
    32    
     32
    3333    protected static final Projection[] lambert4Zones = new Projection[4];
    3434    static {
    35         for (int i=0; i<lambert4Zones.length; i++) {
     35        for (int i = 0; i < lambert4Zones.length; i++) {
    3636            lambert4Zones[i] = Projections.getProjectionByCode("EPSG:"+Integer.toString(27561+i));
    3737        }
    … …  
    3939
    4040    protected static final Projection[] projections = new Projection[]{
    41         lambert93, // France metropolitaine
    42         utm20, // Guadeloupe, Martinique
    43         utm22, // Guyane
    44         utm38, // Mayotte
    45         utm40, // Reunion
     41            lambert93, // France metropolitaine
     42            utm20, // Guadeloupe, Martinique
     43            utm22, // Guyane
     44            utm38, // Mayotte
     45            utm40, // Reunion
    4646    };
    47    
     47
    4848    protected class InternalCsvHandler extends DefaultCsvHandler {
    4949        /*@Override
    … …  
    5555            }
    5656        }*/
    57        
     57
    5858        @Override
    5959        public LatLon getCoor(EastNorth en, String[] fields) {
    … …  
    8989        init();
    9090    }
    91    
     91
    9292    private void init() {
    9393        setShpHandler(new FrenchShpHandler());
    … …  
    124124        return ICON_FR_24;
    125125    }
    126    
     126
    127127    protected static final LatLon getLatLonByDptCode(EastNorth en, String dpt, boolean useCC9) {
     128        // CHECKSTYLE.OFF: LineLength
    128129        if (dpt.equals("971") || dpt.equals("972") || dpt.equals("977") || dpt.equals("978")) {    // Antilles
    129130            return utm20.eastNorth2latlon(en);
    … …  
    157158            throw new IllegalArgumentException("Unsupported department code: "+dpt);
    158159        }
     160        // CHECKSTYLE.ON: LineLength
    159161    }
    160162
    … …  
    170172        replaceFaxPhone(p, dataKey, "contact:fax");
    171173    }
    172    
     174
    173175    protected void replacePhone(OsmPrimitive p, String dataKey) {
    174176        replaceFaxPhone(p, dataKey, "contact:phone");
    175177    }
    176    
     178
    177179    private static final String dayFr = "L|Lu|M|Ma|Me|J|Je|V|Ve|S|Sa|D|Di";
    178180    private static final String dayEn = "Mo|Mo|Tu|Tu|We|Th|Th|Fr|Fr|Sa|Sa|Su|Su";
    179    
     181
    180182    private static final String[] dayFrSplit = dayFr.split("\\|");
    181183    private static final String[] dayEnSplit = dayEn.split("\\|");
    182    
     184
    183185    protected void replaceOpeningHours(OsmPrimitive p, String dataKey) {
    184186        String hours = p.get(dataKey);
    … …  
    194196            String dayNcGroup = "(?:"+dayFr+")";
    195197            // FIXME: this doesn't work yet
    196             for (String sep : new String[]{"-",","}) {
     198            for (String sep : new String[]{"-", ","}) {
    197199                boolean finished = false;
    198200                while (!finished) {
    … …  
    203205                        if (m2.matches()) {
    204206                            String replacement = "";
    205                             for (int i=0; i<m2.groupCount(); i++) {
     207                            for (int i = 0; i < m2.groupCount(); i++) {
    206208                                if (i > 0) {
    207209                                    replacement += sep;
    … …  
    220222        }
    221223    }
    222    
     224
    223225    private String getEnDay(String frDay) {
    224         for (int i=0; i<dayFrSplit.length; i++) {
     226        for (int i = 0; i < dayFrSplit.length; i++) {
    225227            if (dayFrSplit[i].equals(frDay)) {
    226228                return dayEnSplit[i];

  • applications/editors/josm/plugins/opendata/src/org/openstreetmap/josm/plugins/opendata/core/datasets/fr/FrenchShpHandler.java

    r30723 r32545  
    3232            if (sourceCRS instanceof ProjectedCRS) {
    3333                GeodeticDatum datum = ((ProjectedCRS) sourceCRS).getDatum();
    34                 if (datum.getPrimeMeridian().getGreenwichLongitude() > 0.0 && ((ProjectedCRS) sourceCRS).getConversionFromBase().getMathTransform() instanceof LambertConformal2SP) {
     34                if (datum.getPrimeMeridian().getGreenwichLongitude() > 0.0
     35                        && ((ProjectedCRS) sourceCRS).getConversionFromBase().getMathTransform() instanceof LambertConformal2SP) {
    3536                    LambertConformal2SP lambert = (LambertConformal2SP) ((ProjectedCRS) sourceCRS).getConversionFromBase().getMathTransform();
    3637                    Double falseNorthing = get(lambert.getParameterValues(), AbstractProvider.FALSE_NORTHING);

  • applications/editors/josm/plugins/opendata/src/org/openstreetmap/josm/plugins/opendata/core/datasets/it/ItalianConstants.java

    r30723 r32545  
    11// License: GPL. For details, see LICENSE file.
    22package org.openstreetmap.josm.plugins.opendata.core.datasets.it;
    3 
    43
    54public interface ItalianConstants {
    … …  
    87     * Portal
    98     */
    10     public static final String ITALIAN_PORTAL = "http://www.dati.gov.it/content/";
    11    
     9    String ITALIAN_PORTAL = "http://www.dati.gov.it/content/";
     10
    1211    /**
    1312     * Icon
    1413     */
    15     public static final String ICON_IT_24 = "it24.png";
     14    String ICON_IT_24 = "it24.png";
    1615}

  • applications/editors/josm/plugins/opendata/src/org/openstreetmap/josm/plugins/opendata/core/datasets/ke/KenyanConstants.java

    r30723 r32545  
    11// License: GPL. For details, see LICENSE file.
    22package org.openstreetmap.josm.plugins.opendata.core.datasets.ke;
    3 
    43
    54public interface KenyanConstants {
    … …  
    87     * Portal
    98     */
    10     public static final String KENYAN_PORTAL = "https://opendata.go.ke/";
    11    
     9    String KENYAN_PORTAL = "https://opendata.go.ke/";
     10
    1211    /**
    1312     * Icon
    1413     */
    15     public static final String ICON_KE_24 = "ke24.png";
     14    String ICON_KE_24 = "ke24.png";
    1615}

  • applications/editors/josm/plugins/opendata/src/org/openstreetmap/josm/plugins/opendata/core/datasets/nl/DutchConstants.java

    r30723 r32545  
    11// License: GPL. For details, see LICENSE file.
    22package org.openstreetmap.josm.plugins.opendata.core.datasets.nl;
    3 
    43
    54public interface DutchConstants {
    … …  
    87     * Portal
    98     */
    10     public static final String NORWEGIAN_PORTAL = "http://data.overheid.nl/package/";
    11    
     9    String NORWEGIAN_PORTAL = "http://data.overheid.nl/package/";
     10
    1211    /**
    1312     * Icon
    1413     */
    15     public static final String ICON_NL_24 = "nl24.png";
     14    String ICON_NL_24 = "nl24.png";
    1615}

  • applications/editors/josm/plugins/opendata/src/org/openstreetmap/josm/plugins/opendata/core/datasets/no/NorwegianConstants.java

    r30723 r32545  
    11// License: GPL. For details, see LICENSE file.
    22package org.openstreetmap.josm.plugins.opendata.core.datasets.no;
    3 
    43
    54public interface NorwegianConstants {
    … …  
    87     * Portal
    98     */
    10     public static final String NORWEGIAN_PORTAL = "http://data.norge.no/data/";
    11    
     9    String NORWEGIAN_PORTAL = "http://data.norge.no/data/";
     10
    1211    /**
    1312     * Icon
    1413     */
    15     public static final String ICON_NO_24 = "no24.png";
     14    String ICON_NO_24 = "no24.png";
    1615}

  • applications/editors/josm/plugins/opendata/src/org/openstreetmap/josm/plugins/opendata/core/datasets/nz/NewZealanderConstants.java

    r30723 r32545  
    11// License: GPL. For details, see LICENSE file.
    22package org.openstreetmap.josm.plugins.opendata.core.datasets.nz;
    3 
    43
    54public interface NewZealanderConstants {
    … …  
    87     * Portal
    98     */
    10     public static final String NEW_ZEALANDER_PORTAL = "http://data.govt.nz/dataset/";
    11    
     9    String NEW_ZEALANDER_PORTAL = "http://data.govt.nz/dataset/";
     10
    1211    /**
    1312     * Icon
    1413     */
    15     public static final String ICON_NZ_24 = "nz24.png";
     14    String ICON_NZ_24 = "nz24.png";
    1615}

  • applications/editors/josm/plugins/opendata/src/org/openstreetmap/josm/plugins/opendata/core/datasets/pt/PortugueseConstants.java

    r30723 r32545  
    11// License: GPL. For details, see LICENSE file.
    22package org.openstreetmap.josm.plugins.opendata.core.datasets.pt;
    3 
    43
    54public interface PortugueseConstants {
    … …  
    87     * Portal
    98     */
    10     public static final String PORTUGUESE_PORTAL = "http://www.dados.gov.pt/PT/CatalogoDados/Dados.aspx?name=";
    11    
     9    String PORTUGUESE_PORTAL = "http://www.dados.gov.pt/PT/CatalogoDados/Dados.aspx?name=";
     10
    1211    /**
    1312     * Icon
    1413     */
    15     public static final String ICON_PT_24 = "pt24.png";
     14    String ICON_PT_24 = "pt24.png";
    1615}

  • applications/editors/josm/plugins/opendata/src/org/openstreetmap/josm/plugins/opendata/core/datasets/uk/BritishConstants.java

    r30723 r32545  
    11// License: GPL. For details, see LICENSE file.
    22package org.openstreetmap.josm.plugins.opendata.core.datasets.uk;
    3 
    43
    54public interface BritishConstants {
    … …  
    87     * Portal
    98     */
    10     public static final String BRITISH_PORTAL = "http://data.gov.uk/dataset/";
    11    
     9    String BRITISH_PORTAL = "http://data.gov.uk/dataset/";
     10
    1211    /**
    1312     * Icon
    1413     */
    15     public static final String ICON_UK_24 = "uk24.png";
     14    String ICON_UK_24 = "uk24.png";
    1615}

  • applications/editors/josm/plugins/opendata/src/org/openstreetmap/josm/plugins/opendata/core/datasets/us/AmericanConstants.java

    r30723 r32545  
    11// License: GPL. For details, see LICENSE file.
    22package org.openstreetmap.josm.plugins.opendata.core.datasets.us;
    3 
    43
    54public interface AmericanConstants {
    … …  
    87     * Portal
    98     */
    10     public static final String AMERICAN_PORTAL = "http://explore.data.gov/";
    11    
     9    String AMERICAN_PORTAL = "http://explore.data.gov/";
     10
    1211    /**
    1312     * Icon
    1413     */
    15     public static final String ICON_US_24 = "us24.png";
     14    String ICON_US_24 = "us24.png";
    1615}

  • applications/editors/josm/plugins/opendata/src/org/openstreetmap/josm/plugins/opendata/core/gui/DialogPrompter.java

    r30723 r32545  
    88import org.openstreetmap.josm.gui.ExtendedDialog;
    99
    10 public class DialogPrompter <T extends ExtendedDialog> implements Runnable {
    11    
     10public class DialogPrompter<T extends ExtendedDialog> implements Runnable {
     11
    1212    private T dialog;
    1313    private int value = -1;
    1414
    15     protected T buildDialog() {return null;} // To be overriden if needed
    16    
     15    protected T buildDialog() {
     16        // To be overriden if needed
     17        return null;
     18    }
     19
    1720    public DialogPrompter() {
    1821        this(null);
    1922    }
    20    
     23
    2124    public DialogPrompter(T dialog) {
    2225        this.dialog = dialog;
    2326    }
    24    
     27
    2528    public final T getDialog() {
    2629        return dialog;
    2730    }
    28    
     31
    2932    @Override
    3033    public final void run() {
    … …  
    3639        }
    3740    }
    38    
     41
    3942    public final DialogPrompter<T> promptInEdt() {
    4043        if (SwingUtilities.isEventDispatchThread()) {

  • applications/editors/josm/plugins/opendata/src/org/openstreetmap/josm/plugins/opendata/core/gui/ModuleListPanel.java

    r30340 r32545  
    2727 *
    2828 */
    29 public class ModuleListPanel extends VerticallyScrollablePanel{
     29public class ModuleListPanel extends VerticallyScrollablePanel {
    3030    private ModulePreferencesModel model;
    3131
    … …  
    7070        gbc.anchor = GridBagConstraints.CENTER;
    7171        gbc.fill = GridBagConstraints.BOTH;
    72         gbc.insets = new Insets(40,0,40,0);
     72        gbc.insets = new Insets(40, 0, 40, 0);
    7373        gbc.weightx = 1.0;
    7474        gbc.weighty = 1.0;
    7575
    7676        HtmlPanel hint = new HtmlPanel();
    77         hint.setText(
    78                 "<html>"
     77        hint.setText("<html>"
    7978                + tr("Please click on <strong>Download list</strong> to download and display a list of available modules.")
    8079                + "</html>"
    81         );
     80                );
    8281        add(hint, gbc);
    8382    }
    … …  
    107106            cbModule.setSelected(selected);
    108107            cbModule.setToolTipText(formatCheckboxTooltipText(pi));
    109             cbModule.addActionListener(new ActionListener(){
     108            cbModule.addActionListener(new ActionListener() {
     109                @Override
    110110                public void actionPerformed(ActionEvent e) {
    111111                    model.setModuleSelected(pi.getName(), cbModule.isSelected());
    … …  
    119119            gbc.gridx = 0;
    120120            gbc.gridy = ++row;
    121             gbc.insets = new Insets(5,5,0,5);
     121            gbc.insets = new Insets(5, 5, 0, 5);
    122122            gbc.weighty = 0.0;
    123123            gbc.weightx = 0.0;
    … …  
    131131            description.setText(pi.getDescriptionAsHtml());
    132132            description.getEditorPane().addHyperlinkListener(new HyperlinkListener() {
     133                @Override
    133134                public void hyperlinkUpdate(HyperlinkEvent e) {
    134                     if(e.getEventType() == EventType.ACTIVATED) {
     135                    if (e.getEventType() == EventType.ACTIVATED) {
    135136                        OpenBrowser.displayUrl(e.getURL().toString());
    136137                    }
    … …  
    140141            gbc.gridx = 1;
    141142            gbc.gridy = ++row;
    142             gbc.insets = new Insets(3,25,5,5);
     143            gbc.insets = new Insets(3, 25, 5, 5);
    143144            gbc.weighty = 1.0;
    144145            add(description, gbc);

  • applications/editors/josm/plugins/opendata/src/org/openstreetmap/josm/plugins/opendata/core/gui/ModulePreference.java

    r31048 r32545  
    6767        Collection<ModuleInformation> failed = task.getFailedModules();
    6868        StringBuilder sb = new StringBuilder();
    69         if (! downloaded.isEmpty()) {
     69        if (!downloaded.isEmpty()) {
    7070            sb.append(trn(
    7171                    "The following module has been downloaded <strong>successfully</strong>:",
    … …  
    7373                    downloaded.size(),
    7474                    downloaded.size()
    75             ));
     75                    ));
    7676            sb.append("<ul>");
    77             for(ModuleInformation pi: downloaded) {
     77            for (ModuleInformation pi : downloaded) {
    7878                sb.append("<li>").append(pi.name).append(" (").append(pi.version).append(")").append("</li>");
    7979            }
    8080            sb.append("</ul>");
    8181        }
    82         if (! failed.isEmpty()) {
     82        if (!failed.isEmpty()) {
    8383            sb.append(trn(
    8484                    "Downloading the following module has <strong>failed</strong>:",
    … …  
    8686                    failed.size(),
    8787                    failed.size()
    88             ));
     88                    ));
    8989            sb.append("<ul>");
    90             for(ModuleInformation pi: failed) {
     90            for (ModuleInformation pi : failed) {
    9191                sb.append("<li>").append(pi.name).append("</li>");
    9292            }
    … …  
    108108
    109109    protected JPanel buildSearchFieldPanel() {
    110         JPanel pnl  = new JPanel(new GridBagLayout());
    111         pnl.setBorder(BorderFactory.createEmptyBorder(5,5,5,5));
     110        JPanel pnl = new JPanel(new GridBagLayout());
     111        pnl.setBorder(BorderFactory.createEmptyBorder(5, 5, 5, 5));
    112112        GridBagConstraints gc = new GridBagConstraints();
    113113
    … …  
    115115        gc.fill = GridBagConstraints.HORIZONTAL;
    116116        gc.weightx = 0.0;
    117         gc.insets = new Insets(0,0,0,3);
     117        gc.insets = new Insets(0, 0, 0, 3);
    118118        pnl.add(new JLabel(tr("Search:")), gc);
    119119
    … …  
    128128
    129129    protected JPanel buildActionPanel() {
    130         JPanel pnl = new JPanel(new GridLayout(1,3));
     130        JPanel pnl = new JPanel(new GridLayout(1, 3));
    131131
    132132        pnl.add(new JButton(new DownloadAvailableModulesAction()));
    … …  
    139139        JPanel pnl = new JPanel(new BorderLayout());
    140140        pnl.add(buildSearchFieldPanel(), BorderLayout.NORTH);
    141         model  = new ModulePreferencesModel();
     141        model = new ModulePreferencesModel();
    142142        spModulePreferences = new JScrollPane(pnlModulePreferences = new ModuleListPanel(model));
    143143        spModulePreferences.setHorizontalScrollBarPolicy(JScrollPane.HORIZONTAL_SCROLLBAR_NEVER);
    144144        spModulePreferences.setVerticalScrollBarPolicy(JScrollPane.VERTICAL_SCROLLBAR_AS_NEEDED);
    145145        spModulePreferences.getVerticalScrollBar().addComponentListener(
    146                 new ComponentAdapter(){
     146                new ComponentAdapter() {
    147147                    @Override
    148148                    public void componentShown(ComponentEvent e) {
    149149                        spModulePreferences.setBorder(UIManager.getBorder("ScrollPane.border"));
    150150                    }
     151
    151152                    @Override
    152153                    public void componentHidden(ComponentEvent e) {
    … …  
    154155                    }
    155156                }
    156         );
     157                );
    157158
    158159        pnl.add(spModulePreferences, BorderLayout.CENTER);
    … …  
    181182                        tr("Accept the new module sites and close the dialog"),
    182183                        null /* no special help topic */
    183                 ),
     184                        ),
    184185                new ButtonSpec(
    185186                        tr("Cancel"),
    … …  
    187188                        tr("Close the dialog"),
    188189                        null /* no special help topic */
    189                 )
     190                        )
    190191        };
    191192        ModuleConfigurationSitesPanel pnl = new ModuleConfigurationSitesPanel();
    … …  
    200201                options[0],
    201202                null /* no help topic */
    202         );
     203                );
    203204        if (answer != 0 /* OK */)
    204205            return;
    … …  
    218219    @Override
    219220    public boolean ok() {
    220         if (! modulePreferencesActivated)
     221        if (!modulePreferencesActivated)
    221222            return false;
    222223        if (model.isActiveModulesChanged()) {
    … …  
    260261    class DownloadAvailableModulesAction extends AbstractAction {
    261262
    262         public DownloadAvailableModulesAction() {
    263             putValue(NAME,tr("Download list"));
     263        DownloadAvailableModulesAction() {
     264            putValue(NAME, tr("Download list"));
    264265            putValue(SHORT_DESCRIPTION, tr("Download the list of available modules"));
    265266            putValue(SMALL_ICON, ImageProvider.get("download"));
    … …  
    292293     */
    293294    class UpdateSelectedModulesAction extends AbstractAction {
    294         public UpdateSelectedModulesAction() {
    295             putValue(NAME,tr("Update modules"));
     295        UpdateSelectedModulesAction() {
     296            putValue(NAME, tr("Update modules"));
    296297            putValue(SHORT_DESCRIPTION, tr("Update the selected modules"));
    297298            putValue(SMALL_ICON, ImageProvider.get("dialogs", "refresh"));
    … …  
    314315                    !failed.isEmpty() ? JOptionPane.WARNING_MESSAGE : JOptionPane.INFORMATION_MESSAGE,
    315316                            HelpUtil.ht("/Preferences/Modules")
    316             );
     317                    );
    317318        }
    318319
    … …  
    327328                            JOptionPane.INFORMATION_MESSAGE,
    328329                            null // FIXME: provide help context
    329                     );
     330                            );
    330331                }
    331332            });
    … …  
    340341                    toUpdate,
    341342                    tr("Update modules")
    342             );
     343                    );
    343344            // the async task for downloading module information
    344             final ReadRemoteModuleInformationTask moduleInfoDownloadTask = new ReadRemoteModuleInformationTask(OdPreferenceSetting.getModuleSites());
     345            final ReadRemoteModuleInformationTask moduleInfoDownloadTask =
     346                    new ReadRemoteModuleInformationTask(OdPreferenceSetting.getModuleSites());
    345347
    346348            // to be run asynchronously after the module download
    … …  
    374376                    //
    375377                    Iterator<ModuleInformation> it = toUpdate.iterator();
    376                     while(it.hasNext()) {
     378                    while (it.hasNext()) {
    377379                        ModuleInformation pi = it.next();
    378380                        if (!pi.isUpdateRequired()) {
    … …  
    395397    }
    396398
    397 
    398399    /**
    399400     * The action for configuring the module download sites
    … …  
    401402     */
    402403    class ConfigureSitesAction extends AbstractAction {
    403         public ConfigureSitesAction() {
    404             putValue(NAME,tr("Configure sites..."));
     404        ConfigureSitesAction() {
     405            putValue(NAME, tr("Configure sites..."));
    405406            putValue(SHORT_DESCRIPTION, tr("Configure the list of sites where modules are downloaded from"));
    406407            putValue(SMALL_ICON, ImageProvider.get("dialogs", "settings"));
    … …  
    420421    class ModulePreferenceActivationListener implements ChangeListener {
    421422        private Component pane;
    422         public ModulePreferenceActivationListener(Component preferencesPane) {
     423        ModulePreferenceActivationListener(Component preferencesPane) {
    423424            pane = preferencesPane;
    424425        }
    … …  
    426427        @Override
    427428        public void stateChanged(ChangeEvent e) {
    428             JTabbedPane tp = (JTabbedPane)e.getSource();
     429            JTabbedPane tp = (JTabbedPane) e.getSource();
    429430            if (tp.getSelectedComponent() == pane) {
    430431                readLocalModuleInformation();
    … …  
    464465    }
    465466
    466     static private class ModuleConfigurationSitesPanel extends JPanel {
     467    private static class ModuleConfigurationSitesPanel extends JPanel {
    467468
    468469        private DefaultListModel<String> model;
    … …  
    478479            add(new JScrollPane(list), GBC.std().fill());
    479480            JPanel buttons = new JPanel(new GridBagLayout());
    480             buttons.add(new JButton(new AbstractAction(tr("Add")){
     481            buttons.add(new JButton(new AbstractAction(tr("Add")) {
    481482                @Override
    482483                public void actionPerformed(ActionEvent e) {
    … …  
    486487                            tr("Enter URL"),
    487488                            JOptionPane.QUESTION_MESSAGE
    488                     );
     489                            );
    489490                    if (s != null) {
    490491                        model.addElement(s);
    … …  
    492493                }
    493494            }), GBC.eol().fill(GBC.HORIZONTAL));
    494             buttons.add(new JButton(new AbstractAction(tr("Edit")){
     495            buttons.add(new JButton(new AbstractAction(tr("Edit")) {
    495496                @Override
    496497                public void actionPerformed(ActionEvent e) {
    … …  
    501502                                tr("Warning"),
    502503                                JOptionPane.WARNING_MESSAGE
    503                         );
     504                                );
    504505                        return;
    505506                    }
    506                     String s = (String)JOptionPane.showInputDialog(
     507                    String s = (String) JOptionPane.showInputDialog(
    507508                            Main.parent,
    508509                            tr("Edit Open Data Module description URL."),
    … …  
    512513                            null,
    513514                            list.getSelectedValue()
    514                     );
     515                            );
    515516                    if (s != null) {
    516517                        model.setElementAt(s, list.getSelectedIndex());
    … …  
    518519                }
    519520            }), GBC.eol().fill(GBC.HORIZONTAL));
    520             buttons.add(new JButton(new AbstractAction(tr("Delete")){
     521            buttons.add(new JButton(new AbstractAction(tr("Delete")) {
    521522                @Override
    522523                public void actionPerformed(ActionEvent event) {
    … …  
    527528                                tr("Warning"),
    528529                                JOptionPane.WARNING_MESSAGE
    529                         );
     530                                );
    530531                        return;
    531532                    }
    … …  
    536537        }
    537538
    538         public ModuleConfigurationSitesPanel() {
     539        ModuleConfigurationSitesPanel() {
    539540            build();
    540541        }
    … …  
    543544            if (model.getSize() == 0) return Collections.emptyList();
    544545            List<String> ret = new ArrayList<>(model.getSize());
    545             for (int i=0; i< model.getSize();i++){
     546            for (int i = 0; i < model.getSize(); i++) {
    546547                ret.add(model.get(i));
    547548            }

  • applications/editors/josm/plugins/opendata/src/org/openstreetmap/josm/plugins/opendata/core/gui/ModulePreferencesModel.java

    r30723 r32545  
    3131        return Main.pref.getCollection(OdConstants.PREF_MODULES, def);
    3232    }
    33    
     33
    3434    public ModulePreferencesModel() {
    3535        currentActiveModules = new HashSet<>();
    … …  
    7575    }
    7676
    77     protected  void updateAvailableModule(ModuleInformation other) {
     77    protected void updateAvailableModule(ModuleInformation other) {
    7878        if (other == null) return;
    7979        ModuleInformation pi = getModuleInformation(other.name);
    … …  
    148148                availableModules,
    149149                new Comparator<ModuleInformation>() {
     150                    @Override
    150151                    public int compare(ModuleInformation o1, ModuleInformation o2) {
    151152                        String n1 = o1.getName() == null ? "" : o1.getName().toLowerCase();
    … …  
    154155                    }
    155156                }
    156         );
     157                );
    157158    }
    158159
    … …  
    193194        ModuleInformation pi = getModuleInformation(name);
    194195        if (pi != null) {
    195             selectedModulesMap.put(pi,selected);
     196            selectedModulesMap.put(pi, selected);
    196197            if (pi.isUpdateRequired()) {
    197198                pendingDownloads.add(pi.name);
    … …  
    209210     * @param modules the list of modules to clear for a pending download
    210211     */
    211     public void clearPendingModules(Collection<ModuleInformation> modules){
     212    public void clearPendingModules(Collection<ModuleInformation> modules) {
    212213        if (modules == null || modules.isEmpty()) return;
    213         for(ModuleInformation pi: modules) {
     214        for (ModuleInformation pi : modules) {
    214215            pendingDownloads.remove(pi.name);
    215216        }
    … …  
    324325    public boolean isActiveModulesChanged() {
    325326        Set<String> newActiveModules = getSelectedModuleNames();
    326         return ! newActiveModules.equals(currentActiveModules);
     327        return !newActiveModules.equals(currentActiveModules);
    327328    }
    328329
    … …  
    355356                }
    356357                oldinfo.localversion = newinfo.version;
    357             } catch(ModuleException e) {
     358            } catch (ModuleException e) {
    358359                e.printStackTrace();
    359360            }

  • applications/editors/josm/plugins/opendata/src/org/openstreetmap/josm/plugins/opendata/core/gui/OdDialog.java

    r32525 r32545  
    3636    private final SideButton diffButton;
    3737    private final SideButton toolsButton;
    38    
     38
    3939    private final List<SideButton> buttons;
    40    
     40
    4141    private final TreeModel treeModel;
    42    
     42
    4343    private OdDataLayer dataLayer;
    44    
     44
    4545    private class DownloadAction extends JosmAction {
    46         public DownloadAction() {
     46        DownloadAction() {
    4747            super(marktr("Download"), "download", tr("Download OSM data corresponding to the current data set."), null, false);
    4848        }
     49
    4950        @Override
    5051        public void actionPerformed(ActionEvent e) {
    … …  
    5758
    5859    private class SelectAction extends JosmAction {
    59         public SelectAction() {
    60             super(marktr("Select"), "dialogs/select", tr("Set the selected elements on the map to the selected items in the list above."), null, false);
     60        SelectAction() {
     61            super(marktr("Select"), "dialogs/select",
     62                    tr("Set the selected elements on the map to the selected items in the list above."), null, false);
    6163        }
     64
    6265        @Override
    6366        public void actionPerformed(ActionEvent e) {
    … …  
    6770
    6871    private class DiffAction extends JosmAction {
    69         public DiffAction() {
     72        DiffAction() {
    7073            super(marktr("Diff"), "dialogs/diff", tr("Perform diff between current data set and existing OSM data."), null, false);
    7174        }
     75
    7276        @Override
    7377        public void actionPerformed(ActionEvent e) {
    … …  
    7781        }
    7882    }
    79    
     83
    8084    private class ToolsAction extends JosmAction {
    81         public ToolsAction() {
     85        ToolsAction() {
    8286            super(marktr("Tools"), "dialogs/utils", tr("Open tools menu for this data."), null, false);
    8387        }
     88
    8489        @Override
    8590        public void actionPerformed(ActionEvent e) {
    … …  
    8994
    9095    public OdDialog() {
    91         super("OpenData", OdConstants.ICON_CORE_24, tr("Open the OpenData window."), 
     96        super("OpenData", OdConstants.ICON_CORE_24, tr("Open the OpenData window."),
    9297                Shortcut.registerShortcut("subwindow:opendata", tr("Toggle: {0}", "OpenData"),
    9398                        KeyEvent.VK_A, Shortcut.ALT_CTRL_SHIFT), 150, false, OdPreferenceSetting.class);
    94        
     99
    95100        this.buttons = Arrays.asList(new SideButton[] {
    96                 /*selectButton =*/ new SideButton(new SelectAction()), 
    97                 downloadButton = new SideButton(new DownloadAction()), 
     101                /*selectButton =*/ new SideButton(new SelectAction()),
     102                downloadButton = new SideButton(new DownloadAction()),
    98103                diffButton = new SideButton(new DiffAction()),
    99104                toolsButton = new SideButton(new ToolsAction())
    100105        });
    101        
     106
    102107        this.toolsButton.addMouseListener(new MouseAdapter() {
    103108            @Override
    … …  
    112117            }
    113118        });
    114        
     119
    115120        disableAllButtons();
    116        
     121
    117122        this.treeModel = new DefaultTreeModel(null); // TODO: treeNode
    118123        this.dataLayer = null;
    119        
     124
    120125        createLayout(new JTree(treeModel), true, buttons);
    121        
     126
    122127        Main.getLayerManager().addActiveLayerChangeListener(this);
    123128    }
    … …  
    137142            dataLayer = null;
    138143        }
    139        
     144
    140145        if (dataLayer != null) {
    141146            if (dataLayer.osmLayer == null) {

  • applications/editors/josm/plugins/opendata/src/org/openstreetmap/josm/plugins/opendata/core/gui/OdPreferenceSetting.java

    r30723 r32545  
    3434    //private final JRadioButton rbCC43 = new JRadioButton(tr("CC43"));
    3535    //private final JRadioButton rbWGS84 = new JRadioButton(tr("WGS84"));
    36    
     36
    3737    private final JTextField oapi = new JTextField();
    3838    private final JTextField xapi = new JTextField();
    39    
     39
    4040    private final JCheckBox rawData = new JCheckBox(tr("Raw data"));
    41    
     41
    4242    public final JTabbedPane tabPane = new JTabbedPane();
    4343    public JPanel masterPanel;
    44    
     44
    4545    private final ModulePreference modulePref = new ModulePreference();
    4646
    … …  
    5151                        "before any upload of data loaded by this plugin."));
    5252    }
    53    
     53
    5454    /**
    5555     * Replies the collection of module site URLs from where module lists can be downloaded
    5656     *
    57      * @return
     57     * @return the collection of module site URLs from where module lists can be downloaded
    5858     */
    5959    public static final Collection<String> getModuleSites() {
    … …  
    6969        Main.pref.putCollection(OdConstants.PREF_MODULES_SITES, sites);
    7070    }
    71    
     71
    7272    @Override
    7373    public void addGui(PreferenceTabbedPane gui) {
    … …  
    7575        modulePref.addGui(gui);
    7676        tabPane.add(createGeneralSettings());
    77        
     77
    7878        JScrollPane scrollpane = new JScrollPane(tabPane);
    79         scrollpane.setBorder(BorderFactory.createEmptyBorder( 0, 0, 0, 0 ));
     79        scrollpane.setBorder(BorderFactory.createEmptyBorder(0, 0, 0, 0));
    8080        masterPanel.add(scrollpane, GBC.eol().fill(GBC.BOTH));
    8181    }
    … …  
    8383    protected JPanel createGeneralSettings() {
    8484        JPanel general = new JPanel(new GridBagLayout());
    85         general.setBorder(BorderFactory.createEmptyBorder(0,0,0,0));
     85        general.setBorder(BorderFactory.createEmptyBorder(0, 0, 0, 0));
    8686        general.setName(tr("General settings"));
    8787
    … …  
    9090        rawData.setToolTipText(tr("Import only raw data (i.e. do not add/delete tags or replace them by standard OSM tags)"));
    9191        general.add(rawData, GBC.eop().insets(0, 0, 0, 0));
    92        
     92
    9393        // separator
    9494        general.add(new JSeparator(SwingConstants.HORIZONTAL), GBC.eol().fill(GBC.HORIZONTAL));
    95        
     95
    9696        // option to select the coordinates to use
    97 /*        JLabel jLabelRes = new JLabel(tr("Coordinates system to read in CSV files:"));
     97        /*        JLabel jLabelRes = new JLabel(tr("Coordinates system to read in CSV files:"));
    9898        p.add(jLabelRes, GBC.std().insets(0, 5, 10, 0));
    9999        ButtonGroup bgCoordinates = new ButtonGroup();
    … …  
    116116        general.add(jLabelOapi, GBC.std().insets(0, 5, 10, 0));
    117117        general.add(oapi, GBC.eol().fill(GBC.HORIZONTAL).insets(5, 5, 200, 5));
    118        
     118
    119119        // option to set the XAPI server
    120120        JLabel jLabelXapi = new JLabel(tr("XAPI server:"));
    … …  
    123123        general.add(jLabelXapi, GBC.std().insets(0, 5, 10, 0));
    124124        general.add(xapi, GBC.eol().fill(GBC.HORIZONTAL).insets(5, 5, 200, 5));
    125        
     125
    126126        // end of dialog, scroll bar
    127127        general.add(Box.createVerticalGlue(), GBC.eol().fill(GBC.VERTICAL));
    128        
     128
    129129        return general;
    130130    }
    131    
     131
    132132    @Override
    133133    public boolean ok() {
    134134        boolean result = modulePref.ok();
    135            //Main.pref.put(PREF_COORDINATES, rbWGS84.isSelected() ? VALUE_WGS84 : VALUE_CC9ZONES);
    136            Main.pref.put(OdConstants.PREF_OAPI, oapi.getText());
    137            Main.pref.put(OdConstants.PREF_XAPI, xapi.getText());
    138            Main.pref.put(OdConstants.PREF_RAWDATA, rawData.isSelected());
    139        
     135        //Main.pref.put(PREF_COORDINATES, rbWGS84.isSelected() ? VALUE_WGS84 : VALUE_CC9ZONES);
     136        Main.pref.put(OdConstants.PREF_OAPI, oapi.getText());
     137        Main.pref.put(OdConstants.PREF_XAPI, xapi.getText());
     138        Main.pref.put(OdConstants.PREF_RAWDATA, rawData.isSelected());
     139
    140140        // create a task for downloading modules if the user has activated, yet not downloaded,
    141141        // new modules
    … …  
    143143        final List<ModuleInformation> toDownload = modulePref.getModulesScheduledForUpdateOrDownload();
    144144        final ModuleDownloadTask task;
    145         if (toDownload != null && ! toDownload.isEmpty()) {
     145        if (toDownload != null && !toDownload.isEmpty()) {
    146146            task = new ModuleDownloadTask(masterPanel, toDownload, tr("Download modules"));
    147147        } else {
    148148            task = null;
    149149        }
    150        
     150
    151151        // this is the task which will run *after* the modules are downloaded
    152152        //
    153153        final Runnable continuation = new Runnable() {
     154            @Override
    154155            public void run() {
    155156                boolean requiresRestart = false;
    … …  
    195196            Main.worker.submit(
    196197                    new Runnable() {
     198                        @Override
    197199                        public void run() {
    198200                            SwingUtilities.invokeLater(continuation);

  • applications/editors/josm/plugins/opendata/src/org/openstreetmap/josm/plugins/opendata/core/gui/ProjectionChooser.java

    r30723 r32545  
    5252    private JLabel projectionName = new JLabel();
    5353    private JLabel bounds = new JLabel();
    54    
     54
    5555    /**
    5656     * This is the panel holding all projection preferences
    … …  
    6868        this(parent, tr("Projection method"), new String[] {tr("OK"), tr("Cancel")});
    6969    }
    70    
     70
    7171    protected ProjectionChooser(Component parent, String title, String[] buttonTexts) {
    7272        super(parent, title, buttonTexts);
    … …  
    7474        addGui();
    7575    }
    76    
     76
    7777    public void addGui() {
    78         projPanel.setBorder(BorderFactory.createEmptyBorder( 0, 0, 0, 0 ));
     78        projPanel.setBorder(BorderFactory.createEmptyBorder(0, 0, 0, 0));
    7979        projPanel.setLayout(new GridBagLayout());
    80         projPanel.add(new JLabel(tr("Projection method")), GBC.std().insets(5,5,0,5));
    81         projPanel.add(GBC.glue(5,0), GBC.std().fill(GBC.HORIZONTAL));
    82         projPanel.add(projectionCombo, GBC.eop().fill(GBC.HORIZONTAL).insets(0,5,5,5));
    83         projPanel.add(projectionCodeLabel = new JLabel(tr("Projection code")), GBC.std().insets(25,5,0,5));
    84         projPanel.add(projectionCodeGlue = GBC.glue(5,0), GBC.std().fill(GBC.HORIZONTAL));
    85         projPanel.add(projectionCode, GBC.eop().fill(GBC.HORIZONTAL).insets(0,5,5,5));
    86         projPanel.add(projectionNameLabel = new JLabel(tr("Projection name")), GBC.std().insets(25,5,0,5));
    87         projPanel.add(projectionNameGlue = GBC.glue(5,0), GBC.std().fill(GBC.HORIZONTAL));
    88         projPanel.add(projectionName, GBC.eop().fill(GBC.HORIZONTAL).insets(0,5,5,5));
    89         projPanel.add(new JLabel(tr("Bounds")), GBC.std().insets(25,5,0,5));
    90         projPanel.add(GBC.glue(5,0), GBC.std().fill(GBC.HORIZONTAL));
    91         projPanel.add(bounds, GBC.eop().fill(GBC.HORIZONTAL).insets(0,5,5,5));
    92         projPanel.add(projSubPrefPanelWrapper, GBC.eol().fill(GBC.HORIZONTAL).insets(20,5,5,5));
     80        projPanel.add(new JLabel(tr("Projection method")), GBC.std().insets(5, 5, 0, 5));
     81        projPanel.add(GBC.glue(5, 0), GBC.std().fill(GBC.HORIZONTAL));
     82        projPanel.add(projectionCombo, GBC.eop().fill(GBC.HORIZONTAL).insets(0, 5, 5, 5));
     83        projPanel.add(projectionCodeLabel = new JLabel(tr("Projection code")), GBC.std().insets(25, 5, 0, 5));
     84        projPanel.add(projectionCodeGlue = GBC.glue(5, 0), GBC.std().fill(GBC.HORIZONTAL));
     85        projPanel.add(projectionCode, GBC.eop().fill(GBC.HORIZONTAL).insets(0, 5, 5, 5));
     86        projPanel.add(projectionNameLabel = new JLabel(tr("Projection name")), GBC.std().insets(25, 5, 0, 5));
     87        projPanel.add(projectionNameGlue = GBC.glue(5, 0), GBC.std().fill(GBC.HORIZONTAL));
     88        projPanel.add(projectionName, GBC.eop().fill(GBC.HORIZONTAL).insets(0, 5, 5, 5));
     89        projPanel.add(new JLabel(tr("Bounds")), GBC.std().insets(25, 5, 0, 5));
     90        projPanel.add(GBC.glue(5, 0), GBC.std().fill(GBC.HORIZONTAL));
     91        projPanel.add(bounds, GBC.eop().fill(GBC.HORIZONTAL).insets(0, 5, 5, 5));
     92        projPanel.add(projSubPrefPanelWrapper, GBC.eol().fill(GBC.HORIZONTAL).insets(20, 5, 5, 5));
    9393
    9494        selectedProjectionChanged((ProjectionChoice) projectionCombo.getSelectedItem());
    … …  
    102102        setContent(projPanel);
    103103    }
    104    
     104
    105105    private void selectedProjectionChanged(final ProjectionChoice pc) {
    106106        // Don't try to update if we're still starting up
    107107        int size = projPanel.getComponentCount();
    108         if(size < 1)
     108        if (size < 1)
    109109            return;
    110110
    … …  
    124124        updateMeta(pc);
    125125    }
    126    
     126
    127127    private void updateMeta(ProjectionChoice pc) {
    128128        pc.setPreferences(pc.getPreferences(projSubPrefPanel));
    … …  
    132132        Bounds b = proj.getWorldBoundsLatLon();
    133133        CoordinateFormat cf = CoordinateFormat.getDefaultFormat();
    134         bounds.setText(b.getMin().lonToString(cf)+", "+b.getMin().latToString(cf)+" : "+b.getMax().lonToString(cf)+", "+b.getMax().latToString(cf));
     134        bounds.setText(b.getMin().lonToString(cf)+", "+b.getMin().latToString(cf)+" : "+
     135                       b.getMax().lonToString(cf)+", "+b.getMax().latToString(cf));
    135136        boolean showCode = true;
    136137        boolean showName = false;

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

    r30723 r32545  
    1010
    1111public class LambertCC9ZonesProjectionPatterns extends ProjectionPatterns {
    12    
     12
    1313    public static final Projection[] lambertCC9Zones = new Projection[9];
    1414    static {
    15         for (int i=0; i<lambertCC9Zones.length; i++) {
     15        for (int i = 0; i < lambertCC9Zones.length; i++) {
    1616            lambertCC9Zones[i] = Projections.getProjectionByCode("EPSG:"+Integer.toString(3942+i));
    1717        }
    … …  
    3232        String ccy = mx.group(1);
    3333        if (!ccx.equals(ccy)) {
    34             throw new IllegalArgumentException(tr("''Lambert CC 9 zones'' coordinates found with different zone codes for X and Y: "+xFieldName+", "+yFieldName));
     34            throw new IllegalArgumentException(
     35                    tr("''Lambert CC 9 zones'' coordinates found with different zone codes for X and Y: "+xFieldName+", "+yFieldName));
    3536        }
    3637        return lambertCC9Zones[Integer.parseInt(ccx)-42];

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

    r30795 r32545  
    2222import javax.xml.validation.SchemaFactory;
    2323import javax.xml.validation.Validator;
     24
     25import org.openstreetmap.josm.Main;
     26import org.openstreetmap.josm.data.coor.LatLon;
     27import org.openstreetmap.josm.data.osm.DataSet;
     28import org.openstreetmap.josm.data.osm.Node;
     29import org.openstreetmap.josm.data.osm.OsmPrimitive;
     30import org.openstreetmap.josm.data.osm.Relation;
     31import org.openstreetmap.josm.data.osm.RelationMember;
     32import org.openstreetmap.josm.gui.progress.ProgressMonitor;
     33import org.openstreetmap.josm.io.AbstractReader;
     34import org.openstreetmap.josm.plugins.opendata.core.datasets.AbstractDataSetHandler;
     35import org.openstreetmap.josm.plugins.opendata.core.datasets.fr.FrenchConstants;
     36import org.xml.sax.SAXException;
    2437
    2538import neptune.ChouetteAreaType;
    … …  
    3750import neptune.StopPointType;
    3851import neptune.TridentObjectType;
    39 
    40 import org.openstreetmap.josm.Main;
    41 import org.openstreetmap.josm.data.coor.LatLon;
    42 import org.openstreetmap.josm.data.osm.DataSet;
    43 import org.openstreetmap.josm.data.osm.Node;
    44 import org.openstreetmap.josm.data.osm.OsmPrimitive;
    45 import org.openstreetmap.josm.data.osm.Relation;
    46 import org.openstreetmap.josm.data.osm.RelationMember;
    47 import org.openstreetmap.josm.gui.progress.ProgressMonitor;
    48 import org.openstreetmap.josm.io.AbstractReader;
    49 import org.openstreetmap.josm.plugins.opendata.core.datasets.AbstractDataSetHandler;
    50 import org.openstreetmap.josm.plugins.opendata.core.datasets.fr.FrenchConstants;
    51 import org.xml.sax.SAXException;
    5252
    5353/**
    … …  
    127127        Unmarshaller u = jc.createUnmarshaller();
    128128        @SuppressWarnings("unchecked")
    129         JAXBElement<T> doc = (JAXBElement<T>)u.unmarshal(inputStream);
     129        JAXBElement<T> doc = (JAXBElement<T>) u.unmarshal(inputStream);
    130130        return doc.getValue();
    131131    }
    132132
    133     private final void linkTridentObjectToOsmPrimitive(TridentObjectType object, OsmPrimitive p) {
     133    private void linkTridentObjectToOsmPrimitive(TridentObjectType object, OsmPrimitive p) {
    134134        p.put("ref:neptune", object.getObjectId());
    135135        if (tridentObjects.put(object.getObjectId(), p) != null) {
    … …  
    138138    }
    139139
    140     protected Node createNode(LatLon latlon){
     140    protected Node createNode(LatLon latlon) {
    141141        Node n = new Node(latlon);
    142142        ds.addPrimitive(n);
    … …  
    152152    }
    153153
    154     protected Relation createRelation(String type){
     154    protected Relation createRelation(String type) {
    155155        Relation r = new Relation();
    156156        r.put("type", type);
    … …  
    159159    }
    160160
    161     protected Relation createPtRelation(String pt, TridentObjectType object){
     161    protected Relation createPtRelation(String pt, TridentObjectType object) {
    162162        Relation r = createRelation(OSM_PUBLIC_TRANSPORT);
    163163        r.put(OSM_PUBLIC_TRANSPORT, pt);
    … …  
    349349    }
    350350
    351     private static final boolean addStopToRoute(Relation route, OsmPrimitive stop) {
    352         if (route.getMembersCount() == 0 || !route.getMember(route.getMembersCount()-1).getMember().equals(stop) ) {
     351    private static boolean addStopToRoute(Relation route, OsmPrimitive stop) {
     352        if (route.getMembersCount() == 0 || !route.getMember(route.getMembersCount()-1).getMember().equals(stop)) {
    353353            route.addMember(new RelationMember(stop.get(OSM_PUBLIC_TRANSPORT), stop));
    354354            return true;

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

    r31937 r32545  
    4242    private File file;
    4343    private String filename;
    44    
     44
    4545    /**
    4646     * File readers
    … …  
    5858        FILE_READERS.put(OdConstants.TAB_EXT, TabReader.class);
    5959    }
    60    
     60
    6161    public static final Map<String, Class<? extends AbstractReader>> FILE_AND_ARCHIVE_READERS = new HashMap<>(FILE_READERS);
    6262    static {
    … …  
    7272        this.promptUser = promptUser;
    7373    }
    74    
     74
    7575    public final File getReadFile() {
    7676        return file;
    … …  
    105105        } else if (contentType.startsWith("text/csv")) {
    106106            return CsvReader.class;
    107         } else if (contentType.startsWith("text/plain")) {//TODO: extract charset
     107        } else if (contentType.startsWith("text/plain")) { //TODO: extract charset
    108108            return CsvReader.class;
    109109        } else if (contentType.startsWith("tdyn/html")) {
    … …  
    144144            }
    145145            if (readerClass == null) {
    146                    throw new OsmTransferException("Cannot find appropriate reader !");//TODO handler job ?
     146                throw new OsmTransferException("Cannot find appropriate reader !"); //TODO handler job ?
    147147            } else if (findReaderByExtension(url) != null) {
    148148                filename = url.substring(url.lastIndexOf('/')+1);
    … …  
    150150            instance = progressMonitor.createSubTaskMonitor(ProgressMonitor.ALL_TICKS, false);
    151151            if (readerClass.equals(ZipReader.class) || readerClass.equals(SevenZipReader.class)) {
    152                 ArchiveReader zipReader = readerClass.equals(ZipReader.class) 
     152                ArchiveReader zipReader = readerClass.equals(ZipReader.class)
    153153                        ? new ZipReader(in, handler, promptUser) : new SevenZipReader(in, handler, promptUser);
    154                 DataSet ds = zipReader.parseDoc(instance);
    155                 file = zipReader.getReadFile();
    156                 return ds;
     154                        DataSet ds = zipReader.parseDoc(instance);
     155                        file = zipReader.getReadFile();
     156                        return ds;
    157157            } else if (readerClass.equals(KmlReader.class)) {
    158158                return KmlReader.parseDataSet(in, instance);
    … …  
    189189                    in.close();
    190190                }
    191             } catch (Exception e) {/* ignore it */}
     191            } catch (Exception e) {
     192                Main.trace(e);
     193            }
    192194        }
    193195    }

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

    r30723 r32545  
    99import org.openstreetmap.josm.plugins.opendata.core.OdConstants;
    1010
    11 public class OsmDownloader {
    12    
    13     public static final void downloadOapi(String oapiReq) {
     11public final class OsmDownloader {
     12
     13    private OsmDownloader() {
     14        // Hide default constructor for utilities classes
     15    }
     16
     17    public static void downloadOapi(String oapiReq) {
    1418        if (oapiReq != null) {
    1519            try {
    … …  
    2327        }
    2428    }
    25    
    26     public static final void downloadXapi(Collection<String> xapiReqs) {
     29
     30    public static void downloadXapi(Collection<String> xapiReqs) {
    2731        if (xapiReqs != null) {
    2832            String xapiServer = Main.pref.get(OdConstants.PREF_XAPI, OdConstants.DEFAULT_XAPI);

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

    r30723 r32545  
    55
    66    public enum OaQueryType {
    7         NODE ("node"),
    8         WAY ("way"),
    9         RELATION ("relation");
     7        NODE("node"),
     8        WAY("way"),
     9        RELATION("relation");
     10
    1011        @Override
    11         public String toString() { return this.value; }
    12         private OaQueryType(final String value) { this.value = value; }
     12        public String toString() {
     13            return this.value;
     14        }
     15
     16        OaQueryType(final String value) {
     17            this.value = value;
     18        }
     19
    1320        private final String value;
    1421    }
    1522
    1623    public enum OaRecurseType {
    17         RELATION_RELATION ("relation-relation"),
    18         RELATION_BACKWARDS ("relation-backwards"),
    19         RELATION_WAY ("relation-way"),
    20         RELATION_NODE ("relation-node"),
    21         WAY_NODE ("way-node"),
    22         WAY_RELATION ("way-relation"),
    23         NODE_RELATION ("node-relation"),
    24         NODE_WAY ("node-way");
     24        RELATION_RELATION("relation-relation"),
     25        RELATION_BACKWARDS("relation-backwards"),
     26        RELATION_WAY("relation-way"),
     27        RELATION_NODE("relation-node"),
     28        WAY_NODE("way-node"),
     29        WAY_RELATION("way-relation"),
     30        NODE_RELATION("node-relation"),
     31        NODE_WAY("node-way");
     32
    2533        @Override
    26         public String toString() { return this.value; }
    27         private OaRecurseType(final String value) { this.value = value; }
     34        public String toString() {
     35            return this.value;
     36        }
     37
     38        OaRecurseType(final String value) {
     39            this.value = value;
     40        }
     41
    2842        private final String value;
    2943    }
    30    
     44
    3145    public static final String union(String ... queries) {
    3246        String result = "<union>\n";
    … …  
    3953        return result;
    4054    }
    41    
     55
    4256    public static final String query(String bbox, OaQueryType type, String ... conditions) {
    4357        String result = "<query type=\""+type+"\" >\n";
    … …  
    6579        return result;
    6680    }
    67    
     81
    6882    public static final String print() {
    6983        return "<print mode=\"meta\"/>";
    7084    }
    71    
     85
    7286    public static final String hasKey(String key) {
    7387        return hasKey(key, null);

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

    r30723 r32545  
    66public interface ArchiveHandler {
    77
    8     public void notifyTempFileWritten(File file);
     8    void notifyTempFileWritten(File file);
    99
    10     public boolean skipXsdValidation();
     10    boolean skipXsdValidation();
    1111
    12     public void setSkipXsdValidation(boolean skip);
     12    void setSkipXsdValidation(boolean skip);
    1313}

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

    r30795 r32545  
    5252        this.promptUser = promptUser;
    5353    }
    54    
     54
    5555    public final File getReadFile() {
    5656        return file;
    5757    }
    58    
     58
    5959    protected abstract void extractArchive(final File temp, final List<File> candidates) throws IOException, FileNotFoundException;
    60    
     60
    6161    protected abstract String getTaskMessage();
    62    
     62
    6363    protected Collection<File> getDocsToParse(final File temp, final ProgressMonitor progressMonitor) throws FileNotFoundException, IOException {
    6464        final List<File> candidates = new ArrayList<>();
    65        
     65
    6666        if (progressMonitor != null) {
    6767            progressMonitor.beginTask(getTaskMessage());
    6868        }
    6969        extractArchive(temp, candidates);
    70        
     70
    7171        if (promptUser && candidates.size() > 1) {
    7272            DialogPrompter<CandidateChooser> prompt = new DialogPrompter<CandidateChooser>() {
    … …  
    8484        return Collections.emptyList();
    8585    }
    86    
    87     public Map<File, DataSet> parseDocs(final ProgressMonitor progressMonitor) throws IOException, XMLStreamException, FactoryConfigurationError, JAXBException {
     86
     87    public Map<File, DataSet> parseDocs(final ProgressMonitor progressMonitor)
     88            throws IOException, XMLStreamException, FactoryConfigurationError, JAXBException {
    8889        Map<File, DataSet> result = new HashMap<>();
    8990        File temp = OdUtils.createTempDir();
    … …  
    104105        return result;
    105106    }
    106    
    107     public DataSet parseDoc(final ProgressMonitor progressMonitor) throws IOException, XMLStreamException, FactoryConfigurationError, JAXBException  {
     107
     108    public DataSet parseDoc(final ProgressMonitor progressMonitor)
     109            throws IOException, XMLStreamException, FactoryConfigurationError, JAXBException {
    108110        File temp = OdUtils.createTempDir();
    109        
     111
    110112        try {
    111             file = null;       
     113            file = null;
    112114            Collection<File> files = getDocsToParse(temp, progressMonitor);
    113115            if (!files.isEmpty()) {
    114116                file = files.iterator().next();
    115117            }
    116            
     118
    117119            DataSet from = getDataForFile(file, progressMonitor);
    118120            if (from != null) {
    … …  
    127129            }
    128130        }
    129        
     131
    130132        return ds;
    131133    }
    … …  
    186188        }
    187189        // Special treatment for XML files (check supported XSD), unless handler explicitely skip it
    188         if (XmlImporter.XML_FILE_FILTER.accept(file) && ((archiveHandler != null && archiveHandler.skipXsdValidation()) 
     190        if (XmlImporter.XML_FILE_FILTER.accept(file) && ((archiveHandler != null && archiveHandler.skipXsdValidation())
    189191                || OdPlugin.getInstance().xmlImporter.acceptFile(file))) {
    190192            candidates.add(file);

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

    r30723 r32545  
    3232        this(parent, tr("File to load"), new String[] {tr("OK"), tr("Cancel")}, candidates);
    3333    }
    34    
     34
    3535    private class Renderer extends DefaultListCellRenderer {
    3636
    … …  
    4040            super.getListCellRendererComponent(list, value, index, isSelected, cellHasFocus);
    4141            if (value instanceof File) {
    42                 setText(((File)value).getName());
     42                setText(((File) value).getName());
    4343            }
    4444            return this;
    4545        }
    4646    }
    47    
     47
    4848    protected CandidateChooser(Component parent, String title, String[] buttonTexts, List<File> candidates) {
    4949        super(parent, title, buttonTexts);
    … …  
    5252        addGui(candidates);
    5353    }
    54    
     54
    5555    public void addGui(List<File> candidates) {
    56         projPanel.setBorder(BorderFactory.createEmptyBorder( 0, 0, 0, 0 ));
     56        projPanel.setBorder(BorderFactory.createEmptyBorder(0, 0, 0, 0));
    5757        projPanel.setLayout(new GridBagLayout());
    58         projPanel.add(new JLabel(tr("File to load")), GBC.std().insets(5,5,0,5));
    59         projPanel.add(GBC.glue(5,0), GBC.std().fill(GBC.HORIZONTAL));
     58        projPanel.add(new JLabel(tr("File to load")), GBC.std().insets(5, 5, 0, 5));
     59        projPanel.add(GBC.glue(5, 0), GBC.std().fill(GBC.HORIZONTAL));
    6060        /*for (File file : candidates) {
    6161            JCheckBox cbox = new JCheckBox(file.getName());
    6262            checkBoxes.put(cbox, file);
    63             projPanel.add(cbox, GBC.eop().fill(GBC.HORIZONTAL).insets(0,5,5,5));
     63            projPanel.add(cbox, GBC.eop().fill(GBC.HORIZONTAL).insets(0, 5, 5, 5));
    6464        }*/
    65         projPanel.add(fileCombo, GBC.eop().fill(GBC.HORIZONTAL).insets(0,5,5,5));
     65        projPanel.add(fileCombo, GBC.eop().fill(GBC.HORIZONTAL).insets(0, 5, 5, 5));
    6666        setContent(projPanel);
    6767    }
    68    
     68
    6969    /*public List<File> getSelectedFiles() {
    7070        List<File> result = new ArrayList<File>();
    … …  
    7676        return result;
    7777    }*/
    78    
     78
    7979    public File getSelectedFile() {
    8080        return (File) fileCombo.getSelectedItem();

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

    r31924 r32545  
    3333
    3434    private final IInArchive archive = new Handler();
    35    
     35
    3636    public SevenZipReader(InputStream in, AbstractDataSetHandler handler, boolean promptUser) throws IOException {
    3737        super(handler, handler != null ? handler.getArchiveHandler() : null, promptUser);
    … …  
    5252        }
    5353    }
    54    
    55     public static DataSet parseDataSet(InputStream in, AbstractDataSetHandler handler, ProgressMonitor instance, boolean promptUser) 
     54
     55    public static DataSet parseDataSet(InputStream in, AbstractDataSetHandler handler, ProgressMonitor instance, boolean promptUser)
    5656            throws IOException, XMLStreamException, FactoryConfigurationError, JAXBException {
    5757        return new SevenZipReader(in, handler, promptUser).parseDoc(instance);
    5858    }
    5959
    60     public static Map<File, DataSet> parseDataSets(InputStream in, AbstractDataSetHandler handler, ProgressMonitor instance, boolean promptUser) 
     60    public static Map<File, DataSet> parseDataSets(InputStream in, AbstractDataSetHandler handler, ProgressMonitor instance, boolean promptUser)
    6161            throws IOException, XMLStreamException, FactoryConfigurationError, JAXBException {
    6262        return new SevenZipReader(in, handler, promptUser).parseDocs(instance);
    … …  
    7373        archive.close();
    7474    }
    75    
     75
    7676    private class ExtractCallback extends ArchiveExtractCallback {
    7777        private final List<File> candidates;
    78        
    79         public ExtractCallback(IInArchive archive, File tempDir, List<File> candidates) {
     78
     79        ExtractCallback(IInArchive archive, File tempDir, List<File> candidates) {
    8080            Init(archive);
    8181            super.outputDir = tempDir.getPath();
    … …  
    8383        }
    8484
    85         @Override 
     85        @Override
    8686        public int GetStream(int index, OutputStream[] outStream, int askExtractMode) throws IOException {
    8787            int res = super.GetStream(index, outStream, askExtractMode);

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

    r30795 r32545  
    2323    protected static final String VERSION_450 = "450";
    2424
    25     protected static final String CHARSET_WINDOWS_LATIN    = "WindowsLatin1";
     25    protected static final String CHARSET_WINDOWS_LATIN = "WindowsLatin1";
    2626    protected static final String CHARSET_WINDOWS_CYRILLIC = "WindowsCyrillic";
    2727    protected static final String CHARSET_NEUTRAL = "Neutral";
    28     protected static final String CHARSET_MAC     = "MacRoman";
    29    
     28    protected static final String CHARSET_MAC = "MacRoman";
     29
    3030    protected BufferedReader headerReader;
    3131
    … …  
    4747        return dataFile;
    4848    }
    49    
     49
    5050    protected final BufferedReader getDataReader(File headerFile, String extension, Charset charset) throws IOException {
    5151        File dataFile = getDataFile(headerFile, extension);
    … …  
    7676        version = words[1];
    7777    }
    78    
     78
    7979    protected void parseColumns(String[] words) {
    8080        columns = new ArrayList<>();
    … …  
    8888                line = line.replace("  ", " ");
    8989            }
    90             String [] words = line.isEmpty() ? null : line.trim().split(" ");
     90            String[] words = line.isEmpty() ? null : line.trim().split(" ");
    9191            if (words != null && words.length > 0) {
    9292                parseHeaderLine(words);
    … …  
    9494        }
    9595    }
    96    
     96
    9797    protected abstract void parseHeaderLine(String[] words) throws IOException;
    9898}

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

    r30795 r32545  
    3737public class DefaultShpHandler extends DefaultGeographicHandler implements ShpHandler {
    3838
    39     private static final List<Pair<org.opengis.referencing.datum.Ellipsoid, Ellipsoid>> 
    40         ellipsoids = new ArrayList<>();
     39    private static final List<Pair<org.opengis.referencing.datum.Ellipsoid, Ellipsoid>>
     40    ellipsoids = new ArrayList<>();
    4141    static {
    4242        ellipsoids.add(new Pair<org.opengis.referencing.datum.Ellipsoid, Ellipsoid>(DefaultEllipsoid.GRS80, Ellipsoid.GRS80));
    4343        ellipsoids.add(new Pair<org.opengis.referencing.datum.Ellipsoid, Ellipsoid>(DefaultEllipsoid.WGS84, Ellipsoid.WGS84));
    4444    }
    45    
     45
    4646    protected static final Double get(ParameterValueGroup values, ParameterDescriptor<?> desc) {
    4747        return (Double) values.parameter(desc.getName().getCode()).getValue();
    4848    }
    49    
    50     private static final boolean equals(Double a, Double b) {
    51         boolean res = Math.abs(a - b) <= Main.pref.getDouble(OdConstants.PREF_CRS_COMPARISON_TOLERANCE, OdConstants.DEFAULT_CRS_COMPARISON_TOLERANCE);
     49
     50    private static boolean equals(Double a, Double b) {
     51        boolean res = Math.abs(a - b) <= Main.pref.getDouble(
     52                OdConstants.PREF_CRS_COMPARISON_TOLERANCE, OdConstants.DEFAULT_CRS_COMPARISON_TOLERANCE);
    5253        if (Main.pref.getBoolean(OdConstants.PREF_CRS_COMPARISON_DEBUG, false)) {
    5354            Main.debug("Comparing "+a+" and "+b+" -> "+res);
    5455        }
    55         return res; 
     56        return res;
    5657    }
    57    
     58
    5859    private Charset dbfCharset = null;
    5960
    … …  
    8081                                    ParameterValueGroup values = lambert.getParameterValues();
    8182                                    Parameters params = ((LambertConformalConic) ap.getProj()).getParameters();
    82                                    
     83
    8384                                    ok = ok ? equals(get(values, AbstractProvider.LATITUDE_OF_ORIGIN), params.latitudeOrigin) : ok;
    8485                                    ok = ok ? equals(get(values, AbstractProvider.CENTRAL_MERIDIAN), ap.getCentralMeridian()) : ok;
    … …  
    8687                                    ok = ok ? equals(get(values, AbstractProvider.FALSE_EASTING), ap.getFalseEasting()) : ok;
    8788                                    ok = ok ? equals(get(values, AbstractProvider.FALSE_NORTHING), ap.getFalseNorthing()) : ok;
    88                                    
     89
    8990                                    if (lambert instanceof LambertConformal2SP && params instanceof Parameters2SP) {
    9091                                        Parameters2SP param = (Parameters2SP) params;
    91                                         ok = ok ? equals(Math.min(get(values, AbstractProvider.STANDARD_PARALLEL_1),get(values, AbstractProvider.STANDARD_PARALLEL_2)),
    92                                                          Math.min(param.standardParallel1, param.standardParallel2)) : ok;
    93                                         ok = ok ? equals(Math.max(get(values, AbstractProvider.STANDARD_PARALLEL_1), get(values, AbstractProvider.STANDARD_PARALLEL_2)),
    94                                                          Math.max(param.standardParallel1, param.standardParallel2)) : ok;
    95                                        
     92                                        ok = ok ? equals(Math.min(get(values, AbstractProvider.STANDARD_PARALLEL_1),
     93                                                get(values, AbstractProvider.STANDARD_PARALLEL_2)),
     94                                                Math.min(param.standardParallel1, param.standardParallel2)) : ok;
     95                                        ok = ok ? equals(Math.max(get(values, AbstractProvider.STANDARD_PARALLEL_1),
     96                                                get(values, AbstractProvider.STANDARD_PARALLEL_2)),
     97                                                Math.max(param.standardParallel1, param.standardParallel2)) : ok;
     98
    9699                                    } else if (!(lambert instanceof LambertConformal1SP && params instanceof Parameters1SP)) {
    97100                                        ok = false;
    … …  
    130133        dbfCharset = charset;
    131134    }
    132    
     135
    133136    @Override
    134137    public Charset getDbfCharset() {

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

    r30723 r32545  
    88
    99public interface GeographicHandler {
    10    
    11     public void setPreferMultipolygonToSimpleWay(boolean prefer);
    1210
    13     public boolean preferMultipolygonToSimpleWay();
     11    void setPreferMultipolygonToSimpleWay(boolean prefer);
    1412
    15     public void setCheckNodeProximity(boolean check);
    16    
    17     public boolean checkNodeProximity();
    18    
    19     public void setUseNodeMap(boolean use);
    20    
    21     public boolean useNodeMap();
    22    
    23     public CoordinateReferenceSystem getCrsFor(String crsName) throws NoSuchAuthorityCodeException, FactoryException;
     13    boolean preferMultipolygonToSimpleWay();
    2414
    25     public MathTransform findMathTransform(CoordinateReferenceSystem sourceCRS, CoordinateReferenceSystem targetCRS, boolean lenient) throws FactoryException;
     15    void setCheckNodeProximity(boolean check);
     16
     17    boolean checkNodeProximity();
     18
     19    void setUseNodeMap(boolean use);
     20
     21    boolean useNodeMap();
     22
     23    CoordinateReferenceSystem getCrsFor(String crsName) throws NoSuchAuthorityCodeException, FactoryException;
     24
     25    MathTransform findMathTransform(CoordinateReferenceSystem sourceCRS, CoordinateReferenceSystem targetCRS, boolean lenient)
     26            throws FactoryException;
    2627}

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

    r32494 r32545  
    6666        }
    6767    }
    68    
     68
    6969    private final GeographicHandler handler;
    7070    private final GeographicHandler[] defaultHandlers;
    71    
     71
    7272    protected final Map<LatLon, Node> nodes;
    7373
    … …  
    8080        this.defaultHandlers = defaultHandlers;
    8181    }
    82    
     82
    8383    protected Node getNode(Point p, LatLon key) {
    8484        Node n = nodes.get(key);
    … …  
    120120        return n;
    121121    }
    122    
     122
    123123    protected Node createOrGetEmptyNode(Point p) throws MismatchedDimensionException, TransformException {
    124124        Point p2 = (Point) JTS.transform(p, transform);
    … …  
    140140        return n;
    141141    }
    142        
     142
    143143    protected <T extends OsmPrimitive> T addOsmPrimitive(T p) {
    144144        ds.addPrimitive(p);
    … …  
    154154        Way tempWay = new Way();
    155155        if (ls != null) {
    156             // Build list of nodes 
    157             for (int i=0; i<ls.getNumPoints(); i++) {
     156            // Build list of nodes
     157            for (int i = 0; i < ls.getNumPoints(); i++) {
    158158                try {
    159159                    tempWay.addNode(createOrGetNode(ls.getPointN(i)));
    … …  
    195195        r.addMember(new RelationMember(role, w));
    196196    }
    197    
     197
    198198    /**
    199199     * returns true if the user wants to cancel, false if they
    … …  
    207207                        tr("Cannot transform to WGS84"),
    208208                        new String[] {tr("Cancel"), tr("Continue")});
     209                // CHECKSTYLE.OFF: LineLength
    209210                dlg.setContent("<html>" +
    210211                        tr("JOSM was unable to find a strict mathematical transformation between ''{0}'' and WGS84.<br /><br />"+
    211                                 "Do you want to try a <i>lenient</i> method, which will perform a non-precise transformation (<b>with location errors up to 1 km</b>) ?<br/><br/>"+
    212                                 "If so, <b>do NOT upload</b> such data to OSM !", crs.getName())+
     212                        "Do you want to try a <i>lenient</i> method, which will perform a non-precise transformation (<b>with location errors up to 1 km</b>) ?<br/><br/>"+
     213                        "If so, <b>do NOT upload</b> such data to OSM !", crs.getName())+
    213214                        "</html>");
    214                dlg.setButtonIcons(new Icon[] {
     215                // CHECKSTYLE.ON: LineLength
     216                dlg.setButtonIcons(new Icon[] {
    215217                        new ImageProvider("cancel").setMaxSize(ImageSizes.LARGEICON).get(),
    216218                        new ImageProvider("ok").setMaxSize(ImageSizes.LARGEICON).addOverlay(
    217                                 new ImageOverlay(new ImageProvider("warning-small"), 0.5,0.5,1.0,1.0)).get()});
     219                                new ImageOverlay(new ImageProvider("warning-small"), 0.5, 0.5, 1.0, 1.0)).get()});
    218220                dlg.setToolTipTexts(new String[] {
    219221                        tr("Cancel"),
    … …  
    225227        }.promptInEdt().getValue() != 2;
    226228    }
    227    
    228     private static final void compareDebug(CoordinateReferenceSystem crs1, CoordinateReferenceSystem crs2) {
     229
     230    private static void compareDebug(CoordinateReferenceSystem crs1, CoordinateReferenceSystem crs2) {
    229231        Main.debug("-- COMPARING "+crs1.getName()+" WITH "+crs2.getName()+" --");
    230232        compareDebug("class", crs1.getClass(), crs2.getClass());
    … …  
    235237            Integer dim2 = cs2.getDimension();
    236238            if (compareDebug("cs.dim", dim1, dim2)) {
    237                 for (int i = 0; i<dim1; i++) {
     239                for (int i = 0; i < dim1; i++) {
    238240                    compareDebug("cs.axis"+i, cs1.getAxis(i), cs1.getAxis(i));
    239241                }
    … …  
    258260        Main.debug("-- COMPARING FINISHED --");
    259261    }
    260    
    261     private static final boolean compareDebug(String text, Object o1, Object o2) {
     262
     263    private static boolean compareDebug(String text, Object o1, Object o2) {
    262264        return compareDebug(text, o1.equals(o2), o1, o2);
    263265    }
    264    
    265     private static final boolean compareDebug(String text, IdentifiedObject o1, IdentifiedObject o2) {
    266         return compareDebug(text, (AbstractIdentifiedObject)o1, (AbstractIdentifiedObject)o2);
    267     }
    268    
    269     private static final boolean compareDebug(String text, AbstractIdentifiedObject o1, AbstractIdentifiedObject o2) {
     266
     267    private static boolean compareDebug(String text, IdentifiedObject o1, IdentifiedObject o2) {
     268        return compareDebug(text, (AbstractIdentifiedObject) o1, (AbstractIdentifiedObject) o2);
     269    }
     270
     271    private static boolean compareDebug(String text, AbstractIdentifiedObject o1, AbstractIdentifiedObject o2) {
    270272        return compareDebug(text, o1.equals(o2, false), o1, o2);
    271273    }
    272274
    273     private static final boolean compareDebug(String text, boolean result, Object o1, Object o2) {
     275    private static boolean compareDebug(String text, boolean result, Object o1, Object o2) {
    274276        Main.debug(text + ": " + result + "("+o1+", "+o2+")");
    275277        return result;
    276278    }
    277    
    278     protected void findMathTransform(Component parent, boolean findSimiliarCrs) throws FactoryException, UserCancelException, GeoMathTransformException {
     279
     280    protected void findMathTransform(Component parent, boolean findSimiliarCrs)
     281            throws FactoryException, UserCancelException, GeoMathTransformException {
    279282        try {
    280283            transform = CRS.findMathTransform(crs, wgs84);
    281284        } catch (OperationNotFoundException e) {
    282285            Main.info(crs.getName()+": "+e.getMessage()); // Bursa wolf parameters required.
    283            
    284             if (findSimiliarCrs) { 
     286
     287            if (findSimiliarCrs) {
    285288                List<CoordinateReferenceSystem> candidates = new ArrayList<>();
    286                
     289
    287290                // Find matching CRS with Bursa Wolf parameters in EPSG database
    288291                for (String code : CRS.getAuthorityFactory(false).getAuthorityCodes(ProjectedCRS.class)) {
    … …  
    290293                        CoordinateReferenceSystem candidate = CRS.decode(code);
    291294                        if (candidate instanceof AbstractCRS && crs instanceof AbstractIdentifiedObject) {
    292                            
    293                             Hints.putSystemDefault(Hints.COMPARISON_TOLERANCE,
    294                                     Main.pref.getDouble(OdConstants.PREF_CRS_COMPARISON_TOLERANCE, OdConstants.DEFAULT_CRS_COMPARISON_TOLERANCE));
    295                             if (((AbstractCRS)candidate).equals((AbstractIdentifiedObject)crs, false)) {
     295
     296                            Hints.putSystemDefault(Hints.COMPARISON_TOLERANCE, Main.pref.getDouble(
     297                                    OdConstants.PREF_CRS_COMPARISON_TOLERANCE, OdConstants.DEFAULT_CRS_COMPARISON_TOLERANCE));
     298                            if (((AbstractCRS) candidate).equals((AbstractIdentifiedObject) crs, false)) {
    296299                                Main.info("Found a potential CRS: "+candidate.getName());
    297300                                candidates.add(candidate);
    … …  
    302305                        }
    303306                    } catch (FactoryException ex) {
    304                         // Silently ignore exceptions
    305                     }
    306                 }
    307                
     307                        Main.trace(ex);
     308                    }
     309                }
     310
    308311                if (candidates.size() > 1) {
    309312                    Main.warn("Found several potential CRS: "+Arrays.toString(candidates.toArray()));
    310313                    // TODO: ask user which one to use
    311314                }
    312                
     315
    313316                if (candidates.size() > 0) {
    314317                    CoordinateReferenceSystem newCRS = candidates.get(0);
    … …  
    320323                }
    321324            }
    322            
     325
    323326            if (transform == null) {
    324327                if (handler != null) {

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

    r31655 r32545  
    6262        public MathTransform transform;
    6363        public int dim;
    64         public CrsData(CoordinateReferenceSystem crs, MathTransform transform, int dim) {
     64        CrsData(CoordinateReferenceSystem crs, MathTransform transform, int dim) {
    6565            this.crs = crs;
    6666            this.transform = transform;
    … …  
    7777    }
    7878
    79     public static DataSet parseDataSet(InputStream in, AbstractDataSetHandler handler, ProgressMonitor instance) throws IOException, XMLStreamException {
     79    public static DataSet parseDataSet(InputStream in, AbstractDataSetHandler handler, ProgressMonitor instance)
     80            throws IOException, XMLStreamException {
    8081        InputStreamReader ir = UTFInputStreamReader.create(in, OdConstants.UTF8);
    8182        XMLStreamReader parser = XMLInputFactory.newInstance().createXMLStreamReader(ir);
    … …  
    8788    }
    8889
    89     private final boolean isElement(String element) {
     90    private boolean isElement(String element) {
    9091        return parser.getLocalName().matches("(gml:)?"+element);
    9192    }
    9293
    93     private DataSet parseDoc(ProgressMonitor instance) throws XMLStreamException, GeoCrsException, FactoryException, GeoMathTransformException, MismatchedDimensionException, TransformException {
     94    private DataSet parseDoc(ProgressMonitor instance) throws XMLStreamException, GeoCrsException, FactoryException,
     95    GeoMathTransformException, MismatchedDimensionException, TransformException {
    9496        Component parent = instance != null ? instance.getWindowParent() : Main.parent;
    9597        while (parser.hasNext()) {
    … …  
    147149    }
    148150
    149     private void parseFeatureMember(Component parent) throws XMLStreamException, GeoCrsException, FactoryException, UserCancelException, GeoMathTransformException, MismatchedDimensionException, TransformException {
     151    private void parseFeatureMember(Component parent) throws XMLStreamException, GeoCrsException, FactoryException,
     152    UserCancelException, GeoMathTransformException, MismatchedDimensionException, TransformException {
    150153        List<OsmPrimitive> list = new ArrayList<>();
    151154        Way way = null;

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

    r30723 r32545  
    1414
    1515public class KmlKmzImporter extends AbstractImporter {
    16    
     16
    1717    public static final ExtensionFileFilter KML_KMZ_FILE_FILTER = new ExtensionFileFilter(
    18             OdConstants.KML_EXT+","+OdConstants.KMZ_EXT, OdConstants.KMZ_EXT, tr("KML/KMZ files") + " (*."+OdConstants.KML_EXT+",*."+OdConstants.KMZ_EXT+")");
     18            OdConstants.KML_EXT+","+OdConstants.KMZ_EXT, OdConstants.KMZ_EXT, tr("KML/KMZ files") +
     19            " (*."+OdConstants.KML_EXT+",*."+OdConstants.KMZ_EXT+")");
    1920
    2021    public KmlKmzImporter() {

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

    r32130 r32545  
    3232public class KmlReader extends AbstractReader {
    3333
     34    // CHECKSTYLE.OFF: SingleSpaceSeparator
    3435    public static final String KML_PLACEMARK   = "Placemark";
    3536    public static final String KML_NAME        = "name";
    … …  
    4344    public static final String KML_LINEAR_RING = "LinearRing";
    4445    public static final String KML_COORDINATES = "coordinates";
    45    
     46    // CHECKSTYLE.ON: SingleSpaceSeparator
     47
    4648    public static Pattern COLOR_PATTERN = Pattern.compile("\\p{XDigit}{8}");
    4749
    4850    private XMLStreamReader parser;
    4951    private Map<LatLon, Node> nodes = new HashMap<>();
    50    
     52
    5153    public KmlReader(XMLStreamReader parser) {
    5254        this.parser = parser;
    5355    }
    5456
    55     public static DataSet parseDataSet(InputStream in, ProgressMonitor instance) throws IOException, XMLStreamException, FactoryConfigurationError {
     57    public static DataSet parseDataSet(InputStream in, ProgressMonitor instance)
     58            throws IOException, XMLStreamException, FactoryConfigurationError {
    5659        InputStreamReader ir = UTFInputStreamReader.create(in);
    5760        XMLStreamReader parser = XMLInputFactory.newInstance().createXMLStreamReader(ir);
    … …  
    7174        return ds;
    7275    }
    73    
     76
    7477    private static boolean keyIsIgnored(String key) {
    7578        for (ProjectionPatterns pp : OdConstants.PROJECTIONS) {
    … …  
    8083        return false;
    8184    }
    82    
     85
    8386    private void parsePlaceMark(DataSet ds) throws XMLStreamException {
    8487        List<OsmPrimitive> list = new ArrayList<>();
    … …  
    9598                    if (COLOR_PATTERN.matcher(s).matches()) {
    9699                        // KML color format is aabbggrr, convert it to OSM (web) format: #rrggbb
    97                         tags.put(KML_COLOR, '#'+s.substring(6,8)+s.substring(4,6)+s.substring(2,4));
     100                        tags.put(KML_COLOR, '#'+s.substring(6, 8)+s.substring(4, 6)+s.substring(2, 4));
    98101                    }
    99102                } else if (parser.getLocalName().equals(KML_NAME)) {
    … …  
    122125                } else if (parser.getLocalName().equals(KML_COORDINATES)) {
    123126                    String[] tab = parser.getElementText().trim().split("\\s");
    124                     for (int i = 0; i < tab.length; i ++) {
     127                    for (int i = 0; i < tab.length; i++) {
    125128                        String[] values = tab[i].split(",");
    126129                        if (values.length >= 2) {

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

    r30723 r32545  
    2020
    2121    private ZipInputStream zis;
    22    
     22
    2323    public KmzReader(ZipInputStream zis) {
    2424        this.zis = zis;
    2525    }
    2626
    27     public static DataSet parseDataSet(InputStream in, ProgressMonitor instance) throws IOException, XMLStreamException, FactoryConfigurationError {
     27    public static DataSet parseDataSet(InputStream in, ProgressMonitor instance)
     28            throws IOException, XMLStreamException, FactoryConfigurationError {
    2829        return new KmzReader(new ZipInputStream(in)).parseDoc(instance);
    2930    }
    3031
    31     private DataSet parseDoc(ProgressMonitor instance) throws IOException, XMLStreamException, FactoryConfigurationError  {
     32    private DataSet parseDoc(ProgressMonitor instance) throws IOException, XMLStreamException, FactoryConfigurationError {
    3233        ZipEntry entry;
    3334        do {
    … …  
    5960            buffer = out.toByteArray();
    6061        }
    61        
     62
    6263        return KmlReader.parseDataSet(new ByteArrayInputStream(buffer), instance);
    6364    }

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

    r30730 r32545  
    3131 */
    3232public enum MifDatum {
     33    // CHECKSTYLE.OFF: LineLength
    3334    ADINDAN(1, "Ethiopia, Mali, Senegal, Sudan", CLARKE_1880),
    3435    AFGOOYE(2, "Somalia", KRASSOVSKY),
    … …  
    153154    ZANDERIJ(106, "", INTERNATIONAL),
    154155    CUSTOM(999, null, null);
     156    // CHECKSTYLE.ON: LineLength
    155157
    156158    private final Integer code;
    157159    private final String area;
    158160    private final MifEllipsoid ellipsoid;
    159     private MifDatum(Integer code, String area, MifEllipsoid ellipsoid) {
     161
     162    MifDatum(Integer code, String area, MifEllipsoid ellipsoid) {
    160163        this.code = code;
    161164        this.area = area;
    162165        this.ellipsoid = ellipsoid;
    163166    }
     167
    164168    public final Integer getCode() {
    165169        return code;
    166170    }
     171
    167172    public final String getArea() {
    168173        return area;
    169174    }
     175
    170176    public final MifEllipsoid getEllipsoid() {
    171177        return ellipsoid;
    172178    }
     179
    173180    public static MifDatum forCode(Integer code) {
    174181        for (MifDatum p : values()) {

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

    r30583 r32545  
    66public interface MifHandler extends GeographicHandler {
    77
    8     public void setCoordSysNonEarthProjection(Projection p);
    9    
    10     public Projection getCoordSysNonEarthProjection();
     8    void setCoordSysNonEarthProjection(Projection p);
     9
     10    Projection getCoordSysNonEarthProjection();
    1111}

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

    r30723 r32545  
    3636    Transverse_Mercator_modified_for_Danish_System_45_Bornholm(23, "tmerc"),
    3737    Transverse_Mercator_modified_for_Finnish_KKJ(24, "tmerc");
    38    
     38
    3939    private final Integer code;
    4040    private final String proj4id;
    41    
    42     private MifProjection(Integer code, String proj4id) {
     41
     42    MifProjection(Integer code, String proj4id) {
    4343        this.code = code;
    4444        this.proj4id = proj4id;
    4545    }
    46    
     46
    4747    public final Integer getCode() {
    4848        return code;
    4949    }
    50    
     50
    5151    /**
    5252     * Replies the Proj.4 identifier.
    … …  
    5858        return proj4id;
    5959    }
    60    
     60
    6161    public static MifProjection forCode(Integer code) {
    6262        for (MifProjection p : values()) {

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

    r31655 r32545  
    4444 * These files have been stored in reference directory to avoid future dead links.
    4545 */
    46 public class MifReader extends AbstractMapInfoReader {
     46public final class MifReader extends AbstractMapInfoReader {
    4747
    4848    private enum State {
    … …  
    263263            if ((datum == GEODETIC_REFERENCE_SYSTEM_1980_GRS_80 || datum == CUSTOM) && equals(originLon, 3.0)) {
    264264                // This sounds good for Lambert 93 or Lambert CC 9
    265                 if (equals(originLat, 46.5) && equals(stdP1, 44.0) && equals(stdP2, 49.0) && equals(falseEasting, 700000.0) && equals(falseNorthing, 6600000.0)) {
     265                if (equals(originLat, 46.5) && equals(stdP1, 44.0) && equals(stdP2, 49.0)
     266                        && equals(falseEasting, 700000.0) && equals(falseNorthing, 6600000.0)) {
    266267                    josmProj = Projections.getProjectionByCode("EPSG:2154"); // Lambert 93
    267268                } else if (equals(falseEasting, 1700000.0)) {
    268                     for (int i=0; josmProj == null && i<9; i++) {
    269                         if (equals(originLat, 42.0+i) && equals(stdP1, 41.25+i) && equals(stdP2, 42.75+i) && equals(falseNorthing, (i+1)*1000000.0 + 200000.0)) {
     269                    for (int i = 0; josmProj == null && i < 9; i++) {
     270                        if (equals(originLat, 42.0+i) && equals(stdP1, 41.25+i) && equals(stdP2, 42.75+i)
     271                                && equals(falseNorthing, (i+1)*1000000.0 + 200000.0)) {
    270272                            josmProj = Projections.getProjectionByCode("EPSG:"+Integer.toString(3942 + i)); // LambertCC9Zones
    271273                        }
    … …  
    317319
    318320    private void parseCoordSys(String[] words) {
    319         for (int i = 0; i<words.length; i++) {
     321        for (int i = 0; i < words.length; i++) {
    320322            words[i] = words[i].replace(",", "");
    321323        }
    … …  
    327329            parseCoordSysSyntax2(words);
    328330
     331            // CHECKSTYLE.OFF: LineLength
    329332            // Syntax2 is not meant to be used for maps, and still... # 9592 happened
    330333            // From MapInfo documentation:
    … …  
    334337            // the coordinate system. If the Projection clause is omitted, MapBasic uses a longitude, latitude coordinate system using the North American Datum of 1927 (NAD-27).
    335338            // Use syntax 2 to explicitly define a non-Earth coordinate system, such as the coordinate system used in a floor plan or other CAD drawing.
     339            // CHECKSTYLE.ON: LineLength
    336340
    337341            if (handler != null && handler.getMifHandler() != null && handler.getMifHandler().getCoordSysNonEarthProjection() != null) {
    … …  
    596600                    Main.error("Incoherence between MID and MIF files ("+columns.size()+" columns vs "+fields.length+" fields)");
    597601                }
    598                 for (int i=0; i<Math.min(columns.size(), fields.length); i++) {
     602                for (int i = 0; i < Math.min(columns.size(), fields.length); i++) {
    599603                    String field = fields[i].trim();
    600604                    /*if (field.startsWith("\"") && field.endsWith("\"")) {
    … …  
    609613    }
    610614
    611     protected final Node createNode(String x, String y) {
     615    protected Node createNode(String x, String y) {
    612616        Node node = new Node(josmProj.eastNorth2latlon(new EastNorth(Double.parseDouble(x), Double.parseDouble(y))));
    613617        ds.addPrimitive(node);
    … …  
    617621    /** Compare two doubles within a default epsilon */
    618622    public static boolean equals(Double a, Double b) {
    619         if (a==b) return true;
     623        if (a == b) return true;
    620624        // If the difference is less than epsilon, treat as equal.
    621625        return Math.abs(a - b) < 0.0000001;

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

    r30723 r32545  
    1515
    1616public class MifTabImporter extends AbstractImporter {
    17    
     17
    1818    public static final ExtensionFileFilter MIF_TAB_FILE_FILTER = new ExtensionFileFilter(
    19             OdConstants.MIF_EXT+","+OdConstants.TAB_EXT, OdConstants.MIF_EXT, tr("MapInfo files") + " (*."+OdConstants.MIF_EXT+",*."+OdConstants.TAB_EXT+")");
     19            OdConstants.MIF_EXT+","+OdConstants.TAB_EXT, OdConstants.MIF_EXT,
     20            tr("MapInfo files") + " (*."+OdConstants.MIF_EXT+",*."+OdConstants.TAB_EXT+")");
    2021
    2122    public MifTabImporter() {

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

    r30723 r32545  
    2222    US_Survey_Feet(8),
    2323    Yards(4);
    24        
     24
    2525    private final Integer code;
    26     private MifUnit(Integer code) {
     26
     27    MifUnit(Integer code) {
    2728        this.code = code;
    2829    }
     30
    2931    public final Integer getCode() {
    3032        return code;
    3133    }
     34
    3235    public static MifUnit forCode(Integer code) {
    3336        for (MifUnit p : values()) {

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

    r30723 r32545  
    1010public interface ShpHandler extends GeographicHandler {
    1111
    12     public void notifyFeatureParsed(Object feature, DataSet result, Set<OsmPrimitive> featurePrimitives);
     12    void notifyFeatureParsed(Object feature, DataSet result, Set<OsmPrimitive> featurePrimitives);
    1313
    14     public void setDbfCharset(Charset charset);
    15    
    16     public Charset getDbfCharset();
     14    void setDbfCharset(Charset charset);
     15
     16    Charset getDbfCharset();
    1717}

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

    r32139 r32545  
    8181    }
    8282
    83     private void parseFeature(Feature feature, final Component parent)
    84             throws UserCancelException, GeoMathTransformException, FactoryException, GeoCrsException, MismatchedDimensionException, TransformException {
     83    private void parseFeature(Feature feature, final Component parent) throws UserCancelException, GeoMathTransformException,
     84    FactoryException, GeoCrsException, MismatchedDimensionException, TransformException {
    8585        featurePrimitives.clear();
    8686        GeometryAttribute geometry = feature.getDefaultGeometryProperty();
    … …  
    101101                                    tr("Warning: CRS not found"),
    102102                                    JOptionPane.YES_NO_CANCEL_OPTION
    103                             )) {
     103                                    )) {
    104104                                crs = wgs84;
    105105                            }
    … …  
    131131                    Way w = null;
    132132
    133                     for (int i=0; i<nGeometries; i++) {
     133                    for (int i = 0; i < nGeometries; i++) {
    134134                        Geometry g = mp.getGeometryN(i);
    135135                        if (g instanceof Polygon) {
    … …  
    137137                            // Do not create relation if there's only one polygon without interior ring
    138138                            // except if handler prefers it
    139                             if (r == null && (nGeometries > 1 || p.getNumInteriorRing() > 0 || (handler != null && handler.preferMultipolygonToSimpleWay()))) {
     139                            if (r == null && (nGeometries > 1 || p.getNumInteriorRing() > 0 ||
     140                                    (handler != null && handler.preferMultipolygonToSimpleWay()))) {
    140141                                r = createMultipolygon();
    141142                            }
    … …  
    143144                            if (r != null) {
    144145                                addWayToMp(r, "outer", w);
    145                                 for (int j=0; j<p.getNumInteriorRing(); j++) {
     146                                for (int j = 0; j < p.getNumInteriorRing(); j++) {
    146147                                    addWayToMp(r, "inner", createOrGetWay(p.getInteriorRingN(j)));
    147148                                }
    … …  
    211212                String typeName = typeNames[0];
    212213
    213                 FeatureSource<?,?> featureSource = dataStore.getFeatureSource(typeName);
    214                 FeatureCollection<?,?> collection = featureSource.getFeatures();
     214                FeatureSource<?, ?> featureSource = dataStore.getFeatureSource(typeName);
     215                FeatureCollection<?, ?> collection = featureSource.getFeatures();
    215216                FeatureIterator<?> iterator = collection.features();
    216217
    … …  
    259260    }
    260261
    261     private static final void readNonGeometricAttributes(Feature feature, OsmPrimitive primitive) {
     262    private static void readNonGeometricAttributes(Feature feature, OsmPrimitive primitive) {
    262263        try {
    263264            for (Property prop : feature.getProperties()) {

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

    r30795 r32545  
    3232    private Charset datCharset;
    3333    private final AbstractDataSetHandler handler;
    34    
     34
    3535    public TabReader(AbstractDataSetHandler handler) {
    3636        this.handler = handler;
    … …  
    4141        return new TabReader(handler).parse(in, file, instance, Charset.forName(OdConstants.ISO8859_15));
    4242    }
    43        
     43
    4444    private class TabOsmReader extends SpreadSheetReader {
    4545
    4646        private final DbaseFileReader dbfReader;
    47         public TabOsmReader(SpreadSheetHandler handler, TabFiles tabFiles) throws IOException {
     47        TabOsmReader(SpreadSheetHandler handler, TabFiles tabFiles) throws IOException {
    4848            super(handler);
    4949            this.dbfReader = new DbaseFileReader(tabFiles, false, datCharset, null);
    … …  
    6363            List<String> result = new ArrayList<>();
    6464            Row row = dbfReader.readRow();
    65             for (int i=0; i<columns.size(); i++) {
     65            for (int i = 0; i < columns.size(); i++) {
    6666                Object o = row.read(i);
    6767                if (o != null) {

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

    r30723 r32545  
    66public interface CsvHandler extends SpreadSheetHandler {
    77
    8     public void setCharset(Charset charset);
     8    void setCharset(Charset charset);
    99
    10     public void setCharset(String charset);
     10    void setCharset(String charset);
    1111
    12     public Charset getCharset();
     12    Charset getCharset();
    1313
    14     public void setSeparator(String sep);
    15    
    16     public String getSeparator();
     14    void setSeparator(String sep);
     15
     16    String getSeparator();
    1717}

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

    r30738 r32545  
    1010import java.io.InputStream;
    1111
     12import org.openstreetmap.josm.Main;
    1213import org.openstreetmap.josm.actions.ExtensionFileFilter;
    1314import org.openstreetmap.josm.data.osm.DataSet;
    … …  
    2223            OdConstants.CSV_EXT, OdConstants.CSV_EXT, tr("CSV files") + " (*."+OdConstants.CSV_EXT+")");
    2324
    24     public static final String COLOMBUS_HEADER = "INDEX,TAG,DATE,TIME,LATITUDE N/S,LONGITUDE E/W,HEIGHT,SPEED,HEADING,FIX MODE,VALID,PDOP,HDOP,VDOP,VOX";
     25    public static final String COLOMBUS_HEADER =
     26            "INDEX,TAG,DATE,TIME,LATITUDE N/S,LONGITUDE E/W,HEIGHT,SPEED,HEADING,FIX MODE,VALID,PDOP,HDOP,VDOP,VOX";
    2527
    2628    public CsvImporter() {
    … …  
    5052                result = line != null && line.equalsIgnoreCase(COLOMBUS_HEADER);
    5153            } catch (IOException e) {
    52                 // Ignore exceptions
     54                Main.trace(e);
    5355            }
    5456        }

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

    r30723 r32545  
    1919    private final Charset charset;
    2020    private String sep;
    21    
     21
    2222    private BufferedReader reader;
    2323    private String line;
    … …  
    3232        this.sep = handler != null && handler.getSeparator() != null ? handler.getSeparator() : defaultSep;
    3333    }
    34    
     34
    3535    public static DataSet parseDataSet(InputStream in, AbstractDataSetHandler handler, ProgressMonitor instance) throws IOException {
    3636        CsvHandler csvHandler = null;
    … …  
    6565        return splitLine();
    6666    }
    67    
    68     private final String[] splitLine() {
     67
     68    private String[] splitLine() {
    6969        if (line != null) {
    7070            return OdUtils.stripQuotesAndExtraChars(line.split(sep), sep);

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

    r30723 r32545  
    1212public interface SpreadSheetHandler {
    1313
    14     public void setSheetNumber(int n);
    15    
    16     public int getSheetNumber();
     14    void setSheetNumber(int n);
    1715
    18     public void setHandlesProjection(boolean handle);
    19    
    20     public boolean handlesProjection();
     16    int getSheetNumber();
    2117
    22     public LatLon getCoor(EastNorth en, String[] fields);
     18    void setHandlesProjection(boolean handle);
    2319
    24     public void setLineNumber(int n);
    25    
    26     public int getLineNumber();
     20    boolean handlesProjection();
    2721
    28     public void setXCol(int i);
     22    LatLon getCoor(EastNorth en, String[] fields);
    2923
    30     public void setYCol(int i);
    31    
    32     public int getXCol();
     24    void setLineNumber(int n);
    3325
    34     public int getYCol();
     26    int getLineNumber();
    3527
    36     public void nodesAdded(DataSet ds, Map<CoordinateColumns, Node> nodes, String[] fields, int lineNumber);
     28    void setXCol(int i);
     29
     30    void setYCol(int i);
     31
     32    int getXCol();
     33
     34    int getYCol();
     35
     36    void nodesAdded(DataSet ds, Map<CoordinateColumns, Node> nodes, String[] fields, int lineNumber);
    3737}

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

    r30795 r32545  
    2828
    2929public abstract class SpreadSheetReader extends AbstractReader {
    30    
     30
    3131    private static final NumberFormat formatFrance = NumberFormat.getInstance(Locale.FRANCE);
    3232    private static final NumberFormat formatUK = NumberFormat.getInstance(Locale.UK);
    33    
     33
    3434    protected final SpreadSheetHandler handler;
    3535
    … …  
    3939
    4040    protected static double parseDouble(String value) throws ParseException {
    41         if (value.contains(",")) { 
     41        if (value.contains(",")) {
    4242            return formatFrance.parse(value.replace(" ", "")).doubleValue();
    4343        } else {
    … …  
    4545        }
    4646    }
    47    
     47
    4848    protected abstract void initResources(InputStream in, ProgressMonitor progressMonitor) throws IOException;
    49    
     49
    5050    protected abstract String[] readLine(ProgressMonitor progressMonitor) throws IOException;
    51    
     51
    5252    protected final int getSheetNumber() {
    5353        return handler != null && handler.getSheetNumber() > -1 ? handler.getSheetNumber() : 0;
    5454    }
    55    
     55
    5656    protected final int getLineNumber() {
    5757        return handler != null ? handler.getLineNumber() : -1;
    5858    }
    59    
     59
    6060    public static class CoordinateColumns {
    6161        public Projection proj = null;
    … …  
    6565            return xCol > -1 && yCol > -1;
    6666        }
     67
    6768        @Override
    6869        public String toString() {
    … …  
    7071        }
    7172    }
    72    
    73     private final CoordinateColumns addCoorColIfNeeded(List<CoordinateColumns> columns, CoordinateColumns col) {
     73
     74    private CoordinateColumns addCoorColIfNeeded(List<CoordinateColumns> columns, CoordinateColumns col) {
    7475        if (col == null || col.isOk()) {
    7576            columns.add(col = new CoordinateColumns());
    … …  
    7778        return col;
    7879    }
    79    
     80
    8081    public DataSet doParse(String[] header, ProgressMonitor progressMonitor) throws IOException {
    8182        Main.info("Header: "+Arrays.toString(header));
    82        
     83
    8384        Map<ProjectionPatterns, List<CoordinateColumns>> projColumns = new HashMap<>();
    84        
    85         for (int i = 0; i<header.length; i++) {
     85
     86        for (int i = 0; i < header.length; i++) {
    8687            for (ProjectionPatterns pp : OdConstants.PROJECTIONS) {
    8788                List<CoordinateColumns> columns = projColumns.get(pp);
    … …  
    101102
    102103        final List<CoordinateColumns> columns = new ArrayList<>();
    103        
     104
    104105        for (ProjectionPatterns pp : projColumns.keySet()) {
    105106            for (CoordinateColumns col : projColumns.get(pp)) {
    … …  
    116117
    117118        boolean projFound = false;
    118        
     119
    119120        for (CoordinateColumns c : columns) {
    120121            if (c.proj != null) {
    … …  
    123124            }
    124125        }
    125        
     126
    126127        if (projFound) {
    127128            // projection identified, do nothing
    … …  
    140141                }
    141142            }
    142            
     143
    143144        } else {
    144145            throw new IllegalArgumentException(tr("No valid coordinates have been found."));
    … …  
    152153            message += c.proj + "("+header[c.xCol]+", "+header[c.yCol]+")";
    153154        }
    154        
     155
    155156        Main.info("Loading data using projections "+message);
    156        
     157
    157158        final DataSet ds = new DataSet();
    158159        int lineNumber = 1;
    159        
     160
    160161        String[] fields;
    161162        while ((fields = readLine(progressMonitor)) != null) {
    … …  
    165166                handler.setYCol(-1);
    166167            }
    167            
     168
    168169            final Map<CoordinateColumns, EastNorth> ens = new HashMap<>();
    169170            final Map<CoordinateColumns, Node> nodes = new HashMap<>();
    … …  
    172173                ens.put(c, new EastNorth(Double.NaN, Double.NaN));
    173174            }
    174            
     175
    175176            if (fields.length > header.length) {
    176177                Main.warn(tr("Invalid file. Bad length on line {0}. Expected {1} columns, got {2}.", lineNumber, header.length, fields.length));
    177178                Main.warn(Arrays.toString(fields));
    178179            }
    179            
    180             for (int i = 0; i<Math.min(fields.length, header.length); i++) {
     180
     181            for (int i = 0; i < Math.min(fields.length, header.length); i++) {
    181182                try {
    182183                    boolean coordinate = false;
    … …  
    195196                                handler.setYCol(i);
    196197                            }
    197                         }                           
     198                        }
    198199                    }
    199200                    if (!coordinate) {
    … …  
    219220                        firstNode = n;
    220221                    }
    221                     if (n == firstNode || n.getCoor().greatCircleDistance(firstNode.getCoor()) > Main.pref.getDouble(OdConstants.PREF_TOLERANCE, OdConstants.DEFAULT_TOLERANCE)) {
     222                    if (n == firstNode || n.getCoor().greatCircleDistance(firstNode.getCoor())
     223                            > Main.pref.getDouble(OdConstants.PREF_TOLERANCE, OdConstants.DEFAULT_TOLERANCE)) {
    222224                        ds.addPrimitive(n);
    223225                    } else {
    … …  
    230232            }
    231233        }
    232        
     234
    233235        return ds;
    234236    }
    235    
     237
    236238    public final DataSet parse(InputStream in, ProgressMonitor progressMonitor) throws IOException {
    237        
     239
    238240        initResources(in, progressMonitor);
    239        
     241
    240242        String[] header = null;
    241243        int length = 0;
    242244        int n = 0;
    243        
     245
    244246        while (header == null || length == 0) {
    245247            n++;
    … …  
    254256            }
    255257        }
    256        
     258
    257259        return doParse(header, progressMonitor);
    258260    }

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

    r32525 r32545  
    3636    public OdDiffLayer diffLayer;
    3737    public OdOsmDataLayer osmLayer;
    38    
     38
    3939    public final AbstractDataSetHandler handler;
    40    
     40
    4141    private Bounds bounds;
    42    
     42
    4343    public OdDataLayer(DataSet data, String name, File associatedFile, AbstractDataSetHandler handler) {
    4444        super(data, name, associatedFile);
    … …  
    5757        Main.getLayerManager().addLayerChangeListener(this);
    5858    }
    59    
     59
    6060    @Override public ImageProvider getBaseIconProvider() {
    6161        return OdUtils.getImageProvider(handler != null ? handler.getDataLayerIconName() : OdConstants.ICON_CORE_16);
    … …  
    7474        }
    7575    }
    76    
     76
    7777    public void addDiffLayer(OdDiffLayer layer) {
    7878        removeDiffLayer();
    … …  
    8080        Main.getLayerManager().addLayer(diffLayer);
    8181    }
    82    
     82
    8383    public void removeDiffLayer() {
    8484        if (diffLayer != null) {
    … …  
    8787        }
    8888    }
    89    
     89
    9090    public final void downloadOsmData() {
    9191        String oapiReq = handler.getOverpassApiRequest(bounds);
    … …  
    140140        }
    141141        if (this.handler != null) {
    142             if (this.handler.getWikiURL() != null || this.handler.getLocalPortalURL() != null || this.handler.getNationalPortalURL() != null) { 
     142            if (this.handler.getWikiURL() != null || this.handler.getLocalPortalURL() != null || this.handler.getNationalPortalURL() != null) {
    143143                result.add(SeparatorLayerAction.INSTANCE);
    144144            }
    145             if (this.handler.getWikiURL() != null) { 
    146                 result.add(new OpenLinkAction(this.handler.getWikiURL(), OdConstants.ICON_OSM_24, 
     145            if (this.handler.getWikiURL() != null) {
     146                result.add(new OpenLinkAction(this.handler.getWikiURL(), OdConstants.ICON_OSM_24,
    147147                        tr("View OSM Wiki page"), tr("Launch browser to the OSM Wiki page of the selected data set")));
    148148            }
    149             if (this.handler.getLocalPortalURL() != null) { 
    150                 result.add(new OpenLinkAction(this.handler.getLocalPortalURL(), this.handler.getLocalPortalIconName(), 
     149            if (this.handler.getLocalPortalURL() != null) {
     150                result.add(new OpenLinkAction(this.handler.getLocalPortalURL(), this.handler.getLocalPortalIconName(),
    151151                        tr("View Local Portal page"), tr("Launch browser to the local portal page of the selected data set")));
    152152            }
    153             if (this.handler.getNationalPortalURL() != null) { 
    154                 result.add(new OpenLinkAction(this.handler.getNationalPortalURL(), this.handler.getNationalPortalIconName(), 
     153            if (this.handler.getNationalPortalURL() != null) {
     154                result.add(new OpenLinkAction(this.handler.getNationalPortalURL(), this.handler.getNationalPortalIconName(),
    155155                        tr("View National Portal page"), tr("Launch browser to the national portal page of the selected data set")));
    156156            }
    … …  
    158158                License lic = this.handler.getLicense();
    159159                if (lic.getURL() != null && lic.getURL().getProtocol().startsWith("http")) {
    160                     result.add(new OpenLinkAction(lic.getURL(), OdConstants.ICON_AGREEMENT_24, 
     160                    result.add(new OpenLinkAction(lic.getURL(), OdConstants.ICON_AGREEMENT_24,
    161161                            tr("View License"), tr("Launch browser to the license page of the selected data set")));
    162162                }
    163163                if (lic.getSummaryURL() != null && lic.getSummaryURL().getProtocol().startsWith("http")) {
    164                     result.add(new OpenLinkAction(lic.getSummaryURL(), OdConstants.ICON_AGREEMENT_24, 
     164                    result.add(new OpenLinkAction(lic.getSummaryURL(), OdConstants.ICON_AGREEMENT_24,
    165165                            tr("View License (summary)"), tr("Launch browser to the summary license page of the selected data set")));
    166166                }
    167                 if ((lic.getURL() != null && !lic.getURL().getProtocol().startsWith("http")) || (lic.getSummaryURL() != null && !lic.getSummaryURL().getProtocol().startsWith("http"))) {
     167                if ((lic.getURL() != null && !lic.getURL().getProtocol().startsWith("http")) ||
     168                        (lic.getSummaryURL() != null && !lic.getSummaryURL().getProtocol().startsWith("http"))) {
    168169                    result.add(new ViewLicenseAction(lic, tr("View License"), tr("View the license of the selected data set")));
    169170                }

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

    r30723 r32545  
    2323
    2424    private final OdDataLayer dataLayer;
    25    
    26     public final List<Pair<OsmPrimitive,OsmPrimitive>> differentPrimitives;
     25
     26    public final List<Pair<OsmPrimitive, OsmPrimitive>> differentPrimitives;
    2727    public final List<OsmPrimitive> onlyInTlsPrimitives;
    2828    public final List<OsmPrimitive> onlyInOsmPrimitives;
    29    
     29
    3030    public OdDiffLayer(OdDataLayer dataLayer, String name) {
    3131        super(name);
    … …  
    3636        initDiff(dataLayer.data, dataLayer.osmLayer.data);
    3737    }
    38    
     38
    3939    private void initDiff(DataSet tlsData, DataSet osmData) {
    4040        for (OsmPrimitive p1 : tlsData.allPrimitives()) {
    … …  
    5656        }
    5757    }
    58    
     58
    5959    private double distance(OsmPrimitive p1, OsmPrimitive p2) {
    6060        return p1.getBBox().getCenter().greatCircleDistance(p2.getBBox().getCenter());
    6161    }
    62    
     62
    6363    private OsmPrimitive findPrimitiveAt(DataSet dataSet, OsmPrimitive source) {
    6464        double maxDistance = Main.pref.getDouble(OdConstants.PREF_MAXDISTANCE, OdConstants.DEFAULT_MAXDISTANCE);
    … …  
    6969        for (OsmPrimitive p : dataSet.allPrimitives()) {
    7070            if (dataLayer.handler.isRelevant(p)) {
    71                 double dist = distance(source, p); 
     71                double dist = distance(source, p);
    7272                if (dist <= maxDistance) {
    7373                    if (dataLayer.handler.equals(p, source)) {

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

    r30723 r32545  
    33
    44public interface OdLayer {
    5     public OdDataLayer getDataLayer();
     5    OdDataLayer getDataLayer();
    66}

  • applications/editors/josm/plugins/opendata/src/org/openstreetmap/josm/plugins/opendata/core/licenses/License.java

    r30723 r32545  
    1313
    1414public abstract class License {
    15    
     15
    1616    public static final ODbL ODbL = new ODbL();
    1717    public static final LOOL LOOL = new LOOL();
    18    
     18
    1919    private final Map<String, URL> urls = new HashMap<>();
    2020    private final Map<String, URL> summaryURLs = new HashMap<>();
    21    
     21
    2222    private Icon icon;
    23    
    24     private static final URL getURL(Map<String, URL> map) {
     23
     24    private static URL getURL(Map<String, URL> map) {
    2525        // Find URL for current language
    2626        String lang = OdUtils.getJosmLanguage();
    … …  
    3737        // If not found, return first non-null url
    3838        if (map.keySet().size() > 0) {
    39             for (Iterator<String> it=map.keySet().iterator(); it.hasNext(); ) {
     39            for (Iterator<String> it = map.keySet().iterator(); it.hasNext();) {
    4040                url = map.get(it.next());
    4141                if (url != null) {
    … …  
    5555        return getURL(summaryURLs);
    5656    }
    57    
     57
    5858    public final void setURL(URL url) {
    5959        setURL(url, "en");

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

    r30731 r32545  
    115115    }
    116116
    117     private final List<AbstractDataSetHandler> getInstanciatedHandlers() {
     117    private List<AbstractDataSetHandler> getInstanciatedHandlers() {
    118118        if (instanciatedHandlers.isEmpty()) {
    119119            instanciatedHandlers.addAll(getNewlyInstanciatedHandlers());

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

    r30723 r32545  
    99public interface Module {
    1010
    11     public String getDisplayedName();
     11    String getDisplayedName();
    1212
    13     public List<Class<? extends AbstractDataSetHandler>> getHandlers();
     13    List<Class<? extends AbstractDataSetHandler>> getHandlers();
    1414
    15     public List<AbstractDataSetHandler> getNewlyInstanciatedHandlers();
     15    List<AbstractDataSetHandler> getNewlyInstanciatedHandlers();
    1616
    17     public SourceProvider getMapPaintStyleSourceProvider();
    18    
    19     public SourceProvider getPresetSourceProvider();
    20    
    21     public ModuleInformation getModuleInformation();
     17    SourceProvider getMapPaintStyleSourceProvider();
     18
     19    SourceProvider getPresetSourceProvider();
     20
     21    ModuleInformation getModuleInformation();
    2222}

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

    r30795 r32545  
    3232 *
    3333 */
    34 public class ModuleDownloadTask extends PleaseWaitRunnable{
     34public class ModuleDownloadTask extends PleaseWaitRunnable {
    3535    private final Collection<ModuleInformation> toUpdate = new LinkedList<>();
    3636    private final Collection<ModuleInformation> failed = new LinkedList<>();
    … …  
    4848     * @throws IllegalArgumentException thrown if toUpdate is null
    4949     */
    50     public ModuleDownloadTask(Component parent, Collection<ModuleInformation> toUpdate, String title) throws IllegalArgumentException{
     50    public ModuleDownloadTask(Component parent, Collection<ModuleInformation> toUpdate, String title) throws IllegalArgumentException {
    5151        super(parent, title == null ? "" : title, false /* don't ignore exceptions */);
    5252        CheckParameterUtil.ensureParameterNotNull(toUpdate, "toUpdate");
    … …  
    6363     */
    6464    public ModuleDownloadTask(ProgressMonitor monitor, Collection<ModuleInformation> toUpdate, String title) {
    65         super(title, monitor == null? NullProgressMonitor.INSTANCE: monitor, false /* don't ignore exceptions */);
     65        super(title, monitor == null ? NullProgressMonitor.INSTANCE : monitor, false /* don't ignore exceptions */);
    6666        CheckParameterUtil.ensureParameterNotNull(toUpdate, "toUpdate");
    6767        this.toUpdate.addAll(toUpdate);
    … …  
    7474     * @throws IllegalArgumentException thrown if toUpdate is null
    7575     */
    76     public void setModulesToDownload(Collection<ModuleInformation> toUpdate) throws IllegalArgumentException{
     76    public void setModulesToDownload(Collection<ModuleInformation> toUpdate) throws IllegalArgumentException {
    7777        CheckParameterUtil.ensureParameterNotNull(toUpdate, "toUpdate");
    7878        this.toUpdate.clear();
    … …  
    8282    @Override protected void cancel() {
    8383        this.canceled = true;
    84         synchronized(this) {
     84        synchronized (this) {
    8585            if (downloadConnection != null) {
    8686                downloadConnection.disconnect();
    … …  
    9191    @Override protected void finish() {}
    9292
    93     protected void download(ModuleInformation pi, File file) throws ModuleDownloadException{
     93    protected void download(ModuleInformation pi, File file) throws ModuleDownloadException {
    9494        try {
    9595            if (pi.downloadlink == null) {
    … …  
    9999            }
    100100            URL url = new URL(pi.downloadlink);
    101             synchronized(this) {
    102                 downloadConnection = (HttpURLConnection)url.openConnection();
     101            synchronized (this) {
     102                downloadConnection = (HttpURLConnection) url.openConnection();
    103103                downloadConnection.setRequestProperty("Cache-Control", "no-cache");
    104                 downloadConnection.setRequestProperty("User-Agent",Version.getInstance().getAgentString());
     104                downloadConnection.setRequestProperty("User-Agent", Version.getInstance().getAgentString());
    105105                downloadConnection.setRequestProperty("Host", url.getHost());
    106106                downloadConnection.connect();
    107107            }
    108108            try (
    109                 InputStream in = downloadConnection.getInputStream();
    110                 OutputStream out = new FileOutputStream(file)
    111             ) {
     109                    InputStream in = downloadConnection.getInputStream();
     110                    OutputStream out = new FileOutputStream(file)
     111                    ) {
    112112                byte[] buffer = new byte[8192];
    113113                for (int read = in.read(buffer); read != -1; read = in.read(buffer)) {
    … …  
    115115                }
    116116            }
    117         } catch(MalformedURLException e) {
    118             String msg = tr("Warning: Cannot download module ''{0}''. Its download link ''{1}'' is not a valid URL. Skipping download.", pi.name, pi.downloadlink);
     117        } catch (MalformedURLException e) {
     118            String msg = tr("Warning: Cannot download module ''{0}''. Its download link ''{1}'' is not a valid URL. Skipping download.",
     119                    pi.name, pi.downloadlink);
    119120            Main.warn(msg);
    120121            throw new ModuleDownloadException(msg);
    … …  
    124125            throw new ModuleDownloadException(e);
    125126        } finally {
    126             synchronized(this) {
     127            synchronized (this) {
    127128                downloadConnection = null;
    128129            }
    … …  
    147148            try {
    148149                download(d, moduleFile);
    149             } catch(ModuleDownloadException e) {
     150            } catch (ModuleDownloadException e) {
    150151                e.printStackTrace();
    151152                failed.add(d);
    … …  
    160161     * Replies true if the task was canceled by the user
    161162     *
    162      * @return
     163     * @return {@code true} if the task was canceled by the user
    163164     */
    164165    public boolean isCanceled() {

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

    r31008 r32545  
    5656 *
    5757 */
    58 public class ModuleHandler {
     58public final class ModuleHandler {
    5959
    6060    /**
    6161     * All installed and loaded modules (resp. their main classes)
    6262     */
    63     public final static Collection<Module> moduleList = new LinkedList<>();
     63    public static final Collection<Module> moduleList = new LinkedList<>();
    6464
    6565    /**
    … …  
    7777    }
    7878
     79    private ModuleHandler() {
     80        // Hide default constructor for utilities classes
     81    }
     82
    7983    public static Collection<ClassLoader> getResourceClassLoaders() {
    8084        return Collections.unmodifiableCollection(sources);
    … …  
    97101        long last = Main.pref.getLong("opendata.modulemanager.lastupdate", 0);
    98102        Integer maxTime = Main.pref.getInteger("opendata.modulemanager.time-based-update.interval", 60);
    99         long d = (tim - last) / (24 * 60 * 60 * 1000l);
     103        long d = (tim - last) / (24 * 60 * 60 * 1000L);
    100104        if ((last <= 0) || (maxTime <= 0)) {
    101105            Main.pref.put("opendata.modulemanager.lastupdate", Long.toString(tim));
    102106        } else if (d > maxTime) {
    103107            message =
    104                 "<html>"
    105                 + tr("Last module update more than {0} days ago.", d)
    106                 + "</html>";
     108                    "<html>"
     109                            + tr("Last module update more than {0} days ago.", d)
     110                            + "</html>";
    107111            togglePreferenceKey = "opendata.modulemanager.time-based-update.policy";
    108112        }
    109113        if (message == null) return false;
    110114
    111         ButtonSpec [] options = new ButtonSpec[] {
     115        ButtonSpec[] options = new ButtonSpec[] {
    112116                new ButtonSpec(
    113117                        tr("Update modules"),
    … …  
    115119                        tr("Click to update the activated modules"),
    116120                        null /* no specific help context */
    117                 ),
     121                        ),
    118122                new ButtonSpec(
    119123                        tr("Skip update"),
    … …  
    121125                        tr("Click to skip updating the activated modules"),
    122126                        null /* no specific help context */
    123                 )
     127                        )
    124128        };
    125129
    … …  
    158162                options[0],
    159163                null
    160         );
     164                );
    161165
    162166        if (pnlMessage.isRememberDecision()) {
    … …  
    247251                        + "Delete from preferences?</html>", module.name, module.className);
    248252            }
    249         }  catch (Exception e) {
     253        } catch (Exception e) {
    250254            Main.error(e);
    251255        }
    … …  
    309313            try {
    310314                future.get();
    311             } catch(ExecutionException | InterruptedException e) {
     315            } catch (ExecutionException | InterruptedException e) {
    312316                Main.error(e);
    313317                return null;
    … …  
    344348                JOptionPane.WARNING_MESSAGE,
    345349                HelpUtil.ht("/Module/Loading#MissingModuleInfos")
    346         );
     350                );
    347351    }
    348352
    … …  
    356360    public static List<ModuleInformation> buildListOfModulesToLoad(Component parent) {
    357361        Set<String> modules = new HashSet<>();
    358         modules.addAll(Main.pref.getCollection(OdConstants.PREF_MODULES,  new LinkedList<String>()));
     362        modules.addAll(Main.pref.getCollection(OdConstants.PREF_MODULES, new LinkedList<String>()));
    359363        if (System.getProperty("josm."+OdConstants.PREF_MODULES) != null) {
    360364            modules.addAll(Arrays.asList(System.getProperty("josm."+OdConstants.PREF_MODULES).split(",")));
    … …  
    382386                "Updating the following modules has failed:",
    383387                modules.size()
    384         )
    385         );
     388                )
     389                );
    386390        sb.append("<ul>");
    387391        for (ModuleInformation pi: modules) {
    … …  
    393397                "Please open the Preference Dialog after JOSM has started and try to update them manually.",
    394398                modules.size()
    395         ));
     399                ));
    396400        sb.append("</html>");
    397401        HelpAwareOptionPane.showOptionDialog(
    … …  
    401405                JOptionPane.ERROR_MESSAGE,
    402406                HelpUtil.ht("/Module/Loading#FailedModuleUpdated")
    403         );
     407                );
    404408    }
    405409
    … …  
    412416     * @throws IllegalArgumentException thrown if modules is null
    413417     */
    414     public static List<ModuleInformation>  updateModules(Component parent,
     418    public static List<ModuleInformation> updateModules(Component parent,
    415419            List<ModuleInformation> modules, ProgressMonitor monitor)
    416             throws IllegalArgumentException{
     420                    throws IllegalArgumentException {
    417421        CheckParameterUtil.ensureParameterNotNull(modules, "modules");
    418422        if (monitor == null) {
    … …  
    426430            //
    427431            ReadRemoteModuleInformationTask task1 = new ReadRemoteModuleInformationTask(
    428                     monitor.createSubTaskMonitor(1,false),
     432                    monitor.createSubTaskMonitor(1, false),
    429433                    OdPreferenceSetting.getModuleSites()
    430             );
     434                    );
    431435            Future<?> future = service.submit(task1);
    432436            try {
    433437                future.get();
    434438                modules = buildListOfModulesToLoad(parent);
    435             } catch(ExecutionException e) {
     439            } catch (ExecutionException e) {
    436440                Main.warn(tr("Warning: failed to download module information list"));
    437441                e.printStackTrace();
    438442                // don't abort in case of error, continue with downloading modules below
    439             } catch(InterruptedException e) {
     443            } catch (InterruptedException e) {
    440444                Main.warn(tr("Warning: failed to download module information list"));
    441445                e.printStackTrace();
    … …  
    446450            //
    447451            Collection<ModuleInformation> modulesToUpdate = new ArrayList<>();
    448             for(ModuleInformation pi: modules) {
     452            for (ModuleInformation pi: modules) {
    449453                if (pi.isUpdateRequired()) {
    450454                    modulesToUpdate.add(pi);
    … …  
    456460                //
    457461                ModuleDownloadTask task2 = new ModuleDownloadTask(
    458                         monitor.createSubTaskMonitor(1,false),
     462                        monitor.createSubTaskMonitor(1, false),
    459463                        modulesToUpdate,
    460464                        tr("Update modules")
    461                 );
     465                        );
    462466
    463467                future = service.submit(task2);
    464468                try {
    465469                    future.get();
    466                 } catch(ExecutionException e) {
     470                } catch (ExecutionException e) {
    467471                    e.printStackTrace();
    468472                    alertFailedModuleUpdate(parent, modulesToUpdate);
    469473                    return modules;
    470                 } catch(InterruptedException e) {
     474                } catch (InterruptedException e) {
    471475                    e.printStackTrace();
    472476                    alertFailedModuleUpdate(parent, modulesToUpdate);
    … …  
    475479                // notify user if downloading a locally installed module failed
    476480                //
    477                 if (! task2.getFailedModules().isEmpty()) {
     481                if (!task2.getFailedModules().isEmpty()) {
    478482                    alertFailedModuleUpdate(parent, task2.getFailedModules());
    479483                    return modules;
    … …  
    497501     */
    498502    public static boolean confirmDisableModule(Component parent, String reason, String name) {
    499         ButtonSpec [] options = new ButtonSpec[] {
     503        ButtonSpec[] options = new ButtonSpec[] {
    500504                new ButtonSpec(
    501505                        tr("Disable module"),
    … …  
    503507                        tr("Click to delete the module ''{0}''", name),
    504508                        null /* no specific help context */
    505                 ),
     509                        ),
    506510                new ButtonSpec(
    507511                        tr("Keep module"),
    … …  
    509513                        tr("Click to keep the module ''{0}''", name),
    510514                        null /* no specific help context */
    511                 )
     515                        )
    512516        };
    513517        int ret = HelpAwareOptionPane.showOptionDialog(
    … …  
    520524                options[0],
    521525                null // FIXME: add help topic
    522         );
     526                );
    523527        return ret == 0;
    524528    }
    … …  
    543547    public static void installDownloadedModules(boolean dowarn) {
    544548        File moduleDir = OdPlugin.getInstance().getModulesDirectory();
    545         if (! moduleDir.exists() || ! moduleDir.isDirectory() || ! moduleDir.canWrite())
     549        if (!moduleDir.exists() || !moduleDir.isDirectory() || !moduleDir.canWrite())
    546550            return;
    547551
    … …  
    550554            public boolean accept(File dir, String name) {
    551555                return name.endsWith(".jar.new");
    552             }});
     556            } });
    553557
    554558        for (File updatedModule : files) {
    … …  
    556560            File module = new File(filePath.substring(0, filePath.length() - 4));
    557561            String moduleName = updatedModule.getName().substring(0, updatedModule.getName().length() - 8);
     562            // CHECKSTYLE.OFF: LineLength
    558563            if (module.exists()) {
    559564                if (!module.delete() && dowarn) {
    … …  
    567572                Main.warn(tr("Warning: failed to install already downloaded module ''{0}''. Skipping installation. JOSM is still going to load the old module version.", moduleName));
    568573            }
     574            // CHECKSTYLE.ON: LineLength
    569575        }
    570576        return;
    … …  
    736742    }*/
    737743
    738     static private class UpdateModulesMessagePanel extends JPanel {
     744    private static class UpdateModulesMessagePanel extends JPanel {
    739745        private JMultilineLabel lblMessage;
    740746        private JCheckBox cbDontShowAgain;
    … …  
    747753            gc.weightx = 1.0;
    748754            gc.weighty = 1.0;
    749             gc.insets = new Insets(5,5,5,5);
     755            gc.insets = new Insets(5, 5, 5, 5);
    750756            add(lblMessage = new JMultilineLabel(""), gc);
    751757            lblMessage.setFont(lblMessage.getFont().deriveFont(Font.PLAIN));
    … …  
    754760            gc.fill = GridBagConstraints.HORIZONTAL;
    755761            gc.weighty = 0.0;
    756             add(cbDontShowAgain = new JCheckBox(tr("Do not ask again and remember my decision (go to Preferences->Modules to change it later)")), gc);
     762            add(cbDontShowAgain = new JCheckBox(
     763                    tr("Do not ask again and remember my decision (go to Preferences->Modules to change it later)")), gc);
    757764            cbDontShowAgain.setFont(cbDontShowAgain.getFont().deriveFont(Font.PLAIN));
    758765        }
    759766
    760         public UpdateModulesMessagePanel() {
     767        UpdateModulesMessagePanel() {
    761768            build();
    762769        }
    … …  
    769776            String policy = Main.pref.get(preferencesKey, "ask");
    770777            policy = policy.trim().toLowerCase();
    771             cbDontShowAgain.setSelected(! policy.equals("ask"));
     778            cbDontShowAgain.setSelected(!policy.equals("ask"));
    772779        }
    773780

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

    r32071 r32545  
    7272     * @throws ModuleException thrown if reading the manifest file fails
    7373     */
    74     public ModuleInformation(File file, String name) throws ModuleException{
     74    public ModuleInformation(File file, String name) throws ModuleException {
    7575        this.name = name;
    7676        this.file = file;
    … …  
    103103            Manifest manifest = new Manifest();
    104104            manifest.read(manifestStream);
    105             if(url != null) {
     105            if (url != null) {
    106106                downloadlink = url;
    107107            }
    … …  
    133133    }
    134134
    135     private static final ImageIcon extractIcon(String iconPath, File jarFile, boolean suppressWarnings) {
    136         return new ImageProvider(iconPath).setArchive(jarFile).setMaxWidth(24).setMaxHeight(24).setOptional(true).setSuppressWarnings(suppressWarnings).get();
     135    private static ImageIcon extractIcon(String iconPath, File jarFile, boolean suppressWarnings) {
     136        return new ImageProvider(iconPath).setArchive(jarFile).setMaxWidth(24).setMaxHeight(24).setOptional(true)
     137                .setSuppressWarnings(suppressWarnings).get();
    137138    }
    138139
    … …  
    156157        link = s;
    157158        s = attr.getValue(lang+"Module-Description");
    158         if(s == null) {
     159        if (s == null) {
    159160            s = attr.getValue("Module-Description");
    160             if(s != null) {
     161            if (s != null) {
    161162                s = tr(s);
    162163            }
    … …  
    240241        if (className == null)
    241242            return null;
    242         try{
     243        try {
    243244            return (Class<? extends Module>) Class.forName(className, true, classLoader);
    244245        } catch (Exception t) {
    … …  
    276277    public boolean isUpdateRequired(String referenceVersion) {
    277278        if (this.downloadlink == null) return false;
    278         if (this.version == null && referenceVersion!= null)
     279        if (this.version == null && referenceVersion != null)
    279280            return true;
    280281        if (this.version != null && !this.version.equals(referenceVersion))
    … …  
    311312    public boolean matches(String filter) {
    312313        if (filter == null) return true;
    313         String words[] = filter.split("\\s+");
     314        String[] words = filter.split("\\s+");
    314315        for (String word: words) {
    315316            if (matches(word, name)

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

    r30532 r32545  
    1616 * A parser for the module list provided by an opendata Module Download Site.
    1717 *
    18  * See <a href="http://svn.openstreetmap.org/applications/editors/josm/plugins/opendata/modules.txt">http://svn.openstreetmap.org/applications/editors/josm/plugins/opendata/modules.txt</a>
     18 * See <a href="http://svn.openstreetmap.org/applications/editors/josm/plugins/opendata/modules.txt">OSM SVN</a>
    1919 * for a sample of the document. The format is a custom format, kind of mix of CSV and RFC822 style
    2020 * name/value-pairs.
    … …  
    3030     * @param manifest the module manifest
    3131     * @return a module information object
    32      * @throws ModuleListParseException
     32     * @throws ModuleListParseException if the list cannot be parsed
    3333     */
    34     protected ModuleInformation createInfo(String name, String url, String manifest) throws ModuleListParseException{
     34    protected ModuleInformation createInfo(String name, String url, String manifest) throws ModuleListParseException {
    3535        try {
    3636            return new ModuleInformation(
    … …  
    3838                    name.substring(0, name.length() - 4),
    3939                    url
    40             );
    41         } catch(UnsupportedEncodingException e) {
     40                    );
     41        } catch (UnsupportedEncodingException e) {
    4242            throw new ModuleListParseException(tr("Failed to create module information from manifest for module ''{0}''", name), e);
    4343        } catch (ModuleException e) {
    … …  
    4949     * Parses a module information document and replies a list of module information objects.
    5050     *
    51      * See <a href="http://svn.openstreetmap.org/applications/editors/josm/plugins/opendata/modules.txt">http://svn.openstreetmap.org/applications/editors/josm/plugins/opendata/modules.txt</a>
     51     * See <a href="http://svn.openstreetmap.org/applications/editors/josm/plugins/opendata/modules.txt">OSM SVN</a>
    5252     * for a sample of the document. The format is a custom format, kind of mix of CSV and RFC822 style
    5353     * name/value-pairs.
    … …  
    5757     * @throws ModuleListParseException thrown if something goes wrong while parsing
    5858     */
    59     public List<ModuleInformation> parse(InputStream in) throws ModuleListParseException{
     59    public List<ModuleInformation> parse(InputStream in) throws ModuleListParseException {
    6060        List<ModuleInformation> ret = new LinkedList<>();
    6161        BufferedReader r = null;
    … …  
    7373                }
    7474                B;
    75             */
     75             */
    7676            for (String line = r.readLine(); line != null; line = r.readLine()) {
    7777                if (line.startsWith("\t")) {
    … …  
    9595                    }
    9696                }
    97                 String x[] = line.split(";");
    98                 if(x.length != 2)
    99                   throw new IOException(tr("Illegal entry in module list."));
     97                String[] x = line.split(";");
     98                if (x.length != 2)
     99                    throw new IOException(tr("Illegal entry in module list."));
    100100                name = x[0];
    101101                url = x[1];

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

    r30795 r32545  
    3131 *   <li>.jar.new-files, assuming that these are downloaded but not yet installed modules</li>
    3232 *   <li>cached lists of available modules, downloaded for instance from
    33  *   <a href="http://svn.openstreetmap.org/applications/editors/josm/plugins/opendata/modules.txt">http://svn.openstreetmap.org/applications/editors/josm/plugins/opendata/modules.txt</a></li>
     33 *   <a href="http://svn.openstreetmap.org/applications/editors/josm/plugins/opendata/modules.txt">OSM SVN</a></li>
    3434 * </ul>
    35  *
    3635 */
    3736public class ReadLocalModuleInformationTask extends PleaseWaitRunnable {
    … …  
    4544
    4645    public ReadLocalModuleInformationTask(ProgressMonitor monitor) {
    47         super(tr("Reading local module information.."),monitor, false);
     46        super(tr("Reading local module information.."), monitor, false);
    4847        availableModules = new HashMap<>();
    4948    }
    … …  
    5756    protected void finish() {}
    5857
    59     protected void processJarFile(File f, String moduleName) throws ModuleException{
     58    protected void processJarFile(File f, String moduleName) throws ModuleException {
    6059        ModuleInformation info = new ModuleInformation(
    6160                f,
    6261                moduleName
    63         );
     62                );
    6463        if (!availableModules.containsKey(info.getName())) {
    6564            info.localversion = info.version;
    … …  
    8483                    }
    8584                }
    86         );
     85                );
    8786        if (siteCacheFiles == null || siteCacheFiles.length == 0)
    8887            return;
    … …  
    110109                    }
    111110                }
    112         );
     111                );
    113112        if (siteCacheFiles == null || siteCacheFiles.length == 0)
    114113            return;
    … …  
    121120                if (pi.icon == null && pi.iconPath != null) {
    122121                    pi.icon = new ImageProvider(pi.name+".jar/"+pi.iconPath)
    123                                     .setArchive(f)
    124                                     .setMaxWidth(24)
    125                                     .setMaxHeight(24)
    126                                     .setOptional(true).get();
     122                            .setArchive(f)
     123                            .setMaxWidth(24)
     124                            .setMaxHeight(24)
     125                            .setOptional(true).get();
    127126                }
    128127            }
    … …  
    139138                    }
    140139                }
    141         );
     140                );
    142141        if (moduleFiles == null || moduleFiles.length == 0)
    143142            return;
    … …  
    155154                    processJarFile(f, moduleName);
    156155                }
    157             } catch(ModuleException e){
     156            } catch (ModuleException e) {
    158157                Main.warn(tr("Warning: Failed to scan file ''{0}'' for module information. Skipping.", fname));
    159158                e.printStackTrace();
    … …  
    176175    }
    177176
    178     protected void processLocalModuleInformationFile(File file) throws ModuleListParseException{
     177    protected void processLocalModuleInformationFile(File file) throws ModuleListParseException {
    179178        try (FileInputStream fin = new FileInputStream(file)) {
    180179            List<ModuleInformation> pis = new ModuleListParser().parse(fin);
    … …  
    186185                availableModules.put(pi.name, pi);
    187186            }
    188         } catch(IOException e) {
     187        } catch (IOException e) {
    189188            throw new ModuleListParseException(e);
    190189        }
    … …  
    194193        for (Module module : ModuleHandler.moduleList) {
    195194            ModuleInformation info = module.getModuleInformation();
    196             if (canceled)return;
     195            if (canceled) return;
    197196            if (!availableModules.containsKey(info.name)) {
    198197                availableModules.put(info.name, info);
    … …  
    212211                    getProgressMonitor().createSubTaskMonitor(1, false),
    213212                    new File(location)
    214             );
     213                    );
    215214            getProgressMonitor().worked(1);
    216             if (canceled)return;
     215            if (canceled) return;
    217216        }
    218217        analyseInProcessModules();
    219218        getProgressMonitor().worked(1);
    220         if (canceled)return;
     219        if (canceled) return;
    221220        getProgressMonitor().worked(1);
    222221    }

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

    r30795 r32545  
    4646    private List<ModuleInformation> availableModules;
    4747
    48     protected enum CacheType {PLUGIN_LIST, ICON_LIST}
    49 
    50     protected void init(Collection<String> sites){
     48    protected enum CacheType { PLUGIN_LIST, ICON_LIST }
     49
     50    protected void init(Collection<String> sites) {
    5151        this.sites = sites;
    5252        if (sites == null) {
    … …  
    5454        }
    5555        availableModules = new LinkedList<>();
    56 
    57     }
     56    }
     57
    5858    /**
    5959     * Creates the task
    … …  
    7373     */
    7474    public ReadRemoteModuleInformationTask(ProgressMonitor monitor, Collection<String> sites) {
    75         super(tr("Download module list..."), monitor == null ? NullProgressMonitor.INSTANCE: monitor, false /* don't ignore exceptions */);
     75        super(tr("Download module list..."), monitor == null ? NullProgressMonitor.INSTANCE : monitor, false /* don't ignore exceptions */);
    7676        init(sites);
    7777    }
    … …  
    8080    protected void cancel() {
    8181        canceled = true;
    82         synchronized(this) {
     82        synchronized (this) {
    8383            if (connection != null) {
    8484                connection.disconnect();
    … …  
    110110            }
    111111            String path = url.getPath();
    112             for (int i =0;i<path.length()-4; i++) {
     112            for (int i = 0; i < path.length()-4; i++) {
    113113                char c = path.charAt(i);
    114114                if (Character.isLetterOrDigit(c)) {
    … …  
    127127            }
    128128            name = sb.toString();
    129         } catch(MalformedURLException e) {
     129        } catch (MalformedURLException e) {
    130130            name = "site-unknown.txt";
    131131        }
    … …  
    150150
    151151            URL url = new URL(printsite);
    152             synchronized(this) {
    153                 connection = (HttpURLConnection)url.openConnection();
     152            synchronized (this) {
     153                connection = (HttpURLConnection) url.openConnection();
    154154                connection.setRequestProperty("Cache-Control", "no-cache");
    155                 connection.setRequestProperty("User-Agent",Version.getInstance().getAgentString());
     155                connection.setRequestProperty("User-Agent", Version.getInstance().getAgentString());
    156156                connection.setRequestProperty("Host", url.getHost());
    157157                connection.setRequestProperty("Accept-Charset", "utf-8");
    … …  
    159159            try (BufferedReader in = new BufferedReader(new InputStreamReader(connection.getInputStream(), "UTF-8"))) {
    160160                String line;
    161                 while((line = in.readLine()) != null) {
     161                while ((line = in.readLine()) != null) {
    162162                    sb.append(line).append("\n");
    163163                }
    164164                return sb.toString();
    165165            }
    166         } catch(MalformedURLException e) {
     166        } catch (MalformedURLException e) {
    167167            if (canceled) return null;
    168168            e.printStackTrace();
    169169            return null;
    170         } catch(IOException e) {
     170        } catch (IOException e) {
    171171            if (canceled) return null;
    172172            e.printStackTrace();
    173173            return null;
    174174        } finally {
    175             synchronized(this) {
     175            synchronized (this) {
    176176                if (connection != null) {
    177177                    connection.disconnect();
    … …  
    197197
    198198            URL url = new URL(site);
    199             synchronized(this) {
    200                 connection = (HttpURLConnection)url.openConnection();
     199            synchronized (this) {
     200                connection = (HttpURLConnection) url.openConnection();
    201201                connection.setRequestProperty("Cache-Control", "no-cache");
    202                 connection.setRequestProperty("User-Agent",Version.getInstance().getAgentString());
     202                connection.setRequestProperty("User-Agent", Version.getInstance().getAgentString());
    203203                connection.setRequestProperty("Host", url.getHost());
    204204            }
    205205            try (
    206                 InputStream in = connection.getInputStream();
    207                 OutputStream out = new FileOutputStream(destFile)
    208             ) {
     206                    InputStream in = connection.getInputStream();
     207                    OutputStream out = new FileOutputStream(destFile)
     208                    ) {
    209209                byte[] buffer = new byte[8192];
    210210                for (int read = in.read(buffer); read != -1; read = in.read(buffer)) {
    … …  
    212212                }
    213213            }
    214         } catch(MalformedURLException e) {
     214        } catch (MalformedURLException e) {
    215215            if (canceled) return;
    216216            e.printStackTrace();
    217217            return;
    218         } catch(IOException e) {
     218        } catch (IOException e) {
    219219            if (canceled) return;
    220220            e.printStackTrace();
    221221            return;
    222222        } finally {
    223             synchronized(this) {
     223            synchronized (this) {
    224224                if (connection != null) {
    225225                    connection.disconnect();
    … …  
    232232            if (pi.icon == null && pi.iconPath != null) {
    233233                pi.icon = new ImageProvider(pi.name+".jar/"+pi.iconPath)
    234                                 .setArchive(destFile)
    235                                 .setMaxWidth(24)
    236                                 .setMaxHeight(24)
    237                                 .setOptional(true).get();
     234                        .setArchive(destFile)
     235                        .setMaxWidth(24)
     236                        .setMaxHeight(24)
     237                        .setOptional(true).get();
    238238            }
    239239        }
    … …  
    250250            File moduleDir = OdPlugin.getInstance().getModulesDirectory();
    251251            if (!moduleDir.exists()) {
    252                 if (! moduleDir.mkdirs()) {
     252                if (!moduleDir.mkdirs()) {
    253253                    Main.warn(tr("Warning: failed to create module directory ''{0}''. Cannot cache module list from module site ''{1}''.",
    254254                            moduleDir.toString(), site));
    … …  
    277277            InputStream in = new ByteArrayInputStream(doc.getBytes("UTF-8"));
    278278            availableModules.addAll(new ModuleListParser().parse(in));
    279         } catch(UnsupportedEncodingException e) {
     279        } catch (UnsupportedEncodingException e) {
    280280            Main.error(tr("Failed to parse module list document from site ''{0}''. Skipping site. Exception was: {1}", site, e.toString()));
    281281            e.printStackTrace();
    282         } catch(ModuleListParseException e) {
     282        } catch (ModuleListParseException e) {
    283283            Main.error(tr("Failed to parse module list document from site ''{0}''. Skipping site. Exception was: {1}", site, e.toString()));
    284284            e.printStackTrace();
    … …  
    295295        List<File> siteCacheFiles = new LinkedList<>();
    296296        for (String location : ModuleInformation.getModuleLocations()) {
    297             File [] f = new File(location).listFiles(
     297            File[] f = new File(location).listFiles(
    298298                    new FilenameFilter() {
    299299                        @Override
    300300                        public boolean accept(File dir, String name) {
    301301                            return name.matches("^([0-9]+-)?site.*\\.txt$") ||
    302                             name.matches("^([0-9]+-)?site.*-icons\\.zip$");
     302                                    name.matches("^([0-9]+-)?site.*-icons\\.zip$");
    303303                        }
    304304                    }
    305             );
    306             if(f != null && f.length > 0) {
     305                    );
     306            if (f != null && f.length > 0) {
    307307                siteCacheFiles.addAll(Arrays.asList(f));
    308308            }
    … …  
    316316            siteCacheFiles.remove(createSiteCacheFile(moduleDir, site, CacheType.PLUGIN_LIST));
    317317            siteCacheFiles.remove(createSiteCacheFile(moduleDir, site, CacheType.ICON_LIST));
    318             if(list != null)
    319             {
     318            if (list != null) {
    320319                getProgressMonitor().worked(1);
    321320                cacheModuleList(site, list);
    … …  
    327326                if (canceled) return;
    328327            }
    329             downloadModuleIcons(site.replace(".txt", "")+"-icons.zip", createSiteCacheFile(moduleDir, site, CacheType.ICON_LIST), getProgressMonitor().createSubTaskMonitor(0, false));
    330         }
    331         for (File file: siteCacheFiles) /* remove old stuff or whole update process is broken */
    332         {
     328            downloadModuleIcons(site.replace(".txt", "")+"-icons.zip",
     329                    createSiteCacheFile(moduleDir, site, CacheType.ICON_LIST), getProgressMonitor().createSubTaskMonitor(0, false));
     330        }
     331        for (File file: siteCacheFiles) {
     332            // remove old stuff or whole update process is broken
    333333            file.delete();
    334334        }
    … …  
    337337    /**
    338338     * Replies true if the task was canceled
    339      * @return
     339     * @return {@code true} if the task was canceled
    340340     */
    341341    public boolean isCanceled() {

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

    r30795 r32545  
    2626                result += " ";
    2727            }
    28             result += dictionary.containsKey(word) ?  dictionary.get(word) : word;
     28            result += dictionary.containsKey(word) ? dictionary.get(word) : word;
    2929        }
    3030        return result;
    … …  
    9595        } else if (label.equals("Jardin")) {
    9696            return label;
    97         } else if (label.startsWith("L'") || label.equals("La") || label.equals("Le") || label.equals("Les") || label.equals("Saint")) { // Lieux-dits
     97        } else if (label.startsWith("L'") || label.equals("La") || label.equals("Le") || label.equals("Les") ||
     98                label.equals("Saint")) { // Lieux-dits
    9899            return label;
    99100        } else if (label.equals("Lot")) {

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

    r31116 r32545  
    2121
    2222public abstract class OdUtils {
    23    
     23
    2424    private static final String TEMP_DIR_PREFIX = "josm_opendata_temp_";
    25    
     25
    2626    public static final boolean isMultipolygon(OsmPrimitive p) {
    2727        return p instanceof Relation && ((Relation) p).isMultipolygon();
    2828    }
    29    
     29
    3030    public static final String[] stripQuotesAndExtraChars(String[] split, String sep) {
    3131        List<String> result = new ArrayList<>();
    3232        boolean append = false;
    33         for (int i = 0; i<split.length; i++) {
     33        for (int i = 0; i < split.length; i++) {
    3434            if (append) {
    3535                int index = result.size()-1;
    … …  
    5353        return result.toArray(new String[0]);
    5454    }
    55    
     55
    5656    public static final ImageIcon getImageIcon(String iconName) {
    5757        return getImageIcon(iconName, false);
    5858    }
    59    
     59
    6060    public static final ImageIcon getImageIcon(String iconName, boolean optional) {
    6161        return getImageProvider(iconName, optional).get();
    6262    }
    63    
     63
    6464    public static final ImageProvider getImageProvider(String iconName) {
    6565        return getImageProvider(iconName, false);
    6666    }
    67    
     67
    6868    public static final ImageProvider getImageProvider(String iconName, boolean optional) {
    6969        return new ImageProvider(iconName).setOptional(optional).setAdditionalClassLoaders(ModuleHandler.getResourceClassLoaders());
    7070    }
    71    
     71
    7272    public static final String getJosmLanguage() {
    7373        String lang = Main.pref.get("language");
    … …  
    7777        return lang;
    7878    }
    79    
     79
    8080    public static final double convertMinuteSecond(double minute, double second) {
    8181        return (minute/60.0) + (second/3600.0);
    … …  
    8585        return degree + convertMinuteSecond(minute, second);
    8686    }
    87    
     87
    8888    public static final File createTempDir() throws IOException {
    8989        final File temp = File.createTempFile(TEMP_DIR_PREFIX, Long.toString(System.nanoTime()));
    … …  
    9696            throw new IOException("Could not create temp directory: " + temp.getAbsolutePath());
    9797        }
    98        
     98
    9999        return temp;
    100100    }
    101    
     101
    102102    public static final void deleteDir(File dir) {
    103103        for (File file : dir.listFiles()) {
    … …  
    110110        }
    111111    }
    112    
     112
    113113    public static final void deletePreviousTempDirs() {
    114114        File tmpDir = new File(System.getProperty("java.io.tmpdir"));

  • applications/editors/josm/plugins/opendata/test/unit/org/openstreetmap/josm/plugins/opendata/core/io/geographic/MifReaderTest.java

    r32355 r32545  
    2626    @Rule
    2727    public JOSMTestRules rules = new JOSMTestRules().preferences();
    28    
     28
    2929    private static AbstractDataSetHandler newHandler(final String epsgCode) {
    3030        AbstractDataSetHandler handler = new AbstractDataSetHandler() {
    … …  
    3333                return true;
    3434            }
     35
    3536            @Override
    3637            public void updateDataSet(DataSet ds) {
    … …  
    4243        return handler;
    4344    }
    44    
     45
    4546    /**
    4647     * Non-regression test for ticket <a href="https://josm.openstreetmap.de/ticket/9592">#9592</a>

  • applications/editors/josm/plugins/opendata/test/unit/org/openstreetmap/josm/plugins/opendata/core/io/geographic/ShpReaderTest.java

    r32360 r32545  
    3131
    3232    /**
    33      * Non-regression test for ticket <a href="https://josm.openstreetmap.de/ticket/12714">#12714/a>
     33     * Non-regression test for ticket <a href="https://josm.openstreetmap.de/ticket/12714">#12714</a>
    3434     * @throws IOException if an error occurs during reading
    3535     */
    … …  
    5555        }
    5656    }
    57    
     57
    5858    /**
    5959     * Non-regression test for ticket <a href="https://josm.openstreetmap.de/ticket/8309">#8309</a>

  • applications/editors/josm/plugins/opendata/test/unit/org/openstreetmap/josm/plugins/opendata/core/io/tabular/CsvReaderTest.java

    r32355 r32545  
    2929    @Rule
    3030    public JOSMTestRules rules = new JOSMTestRules().preferences();
    31    
     31
    3232    private static AbstractDataSetHandler newHandler(final String epsgCode) {
    3333        AbstractDataSetHandler handler = new AbstractDataSetHandler() {
    … …  
    3636                return true;
    3737            }
     38
    3839            @Override
    3940            public void updateDataSet(DataSet ds) {
    … …  
    4546                return true;
    4647            }
     48
    4749            @Override
    4850            public LatLon getCoor(EastNorth en, String[] fields) {
    … …  
    5254        return handler;
    5355    }
    54    
     56
    5557    /**
    5658     * Non-regression test for ticket <a href="https://josm.openstreetmap.de/ticket/10214">#10214</a>
    … …  
    6365        }
    6466    }
    65    
     67
    6668    /**
    6769     * Non-regression test for ticket <a href="https://josm.openstreetmap.de/ticket/8805">#8805</a>

  • applications/editors/josm/plugins/opendata/util/opendata/ModuleListGenerator.java

    r30910 r32545  
    1515import java.util.zip.ZipOutputStream;
    1616
    17 public class ModuleListGenerator {
     17public final class ModuleListGenerator {
    1818
    19         /**
    20          * @param args
    21          */
    22         public static void main(String[] args) {
    23                 final String url = "http://svn.openstreetmap.org/applications/editors/josm/plugins/opendata/dist/";
    24                 String baseDir = "";
    25                 if (args.length > 0) {
    26                         baseDir = args[0];
    27                 }
    28                 try (
    29                         BufferedWriter list = new BufferedWriter(new FileWriter(baseDir+"modules.txt"));
    30                         ZipOutputStream zip = new ZipOutputStream(new FileOutputStream(baseDir+"modules-icons.zip"));
    31                 ) {
    32                         for (File file : new File(baseDir+"dist").listFiles(new FilenameFilter() {
    33                                 @Override
    34                                 public boolean accept(File dir, String name) {
    35                                         return name.endsWith(".jar");
    36                                 }
    37                         })) {
    38                                 try {
    39                                         String filename = file.getName();
    40                                         System.out.println("Processing "+filename);
    41                                         list.write(filename+";"+url+filename); list.newLine();
    42                                         Manifest mf = new JarFile(file).getManifest();
    43                                         for (Object att : mf.getMainAttributes().keySet()) {
    44                                                 Object value = mf.getMainAttributes().get(att);
    45                                                 if (value != null) {
    46                                                         list.write("\t"+att+": "+value.toString()); list.newLine();
    47                                                         if (att.toString().equals("Module-Icon")) {
    48                                                                 // Directory with jar name, including extension
    49                                                                 String name = filename+"/";
    50                                                                 zip.putNextEntry(new ZipEntry(name));
    51                                                                 // Directory tree to image
    52                                                                 String[] items = value.toString().split("/");
    53                                                                 for (int i=0; i<items.length-1; i++) {
    54                                                                         zip.putNextEntry(new ZipEntry(name += items[i]+"/"));
    55                                                                 }
    56                                                                 // Image file
    57                                                                 zip.putNextEntry(new ZipEntry(name += items[items.length-1]));
    58                                                                 try {
    59                                                                         FileInputStream in;
    60                                                                         try {
    61                                                                                 in = new FileInputStream(baseDir+"modules/"+filename.replace(".jar", "/")+value.toString());
    62                                                                         } catch (FileNotFoundException e) {
    63                                                                                 // If not in module dir, may be in main images directory
    64                                                                                 if (value.toString().startsWith("images/")) {
    65                                                                                         in = new FileInputStream(baseDir+value.toString());
    66                                                                                 } else {
    67                                                                                         throw e;
    68                                                                                 }
    69                                                                         }
    70                                                                         try {
    71                                                                                 byte[] buffer = new byte[4096];
    72                                                                                 int n = -1;
    73                                                                                 while ((n = in.read(buffer)) > 0) {
    74                                                                                         zip.write(buffer, 0, n);
    75                                                                                 }
    76                                                                         } finally {
    77                                                                                 in.close();
    78                                                                         }
    79                                                                 } catch (IOException e) {
    80                                                                         System.err.println("Cannot load Image-Icon: "+value.toString());
    81                                                                 } finally {
    82                                                                         zip.closeEntry();
    83                                                                 }
    84                                                         }
    85                                                 }
    86                                         }
    87                                        
    88                                 } catch (IOException e) {
    89                                         e.printStackTrace();
    90                                 }
    91                         }
    92                 } catch (IOException e) {
    93                         e.printStackTrace();
    94                 }
    95         }
     19    private ModuleListGenerator() {
     20        // Hide default constructor for utilities classes
     21    }
     22
     23    /**
     24     * @param args Main program arguments
     25     */
     26    public static void main(String[] args) {
     27        final String url = "http://svn.openstreetmap.org/applications/editors/josm/plugins/opendata/dist/";
     28        String baseDir = "";
     29        if (args.length > 0) {
     30            baseDir = args[0];
     31        }
     32        try (
     33                BufferedWriter list = new BufferedWriter(new FileWriter(baseDir+"modules.txt"));
     34                ZipOutputStream zip = new ZipOutputStream(new FileOutputStream(baseDir+"modules-icons.zip"));
     35                ) {
     36            for (File file : new File(baseDir+"dist").listFiles(new FilenameFilter() {
     37                @Override
     38                public boolean accept(File dir, String name) {
     39                    return name.endsWith(".jar");
     40                }
     41            })) {
     42                try {
     43                    String filename = file.getName();
     44                    System.out.println("Processing "+filename);
     45                    list.write(filename+";"+url+filename); list.newLine();
     46                    Manifest mf = new JarFile(file).getManifest();
     47                    for (Object att : mf.getMainAttributes().keySet()) {
     48                        Object value = mf.getMainAttributes().get(att);
     49                        if (value != null) {
     50                            list.write("\t"+att+": "+value.toString()); list.newLine();
     51                            if (att.toString().equals("Module-Icon")) {
     52                                // Directory with jar name, including extension
     53                                String name = filename+"/";
     54                                zip.putNextEntry(new ZipEntry(name));
     55                                // Directory tree to image
     56                                String[] items = value.toString().split("/");
     57                                for (int i = 0; i < items.length-1; i++) {
     58                                    zip.putNextEntry(new ZipEntry(name += items[i]+"/"));
     59                                }
     60                                // Image file
     61                                zip.putNextEntry(new ZipEntry(name += items[items.length-1]));
     62                                try {
     63                                    FileInputStream in;
     64                                    try {
     65                                        in = new FileInputStream(baseDir+"modules/"+filename.replace(".jar", "/")+value.toString());
     66                                    } catch (FileNotFoundException e) {
     67                                        // If not in module dir, may be in main images directory
     68                                        if (value.toString().startsWith("images/")) {
     69                                            in = new FileInputStream(baseDir+value.toString());
     70                                        } else {
     71                                            throw e;
     72                                        }
     73                                    }
     74                                    try {
     75                                        byte[] buffer = new byte[4096];
     76                                        int n = -1;
     77                                        while ((n = in.read(buffer)) > 0) {
     78                                            zip.write(buffer, 0, n);
     79                                        }
     80                                    } finally {
     81                                        in.close();
     82                                    }
     83                                } catch (IOException e) {
     84                                    System.err.println("Cannot load Image-Icon: "+value.toString());
     85                                } finally {
     86                                    zip.closeEntry();
     87                                }
     88                            }
     89                        }
     90                    }
     91
     92                } catch (IOException e) {
     93                    e.printStackTrace();
     94                }
     95            }
     96        } catch (IOException e) {
     97            e.printStackTrace();
     98        }
     99    }
    96100}
Note: See TracChangeset for help on using the changeset viewer.