Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

VersionB PropertyValue.ValueCase return 20 after setting DataType to DataType.PropertySetList #72

Closed
thuyenle1999 opened this issue Feb 23, 2024 · 2 comments
Assignees
Labels
bug Something isn't working

Comments

@thuyenle1999
Copy link

I would like to report an issue when setting VersionB PropertyValue's DataType to DataType.PropertySetList.

        var propertyValue = new PropertyValue()
        {
            DataType = DataType.PropertySetList, 
            PropertySetsValue = new PropertySetList()
        };

When I checked the ValueCase of the PropertyValue, it somehow returned 20, which indicates PropertySet, rather than 21, which indicates PropertySetList.

I can force the ValueCase to 21, but I think it should be set to 21 automatically.

I hope I have explained the problem clearly as English isn't my mother tongue.

Thank you very much for your useful library!

@SeppPenner SeppPenner self-assigned this Feb 23, 2024
@SeppPenner SeppPenner added the bug Something isn't working label Feb 23, 2024
@SeppPenner
Copy link
Owner

I'm currently reworking the total conversion and value assignment and try to back all those things by unit tests. Stay tuned, I hope to finish this soon.

@SeppPenner
Copy link
Owner

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants