[jboss-jira] [JBoss JIRA] Commented: (JBRULES-1778) Drools builder raises errors for non-decision tables XLS files
Michael Anstis (JIRA)
jira-events at lists.jboss.org
Wed Sep 21 04:49:26 EDT 2011
[ https://issues.jboss.org/browse/JBRULES-1778?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12629568#comment-12629568 ]
Michael Anstis commented on JBRULES-1778:
-----------------------------------------
The problem is that we attempt to compile all XLS resources in the project when building the project (no surprise there). However, how can we distinguish between an XLS that isn't a Decision Table and one that is but it just in error? I.E. even if we checked for the presence of "RuleTable" what's to say it's not a Decision Table. I note we don't build CSV files (as I suspect the same issue arises - how can we be certain an arbitrary CSV file is means to be a Decision Table). I propose we either (1) remove XLS validation (consistent with CSV handling) or (2) report it as a warning rather than an error. Any preference?
> Drools builder raises errors for non-decision tables XLS files
> --------------------------------------------------------------
>
> Key: JBRULES-1778
> URL: https://issues.jboss.org/browse/JBRULES-1778
> Project: Drools
> Issue Type: Bug
> Security Level: Public(Everyone can see)
> Components: drools-eclipse (expert)
> Affects Versions: 5.0.0.M1
> Reporter: Edson Tirelli
> Assignee: Michael Anstis
> Fix For: FUTURE
>
>
> As it is now, the eclipse "drools builder" is raising errors for any XLS file that is not a decision table. For instance, in drools-examples there is the ExampleCheese.xls file used for a drools-template example that acuses the following error:
> No RuleTable's were found in spreadsheet. ExampleCheese.xls drools-examples/src/main/rules/org/drools/examples Unknown Drools Error
--
This message is automatically generated by JIRA.
For more information on JIRA, see: http://www.atlassian.com/software/jira
More information about the jboss-jira
mailing list