Codejock Developer Community : Selection and sel change notification bug http://forum.codejock.com/ Copyright (c) 2006-2013 Web Wiz Forums - All Rights Reserved. Fri, 05 Jun 2026 07:57:01 +0000 Tue, 20 Jan 2009 16:50:41 +0000 http://blogs.law.harvard.edu/tech/rss Web Wiz Forums 12.04 360 forum.codejock.com/RSS_post_feed.asp?TID=13154 <![CDATA[Codejock Developer Community]]> http://forum.codejock.com/forum_images/codejock-logo.gif http://forum.codejock.com/ <![CDATA[Selection and sel change notification bug : excellent, thanks so much! ]]> http://forum.codejock.com/forum_posts.asp?TID=13154&PID=44737&title=selection-and-sel-change-notification-bug#44737 Author: adrien
Subject: 13154
Posted: 20 January 2009 at 4:50pm

excellent, thanks so much!]]> Tue, 20 Jan 2009 16:50:41 +0000 http://forum.codejock.com/forum_posts.asp?TID=13154&PID=44737&title=selection-and-sel-change-notification-bug#44737 <![CDATA[Selection and sel change notification bug : Hi, finally I implemented bool...]]> http://forum.codejock.com/forum_posts.asp?TID=13154&PID=44697&title=selection-and-sel-change-notification-bug#44697 Author: mdoubson
Subject: 13154
Posted: 19 January 2009 at 9:42am

Hi,
 finally I implemented bool flag InitialSelectionEnable to control Populate function - you can call it ONCE and not auto-selection for Populate calls. It exposed to VB also
]]> Mon, 19 Jan 2009 09:42:32 +0000 http://forum.codejock.com/forum_posts.asp?TID=13154&PID=44697&title=selection-and-sel-change-notification-bug#44697 <![CDATA[Selection and sel change notification bug : P.S. Check - how many cases of...]]> http://forum.codejock.com/forum_posts.asp?TID=13154&PID=44675&title=selection-and-sel-change-notification-bug#44675 Author: mdoubson
Subject: 13154
Posted: 19 January 2009 at 12:13am

P.S. Check - how many cases of Populate() calls you have - you need to modify all with same approach (3 lines instead of 1)
 
You can also get rid from Focus

wndReport.SelectionEnable(FALSE);

wndReport.Populate();

wndReport.ShowRowFocus(FALSE);

wndReport.SelectionEnable(TRUE);

and turn it back when you need using call:

wndReport.ShowRowFocus(TRUE);

]]> Mon, 19 Jan 2009 00:13:23 +0000 http://forum.codejock.com/forum_posts.asp?TID=13154&PID=44675&title=selection-and-sel-change-notification-bug#44675 <![CDATA[Selection and sel change notification bug : I can agree with last sentence...]]> http://forum.codejock.com/forum_posts.asp?TID=13154&PID=44674&title=selection-and-sel-change-notification-bug#44674 Author: mdoubson
Subject: 13154
Posted: 19 January 2009 at 12:11am

I can agree with last sentence - this is result of Populate function call. This function check IsSelectionEnabled() flag and if TRUE - make selection internally - without notification. If you call

wndReport.SelectionEnable(FALSE);

wndReport.Populate();

wndReport.SelectionEnable(TRUE);

you will have expected behaviour. After this point keyboard steps or mouse click will generate same effect - and will send notification to app. So you can have only one your own function

OnReportSelectionChanged for customization. If it what you want? If so - no need to have mode complicated model I propose you because my idea realized different reaction on keyboard and on mouse! (It have sense in some business model!)

]]> Mon, 19 Jan 2009 00:11:24 +0000 http://forum.codejock.com/forum_posts.asp?TID=13154&PID=44674&title=selection-and-sel-change-notification-bug#44674 <![CDATA[Selection and sel change notification bug : p.p.s (sorry) failing that if...]]> http://forum.codejock.com/forum_posts.asp?TID=13154&PID=44673&title=selection-and-sel-change-notification-bug#44673 Author: adrien
Subject: 13154
Posted: 18 January 2009 at 11:55pm

p.p.s (sorry) failing that if the report control must select a row of its own accord, it should notify this.]]> Sun, 18 Jan 2009 23:55:50 +0000 http://forum.codejock.com/forum_posts.asp?TID=13154&PID=44673&title=selection-and-sel-change-notification-bug#44673 <![CDATA[Selection and sel change notification bug : p.s. I&#039;m not trying to get...]]> http://forum.codejock.com/forum_posts.asp?TID=13154&PID=44672&title=selection-and-sel-change-notification-bug#44672 Author: adrien
Subject: 13154
Posted: 18 January 2009 at 11:54pm

p.s. I'm not trying to get it to behave differently between keyboard vs mouse user input.

I'm trying to get it to stop selecting the row by itself, with NO human input.
]]> Sun, 18 Jan 2009 23:54:26 +0000 http://forum.codejock.com/forum_posts.asp?TID=13154&PID=44672&title=selection-and-sel-change-notification-bug#44672 <![CDATA[Selection and sel change notification bug : My complaint is that the control...]]> http://forum.codejock.com/forum_posts.asp?TID=13154&PID=44671&title=selection-and-sel-change-notification-bug#44671 Author: adrien
Subject: 13154
Posted: 18 January 2009 at 11:45pm

My complaint is that the control selects a row when there was

a) no click on the row
b) no keyboard input (e.g no keyboard selection)

furthermore it selects said row without sending XTP_NM_REPORT_SELCHANGED

If going from 0 selected rows to 1 selected row is not a change in selection then I don't know what is.

So, I guess we disagree whether this is a bug or not, I remain unconvinced that it is not.

I'm not going to go through the dozens of places I use report control to add 3 handlers to "fix" this problem, instead I will more likely have to hack the report control source to have it simply not decide to randomly select rows all by itself. ]]> Sun, 18 Jan 2009 23:45:41 +0000 http://forum.codejock.com/forum_posts.asp?TID=13154&PID=44671&title=selection-and-sel-change-notification-bug#44671 <![CDATA[Selection and sel change notification bug : I know that it works - I tested...]]> http://forum.codejock.com/forum_posts.asp?TID=13154&PID=44669&title=selection-and-sel-change-notification-bug#44669 Author: mdoubson
Subject: 13154
Posted: 18 January 2009 at 10:44pm

I know that it works - I tested it before sending to you. You should consider different situation with the way you react on mouse click - sometims such click means select a word in InPlace editor (normal behaviour of edit control), sometimes you allow to select subitem, sometimes - not.
Don't expect ReportControl to change existed behaviour - maximum I can do if will be enough client asking about such feature - introduce some binary or enumerated flag - mouse click VS keyboard navigation - select or not  (looks like 4 different cases - NO-NO, YES-NO and so on).
But not in release 13 for sure.
But usually such specific reaction implemented on app level - this is way control is sending notification and app can choose that to do...
 
Btw - as I explain you in very begining of our discussion - you use wrong term in this topic title as  this is NOT a BUG - this is proper reaction on specific action - Focus vs Selection.
]]> Sun, 18 Jan 2009 22:44:53 +0000 http://forum.codejock.com/forum_posts.asp?TID=13154&PID=44669&title=selection-and-sel-change-notification-bug#44669 <![CDATA[Selection and sel change notification bug : ok I guess that would work... or...]]> http://forum.codejock.com/forum_posts.asp?TID=13154&PID=44668&title=selection-and-sel-change-notification-bug#44668 Author: adrien
Subject: 13154
Posted: 18 January 2009 at 10:32pm

ok I guess that would work...

or maybe the report control could be fixed?

I've got quite a few report controls I'm using that do this...]]> Sun, 18 Jan 2009 22:32:57 +0000 http://forum.codejock.com/forum_posts.asp?TID=13154&PID=44668&title=selection-and-sel-change-notification-bug#44668 <![CDATA[Selection and sel change notification bug : Essentially you should deal on...]]> http://forum.codejock.com/forum_posts.asp?TID=13154&PID=44666&title=selection-and-sel-change-notification-bug#44666 Author: mdoubson
Subject: 13154
Posted: 18 January 2009 at 7:32pm

Essentially you should deal on app level with 3 events:

ON_NOTIFY(XTP_NM_REPORT_SELCHANGED, XTP_ID_REPORT_CONTROL, OnReportSelectionChanged)

ON_NOTIFY(XTP_NM_REPORT_FOCUS_CHANGING, XTP_ID_REPORT_CONTROL, OnReportFocusChanged)

ON_NOTIFY(XTP_NM_REPORT_LBUTTONDOWN, XTP_ID_REPORT_CONTROL, OnReportLbDown)

void CTaskListView::OnReportLbDown(NMHDR* /*pNotifyStruct*/, LRESULT* /*result*/)

{

m_bAllowSelect = TRUE;

}

void CTaskListView::OnReportFocusChanged(NMHDR* /*pNotifyStruct*/, LRESULT* result)

{

CXTPReportSelectedRows* pSelectedRows = GetReportCtrl().GetSelectedRows();

if (pSelectedRows)

pSelectedRows->Clear();

}

void CTaskListView::OnReportSelectionChanged(NMHDR* /*pNotifyStruct*/, LRESULT* /*result*/)

{

CXTPReportSelectedRows* pSelectedRows = GetReportCtrl().GetSelectedRows();

if (pSelectedRows)

{

if (!m_bAllowSelect)

pSelectedRows->Clear();

else

{

m_bAllowSelect = FALSE;

pSelectedRows->Add(GetReportCtrl().GetFocusedRow());

}

}

}

and flag
BOOL m_bAllowSelect;
which you should to be FALSE end the end of initialization  together with cleaning selection:
 

int CTaskListView::OnCreate(LPCREATESTRUCT lpCreateStruct)

{

........
m_bAllowSelect = FALSE;

GetReportCtrl().GetSelectedRows()->Clear();

}
 
]]> Sun, 18 Jan 2009 19:32:51 +0000 http://forum.codejock.com/forum_posts.asp?TID=13154&PID=44666&title=selection-and-sel-change-notification-bug#44666