Ignore:
Timestamp:
2014-01-19T01:17:29+01:00 (11 years ago)
Author:
donvip
Message:

[josm_opendata] fix module/handlers conflicts in toolbar actions with same name

Location:
applications/editors/josm/plugins/opendata
Files:
5 edited

Legend:

Unmodified
Added
Removed

  • applications/editors/josm/plugins/opendata/modules/fr.datagouvfr/src/org/openstreetmap/josm/plugins/opendata/modules/fr/datagouvfr/datasets/ecologie/ForetsPubliquesHandler.java

    r28054 r30211  
    121121        }
    122122
    123         /* (non-Javadoc)
    124          * @see org.openstreetmap.josm.plugins.opendata.core.datasets.AbstractDataSetHandler#getDataURLs()
    125          */
    126123        @Override
    127124        public List<Pair<String, URL>> getDataURLs() {
    … …  
    139136        }
    140137
    141         private Pair<String, URL> getForetURL(String code, String name) throws MalformedURLException {
    142                 return new Pair<String, URL>(name, new URL(FRENCH_PORTAL+"var/download/"+"for_publ_v2011_reg"+code+".zip"));
     138        private Pair<String, URL> getForetURL(String code, String regionName) throws MalformedURLException {
     139                return new Pair<String, URL>("PublicForests_"+regionName, new URL(FRENCH_PORTAL+"var/download/"+"for_publ_v2011_reg"+code+".zip"));
    143140        }
    144141}

  • applications/editors/josm/plugins/opendata/modules/fr.datagouvfr/src/org/openstreetmap/josm/plugins/opendata/modules/fr/datagouvfr/datasets/hydrologie/EauxDeSurfaceHandler.java

    r29679 r30211  
    5353                new WaterAgency("B1", "Meuse", "Meuse-30381855"),
    5454                new WaterAgency("B2", "Sambre", "Sambre-30381857"),
    55                 new WaterAgency("C", "Rhin", "Rhin-30381951"),
     55                new WaterAgency("C",  "Rhin", "Rhin-30381951"),
    5656                new WaterAgency("D",  "Rhône Méditerranée", "Rhône-Méditerranée-30382014"),
    5757                new WaterAgency("E",  "Corse", "Corse-30381905"),
    … …  
    106106        }
    107107       
    108         /* (non-Javadoc)
    109          * @see org.openstreetmap.josm.plugins.opendata.core.datasets.AbstractDataSetHandler#getDataURLs()
    110          */
    111108        @Override
    112109        public List<Pair<String, URL>> getDataURLs() {
    113110                List<Pair<String, URL>> result = new ArrayList<Pair<String,URL>>();
    114111                try {
    115                         for (int i =0; i<waterAgencies.length; i++) {
    116                                 result.add(getDownloadURL(waterAgencies[i]));
     112                        for (WaterAgency wa : waterAgencies) {
     113                                result.add(getDownloadURL(wa));
    117114                        }
    118115                } catch (MalformedURLException e) {
    … …  
    123120
    124121        private Pair<String, URL> getDownloadURL(WaterAgency a) throws MalformedURLException {
    125                 return new Pair<String, URL>(a.name, new URL("http://www.rapportage.eaufrance.fr/sites/default/files/SIG/FR"+a.code+"_SW.zip"));
     122                return new Pair<String, URL>("SurfaceWater_"+a.name, new URL("http://www.rapportage.eaufrance.fr/sites/default/files/SIG/FR"+a.code+"_SW.zip"));
    126123        }
    127124       

  • applications/editors/josm/plugins/opendata/modules/fr.lemans/src/org/openstreetmap/josm/plugins/opendata/modules/fr/lemans/datasets/LeMansDataSetHandler.java

    r28113 r30211  
    7272        }
    7373
    74         /* (non-Javadoc)
    75          * @see org.openstreetmap.josm.plugins.opendata.core.datasets.AbstractDataSetHandler#getSource()
    76          */
    7774        @Override
    7875        public String getSource() {
    … …  
    8077        }
    8178       
    82         /* (non-Javadoc)
    83          * @see org.openstreetmap.josm.plugins.opendata.core.datasets.AbstractDataSetHandler#getLicenseURL()
    84          */
    8579        /*@Override
    8680        public URL getLicenseURL() {
    … …  
    9387        }*/
    9488
    95         /* (non-Javadoc)
    96          * @see org.openstreetmap.josm.plugins.opendata.core.datasets.AbstractDataSetHandler#getDataURLs()
    97          */
    9889        @Override
    9990        public List<Pair<String, URL>> getDataURLs() {
    10091                List<Pair<String, URL>> result = new ArrayList<Pair<String,URL>>();
    10192                try {
    102                         if (kmzUuid != null && !kmzUuid.isEmpty()) result.add(new Pair<String, URL>("KMZ", new URL(PORTAL + "download.do?uuid=" + kmzUuid)));
    103                         if (shpUuid != null && !shpUuid.isEmpty()) result.add(new Pair<String, URL>("SHP", new URL(PORTAL + "download.do?uuid=" + shpUuid)));
     93                        if (kmzUuid != null && !kmzUuid.isEmpty()) result.add(new Pair<String, URL>(getName() + " (KMZ)", new URL(PORTAL + "download.do?uuid=" + kmzUuid)));
     94                        if (shpUuid != null && !shpUuid.isEmpty()) result.add(new Pair<String, URL>(getName() + " (SHP)", new URL(PORTAL + "download.do?uuid=" + shpUuid)));
    10495                } catch (MalformedURLException e) {
    10596                        e.printStackTrace();

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

    r29854 r30211  
    121121                JMenu moduleMenu = null;
    122122                for (AbstractDataSetHandler handler: module.getNewlyInstanciatedHandlers()) {
    123                         if (handler.getDataURL() != null || (handler.getDataURLs() != null && !handler.getDataURLs().isEmpty())) {
     123                    URL dataURL = handler.getDataURL();
     124                    List<Pair<String, URL>> dataURLs = handler.getDataURLs();
     125                        if (dataURL != null || (dataURLs != null && !dataURLs.isEmpty())) {
    124126                                if (moduleMenu == null) {
    125127                                        moduleMenu = getModuleMenu(module);
    … …  
    142144                                }
    143145                                JMenuItem handlerItem = null;
    144                                 if (handler.getDataURL() != null) {
    145                                     handlerItem = endMenu.add(new DownloadDataAction(handlerName, handler.getDataURL()));
    146                                 } else if (handler.getDataURLs() != null) {
     146                                if (dataURL != null) {
     147                                    handlerItem = endMenu.add(new DownloadDataAction(module.getDisplayedName()+"_"+handlerName, dataURL));
     148                                } else if (dataURLs != null) {
    147149                                        JMenu handlerMenu = new JMenu(handlerName);
    148150                                        JMenuItem item = null;
    149                                         for (Pair<String, URL> pair : handler.getDataURLs()) {
     151                                        for (Pair<String, URL> pair : dataURLs) {
    150152                                                if (pair != null && pair.a != null && pair.b != null) {
    151153                                                        item = handlerMenu.add(new DownloadDataAction(pair.a, pair.b));

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

    r29686 r30211  
    427427       
    428428        public boolean acceptsUrl(String url) {
    429                 if (getDataURL() != null && url.equals(getDataURL().toString())) {
     429            URL dataURL = getDataURL();
     430                if (dataURL != null && url.equals(dataURL.toString())) {
    430431                        return true;
    431432                }
    432                 if (getDataURLs() != null) {
    433                         for (Pair<String, URL> pair : getDataURLs()) {
     433                List<Pair<String, URL>> dataURLs = getDataURLs();
     434                if (dataURLs != null) {
     435                        for (Pair<String, URL> pair : dataURLs) {
    434436                                if (pair.b != null && url.equals(pair.b.toString())) {
    435437                                        return true;
Note: See TracChangeset for help on using the changeset viewer.