Ignore:
Timestamp:
2017-08-25T21:51:00+02:00 (2 years ago)
Author:
Don-vip
Message:

see #15182 - code refactoring to avoid dependence on GUI packages from Preferences

Location:
trunk/src/org/openstreetmap/josm/data
Files:
7 added
5 edited
3 moved

Legend:

Unmodified
Added
Removed
  • trunk/src/org/openstreetmap/josm/data/Preferences.java

    r12634 r12649  
    6868import org.openstreetmap.josm.data.preferences.Setting;
    6969import org.openstreetmap.josm.data.preferences.StringSetting;
    70 import org.openstreetmap.josm.gui.preferences.SourceEditor.ExtendedSourceEntry;
    71 import org.openstreetmap.josm.gui.preferences.validator.ValidatorTagCheckerRulesPreference;
    72 import org.openstreetmap.josm.gui.preferences.validator.ValidatorTagCheckerRulesPreference.RulePrefHelper;
     70import org.openstreetmap.josm.data.preferences.sources.ExtendedSourceEntry;
     71import org.openstreetmap.josm.data.preferences.sources.ValidatorPrefHelper;
    7372import org.openstreetmap.josm.io.OfflineAccessException;
    7473import org.openstreetmap.josm.io.OnlineResource;
     
    17141713            List<Map<String, String>> l = new ArrayList<>(((MapListSetting) setting).getValue());
    17151714            if (l.stream().noneMatch(x -> x.containsValue(url))) {
    1716                 RulePrefHelper helper = ValidatorTagCheckerRulesPreference.RulePrefHelper.INSTANCE;
     1715                ValidatorPrefHelper helper = ValidatorPrefHelper.INSTANCE;
    17171716                Optional<ExtendedSourceEntry> val = helper.getDefault().stream().filter(x -> url.equals(x.url)).findFirst();
    17181717                if (val.isPresent()) {
  • trunk/src/org/openstreetmap/josm/data/preferences/sources/SourceEntry.java

    r12648 r12649  
    11// License: GPL. For details, see LICENSE file.
    2 package org.openstreetmap.josm.gui.preferences;
     2package org.openstreetmap.josm.data.preferences.sources;
    33
    44import java.io.File;
     
    1313 * A source entry primarily used to save the user's selection of mappaint styles,
    1414 * but also for preset sources or validator rules.
     15 * @since 12649 (moved from gui.preferences package)
    1516 * @since 3796
    1617 */
  • trunk/src/org/openstreetmap/josm/data/preferences/sources/SourceProvider.java

    r12648 r12649  
    11// License: GPL. For details, see LICENSE file.
    2 package org.openstreetmap.josm.gui.preferences;
     2package org.openstreetmap.josm.data.preferences.sources;
    33
    44import java.util.Collection;
     
    88 *
    99 * Used by plugins to offer additional SourceEntrys to the user.
     10 * @since 12649 (moved from gui.preferences package)
    1011 */
    1112@FunctionalInterface
  • trunk/src/org/openstreetmap/josm/data/preferences/sources/SourceType.java

    r12648 r12649  
    11// License: GPL. For details, see LICENSE file.
    2 package org.openstreetmap.josm.gui.preferences;
     2package org.openstreetmap.josm.data.preferences.sources;
    33
    44/**
    55 * The different types of source entries.
     6 * @since 12649 (moved from gui.preferences package)
    67 * @since 6670
    78 */
  • trunk/src/org/openstreetmap/josm/data/validation/OsmValidator.java

    r12636 r12649  
    2626
    2727import org.openstreetmap.josm.Main;
     28import org.openstreetmap.josm.data.preferences.sources.ValidatorPrefHelper;
    2829import org.openstreetmap.josm.data.validation.tests.Addresses;
    2930import org.openstreetmap.josm.data.validation.tests.ApiCapabilitiesTest;
     
    6162import org.openstreetmap.josm.gui.layer.ValidatorLayer;
    6263import org.openstreetmap.josm.gui.preferences.projection.ProjectionPreference;
    63 import org.openstreetmap.josm.gui.preferences.validator.ValidatorPreference;
    6464import org.openstreetmap.josm.tools.Logging;
    6565import org.openstreetmap.josm.tools.Utils;
     
    185185    private static void loadIgnoredErrors() {
    186186        ignoredErrors.clear();
    187         if (ValidatorPreference.PREF_USE_IGNORE.get()) {
     187        if (ValidatorPrefHelper.PREF_USE_IGNORE.get()) {
    188188            Path path = Paths.get(getValidatorDir()).resolve("ignorederrors");
    189189            if (path.toFile().exists()) {
     
    235235     */
    236236    public static synchronized void initializeErrorLayer() {
    237         if (!ValidatorPreference.PREF_LAYER.get())
     237        if (!ValidatorPrefHelper.PREF_LAYER.get())
    238238            return;
    239239        if (errorLayer == null) {
     
    278278    private static void applyPrefs(Map<String, Test> tests, boolean beforeUpload) {
    279279        for (String testName : Main.pref.getCollection(beforeUpload
    280         ? ValidatorPreference.PREF_SKIP_TESTS_BEFORE_UPLOAD : ValidatorPreference.PREF_SKIP_TESTS)) {
     280        ? ValidatorPrefHelper.PREF_SKIP_TESTS_BEFORE_UPLOAD : ValidatorPrefHelper.PREF_SKIP_TESTS)) {
    281281            Test test = tests.get(testName);
    282282            if (test != null) {
  • trunk/src/org/openstreetmap/josm/data/validation/tests/MapCSSTagChecker.java

    r12620 r12649  
    4040import org.openstreetmap.josm.data.osm.OsmUtils;
    4141import org.openstreetmap.josm.data.osm.Tag;
     42import org.openstreetmap.josm.data.preferences.sources.SourceEntry;
     43import org.openstreetmap.josm.data.preferences.sources.ValidatorPrefHelper;
    4244import org.openstreetmap.josm.data.validation.Severity;
    4345import org.openstreetmap.josm.data.validation.Test;
     
    5860import org.openstreetmap.josm.gui.mappaint.mapcss.parsergen.MapCSSParser;
    5961import org.openstreetmap.josm.gui.mappaint.mapcss.parsergen.ParseException;
    60 import org.openstreetmap.josm.gui.preferences.SourceEntry;
    61 import org.openstreetmap.josm.gui.preferences.validator.ValidatorPreference;
    62 import org.openstreetmap.josm.gui.preferences.validator.ValidatorTagCheckerRulesPreference;
    6362import org.openstreetmap.josm.io.CachedFile;
    6463import org.openstreetmap.josm.io.IllegalDataException;
     
    714713    @Override
    715714    public void check(OsmPrimitive p) {
    716         errors.addAll(getErrorsForPrimitive(p, ValidatorPreference.PREF_OTHER.get()));
     715        errors.addAll(getErrorsForPrimitive(p, ValidatorPrefHelper.PREF_OTHER.get()));
    717716    }
    718717
     
    748747    public synchronized void initialize() throws Exception {
    749748        checks.clear();
    750         for (SourceEntry source : new ValidatorTagCheckerRulesPreference.RulePrefHelper().get()) {
     749        for (SourceEntry source : new ValidatorPrefHelper().get()) {
    751750            if (!source.active) {
    752751                continue;
  • trunk/src/org/openstreetmap/josm/data/validation/tests/TagChecker.java

    r12620 r12649  
    3636import org.openstreetmap.josm.data.osm.OsmUtils;
    3737import org.openstreetmap.josm.data.osm.Tag;
     38import org.openstreetmap.josm.data.preferences.sources.ValidatorPrefHelper;
    3839import org.openstreetmap.josm.data.validation.Severity;
    3940import org.openstreetmap.josm.data.validation.Test.TagTest;
    4041import org.openstreetmap.josm.data.validation.TestError;
    4142import org.openstreetmap.josm.data.validation.util.Entities;
    42 import org.openstreetmap.josm.gui.preferences.validator.ValidatorPreference;
    4343import org.openstreetmap.josm.gui.progress.ProgressMonitor;
    4444import org.openstreetmap.josm.gui.tagging.presets.TaggingPreset;
     
    8080
    8181    /** The preferences prefix */
    82     protected static final String PREFIX = ValidatorPreference.PREFIX + "." + TagChecker.class.getSimpleName();
     82    protected static final String PREFIX = ValidatorPrefHelper.PREFIX + "." + TagChecker.class.getSimpleName();
    8383
    8484    /**
     
    293293            }
    294294            // TODO directionKeys are no longer in OsmPrimitive (search pattern is used instead)
    295             for (String a : Main.pref.getCollection(ValidatorPreference.PREFIX + ".knownkeys",
     295            for (String a : Main.pref.getCollection(ValidatorPrefHelper.PREFIX + ".knownkeys",
    296296                    Arrays.asList("is_in", "int_ref", "fixme", "population"))) {
    297297                additionalPresetsValueData.putVoid(a);
  • trunk/src/org/openstreetmap/josm/data/validation/tests/UnconnectedWays.java

    r12636 r12649  
    2727import org.openstreetmap.josm.data.osm.QuadBuckets;
    2828import org.openstreetmap.josm.data.osm.Way;
     29import org.openstreetmap.josm.data.preferences.sources.ValidatorPrefHelper;
    2930import org.openstreetmap.josm.data.projection.Ellipsoid;
    3031import org.openstreetmap.josm.data.validation.Severity;
     
    3233import org.openstreetmap.josm.data.validation.TestError;
    3334import org.openstreetmap.josm.gui.MainApplication;
    34 import org.openstreetmap.josm.gui.preferences.validator.ValidatorPreference;
    3535import org.openstreetmap.josm.gui.progress.ProgressMonitor;
    3636import org.openstreetmap.josm.tools.Logging;
     
    138138
    139139    protected static final int UNCONNECTED_WAYS = 1301;
    140     protected static final String PREFIX = ValidatorPreference.PREFIX + "." + UnconnectedWays.class.getSimpleName();
     140    protected static final String PREFIX = ValidatorPrefHelper.PREFIX + "." + UnconnectedWays.class.getSimpleName();
    141141
    142142    private Set<MyWaySegment> ways;
Note: See TracChangeset for help on using the changeset viewer.