{"action":"closed","issue":{"url":"https://api.github.com/repos/languagetool-org/languagetool/issues/625","repository_url":"https://api.github.com/repos/languagetool-org/languagetool","labels_url":"https://api.github.com/repos/languagetool-org/languagetool/issues/625/labels{/name}","comments_url":"https://api.github.com/repos/languagetool-org/languagetool/issues/625/comments","events_url":"https://api.github.com/repos/languagetool-org/languagetool/issues/625/events","html_url":"https://github.com/languagetool-org/languagetool/issues/625","id":193294147,"number":625,"title":"Conflict between testrules message and LT behaviour with chunks","user":{"login":"MikeUnwalla","id":5816160,"avatar_url":"https://avatars.githubusercontent.com/u/5816160?v=3","gravatar_id":"","url":"https://api.github.com/users/MikeUnwalla","html_url":"https://github.com/MikeUnwalla","followers_url":"https://api.github.com/users/MikeUnwalla/followers","following_url":"https://api.github.com/users/MikeUnwalla/following{/other_user}","gists_url":"https://api.github.com/users/MikeUnwalla/gists{/gist_id}","starred_url":"https://api.github.com/users/MikeUnwalla/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/MikeUnwalla/subscriptions","organizations_url":"https://api.github.com/users/MikeUnwalla/orgs","repos_url":"https://api.github.com/users/MikeUnwalla/repos","events_url":"https://api.github.com/users/MikeUnwalla/events{/privacy}","received_events_url":"https://api.github.com/users/MikeUnwalla/received_events","type":"User","site_admin":false},"labels":[],"state":"closed","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":3,"created_at":"2016-12-03T15:36:49Z","updated_at":"2016-12-03T16:20:22Z","closed_at":"2016-12-03T16:20:22Z","body":"Using LT 3.6 snapshot 2016/11/30\r\n\r\nSentence to test: Do not let the water drain quickly from the container.\r\n\r\nTagger result for the test sentence:\r\n`<S> Do[do/VB,do/VBP,B-VP] not[not/RB,I-VP] let[let/VB,I-VP] the[the/DT,B-NP-singular] water[water/NN:U,I-NP-singular] drain[drain/NN,drain/VB,drain/VBP,E-NP-singular] quickly[quickly/RB,B-ADVP] from[from/IN,from/RP,B-PP] the[the/DT,B-NP-singular] container[container/NN,E-NP-singular] .[./.,</S>,O] `\r\n\r\nThis grammar rule uses a chunk B-ADVP to find adverbs:\r\n```\r\n<rule id=\"chunker_test\" name=\"chunker test\">\r\n    <pattern>\r\n        <token chunk=\"B-ADVP\"/>\r\n    </pattern>\r\n    <message>'\\1' is an adverb.</message>\r\n    <example correction=\"\">Do not let the water drain <marker>quickly</marker> from the container.</example>\r\n    <example>This sentence is correct.</example>\r\n</rule>\r\n```\r\n\r\nLT gives the expected result:\r\n![chunker-test](https://cloud.githubusercontent.com/assets/5816160/20860299/ad76c04e-b96c-11e6-8958-e0a22e3ebca1.png)\r\n\r\nPart of the output from 'testrules':\r\n```\r\nRunning XML validation for en/grammar.xml...\r\ncvc-enumeration-valid: Value 'B-ADVP' is not facet-valid with respect to enumeration '[B-NP-singular, B-NP-plural, I-NP-\r\nsingular, I-NP-plural, E-NP-singular, E-NP-plural, B-VP, I-VP, B-PP, I-PP, containsTypewriterApostrophe]'. It must be a\r\nvalue from the enumeration. Problem found at line 49, column 32.\r\nException in thread \"main\" java.io.IOException: Cannot load or parse '/org/languagetool/rules/en/grammar.xml'\r\n        at org.languagetool.XMLValidator.validateWithXmlSchema(XMLValidator.java:123)\r\n        at org.languagetool.rules.patterns.PatternRuleTest.validatePatternFile(PatternRuleTest.java:226)\r\n        at org.languagetool.rules.patterns.PatternRuleTest.runTestForLanguage(PatternRuleTest.java:155)\r\n        at org.languagetool.rules.patterns.PatternRuleTest.runGrammarRulesFromXmlTestIgnoringLanguages(PatternRuleTest.j\r\nava:150)\r\n        at org.languagetool.rules.patterns.PatternRuleTest.main(PatternRuleTest.java:571)\r\nCaused by: org.xml.sax.SAXParseException; lineNumber: 49; columnNumber: 32; cvc-enumeration-valid: Value 'B-ADVP' is not\r\n facet-valid with respect to enumeration '[B-NP-singular, B-NP-plural, I-NP-singular, I-NP-plural, E-NP-singular, E-NP-p\r\nlural, B-VP, I-VP, B-PP, I-PP, containsTypewriterApostrophe]'. It must be a value from the enumeration.\r\n```\r\nIf 'testrules' is correct and B-ADVP (and others) should not be used, could LT give an error message when it is started?"}}
{"actor":{"display_login":"MikeUnwalla"}}