QuickFix for removing whitespaces

[DavidK]

Hello,

I am still new to creating schematron rules and currently trying to develop a quickfix that removes all whitespace within a <p> tag. The schematron rule works correctly, but the quickfix also eliminates spaces before and after inline elements. Is there a simple solution for this, or do I need to reconstruct the p-tag using the concat function?
I would be grateful for any help.

Code: Select all

  <pattern id="normalize-p-tag">
    <rule context="p">
      <assert test="normalize-space(.) = ." sqf:fix="normalize">whitespaces found</assert>
      <sqf:fix id="normalize">
        <sqf:description>
          <sqf:title>normalize p-tag</sqf:title>
        </sqf:description>
        <sqf:replace match="node()" select="normalize-space(.)"/>
      </sqf:fix>
    </rule>
  </pattern>

[tavy]

Hello,

I believe you can create a quick fix that normalizes spaces while adding a space before and after the text. For the first and last pieces of text, you can add a space after and before them, respectively.

Code: Select all

<sqf:fix id="normalize">
    <sqf:description>
        <sqf:title>normalize p-tag</sqf:title>
    </sqf:description>
    <sqf:replace match="text()[position() = 1]" select="concat(normalize-space(), ' ')"/>
    <sqf:replace match="text()[position() != 1 and position() != last()]" select="concat(' ', normalize-space(), ' ')"/>
    <sqf:replace match="text()[position() = last()]" select="concat(' ', normalize-space())"/>
</sqf:fix>

Best Regards,
Octavian

[DavidK]

Hi Octavian,

thank you for your solution! I’ve already learned something from it.

I added the line <sqf:replace match="text()" select="normalize-space()"/> at the beginning to ensure that even <p> tags without inline elements are normalized. Also, because text within <p>tags without any inline elements would be position = 1 and position = last at the same time (correct me if I'm wrong) and your quickfix would add spaces in that case, I added a second condition in your first and last match statement.

Code: Select all

  <sqf:fix id="normalize">
        <sqf:description>
          <sqf:title>normalize p-tag</sqf:title>
        </sqf:description>

        <sqf:replace match="text()" select="normalize-space()"/>
        
        <sqf:replace match="text()[position() = 1 and position() != last()]" select="concat(normalize-space(), ' ')"/>

        <sqf:replace match="text()[position() != 1 and position() != last()]" select="concat(' ', normalize-space(), ' ')"/>
        
        <sqf:replace match="text()[position() = last() and position() != 1]" select="concat(' ', normalize-space())"/>
      </sqf:fix> 

Even though the logic seems fine to me, it still doesn’t quite work as expected.

This already works:
<p>_text_</p> <p>text</p>
<p>_text_<xref></xref>_text_</p> <p>text_<xref></xref>_text</p>

This doesnt work:
<p>_text_<xref></xref></p> <p>text<xref></xref></p> (no space before the inline element)
<p>_text_<xref></xref>_text_<xref></xref></p> <p>_text_<xref></xref>_text<xref></xref></p> (space after the second text element gets removed)

Do you see, why it's not working?

[tavy]

Hi David,

If you have inline elements at the beginning and end of a paragraph, you need to verify the position of the nodes within the paragraph. Additionally, check if the node is a text node. It should look something like this:

Code: Select all

<sqf:fix id="normalize">
    <sqf:description>
        <sqf:title>normalize p-tag</sqf:title>
    </sqf:description>
    <sqf:replace match="node()[position() = 1][self::text()]" select="concat(normalize-space(), ' ')"/>
    <sqf:replace match="node()[position() != 1 and position() != last()][self::text()]" select="concat(' ', normalize-space(), ' ')"/>
    <sqf:replace match="node()[position() = last()][self::text()]" select="concat(' ', normalize-space())"/>
</sqf:fix>

Best Regards,
Octavian