Update restrictReferences logic for removing unused definitions.

Remove unused definitions not defined in this spec.

Author: gkellogg

common/common.js

@@ -5,11 +5,8 @@
 //
 // Add class "preserve" to a definition to ensure it is not removed.
 //
-// the termlist is in a block of class "termlist", so make sure that
-// has an ID and put that ID into the termLists array so we can
-// interrogate all of the included termlists later.
+// the termlist is in a block of class "termlist".
 const termNames = [] ;
-const termLists = [] ;
 const termsReferencedByTerms = [] ;
 
 function restrictReferences(utils, content) {
@@ -33,7 +30,6 @@ function restrictReferences(utils, content) {
 
   const $container = $(".termlist", base) ;
   const containerID = $container.makeID("", "terms") ;
-  termLists.push(containerID) ;
   return (base.innerHTML);
 }
 
@@ -99,14 +95,25 @@ require(["core/pubsubhub"], (respecEvents) => {
       // delete any terms that were not referenced.
       for (const term in termNames) {
         const $p = $("#"+term);
-        if ($p.length > 0) {
-          const tList = $p.getDfnTitles();
-          $p.parent().next().remove(); // remove dd
-          $p.remove();                 // remove dt
-          for (const item of tList) {
-            if (respecConfig.definitionMap[item]) {
-              delete respecConfig.definitionMap[item];
-            }
+        // Remove term definitions inside a dt, where data-cite does not start with shortname
+        if ($p === undefined) { continue; }
+        if (!$p.parent().is("dt")) { continue; }
+        if (($p.data("cite") || "").toLowerCase().startsWith(respecConfig.shortName)) { continue; }
+
+        const $dt = $p.parent();
+        const $dd = $dt.next();
+
+        // If the associated dd contains a dfn which is _not_ in termNames, warn
+        if ($dd.children("dfn").length > 0) {
+          console.log(term + " definition contains definitions " + $dd.children("dfn").attr("id"))
+        }
+        console.log("drop term " + term);
+        const tList = $p.getDfnTitles();
+        $dd.remove(); // remove dd
+        $dt.remove(); // remove dt
+        for (const item of tList) {
+          if (respecConfig.definitionMap[item]) {
+            delete respecConfig.definitionMap[item];
           }
         }
       }

index.html

@@ -352,8 +352,7 @@ <h2>Terminology</h2>
       and defines terms specific to JSON-LD.</p>
 
     <div data-include="common/terms.html"
-         data-oninclude="exportReferences">
-      <!-- Mark these terms for export to be shared by related specs-->
+         data-oninclude="restrictReferences">
     </div>
 
   <section>
@@ -362,8 +361,7 @@ <h4>Algorithm Terms</h4>
     <p>The Following terms are used within specific algorithms.</p>
 
     <div data-include="common/algorithm-terms.html"
-         data-oninclude="exportReferences">
-      <!-- Mark these terms for export to be shared by related specs-->
+         data-oninclude="restrictReferences">
     </div>
   </section>