Bug 745030

Summary: Maven Bulk Importer can't work with multiple files
Product: [JBoss] JBoss Enterprise BRMS Platform 5 Reporter: Tomas Schlosser <tschloss>
Component: BRM (Guvnor)Assignee: manstis
Status: NEW --- QA Contact: Lukáš Petrovický <lpetrovi>
Severity: low Docs Contact:
Priority: unspecified    
Version: BRMS 5.2.0.GACC: atangrin, lpetrovi
Target Milestone: ---   
Target Release: ---   
Hardware: Unspecified   
OS: Unspecified   
Whiteboard:
Fixed In Version: Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: Type: Bug
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:

Description Tomas Schlosser 2011-10-11 08:50:40 UTC
Description of problem:
As long as the only files in the folder are drl files, MBI can import them all, but if it's one xls and any other type (drl or xls) MBI ends with "Can't parse more than one .xls decision table file in a single directory".

Version-Release number of selected component (if applicable):
BRMS-5.2.0.CR1

How reproducible:
Every time

Steps to Reproduce:
1. open included test case
2. run it
  
Actual results:
Exception in thread "main" org.drools.compiler.DroolsParserException: Can't parse more than one .xls decision table file in a single directory [/home/tschloss/QA/BRMS/maven-bulk-importer-test/test-resources/org/jboss/qa/guvnor]
	at org.jboss.drools.guvnor.importgenerator.PackageFile.parseRuleFiles(PackageFile.java:157)
	at org.jboss.drools.guvnor.importgenerator.PackageFile.buildPackageForDirectory(PackageFile.java:115)
	at org.jboss.drools.guvnor.importgenerator.PackageFile.buildPackageForDirectory(PackageFile.java:120)
	at org.jboss.drools.guvnor.importgenerator.PackageFile.buildPackageForDirectory(PackageFile.java:120)
	at org.jboss.drools.guvnor.importgenerator.PackageFile.buildPackageForDirectory(PackageFile.java:120)
	at org.jboss.drools.guvnor.importgenerator.PackageFile.buildPackages(PackageFile.java:89)
	at org.jboss.drools.guvnor.importgenerator.ImportFileGenerator.run(ImportFileGenerator.java:272)
	at org.jboss.qa.guvnor.MavenBulkImporterTest.prepareXmlRepository(MavenBulkImporterTest.java:297)
	at org.jboss.qa.guvnor.MavenBulkImporterTest.main(MavenBulkImporterTest.java:47)

Expected results:
Imported xls decision table and all rules

Additional info:

Comment 2 Geoffrey De Smet 2012-04-05 09:01:01 UTC
Bulk importer has had a major refactor for 5.4 (Mat's pull request), making it hard to backport patches.

This looks like a new feature, so it should be implemented in the community version first.

Comment 3 Geoffrey De Smet 2012-04-11 13:21:21 UTC
Lukas,
Are you ok with postponing these issues for 5.4 (see message above)?
If ok, please remove jboss‑brms‑5.3.0+.

Comment 4 Lukáš Petrovický 2012-04-12 06:23:10 UTC
(In reply to comment #3)
> Lukas,
> Are you ok with postponing these issues for 5.4 (see message above)?
> If ok, please remove jboss‑brms‑5.3.0+.

I can't "-", but I can "?". Anyway, Maven Bulk Importer is out of scope for the release, that much is clear.

Comment 6 Geoffrey De Smet 2013-03-05 13:39:14 UTC
Reassign to the guvnor team, now that I am working full time on planner. So these issues aren't forgotten.