Gathering string utils

CHANGELOG.md

@@ -4,8 +4,23 @@
 Changelog of Violations lib.
 
 ## Unreleased
+### GitHub [#28](https://github.com/tomasbjerre/violations-lib/issues/28) Single quotes in xml are printed as ' 
+
+**Un-escape XML when reading attributes.**
+
+ * Fixes 
+
+[9b10f800c7b7941](https://github.com/tomasbjerre/violations-lib/commit/9b10f800c7b7941) Sam Judd *2017-12-31 03:25:14*
+
+
+## 1.49
 ### No issue
 
+**Relocating to correct Java identifier**
+
+
+[8e64583ce0fec03](https://github.com/tomasbjerre/violations-lib/commit/8e64583ce0fec03) Tomas Bjerre *2017-12-30 18:32:34*
+
 **Doc**
 
 

src/main/java/se/bjurr/violations/lib/parsers/AndroidLintParser.java

@@ -4,10 +4,10 @@
 import static se.bjurr.violations.lib.model.SEVERITY.INFO;
 import static se.bjurr.violations.lib.model.SEVERITY.WARN;
 import static se.bjurr.violations.lib.model.Violation.violationBuilder;
-import static se.bjurr.violations.lib.parsers.ViolationParserUtils.findIntegerAttribute;
-import static se.bjurr.violations.lib.parsers.ViolationParserUtils.getAttribute;
-import static se.bjurr.violations.lib.parsers.ViolationParserUtils.getChunks;
 import static se.bjurr.violations.lib.reports.Parser.ANDROIDLINT;
+import static se.bjurr.violations.lib.util.ViolationParserUtils.findIntegerAttribute;
+import static se.bjurr.violations.lib.util.ViolationParserUtils.getAttribute;
+import static se.bjurr.violations.lib.util.ViolationParserUtils.getChunks;
 
 import java.util.ArrayList;
 import java.util.List;

src/main/java/se/bjurr/violations/lib/parsers/CLangParser.java

@@ -5,9 +5,9 @@
 import static se.bjurr.violations.lib.model.SEVERITY.INFO;
 import static se.bjurr.violations.lib.model.SEVERITY.WARN;
 import static se.bjurr.violations.lib.model.Violation.violationBuilder;
-import static se.bjurr.violations.lib.parsers.ViolationParserUtils.getLines;
 import static se.bjurr.violations.lib.reports.Parser.CLANG;
 import static se.bjurr.violations.lib.util.Utils.isNullOrEmpty;
+import static se.bjurr.violations.lib.util.ViolationParserUtils.getLines;
 
 import java.util.ArrayList;
 import java.util.List;

src/main/java/se/bjurr/violations/lib/parsers/CPDParser.java

@@ -5,9 +5,9 @@
 import static se.bjurr.violations.lib.model.SEVERITY.INFO;
 import static se.bjurr.violations.lib.model.SEVERITY.WARN;
 import static se.bjurr.violations.lib.model.Violation.violationBuilder;
-import static se.bjurr.violations.lib.parsers.ViolationParserUtils.getAttribute;
-import static se.bjurr.violations.lib.parsers.ViolationParserUtils.getIntegerAttribute;
 import static se.bjurr.violations.lib.reports.Parser.CODENARC;
+import static se.bjurr.violations.lib.util.ViolationParserUtils.getAttribute;
+import static se.bjurr.violations.lib.util.ViolationParserUtils.getIntegerAttribute;
 
 import java.io.ByteArrayInputStream;
 import java.io.InputStream;

src/main/java/se/bjurr/violations/lib/parsers/CPPCheckParser.java

@@ -4,10 +4,10 @@
 import static se.bjurr.violations.lib.model.SEVERITY.INFO;
 import static se.bjurr.violations.lib.model.SEVERITY.WARN;
 import static se.bjurr.violations.lib.model.Violation.violationBuilder;
-import static se.bjurr.violations.lib.parsers.ViolationParserUtils.getAttribute;
-import static se.bjurr.violations.lib.parsers.ViolationParserUtils.getChunks;
-import static se.bjurr.violations.lib.parsers.ViolationParserUtils.getIntegerAttribute;
 import static se.bjurr.violations.lib.reports.Parser.CPPCHECK;
+import static se.bjurr.violations.lib.util.ViolationParserUtils.getAttribute;
+import static se.bjurr.violations.lib.util.ViolationParserUtils.getChunks;
+import static se.bjurr.violations.lib.util.ViolationParserUtils.getIntegerAttribute;
 
 import java.util.ArrayList;
 import java.util.List;

src/main/java/se/bjurr/violations/lib/parsers/CSSLintParser.java

@@ -4,11 +4,11 @@
 import static se.bjurr.violations.lib.model.SEVERITY.INFO;
 import static se.bjurr.violations.lib.model.SEVERITY.WARN;
 import static se.bjurr.violations.lib.model.Violation.violationBuilder;
-import static se.bjurr.violations.lib.parsers.ViolationParserUtils.findAttribute;
-import static se.bjurr.violations.lib.parsers.ViolationParserUtils.findIntegerAttribute;
-import static se.bjurr.violations.lib.parsers.ViolationParserUtils.getAttribute;
-import static se.bjurr.violations.lib.parsers.ViolationParserUtils.getChunks;
 import static se.bjurr.violations.lib.reports.Parser.CSSLINT;
+import static se.bjurr.violations.lib.util.ViolationParserUtils.findAttribute;
+import static se.bjurr.violations.lib.util.ViolationParserUtils.findIntegerAttribute;
+import static se.bjurr.violations.lib.util.ViolationParserUtils.getAttribute;
+import static se.bjurr.violations.lib.util.ViolationParserUtils.getChunks;
 
 import java.util.ArrayList;
 import java.util.List;

src/main/java/se/bjurr/violations/lib/parsers/CheckStyleParser.java

@@ -4,12 +4,12 @@
 import static se.bjurr.violations.lib.model.SEVERITY.INFO;
 import static se.bjurr.violations.lib.model.SEVERITY.WARN;
 import static se.bjurr.violations.lib.model.Violation.violationBuilder;
-import static se.bjurr.violations.lib.parsers.ViolationParserUtils.findAttribute;
-import static se.bjurr.violations.lib.parsers.ViolationParserUtils.findIntegerAttribute;
-import static se.bjurr.violations.lib.parsers.ViolationParserUtils.getAttribute;
-import static se.bjurr.violations.lib.parsers.ViolationParserUtils.getChunks;
-import static se.bjurr.violations.lib.parsers.ViolationParserUtils.getIntegerAttribute;
 import static se.bjurr.violations.lib.reports.Parser.CHECKSTYLE;
+import static se.bjurr.violations.lib.util.ViolationParserUtils.findAttribute;
+import static se.bjurr.violations.lib.util.ViolationParserUtils.findIntegerAttribute;
+import static se.bjurr.violations.lib.util.ViolationParserUtils.getAttribute;
+import static se.bjurr.violations.lib.util.ViolationParserUtils.getChunks;
+import static se.bjurr.violations.lib.util.ViolationParserUtils.getIntegerAttribute;
 
 import java.util.ArrayList;
 import java.util.List;

src/main/java/se/bjurr/violations/lib/parsers/CodeNarcParser.java

@@ -5,9 +5,9 @@
 import static se.bjurr.violations.lib.model.SEVERITY.INFO;
 import static se.bjurr.violations.lib.model.SEVERITY.WARN;
 import static se.bjurr.violations.lib.model.Violation.violationBuilder;
-import static se.bjurr.violations.lib.parsers.ViolationParserUtils.getAttribute;
-import static se.bjurr.violations.lib.parsers.ViolationParserUtils.getIntegerAttribute;
 import static se.bjurr.violations.lib.reports.Parser.CODENARC;
+import static se.bjurr.violations.lib.util.ViolationParserUtils.getAttribute;
+import static se.bjurr.violations.lib.util.ViolationParserUtils.getIntegerAttribute;
 
 import java.io.ByteArrayInputStream;
 import java.io.InputStream;

src/main/java/se/bjurr/violations/lib/parsers/CppLintParser.java

@@ -5,9 +5,9 @@
 import static se.bjurr.violations.lib.model.SEVERITY.INFO;
 import static se.bjurr.violations.lib.model.SEVERITY.WARN;
 import static se.bjurr.violations.lib.model.Violation.violationBuilder;
-import static se.bjurr.violations.lib.parsers.ViolationParserUtils.getLines;
-import static se.bjurr.violations.lib.parsers.ViolationParserUtils.getParts;
 import static se.bjurr.violations.lib.reports.Parser.CPPLINT;
+import static se.bjurr.violations.lib.util.ViolationParserUtils.getLines;
+import static se.bjurr.violations.lib.util.ViolationParserUtils.getParts;
 
 import java.util.ArrayList;
 import java.util.List;

src/main/java/se/bjurr/violations/lib/parsers/DocFXParser.java

@@ -11,7 +11,7 @@
 import java.util.logging.Logger;
 import se.bjurr.violations.lib.model.SEVERITY;
 import se.bjurr.violations.lib.model.Violation;
-import se.bjurr.violations.lib.util.HTMLUtils;
+import se.bjurr.violations.lib.util.StringUtils;
 
 public class DocFXParser implements ViolationsParser {
   private static Logger LOG = Logger.getLogger(DocFXParser.class.getName());
@@ -32,7 +32,7 @@ public List<Violation> parseReportOutput(String reportContent) throws Exception
         LOG.log(Level.FINE, "Ignoring " + rawLineToParse + " because there is not file attribute");
         continue;
       }
-      final String file = HTMLUtils.htmlDecode(fileEncoded);
+      final String file = StringUtils.htmlDecode(fileEncoded);
       final String code = (String) parsedMap.get("code");
       final String lineString = (String) parsedMap.get("line");
       Integer lineInt;

src/main/java/se/bjurr/violations/lib/parsers/FindbugsParser.java

@@ -2,13 +2,13 @@
 
 import static java.util.logging.Level.SEVERE;
 import static se.bjurr.violations.lib.model.Violation.violationBuilder;
-import static se.bjurr.violations.lib.parsers.ViolationParserUtils.findIntegerAttribute;
-import static se.bjurr.violations.lib.parsers.ViolationParserUtils.getAttribute;
-import static se.bjurr.violations.lib.parsers.ViolationParserUtils.getChunks;
-import static se.bjurr.violations.lib.parsers.ViolationParserUtils.getContent;
-import static se.bjurr.violations.lib.parsers.ViolationParserUtils.getIntegerAttribute;
 import static se.bjurr.violations.lib.reports.Parser.FINDBUGS;
 import static se.bjurr.violations.lib.util.Utils.isNullOrEmpty;
+import static se.bjurr.violations.lib.util.ViolationParserUtils.findIntegerAttribute;
+import static se.bjurr.violations.lib.util.ViolationParserUtils.getAttribute;
+import static se.bjurr.violations.lib.util.ViolationParserUtils.getChunks;
+import static se.bjurr.violations.lib.util.ViolationParserUtils.getContent;
+import static se.bjurr.violations.lib.util.ViolationParserUtils.getIntegerAttribute;
 
 import java.io.ByteArrayInputStream;
 import java.io.IOException;

src/main/java/se/bjurr/violations/lib/parsers/Flake8Parser.java

@@ -5,9 +5,9 @@
 import static se.bjurr.violations.lib.model.SEVERITY.INFO;
 import static se.bjurr.violations.lib.model.SEVERITY.WARN;
 import static se.bjurr.violations.lib.model.Violation.violationBuilder;
-import static se.bjurr.violations.lib.parsers.ViolationParserUtils.getLines;
 import static se.bjurr.violations.lib.reports.Parser.FLAKE8;
 import static se.bjurr.violations.lib.util.Utils.isNullOrEmpty;
+import static se.bjurr.violations.lib.util.ViolationParserUtils.getLines;
 
 import java.util.ArrayList;
 import java.util.List;

src/main/java/se/bjurr/violations/lib/parsers/FxCopParser.java

@@ -6,9 +6,9 @@
 import static se.bjurr.violations.lib.model.SEVERITY.INFO;
 import static se.bjurr.violations.lib.model.SEVERITY.WARN;
 import static se.bjurr.violations.lib.model.Violation.violationBuilder;
-import static se.bjurr.violations.lib.parsers.ViolationParserUtils.getAttribute;
-import static se.bjurr.violations.lib.parsers.ViolationParserUtils.getIntegerAttribute;
 import static se.bjurr.violations.lib.reports.Parser.FXCOP;
+import static se.bjurr.violations.lib.util.ViolationParserUtils.getAttribute;
+import static se.bjurr.violations.lib.util.ViolationParserUtils.getIntegerAttribute;
 
 import java.io.ByteArrayInputStream;
 import java.io.InputStream;
@@ -20,6 +20,7 @@
 import se.bjurr.violations.lib.model.SEVERITY;
 import se.bjurr.violations.lib.model.Violation;
 import se.bjurr.violations.lib.util.Optional;
+import se.bjurr.violations.lib.util.ViolationParserUtils;
 
 public class FxCopParser implements ViolationsParser {
   private static Logger LOG = Logger.getLogger(FxCopParser.class.getSimpleName());

src/main/java/se/bjurr/violations/lib/parsers/GendarmeParser.java

@@ -5,8 +5,8 @@
 import static se.bjurr.violations.lib.model.SEVERITY.INFO;
 import static se.bjurr.violations.lib.model.SEVERITY.WARN;
 import static se.bjurr.violations.lib.model.Violation.violationBuilder;
-import static se.bjurr.violations.lib.parsers.ViolationParserUtils.getAttribute;
 import static se.bjurr.violations.lib.reports.Parser.GENDARME;
+import static se.bjurr.violations.lib.util.ViolationParserUtils.getAttribute;
 
 import java.io.ByteArrayInputStream;
 import java.io.InputStream;

src/main/java/se/bjurr/violations/lib/parsers/GoLintParser.java

@@ -5,9 +5,9 @@
 import static se.bjurr.violations.lib.model.SEVERITY.INFO;
 import static se.bjurr.violations.lib.model.SEVERITY.WARN;
 import static se.bjurr.violations.lib.model.Violation.violationBuilder;
-import static se.bjurr.violations.lib.parsers.ViolationParserUtils.getLines;
 import static se.bjurr.violations.lib.reports.Parser.GOLINT;
 import static se.bjurr.violations.lib.util.Utils.isNullOrEmpty;
+import static se.bjurr.violations.lib.util.ViolationParserUtils.getLines;
 
 import java.util.ArrayList;
 import java.util.List;

src/main/java/se/bjurr/violations/lib/parsers/JCReportParser.java

@@ -5,9 +5,9 @@
 import static se.bjurr.violations.lib.model.SEVERITY.INFO;
 import static se.bjurr.violations.lib.model.SEVERITY.WARN;
 import static se.bjurr.violations.lib.model.Violation.violationBuilder;
-import static se.bjurr.violations.lib.parsers.ViolationParserUtils.getAttribute;
-import static se.bjurr.violations.lib.parsers.ViolationParserUtils.getIntegerAttribute;
 import static se.bjurr.violations.lib.reports.Parser.JCREPORT;
+import static se.bjurr.violations.lib.util.ViolationParserUtils.getAttribute;
+import static se.bjurr.violations.lib.util.ViolationParserUtils.getIntegerAttribute;
 
 import java.io.ByteArrayInputStream;
 import java.io.InputStream;

src/main/java/se/bjurr/violations/lib/parsers/JSHintParser.java

@@ -4,10 +4,10 @@
 import static se.bjurr.violations.lib.model.SEVERITY.INFO;
 import static se.bjurr.violations.lib.model.SEVERITY.WARN;
 import static se.bjurr.violations.lib.model.Violation.violationBuilder;
-import static se.bjurr.violations.lib.parsers.ViolationParserUtils.getAttribute;
-import static se.bjurr.violations.lib.parsers.ViolationParserUtils.getChunks;
-import static se.bjurr.violations.lib.parsers.ViolationParserUtils.getIntegerAttribute;
 import static se.bjurr.violations.lib.reports.Parser.JSHINT;
+import static se.bjurr.violations.lib.util.ViolationParserUtils.getAttribute;
+import static se.bjurr.violations.lib.util.ViolationParserUtils.getChunks;
+import static se.bjurr.violations.lib.util.ViolationParserUtils.getIntegerAttribute;
 
 import java.util.ArrayList;
 import java.util.List;

src/main/java/se/bjurr/violations/lib/parsers/LintParser.java

@@ -4,11 +4,11 @@
 import static se.bjurr.violations.lib.model.SEVERITY.INFO;
 import static se.bjurr.violations.lib.model.SEVERITY.WARN;
 import static se.bjurr.violations.lib.model.Violation.violationBuilder;
-import static se.bjurr.violations.lib.parsers.ViolationParserUtils.findIntegerAttribute;
-import static se.bjurr.violations.lib.parsers.ViolationParserUtils.getAttribute;
-import static se.bjurr.violations.lib.parsers.ViolationParserUtils.getChunks;
-import static se.bjurr.violations.lib.parsers.ViolationParserUtils.getIntegerAttribute;
 import static se.bjurr.violations.lib.reports.Parser.LINT;
+import static se.bjurr.violations.lib.util.ViolationParserUtils.findIntegerAttribute;
+import static se.bjurr.violations.lib.util.ViolationParserUtils.getAttribute;
+import static se.bjurr.violations.lib.util.ViolationParserUtils.getChunks;
+import static se.bjurr.violations.lib.util.ViolationParserUtils.getIntegerAttribute;
 
 import java.util.ArrayList;
 import java.util.List;

src/main/java/se/bjurr/violations/lib/parsers/MyPyParser.java

@@ -4,8 +4,8 @@
 import static se.bjurr.violations.lib.model.SEVERITY.ERROR;
 import static se.bjurr.violations.lib.model.SEVERITY.INFO;
 import static se.bjurr.violations.lib.model.Violation.violationBuilder;
-import static se.bjurr.violations.lib.parsers.ViolationParserUtils.getLines;
 import static se.bjurr.violations.lib.reports.Parser.MYPY;
+import static se.bjurr.violations.lib.util.ViolationParserUtils.getLines;
 
 import java.util.ArrayList;
 import java.util.List;

src/main/java/se/bjurr/violations/lib/parsers/PMDParser.java

@@ -4,13 +4,13 @@
 import static se.bjurr.violations.lib.model.SEVERITY.INFO;
 import static se.bjurr.violations.lib.model.SEVERITY.WARN;
 import static se.bjurr.violations.lib.model.Violation.violationBuilder;
-import static se.bjurr.violations.lib.parsers.ViolationParserUtils.findAttribute;
-import static se.bjurr.violations.lib.parsers.ViolationParserUtils.findIntegerAttribute;
-import static se.bjurr.violations.lib.parsers.ViolationParserUtils.getAttribute;
-import static se.bjurr.violations.lib.parsers.ViolationParserUtils.getChunks;
-import static se.bjurr.violations.lib.parsers.ViolationParserUtils.getContent;
-import static se.bjurr.violations.lib.parsers.ViolationParserUtils.getIntegerAttribute;
 import static se.bjurr.violations.lib.reports.Parser.PMD;
+import static se.bjurr.violations.lib.util.ViolationParserUtils.findAttribute;
+import static se.bjurr.violations.lib.util.ViolationParserUtils.findIntegerAttribute;
+import static se.bjurr.violations.lib.util.ViolationParserUtils.getAttribute;
+import static se.bjurr.violations.lib.util.ViolationParserUtils.getChunks;
+import static se.bjurr.violations.lib.util.ViolationParserUtils.getContent;
+import static se.bjurr.violations.lib.util.ViolationParserUtils.getIntegerAttribute;
 
 import java.util.ArrayList;
 import java.util.List;

src/main/java/se/bjurr/violations/lib/parsers/PerlCriticParser.java

@@ -5,9 +5,9 @@
 import static se.bjurr.violations.lib.model.SEVERITY.INFO;
 import static se.bjurr.violations.lib.model.SEVERITY.WARN;
 import static se.bjurr.violations.lib.model.Violation.violationBuilder;
-import static se.bjurr.violations.lib.parsers.ViolationParserUtils.getLines;
-import static se.bjurr.violations.lib.parsers.ViolationParserUtils.getParts;
 import static se.bjurr.violations.lib.reports.Parser.CPPLINT;
+import static se.bjurr.violations.lib.util.ViolationParserUtils.getLines;
+import static se.bjurr.violations.lib.util.ViolationParserUtils.getParts;
 
 import java.util.ArrayList;
 import java.util.List;

src/main/java/se/bjurr/violations/lib/parsers/PiTestParser.java

@@ -2,11 +2,11 @@
 
 import static se.bjurr.violations.lib.model.SEVERITY.WARN;
 import static se.bjurr.violations.lib.model.Violation.violationBuilder;
-import static se.bjurr.violations.lib.parsers.ViolationParserUtils.getAttribute;
-import static se.bjurr.violations.lib.parsers.ViolationParserUtils.getChunks;
-import static se.bjurr.violations.lib.parsers.ViolationParserUtils.getContent;
-import static se.bjurr.violations.lib.parsers.ViolationParserUtils.getIntegerContent;
 import static se.bjurr.violations.lib.reports.Parser.PITEST;
+import static se.bjurr.violations.lib.util.ViolationParserUtils.getAttribute;
+import static se.bjurr.violations.lib.util.ViolationParserUtils.getChunks;
+import static se.bjurr.violations.lib.util.ViolationParserUtils.getContent;
+import static se.bjurr.violations.lib.util.ViolationParserUtils.getIntegerContent;
 
 import java.util.ArrayList;
 import java.util.HashMap;

src/main/java/se/bjurr/violations/lib/parsers/PyDocStyleParser.java

@@ -3,12 +3,13 @@
 import static java.lang.Integer.parseInt;
 import static se.bjurr.violations.lib.model.SEVERITY.ERROR;
 import static se.bjurr.violations.lib.model.Violation.violationBuilder;
-import static se.bjurr.violations.lib.parsers.ViolationParserUtils.getParts;
 import static se.bjurr.violations.lib.reports.Parser.PYDOCSTYLE;
+import static se.bjurr.violations.lib.util.ViolationParserUtils.getParts;
 
 import java.util.ArrayList;
 import java.util.List;
 import se.bjurr.violations.lib.model.Violation;
+import se.bjurr.violations.lib.util.ViolationParserUtils;
 
 public class PyDocStyleParser implements ViolationsParser {
 

src/main/java/se/bjurr/violations/lib/parsers/PyLintParser.java

@@ -5,8 +5,8 @@
 import static se.bjurr.violations.lib.model.SEVERITY.INFO;
 import static se.bjurr.violations.lib.model.SEVERITY.WARN;
 import static se.bjurr.violations.lib.model.Violation.violationBuilder;
-import static se.bjurr.violations.lib.parsers.ViolationParserUtils.getLines;
 import static se.bjurr.violations.lib.reports.Parser.PYLINT;
+import static se.bjurr.violations.lib.util.ViolationParserUtils.getLines;
 
 import java.util.ArrayList;
 import java.util.List;

src/main/java/se/bjurr/violations/lib/parsers/ResharperParser.java

@@ -4,11 +4,11 @@
 import static se.bjurr.violations.lib.model.SEVERITY.INFO;
 import static se.bjurr.violations.lib.model.SEVERITY.WARN;
 import static se.bjurr.violations.lib.model.Violation.violationBuilder;
-import static se.bjurr.violations.lib.parsers.ViolationParserUtils.findAttribute;
-import static se.bjurr.violations.lib.parsers.ViolationParserUtils.findIntegerAttribute;
-import static se.bjurr.violations.lib.parsers.ViolationParserUtils.getAttribute;
-import static se.bjurr.violations.lib.parsers.ViolationParserUtils.getChunks;
 import static se.bjurr.violations.lib.reports.Parser.RESHARPER;
+import static se.bjurr.violations.lib.util.ViolationParserUtils.findAttribute;
+import static se.bjurr.violations.lib.util.ViolationParserUtils.findIntegerAttribute;
+import static se.bjurr.violations.lib.util.ViolationParserUtils.getAttribute;
+import static se.bjurr.violations.lib.util.ViolationParserUtils.getChunks;
 
 import java.util.ArrayList;
 import java.util.HashMap;

src/main/java/se/bjurr/violations/lib/parsers/SbtScalacParser.java

@@ -5,8 +5,8 @@
 import static se.bjurr.violations.lib.model.SEVERITY.INFO;
 import static se.bjurr.violations.lib.model.SEVERITY.WARN;
 import static se.bjurr.violations.lib.model.Violation.violationBuilder;
-import static se.bjurr.violations.lib.parsers.ViolationParserUtils.getLines;
 import static se.bjurr.violations.lib.reports.Parser.SBTSCALAC;
+import static se.bjurr.violations.lib.util.ViolationParserUtils.getLines;
 
 import java.util.ArrayList;
 import java.util.List;