extra blank line with `<br/>` and XPath conditional

[pr-apes]

@pgundlach,

the shorter text suggested in https://github.com/speedata/publisher/discussions/478#discussioncomment-5345430 might have hit a bug (or a rather uncommon behavior).

Given the following data.xml contents:

<data>
  <row>
    <cell>a b</cell>
    <cell>a&lt;br/&gt;b</cell>
    <cell>a b</cell>
  </row>
</data>

And the following layout contents:

<Layout xmlns="urn:speedata.de:2009/publisher/en"
  xmlns:sd="urn:speedata:2009/publisher/functions/en">

  <Record element="data">
    <Output>
      <Text>
        <ForAll select="row">
          <Switch>
            <Case test="position() > 0">
              <ForAll select="cell">
                <Paragraph>
                  <Value select="if (position() > 0) then sd:decode-html(.) else" />
                  <Value select="sd:decode-html(.)" />
                </Paragraph>
              </ForAll>
            </Case>
          </Switch>
        </ForAll>
      </Text>
    </Output>
  </Record>

</Layout>

Final output is:

It seems that <Value select="if (position() > 0) then sd:decode-html(.) else" /> interprets <br/> as a full blank line (which is wrong) and <Value select="sd:decode-html(.)" /> handles it as a line break inside the same paragraph (which is right).

I have no idea what might be triggering this.

It would be great, if you could take a look at it.

Many thanks for your help.

[pgundlach]

Interesting, this looks strange. Thank you for posting this issue!

[pgundlach]

I believe this should be fixed by rewriting the XPath parser, which will take some time.

[pr-apes]

Many thanks for your reply, @pgundlach.

This doesn't seem to be a minor task.

I'm still making some tests, so it may be wise to close the issue (myself).

If I find something that I cannot avoid, I will reopen the issue.

Many thanks for your help.