Hi at all.

I'm student in this years Google Sommer of Code program. I'm contributing
BIRT with adding new chart types like donut, venn and radar.

As I tried creating new SeriesDefinition for the donut chart I found a
conflict in build()-method of Generator.class.


Chart cmRuntime = (Chart) EcoreUtil.copy( cmDesignTime );

This line of code loses the new SeriesDefintion.

What I did:

* replace this line with
Chart cmRuntime = cmDesignTime;

I found a patch which is a kind of workaround and does the same as I did,
I think:


### Eclipse Workspace Patch 1.0 #P org.eclipse.birt.chart.engine
Index: src/org/eclipse/birt/chart/factory/Generator.java
===================================================================
RCS file:
/cvsroot/birt/source/org.eclipse.birt.chart.engine/src/org/eclipse/birt/chart/factory/Generator.java,
v retrieving revision 1.75.2.2 diff -u -r1.75.2.2 Generator.java ---
src/org/eclipse/birt/chart/factory/Generator.java 12 Dec 2008 03:42:15
-0000 1.75.2.2
+++ src/org/eclipse/birt/chart/factory/Generator.java 19 Jun 2009 13:10:37
-0000 @@ -926,7 +926,8 @@ {
// re-init chart script context.
ChartScriptContext csc = new ChartScriptContext( );
- Chart cmRunTime = (Chart) EcoreUtil.copy( cmDesignTime );
+ // Chart cmRunTime = (Chart) EcoreUtil.copy( cmDesignTime );
+ Chart cmRunTime = cmDesignTime;
csc.setChartInstance( cmRunTime );
csc.setExternalContext( externalContext );
csc.setULocale( rtc.getULocale( ) );



My question here is what's the solution? Is this a known bug, is this
patch a permanent solution or just a temporary?

Best regards,
Karsten