Bug 744765 - Maven Bulk Importer cannot merge content of multiple files
Summary: Maven Bulk Importer cannot merge content of multiple files
Keywords:
Status: NEW
Alias: None
Product: JBoss Enterprise BRMS Platform 5
Classification: JBoss
Component: BRM (Guvnor)
Version: BRMS 5.2.0.GA
Hardware: Unspecified
OS: Unspecified
unspecified
low
Target Milestone: ---
: ---
Assignee: manstis
QA Contact: Lukáš Petrovický
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2011-10-10 11:57 UTC by Tomas Schlosser
Modified: 2022-11-15 23:14 UTC (History)
2 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed:
Type: Bug


Attachments (Terms of Use)
Simple reproducer (12.40 KB, application/x-zip-compressed)
2011-10-10 11:59 UTC, Tomas Schlosser
no flags Details

Description Tomas Schlosser 2011-10-10 11:57:34 UTC
Description of problem:
When multiple files contain declaration/import/global they are not properly merged and only one package's header is used.

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

How reproducible:
Every time

Steps to Reproduce:
1. Open attached project
2. Run main method
3. Check test-resources/repository.xml
  
Actual results:
          <sv:property sv:name="drools:content" sv:type="String"><sv:value><![CDATA[package org.jboss.qa.guvnor

declare Dog
	name : String
end




declare Dog
	name : String
end

]]></sv:value></sv:property>

Expected results:
          <sv:property sv:name="drools:content" sv:type="String"><sv:value><![CDATA[package org.jboss.qa.guvnor

import java.util.List

global String logPrefix

declare Dog
	name : String
end

]]></sv:value></sv:property>


Additional info:
Also please note that Dog declared fact is mentioned twice!

Comment 1 Tomas Schlosser 2011-10-10 11:59:18 UTC
Created attachment 527223 [details]
Simple reproducer

Comment 3 Geoffrey De Smet 2012-04-05 09:00:38 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 4 Geoffrey De Smet 2012-04-11 13:21:06 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 5 Lukáš Petrovický 2012-04-12 06:23:45 UTC
I can't "-", but I can "?". Anyway, Maven Bulk Importer is out of scope for the release, that much is clear.

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


Note You need to log in before you can comment on or make changes to this bug.