Bug 780061 (SOA-2426)

Summary: Decision Table: String Index Out Of Range
Product: [JBoss] JBoss Enterprise SOA Platform 5 Reporter: Lukáš Petrovický <lpetrovi>
Component: Documentation, JBoss RulesAssignee: Jervis Liu <jliu>
Status: CLOSED WONTFIX QA Contact:
Severity: high Docs Contact:
Priority: high    
Version: 5.1.0.ER2CC: mproctor
Target Milestone: ---Keywords: Regression
Target Release: 5.2.0 GA   
Hardware: Unspecified   
OS: Unspecified   
URL: http://jira.jboss.org/jira/browse/SOA-2426
Whiteboard:
Fixed In Version: Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2011-06-20 19:48:47 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
DiscountDT.xls
none
dt.png none

Description Lukáš Petrovický 2010-10-15 09:50:13 UTC
Affects: Release Notes
project_key: SOA

The attached XLS decision table, when used in a knowledge base, produces the following exception:

java.lang.StringIndexOutOfBoundsException: String index out of range: 78
	at java.lang.String.checkBounds(String.java:409)
	at java.lang.String.<init>(String.java:450)
	at jxl.biff.StringHelper.getString(StringHelper.java:164)
	at jxl.read.biff.WriteAccessRecord.<init>(WriteAccessRecord.java:56)
	at jxl.read.biff.WorkbookParser.parse(WorkbookParser.java:821)
	at jxl.Workbook.getWorkbook(Workbook.java:271)
	at jxl.Workbook.getWorkbook(Workbook.java:253)
	at org.drools.decisiontable.parser.xls.ExcelParser.parseFile(ExcelParser.java:74)
	at org.drools.decisiontable.SpreadsheetCompiler.compile(SpreadsheetCompiler.java:90)
	at org.drools.decisiontable.SpreadsheetCompiler.compile(SpreadsheetCompiler.java:69)
	at org.drools.decisiontable.DecisionTableProviderImpl.compileStream(DecisionTableProviderImpl.java:29)
	at org.drools.decisiontable.DecisionTableProviderImpl.loadFromInputStream(DecisionTableProviderImpl.java:19)
	at org.drools.compiler.DecisionTableFactory.loadFromInputStream(DecisionTableFactory.java:18)
	at org.drools.compiler.PackageBuilder.addKnowledgeResource(PackageBuilder.java:482)
	at org.drools.builder.impl.KnowledgeBuilderImpl.add(KnowledgeBuilderImpl.java:34)
        ... (Rest of the stack trace omitted, because it originally came from the ESB BRP action.)

I intended to create a reproducer for the issue, but then I found out that simply putting the XLS in a Drools project in JBDS results in the very same error. (See attached screenshot.)

This is a regression compared to SOA 5.0.

Comment 1 Lukáš Petrovický 2010-10-15 09:51:31 UTC
The attachments mentioned in the bug description.

Comment 2 Lukáš Petrovický 2010-10-15 09:51:31 UTC
Attachment: Added: DiscountDT.xls
Attachment: Added: dt.png


Comment 3 Lukáš Petrovický 2010-10-15 10:01:53 UTC
To clarify - the Eclipse used isn't actually JBDS. It's Eclipse 3.6 w. latest JBoss Tools milestone. (aka. the future JBDS 4.)

Comment 4 Lukáš Petrovický 2010-10-15 10:25:48 UTC
Link: Added: This issue related BRMS-388


Comment 5 Lukáš Petrovický 2010-10-27 08:09:21 UTC
See the linked BRMS issue for resolution. This is now a release notes issue.

Comment 6 John Graham 2010-11-18 17:19:25 UTC
Updated for documentation release note requirement for 5.1

Comment 7 Anne-Louise Tangring 2010-12-06 16:56:27 UTC
Affects: Added: [Release Notes]


Comment 9 Dana Mison 2011-01-05 00:12:24 UTC
Writer: Added: Darrin


Comment 10 Dana Mison 2011-01-27 08:34:51 UTC
Release Notes Docs Status: Added: Documented as Known Issue
Release Notes Text: Added: Importing an Microsoft Excel spreadsheet into in a Knowledge Base will cause an exception to be thrown ( StringIndexOutOfBoundsException ) if the spreadsheet was created in Excel 95 or earlier. This is because of an issue in the JXL library which is used to handle these files. This can be worked around by opening and saving the spreadsheet in Microsoft Excel 97 or greater or with OpenOffice.org Calc. 

This will be fixed in a future release. 


Comment 11 Dana Mison 2011-01-27 08:36:35 UTC
Hi Jervis, I've added this to the Known Issues for SOA 5.1 - is this all that is required to be done for this issue does it need to be kept open for a future fix ?

Comment 13 Len DiMaggio 2011-09-29 16:22:12 UTC
Link: Added: This issue relates to SOA-3409