Codejock Developer Community : XTP_NM_REPORT_SORTORDERCHANGED crash

XTP_NM_REPORT_SORTORDERCHANGED crash : No, I do. You can use derived...

Sun, 08 Nov 2009 11:28:00 +0000

Author: mdoubson
Subject: 15532
Posted: 08 November 2009 at 11:28am

No, I do. You can use derived classes with extra details used

XTP_NM_REPORT_SORTORDERCHANGED crash : FACT: My post shows a bug...

Wed, 04 Nov 2009 11:28:36 +0000

Author: znakeeye
Subject: 15532
Posted: 04 November 2009 at 11:28am

FACT: My post shows a bug OR erroneous documentation.

You obviously did not read it.

XTP_NM_REPORT_SORTORDERCHANGED crash : piece of XTPRecordHeader.cpp...

Mon, 02 Nov 2009 11:46:05 +0000

Author: mdoubson
Subject: 15532
Posted: 02 November 2009 at 11:46am

piece of XTPRecordHeader.cpp file ver 12.1:

m_pControl->SendNotifyMessage(XTP_NM_REPORT_PRESORTORDERCHANGED);

if (bGroupOrderChanged)

m_pControl->Populate();

else

m_pControl->ReSortRows();

m_pControl->SendNotifyMessage(XTP_NM_REPORT_SORTORDERCHANGED);

XTP_NM_REPORT_SORTORDERCHANGED crash : Using 12.0.1. Same code for 13.2... From...

Mon, 02 Nov 2009 10:39:12 +0000

Author: znakeeye
Subject: 15532
Posted: 02 November 2009 at 10:39am

Using 12.0.1. Same code for 13.2...

From source comments:

This is what the DOCUMENTATION SAYS:

// BEGIN_MESSAGE_MAP(CReportSampleView, CXTPReportView)
//     ON_NOTIFY(XTP_NM_REPORT_SORTORDERCHANGED, XTP_ID_REPORT_CONTROL, OnReportSortOrderChanged)
// END_MESSAGE_MAP()
//
// void CReportSampleView::OnReportSortOrderChanged(NMHDR* pNotifyStruct, LRESULT* /*result*/)
// {
//     XTP_NM_REPORTRECORDITEM* pItemNotify = (XTP_NM_REPORTRECORDITEM*) pNotifyStruct;
//     ASSERT(pItemNotify != NULL);
//
//     // TODO: Handle command.
// }

From source:

This is what the CODE DOES:

m_pControl->SendNotifyMessage(XTP_NM_REPORT_PRESORTORDERCHANGED); // Second parameter is NULL!

LRESULT CXTPReportControl::SendNotifyMessage(UINT nMessage, NMHDR* pNMHDR) const <------ NULL!
{
if (!IsWindow(m_hWnd))
return 0;

NMHDR nmhdr;
if (pNMHDR == NULL)
pNMHDR = &nmhdr; <---------- assignment!

pNMHDR->hwndFrom = GetSafeHwnd();
pNMHDR->idFrom = GetDlgCtrlID();
pNMHDR->code = nMessage;

CWnd *pOwner = GetOwner();
if (pOwner && IsWindow(pOwner->m_hWnd))
return pOwner->SendMessage(WM_NOTIFY, pNMHDR->idFrom, (LPARAM)pNMHDR); <--- NOT the struct we expect!
else
return 0;
}

WM_NOTIFY does NOT send a XTP_NM_REPORTRECORDITEM struct for XTP_NM_REPORT_PRESORTORDERCHANGED message! So you will get a CRASH!

It's a bug. Period. End of discussion. Please fix it.