Oxygen XML Forum

Posted: **Tue Oct 29, 2024 6:00 pm**

Hello,

I am still new to creating schematron rules and currently trying to develop a quickfix that removes all whitespace within a 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>

Posted: **Wed Oct 30, 2024 12:17 pm**

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

Posted: **Thu Oct 31, 2024 10:12 am**

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 tags without inline elements are normalized. Also, because text within 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:
_text_

text
_text_<xref></xref>_text_

text_<xref></xref>_text

This doesnt work:
_text_<xref></xref>

text<xref></xref> (no space before the inline element)
_text_<xref></xref>_text_<xref></xref>

_text_<xref></xref>_text<xref></xref> (space after the second text element gets removed)

Do you see, why it's not working?

Posted: **Thu Oct 31, 2024 10:49 am**

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

Oxygen XML Forum

QuickFix for removing whitespaces

QuickFix for removing whitespaces

Re: QuickFix for removing whitespaces

Re: QuickFix for removing whitespaces

Re: QuickFix for removing whitespaces