-
Notifications
You must be signed in to change notification settings - Fork 71
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
'rdau' RDF namespace is not registered #1520
Comments
Nevermind, |
Maybe we should do a review to see if all the namespaces for all the default mappings are registered? |
Maybe we should finally just make this namespace thing configurable.... Note: this is untested code... just something that popped into mind and written up real quick. rdf_map.schema.yml
rdf_map.yml
islandora.module
This is, of course, less efficient that simply returning a hard-coded list, but much easier for sites to update; especially with Features. Also, I recently learned that we can add weight to our module so it's module file is processed after others, making it more likely that our config will override other mappings set. |
I like the idea of auto-discovery. Having to manage the mappings and their corresponding namespaces separately is going to be a PITA in the long run. |
|
Specifically, I find http://www.rdaregistry.info to be kind of an unstable site in general. Maybe it was being weird when you were looking at it? Currently the link above (http://www.rdaregistry.info/Elements/u/#P60329) does resolve to the "has edition statement" property. It is also registered at http://metadataregistry.org/schemaprop/show/id/14925.html |
I can take this for the sprint. Simply adding the namespace is simple enough, but would folks like me to go ahead and move the namespaces to config as I suggested above? If so, should it go in the islandora module or in the JSON-LD module? Thoughts, @Islandora/8-x-committers ? |
I am not a committer, but +1 on making the namespaces configurable (preferably via the UI in such a way that metadata managers will be able to add namespaces as needed). |
Resolved via Islandora/islandora@5644455 and Islandora/jsonld@b7acb80 You can now configure namespace prefixes through the UI in jsonld's config page. Be sure to run the update hook after fetching the newest code. We should make note to that effect in #1789 as well. |
config/install/rdf.mapping.node.islandora_object.yml
defines the following mapping:but the
rdau
namespace is not registered (or at least isn't known to therdf_get_namespaces()
function). Should we add that namespace toislandora_rdf_namespaces()
?I think
rdau
is used as the namespace for http://www.rdaregistry.info/, but I can't find P60329 in any of the ontologies registered there. Metadata IG folks, any suggestions?The text was updated successfully, but these errors were encountered: