Page 1 of 1

[FRQ]:expose more parameter for event OnBeforeAdd

Posted: Tue Feb 14, 2012 8:40 am
by 16561241
Hello

It will be better to expose more parameter for event OnBeforeAdd for TChartSeries as:

TSeriesOnBeforeAdd=Function(Sender:TChartSeries;Const AXValue,AYValue:TChartValue;
Const ALabel:String; AColor:TColor):Boolean of object;


This will give user a much more easy way to eliminate un-qualified value be added to series.

Thanks.

Re: [FRQ]:expose more parameter for event OnBeforeAdd

Posted: Tue Feb 14, 2012 11:53 am
by yeray
Hi,

I've added it to the wish list to be studied for inclusion in future releases (TV52016034).
Thanks for reporting it.

Re: [FRQ]:expose more parameter for event OnBeforeAdd

Posted: Tue Feb 14, 2012 1:08 pm
by 16561241
Hello

Thanks for quick response.

However, I need to modify my request as following to avoid introducing trouble to your component.

1. the suggested new definition of TSeriesOnBeforeAdd be modified as:
TSeriesOnBeforeAdd=Function(Sender:TChartSeries;Const AXValue,AYValue:TChartValue;
Const ALabel:String; var AColor:TColor):Boolean of object;
AColor is now a VAR parameter.
This is very useful if the user want to replace the unqualified value to a NULL value by setting:
AColor:=clNone;
By this NULL value be added instead of an unqualified value, the chart will looked more neat and clean.
You case see the different in attachment

2. Please also remember to make test on 3D series in case
2.1 a false value is returned by this event
2.2 a true value is returned by this event, but Y value is replaced by a NULL

Re: [FRQ]:expose more parameter for event OnBeforeAdd

Posted: Wed Feb 15, 2012 9:15 am
by yeray
Hi,

Thanks again for your comments.
All the possibilities we can think on will be tested when this enhancement will be studied for inclusion, but I've added your comments to the ticket description for not to forget the concrete aspects you mentioned.