Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.

Bug 315444

Summary:	API for IStyleProcessor changed in bug 313652
Product:	z_Archived	Reporter:	Kathy Chan <kathy>
Component:	BIRT	Assignee:	Birt-Chart-inbox <Birt-Chart-inbox>
Status:	RESOLVED WONTFIX	QA Contact:
Severity:	normal
Priority:	P3	CC:	Lionel.wyl
Version:	2.6.0
Target Milestone:	2.6.1
Hardware:	PC
OS:	Windows XP
Whiteboard:

Description Kathy Chan

2010-06-02 14:23:13 EDT

Just want to confirm that the API change for IStyleProcessor as a result of bug 313652 is intentional or not?

TPTP started noticing the problem when we picked up BIRT RC2 this week.  This resulted in a TPTP build break which we need to react to (TPTP bug 315200 has the detail on the problem caused by this change).  In short, the BIRT class that changed API (checked in on 05/19 for RC2) was: 

org.eclipse.birt.chart.style.IStyleProcessor 

A few methods such as processStyle, defaultBgColor, setDefaultBackgroundColor and getDefaultBackgroudColor were introduced. 

Just want to confirm that this change in API is intentional and is going to stay.  Since this API change is very late in the Helios cycle (after M7), we would appreciate if API changes are required late in the cycle is announced in the Eclipes Cross Project distribution list.  Thanks!

Comment 1 Yulin Wang

2010-06-02 21:57:26 EDT

The change is going to stay.
Could you customize your style processor by extending BaseStyleProcessor in the same package, rather than implementing IStyleProcessor? New base class will minimize the interface change.

Comment 2 Yulin Wang

2010-06-12 04:57:04 EDT

Set wontfix since this is must-change and user has a workaround.