View Issue Details

IDProjectCategoryView StatusLast Update
000278910000-009: Alarms and ConditionsSpecpublic2015-03-24 16:49
ReporterNathan PocockAssigned ToPaul Hunkar  
PrioritynormalSeverityminorReproducibilityalways
Status closedResolutionfixed 
Product Version1.02 
Target Version1.03Fixed in Version1.03 
Summary0002789: Table 28 - Confirm Result Codes: BadNodeIdUnknown description wrong for Types
Description

CMPWG 5/29/2014:

The result codes for the Confirm() method are shown in P9 Table 28. The description for Bad_NodeIdUnknown seems to be wrong about using this code whenever the method is called against for a type-definition:

"See Part 4 for the description of this result code. Used to indicate that the specified Condition is not valid or that the Method was called on the ConditionType Node."

We think that the last sentence needs to be removed.

TagsNo tags attached.
Commit Version
Fix Due Date

Activities

Jim Luth

2014-06-24 17:26

administrator   ~0005368

Confirm and Ack are identical in this respect and they must be called on instance nodes, never on the type. This sentence just clarifies this particular case of Invalid NodeID.

Nathan Pocock

2014-06-26 19:12

viewer   ~0005371

CMPWG Jun-26-2014:

Thanks for the feedback. We wanted to clarify that Confirm should actually be called against a type definition, kind of... the opening paragraph in 5.7.4 (Confirm Method) states:

"However, some Servers do not expose Condition instances in the AddressSpace. Therefore all Servers shall allow Clients to call the Confirm Method by specifying ConditionId as the ObjectId and the well known NodeId of the Method declaration on the AcknowledgeableConditionType as the MethodId. The Method cannot be called on the AcknowledgeableConditionType Node."

However, we still think that BadNodeIdUnknown is the wrong choice of code here. The Node is known. But we wonder if another problem we found may have accidentally resulted in BadNodeIdUnknown being used:

Part 4 Table 63 (Call Operation Level Result Codes) shows:

  • Bad_NodeIdInvalid See Table 166 for the description of this result code.
    Used to indicate that the specified object is not valid.
  • Bad_NodeIdUnknown See Table 166 for the description of this result code.
    Used to indicate that the specified object is not valid.

Note: two different codes showing the exact same description.

Based on the description of BadNodeIdInvalid, it seems to be the best fit for this situation.

Karl Deiretsbacher

2014-07-29 16:57

developer   ~0005380

2014-07-29: UA meeting

We agreed that the error description should be improved.
Proposal: Change error description into "Used to indicate that the specified ObjectId does not refer to a valid Condition"

Paul Hunkar

2014-12-05 13:50

developer   ~0005675

Reviewed all error codes and fix description and the actual error codes as needed (made all methods more consistent)

Jim Luth

2015-03-24 16:49

administrator   ~0005972

Agreed to changes in telecon.

Issue History

Date Modified Username Field Change
2014-05-29 19:45 Nathan Pocock New Issue
2014-06-24 17:26 Jim Luth Note Added: 0005368
2014-06-24 17:26 Jim Luth Status new => closed
2014-06-24 17:26 Jim Luth Resolution open => no change required
2014-06-26 19:12 Nathan Pocock Note Added: 0005371
2014-06-26 19:12 Nathan Pocock Status closed => assigned
2014-06-26 19:12 Nathan Pocock Assigned To => Jim Luth
2014-06-26 20:43 Jim Luth Assigned To Jim Luth =>
2014-06-26 20:43 Jim Luth Status assigned => new
2014-07-29 16:57 Karl Deiretsbacher Note Added: 0005380
2014-07-29 16:57 Karl Deiretsbacher Status new => assigned
2014-07-29 16:57 Karl Deiretsbacher Assigned To => Paul Hunkar
2014-08-19 17:34 Jim Luth Category (No Category) => Spec
2014-08-19 17:34 Jim Luth Target Version => 1.03
2014-12-05 13:50 Paul Hunkar Note Added: 0005675
2014-12-05 13:50 Paul Hunkar Status assigned => resolved
2014-12-05 13:51 Paul Hunkar Status resolved => feedback
2014-12-05 13:51 Paul Hunkar Resolution no change required => reopened
2014-12-05 13:51 Paul Hunkar Status feedback => resolved
2014-12-05 13:51 Paul Hunkar Resolution reopened => fixed
2015-03-24 16:49 Jim Luth Note Added: 0005972
2015-03-24 16:49 Jim Luth Status resolved => closed
2015-03-24 16:49 Jim Luth Fixed in Version => 1.03