View Issue Details

IDProjectCategoryView StatusLast Update
0005052JEDI VCL00 JVCL Componentspublic2013-12-13 11:21
Reporterabbe2003czAssigned Toobones 
PrioritynormalSeveritymajorReproducibilityalways
Status resolvedResolutionsuspended 
Product Version3.36 
Target VersionDaily / GITFixed in Version 
Summary0005052: TJvDBGrid handles OnKeyPress event twice
DescriptionA TJvDBGrid handles an OnKeyPress() event twice. One time correctly, one time with var Key equal to #0.
Notice: "dgEditing" in "Options" must be set to "true".
Additional Information  begin
>>>> inherited; <<<<< ...ist it necessary? (line 2720)

    if Assigned(FCurrentControl) then
    begin
      if FCurrentControl.Visible then
        PostMessage(FCurrentControl.Handle, WM_CHAR, Msg.CharCode, Msg.KeyData);
    end
    else
      if InplaceEditor = nil then
        DoKeyPress(Msg); // This is needed to trigger an onKeyPressed event when the
                         // default editor hasn't been created because the data type
                         // of the selected field is binary or memo.
  end;
TagsNo tags attached.

Activities

wpostma416

2009-12-11 17:32

developer   ~0016989

from the user saying "Notice: "dgEditing" in "Options" must be set to "true". " I get the idea that this "twice" bug might be a conflict between the JVCL component and the changes in the base VCL class in Delphi 2010.

In other words, this bug probably isn't a bug, until we hit Delphi 2010, which as one of its not-so-bright moves, changed how grid header click handling works. Introducing the breaking change "dgEditing" into Options. (It's on by default in all projects to which you add a DBGrid, but off whenever an existing Delphi 2009 or earlier project is imported.)

A change that is {$ifdefd for delphi 2010} and higher is probably likely to be needed here.

obones

2012-02-27 16:10

administrator   ~0019587

The sources of a demo application would be really nice to have here

obones

2012-06-11 17:13

administrator   ~0019834

Any news?

obones

2013-12-13 11:21

administrator   ~0020765

No news, suspending the issue

Issue History

Date Modified Username Field Change
2009-12-09 19:57 abbe2003cz New Issue
2009-12-11 17:32 wpostma416 Note Added: 0016989
2009-12-18 12:05 obones Status new => acknowledged
2009-12-18 12:05 obones Target Version => Daily / SVN
2012-02-27 16:10 obones Note Added: 0019587
2012-02-27 16:10 obones Status acknowledged => feedback
2012-06-11 17:13 obones Note Added: 0019834
2013-12-13 11:21 obones Note Added: 0020765
2013-12-13 11:21 obones Status feedback => resolved
2013-12-13 11:21 obones Resolution open => suspended
2013-12-13 11:21 obones Assigned To => obones