From f94865ecb25265f838282d3c9b25143c1328fbcf Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?R=C3=B3bert=20Papp=20=28TWiStErRob=29?=
 <papp.robert.s@gmail.com>
Date: Sun, 31 Jul 2022 14:35:55 +0100
Subject: [PATCH] Fix #239: add link to module

---
 quality/src/main/resources/violations.xsl | 14 +++++++++++---
 1 file changed, 11 insertions(+), 3 deletions(-)

diff --git a/quality/src/main/resources/violations.xsl b/quality/src/main/resources/violations.xsl
index 12ffdf292..8a6d7efc7 100644
--- a/quality/src/main/resources/violations.xsl
+++ b/quality/src/main/resources/violations.xsl
@@ -123,14 +123,17 @@
 				">
 					<xsl:sort />
 					<xsl:variable name="module" select="." />
+					<xsl:variable name="moduleId" select="concat($ruleId, '-', $module)" />
 					<xsl:variable name="moduleCount" select="count($sameRuleViolations[location/@module = $module])" />
 					<span class="toc-count">
 						<xsl:value-of select="$moduleCount" />
 					</span>
 					×
-					<code class="module">
-						<xsl:value-of select="$module" />
-					</code>
+					<a href="#{$moduleId}">
+						<code class="module" data-id="toc-{$moduleId}">
+							<xsl:value-of select="$module" />
+						</code>
+					</a>
 					<xsl:if test="position() != last()" xml:space="preserve">, </xsl:if>
 				</xsl:for-each>
 				)
@@ -162,6 +165,11 @@
 				 is also `true` and not `no node` coerced to `false`) -->
 			<a name="{details/@category}-{source/@reporter}-{details/@rule}" />
 		</xsl:if>
+		<xsl:if test="details/@rule != concat(preceding-sibling::violation[1]/details/@rule, '') or location/@module != concat(preceding-sibling::violation[1]/location/@module, '')">
+			<!-- concat in test makes sure the first violation (which has no preceding nodes)
+				 is also `true` and not `no node` coerced to `false`) -->
+			<a name="{details/@category}-{source/@reporter}-{details/@rule}-{location/@module}" />
+		</xsl:if>
 		<div class="violation" xml:space="preserve">
 			<span class="title">
 				<!-- @formatter:off -->