From 6353c32b0a17f1af8ddc7edec2d853efee97f789 Mon Sep 17 00:00:00 2001
From: Jeremy Landis <jeremylandis@hotmail.com>
Date: Fri, 5 Apr 2024 19:53:29 -0400
Subject: [PATCH] [logging] Make sure we use appropriate logger slf4j 1.7.36 **

** Because we only use this with maven and to be in line with maven, use 1.7.36 since that is what maven is still using.
---
 pom.xml                                       | 29 ++++++++++++++++++-
 .../xml/lib/XmlDocumentFormatter.java         | 16 ++++++----
 2 files changed, 39 insertions(+), 6 deletions(-)

diff --git a/pom.xml b/pom.xml
index aaf4030..43429d8 100644
--- a/pom.xml
+++ b/pom.xml
@@ -8,7 +8,7 @@
     SPDX-License-Identifier: EPL-2.0
 
 -->
-<project xmlns="http://maven.apache.org/POM/4.0.0" xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance" xsi:schemaLocation="http://maven.apache.org/POM/4.0.0 http://maven.apache.org/xsd/maven-4.0.0.xsd">
+<project xmlns="http://maven.apache.org/POM/4.0.0" xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance" xsi:schemaLocation="http://maven.apache.org/POM/4.0.0 https://maven.apache.org/xsd/maven-4.0.0.xsd">
   <modelVersion>4.0.0</modelVersion>
   <parent>
     <groupId>net.revelc.code</groupId>
@@ -49,4 +49,31 @@ SPDX-License-Identifier: EPL-2.0
     <!-- rat plugin is no good at validating EPL headers out of the box -->
     <rat.skip>true</rat.skip>
   </properties>
+  <dependencies>
+    <dependency>
+      <groupId>org.slf4j</groupId>
+      <artifactId>slf4j-api</artifactId>
+      <version>1.7.36</version>
+    </dependency>
+    <dependency>
+      <groupId>org.slf4j</groupId>
+      <artifactId>slf4j-simple</artifactId>
+      <version>1.7.36</version>
+      <scope>test</scope>
+    </dependency>
+  </dependencies>
+  <build>
+    <plugins>
+      <plugin>
+        <groupId>org.apache.maven.plugins</groupId>
+        <artifactId>maven-dependency-plugin</artifactId>
+        <configuration>
+          <ignoredUnusedDeclaredDependencies>
+            <!-- ignore sfl4j-simple as required for testing -->
+            <ignore>org.slf4j:slf4j-simple</ignore>
+          </ignoredUnusedDeclaredDependencies>
+        </configuration>
+      </plugin>
+    </plugins>
+  </build>
 </project>
diff --git a/src/main/java/net/revelc/code/formatter/xml/lib/XmlDocumentFormatter.java b/src/main/java/net/revelc/code/formatter/xml/lib/XmlDocumentFormatter.java
index 974e6e2..5dbf4d9 100644
--- a/src/main/java/net/revelc/code/formatter/xml/lib/XmlDocumentFormatter.java
+++ b/src/main/java/net/revelc/code/formatter/xml/lib/XmlDocumentFormatter.java
@@ -17,6 +17,8 @@
 import javax.xml.XMLConstants;
 import javax.xml.parsers.SAXParser;
 import javax.xml.parsers.SAXParserFactory;
+import org.slf4j.Logger;
+import org.slf4j.LoggerFactory;
 import org.xml.sax.ErrorHandler;
 import org.xml.sax.InputSource;
 import org.xml.sax.SAXException;
@@ -25,6 +27,8 @@
 
 public class XmlDocumentFormatter {
 
+    private static final Logger logger = LoggerFactory.getLogger(XmlDocumentFormatter.class);
+
     private final String fDefaultLineDelimiter;
     private final FormattingPreferences prefs;
 
@@ -109,7 +113,8 @@ public String format(String documentText) {
             }
             reader.close();
         } catch (IOException e) {
-            System.out.println(e.getMessage());
+            logger.error("{}", e.getMessage());
+            logger.debug("", e);
         }
         return state.out.toString();
     }
@@ -129,12 +134,13 @@ private void validateWellFormedness(String documentText) {
             XMLReader reader = parser.getXMLReader();
             reader.setErrorHandler(errorHandler);
             reader.parse(new InputSource(new StringReader(documentText)));
-        } catch (Exception exception) {
-            if (!(exception instanceof SAXParseException)
+        } catch (Exception e) {
+            if (!(e instanceof SAXParseException)
                     || !prefs.getWellFormedValidation().equals(FormattingPreferences.WARN)) {
-                throw new IllegalArgumentException(exception);
+                throw new IllegalArgumentException(e);
             }
-            System.err.println("WARN: " + exception.getMessage());
+            logger.error("WARN: {}", e.getMessage());
+            logger.debug("", e);
         }
     }