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

Small improvements in consistency checking and TransformData #262

Merged
merged 5 commits into from Sep 19, 2016

Conversation

benjamingeer
Copy link

  • Add a transformation to TransformData to add missing knora-base:isDeleted statements.
  • Add a cardinality requiring an rdfs:label on every knora-base:Resource.
  • Improve error-handling if an rdfs:label is missing in a search result.
  • Add a consistency rule that forbids empty strings as objects.
  • Fix the Dokubib ontology so it's consistent with exported test data.

Benjamin Geer added 3 commits September 14, 2016 16:45
…sformData.

- Add a cardinality requiring an rdfs:label on every knora-base:Resource.
- Improve error-handling if an rdfs:label is missing in a search result.
- Add a consistency rule that forbids empty strings as objects.
- Fix the Dokubib ontology so it's consistent with exported test data.
- Make error handling consistent in search responder.
@benjamingeer benjamingeer added the enhancement improve existing code or new feature label Sep 14, 2016
@benjamingeer benjamingeer added this to the Beta Release milestone Sep 14, 2016
@tobiasschweizer
Copy link
Contributor

tobiasschweizer commented Sep 16, 2016

Shouldn't there be a check in the InputValidation in toSparqlEncodedString is the string is empty? If so, we should throw an error.

Otherwise when using Knora with fuseki we would end up with empty strings whereas when using graphdb a consistency rule would be violated. And then you would want to load the data dumped from fuseki into graphdb and it does not work.

@benjamingeer
Copy link
Author

OK, how's this?

@tobiasschweizer
Copy link
Contributor

It is beautiful!

Please add an E2E test for it.

@benjamingeer benjamingeer merged commit aacb46a into develop Sep 19, 2016
@benjamingeer benjamingeer deleted the wip/transform-deleted branch September 19, 2016 12:31
@benjamingeer benjamingeer mentioned this pull request Jul 5, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement improve existing code or new feature
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants