View Issue Details
ID | Project | Category | View Status | Date Submitted | Last Update |
---|---|---|---|---|---|
0000116 | .NET API | Implementation Bug | public | 2007-02-22 21:35 | 2008-05-10 08:50 |
Reporter | Simon Guard | Assigned To | Randy Armstrong | ||
Priority | normal | Severity | major | Reproducibility | always |
Status | resolved | Resolution | fixed | ||
Summary | 0000116: HDA.NET Browser does not report branch as having children if GetItemID fails. | ||||
Description | This may not be a bug if I have not implemented IOPCHDA_Browser::GetItemID properly. In my implementation of IOPCHDA_Browser::GetItemID I return E_FAIL if a 'pure' branch is provided for szNode. If this behaviour is wrong then this is not a bug. The problem I have is that BrowseElement.HasChildren is not being set for branches when GetItemID returns E_FAIL. The following code is the problem: OpcCom.Hda.Browser.cs: line 345 // create new element objects.
} I think the correct behaviour should be to set 'element.HasChildren = isBranch' regardless of whether GetItemId fails or not (or call m_Browser.GetBranchPosition instead of GetItemId to get the element.ItemName when isBranch is true). Note that this problem exists in OPC.NET API 1.30 and 2.00. | ||||
Tags | No tags attached. | ||||
|
Here is a fix that works for me... /// <summary> return branch; /// <summary>
... |
|
Changed code to: BrowseElement element = new BrowseElement(); element.Name = name; try elements.Add(element); |
|
Will be part of Release 2.00.101 |
Date Modified | Username | Field | Change |
---|---|---|---|
2007-02-22 21:35 | Simon Guard | New Issue | |
2007-02-22 22:25 | Simon Guard | Note Added: 0000167 | |
2007-02-22 22:56 | Randy Armstrong | Status | new => assigned |
2007-02-22 22:56 | Randy Armstrong | Assigned To | => Randy Armstrong |
2007-02-22 22:57 | Randy Armstrong | Status | assigned => resolved |
2007-02-22 22:57 | Randy Armstrong | Resolution | open => fixed |
2007-02-22 22:57 | Randy Armstrong | Note Added: 0000168 | |
2007-02-23 03:31 | Randy Armstrong | Status | resolved => acknowledged |
2007-02-28 23:24 | Randy Armstrong | Status | acknowledged => confirmed |
2008-05-10 08:50 | Randy Armstrong | Status | confirmed => resolved |
2008-05-10 08:50 | Randy Armstrong | Note Added: 0000697 |