Bug 724504 (BRMS-450)

Summary: BRMS 5.1, ER4 build - with Modeshape as the repo - the mortgage samples fail to load
Product: [JBoss] JBoss Enterprise BRMS Platform 5 Reporter: Len DiMaggio <ldimaggi>
Component: ExamplesAssignee: Tihomir Surdilovic <tsurdilo>
Status: CLOSED NEXTRELEASE QA Contact:
Severity: unspecified Docs Contact:
Priority: urgent    
Version: unspecifiedKeywords: TestBlocker
Target Milestone: ---   
Target Release: 5.1.0 GA   
Hardware: Unspecified   
OS: Unspecified   
URL: http://jira.jboss.org/jira/browse/BRMS-450
Whiteboard:
Fixed In Version: Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
BRMS 5.1 ER4
Last Closed: 2010-12-03 04:21:16 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:
Attachments:
Description Flags
Screen shot 2010-11-22 at 1.49.29 PM.png none

Description Len DiMaggio 2010-11-22 20:42:05 UTC
Affects Testing: Blocks Testing
securitylevel_name: Public

See attached screenshot.

Comment 2 Len DiMaggio 2010-11-22 20:42:59 UTC
Attachment: Added: Screen shot 2010-11-22 at 1.49.29 PM.png


Comment 3 Tihomir Surdilovic 2010-11-23 14:38:11 UTC
The issue seems to be that ModeShape does not know how to deal with base64Binary encoded values. In the Guvnor repository xml for mortgages:
<sv:node sv:name="ApplicantDsl">
<sv:property sv:name="jcr:primaryType" sv:type="Name">
<sv:value>drools:assetNodeType</sv:value>
</sv:property>
.....
<sv:property sv:name="drools:content" sv:type="String">
<sv:value xsi:type="xs:base64Binary">W3doZW5dV2hlbiB0aGUgY3JlZGl0IHJhdGluZyBpcyB7cmF0aW5nOkVOVU06QXBwbGljYW50LmNyZWRpdFJhdGluZ30gPSBhcHBsaWNhbnQ6QXBwbGljYW50KGNyZWRpdFJhdGluZz09IntyYXRpbmd9IikNClt0aGVuXUFwcHJvdmUgdGhlIGxvYW4gPSBhcHBsaWNhbnQuc2V0QXBwcm92ZWQodHJ1ZSkNCg0KW3doZW5dV2hlbiB0aGUgYXBwbGljYW50IGRhdGVzIGlzIGFmdGVyIHtkb3M6REFURTpkZWZhdWx0fSA9IGFwcGxpY2FudDpBcHBsaWNhbnQoYXBwbGljYXRpb25EYXRlPiJ7ZG9zfSIpDQoNClt3aGVuXVdoZW4gdGhlIGFwcGxpY2FudCBhcHByb3ZhbCBpcyB7Ym9vbDpCT09MRUFOOmNoZWNrZWR9ID0gYXBwbGljYW50OkFwcGxpY2FudChhcHByb3ZlZD09e2Jvb2x9KQ0KDQpbd2hlbl1XaGVuIHRoZSBhZ2VzIGlzIGxlc3MgdGhhbiB7bnVtOjE/WzAtOV0/WzAtOV19ID0gYXBwbGljYW50OkFwcGxpY2FudChhZ2U8e251bX0p
</sv:value>
</sv:property>
......
</sv:node> 


I will try to change this to just string value in the export xml and it should help, also will file a MS Jira as seems to be a missing feature currently.



Comment 4 prakash aradhya 2010-11-23 14:38:48 UTC
Ideally this should be fixed in Modeshape.  If this is not possible at all, we may need to tackle it in the Sample.  But lets make sure we put that as a known issue that 64bit encoding works only with JackRabbit and not currently in Modeshape

Comment 5 Van Halbert 2010-11-23 14:39:53 UTC
can we get the example files used and how one would duplicate.

Comment 6 Tihomir Surdilovic 2010-11-23 14:44:49 UTC
Link: Added: This issue is related to MODE-1026


Comment 7 Tihomir Surdilovic 2010-11-23 14:48:13 UTC
I created MODE-1026 an put this:
****
Attached the repository xml Guvnor is using as the samples repository.
Let me know if you need anything else?
The steps on how to reproduce this error are described in https://access.redhat.com/kb/docs/DOC-43075 
****

let me know if you need anything else.

Comment 8 Randall Hauch 2010-11-23 18:50:47 UTC
Attachment: Added: MODE-1026-fix-src.patch
Attachment: Added: MODE-1026-fix-test.patch


Comment 9 Randall Hauch 2010-11-23 18:54:03 UTC
Attachment: Removed: MODE-1026-fix-src.patch 


Comment 10 Randall Hauch 2010-11-23 18:54:09 UTC
Attachment: Removed: MODE-1026-fix-test.patch 


Comment 11 Randall Hauch 2010-11-23 19:04:54 UTC
The fix for MODE-1026 has been committed into our SVN (both trunk and the '2.2.x' branch). I've also attached a src patch and test patch to the issue, in case they are needed to create a patch JAR. If this fixes the import issue, please mark MODE-1026 as resolved/closed; otherwise, please provide more information regarding the new failure.

Comment 12 Tihomir Surdilovic 2010-12-03 04:21:16 UTC
resolved for ER5

Comment 13 David Le Sage 2010-12-08 05:24:37 UTC
Labels: Added: rn-dlesage rn-not-needed


Comment 14 David Le Sage 2010-12-08 05:24:49 UTC
Release Notes Docs Status: Added: Not Required
Writer: Added: dlesage