Changeset 15989 in josm for trunk/src/org/openstreetmap/josm/gui/mappaint

Timestamp:

2020-03-01T23:36:09+01:00 (4 years ago)

Author:

simon04

Message:

see #18802 - MapCSSStyleIndex: rename/move

Location:

trunk/src/org/openstreetmap/josm/gui/mappaint/mapcss

Files:

• MapCSSStyleIndex.java (moved) (moved from trunk/src/org/openstreetmap/josm/data/validation/tests/MapCSSTagCheckerIndex.java ) (5 diffs)
• MapCSSStyleSource.java (modified) (2 diffs)

trunk/src/org/openstreetmap/josm/gui/mappaint/mapcss/MapCSSStyleIndex.java

@@ -1 +1 @@
 // License: GPL. For details, see LICENSE file.
-package org.openstreetmap.josm.data.validation.tests;
+package org.openstreetmap.josm.gui.mappaint.mapcss;
 
 import static org.openstreetmap.josm.tools.I18n.tr;
 
 import java.text.MessageFormat;
-import java.util.Collection;
-import java.util.HashMap;
 import java.util.Iterator;
 import java.util.Map;
@@ -17 +15 @@
 import org.openstreetmap.josm.data.osm.IWay;
 import org.openstreetmap.josm.data.osm.OsmUtils;
-import org.openstreetmap.josm.data.validation.Severity;
-import org.openstreetmap.josm.data.validation.tests.MapCSSTagChecker.TagCheck;
-import org.openstreetmap.josm.gui.mappaint.mapcss.Declaration;
-import org.openstreetmap.josm.gui.mappaint.mapcss.MapCSSRule;
 import org.openstreetmap.josm.gui.mappaint.mapcss.MapCSSStyleSource.MapCSSRuleIndex;
-import org.openstreetmap.josm.gui.mappaint.mapcss.Selector;
 import org.openstreetmap.josm.tools.JosmRuntimeException;
 import org.openstreetmap.josm.tools.Logging;
-import org.openstreetmap.josm.tools.MultiMap;
 
 /**
- * Helper class for {@link MapCSSTagChecker} to store indexes of rules
- * @author Gerd
- *
+ * Store indexes of {@link MapCSSRule}s using {@link MapCSSRuleIndex} differentiated by {@linkplain Selector#getBase() base}
  */
-public final class MapCSSTagCheckerIndex {
-    final Map<Declaration, TagCheck> ruleToCheckMap = new HashMap<>();
-
-    static final boolean ALL_TESTS = true;
-    static final boolean ONLY_SELECTED_TESTS = false;
+public final class MapCSSStyleIndex {
 
     /**
@@ -63 +49 @@
     final MapCSSRuleIndex canvasRules = new MapCSSRuleIndex();
 
-    static MapCSSTagCheckerIndex createMapCSSTagCheckerIndex(MultiMap<String, TagCheck> checks, boolean includeOtherSeverity, boolean allTests) {
-        final MapCSSTagCheckerIndex index = new MapCSSTagCheckerIndex();
-        final Stream<MapCSSRule> ruleStream = checks.values().stream()
-                .flatMap(Collection::stream)
-                // Ignore "information" level checks if not wanted, unless they also set a MapCSS class
-                .filter(c -> includeOtherSeverity || Severity.OTHER != c.getSeverity() || !c.setClassExpressions.isEmpty())
-                .filter(c -> allTests || c.rule.selectors.stream().anyMatch(Selector.ChildOrParentSelector.class::isInstance))
-                .peek(c -> index.ruleToCheckMap.put(c.rule.declaration, c))
-                .map(c -> c.rule);
-        index.buildIndex(ruleStream);
-        return index;
-    }
-
     /**
      * Clear the index.
@@ -82 +55 @@
      */
     public void clear() {
-        ruleToCheckMap.clear();
         nodeRules.clear();
         wayRules.clear();
@@ -192 +164 @@
         return get(osm).getRuleCandidates(osm);
     }
-
-    /**
-     * return the TagCheck for which the given indexed rule was created.
-     * @param rule an indexed rule
-     * @return the original TagCheck
-     */
-    public TagCheck getCheck(MapCSSRule rule) {
-        return ruleToCheckMap.get(rule.declaration);
-    }
 }

trunk/src/org/openstreetmap/josm/gui/mappaint/mapcss/MapCSSStyleSource.java

@@ -39 +39 @@
 import org.openstreetmap.josm.data.osm.Tagged;
 import org.openstreetmap.josm.data.preferences.sources.SourceEntry;
-import org.openstreetmap.josm.data.validation.tests.MapCSSTagCheckerIndex;
 import org.openstreetmap.josm.gui.mappaint.Cascade;
 import org.openstreetmap.josm.gui.mappaint.Environment;
@@ -86 +85 @@
      * Index of rules in this style file
      */
-    private final MapCSSTagCheckerIndex ruleIndex = new MapCSSTagCheckerIndex();
+    private final MapCSSStyleIndex ruleIndex = new MapCSSStyleIndex();
 
     private Color backgroundColorOverride;