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

Cannot remove @solid/context #36

Open
matthieu-fesselier opened this issue Sep 16, 2019 · 4 comments
Open

Cannot remove @solid/context #36

matthieu-fesselier opened this issue Sep 16, 2019 · 4 comments
Assignees
Labels
question Further information is requested

Comments

@matthieu-fesselier
Copy link

My current context is:

{
  '@vocab': 'http://happy-dev.fr/owl/#',
  rdf: 'http://www.w3.org/1999/02/22-rdf-syntax-ns#',
  rdfs: 'http://www.w3.org/2000/01/rdf-schema#',
  ldp: 'http://www.w3.org/ns/ldp#',
  foaf: 'http://xmlns.com/foaf/0.1/',
  name: 'rdfs:label',
  acl: 'http://www.w3.org/ns/auth/acl#',
  permissions: 'acl:accessControl',
  mode: 'acl:mode'
};

The @vocab defines our default which is used for almost all our properties. With the @solid context, some properties are overriden. For example, when we try to target http://happy-dev.fr/owl/#firstName, we get http://xmlns.com/foaf/0.1/givenName.

Is it possible to use only our context and not the solid one?

@RubenVerborgh
Copy link
Member

@matthieu-fesselier
Copy link
Author

The existing properties are favored only over our default which is defined by '@vocab': 'http://happy-dev.fr/owl/#'
For the properties we define explicitely, it works fine

@RubenVerborgh
Copy link
Member

Right… then perhaps it is a feature instead of a bug. i.e., you probably want to explicitly override existing properties.

But I would actually recommend against that, given that the same Solid LDflex expressions then get different meanings. If you want to get really custom, perhaps consider distributing a custom configuration of ldflex rather than extending @solid/query-ldflex. Or in other words: @solid/query-ldflex is a configuration of ldflex; maybe you want your own configuration?

@RubenVerborgh RubenVerborgh added the question Further information is requested label Sep 16, 2019
@RubenVerborgh RubenVerborgh self-assigned this Sep 16, 2019
@rubensworks
Copy link
Contributor

That does indeed sound like the intended JSON-LD context semantics.
Explicitly defined terms (such as firstName in the Solid context)
have priority over @vocab.

Perhaps a flag could be added to context.extend() to exclude the default context? The the user is responsible for the full context.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
question Further information is requested
Projects
None yet
Development

No branches or pull requests

3 participants