[checkstyle] OnlyTabIndentationCheck to report filename

Signed-off-by: Holger Friedrich <mail@holger-friedrich.de>

Author: holgerfriedrich

custom-checks/checkstyle/src/main/java/org/openhab/tools/analysis/checkstyle/OnlyTabIndentationCheck.java

@@ -18,6 +18,7 @@
 import org.openhab.tools.analysis.checkstyle.api.AbstractStaticCheck;
 
 import com.puppycrawl.tools.checkstyle.api.FileText;
+import com.puppycrawl.tools.checkstyle.api.MessageDispatcher;
 
 /**
  * Checks whether whitespace characters are use instead of tabs in xml files
@@ -42,7 +43,11 @@ public void setFileTypes(String[] value) {
 
     @Override
     protected void processFiltered(File file, FileText fileText) {
+        MessageDispatcher dispatcher = getMessageDispatcher();
+        dispatcher.fireFileStarted(file.getPath());
         processTabIndentationCheck(fileText);
+        fireErrors(file.getAbsolutePath());
+        dispatcher.fireFileFinished(file.getPath());
     }
 
     private void processTabIndentationCheck(FileText fileText) {

sat-plugin/src/main/resources/report/prepare_checkstyle.xslt

@@ -34,9 +34,37 @@
 	<xsl:template match="file">
 		<xsl:variable name="new_name">
 		<xsl:choose>
-			<xsl:when test="contains(@name,'feature.xml')">
+			<xsl:when test="ends-with(@name,'.xml')">
+			    <!-- this should find the last occurrence of /src/ and construct
+				a new path from parent folders name, /src/ and remaining path/filename 
+				 -->
 				<xsl:variable name="temp_name" select="translate(@name, '\', '/')" />
-				<xsl:value-of select="concat('src/', substring-after($temp_name, '/src/'))" />
+				<xsl:variable name="after_last_src">
+					<xsl:call-template name="substring-after-last">
+						<xsl:with-param name="input" select="$temp_name" />
+						<xsl:with-param name="marker" select="'/src/'" />
+					</xsl:call-template>
+				</xsl:variable>
+				<xsl:variable name="before_last_src" select="substring-before($temp_name, concat('/src/', $after_last_src))" />
+				<xsl:variable name="parent">
+					<xsl:choose>
+						<xsl:when test="string-length($before_last_src) &gt; 0">
+							<xsl:call-template name="substring-after-last">
+								<xsl:with-param name="input" select="$before_last_src" />
+								<xsl:with-param name="marker" select="'/'" />
+							</xsl:call-template>
+						</xsl:when>
+						<xsl:otherwise />
+					</xsl:choose>
+				</xsl:variable>
+				<xsl:choose>
+					<xsl:when test="string-length($parent) &gt; 0">
+						<xsl:value-of select="concat($parent, '/src/', $after_last_src)" />
+					</xsl:when>
+					<xsl:otherwise>
+						<xsl:value-of select="concat('src/', $after_last_src)" />
+					</xsl:otherwise>
+				</xsl:choose>
 			</xsl:when>
 			<xsl:when test="contains(@name,'src')">
 				<xsl:variable name="temp_name" select="translate(@name, '/', '.')" />