View Issue Details
ID | Project | Category | View Status | Date Submitted | Last Update |
---|---|---|---|---|---|
0004788 | NodeSets, XSDs and Generated Code | Implementation Bug | public | 2019-06-19 09:43 | 2019-07-23 15:18 |
Reporter | Christian von Arnim | Assigned To | Randy Armstrong | ||
Priority | normal | Severity | minor | Reproducibility | always |
Status | closed | Resolution | fixed | ||
Summary | 0004788: Commas in CSV fields are not enclosed in double quotes | ||||
Description | I stumbled across th message when I watched the StatusCode.csv on github (Se also attached image, or https://github.com/OPCFoundation/UA-Nodeset/blob/044132f275185481c435d0396014e4e9f6836f9e/Schema/StatusCode.csv): The reason is, for example, in the following line, which has a comma in the description text. https://github.com/OPCFoundation/UA-Nodeset/blob/044132f275185481c435d0396014e4e9f6836f9e/Schema/StatusCode.csv#L50 The rfc4180 suggests adding double quotes for these fields. From: https://tools.ietf.org/html/rfc4180
So I would suggest adding double quotes to all CSV fields that (can) contain a comma or other special characters. This change might break some existing parsers of these files, but will improve overall compatibility. This may also be a problem in other CSV files in the repository. | ||||
Tags | Schema | ||||
Attached Files | |||||
Commit Version | |||||
Fix Due Date | |||||
Date Modified | Username | Field | Change |
---|---|---|---|
2019-06-19 09:43 | Christian von Arnim | New Issue | |
2019-06-19 09:43 | Christian von Arnim | Tag Attached: Schema | |
2019-06-19 09:43 | Christian von Arnim | File Added: GithubCsvError.PNG | |
2019-06-25 15:54 | Jim Luth | Assigned To | => Randy Armstrong |
2019-06-25 15:54 | Jim Luth | Status | new => assigned |
2019-07-17 22:46 | Randy Armstrong | Status | assigned => resolved |
2019-07-17 22:46 | Randy Armstrong | Resolution | open => fixed |
2019-07-17 22:46 | Randy Armstrong | Note Added: 0010500 | |
2019-07-23 15:18 | Randy Armstrong | Status | resolved => closed |
2019-07-23 15:18 | Randy Armstrong | Note Added: 0010561 |