xslt-quality_writing.sch

<?xml version="1.0" encoding="UTF-8"?>
<pattern xmlns="http://purl.oclc.org/dsdl/schematron" 
  xmlns:xsl="http://www.w3.org/1999/XSL/Transform" 
  xmlns:xs="http://www.w3.org/2001/XMLSchema"
  xmlns:xd="http://www.oxygenxml.com/ns/doc/xsl"
  xmlns:sqf="http://www.schematron-quickfix.com/validator/process"
  xml:lang="en"
  id="xslt-quality_writing">
  
  <xd:doc>
    <xd:desc>
      <xd:p>These rules are about XSLT writing quality</xd:p>
    </xd:desc>
  </xd:doc>
  
  <rule context="xsl:attribute | xsl:namespace | xsl:variable | xsl:param | xsl:with-param">
    <xd:doc>
      <xd:desc xml:lang="en">If you don't need special XSLT instruction to calculate a variable, a param (or any other attribute, namespace) then use the short "select=" syntax that make your code less verbose and easier to read.</xd:desc>
      <xd:desc xml:lang="fr">Si vous n'avez pas besoin d'instruction XSLT spécifiques pour calculer une variable, un paramètre (ou tout autre attribut, namespace) utilisez la syntaxe rapide "select=" qui rendra votre code moins verbeux et plus facile à lire.</xd:desc>
    </xd:doc>
    <report id="xslt-quality_use-select-attribute-when-possible"
      test="not(@select) and (count(* | text()[normalize-space(.)]) = 1) and (count(xsl:value-of | xsl:sequence | text()[normalize-space(.)]) = 1)"
      sqf:fix="sqf_move-text-to-select-attribute">
      Use @select to assign a value to <name/>
    </report>
  </rule>
  
  <sqf:fixes>
    <sqf:fix id="sqf_move-text-to-select-attribute">
      <sqf:description>
        <sqf:title xml:lang="en">Move the text content to a "select" attribute</sqf:title>
      </sqf:description>
      <sqf:add match="." target="select" node-type="attribute">'<value-of select="."/>'</sqf:add>
      <sqf:delete match="./text()"/>
    </sqf:fix>
  </sqf:fixes>
  
</pattern>