pldn / LDWizard

🧙 LDWizard: A generic framework for simplifying the creation of linked data. Supported by the PLDN community.
European Union Public License 1.2
13 stars 7 forks source link

Perform IRI validation for base IRI and class fields #26

Closed wouterbeek closed 1 year ago

wouterbeek commented 4 years ago

Observed

Currently IRI validation is performed when entering property IRIs, but not when entering base IRIs or class IRIs.

Expected

All IRI fields to be validated, including the base IRI and class fields.

Steps to reproduce

  1. Enter a text that is not a valid IRI in either the base IRI or class field.
  2. Export to RDF and notice that the resulting export is not valid.

Screenshot from 2020-09-28 10-21-25

LaurensRietveld commented 3 years ago

This is an unresolved issue atm:

For consistency reasons, we should go for one of the two options below:

Leaving this one open for now, and let's revisit once we've made a decision

MPvHarmelen commented 3 years ago

See also https://www.npmjs.com/package/uri-js for IRI parsing and serialization (but not validation)

wouterbeek commented 2 years ago

It is unfortunate that validating IRIs is an unsolved computer science problem in 2022 :-P

We should still apply URI validation (many off-the-shelf libraries for that, probably in the standard library for ECMAScript).