XMLStreamFilter with HTMLSubfilter doesn't group back the nested translation tags correctly with M22

[Former user]

Original issue 375 created by 143.ravik... on 2013-10-28T12:10:12.000Z:

The following is my source file content -

<SOLUTIONS>
<![CDATA[Attachments in <img width="13" height="15" src="mail-p_attach_all_01.png" alt="paper clip" /> end tag]]>
</SOLUTIONS>

This generates the following text units -

<trans-unit id="1" resname="{group:sd1,tu:sd1_tu1}" xml:space="preserve">
<source xml:lang="en">Attachments in <ph id="1">[#$sd1_dp1]</ph> end tag</source>
<target xml:lang="de-DE" state="new"><mrk mid="0" mtype="seg">Attachments in <ph id="1">[#$sd1_dp1]</ph> end tag</mrk></target>
</trans-unit>
<trans-unit id="2" resname="{group:sd1,tu:sd1_tu2}" xml:space="preserve">
<source xml:lang="en">paper clip</source>
<target xml:lang="de-DE" state="new"><mrk mid="0" mtype="seg">paper clip</mrk></target>
</trans-unit>

It fails to insert the translation for the place holder with the generated file content as -

<SOLUTIONS>
<![CDATA[Attachments in -ERR:REF-NOT-FOUND- end tag]]>
</SOLUTIONS>

The yml file used by by my html filter contains -

assumeWellformed: true
global_cdata_subfilter: okf_html
preserve_whitespace: false

elements:
'.*':
ruleTypes: [EXCLUDE]
solutions:
ruleTypes: [INCLUDE]

[Former user]

Comment 1. originally posted by @ysavourel on 2013-10-28T12:15:46.000Z:

[Former user]

Comment 2. originally posted by @ysavourel on 2013-10-28T14:10:56.000Z:

[Former user]

Comment 3. originally posted by 143.ravik... on 2013-11-22T14:40:24.000Z:

were you able to reproduce it ?
Let me know if you need any more info, will be glad to provide.

[Former user]

Comment 4. originally posted by @jhargraveiii on 2013-11-22T15:19:17.000Z:

God timing. I'm working on similar issues today. I will add a unit test to our current dev branch and try to reproduce it. The snippet and rules should be enough.

[Former user]

Comment 5. originally posted by @ysavourel on 2013-11-22T18:17:39.000Z:

When I implement a simple test case I don't see the error. Could this be an xliff issue? The tests pass in M22 and M24

See attached unit test

[Former user]

Comment 6. originally posted by @ysavourel on 2013-11-22T18:45:13.000Z:

even running a full rainbow xliff project this seems to work - at least in the latest M24 snapshot

[Jim Hargrave (OLD)]

fixed