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

Incorrect encoding of publicationDate in ModelDesign XML file #50

Open
eoursel opened this issue Aug 10, 2021 · 0 comments
Open

Incorrect encoding of publicationDate in ModelDesign XML file #50

eoursel opened this issue Aug 10, 2021 · 0 comments
Labels
bug Something isn't working
Milestone

Comments

@eoursel
Copy link

eoursel commented Aug 10, 2021

Describe the bug

Tried with the DI model to generate modelDesign with asp. It appears that publication dates do not have a correct encoding; As a result of that the ModelCompiler stops with an exception.

To Reproduce

Steps to reproduce the behavior:

%ASPDIR%\asp "..\Opc.Ua.Di.NodeSet2.xml" -e "publish\Opc.Ua.Di.Model.xml" -s XMLstylesheet -n "http://opcfoundation.org/UA/DI/" %MODELCOMPILER% -d2 "publish\Opc.Ua.Di.Model.xml" -cg "publish\Opc.Ua.Di.Model.csv" -o2 "publish"

The ModelDesign file contains

 <Namespaces>
    <Namespace Name="Name0" Prefix="Prefix0" InternalPrefix="http://opcfoundation.org/UA/" XmlNamespace="http://opcfoundation.org/UA/" XmlPrefix="Prefix1" FilePath="" Version="1.04.8" PublicationDate="20/11/2020">http://opcfoundation.org/UA/</Namespace>
    <Namespace Name="Name2" Prefix="Prefix2" InternalPrefix="http://opcfoundation.org/UA/DI/" XmlNamespace="http://opcfoundation.org/UA/DI/" XmlPrefix="Prefix3" FilePath="" Version="1.03.0" PublicationDate="09/03/2021">http://opcfoundation.org/UA/DI/</Namespace>
  </Namespaces>

Expected behavior

Obviously publication date should be something like PublicationDate="2021-03-09T00:00:00Z"

Screenshots

image

Additional context

Perhaps it is because my machine has a french Locale? It could be the difference with your machine. Anyway, the encoding of the XML file should be independant of the locale of the machine, right?

Best regards

@eoursel eoursel added the bug Something isn't working label Aug 10, 2021
@mpostol mpostol added this to the backlog milestone Jul 17, 2022
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