Changeset 3803 in josm


Ignore:
Timestamp:
Jan 22, 2011 7:27:33 PM (2 years ago)
Author:
bastiK
Message:

Don't merge all mappaitn style rules into one StyleSet, but keep them as separate StyleSources. This allows switching styles on and off at runtime.

Location:
trunk/src/org/openstreetmap/josm
Files:
7 edited

Legend:

Unmodified
Added
Removed

  • trunk/src/org/openstreetmap/josm/data/osm/visitor/paint/MapPaintVisitor.java

    r3800 r3803  
    3434import org.openstreetmap.josm.gui.mappaint.AreaElemStyle; 
    3535import org.openstreetmap.josm.gui.mappaint.ElemStyle; 
     36import org.openstreetmap.josm.gui.mappaint.ElemStyles; 
    3637import org.openstreetmap.josm.gui.mappaint.IconElemStyle; 
    3738import org.openstreetmap.josm.gui.mappaint.LineElemStyle; 
    3839import org.openstreetmap.josm.gui.mappaint.MapPaintStyles; 
    3940import org.openstreetmap.josm.gui.mappaint.SimpleNodeElemStyle; 
    40 import org.openstreetmap.josm.gui.mappaint.StyleSource; 
    4141 
    4242public class MapPaintVisitor implements PaintVisitor { 
    … …  
    4949    private boolean drawRestriction; 
    5050    private boolean leftHandTraffic; 
    51     private StyleSource styles; 
     51    private ElemStyles styles; 
    5252    private double circum; 
    5353    private double dist; 
    … …  
    535535        zoomLevelDisplay = Main.pref.getBoolean("mappaint.zoomLevelDisplay", false); 
    536536        circum = nc.getDist100Pixel(); 
    537         styles = MapPaintStyles.getStyles().getStyleSet(); 
     537        styles = MapPaintStyles.getStyles(); 
    538538        drawMultipolygon = Main.pref.getBoolean("mappaint.multipolygon", true); 
    539539        drawRestriction = Main.pref.getBoolean("mappaint.restriction", true); 

  • trunk/src/org/openstreetmap/josm/data/validation/tests/MultipolygonTest.java

    r3799 r3803  
    2424import org.openstreetmap.josm.gui.mappaint.AreaElemStyle; 
    2525import org.openstreetmap.josm.gui.mappaint.ElemStyle; 
     26import org.openstreetmap.josm.gui.mappaint.ElemStyles; 
    2627import org.openstreetmap.josm.gui.mappaint.MapPaintStyles; 
    27 import org.openstreetmap.josm.gui.mappaint.StyleSource; 
    2828 
    2929public class MultipolygonTest extends Test { 
    … …  
    4141    protected static final int NO_STYLE_POLYGON = 1611; 
    4242 
    43     private static StyleSource styles; 
     43    private static ElemStyles styles; 
    4444 
    4545    private final List<List<Node>> nonClosedWays = new ArrayList<List<Node>>(); 
    … …  
    5252    @Override 
    5353    public void initialize() throws Exception { 
    54         styles = MapPaintStyles.getStyles().getStyleSet(); 
     54        styles = MapPaintStyles.getStyles(); 
    5555    } 
    5656 

  • trunk/src/org/openstreetmap/josm/gui/mappaint/AreaElemStyle.java

    r3719 r3803  
    4545    } 
    4646 
    47     public ElemStyle getLineStyle() { 
     47    public LineElemStyle getLineStyle() { 
    4848        return line; 
    4949    } 

  • trunk/src/org/openstreetmap/josm/gui/mappaint/ElemStyleHandler.java

    r3719 r3803  
    1515public class ElemStyleHandler extends DefaultHandler 
    1616{ 
    17     boolean inDoc, inRule, inCondition, inLine, inLineMod, inIcon, inArea, inScaleMax, inScaleMin; 
    18     boolean hadLine, hadLineMod, hadIcon, hadArea; 
    19     ElemStyles styles; 
    20     String styleName; 
    21     RuleElem rule = new RuleElem(); 
     17    private boolean inDoc, inRule, inCondition, inLine, inLineMod, inIcon, inArea, inScaleMax, inScaleMin; 
     18    private boolean hadLine, hadLineMod, hadIcon, hadArea; 
     19    private RuleElem rule = new RuleElem(); 
     20 
     21    StyleSource style; 
    2222 
    2323    static class RuleElem { 
    … …  
    4343    } 
    4444 
    45     public ElemStyleHandler(String name) { 
    46         styleName = name; 
     45    public ElemStyleHandler(StyleSource style) { 
     46        this.style = style; 
    4747        inDoc=inRule=inCondition=inLine=inIcon=inArea=false; 
    4848        rule.init(); 
    49         styles = MapPaintStyles.getStyles(); 
    5049    } 
    5150 
    … …  
    5453        int i = colString.indexOf("#"); 
    5554        Color ret; 
    56         if(i < 0) { 
    57             ret = Main.pref.getColor("mappaint."+styleName+"."+colString, Color.red); 
     55        if (i < 0) { 
     56            ret = Main.pref.getColor("mappaint."+style.getPrefName()+"."+colString, Color.red); 
    5857        } else if(i == 0) { 
    5958            ret = ColorHelper.html2color(colString); 
    6059        } else { 
    61             ret = Main.pref.getColor("mappaint."+styleName+"."+colString.substring(0,i), 
     60            ret = Main.pref.getColor("mappaint."+style.getPrefName()+"."+colString.substring(0,i), 
    6261                    ColorHelper.html2color(colString.substring(i))); 
    6362        } 
    … …  
    7473 
    7574    private void error(String message) { 
    76         System.out.println(styleName + " (" + rule.rule.key + "=" + rule.rule.value + "): " + message); 
     75        System.out.println(style.getDisplayString() + " (" + rule.rule.key + "=" + rule.rule.value + "): " + message); 
    7776    } 
    7877 
    … …  
    141140            } else if (qName.equals("rules")) 
    142141            { 
    143                 if(styleName == null) 
    144                 { 
    145                     String n = atts.getValue("name"); 
    146                     if(n == null) { 
    147                         n = "standard"; 
    148                     } 
    149                     styleName = n; 
     142                if (style.name == null) { 
     143                    style.name = atts.getValue("name"); 
     144                } 
     145                if (style.shortdescription == null) { 
     146                    style.shortdescription = atts.getValue("shortdescription"); 
    150147                } 
    151148            } 
    … …  
    203200                { 
    204201                    if (atts.getQName(count).equals("src")) { 
    205                         ImageIcon icon = MapPaintStyles.getIcon(atts.getValue(count), styleName); 
     202                        ImageIcon icon = MapPaintStyles.getIcon(atts.getValue(count), style.getPrefName()); 
    206203                        hadIcon = (icon != null); 
    207204                        rule.icon.icon = icon; 
    … …  
    242239            if(hadLine) 
    243240            { 
    244                 styles.add(styleName, rule.rule, rule.rules, 
     241                style.add(rule.rule, rule.rules, 
    245242                        new LineElemStyle(rule.line, rule.scaleMax, rule.scaleMin)); 
    246243            } 
    247244            if(hadLineMod) 
    248245            { 
    249                 styles.addModifier(styleName, rule.rule, rule.rules, 
     246                style.addModifier(rule.rule, rule.rules, 
    250247                        new LineElemStyle(rule.linemod, rule.scaleMax, rule.scaleMin)); 
    251248            } 
    252249            if(hadIcon) 
    253250            { 
    254                 styles.add(styleName, rule.rule, rule.rules, 
     251                style.add(rule.rule, rule.rules, 
    255252                        new IconElemStyle(rule.icon, rule.scaleMax, rule.scaleMin)); 
    256253            } 
    257254            if(hadArea) 
    258255            { 
    259                 styles.add(styleName, rule.rule, rule.rules, 
     256                style.add(rule.rule, rule.rules, 
    260257                        new AreaElemStyle(rule.area, rule.scaleMax, rule.scaleMin)); 
    261258            } 

  • trunk/src/org/openstreetmap/josm/gui/mappaint/ElemStyles.java

    r3799 r3803  
    22package org.openstreetmap.josm.gui.mappaint; 
    33 
     4import static org.openstreetmap.josm.tools.Utils.equal; 
     5 
     6import java.util.ArrayList; 
    47import java.util.Collection; 
    5 import java.util.Collections; 
    6 import java.util.HashMap; 
    7 import java.util.Iterator; 
    8 import java.util.LinkedList; 
     8import java.util.HashSet; 
    99import java.util.List; 
     10import java.util.Set; 
    1011 
    1112import org.openstreetmap.josm.Main; 
    1213import org.openstreetmap.josm.data.osm.Node; 
    1314import org.openstreetmap.josm.data.osm.OsmPrimitive; 
    14 import org.openstreetmap.josm.data.osm.OsmUtils; 
    1515import org.openstreetmap.josm.data.osm.Way; 
     16import org.openstreetmap.josm.tools.FilteredCollection; 
     17import org.openstreetmap.josm.tools.Predicate; 
    1618 
    17 public class ElemStyles 
    18 { 
    19     HashMap<String, StyleSource> styleSet; 
     19public class ElemStyles { 
     20    private List<StyleSource> styleSources; 
     21 
    2022    public ElemStyles() 
    2123    { 
    22         styleSet = new HashMap<String, StyleSource>(); 
     24        styleSources = new ArrayList<StyleSource>(); 
    2325    } 
    2426 
    25     public void add(String name, Rule r, Collection<Rule> rules, LineElemStyle style) 
    26     { 
    27         if(rules != null) 
    28         { 
    29             style.rules = rules; 
    30             getStyleSet(name, true).linesList.add(style); 
    31         } 
    32         else 
    33         { 
    34             String key = r.getKey(); 
    35             style.code = key; 
    36             getStyleSet(name, true).lines.put(key, style); 
     27    public void add(StyleSource style) { 
     28        styleSources.add(style); 
     29    } 
     30 
     31    public Collection<StyleSource> getStyleSources() { 
     32        return new FilteredCollection<StyleSource>(styleSources, new Predicate<StyleSource>() { 
     33 
     34            String name = Main.pref.get("mappaint.style", "standard"); 
     35 
     36            @Override 
     37            public boolean evaluate(StyleSource s) { 
     38                return equal(s.getPrefName(), name); 
     39            } 
     40 
     41        }); 
     42    } 
     43 
     44    public ElemStyle get(OsmPrimitive osm) { 
     45        return get(osm, false); 
     46    } 
     47 
     48    public ElemStyle get(OsmPrimitive osm, boolean forceArea) { 
     49        if (!osm.hasKeys()) 
     50            return null; 
     51 
     52        if (osm instanceof Node) { 
     53            IconElemStyle icon = null; 
     54            for (StyleSource s : getStyleSources()) { 
     55                icon = s.getNode(osm, icon); 
     56            } 
     57            return icon; 
     58        } else { 
     59            boolean noclosed; 
     60            if (forceArea) { 
     61                noclosed = false; 
     62            } else { 
     63                noclosed = osm instanceof Way && !((Way) osm).isClosed(); 
     64            } 
     65            AreaElemStyle area = null; 
     66            LineElemStyle line = null; 
     67            ElemStyle result = null; 
     68            for (StyleSource s : getStyleSources()) { 
     69                result = s.get(osm, noclosed, area, line); 
     70                if (result instanceof LineElemStyle) { 
     71                    area = null; 
     72                    line = (LineElemStyle) result; 
     73                } else if (result instanceof AreaElemStyle) { 
     74                    area = (AreaElemStyle) result; 
     75                    if (area.getLineStyle() != null) { 
     76                        line = area.getLineStyle(); 
     77                    } 
     78                } else if (result != null) 
     79                    throw new AssertionError(); 
     80            } 
     81            return result; 
    3782        } 
    3883    } 
    3984 
    40     public void addModifier(String name, Rule r, Collection<Rule> rules, LineElemStyle style) 
    41     { 
    42         if(rules != null) 
    43         { 
    44             style.rules = rules; 
    45             getStyleSet(name, true).modifiersList.add(style); 
     85    public boolean hasAreas() { 
     86        for (StyleSource s : getStyleSources()) { 
     87            if (s.hasAreas()) 
     88                return true; 
    4689        } 
    47         else 
    48         { 
    49             String key = r.getKey(); 
    50             style.code = key; 
    51             getStyleSet(name, true).modifiers.put(key, style); 
    52         } 
     90        return false; 
    5391    } 
    5492 
    55     public void add(String name, Rule r, Collection<Rule> rules, AreaElemStyle style) 
    56     { 
    57         if(rules != null) 
    58         { 
    59             style.rules = rules; 
    60             getStyleSet(name, true).areasList.add(style); 
     93    public boolean isArea(OsmPrimitive osm) { 
     94        for (StyleSource s : getStyleSources()) { 
     95            if (s.isArea(osm)) 
     96                return true; 
    6197        } 
    62         else 
    63         { 
    64             String key = r.getKey(); 
    65             style.code = key; 
    66             getStyleSet(name, true).areas.put(key, style); 
    67         } 
     98        return false; 
    6899    } 
    69100 
    70     public void add(String name, Rule r, Collection<Rule> rules, IconElemStyle style) 
    71     { 
    72         if(rules != null) 
    73         { 
    74             style.rules = rules; 
    75             getStyleSet(name, true).iconsList.add(style); 
     101    public ElemStyle getArea(Way osm) { 
     102        if (osm.hasKeys()) { 
     103            /* force area mode also for unclosed ways */ 
     104            ElemStyle style = get(osm, true); 
     105            if (style != null && style instanceof AreaElemStyle) { 
     106                return style; 
     107            } 
    76108        } 
    77         else 
    78         { 
    79             String key = r.getKey(); 
    80             style.code = key; 
    81             getStyleSet(name, true).icons.put(key, style); 
    82         } 
     109        return null; 
    83110    } 
    84111 
    85     private StyleSource getStyleSet(String name, boolean create) 
    86     { 
    87         if(name == null) { 
    88             name = Main.pref.get("mappaint.style", "standard"); 
    89         } 
    90  
    91         StyleSource s = styleSet.get(name); 
    92         if(create && s == null) 
    93         { 
    94             s = new StyleSource(); 
    95             styleSet.put(name, s); 
    96         } 
    97         return s; 
     112    public IconElemStyle getIcon(OsmPrimitive osm) { 
     113        return osm.hasKeys() ? (IconElemStyle) get(osm) : null; 
    98114    } 
    99115 
    100     /* called from class users, never return null */ 
    101     public StyleSource getStyleSet() 
    102     { 
    103         return getStyleSet(null, true); 
    104     } 
    105  
    106     public Collection<String> getStyleNames() 
    107     { 
    108         return styleSet.keySet(); 
     116    public Collection<String> getStyleNames() { 
     117        Set<String> names = new HashSet<String>(); 
     118        names.add("standard"); 
     119        for (StyleSource s : styleSources) { 
     120            if (s.name != null) { 
     121                names.add(s.name); 
     122            } 
     123        } 
     124        return names; 
    109125    } 
    110126} 

  • trunk/src/org/openstreetmap/josm/gui/mappaint/MapPaintStyles.java

    r3796 r3803  
    7676 
    7777        for (SourceEntry entry : sourceEntries) { 
     78            StyleSource style = new StyleSource(entry); 
    7879            try { 
    79                 XmlObjectParser parser = new XmlObjectParser(new ElemStyleHandler(entry.name)); 
     80                XmlObjectParser parser = new XmlObjectParser(new ElemStyleHandler(style)); 
    8081                MirroredInputStream in = new MirroredInputStream(entry.url); 
    8182                InputStream zip = in.getZipEntry("xml","style"); 
    … …  
    9596                System.err.println(tr("Warning: failed to load Mappaint styles from ''{0}''. Exception was: {1}", entry.url, e.toString())); 
    9697                e.printStackTrace(); 
     98                style.hasError = true; 
    9799            } catch(SAXParseException e) { 
    98100                System.err.println(tr("Warning: failed to parse Mappaint styles from ''{0}''. Error was: [{1}:{2}] {3}", entry.url, e.getLineNumber(), e.getColumnNumber(), e.getMessage())); 
    99101                e.printStackTrace(); 
     102                style.hasError = true; 
    100103            } catch(SAXException e) { 
    101104                System.err.println(tr("Warning: failed to parse Mappaint styles from ''{0}''. Error was: {1}", entry.url, e.getMessage())); 
    102105                e.printStackTrace(); 
     106                style.hasError = true; 
    103107            } 
     108            styles.add(style); 
    104109        } 
    105110        iconDirs = null; 

  • trunk/src/org/openstreetmap/josm/gui/mappaint/StyleSource.java

    r3799 r3803  
    22package org.openstreetmap.josm.gui.mappaint; 
    33 
     4import java.util.Collection; 
    45import java.util.Collections; 
    56import java.util.HashMap; 
    … …  
    78import java.util.LinkedList; 
    89import java.util.List; 
     10import java.util.regex.Matcher; 
     11import java.util.regex.Pattern; 
    912import org.openstreetmap.josm.data.osm.Node; 
    1013import org.openstreetmap.josm.data.osm.OsmPrimitive; 
    1114import org.openstreetmap.josm.data.osm.OsmUtils; 
    1215import org.openstreetmap.josm.data.osm.Way; 
    13  
    14 public class StyleSource { 
    15  
    16     public HashMap<String, IconElemStyle> icons; 
    17     public HashMap<String, LineElemStyle> lines; 
    18     public HashMap<String, AreaElemStyle> areas; 
    19     public HashMap<String, LineElemStyle> modifiers; 
    20     public LinkedList<IconElemStyle> iconsList; 
    21     public LinkedList<LineElemStyle> linesList; 
    22     public LinkedList<AreaElemStyle> areasList; 
    23     public LinkedList<LineElemStyle> modifiersList; 
    24  
    25     public StyleSource() { 
    26         icons = new HashMap<String, IconElemStyle>(); 
    27         lines = new HashMap<String, LineElemStyle>(); 
    28         modifiers = new HashMap<String, LineElemStyle>(); 
    29         areas = new HashMap<String, AreaElemStyle>(); 
    30         iconsList = new LinkedList<IconElemStyle>(); 
    31         linesList = new LinkedList<LineElemStyle>(); 
    32         modifiersList = new LinkedList<LineElemStyle>(); 
    33         areasList = new LinkedList<AreaElemStyle>(); 
    34     } 
    35  
    36     private IconElemStyle getNode(OsmPrimitive primitive) { 
    37         IconElemStyle ret = null; 
     16import org.openstreetmap.josm.gui.preferences.SourceEntry; 
     17 
     18public class StyleSource extends SourceEntry { 
     19 
     20    public final HashMap<String, IconElemStyle> icons = new HashMap<String, IconElemStyle>(); 
     21    public final HashMap<String, LineElemStyle> lines = new HashMap<String, LineElemStyle>(); 
     22    public final HashMap<String, LineElemStyle> modifiers = new HashMap<String, LineElemStyle>(); 
     23    public final HashMap<String, AreaElemStyle> areas = new HashMap<String, AreaElemStyle>(); 
     24    public final LinkedList<IconElemStyle> iconsList = new LinkedList<IconElemStyle>(); 
     25    public final LinkedList<LineElemStyle> linesList = new LinkedList<LineElemStyle>(); 
     26    public final LinkedList<LineElemStyle> modifiersList = new LinkedList<LineElemStyle>(); 
     27    public final LinkedList<AreaElemStyle> areasList = new LinkedList<AreaElemStyle>(); 
     28 
     29    public boolean hasError = false; 
     30 
     31    public StyleSource(String url, String name, String shortdescription) { 
     32        super(url, name, shortdescription, true); 
     33    } 
     34 
     35    public StyleSource(SourceEntry entry) { 
     36        super(entry.url, entry.name, entry.shortdescription, entry.active); 
     37    } 
     38 
     39    public IconElemStyle getNode(OsmPrimitive primitive, IconElemStyle icon) { 
    3840        for (String key : primitive.keySet()) { 
    3941            String val = primitive.get(key); 
    4042            IconElemStyle style; 
    4143            if ((style = icons.get("n" + key + "=" + val)) != null) { 
    42                 if (ret == null || style.priority > ret.priority) { 
    43                     ret = style; 
     44                if (icon == null || style.priority > icon.priority) { 
     45                    icon = style; 
    4446                } 
    4547            } 
    4648            if ((style = icons.get("b" + key + "=" + OsmUtils.getNamedOsmBoolean(val))) != null) { 
    47                 if (ret == null || style.priority > ret.priority) { 
    48                     ret = style; 
     49                if (icon == null || style.priority > icon.priority) { 
     50                    icon = style; 
    4951                } 
    5052            } 
    5153            if ((style = icons.get("x" + key)) != null) { 
    52                 if (ret == null || style.priority > ret.priority) { 
    53                     ret = style; 
     54                if (icon == null || style.priority > icon.priority) { 
     55                    icon = style; 
    5456                } 
    5557            } 
    5658        } 
    5759        for (IconElemStyle s : iconsList) { 
    58             if ((ret == null || s.priority > ret.priority) && s.check(primitive)) { 
    59                 ret = s; 
    60             } 
    61         } 
    62         return ret; 
    63     } 
    64  
    65     private ElemStyle get(OsmPrimitive primitive, boolean noclosed) { 
    66         AreaElemStyle retArea = null; 
    67         LineElemStyle retLine = null; 
    68         String linestring = null; 
    69         HashMap<String, LineElemStyle> over = new HashMap<String, LineElemStyle>(); 
     60            if ((icon == null || s.priority > icon.priority) && s.check(primitive)) { 
     61                icon = s; 
     62            } 
     63        } 
     64        return icon; 
     65    } 
     66 
     67    public ElemStyle get(OsmPrimitive primitive, boolean noclosed, AreaElemStyle area, LineElemStyle line) { 
     68        String lineIdx = null; 
     69        HashMap<String, LineElemStyle> overlayMap = new HashMap<String, LineElemStyle>(); 
    7070        for (String key : primitive.keySet()) { 
    7171            String val = primitive.get(key); 
    … …  
    7373            LineElemStyle styleLine; 
    7474            String idx = "n" + key + "=" + val; 
    75             if ((styleArea = areas.get(idx)) != null && (retArea == null || styleArea.priority > retArea.priority) && (!noclosed || !styleArea.closed)) { 
    76                 retArea = styleArea; 
    77             } 
    78             if ((styleLine = lines.get(idx)) != null && (retLine == null || styleLine.priority > retLine.priority)) { 
    79                 retLine = styleLine; 
    80                 linestring = idx; 
     75            if ((styleArea = areas.get(idx)) != null && (area == null || styleArea.priority > area.priority) && (!noclosed || !styleArea.closed)) { 
     76                area = styleArea; 
     77            } 
     78            if ((styleLine = lines.get(idx)) != null && (line == null || styleLine.priority > line.priority)) { 
     79                line = styleLine; 
     80                lineIdx = idx; 
    8181            } 
    8282            if ((styleLine = modifiers.get(idx)) != null) { 
    83                 over.put(idx, styleLine); 
     83                overlayMap.put(idx, styleLine); 
    8484            } 
    8585            idx = "b" + key + "=" + OsmUtils.getNamedOsmBoolean(val); 
    86             if ((styleArea = areas.get(idx)) != null && (retArea == null || styleArea.priority > retArea.priority) && (!noclosed || !styleArea.closed)) { 
    87                 retArea = styleArea; 
    88             } 
    89             if ((styleLine = lines.get(idx)) != null && (retLine == null || styleLine.priority > retLine.priority)) { 
    90                 retLine = styleLine; 
    91                 linestring = idx; 
     86            if ((styleArea = areas.get(idx)) != null && (area == null || styleArea.priority > area.priority) && (!noclosed || !styleArea.closed)) { 
     87                area = styleArea; 
     88            } 
     89            if ((styleLine = lines.get(idx)) != null && (line == null || styleLine.priority > line.priority)) { 
     90                line = styleLine; 
     91                lineIdx = idx; 
    9292            } 
    9393            if ((styleLine = modifiers.get(idx)) != null) { 
    94                 over.put(idx, styleLine); 
     94                overlayMap.put(idx, styleLine); 
    9595            } 
    9696            idx = "x" + key; 
    97             if ((styleArea = areas.get(idx)) != null && (retArea == null || styleArea.priority > retArea.priority) && (!noclosed || !styleArea.closed)) { 
    98                 retArea = styleArea; 
    99             } 
    100             if ((styleLine = lines.get(idx)) != null && (retLine == null || styleLine.priority > retLine.priority)) { 
    101                 retLine = styleLine; 
    102                 linestring = idx; 
     97            if ((styleArea = areas.get(idx)) != null && (area == null || styleArea.priority > area.priority) && (!noclosed || !styleArea.closed)) { 
     98                area = styleArea; 
     99            } 
     100            if ((styleLine = lines.get(idx)) != null && (line == null || styleLine.priority > line.priority)) { 
     101                line = styleLine; 
     102                lineIdx = idx; 
    103103            } 
    104104            if ((styleLine = modifiers.get(idx)) != null) { 
    105                 over.put(idx, styleLine); 
     105                overlayMap.put(idx, styleLine); 
    106106            } 
    107107        } 
    108108        for (AreaElemStyle s : areasList) { 
    109             if ((retArea == null || s.priority > retArea.priority) && (!noclosed || !s.closed) && s.check(primitive)) { 
    110                 retArea = s; 
     109            if ((area == null || s.priority > area.priority) && (!noclosed || !s.closed) && s.check(primitive)) { 
     110                area = s; 
    111111            } 
    112112        } 
    113113        for (LineElemStyle s : linesList) { 
    114             if ((retLine == null || s.priority > retLine.priority) && s.check(primitive)) { 
    115                 retLine = s; 
     114            if ((line == null || s.priority > line.priority) && s.check(primitive)) { 
     115                line = s; 
    116116            } 
    117117        } 
    118118        for (LineElemStyle s : modifiersList) { 
    119119            if (s.check(primitive)) { 
    120                 over.put(s.getCode(), s); 
    121             } 
    122         } 
    123         over.remove(linestring); 
    124         if (over.size() != 0 && retLine != null) { 
    125             List<LineElemStyle> s = new LinkedList<LineElemStyle>(over.values()); 
    126             Collections.sort(s); 
    127             retLine = new LineElemStyle(retLine, s); 
    128         } 
    129         if (retArea != null) { 
    130             if (retLine != null) { 
    131                 return new AreaElemStyle(retArea, retLine); 
     120                overlayMap.put(s.getCode(), s); 
     121            } 
     122        } 
     123        overlayMap.remove(lineIdx); // do not use overlay if linestyle is from the same rule (example: railway=tram) 
     124        if (!overlayMap.isEmpty() && line != null) { 
     125            List<LineElemStyle> tmp = new LinkedList<LineElemStyle>(); 
     126            if (line.overlays != null) { 
     127                tmp.addAll(line.overlays); 
     128            } 
     129            tmp.addAll(overlayMap.values()); 
     130            Collections.sort(tmp); 
     131            line = new LineElemStyle(line, tmp); 
     132        } 
     133        if (area != null) { 
     134            if (line != null) { 
     135                return new AreaElemStyle(area, line); 
    132136            } else { 
    133                 return retArea; 
    134             } 
    135         } 
    136         return retLine; 
    137     } 
    138  
    139     public ElemStyle get(OsmPrimitive osm) { 
    140         return (!osm.hasKeys()) ? null : ((osm instanceof Node) ? getNode(osm) : get(osm, osm instanceof Way && !((Way) osm).isClosed())); 
    141     } 
    142  
    143     public ElemStyle getArea(Way osm) { 
    144         if (osm.hasKeys()) { 
    145             /* force area mode also for unclosed ways */ 
    146             ElemStyle style = get(osm, false); 
    147             if (style != null && style instanceof AreaElemStyle) { 
    148                 return style; 
    149             } 
    150         } 
    151         return null; 
    152     } 
    153  
    154     public IconElemStyle getIcon(OsmPrimitive osm) { 
    155         return osm.hasKeys() ? getNode(osm) : null; 
     137                return area; 
     138            } 
     139        } 
     140        return line; 
    156141    } 
    157142 
    … …  
    186171        return areas.size() > 0; 
    187172    } 
     173 
     174    public void add(Rule r, Collection<Rule> rules, LineElemStyle style) { 
     175        if(rules != null) 
     176        { 
     177            style.rules = rules; 
     178            linesList.add(style); 
     179        } 
     180        else { 
     181            String key = r.getKey(); 
     182            style.code = key; 
     183            lines.put(key, style); 
     184        } 
     185    } 
     186 
     187    public void addModifier(Rule r, Collection<Rule> rules, LineElemStyle style) { 
     188        if(rules != null) 
     189        { 
     190            style.rules = rules; 
     191            modifiersList.add(style); 
     192        } 
     193        else 
     194        { 
     195            String key = r.getKey(); 
     196            style.code = key; 
     197            modifiers.put(key, style); 
     198        } 
     199    } 
     200 
     201    public void add(Rule r, Collection<Rule> rules, AreaElemStyle style) { 
     202        if(rules != null) 
     203        { 
     204            style.rules = rules; 
     205            areasList.add(style); 
     206        } 
     207        else 
     208        { 
     209            String key = r.getKey(); 
     210            style.code = key; 
     211            areas.put(key, style); 
     212        } 
     213    } 
     214 
     215    public void add(Rule r, Collection<Rule> rules, IconElemStyle style) { 
     216        if(rules != null) 
     217        { 
     218            style.rules = rules; 
     219            iconsList.add(style); 
     220        } 
     221        else 
     222        { 
     223            String key = r.getKey(); 
     224            style.code = key; 
     225            icons.put(key, style); 
     226        } 
     227    } 
     228 
     229    /** 
     230     * the name / identifier that should be used to save custom color values 
     231     * and similar stuff to the preference file 
     232     * @return the identifier; never null. Usually the result is "standard" 
     233     */ 
     234    public String getPrefName() { 
     235        return name == null ? "standard" : name; 
     236    } 
     237 
     238    /** 
     239     * String to show in menus and error messages. 
     240     * @return Usually the shortdescription, but can be the file name 
     241     * if no shortdescription is available. 
     242     */ 
     243    public String getDisplayString() { 
     244        if (shortdescription != null) 
     245            return shortdescription; 
     246        /** 
     247         * extract file part from url, e.g.: 
     248         * http://www.test.com/file.xml?format=text  --> file.xml 
     249         */ 
     250        Pattern p = Pattern.compile("([^/\\\\]*?)([?].*)?$"); 
     251        Matcher m = p.matcher(url); 
     252        if (m.find()) { 
     253            return m.group(1); 
     254        } else { 
     255            System.err.println("Warning: Unexpected URL format: "+url); 
     256            return url; 
     257        } 
     258    } 
    188259} 
Note: See TracChangeset for help on using the changeset viewer.