Codejock Developer Community : UpdateDiagram() buggy!

UpdateDiagram() buggy! : While I'm at it, how...

Wed, 07 Sep 2011 10:46:51 +0000

Author: znakeeye
Subject: 18901
Posted: 07 September 2011 at 10:46am

While I'm at it, how do you set a title text above/under each pie?

UpdateDiagram() buggy! : And... just as usual the most...

Sun, 04 Sep 2011 19:11:29 +0000

Author: znakeeye
Subject: 18901
Posted: 04 September 2011 at 7:11pm

And... just as usual the most critical functions are not virtual. Unbelievable! :(

XTPChartContent.h:

void UpdateDiagram();

CXTPChartDeviceContext* CreateDeviceContext(CXTPChartContainer* pContainer, HDC hDC, CRect rcBounds, BOOL bWindowDC);

UpdateDiagram() buggy! : Consider this type of diagram:Clearly,...

Sun, 04 Sep 2011 19:02:37 +0000

Author: znakeeye
Subject: 18901
Posted: 04 September 2011 at 7:02pm

Consider this type of diagram:

Clearly, the coloring is wrong. I've tried playing with SetColorEach() and SetLegendVisible() but with no success. Digging into the source code reveals the issue. UpdateDiagram() does not check for the above! Palette index should be reset for each series!

void CXTPChartContent::UpdateDiagram()
{
int i;
int nIndex = 0;

    for (i = 0; i < GetSeries()->GetCount(); i++)
    {
        CXTPChartSeries* pSeries = GetSeries()->GetAt(i);
        if (!pSeries->IsVisible())
            continue;

if (!pSeries->GetStyle())
continue;

        if (pSeries->GetStyle()->IsColorEach())
        {
            for (int j = 0; j < pSeries->GetPoints()->GetCount(); j++)
            {
                pSeries->GetPoints()->GetAt(j)->m_nPaletteIndex = nIndex++;
            }
        }
        else
        {
            pSeries->m_nPaletteIndex = nIndex++;
        }
    }
}