Bug 1217255

Summary: [GSS] (6.1.z) migration tool generates non-escaped chars that cause the whole exported package empty
Product: [Retired] JBoss BRMS Platform 6 Reporter: Andrea Hoffer <ahoffer>
Component: Migration ToolAssignee: Jan <jschatte>
Status: CLOSED EOL QA Contact: Tibor Zimanyi <tzimanyi>
Severity: high Docs Contact:
Priority: high    
Version: 6.1.0CC: ahoffer, ghu, jschatte, kverlaen, mbaluch, tzimanyi
Target Milestone: CR1   
Target Release: ---   
Hardware: Unspecified   
OS: Unspecified   
Whiteboard:
Fixed In Version: Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: 1217117 Environment:
Last Closed: 2020-03-27 18:34:31 UTC Type: Bug
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Embargoed:
Bug Depends On: 1217117    
Bug Blocks: 1220521, 1220522    

Comment 2 Andrea Hoffer 2015-04-30 19:30:21 UTC
Description:

When running the migration tool shipped by BPMS 6.1 against repository generated by BRMS 5.2, several migrated rule packages are empty.
The console output of running the migration tool shows some error due to non-escaped characters in the exported xml:

[Fatal Error] xxxx.xml:1:104: The value of attribute "name" associated with an element type "null" must not contain the '<' character.
org.xml.sax.SAXParseException; systemId: file:/xxx/jboss-brms-bpmsuite-6.1.0.GA-redhat-2-migration-tool/bin/tmp-jcr2vfs/xxx.xml; lineNumber: 1; columnNumber: 104; The value of attribute "name" associated with an element type "null" must not contain the '<' character.
	at com.sun.org.apache.xerces.internal.parsers.DOMParser.parse(DOMParser.java:257)


The problematic exported rule in the xxxx.xml looks like:

]]></textContent><history><assets></assets></history></asset><asset name="myasset <= myattrib ..." type="brl" ...>

Please note that the char "<" in the name attribute is supposed to be escaped.

Comment 3 Jan 2015-05-04 12:48:20 UTC
Fixed in master: http://github.com/droolsjbpm/drools-wb/commit/1c7ff68cd

Comment 4 Jan 2015-05-04 16:47:11 UTC
Additionnal fix: http://github.com/droolsjbpm/drools-wb/commit/8acd3d70f

Comment 5 Jan 2015-05-04 16:59:55 UTC
Cherry-picked into 6.2.x: http://github.com/droolsjbpm/drools-wb/compare/620d357...16b697f

Comment 6 Tibor Zimanyi 2015-05-19 11:49:03 UTC
Verified running migration on repositories from parent BZ.