View Issue Details

IDProjectCategoryView StatusLast Update
0006641JEDI VCL00 JVCL Componentspublic2019-07-03 08:02
ReportertstoicescuAssigned To 
PrioritynormalSeverityminorReproducibilityalways
Status newResolutionopen 
Product Version3.48 
Target VersionFixed in Version 
Summary0006641: JvDBTreeView - insert record
Description/Jvcl/Run/JvDBTreeView.pas:
Bug: Editing items's parent in State = dsInsert, it do Post and (wrong) Insert again (a new item). Normally it should edit same item.
so: OldState isn't needed: it should never reinsert new item.
Stack:
TCustomTreeView.CNNotify ... TVN_SELCHANGEDA, TVN_SELCHANGEDW: ... Change
TJvCustomDBTreeView.Change ::
 case OldState of
      dsEdit:
        FDataLink.DataSet.Edit;
      dsInsert:
        FDataLink.DataSet.Insert; /// Edit
    end;
TagsNo tags attached.

Activities

obones

2018-07-18 16:05

administrator   ~0021549

Could you check if the issue is still present in the latest GIT content? If yes, please provide the zipped sources of an application showing this.

mh

2019-04-27 14:00

reporter   ~0021760

I found the code snippet referenced above but it looks exactly like above. Question: what exactly shall be changed there? Shall in dsInsert also the Edit method be called? Or is the code as is already fixed?

tstoicescu

2019-07-03 08:02

reporter   ~0021871

Shall in dsInsert also the Edit method be called? Yes. Don't need to do test case: it, it's logical if already saved last record inserted cannot insert another. Because it was already Post before. Just re edit.
Is the code as is already fixed? No. We put the fix in comment, not reverse.
Thank you.

Issue History

Date Modified Username Field Change
2018-06-25 10:10 tstoicescu New Issue
2018-07-18 16:05 obones Note Added: 0021549
2018-07-18 16:05 obones Status new => feedback
2019-04-27 14:00 mh Note Added: 0021760
2019-07-03 08:02 tstoicescu Note Added: 0021871
2019-07-03 08:02 tstoicescu Status feedback => new