Changeset 11014 in josm for trunk/src


Ignore:
Timestamp:
2016-09-17T21:57:10+02:00 (3 years ago)
Author:
Don-vip
Message:

fix #13630 - proper validation of invalid mapcss rules

File:
1 edited

Legend:

Unmodified
Added
Removed
  • trunk/src/org/openstreetmap/josm/data/validation/tests/MapCSSTagChecker.java

    r10921 r11014  
    301301                        continue;
    302302                    }
    303                     final String val = ai.val instanceof Expression
    304                             ? (String) ((Expression) ai.val).evaluate(new Environment())
    305                             : ai.val instanceof String
    306                             ? (String) ai.val
    307                             : ai.val instanceof Keyword
    308                             ? ((Keyword) ai.val).val
    309                             : null;
    310                     if (ai.key.startsWith("throw")) {
    311                         try {
    312                             final Severity severity = Severity.valueOf(ai.key.substring("throw".length()).toUpperCase(Locale.ENGLISH));
    313                             check.errors.put(ai, severity);
    314                         } catch (IllegalArgumentException e) {
    315                             Main.warn(e, "Unsupported "+ai.key+" instruction. Allowed instructions are "+POSSIBLE_THROWS+'.');
     303                    try {
     304                        final String val = ai.val instanceof Expression
     305                                ? (String) ((Expression) ai.val).evaluate(new Environment())
     306                                : ai.val instanceof String
     307                                ? (String) ai.val
     308                                : ai.val instanceof Keyword
     309                                ? ((Keyword) ai.val).val
     310                                : null;
     311                        if (ai.key.startsWith("throw")) {
     312                            try {
     313                                final Severity severity = Severity.valueOf(ai.key.substring("throw".length()).toUpperCase(Locale.ENGLISH));
     314                                check.errors.put(ai, severity);
     315                            } catch (IllegalArgumentException e) {
     316                                Main.warn(e, "Unsupported "+ai.key+" instruction. Allowed instructions are "+POSSIBLE_THROWS+'.');
     317                            }
     318                        } else if ("fixAdd".equals(ai.key)) {
     319                            check.fixCommands.add(FixCommand.fixAdd(ai.val));
     320                        } else if ("fixRemove".equals(ai.key)) {
     321                            CheckParameterUtil.ensureThat(!(ai.val instanceof String) || !(val != null && val.contains("=")),
     322                                    "Unexpected '='. Please only specify the key to remove!");
     323                            check.fixCommands.add(FixCommand.fixRemove(ai.val));
     324                        } else if ("fixChangeKey".equals(ai.key) && val != null) {
     325                            CheckParameterUtil.ensureThat(val.contains("=>"), "Separate old from new key by '=>'!");
     326                            final String[] x = val.split("=>", 2);
     327                            check.fixCommands.add(FixCommand.fixChangeKey(Tag.removeWhiteSpaces(x[0]), Tag.removeWhiteSpaces(x[1])));
     328                        } else if ("fixDeleteObject".equals(ai.key) && val != null) {
     329                            CheckParameterUtil.ensureThat("this".equals(val), "fixDeleteObject must be followed by 'this'");
     330                            check.deletion = true;
     331                        } else if ("suggestAlternative".equals(ai.key) && val != null) {
     332                            check.alternatives.add(val);
     333                        } else if ("assertMatch".equals(ai.key) && val != null) {
     334                            check.assertions.put(val, Boolean.TRUE);
     335                        } else if ("assertNoMatch".equals(ai.key) && val != null) {
     336                            check.assertions.put(val, Boolean.FALSE);
     337                        } else if ("group".equals(ai.key) && val != null) {
     338                            check.group = val;
     339                        } else {
     340                            throw new IllegalDataException("Cannot add instruction " + ai.key + ": " + ai.val + '!');
    316341                        }
    317                     } else if ("fixAdd".equals(ai.key)) {
    318                         check.fixCommands.add(FixCommand.fixAdd(ai.val));
    319                     } else if ("fixRemove".equals(ai.key)) {
    320                         CheckParameterUtil.ensureThat(!(ai.val instanceof String) || !(val != null && val.contains("=")),
    321                                 "Unexpected '='. Please only specify the key to remove!");
    322                         check.fixCommands.add(FixCommand.fixRemove(ai.val));
    323                     } else if ("fixChangeKey".equals(ai.key) && val != null) {
    324                         CheckParameterUtil.ensureThat(val.contains("=>"), "Separate old from new key by '=>'!");
    325                         final String[] x = val.split("=>", 2);
    326                         check.fixCommands.add(FixCommand.fixChangeKey(Tag.removeWhiteSpaces(x[0]), Tag.removeWhiteSpaces(x[1])));
    327                     } else if ("fixDeleteObject".equals(ai.key) && val != null) {
    328                         CheckParameterUtil.ensureThat("this".equals(val), "fixDeleteObject must be followed by 'this'");
    329                         check.deletion = true;
    330                     } else if ("suggestAlternative".equals(ai.key) && val != null) {
    331                         check.alternatives.add(val);
    332                     } else if ("assertMatch".equals(ai.key) && val != null) {
    333                         check.assertions.put(val, Boolean.TRUE);
    334                     } else if ("assertNoMatch".equals(ai.key) && val != null) {
    335                         check.assertions.put(val, Boolean.FALSE);
    336                     } else if ("group".equals(ai.key) && val != null) {
    337                         check.group = val;
    338                     } else {
    339                         throw new IllegalDataException("Cannot add instruction " + ai.key + ": " + ai.val + '!');
     342                    } catch (IllegalArgumentException e) {
     343                        throw new IllegalDataException(e);
    340344                    }
    341345                }
     
    357361            final MapCSSStyleSource source = new MapCSSStyleSource("");
    358362            final MapCSSParser preprocessor = new MapCSSParser(css, MapCSSParser.LexicalState.PREPROCESSOR);
    359 
    360             css = new StringReader(preprocessor.pp_root(source));
    361             final MapCSSParser parser = new MapCSSParser(css, MapCSSParser.LexicalState.DEFAULT);
     363            final StringReader mapcss = new StringReader(preprocessor.pp_root(source));
     364            final MapCSSParser parser = new MapCSSParser(mapcss, MapCSSParser.LexicalState.DEFAULT);
    362365            parser.sheet(source);
    363             Collection<Throwable> parseErrors = source.getErrors();
    364             assert parseErrors.isEmpty();
    365366            // Ignore "meta" rule(s) from external rules of JOSM wiki
    366367            removeMetaRules(source);
     
    383384                } catch (IllegalDataException e) {
    384385                    Main.error("Cannot add MapCss rule: "+e.getMessage());
    385                     parseErrors.add(e);
    386                 }
    387             }
    388             return new ParseResult(parseChecks, parseErrors);
     386                    source.logError(e);
     387                }
     388            }
     389            return new ParseResult(parseChecks, source.getErrors());
    389390        }
    390391
Note: See TracChangeset for help on using the changeset viewer.