
|
If you were logged in you would be able to see more operations.
|
|
|
AppFuse
Created: 08/Apr/08 06:52 AM
Updated: 03/May/08 10:00 AM
|
|
| Component/s: |
Tools - AMP
|
| Affects Version/s: |
2.0.1
|
| Fix Version/s: |
2.0.2
|
|
|
File Attachments:
|
1.
amp_diff.txt (3 kb)
2.
diff.txt (1 kb)
|
|
Environment:
|
Windows XP Professional, Maven version: 2.0.8, Java version: 1.5.0_04, Appfuse version 2.0.1
|
|
|
I have a entity called MeterLocation which is already properly generated and installed. When I try to add a new entity called Meter appfuse:gen will wrongly think that it is already generated and when it tries to install it, it complains about the sample data being missing as this was not generated.
The problem is in src\main\java\org\appfuse\mojo\exporter\AppFuseGeneratorMojo.java:
Line 289: if (!hibernateCfgXml.contains(pojoName)) {
Line 332: if (entity.contains(pojoName)) {
I tried to fix the problem and fixed (fix attached) it for my case, but this causes some of the test cases to fail. The reason is that I used Regular Expressions which match greedily and will not always work correctly. I would suggest the use of a proper XML parser. This will be easiest to implement and will not cause the current tests to fail.
|
|
Description
|
I have a entity called MeterLocation which is already properly generated and installed. When I try to add a new entity called Meter appfuse:gen will wrongly think that it is already generated and when it tries to install it, it complains about the sample data being missing as this was not generated.
The problem is in src\main\java\org\appfuse\mojo\exporter\AppFuseGeneratorMojo.java:
Line 289: if (!hibernateCfgXml.contains(pojoName)) {
Line 332: if (entity.contains(pojoName)) {
I tried to fix the problem and fixed (fix attached) it for my case, but this causes some of the test cases to fail. The reason is that I used Regular Expressions which match greedily and will not always work correctly. I would suggest the use of a proper XML parser. This will be easiest to implement and will not cause the current tests to fail. |
Show » |
|
see amp_diff.txt.