From 9bcf077e8eb3e1cc7e5c158265e80a0aa655a78c Mon Sep 17 00:00:00 2001
From: Grahame Grieve <grahameg@gmail.ccom>
Date: Tue, 22 Oct 2024 09:26:19 +1030
Subject: [PATCH] Add version warnings on IGs to qa.html

---
 .../renderers/ValidationPresenter.java        | 31 +++++++++++++++++--
 pom.xml                                       |  2 +-
 2 files changed, 29 insertions(+), 4 deletions(-)

diff --git a/org.hl7.fhir.publisher.core/src/main/java/org/hl7/fhir/igtools/renderers/ValidationPresenter.java b/org.hl7.fhir.publisher.core/src/main/java/org/hl7/fhir/igtools/renderers/ValidationPresenter.java
index e084cde2..2d1cf8e4 100644
--- a/org.hl7.fhir.publisher.core/src/main/java/org/hl7/fhir/igtools/renderers/ValidationPresenter.java
+++ b/org.hl7.fhir.publisher.core/src/main/java/org/hl7/fhir/igtools/renderers/ValidationPresenter.java
@@ -305,13 +305,14 @@ public ValidationMessage getVm() {
   boolean showReferenceMessages = false;
   boolean noExperimentalContent = false;
   boolean displayWarnings = false;
+  private List<String> versionProblems;
 
   
   public ValidationPresenter(String statedVersion, String igVersion, IGKnowledgeProvider provider, IGKnowledgeProvider altProvider, String root, String packageId, String altPackageId, 
       String toolsVersion, String currentToolsVersion, RealmBusinessRules realm, PreviousVersionComparator previousVersionComparator, IpaComparator ipaComparator, IpsComparator ipsComparator,
       String dependencies, String csAnalysis, String pubReqCheck, String globalCheck, String copyrightYear, IWorkerContext context,
       Set<String> r5Extensions, List<StructureDefinition> modifierExtensions, String draftDependencies,
-      List<FetchedResource> noNarratives, List<FetchedResource> noValidation, boolean noValidate, boolean noGenerate, DependentIGFinder dependentIgs, TerminologyClientManager txServers) {
+      List<FetchedResource> noNarratives, List<FetchedResource> noValidation, boolean noValidate, boolean noGenerate, DependentIGFinder dependentIgs, TerminologyClientManager txServers, List<String> versionProblems) {
     super();
     this.statedVersion = statedVersion;
     this.igVersion = igVersion;
@@ -341,6 +342,7 @@ public ValidationPresenter(String statedVersion, String igVersion, IGKnowledgePr
     this.draftDependencies = draftDependencies;
     this.globalCheck = globalCheck;
     this.txServers = txServers;
+    this.versionProblems = versionProblems;
     ruleDateCutoff = Date.from(LocalDate.now().minusMonths(1).atStartOfDay(ZoneId.systemDefault()).toInstant());
     determineCode();
   }
@@ -733,6 +735,10 @@ public List<FetchedFile> genQAHtml(String title, List<FetchedFile> files, String
     b.append(genEnd());
     b.append(genStartInternal());
     int id = 0;
+    for (String vp : versionProblems) {
+      b.append(genDetails(vp, id));
+      id++;
+    }
     for (ValidationMessage vm : linkErrors) {
       b.append(genDetails(vm, id));
       id++;
@@ -1032,12 +1038,12 @@ public static List<ValidationMessage> filterMessages(List<ValidationMessage> mes
   
   private final String detailsTemplateTxLink = 
       "   <tr style=\"background-color: $color$\">\r\n"+
-      "     <td><b>$path$</b></td><td><b>$level$</b></td><td><b>$msg$</b>$comment$ (from <a href=\"qa-txservers.html#$txsrvr$\">$txsrvr$</ta>, see <a href=\"$tx$\">log</a>)</td>\r\n"+
+      "     <td><b>$path$</b></td><td><b>$level$</b></td><td><b>$msg$</b>$comment$ (from <a href=\"qa-txservers.html#$txsrvr$\">$txsrvr$<ta>, see <a href=\"$tx$\">log</a>)</td>\r\n"+
       "   </tr>\r\n";
   
   private final String detailsTemplateTxNoLink = 
       "   <tr style=\"background-color: $color$\">\r\n"+
-      "     <td><b>$path$</b></td><td><b>$level$</b></td><td><b>$msg$</b>$comment$ (from <a href=\"qa-txservers.html#$txsrvr$\">$txsrvr$</ta>)</td>\r\n"+
+      "     <td><b>$path$</b></td><td><b>$level$</b></td><td><b>$msg$</b>$comment$ (from <a href=\"qa-txservers.html#$txsrvr$\">$txsrvr$<ta>)</td>\r\n"+
       "   </tr>\r\n";
   
   private final String detailsTemplateWithExtraDetails = 
@@ -1543,6 +1549,25 @@ private String genDetails(ValidationMessage vm, int id) {
     return t.render();
   }
 
+  private String genDetails(String vp, int id) {
+    String tid = null;
+    tid = detailsTemplate;
+    ST t = template(tid);
+    t.add("path", "");
+    t.add("pathlink", "");      
+    t.add("level", "warning");
+    t.add("color", colorForLevel(IssueSeverity.WARNING, false));
+    t.add("halfcolor", halfColorForLevel(IssueSeverity.WARNING, false));
+    t.add("id", "l"+id);
+    t.add("mid", "");
+    t.add("msg", vp);
+    t.add("msgdetails", vp);
+    t.add("comment", "");
+    t.add("tx", "");
+    t.add("txsrvr", "");
+    return t.render();
+  }
+
   private String getServer(String server) {
     if (txServers.getServerMap().containsKey(server)) {
       return txServers.getServerMap().get(server).getClient().getAddress();
diff --git a/pom.xml b/pom.xml
index 05d6aae5..299fd592 100644
--- a/pom.xml
+++ b/pom.xml
@@ -8,7 +8,7 @@
     <packaging>pom</packaging>
 
     <properties>
-        <core_version>6.3.32</core_version>
+        <core_version>6.3.33-SNAPSHOT</core_version>
         <maven_surefire_version>3.0.0-M5</maven_surefire_version>
         <apache_poi_version>5.2.1</apache_poi_version>
         <okhttp.version>4.11.0</okhttp.version>