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

Object property hasTopic has two ranges: "Topic" and "EditorialObject" #278

Open
JuergenGrupp opened this issue Jul 14, 2023 · 4 comments · May be fixed by #284
Open

Object property hasTopic has two ranges: "Topic" and "EditorialObject" #278

JuergenGrupp opened this issue Jul 14, 2023 · 4 comments · May be fixed by #284
Assignees
Labels
bug Something isn't working

Comments

@JuergenGrupp
Copy link
Collaborator

The two ranges should be combined with an or-statement:

  1. EditorialObject hasTopic (Topic or EditorialObject)
@JuergenGrupp JuergenGrupp added the bug Something isn't working label Jul 14, 2023
@tormodv
Copy link
Collaborator

tormodv commented Aug 13, 2023

Yes, this is a bug, but there might be more than one. "hasTopic" is a sub-property of hasSubject, but they do not have the same range.
We have other properties without range, and the proper class restriction implicitly defines the range.

Skjermbilde 2023-08-13 kl  20 43 56

So a solution can be to remove the hierarchy and the range statement.

@tormodv
Copy link
Collaborator

tormodv commented Aug 13, 2023

Or should we use "min 0" here too?

@tormodv
Copy link
Collaborator

tormodv commented Aug 14, 2023

What about EditorialObject concerning EditorialObject

or isAbout?

@JuergenGrupp
Copy link
Collaborator Author

I think we should use a new property like "isAbout", and not include it in the above mentioned hierarchy of hasSubject.

@tormodv tormodv linked a pull request Oct 9, 2023 that will close this issue
@tormodv tormodv linked a pull request Oct 9, 2023 that will close this issue
@aro-max aro-max closed this as completed Apr 9, 2024
@aro-max aro-max reopened this Apr 9, 2024
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

Successfully merging a pull request may close this issue.

3 participants