Changeset 11913 in josm for trunk/src


Ignore:
Timestamp:
2017-04-15T01:38:46+02:00 (8 years ago)
Author:
Don-vip
Message:

sonar - squid:S1192 - String literals should not be duplicated

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

Legend:

Unmodified
Added
Removed

  • trunk/src/org/openstreetmap/josm/actions/mapmode/ImproveWayAccuracyAction.java

    r11713 r11913  
    5757        SelectionChangedListener, ModifierListener {
    5858
     59    private static final String CROSSHAIR = "crosshair";
     60
    5961    private enum State {
    6062        SELECTING, IMPROVING
    … …  
    7678    private final Cursor cursorSelect = ImageProvider.getCursor("normal", "mode");
    7779    private final Cursor cursorSelectHover = ImageProvider.getCursor("hand", "mode");
    78     private final Cursor cursorImprove = ImageProvider.getCursor("crosshair", null);
    79     private final Cursor cursorImproveAdd = ImageProvider.getCursor("crosshair", "addnode");
    80     private final Cursor cursorImproveDelete = ImageProvider.getCursor("crosshair", "delete_node");
    81     private final Cursor cursorImproveAddLock = ImageProvider.getCursor("crosshair", "add_node_lock");
    82     private final Cursor cursorImproveLock = ImageProvider.getCursor("crosshair", "lock");
     80    private final Cursor cursorImprove = ImageProvider.getCursor(CROSSHAIR, null);
     81    private final Cursor cursorImproveAdd = ImageProvider.getCursor(CROSSHAIR, "addnode");
     82    private final Cursor cursorImproveDelete = ImageProvider.getCursor(CROSSHAIR, "delete_node");
     83    private final Cursor cursorImproveAddLock = ImageProvider.getCursor(CROSSHAIR, "add_node_lock");
     84    private final Cursor cursorImproveLock = ImageProvider.getCursor(CROSSHAIR, "lock");
    8385
    8486    private Color guideColor;

  • trunk/src/org/openstreetmap/josm/actions/mapmode/SelectAction.java

    r11713 r11913  
    6868public class SelectAction extends MapMode implements ModifierListener, KeyPressReleaseListener, SelectionEnded {
    6969
     70    private static final String NORMAL = "normal";
     71
    7072    /**
    7173     * Select action mode.
    … …  
    8587    // contains all possible cases the cursor can be in the SelectAction
    8688    private enum SelectActionCursor {
    87         rect("normal", /* ICON(cursor/modifier/) */ "selection"),
    88         rect_add("normal", /* ICON(cursor/modifier/) */ "select_add"),
    89         rect_rm("normal", /* ICON(cursor/modifier/) */ "select_remove"),
    90         way("normal", /* ICON(cursor/modifier/) */ "select_way"),
    91         way_add("normal", /* ICON(cursor/modifier/) */ "select_way_add"),
    92         way_rm("normal", /* ICON(cursor/modifier/) */ "select_way_remove"),
    93         node("normal", /* ICON(cursor/modifier/) */ "select_node"),
    94         node_add("normal", /* ICON(cursor/modifier/) */ "select_node_add"),
    95         node_rm("normal", /* ICON(cursor/modifier/) */ "select_node_remove"),
    96         virtual_node("normal", /* ICON(cursor/modifier/) */ "addnode"),
    97         scale(/* ICON(cursor/) */ "scale", null),
    98         rotate(/* ICON(cursor/) */ "rotate", null),
    99         merge(/* ICON(cursor/) */ "crosshair", null),
    100         lasso("normal", /* ICON(cursor/modifier/) */ "rope"),
    101         merge_to_node("crosshair", /* ICON(cursor/modifier/) */ "joinnode"),
     89
     90        rect(NORMAL, "selection"),
     91        rect_add(NORMAL, "select_add"),
     92        rect_rm(NORMAL, "select_remove"),
     93        way(NORMAL, "select_way"),
     94        way_add(NORMAL, "select_way_add"),
     95        way_rm(NORMAL, "select_way_remove"),
     96        node(NORMAL, "select_node"),
     97        node_add(NORMAL, "select_node_add"),
     98        node_rm(NORMAL, "select_node_remove"),
     99        virtual_node(NORMAL, "addnode"),
     100        scale("scale", null),
     101        rotate("rotate", null),
     102        merge("crosshair", null),
     103        lasso(NORMAL, "rope"),
     104        merge_to_node("crosshair", "joinnode"),
    102105        move(Cursor.MOVE_CURSOR);
    103106

  • trunk/src/org/openstreetmap/josm/actions/upload/FixDataHook.java

    r10601 r11913  
    2727public class FixDataHook implements UploadHook {
    2828
     29    private static final String ONEWAY = "oneway";
     30
    2931    /**
    3032     * List of checks to run on data
    … …  
    4042        deprecated.add(new FixDataKey("color",            "colour"));
    4143        deprecated.add(new FixDataTag("highway", "ford",  "ford",    "yes"));
    42         deprecated.add(new FixDataTag("oneway",  "false", "oneway",  "no"));
    43         deprecated.add(new FixDataTag("oneway",  "0",     "oneway",  "no"));
    44         deprecated.add(new FixDataTag("oneway",  "true",  "oneway",  "yes"));
    45         deprecated.add(new FixDataTag("oneway",  "1",     "oneway",  "yes"));
     44        deprecated.add(new FixDataTag(ONEWAY,    "false",  ONEWAY,   "no"));
     45        deprecated.add(new FixDataTag(ONEWAY,    "0",      ONEWAY,   "no"));
     46        deprecated.add(new FixDataTag(ONEWAY,    "true",   ONEWAY,   "yes"));
     47        deprecated.add(new FixDataTag(ONEWAY,    "1",      ONEWAY,   "yes"));
    4648        deprecated.add(new FixDataTag("highway", "stile", "barrier", "stile"));
    4749        // CHECKSTYLE.ON: SingleSpaceSeparator

  • trunk/src/org/openstreetmap/josm/data/Preferences.java

    r11746 r11913  
    106106public class Preferences {
    107107
     108    private static final String COLOR_PREFIX = "color.";
     109
    108110    private static final String[] OBSOLETE_PREF_KEYS = {
    109111      "hdop.factor", /* remove entry after April 2017 */
    … …  
    501503        final Map<String, String> all = new TreeMap<>();
    502504        for (final Entry<String, Setting<?>> e : defaultsMap.entrySet()) {
    503             if (e.getKey().startsWith("color.") && e.getValue() instanceof StringSetting) {
     505            if (e.getKey().startsWith(COLOR_PREFIX) && e.getValue() instanceof StringSetting) {
    504506                StringSetting d = (StringSetting) e.getValue();
    505507                if (d.getValue() != null) {
    … …  
    509511        }
    510512        for (final Entry<String, Setting<?>> e : settingsMap.entrySet()) {
    511             if (e.getKey().startsWith("color.") && (e.getValue() instanceof StringSetting)) {
     513            if (e.getKey().startsWith(COLOR_PREFIX) && (e.getValue() instanceof StringSetting)) {
    512514                all.put(e.getKey().substring(6), ((StringSetting) e.getValue()).getValue());
    513515            }
    … …  
    864866        String colKey = ColorProperty.getColorKey(colName);
    865867        registerColor(colKey, colName);
    866         String colStr = specName != null ? get("color."+specName) : "";
     868        String colStr = specName != null ? get(COLOR_PREFIX+specName) : "";
    867869        if (colStr.isEmpty()) {
    868870            colStr = get(colKey, ColorHelper.color2html(def, true));
    … …  
    888890
    889891    public synchronized Color getDefaultColor(String colKey) {
    890         StringSetting col = Utils.cast(defaultsMap.get("color."+colKey), StringSetting.class);
     892        StringSetting col = Utils.cast(defaultsMap.get(COLOR_PREFIX+colKey), StringSetting.class);
    891893        String colStr = col == null ? null : col.getValue();
    892894        return colStr == null || colStr.isEmpty() ? null : ColorHelper.html2color(colStr);
    … …  
    894896
    895897    public synchronized boolean putColor(String colKey, Color val) {
    896         return put("color."+colKey, val != null ? ColorHelper.color2html(val, true) : null);
     898        return put(COLOR_PREFIX+colKey, val != null ? ColorHelper.color2html(val, true) : null);
    897899    }
    898900
    … …  
    15321534    private void migrateOldColorKeys() {
    15331535        settingsMap.keySet().stream()
    1534                 .filter(key -> key.startsWith("color."))
     1536                .filter(key -> key.startsWith(COLOR_PREFIX))
    15351537                .flatMap(key -> {
    1536                     final String newKey = ColorProperty.getColorKey(key.substring("color.".length()));
     1538                    final String newKey = ColorProperty.getColorKey(key.substring(COLOR_PREFIX.length()));
    15371539                    return key.equals(newKey) || settingsMap.containsKey(newKey)
    15381540                            ? Stream.empty()

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

    r11893 r11913  
    3232    protected static final int CROSSING_WAYS = 601;
    3333
    34     private static final String HIGHWAY = "highway";
    35     private static final String RAILWAY = "railway";
    36     private static final String WATERWAY = "waterway";
     34    static final String HIGHWAY = "highway";
     35    static final String RAILWAY = "railway";
     36    static final String WATERWAY = "waterway";
    3737
    3838    /** All way segments, grouped by cells */

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

    r11893 r11913  
    22package org.openstreetmap.josm.data.validation.tests;
    33
     4import static org.openstreetmap.josm.data.validation.tests.CrossingWays.HIGHWAY;
     5import static org.openstreetmap.josm.data.validation.tests.CrossingWays.RAILWAY;
     6import static org.openstreetmap.josm.data.validation.tests.CrossingWays.WATERWAY;
    47import static org.openstreetmap.josm.tools.I18n.tr;
    58
    … …  
    99102
    100103    private static final String[] TYPES = {
    101             "none", "highway", "railway", "waterway", "boundary", "power", "natural", "landuse", "building"};
     104            "none", HIGHWAY, RAILWAY, WATERWAY, "boundary", "power", "natural", "landuse", "building"};
    102105
    103106    /** The map of potential duplicates.
    … …  
    210213                            .primitives(primitives)
    211214                            .build());
    212                 } else if (typeMap.get("highway")) {
     215                } else if (typeMap.get(HIGHWAY)) {
    213216                    errors.add(TestError.builder(parentTest, Severity.ERROR, DUPLICATE_NODE_HIGHWAY)
    214217                            .message(tr("Highway duplicated nodes"))
    215218                            .primitives(primitives)
    216219                            .build());
    217                 } else if (typeMap.get("railway")) {
     220                } else if (typeMap.get(RAILWAY)) {
    218221                    errors.add(TestError.builder(parentTest, Severity.ERROR, DUPLICATE_NODE_RAILWAY)
    219222                            .message(tr("Railway duplicated nodes"))
    220223                            .primitives(primitives)
    221224                            .build());
    222                 } else if (typeMap.get("waterway")) {
     225                } else if (typeMap.get(WATERWAY)) {
    223226                    errors.add(TestError.builder(parentTest, Severity.ERROR, DUPLICATE_NODE_WATERWAY)
    224227                            .message(tr("Waterway duplicated nodes"))

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

    r11807 r11913  
    22package org.openstreetmap.josm.data.validation.tests;
    33
     4import static org.openstreetmap.josm.data.validation.tests.CrossingWays.HIGHWAY;
    45import static org.openstreetmap.josm.tools.I18n.tr;
    56
    … …  
    7778        if (n.isUsable()) {
    7879            if (!n.hasTag("crossing", "no")
    79              && !(n.hasKey("crossing") && (n.hasTag("highway", "crossing") || n.hasTag("highway", "traffic_signals")))
     80             && !(n.hasKey("crossing") && (n.hasTag(HIGHWAY, "crossing")
     81                                        || n.hasTag(HIGHWAY, "traffic_signals")))
    8082             && n.isReferredByWays(2)) {
    8183                testMissingPedestrianCrossing(n);
    … …  
    9294    public void visit(Way w) {
    9395        if (w.isUsable()) {
    94             if (w.isClosed() && w.hasTag("highway", CLASSIFIED_HIGHWAYS) && w.hasTag("junction", "roundabout")) {
     96            if (w.isClosed() && w.hasTag(HIGHWAY, CLASSIFIED_HIGHWAYS) && w.hasTag("junction", "roundabout")) {
    9597                // TODO: find out how to handle splitted roundabouts (see #12841)
    9698                testWrongRoundabout(w);
    … …  
    110112        for (Node n : new HashSet<>(w.getNodes())) {
    111113            for (Way h : Utils.filteredCollection(n.getReferrers(), Way.class)) {
    112                 String value = h.get("highway");
     114                String value = h.get(HIGHWAY);
    113115                if (h != w && value != null && !value.endsWith("_link")) {
    114116                    List<Way> list = map.get(value);
    … …  
    130132                if (list.size() > 2 || oneway1 == null || oneway2 == null || !oneway1 || !oneway2) {
    131133                    // Error when the highway tags do not match
    132                     String value = w.get("highway");
     134                    String value = w.get(HIGHWAY);
    133135                    if (!value.equals(s)) {
    134136                        errors.add(TestError.builder(this, Severity.WARNING, WRONG_ROUNDABOUT_HIGHWAY)
    135137                                .message(tr("Incorrect roundabout (highway: {0} instead of {1})", value, s))
    136138                                .primitives(w)
    137                                 .fix(() -> new ChangePropertyCommand(w, "highway", s))
     139                                .fix(() -> new ChangePropertyCommand(w, HIGHWAY, s))
    138140                                .build());
    139141                    }
    … …  
    145147
    146148    public static boolean isHighwayLinkOkay(final Way way) {
    147         final String highway = way.get("highway");
     149        final String highway = way.get(HIGHWAY);
    148150        if (highway == null || !highway.endsWith("_link")
    149151                || !IN_DOWNLOADED_AREA.test(way.getNode(0)) || !IN_DOWNLOADED_AREA.test(way.getNode(way.getNodesCount()-1))) {
    … …  
    164166
    165167        return Utils.filteredCollection(referrers, Way.class).stream().anyMatch(
    166                 otherWay -> !way.equals(otherWay) && otherWay.hasTag("highway", highway, highway.replaceAll("_link$", "")));
     168                otherWay -> !way.equals(otherWay) && otherWay.hasTag(HIGHWAY, highway, highway.replaceAll("_link$", "")));
    167169    }
    168170
    … …  
    185187
    186188        for (Way w : OsmPrimitive.getFilteredList(n.getReferrers(), Way.class)) {
    187             String highway = w.get("highway");
     189            String highway = w.get(HIGHWAY);
    188190            if (highway != null) {
    189191                if ("footway".equals(highway) || "path".equals(highway)) {

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

    r11608 r11913  
    22package org.openstreetmap.josm.data.validation.tests;
    33
     4import static org.openstreetmap.josm.data.validation.tests.CrossingWays.HIGHWAY;
     5import static org.openstreetmap.josm.data.validation.tests.CrossingWays.RAILWAY;
    46import static org.openstreetmap.josm.tools.I18n.tr;
    57
    … …  
    9092
    9193                for (WaySegment ws : duplicated) {
    92                     if (ws.way.hasKey("highway")) {
     94                    if (ws.way.hasKey(HIGHWAY)) {
    9395                        highway++;
    94                     } else if (ws.way.hasKey("railway")) {
     96                    } else if (ws.way.hasKey(RAILWAY)) {
    9597                        railway++;
    9698                    }

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

    r11621 r11913  
    22package org.openstreetmap.josm.data.validation.tests;
    33
     4import static org.openstreetmap.josm.data.validation.tests.CrossingWays.HIGHWAY;
     5import static org.openstreetmap.josm.data.validation.tests.CrossingWays.RAILWAY;
    46import static org.openstreetmap.josm.tools.I18n.tr;
    57
    … …  
    5759        @Override
    5860        public boolean isPrimitiveUsable(OsmPrimitive p) {
    59             return super.isPrimitiveUsable(p) && p.hasKey("highway");
     61            return super.isPrimitiveUsable(p) && p.hasKey(HIGHWAY);
    6062        }
    6163    }
    … …  
    181183                        continue;
    182184                    }
    183                     if (en.hasTag("highway", "turning_circle", "bus_stop")
     185                    if (en.hasTag(HIGHWAY, "turning_circle", "bus_stop")
    184186                            || en.hasTag("amenity", "parking_entrance")
    185                             || en.hasTag("railway", "buffer_stop")
     187                            || en.hasTag(RAILWAY, "buffer_stop")
    186188                            || en.isKeyTrue("noexit")
    187189                            || en.hasKey("entrance", "barrier")) {
    … …  
    302304        MyWaySegment(Way w, Node n1, Node n2) {
    303305            this.w = w;
    304             String railway = w.get("railway");
    305             String highway = w.get("highway");
     306            String railway = w.get(RAILWAY);
     307            String highway = w.get(HIGHWAY);
    306308            this.isAbandoned = "abandoned".equals(railway) || w.isKeyTrue("disused");
    307309            this.highway = (highway != null || railway != null) && !isAbandoned;
    … …  
    449451                && !w.hasKey("addr:interpolation")
    450452                // similarly for public transport platforms
    451                 && !w.hasTag("highway", "platform") && !w.hasTag("railway", "platform")
     453                && !w.hasTag(HIGHWAY, "platform") && !w.hasTag(RAILWAY, "platform")
    452454                ) {
    453455            ways.addAll(getWaySegments(w));
    454456            QuadBuckets<Node> set = endnodes;
    455             if (w.hasKey("highway", "railway")) {
     457            if (w.hasKey(HIGHWAY, RAILWAY)) {
    456458                set = endnodesHighway;
    457459            }

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

    r11384 r11913  
    22package org.openstreetmap.josm.data.validation.tests;
    33
     4import static org.openstreetmap.josm.data.validation.tests.CrossingWays.HIGHWAY;
    45import static org.openstreetmap.josm.tools.I18n.tr;
    56
    … …  
    8182        Map<String, String> tags = w.getKeys();
    8283        if (!tags.isEmpty()) {
    83             String highway = tags.get("highway");
     84            String highway = tags.get(HIGHWAY);
    8485            if (highway != null && NAMED_WAYS.contains(highway) && !tags.containsKey("name") && !tags.containsKey("ref")
    8586                    && !"yes".equals(tags.get("noname"))) {

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

    r11587 r11913  
    22package org.openstreetmap.josm.data.validation.tests;
    33
     4import static org.openstreetmap.josm.data.validation.tests.CrossingWays.HIGHWAY;
    45import static org.openstreetmap.josm.tools.I18n.tr;
    56
    … …  
    3031    @Override
    3132    public void visit(Way w) {
    32         if (!w.isUsable() || w.isClosed() || !w.hasKey("highway")) {
     33        if (!w.isUsable() || w.isClosed() || !w.hasKey(HIGHWAY)) {
    3334            return;
    3435        }
    … …  
    3738        List<OsmPrimitive> r = w.firstNode().getReferrers();
    3839        for (OsmPrimitive p : r) {
    39             if (p != w && p.hasKey("highway")) {
     40            if (p != w && p.hasKey(HIGHWAY)) {
    4041                hasway = true;
    4142                break;
    … …  
    5051        r = w.lastNode().getReferrers();
    5152        for (OsmPrimitive p : r) {
    52             if (p != w && p.hasKey("highway")) {
     53            if (p != w && p.hasKey(HIGHWAY)) {
    5354                hasway = true;
    5455                break;
    … …  
    8687    private void addPossibleError(Way w, Node wayNode, OsmPrimitive p, OsmPrimitive area) {
    8788        // Avoid "legal" cases (see #10655)
    88         if (w.hasKey("highway") && wayNode.hasTag("leisure", "slipway") && area.hasTag("natural", "water")) {
     89        if (w.hasKey(HIGHWAY) && wayNode.hasTag("leisure", "slipway") && area.hasTag("natural", "water")) {
    8990            return;
    9091        }

  • trunk/src/org/openstreetmap/josm/gui/dialogs/layer/LayerVisibilityAction.java

    r11510 r11913  
    4040 */
    4141public final class LayerVisibilityAction extends AbstractAction implements IEnabledStateUpdating, LayerAction {
     42    private static final String DIALOGS_LAYERLIST = "dialogs/layerlist";
    4243    private static final int SLIDER_STEPS = 100;
    4344    /**
    … …  
    7071        content.setLayout(new GridBagLayout());
    7172
    72         new ImageProvider("dialogs/layerlist", "visibility").getResource().attachImageIcon(this, true);
     73        new ImageProvider(DIALOGS_LAYERLIST, "visibility").getResource().attachImageIcon(this, true);
    7374        putValue(SHORT_DESCRIPTION, tr("Change visibility of the selected layer."));
    7475
    … …  
    317318        @Override
    318319        public ImageIcon getIcon() {
    319             return ImageProvider.get("dialogs/layerlist", "transparency");
     320            return ImageProvider.get(DIALOGS_LAYERLIST, "transparency");
    320321        }
    321322
    … …  
    355356        @Override
    356357        public ImageIcon getIcon() {
    357            return ImageProvider.get("dialogs/layerlist", "gamma");
     358           return ImageProvider.get(DIALOGS_LAYERLIST, "gamma");
    358359        }
    359360
    … …  
    413414        @Override
    414415        public ImageIcon getIcon() {
    415            return ImageProvider.get("dialogs/layerlist", "sharpness");
     416           return ImageProvider.get(DIALOGS_LAYERLIST, "sharpness");
    416417        }
    417418
    … …  
    450451        @Override
    451452        public ImageIcon getIcon() {
    452            return ImageProvider.get("dialogs/layerlist", "colorfulness");
     453           return ImageProvider.get(DIALOGS_LAYERLIST, "colorfulness");
    453454        }
    454455

  • trunk/src/org/openstreetmap/josm/io/imagery/ImageryReader.java

    r11884 r11913  
    9898
    9999    private static class Parser extends DefaultHandler {
     100        private static final String MAX_ZOOM = "max-zoom";
     101        private static final String MIN_ZOOM = "min-zoom";
     102        private static final String TILE_SIZE = "tile-size";
     103        private static final String TRUE = "true";
     104
    100105        private StringBuilder accumulator = new StringBuilder();
    101106
    … …  
    155160                    metadataHeaders = new HashMap<>();
    156161                    String best = atts.getValue("eli-best");
    157                     if ("true".equals(best)) {
     162                    if (TRUE.equals(best)) {
    158163                        entry.setBestMarked(true);
    159164                    }
    … …  
    164169                        "type",
    165170                        "url",
    166                         "min-zoom",
    167                         "max-zoom",
    168                         "tile-size",
     171                        MIN_ZOOM,
     172                        MAX_ZOOM,
     173                        TILE_SIZE,
    169174                }).contains(qName)) {
    170175                    newState = State.MIRROR_ATTRIBUTE;
    … …  
    184189                        "url",
    185190                        "eula",
    186                         "min-zoom",
    187                         "max-zoom",
     191                        MIN_ZOOM,
     192                        MAX_ZOOM,
    188193                        "attribution-text",
    189194                        "attribution-url",
    … …  
    195200                        "icon",
    196201                        "date",
    197                         "tile-size",
     202                        TILE_SIZE,
    198203                        "valid-georeference",
    199204                }).contains(qName)) {
    … …  
    264269            }
    265270            states.push(newState);
    266             if (newState == State.UNKNOWN && "true".equals(atts.getValue("mandatory"))) {
     271            if (newState == State.UNKNOWN && TRUE.equals(atts.getValue("mandatory"))) {
    267272                skipEntry = true;
    268273            }
    … …  
    319324                        mirrorEntry.setUrl(accumulator.toString());
    320325                        break;
    321                     case "min-zoom":
    322                     case "max-zoom":
     326                    case MIN_ZOOM:
     327                    case MAX_ZOOM:
    323328                        Integer val = null;
    324329                        try {
    … …  
    330335                            mirrorEntry = null;
    331336                        } else {
    332                             if ("min-zoom".equals(qName)) {
     337                            if (MIN_ZOOM.equals(qName)) {
    333338                                mirrorEntry.setDefaultMinZoom(val);
    334339                            } else {
    … …  
    337342                        }
    338343                        break;
    339                     case "tile-size":
     344                    case TILE_SIZE:
    340345                        Integer tileSize = null;
    341346                        try {
    … …  
    383388                case "default":
    384389                    switch (accumulator.toString()) {
    385                     case "true":
     390                    case TRUE:
    386391                        entry.setDefaultEntry(true);
    387392                        break;
    … …  
    399404                    entry.setEulaAcceptanceRequired(accumulator.toString());
    400405                    break;
    401                 case "min-zoom":
    402                 case "max-zoom":
     406                case MIN_ZOOM:
     407                case MAX_ZOOM:
    403408                    Integer val = null;
    404409                    try {
    … …  
    410415                        skipEntry = true;
    411416                    } else {
    412                         if ("min-zoom".equals(qName)) {
     417                        if (MIN_ZOOM.equals(qName)) {
    413418                            entry.setDefaultMinZoom(val);
    414419                        } else {
    … …  
    441446                    entry.setIcon(accumulator.toString());
    442447                    break;
    443                 case "tile-size":
     448                case TILE_SIZE:
    444449                    Integer tileSize = null;
    445450                    try {
Note: See TracChangeset for help on using the changeset viewer.