Home General
New Blog Posts: Merging Reports - Part 1 and Part 2

TppSystemVariable and CalcOrder

edited November 2009 in General
Hi,

RbPro 11.06, Delphi 2006, Vista SP2


The calculation order of TppVariables is not correct if there are also
TppSystemVariables on the same band. The TppSystemVariables are also counted
in the CalcOrder. So if you first place a TppSystemVariable on a band and
then add a TppVariable it wil have CalcOrder = 1.
And then the calculation order dialog is not working correct anymore because
its confused about the number of TppVariables and the CalcOrder of those
variables.

I think the cause is that TppSystemVariables are also added to
TppBand.Variables. Is that necessary? TppSystemVariables are not calculated
as TppVariables and the CalOrder dialog filters them out.

I tried changing the test in TppBand.AddObject

from
if (lComponent is TppCustomVariable) then
AddVariable(lComponent);

to
if (lComponent is TppVariable) then
AddVariable(lComponent);

and everything seems OK. CalcOrder is correct and TppSystemVariables still
give the same result.

Is this modification correct or am i missing something?




Regards,

Max Paay
Quadrant Software bv.
The Netherlands.






__________ Information from ESET NOD32 Antivirus, version of virus signature database 4636 (20091125) __________

The message was checked by ESET NOD32 Antivirus.

http://www.eset.com

Comments

  • edited November 2009
    Hi Max,

    Looking at the changes you made, you are correct that the TppSystemVariable
    probably should not be added to the calc order. I will change this in our
    source and it will be included in the next release.

    --
    Regards,

    Nico Cizik
    Digital Metaphors
    http://www.digital-metaphors.com

    Best Regards,

    Nico Cizik
    Digital Metaphors
    http://www.digital-metaphors.com
  • edited November 2009
    Hi Nico,


    Thanks for your reply, i will leave the modification in then.


    Regards,


    Max Paay
    Quadrant Software bv.
    The Netherlands.


This discussion has been closed.