Fix GetDataTypeId to return specific DataTypeId for IEncodeable types

[Copilot]

Proposed changes

DataTypes.GetDataTypeId(typeof(EUInformation)) returned i=22 (Structure) instead of i=887 (EUInformation). The method failed to extract specific TypeIds from IEncodeable types, returning the generic Structure base type for all encodeable objects.

Changes

Modified TypeInfo.GetDataTypeId(Type type) to instantiate IEncodeable types and extract their TypeId:

// Before: returned Structure (i=22) for all IEncodeable types
DataTypes.GetDataTypeId(typeof(EUInformation));  // i=22

// After: returns specific DataTypeId
DataTypes.GetDataTypeId(typeof(EUInformation));  // i=887
DataTypes.GetDataTypeId(typeof(Range));          // i=884
DataTypes.GetDataTypeId(typeof(Argument));       // i=296

Implementation:

• Check if returned DataTypeId is Structure and type implements IEncodeable
• Create instance via Activator.CreateInstance and extract TypeId property
• Fall back to Structure on instantiation failure
• Handle MissingMethodException, TargetInvocationException, MethodAccessException, NotSupportedException

Tests:

• Added tests for EUInformation, Range, Argument, EnumValueType, TimeZoneDataType
• Validates both Type and instance-based GetDataTypeId calls
• All 7,839 Core tests pass

Related Issues

• Fixes DataTypes.GetDataTypeId return node ID "i=22" instead of "i=887" for EUInformation #1679

Types of changes

• Bugfix (non-breaking change which fixes an issue)
• Enhancement (non-breaking change which adds functionality)
• Test enhancement (non-breaking change to increase test coverage)
• Breaking change (fix or feature that would cause existing functionality to not work as expected, requires version increase of Nuget packages)
• Documentation Update (if none of the other choices apply)

Checklist

• I have read the CONTRIBUTING doc.
• I have signed the CLA.
• I ran tests locally with my changes, all passed.
• I fixed all failing tests in the CI pipelines.
• I fixed all introduced issues with CodeQL and LGTM.
• I have added tests that prove my fix is effective or that my feature works and increased code coverage.
• I have added necessary documentation (if appropriate).
• Any dependent changes have been merged and published in downstream modules.

Further comments

The fix uses reflection to instantiate types at runtime rather than maintaining a hardcoded mapping. This approach scales to custom IEncodeable types and aligns with the existing instance-based GetDataTypeId(object value) implementation, which already extracts TypeId from IEncodeable instances.

Performance impact is minimal—instantiation occurs only once per type when GetDataTypeId is called with a Type parameter, and only for IEncodeable types that would otherwise return Structure.

Original prompt

---

This section details on the original issue you should resolve

<issue_title>DataTypes.GetDataTypeId return node ID "i=22" instead of "i=887" for EUInformation</issue_title>
<issue_description>Type of Issue

• Bug
• Enhancement
• Compliance
• Question
• Help wanted

Describe the Issue

The method NodeId GetDataTypeId(Type type) of DataTypes (DataTypes.Helpers.cs) returns the node ID "i=22" (Structure) when the type passed as argument is Opc.Ua.EUInformation. Why does it not return "i=887"?

GetDataTypeId calls TypeInfo.Construct(type) which does:

            ...
            // handle scalar.
            if (dimensions == null)
            {
                BuiltInType builtInType = GetBuiltInType(name);
            ...

builtInType is then null, and then TypeInfo.Construct(type) return the TypeInfo for BuiltInType.ExtensionObject:

                ...
                // check for encodeable object.
                if (typeof(IEncodeable).GetTypeInfo().IsAssignableFrom(systemType.GetTypeInfo()) || name == "IEncodeable")
                {
                    return new TypeInfo(BuiltInType.ExtensionObject, ValueRanks.Scalar);
                }

This leads to the node ID "i=22" due to the later call to NodeId GetDataTypeId(TypeInfo typeInfo).
Is this normal?

Environment:

• OPC Package Release Version 1.4.367.42

</issue_description>

Comments on the Issue (you are @copilot in this section)

@EthanChangAED Hello @thomas-wirtz-bauman , are you still seeing this issue?

• Fixes DataTypes.GetDataTypeId return node ID "i=22" instead of "i=887" for EUInformation #1679

---

💬 We'd love your input! Share your thoughts on Copilot coding agent in our 2 minute survey.

[CLAassistant]

Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.
_{You have signed the CLA already but the status is still pending? Let us recheck it.}