adding minor comments

src/main/plugin/iso19139.MERIDIAN/index-fields/index.xsl

@@ -99,10 +99,11 @@
     <xsl:variable name="identifier" as="xs:string"
                   select="gmd:fileIdentifier/gco:CharacterString[. != '']"/>
 
-
+<!-- this LanguageCode cannot appear in this context - therefore, this should never exist? Therefore pointless -->
     <xsl:variable name="mainLanguageCode" as="xs:string?"
                   select="gmd:language[1]/gmd:LanguageCode/
                         @codeListValue[normalize-space(.) != '']"/>
+						
     <xsl:variable name="mainLanguage" as="xs:string?"
                   select="if ($mainLanguageCode) then $mainLanguageCode else
                     gmd:language[1]/gco:CharacterString[normalize-space(.) != '']"/>
@@ -112,7 +113,8 @@
                         gmd:languageCode/gmd:LanguageCode/
                           @codeListValue[normalize-space(.) != '']"/>
 
-    <!-- Record is dataset if no hierarchyLevel -->
+    <!-- Record is dataset if no hierarchyLevel 
+	Should series of datasets be a dataset?-->
     <xsl:variable name="isDataset" as="xs:boolean"
                   select="
                       count(gmd:hierarchyLevel[gmd:MD_ScopeCode/@codeListValue='dataset']) > 0 or
@@ -130,7 +132,8 @@
       <documentType>metadata</documentType>
       <documentStandard>iso19139.MERIDIAN</documentStandard>
 
-      <!-- Index the metadata document as XML -->
+      <!-- Index the metadata document as XML
+		This does nothing - should we keep it? -->
       <document>
         <!--<xsl:value-of select="saxon:serialize(., 'default-serialize-mode')"/>-->
       </document>
@@ -196,6 +199,7 @@
 
 
       <!-- # Resource type -->
+	  <!-- why can't we just always grab the scopecode rather than do this pointless test? -->
       <xsl:choose>
         <xsl:when test="$isDataset">
           <resourceType>dataset</resourceType>
@@ -250,6 +254,7 @@
           <resourceTitle>
             <xsl:value-of select="gmd:title/gco:CharacterString/text()"/>
           </resourceTitle>
+		  <!-- hide field if no alt title? -->
           <resourceAltTitle>
             <xsl:value-of
               select="gmd:alternateTitle/gco:CharacterString/text()"/>
@@ -955,7 +960,7 @@
 
   <xsl:template mode="index-contact" match="*[gmd:CI_ResponsibleParty]">
     <xsl:param name="fieldSuffix" select="''" as="xs:string"/>
-
+	<!-- this area needs the personal names!!! -->
     <!-- Select the first child which should be a CI_ResponsibleParty.
     Some records contains more than one CI_ResponsibleParty which is
     not valid and they will be ignored.