Codejock Developer Community : Urgent problem with CXTPPropertyGridItemFont

Urgent problem with CXTPPropertyGridItemFont : Yes, will not be updated. So try...

Mon, 28 May 2007 07:01:18 +0000

Author: Oleg
Subject: 7179
Posted: 28 May 2007 at 7:01am

Yes, will not be updated. So try not delete items while dialogs are visible. use code I wrote in 21 May.

Urgent problem with CXTPPropertyGridItemFont : Good. But what happens if the...

Sun, 27 May 2007 16:20:47 +0000

Author: znakeeye
Subject: 7179
Posted: 27 May 2007 at 4:20pm

Good. But what happens if the item is deleted and re-inserted into the grid while the dialog is visible? Then its value will not be updated when the user presses OK?

Urgent problem with CXTPPropertyGridItemFont : Hi, We added InternalAddRef();...

Tue, 22 May 2007 01:19:01 +0000

Author: Oleg
Subject: 7179
Posted: 22 May 2007 at 1:19am

Hi,

We added InternalAddRef(); in begining of OnInplaceButtonDown and InternalRelease(); in the end, so even if item was removed from grid it will be deleted only in the end of OnInplaceButtonDown method.

Urgent problem with CXTPPropertyGridItemFont : Yes, it works =) What kind of...

Mon, 21 May 2007 07:14:05 +0000

Author: znakeeye
Subject: 7179
Posted: 21 May 2007 at 7:14am

Yes, it works =)

What kind of method will you use for the mutual exclusion? Please implement a general solution that would work for a custom-made propertygrid item where a dialog is shown. Recall your Grid sample where this can be found:

virtual void OnValueChanged(CString strValue)

{

if (strValue == _T(""))

{

CSomeDialog dlg(m_pGrid);
dlg.DoModal();

}

...

}

Thanks!

Urgent problem with CXTPPropertyGridItemFont : You right. Please try this changed...

Mon, 21 May 2007 04:59:15 +0000

Author: Oleg
Subject: 7179
Posted: 21 May 2007 at 4:59am

You right. Please try this changed method:

void CDrawView::OnActivateView(BOOL bActivate, CView* pActiveView,
CView* pDeactiveView)
{
CView::OnActivateView(bActivate, pActiveView, pDeactiveView);

CMainFrame* pMainFrame = (CMainFrame*)AfxGetMainWnd();

if (!bActivate && !pMainFrame->IsWindowEnabled())
return;

// invalidate selections when active status changes
if (m_bActive != bActivate)
{
  if (bActivate) // if becoming active update as if active
   m_bActive = bActivate;
  if (!m_selection.IsEmpty())
   OnUpdate(NULL, HINT_UPDATE_SELECTION, NULL);
  m_bActive = bActivate;

}

if (!bActivate && pMainFrame->MDIGetActive() != GetParentFrame())
{
pMainFrame->UpdatePropertyGridContent(0);
}
}

For next maintaince release we will update our code to not crush even if item was removed while dialog is visible.

Thanks, let me know it it will work for you.

Urgent problem with CXTPPropertyGridItemFont : No, it doesn't - until I...

Mon, 21 May 2007 03:00:29 +0000

Author: znakeeye
Subject: 7179
Posted: 21 May 2007 at 3:00am

No, it doesn't - until I press OK/Cancel. When I close the font dialog, UpdatePropertyGridContent is actually called before DoModal() is finished. I think this is a race condition. It might work in some cases.

Any further ideas?

Urgent problem with CXTPPropertyGridItemFont : Strange. it works for me. If...

Mon, 21 May 2007 01:14:11 +0000

Author: Oleg
Subject: 7179
Posted: 21 May 2007 at 1:14am

Strange. it works for me.

If you set breakpoint after if (!IsWindowEnabled()) return; in the UpdatePropertyGridContent, show font dialog box and switch application does it fired?

Urgent problem with CXTPPropertyGridItemFont : You're on the right track,...

Sat, 19 May 2007 07:15:53 +0000

Author: znakeeye
Subject: 7179
Posted: 19 May 2007 at 7:15am

You're on the right track, but that fix doesn't work.

As soon as UpdatePropertyGridContent is called (not just from that line), the destructor of the font item is called - causing the heap error. I tried adding this in CMainFrame::UpdatePropertyGridContent():

if (!IsWindowEnabled()) return;

... but it didn't help either.

Urgent problem with CXTPPropertyGridItemFont : Hello, Yes, thanks, this is problem...

Sat, 19 May 2007 02:03:38 +0000

Author: Oleg
Subject: 7179
Posted: 19 May 2007 at 2:03am

Hello,

Yes, thanks, this is problem of sample - not code.

in CDrawView::OnActivateView replace

if (!bActivate)
((CMainFrame*)AfxGetMainWnd())->UpdatePropertyGridContent(0);

if (!bActivate && AfxGetMainWnd()->IsWindowEnabled())
((CMainFrame*)AfxGetMainWnd())->UpdatePropertyGridContent(0);

Urgent problem with CXTPPropertyGridItemFont : Open the DrawClient sample, open...

Fri, 18 May 2007 19:27:45 +0000

Author: znakeeye
Subject: 7179
Posted: 18 May 2007 at 7:27pm

Open the DrawClient sample, open MainFrm.cpp and go to line 264.

Then add this:

LOGFONT lf = { 0 };
pCategorySelected->AddChildItem(new CXTPPropertyGridItemFont(_T("font test"), lf));

Debug the application and then click on button of the newly added font property. The font dialog appears. Now Alt+Tab to another application. Switch back to the font dialog and press ok. BOOM! We get a heap error at line 164 in xtppropertygriditemfont.cpp:

CString strValue = FontToString(lf);

In some cases you break deep inside the C heap library. What's going on here?

Please help!