Ignore:
Timestamp:
2016-07-23T18:46:45+02:00 (3 years ago)
Author:
Don-vip
Message:

see #11390 - sonar - squid:S1604 - Java 8: Anonymous inner classes containing only one method should become lambdas

Location:
trunk/src/org/openstreetmap/josm/gui/io
Files:
9 edited

Legend:

Unmodified
Added
Removed
  • trunk/src/org/openstreetmap/josm/gui/io/AbstractPrimitiveTask.java

    r10212 r10611  
    184184            return;
    185185        }
    186         GuiHelper.runInEDTAndWait(new Runnable() {
    187             @Override
    188             public void run() {
    189                 layer.mergeFrom(ds);
    190                 if (zoom && Main.map != null)
    191                     AutoScaleAction.zoomTo(ds.allPrimitives());
    192                 layer.onPostDownloadFromServer();
    193             }
     186        GuiHelper.runInEDTAndWait(() -> {
     187            layer.mergeFrom(ds);
     188            if (zoom && Main.map != null)
     189                AutoScaleAction.zoomTo(ds.allPrimitives());
     190            layer.onPostDownloadFromServer();
    194191        });
    195192    }
  • trunk/src/org/openstreetmap/josm/gui/io/ActionFlagsTableCell.java

    r10305 r10611  
    4040    private final transient CellEditorSupport cellEditorSupport = new CellEditorSupport(this);
    4141
    42     private final transient ActionListener al = new ActionListener() {
    43         @Override
    44         public void actionPerformed(ActionEvent e) {
    45             cellEditorSupport.fireEditingStopped();
    46         }
    47     };
     42    private final transient ActionListener al = e -> cellEditorSupport.fireEditingStopped();
    4843
    4944    /**
  • trunk/src/org/openstreetmap/josm/gui/io/BasicUploadSettingsPanel.java

    r10210 r10611  
    2424import javax.swing.event.ChangeListener;
    2525import javax.swing.event.HyperlinkEvent;
    26 import javax.swing.event.HyperlinkListener;
    2726
    2827import org.openstreetmap.josm.Main;
     
    7069        JEditorPane sourceLabel = new JMultilineLabel("<html><b>" + tr("Specify the data source for the changes")
    7170                + "</b> (<a href=\"urn:changeset-source\">" + tr("obtain from current layers") + "</a>)<b>:</b>");
    72         sourceLabel.addHyperlinkListener(new HyperlinkListener() {
    73             @Override
    74             public void hyperlinkUpdate(HyperlinkEvent e) {
    75                 if (HyperlinkEvent.EventType.ACTIVATED.equals(e.getEventType())) {
    76                     final String source = Main.map.mapView.getLayerInformationForSourceTag();
    77                     hcbUploadSource.setText(Utils.shortenString(source, Changeset.MAX_CHANGESET_TAG_LENGTH));
    78                     // Fix #9965
    79                     changesetSourceModel.setComment(hcbUploadSource.getText());
    80                 }
     71        sourceLabel.addHyperlinkListener(e -> {
     72            if (HyperlinkEvent.EventType.ACTIVATED.equals(e.getEventType())) {
     73                final String source = Main.map.mapView.getLayerInformationForSourceTag();
     74                hcbUploadSource.setText(Utils.shortenString(source, Changeset.MAX_CHANGESET_TAG_LENGTH));
     75                // Fix #9965
     76                changesetSourceModel.setComment(hcbUploadSource.getText());
    8177            }
    8278        });
  • trunk/src/org/openstreetmap/josm/gui/io/CloseChangesetTask.java

    r10212 r10611  
    5858            ExceptionDialogUtil.explainException(lastException);
    5959        }
    60         SwingUtilities.invokeLater(
    61                 new Runnable() {
    62                     @Override
    63                     public void run() {
    64                         ChangesetCache.getInstance().update(closedChangesets);
    65                     }
    66                 }
    67         );
     60        SwingUtilities.invokeLater(() -> ChangesetCache.getInstance().update(closedChangesets));
    6861    }
    6962
  • trunk/src/org/openstreetmap/josm/gui/io/DownloadOpenChangesetsTask.java

    r10212 r10611  
    8585            return;
    8686        }
    87         SwingUtilities.invokeLater(
    88                 new Runnable() {
    89                     @Override public void run() {
    90                         ChangesetCache.getInstance().update(changesets);
    91                     }
    92                 }
    93         );
     87        SwingUtilities.invokeLater(() -> ChangesetCache.getInstance().update(changesets));
    9488    }
    9589
    9690    /**
    97      * Refreshes the user info from the server. This is necessary if we don't know
    98      * the users id yet.
    99      *
     91     * Refreshes the user info from the server. This is necessary if we don't know the users id yet.
    10092     */
    10193    protected void refreshUserIdentity() {
  • trunk/src/org/openstreetmap/josm/gui/io/DownloadPrimitivesWithReferrersTask.java

    r10446 r10611  
    138138        final Set<PrimitiveId> errs = mainTask.getMissingPrimitives();
    139139        if (errs != null && !errs.isEmpty())
    140             GuiHelper.runInEDTAndWait(new Runnable() {
    141                 @Override
    142                 public void run() {
    143                     reportProblemDialog(errs,
    144                             trn("Object could not be downloaded", "Some objects could not be downloaded", errs.size()),
    145                             trn("One object could not be downloaded.<br>",
    146                                     "{0} objects could not be downloaded.<br>",
    147                                     errs.size(),
    148                                     errs.size())
    149                                     + tr("The server replied with response code 404.<br>"
    150                                          + "This usually means, the server does not know an object with the requested id."),
    151                             tr("missing objects:"),
    152                             JOptionPane.ERROR_MESSAGE
    153                             ).showDialog();
    154                 }
    155             });
     140            GuiHelper.runInEDTAndWait(() -> reportProblemDialog(errs,
     141                    trn("Object could not be downloaded", "Some objects could not be downloaded", errs.size()),
     142                    trn("One object could not be downloaded.<br>",
     143                            "{0} objects could not be downloaded.<br>",
     144                            errs.size(),
     145                            errs.size())
     146                            + tr("The server replied with response code 404.<br>"
     147                                 + "This usually means, the server does not know an object with the requested id."),
     148                    tr("missing objects:"),
     149                    JOptionPane.ERROR_MESSAGE
     150                    ).showDialog());
    156151
    157152        // Warm about deleted primitives
     
    165160        }
    166161        if (!del.isEmpty())
    167             GuiHelper.runInEDTAndWait(new Runnable() {
    168                 @Override
    169                 public void run() {
    170                     reportProblemDialog(del,
    171                             trn("Object deleted", "Objects deleted", del.size()),
    172                             trn(
    173                                 "One downloaded object is deleted.",
    174                                 "{0} downloaded objects are deleted.",
    175                                 del.size(),
    176                                 del.size()),
    177                             null,
    178                             JOptionPane.WARNING_MESSAGE
    179                     ).showDialog();
    180                 }
    181             });
     162            GuiHelper.runInEDTAndWait(() -> reportProblemDialog(del,
     163                    trn("Object deleted", "Objects deleted", del.size()),
     164                    trn(
     165                        "One downloaded object is deleted.",
     166                        "{0} downloaded objects are deleted.",
     167                        del.size(),
     168                        del.size()),
     169                    null,
     170                    JOptionPane.WARNING_MESSAGE
     171            ).showDialog());
    182172    }
    183173
  • trunk/src/org/openstreetmap/josm/gui/io/SaveLayersDialog.java

    r10428 r10611  
    579579        @Override
    580580        public void run() {
    581             GuiHelper.runInEDTAndWait(new Runnable() {
    582                 @Override
    583                 public void run() {
    584                     model.setMode(SaveLayersModel.Mode.UPLOADING_AND_SAVING);
    585                     List<SaveLayerInfo> toUpload = model.getLayersToUpload();
    586                     if (!toUpload.isEmpty()) {
    587                         uploadLayers(toUpload);
    588                     }
    589                     List<SaveLayerInfo> toSave = model.getLayersToSave();
    590                     if (!toSave.isEmpty()) {
    591                         saveLayers(toSave);
    592                     }
    593                     model.setMode(SaveLayersModel.Mode.EDITING_DATA);
    594                     if (model.hasUnsavedData()) {
    595                         warnBecauseOfUnsavedData();
    596                         model.setMode(Mode.EDITING_DATA);
    597                         if (canceled) {
    598                             setUserAction(UserAction.CANCEL);
    599                             closeDialog();
    600                         }
    601                     } else {
    602                         setUserAction(UserAction.PROCEED);
     581            GuiHelper.runInEDTAndWait(() -> {
     582                model.setMode(SaveLayersModel.Mode.UPLOADING_AND_SAVING);
     583                List<SaveLayerInfo> toUpload = model.getLayersToUpload();
     584                if (!toUpload.isEmpty()) {
     585                    uploadLayers(toUpload);
     586                }
     587                List<SaveLayerInfo> toSave = model.getLayersToSave();
     588                if (!toSave.isEmpty()) {
     589                    saveLayers(toSave);
     590                }
     591                model.setMode(SaveLayersModel.Mode.EDITING_DATA);
     592                if (model.hasUnsavedData()) {
     593                    warnBecauseOfUnsavedData();
     594                    model.setMode(Mode.EDITING_DATA);
     595                    if (canceled) {
     596                        setUserAction(UserAction.CANCEL);
    603597                        closeDialog();
    604598                    }
     599                } else {
     600                    setUserAction(UserAction.PROCEED);
     601                    closeDialog();
    605602                }
    606603            });
  • trunk/src/org/openstreetmap/josm/gui/io/SaveLayersModel.java

    r10217 r10611  
    77import java.util.ArrayList;
    88import java.util.Collections;
    9 import java.util.Comparator;
    109import java.util.List;
    1110
     
    7271            layerInfo.add(new SaveLayerInfo(layer));
    7372        }
    74         Collections.sort(
    75                 layerInfo,
    76                 new Comparator<SaveLayerInfo>() {
    77                     @Override
    78                     public int compare(SaveLayerInfo o1, SaveLayerInfo o2) {
    79                         return o1.compareTo(o2);
    80                     }
    81                 }
    82         );
     73        Collections.sort(layerInfo, (o1, o2) -> o1.compareTo(o2));
    8374        fireTableDataChanged();
    8475    }
  • trunk/src/org/openstreetmap/josm/gui/io/UploadPrimitivesTask.java

    r10378 r10611  
    219219        // we always clean up the data, even in case of errors. It's possible the data was
    220220        // partially uploaded. Better run on EDT.
    221         //
    222         Runnable r = new Runnable() {
    223             @Override
    224             public void run() {
    225                 layer.cleanupAfterUpload(processedPrimitives);
    226                 layer.onPostUploadToServer();
    227                 ChangesetCache.getInstance().update(changeset);
    228             }
     221        Runnable r = () -> {
     222            layer.cleanupAfterUpload(processedPrimitives);
     223            layer.onPostUploadToServer();
     224            ChangesetCache.getInstance().update(changeset);
    229225        };
    230226
     
    320316        // - to the Upload Dialog
    321317        // - to map editing
    322         GuiHelper.runInEDT(new Runnable() {
    323             @Override
    324             public void run() {
    325                 // if the changeset is still open after this upload we want it to
    326                 // be selected on the next upload
    327                 //
    328                 ChangesetCache.getInstance().update(changeset);
    329                 if (changeset != null && changeset.isOpen()) {
    330                     UploadDialog.getUploadDialog().setSelectedChangesetForNextUpload(changeset);
    331                 }
    332                 if (uploadCanceled) return;
    333                 if (lastException == null) {
    334                     new Notification(
    335                             "<h3>" + tr("Upload successful!") + "</h3>")
    336                             .setIcon(ImageProvider.get("misc", "check_large"))
    337                             .show();
     318        GuiHelper.runInEDT(() -> {
     319            // if the changeset is still open after this upload we want it to be selected on the next upload
     320            ChangesetCache.getInstance().update(changeset);
     321            if (changeset != null && changeset.isOpen()) {
     322                UploadDialog.getUploadDialog().setSelectedChangesetForNextUpload(changeset);
     323            }
     324            if (uploadCanceled) return;
     325            if (lastException == null) {
     326                new Notification(
     327                        "<h3>" + tr("Upload successful!") + "</h3>")
     328                        .setIcon(ImageProvider.get("misc", "check_large"))
     329                        .show();
     330                return;
     331            }
     332            if (lastException instanceof ChangesetClosedException) {
     333                ChangesetClosedException e = (ChangesetClosedException) lastException;
     334                if (e.getSource().equals(ChangesetClosedException.Source.UPDATE_CHANGESET)) {
     335                    handleFailedUpload(lastException);
    338336                    return;
    339337                }
    340                 if (lastException instanceof ChangesetClosedException) {
    341                     ChangesetClosedException e = (ChangesetClosedException) lastException;
    342                     if (e.getSource().equals(ChangesetClosedException.Source.UPDATE_CHANGESET)) {
    343                         handleFailedUpload(lastException);
    344                         return;
    345                     }
    346                     if (strategy.getPolicy() == null)
    347                         /* do nothing if unknown policy */
    348                         return;
    349                     if (e.getSource().equals(ChangesetClosedException.Source.UPLOAD_DATA)) {
    350                         switch(strategy.getPolicy()) {
    351                         case ABORT:
    352                             break; /* do nothing - we return to map editing */
    353                         case AUTOMATICALLY_OPEN_NEW_CHANGESETS:
    354                             break; /* do nothing - we return to map editing */
    355                         case FILL_ONE_CHANGESET_AND_RETURN_TO_UPLOAD_DIALOG:
    356                             // return to the upload dialog
    357                             //
    358                             toUpload.removeProcessed(processedPrimitives);
    359                             UploadDialog.getUploadDialog().setUploadedPrimitives(toUpload);
    360                             UploadDialog.getUploadDialog().setVisible(true);
    361                             break;
    362                         }
    363                     } else {
    364                         handleFailedUpload(lastException);
     338                if (strategy.getPolicy() == null)
     339                    /* do nothing if unknown policy */
     340                    return;
     341                if (e.getSource().equals(ChangesetClosedException.Source.UPLOAD_DATA)) {
     342                    switch(strategy.getPolicy()) {
     343                    case ABORT:
     344                        break; /* do nothing - we return to map editing */
     345                    case AUTOMATICALLY_OPEN_NEW_CHANGESETS:
     346                        break; /* do nothing - we return to map editing */
     347                    case FILL_ONE_CHANGESET_AND_RETURN_TO_UPLOAD_DIALOG:
     348                        // return to the upload dialog
     349                        //
     350                        toUpload.removeProcessed(processedPrimitives);
     351                        UploadDialog.getUploadDialog().setUploadedPrimitives(toUpload);
     352                        UploadDialog.getUploadDialog().setVisible(true);
     353                        break;
    365354                    }
    366355                } else {
    367356                    handleFailedUpload(lastException);
    368357                }
     358            } else {
     359                handleFailedUpload(lastException);
    369360            }
    370361        });
Note: See TracChangeset for help on using the changeset viewer.