Changeset 19258 in josm for trunk/src/org/openstreetmap

Timestamp:

2024-11-15T10:02:07+01:00 (29 hours ago)

Author:

stoecker

Message:

fix #24009 - don't call setting storage in settings setup stage or checkbox state gets lost

File:

• trunk/src/org/openstreetmap/josm/gui/download/OSMDownloadSource.java (modified) (2 diffs)

trunk/src/org/openstreetmap/josm/gui/download/OSMDownloadSource.java

@@ -197 +197 @@
         private final JPanel downloadSourcesPanel;
 
+        private boolean inRestore = false;
+
         private final ChangeListener checkboxChangeListener;
 
@@ -256 +258 @@
         @Override
         public void rememberSettings() {
-            DOWNLOAD_SOURCES.forEach(type -> type.getBooleanProperty().put(type.getCheckBox().isSelected()));
+            if (!inRestore)
+                DOWNLOAD_SOURCES.forEach(type -> type.getBooleanProperty().put(type.getCheckBox().isSelected()));
         }
 
         @Override
         public void restoreSettings() {
+            inRestore = true;
             updateSources();
             DOWNLOAD_SOURCES.forEach(type -> type.getCheckBox().setSelected(type.isEnabled()));
+            inRestore = false;
         }