Changeset 15121 in josm


Ignore:
Timestamp:
2019-05-27T12:37:31+02:00 (7 months ago)
Author:
Don-vip
Message:

sonarqube, javadoc

Location:
trunk
Files:
17 edited

Legend:

Unmodified
Added
Removed
  • trunk/data/gpx-extensions.xsd

    r7937 r15121  
    11<?xml version="1.0" encoding="UTF-8"?>
    22<schema xmlns="http://www.w3.org/2001/XMLSchema" targetNamespace="http://josm.openstreetmap.de/gpx-extensions-1.0"
    3         xmlns:tns="http://josm.openstreetmap.de/gpx-extensions-1.0" elementFormDefault="qualified">
     3    xmlns:tns="http://josm.openstreetmap.de/gpx-extensions-1.0" elementFormDefault="qualified">
    44
    55    <!-- true, if gpx data has been downloaded from the osm server -->
  • trunk/src/org/openstreetmap/josm/actions/relation/ExportRelationToGpxAction.java

    r14682 r15121  
    152152            final HashMap<String, Integer> names = new HashMap<>();
    153153            for (int i = 0; i < flat.size(); i++) {
    154                 if (!wct.get(i).isOnewayLoopBackwardPart) {
    155                     if (!wct.get(i).direction.isRoundabout()) {
    156                         if (!wct.get(i).linkPrev && !trkseg.isEmpty()) {
    157                             gpxData.addTrack(new ImmutableGpxTrack(trk, trkAttr));
    158                             trkAttr.clear();
    159                             trk.clear();
    160                             trkseg.clear();
    161                             trk.add(trkseg);
    162                         }
    163                         if (trkAttr.isEmpty()) {
    164                             flat.get(i).getWay().referrers(Relation.class)
    165                                     .filter(relsFound::contains)
    166                                     .findFirst()
    167                                     .ifPresent(r -> {
    168                                         trkAttr.put("name", r.getName() != null ? r.getName() : r.getId());
    169                                         trkAttr.put("desc", tr("based on osm route relation data, timestamps are synthetic"));
    170                                     });
    171                             GpxData.ensureUniqueName(trkAttr, names);
    172                         }
    173                         List<Node> ln = flat.get(i).getWay().getNodes();
    174                         if (wct.get(i).direction == WayConnectionType.Direction.BACKWARD)
    175                             Collections.reverse(ln);
    176                         for (Node n: ln) {
    177                             trkseg.add(OsmDataLayer.nodeToWayPoint(n, TimeUnit.SECONDS.toMillis(time)));
    178                             time += 1;
    179                         }
     154                WayConnectionType wayConnectionType = wct.get(i);
     155                if (!wayConnectionType.isOnewayLoopBackwardPart && !wayConnectionType.direction.isRoundabout()) {
     156                    if (!wayConnectionType.linkPrev && !trkseg.isEmpty()) {
     157                        gpxData.addTrack(new ImmutableGpxTrack(trk, trkAttr));
     158                        trkAttr.clear();
     159                        trk.clear();
     160                        trkseg.clear();
     161                        trk.add(trkseg);
     162                    }
     163                    if (trkAttr.isEmpty()) {
     164                        flat.get(i).getWay().referrers(Relation.class)
     165                                .filter(relsFound::contains)
     166                                .findFirst()
     167                                .ifPresent(r -> {
     168                                    trkAttr.put("name", r.getName() != null ? r.getName() : r.getId());
     169                                    trkAttr.put("desc", tr("based on osm route relation data, timestamps are synthetic"));
     170                                });
     171                        GpxData.ensureUniqueName(trkAttr, names);
     172                    }
     173                    List<Node> ln = flat.get(i).getWay().getNodes();
     174                    if (wayConnectionType.direction == WayConnectionType.Direction.BACKWARD)
     175                        Collections.reverse(ln);
     176                    for (Node n: ln) {
     177                        trkseg.add(OsmDataLayer.nodeToWayPoint(n, TimeUnit.SECONDS.toMillis(time)));
     178                        time += 1;
    180179                    }
    181180                }
  • trunk/src/org/openstreetmap/josm/data/StructUtils.java

    r14977 r15121  
    147147     * @return the resulting map (same data content as <code>struct</code>)
    148148     */
    149     public static <T> Map<String, String> serializeStruct(T struct, Class<T> klass) {
     149    public static <T> HashMap<String, String> serializeStruct(T struct, Class<T> klass) {
    150150        T structPrototype;
    151151        try {
     
    155155        }
    156156
    157         Map<String, String> hash = new LinkedHashMap<>();
     157        HashMap<String, String> hash = new LinkedHashMap<>();
    158158        for (Field f : klass.getDeclaredFields()) {
    159159            if (f.getAnnotation(StructEntry.class) == null) {
  • trunk/src/org/openstreetmap/josm/data/gpx/GpxData.java

    r14853 r15121  
    461461
    462462        List<GpxTrack> trks = getTracks().stream()
    463             .flatMap(trk -> {
    464                 return trk.getSegments().stream().map(seg -> {
     463            .flatMap(trk -> trk.getSegments().stream().map(seg -> {
    465464                    HashMap<String, Object> attrs = new HashMap<>(trk.getAttributes());
    466465                    ensureUniqueName(attrs, counts);
    467466                    return new ImmutableGpxTrack(Arrays.asList(seg), attrs);
    468                 });
    469             })
     467                }))
    470468            .collect(Collectors.toCollection(ArrayList<GpxTrack>::new));
    471469
  • trunk/src/org/openstreetmap/josm/data/osm/history/HistoryWay.java

    r13907 r15121  
    88import java.util.Date;
    99import java.util.List;
     10import java.util.Objects;
    1011
    1112import org.openstreetmap.josm.data.osm.OsmPrimitiveType;
     
    137138     */
    138139    public boolean isClosed() {
    139         return getNumNodes() >= 3 && nodeIds.get(0) == nodeIds.get(nodeIds.size()-1);
     140        return getNumNodes() >= 3 && Objects.equals(nodeIds.get(0), nodeIds.get(nodeIds.size()-1));
    140141    }
    141142
  • trunk/src/org/openstreetmap/josm/data/validation/Test.java

    r14599 r15121  
    324324     */
    325325    public boolean isCanceled() {
    326         return progressMonitor != null ? progressMonitor.isCanceled() : false;
     326        return progressMonitor != null && progressMonitor.isCanceled();
    327327    }
    328328
  • trunk/src/org/openstreetmap/josm/data/validation/tests/PublicTransportRouteTest.java

    r14473 r15121  
    5252                        .build());
    5353                return;
    54             } else if (member.hasRole("") && OsmPrimitiveType.WAY.equals(member.getType())) {
     54            } else if (member.hasRole("") && OsmPrimitiveType.WAY == member.getType()) {
    5555                membersToCheck.add(member);
    5656                routeNodes.addAll(member.getWay().getNodes());
     
    6767                    || !(i == links.size() - 1 || link.linkNext)
    6868                    || link.direction == null
    69                     || WayConnectionType.Direction.NONE.equals(link.direction);
     69                    || WayConnectionType.Direction.NONE == link.direction;
    7070            if (hasError) {
    7171                errors.add(TestError.builder(this, Severity.WARNING, 3602)
     
    7979        for (RelationMember member : r.getMembers()) {
    8080            if (member.hasRole("stop", "stop_exit_only", "stop_entry_only")
    81                     && OsmPrimitiveType.NODE.equals(member.getType())
     81                    && OsmPrimitiveType.NODE == member.getType()
    8282                    && !routeNodes.contains(member.getNode())) {
    8383                errors.add(TestError.builder(this, Severity.WARNING, 3603)
  • trunk/src/org/openstreetmap/josm/gui/MapView.java

    r15041 r15121  
    102102            @Override
    103103            public void mapPaintStylesUpdated() {
    104                 SwingUtilities.invokeLater(() -> {
     104                SwingUtilities.invokeLater(() ->
    105105                    // Trigger a repaint of all data layers
    106106                    MainApplication.getLayerManager().getLayers()
    107107                        .stream()
    108108                        .filter(layer -> layer instanceof OsmDataLayer)
    109                         .forEach(Layer::invalidate);
    110                 });
     109                        .forEach(Layer::invalidate)
     110                );
    111111            }
    112112
  • trunk/src/org/openstreetmap/josm/gui/io/SaveLayerInfo.java

    r9751 r15121  
    9999     */
    100100    public void setDoSaveToFile(boolean doSaveToFile) {
    101         this.doSaveToFile = isSavable() ? doSaveToFile : false;
     101        this.doSaveToFile = isSavable() && doSaveToFile;
    102102    }
    103103
     
    117117     */
    118118    public void setDoUploadToServer(boolean doUploadToServer) {
    119         this.doUploadToServer = isUploadable() ? doUploadToServer : false;
     119        this.doUploadToServer = isUploadable() && doUploadToServer;
    120120    }
    121121
  • trunk/src/org/openstreetmap/josm/gui/io/UploadDialog.java

    r15117 r15121  
    604604            List<String> forbiddenTerms = Config.getPref().getList(preferencePrefix+".forbidden-terms", defForbidden)
    605605                    .stream().map(UploadAction::lower)
    606                     .filter(x -> uploadValueLc.contains(x) && !exceptions.stream().anyMatch(uploadValueLc::contains))
     606                    .filter(x -> uploadValueLc.contains(x) && exceptions.stream().noneMatch(uploadValueLc::contains))
    607607                    .collect(Collectors.toList());
    608608            if (!forbiddenTerms.isEmpty()) {
  • trunk/src/org/openstreetmap/josm/gui/layer/OsmDataLayer.java

    r14668 r15121  
    11321132    @Override
    11331133    public boolean checkSaveConditions() {
    1134         if (isDataSetEmpty() && 1 != GuiHelper.runInEDTAndWaitAndReturn(() -> {
    1135             return new ExtendedDialog(
     1134        if (isDataSetEmpty() && 1 != GuiHelper.runInEDTAndWaitAndReturn(() ->
     1135            new ExtendedDialog(
    11361136                    MainApplication.getMainFrame(),
    11371137                    tr("Empty document"),
     
    11391139                .setContent(tr("The document contains no data."))
    11401140                .setButtonIcons("save", "cancel")
    1141                 .showDialog().getValue();
    1142         })) {
     1141                .showDialog().getValue()
     1142        )) {
    11431143            return false;
    11441144        }
  • trunk/src/org/openstreetmap/josm/gui/preferences/display/ColorPreference.java

    r14152 r15121  
    406406        }
    407407        for (ColorEntry e : tableModel.getData()) {
    408             if (e.info.getValue() != null) {
    409                 if (e.toProperty().put(e.info.getValue())
    410                         && NamedColorProperty.COLOR_CATEGORY_MAPPAINT.equals(e.info.getCategory())) {
    411                     ret = true;
    412                 }
     408            if (e.info.getValue() != null && e.toProperty().put(e.info.getValue())
     409                    && NamedColorProperty.COLOR_CATEGORY_MAPPAINT.equals(e.info.getCategory())) {
     410                ret = true;
    413411            }
    414412        }
  • trunk/src/org/openstreetmap/josm/gui/preferences/imagery/ImageryProvidersPanel.java

    r15116 r15121  
    319319        activeToolbar.add(new NewEntryAction(ImageryInfo.ImageryType.TMS));
    320320        activeToolbar.add(new NewEntryAction(ImageryInfo.ImageryType.WMTS));
    321         //activeToolbar.add(edit); TODO
    322321        activeToolbar.add(remove);
    323322        add(activeToolbar, GBC.eol().anchor(GBC.NORTH).insets(0, 0, 5, 5));
     
    698697            case 3:
    699698                return info.getExtendedUrl();
    700             }
    701             return null;
     699            default:
     700                return null;
     701            }
    702702        }
    703703
  • trunk/src/org/openstreetmap/josm/gui/preferences/plugin/PluginPreference.java

    r15116 r15121  
    5555import org.openstreetmap.josm.gui.util.GuiHelper;
    5656import org.openstreetmap.josm.gui.widgets.FilterField;
    57 import org.openstreetmap.josm.gui.widgets.JosmTextField;
    5857import org.openstreetmap.josm.plugins.PluginDownloadTask;
    5958import org.openstreetmap.josm.plugins.PluginInformation;
     
    8281    }
    8382
    84     private JosmTextField tfFilter;
    8583    private PluginListPanel pnlPluginPreferences;
    8684    private PluginPreferencesModel model;
     
    190188        gc.gridx = 1;
    191189        gc.weightx = 1.0;
    192         tfFilter = new FilterField().filter(expr -> {
     190        pnl.add(new FilterField().filter(expr -> {
    193191            model.filterDisplayedPlugins(expr);
    194192            pnlPluginPreferences.refreshView();
    195         });
    196         pnl.add(tfFilter, gc);
     193        }), gc);
    197194        return pnl;
    198195    }
  • trunk/src/org/openstreetmap/josm/io/CertificateAmendment.java

    r15006 r15121  
    160160                "0687260331a72403d909f105e69bcf0d32e1bd2493ffc6d9206d11bcd6770739",
    161161                "https://acme-v02.api.letsencrypt.org"),
    162         // Government of Netherlands
     162        // #14649 - Government of Netherlands - for PDOK aerial imagery at ​https://geodata.nationaalgeoregister.nl
    163163        new NativeCertAmend(Collections.singleton("Staat der Nederlanden Root CA - G2"),
    164164                "Staat_der_Nederlanden_Root_CA_-_G2.crt",
    165165                "668c83947da63b724bece1743c31a0e6aed0db8ec5b31be377bb784f91b6716f",
    166166                "https://roottest-g2.pkioverheid.nl"),
    167         // Government of Netherlands
     167        // #14649 - Government of Netherlands - for PDOK aerial imagery at ​https://geodata.nationaalgeoregister.nl
    168168        new NativeCertAmend(Arrays.asList("Government of Netherlands G3", "Staat der Nederlanden Root CA - G3"),
    169169                "Staat_der_Nederlanden_Root_CA_-_G3.crt",
    170170                "3c4fb0b95ab8b30032f432b86f535fe172c185d0fd39865837cf36187fa6f428",
    171171                "https://roottest-g3.pkioverheid.nl"),
    172         // Trusted and used by French Government - https://www.certigna.fr/autorites/index.xhtml?ac=Racine#lracine
     172        // #15178 - Trusted and used by French Government - for cadastre - https://www.certigna.fr/autorites/index.xhtml?ac=Racine#lracine
    173173        new NativeCertAmend(Collections.singleton("Certigna"),
    174174                "Certigna.crt",
    175175                "e3b6a2db2ed7ce48842f7ac53241c7b71d54144bfb40c11f3f1d0b42f5eea12d",
    176176                "https://www.certigna.fr"),
    177         // Trusted and used by Slovakian Government - https://eidas.disig.sk/en/cacert/
     177        // #16307 - Trusted and used by Slovakian Government - https://eidas.disig.sk/en/cacert/
    178178        new NativeCertAmend(Collections.singleton("CA Disig Root R2"),
    179179                "CA_Disig_Root_R2.pem",
    180180                "e23d4a036d7b70e9f595b1422079d2b91edfbb1fb651a0633eaa8a9dc5f80703",
    181181                "https://eidas.disig.sk"),
    182         // Government of Taiwan - https://grca.nat.gov.tw/GRCAeng/index.html
     182        // #17062 - Government of Taiwan - for https://data.gov.tw/license - https://grca.nat.gov.tw/GRCAeng/index.html
    183183        new NativeCertAmend(Arrays.asList("TW Government Root Certification Authority", "Government Root Certification Authority"),
    184184                "Taiwan_GRCA.pem",
  • trunk/src/org/openstreetmap/josm/spi/preferences/DefaultPreferenceChangeEvent.java

    r14977 r15121  
    99 */
    1010public class DefaultPreferenceChangeEvent extends EventObject implements PreferenceChangeEvent {
     11
     12    private static final long serialVersionUID = 1L;
    1113
    1214    private final String key;
  • trunk/src/org/openstreetmap/josm/tools/PlatformHookOsx.java

    r15006 r15121  
    3333
    3434/**
    35  * {@code PlatformHook} implementation for Apple Mac OS X systems.
     35 * {@code PlatformHook} implementation for Apple macOS (formerly Mac OS X) systems.
    3636 * @since 1023
    3737 */
Note: See TracChangeset for help on using the changeset viewer.