Changeset 18367 in josm

Timestamp:

2022-01-27T23:43:27+01:00 (3 years ago)

Author:

taylor.smock

Message:

fix #21797: IOOBE in ImageryProvidersPanel$ImageryDefaultLayerTableModel.getValueAt

This was caused by a race condition in newer JVM's when the cancel button is
pressed. This splits out the code forcing a refresh of the tables into its own
method, and calls that method in the worker thread after we finish
redownloading the imagery list.

File:

• trunk/src/org/openstreetmap/josm/gui/preferences/imagery/ImageryProvidersPanel.java (modified) (1 diff)

trunk/src/org/openstreetmap/josm/gui/preferences/imagery/ImageryProvidersPanel.java

@@ -602 +602 @@
         public void actionPerformed(ActionEvent evt) {
             layerInfo.loadDefaults(true, MainApplication.worker, false);
+            this.updateTableArea();
+            // Ensure that the table is updated properly if someone cancels the download. See JOSM #21797.
+            MainApplication.worker.execute(() -> GuiHelper.runInEDTAndWait(this::updateTableArea));
+        }
+
+        private void updateTableArea() {
             defaultModel.fireTableDataChanged();
             defaultTable.getSelectionModel().clearSelection();