Changeset 12999 in josm for trunk/src

Timestamp:

2017-10-15T12:26:33+02:00 (7 years ago)

Author:

bastiK

Message:

see #15229 - use Config.getPref() in *Property classes

Location:

trunk/src/org/openstreetmap/josm/data/preferences

Files:

• AbstractProperty.java (modified) (4 diffs)
• CollectionProperty.java (modified) (2 diffs)
• ColorProperty.java (modified) (2 diffs)

trunk/src/org/openstreetmap/josm/data/preferences/AbstractProperty.java

@@ -2 +2 @@
 package org.openstreetmap.josm.data.preferences;
 
-import org.openstreetmap.josm.Main;
-import org.openstreetmap.josm.data.Preferences;
+import org.openstreetmap.josm.spi.preferences.Config;
+import org.openstreetmap.josm.spi.preferences.IPreferences;
 import org.openstreetmap.josm.spi.preferences.PreferenceChangeEvent;
 import org.openstreetmap.josm.spi.preferences.PreferenceChangedListener;
@@ -152 +152 @@
      * The preferences object this property is for.
      */
-    protected final Preferences preferences;
+    protected final IPreferences preferences;
     protected final String key;
     protected final T defaultValue;
@@ -164 +164 @@
     public AbstractProperty(String key, T defaultValue) {
         // Main.pref should not change in production but may change during tests.
-        preferences = Main.pref;
+        preferences = Config.getPref();
         this.key = key;
         this.defaultValue = defaultValue;
@@ -227 +227 @@
      * Gets the preferences used for this property.
      * @return The preferences for this property.
-     * @since 10824
-     */
-    protected Preferences getPreferences() {
+     * @since 12999
+     */
+    protected IPreferences getPreferences() {
         return preferences;
     }

trunk/src/org/openstreetmap/josm/data/preferences/CollectionProperty.java

@@ -2 +2 @@
 package org.openstreetmap.josm.data.preferences;
 
+import java.util.ArrayList;
 import java.util.Collection;
 
@@ -27 +28 @@
     @Override
     public Collection<String> get() {
-        return getPreferences().getCollection(getKey(), getDefaultValue());
+        return getPreferences().getList(getKey(), new ArrayList<>(getDefaultValue()));
     }
 
     @Override
     public boolean put(Collection<String> value) {
-        return getPreferences().putCollection(getKey(), value);
+        return getPreferences().putList(getKey(), new ArrayList<>(value));
     }
 }

trunk/src/org/openstreetmap/josm/data/preferences/ColorProperty.java

@@ -5 +5 @@
 import java.util.Locale;
 
+import org.openstreetmap.josm.Main;
 import org.openstreetmap.josm.tools.CheckParameterUtil;
 import org.openstreetmap.josm.tools.ColorHelper;
@@ -36 +37 @@
         CheckParameterUtil.ensureParameterNotNull(defaultValue, "defaultValue");
         this.name = colName;
-        getPreferences().registerColor(getColorKey(colName), colName);
+        Main.pref.registerColor(getColorKey(colName), colName);
     }